Skip to content

DE-159650: Update remaining tests for ES v9 compatibility#27

Closed
sumitkhopade1986 wants to merge 1 commit intoDE-159650-upgrade-elasticsearch-tests-transportfrom
DE-159650-upgrade-elasticsearch-tests-remaining
Closed

DE-159650: Update remaining tests for ES v9 compatibility#27
sumitkhopade1986 wants to merge 1 commit intoDE-159650-upgrade-elasticsearch-tests-transportfrom
DE-159650-upgrade-elasticsearch-tests-remaining

Conversation

@sumitkhopade1986
Copy link
Copy Markdown

Description: Update all remaining test files for ES v9 compatibility — Base, ResultSet, Exception, Query, Client functional tests
Possible impact: tests/Base.php, tests/ResultSetTest.php, tests/ClientFunctionalTest.php, tests/IndexTest.php, tests/Query/, tests/Exception/, and more


Summary

  • Updates tests/Base.php helper methods for ES v9 client setup
  • Updates tests/ClientFunctionalTest.php for v9 response structure
  • Updates tests/ResultSetTest.php, tests/ResultTest.php for v9 search response
  • Updates tests/IndexTest.php, tests/MappingTest.php, tests/SnapshotTest.php
  • Updates tests/Query/CommonTest.php, tests/Query/FunctionScoreTest.php, tests/Query/RangeTest.php
  • Updates tests/Exception/ResponseExceptionTest.php, tests/Exception/PartialShardFailureExceptionTest.php
  • Updates tests/ResponseFunctionalTest.php, tests/ReindexTest.php, tests/StatusTest.php
  • Updates tests/ResultSet/BuilderTest.php, tests/ResultSet/ChainProcessorTest.php, tests/ResultSet/ProcessingBuilderTest.php

Stacked PRs (9/9) — Final PR

opensearchinfra-depsconnectionindex-apiclient-bulksecondary-srctests-bulktests-multi-aggregationtests-transportThis PR

⚠️ Base branch is DE-159650-upgrade-elasticsearch-tests-transport — merge that first.

Test Plan

  • All unit and functional tests pass against ES v9
  • Full test suite green on ES v9.3.0

🤖 Generated with Claude Code

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant