Skip to content

Adds ResumeInfo Tests#142769

Merged
joshua-adams-1 merged 12 commits intoelastic:mainfrom
joshua-adams-1:spacetime-abstract-bulk-by-scroll-request
Feb 27, 2026
Merged

Adds ResumeInfo Tests#142769
joshua-adams-1 merged 12 commits intoelastic:mainfrom
joshua-adams-1:spacetime-abstract-bulk-by-scroll-request

Conversation

@joshua-adams-1
Copy link
Copy Markdown
Contributor

Adds a unit test suite for ResumeInfo, and serialization tests for both ResumeInfo, and it's subclasses.

@joshua-adams-1 joshua-adams-1 self-assigned this Feb 20, 2026
@joshua-adams-1 joshua-adams-1 added >test Issues or PRs that are addressing/adding tests :Distributed/Distributed A catch all label for anything in the Distributed Area. Please avoid if you can. labels Feb 20, 2026
@joshua-adams-1
Copy link
Copy Markdown
Contributor Author

As a note, since the tests heavily rely on randomisation, the newly added tests have been run a couple of thousand times to ensure there are no failure paths

@joshua-adams-1 joshua-adams-1 marked this pull request as ready for review February 25, 2026 12:08
@elasticsearchmachine elasticsearchmachine added the Team:Distributed Meta label for distributed team. label Feb 25, 2026
@elasticsearchmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-distributed (Team:Distributed)

Copy link
Copy Markdown
Contributor

@samxbr samxbr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for adding these, unfortunately the tests failed to compile due to a recent rename (I am to be blamed..), you will need to rename in the tests as well

@joshua-adams-1 joshua-adams-1 merged commit bc4fe22 into elastic:main Feb 27, 2026
35 checks passed
@joshua-adams-1 joshua-adams-1 deleted the spacetime-abstract-bulk-by-scroll-request branch February 27, 2026 16:09
szybia added a commit to szybia/elasticsearch that referenced this pull request Feb 27, 2026
…cations

* upstream/main:
  Warn on API key version mismatch (elastic#143127)
  Fixed wrong malformed value ordering in synthetic source tests (elastic#143187)
  [ML] Fix: required_native_memory_bytes Calculated with Wrong Allocation Count (elastic#143077)
  Add configureBenchmarkLogging calls across the various benchmarks (elastic#143185)
  Mute org.elasticsearch.xpack.esql.CsvIT test {csv-spec:k8s-timeseries-avg-over-time.Avg_over_time_aggregate_metric_double_implicit_casting} elastic#143292
  Give system role permission to invoke shard refresh (elastic#143190)
  Mute testSyntheticSourceWithTranslogSnapshot (elastic#143260)
  Adds ResumeInfo Tests (elastic#142769)
  Use a static method to configure benchmark logging (elastic#143056)
  add connectors release notes (elastic#142884)
  Add CI triage guidance for AI agents (elastic#142994)
  ESQL: Data sources: ZSTD, BZIP2 (elastic#143228)
  [ES|QL] Channels issue when an agg is called with the same field (elastic#142180) (elastic#142269)
  Add support for project routing in reindex requests (elastic#142240)
tballison pushed a commit to tballison/elasticsearch that referenced this pull request Mar 3, 2026
Adds serialisation tests for ResumeInfo and its subclasses.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Distributed/Distributed A catch all label for anything in the Distributed Area. Please avoid if you can. Team:Distributed Meta label for distributed team. >test Issues or PRs that are addressing/adding tests v9.4.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants