-
Notifications
You must be signed in to change notification settings - Fork 1.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Rename] remaining packages and resources in test/framework #364
[Rename] remaining packages and resources in test/framework #364
Conversation
This commit refactors the remaining o.e.index and o.e.test packages in the test/fixtures module. References throughout the codebase are also refactored. Signed-off-by: Nicholas Walter Knize <[email protected]>
✅ DCO Check Passed |
Since this is a big PR - I'll be adding comments incrementally, so we can fix them (if needed) quickly rather than waiting for the full review. |
test/framework/src/main/java/org/opensearch/test/ESTestCase.java
Outdated
Show resolved
Hide resolved
Signed-off-by: Nicholas Walter Knize <[email protected]>
✅ DCO Check Passed |
Signed-off-by: Nicholas Walter Knize <[email protected]>
✅ DCO Check Passed |
client/rest-high-level/src/test/java/org/opensearch/client/core/CountResponseTests.java
Outdated
Show resolved
Hide resolved
test/framework/src/main/java/org/opensearch/test/hamcrest/ElasticsearchAssertions.java
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hard to catch everything in one scan - commented on whatever I can find (We will clean up later if missed something). Thanks 👍
test/framework/src/main/java/org/opensearch/index/store/EsBaseDirectoryTestCase.java
Outdated
Show resolved
Hide resolved
server/src/test/java/org/elasticsearch/index/OpenSearchTieredMergePolicyTests.java
Show resolved
Hide resolved
test/framework/src/test/java/org/elasticsearch/test/AbstractQueryTestCaseTests.java
Outdated
Show resolved
Hide resolved
distribution/tools/plugin-cli/src/test/java/org/opensearch/plugins/ListPluginsCommandTests.java
Show resolved
Hide resolved
test/framework/src/main/java/org/opensearch/test/OpenSearchTestCase.java
Show resolved
Hide resolved
test/framework/src/main/java/org/opensearch/test/rest/package-info.java
Outdated
Show resolved
Hide resolved
Signed-off-by: Nicholas Walter Knize <[email protected]>
This commit refactors the remaining o.e.index and o.e.test packages in the test/fixtures module. References throughout the codebase are also refactored. Signed-off-by: Nicholas Walter Knize <[email protected]>
This commit refactors the remaining o.e.index and o.e.test packages in the test/fixtures module. References throughout the codebase are also refactored. Signed-off-by: Nicholas Walter Knize <[email protected]>
Signed-off-by: dblock <[email protected]>
…h-project#412) * Adding x-amz-content-sha256 header for signed requests (opensearch-project#339) * Adding X-Amz-Content-Sha256 header for signed requests Signed-off-by: Vacha Shah <[email protected]> * Adding CHANGELOG entry Signed-off-by: Vacha Shah <[email protected]> * Adding documentation comment Signed-off-by: Vacha Shah <[email protected]> * Adding tests Signed-off-by: Vacha Shah <[email protected]> * Addressing comments Signed-off-by: Vacha Shah <[email protected]> * Addressing comments Signed-off-by: Vacha Shah <[email protected]> * Removing refresh policy for integ tests for Sigv4 Signed-off-by: Vacha Shah <[email protected]> * Updating the developer guide Signed-off-by: Vacha Shah <[email protected]> Signed-off-by: Vacha Shah <[email protected]> Signed-off-by: MarinaRazumovsky <[email protected]> * Fix issue with completion suggester being parsed as term suggester. (opensearch-project#347) * Fix issue with completion suggester being parsed as term suggester. This commit fixes the issue where a completion suggester search request was being parsed as a term suggester and failing due to "Missing required property 'TermSuggestOption.score'" This solution was originally proposed by Github user @apatrida Signed-off-by: Dan Hart <[email protected]> * Remove commented code Signed-off-by: Dan Hart <[email protected]> * Fix format of changelog item Signed-off-by: Dan Hart <[email protected]> Signed-off-by: Dan Hart <[email protected]> Signed-off-by: MarinaRazumovsky <[email protected]> * Update USER_GUIDE to use exact AWS brand names (opensearch-project#352) Minor update to use exact brand names. "OpenSearch" is rather overloaded to I find it less confusing to be precise in the naming. Signed-off-by: Andrew Ross <[email protected]> Signed-off-by: Andrew Ross <[email protected]> Signed-off-by: MarinaRazumovsky <[email protected]> * Bumps Jackson from 2.14.1 to 2.14.2 (opensearch-project#357) Signed-off-by: Andriy Redko <[email protected]> Signed-off-by: MarinaRazumovsky <[email protected]> * Bulk UpdateOperation misses upsert options (opensearch-project#353) * Bulk UpdateOperation misses upsert options Signed-off-by: Andriy Redko <[email protected]> * Fix formatting Signed-off-by: Andriy Redko <[email protected]> --------- Signed-off-by: Andriy Redko <[email protected]> Signed-off-by: MarinaRazumovsky <[email protected]> * Upgrading compatibility to opensearch 2.5 (opensearch-project#367) * Upgrading compatibility to opensearch 2.5 Signed-off-by: amitai stern <[email protected]> * latest 2.4 version Signed-off-by: amitai stern <[email protected]> * keep existing compatibility matrix rows, this pr adds a new row for versions of opensearch after 2.3.0 Signed-off-by: amitai stern <[email protected]> --------- Signed-off-by: amitai stern <[email protected]> Signed-off-by: MarinaRazumovsky <[email protected]> * add option to set columns into cat indices request Signed-off-by: MarinaRazumovsky <[email protected]> * add changelog Signed-off-by: MarinaRazumovsky <[email protected]> * changes in CatRequestBase, IndicesRequest, NodesRequest, RecoveryRequest and add tests Signed-off-by: MarinaRazumovsky <[email protected]> * Feature/range aggregation fix 369 (opensearch-project#370) * test: create integration tests for date_range and range aggregation (opensearch-project#369) Signed-off-by: Dominik Szczepanczyk <[email protected]> * fix: add a missing key propery to the RangeBucket (opensearch-project#369) Signed-off-by: Dominik Szczepanczyk <[email protected]> * docs: add CHANGELOG entry (opensearch-project#369) Signed-off-by: Dominik Szczepanczyk <[email protected]> --------- Signed-off-by: Dominik Szczepanczyk <[email protected]> Signed-off-by: MarinaRazumovsky <[email protected]> * Fixing integration tests (opensearch-project#375) * Fixing integration tests Signed-off-by: Vacha Shah <[email protected]> * Updating the test with a non-exact check Signed-off-by: Vacha Shah <[email protected]> --------- Signed-off-by: Vacha Shah <[email protected]> Signed-off-by: MarinaRazumovsky <[email protected]> * Adding bulk request example in user guide (opensearch-project#373) Signed-off-by: Vacha Shah <[email protected]> Signed-off-by: MarinaRazumovsky <[email protected]> * Fixing issue when tracktotalhits is disabled (opensearch-project#372) * Fixing issue when tracktotalhits is disabled Signed-off-by: Vacha Shah <[email protected]> * Update Changelog Signed-off-by: Vacha Shah <[email protected]> --------- Signed-off-by: Vacha Shah <[email protected]> Co-authored-by: Daniel (dB.) Doubrovkine <[email protected]> Signed-off-by: MarinaRazumovsky <[email protected]> * changes changelog Signed-off-by: MarinaRazumovsky <[email protected]> * Bump io.github.classgraph:classgraph from 4.8.154 to 4.8.156 (opensearch-project#390) * Bump io.github.classgraph:classgraph from 4.8.154 to 4.8.156 Bumps [io.github.classgraph:classgraph](https://github.com/classgraph/classgraph) from 4.8.154 to 4.8.156. - [Release notes](https://github.com/classgraph/classgraph/releases) - [Commits](classgraph/classgraph@classgraph-4.8.154...classgraph-4.8.156) --- updated-dependencies: - dependency-name: io.github.classgraph:classgraph dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> * Update changelog Signed-off-by: dependabot[bot] <[email protected]> --------- Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Signed-off-by: MarinaRazumovsky <[email protected]> * Add 2PR approval to release workflow (opensearch-project#383) Signed-off-by: Sayali Gaikawad <[email protected]> Signed-off-by: MarinaRazumovsky <[email protected]> * Adding an example to use raw JSON class instead of target classes (opensearch-project#385) Signed-off-by: Vacha Shah <[email protected]> Signed-off-by: MarinaRazumovsky <[email protected]> * Update Gradle to 8.0.2 (opensearch-project#392) Signed-off-by: Andriy Redko <[email protected]> Signed-off-by: MarinaRazumovsky <[email protected]> * Add "mappings" field to IndexSettings object (opensearch-project#382) Signed-off-by: Ilya Lukyanovich <[email protected]> Signed-off-by: MarinaRazumovsky <[email protected]> * Created untriaged issue workflow. (opensearch-project#364) Signed-off-by: dblock <[email protected]> Signed-off-by: MarinaRazumovsky <[email protected]> * Bump io.github.classgraph:classgraph from 4.8.156 to 4.8.157 in /java-client (opensearch-project#403) * Bump io.github.classgraph:classgraph in /java-client Bumps [io.github.classgraph:classgraph](https://github.com/classgraph/classgraph) from 4.8.156 to 4.8.157. - [Release notes](https://github.com/classgraph/classgraph/releases) - [Commits](classgraph/classgraph@classgraph-4.8.156...classgraph-4.8.157) --- updated-dependencies: - dependency-name: io.github.classgraph:classgraph dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> * Update changelog Signed-off-by: dependabot[bot] <[email protected]> --------- Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Signed-off-by: MarinaRazumovsky <[email protected]> * Add buffered lookahead for Jackson (opensearch-project#338) Signed-off-by: luneo7 <[email protected]> Signed-off-by: MarinaRazumovsky <[email protected]> * Mask env values for snapshots (opensearch-project#411) Signed-off-by: Sayali Gaikawad <[email protected]> Signed-off-by: MarinaRazumovsky <[email protected]> * option to set headers and sort into cat requests Signed-off-by: MarinaRazumovsky <[email protected]> * fix test Signed-off-by: MarinaRazumovsky <[email protected]> * fix changelog Signed-off-by: MarinaRazumovsky <[email protected]> * add cat examples into USER_GUID Signed-off-by: MarinaRazumovsky <[email protected]> --------- Signed-off-by: Vacha Shah <[email protected]> Signed-off-by: MarinaRazumovsky <[email protected]> Signed-off-by: Dan Hart <[email protected]> Signed-off-by: Andrew Ross <[email protected]> Signed-off-by: Andriy Redko <[email protected]> Signed-off-by: amitai stern <[email protected]> Signed-off-by: Dominik Szczepanczyk <[email protected]> Signed-off-by: dependabot[bot] <[email protected]> Signed-off-by: Sayali Gaikawad <[email protected]> Signed-off-by: Ilya Lukyanovich <[email protected]> Signed-off-by: dblock <[email protected]> Signed-off-by: luneo7 <[email protected]> Co-authored-by: Vacha Shah <[email protected]> Co-authored-by: Danny Hart <[email protected]> Co-authored-by: Andrew Ross <[email protected]> Co-authored-by: Andriy Redko <[email protected]> Co-authored-by: amitai stern <[email protected]> Co-authored-by: szczepanczykd <[email protected]> Co-authored-by: Daniel (dB.) Doubrovkine <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: Sayali Gaikawad <[email protected]> Co-authored-by: Ilya Lukyanovich <[email protected]> Co-authored-by: Lucas Rogerio Caetano Ferreira <[email protected]>
This PR refactors the remaining o.e.index and o.e.test packages in the
test/framework module. References throughout the codebase are also refactored.
relates #160