-
Notifications
You must be signed in to change notification settings - Fork 53
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
Add resolve API test #506
Add resolve API test #506
Conversation
Signed-off-by: Archer <[email protected]>
Changes AnalysisCommit SHA: 721f951 API ChangesSummaryNO CHANGES ReportThe full API changes report is available at: https://github.com/opensearch-project/opensearch-api-specification/actions/runs/10385601806/artifacts/1810724266 API Coverage
|
Curious if I put this test in the right location. Here is the endpoint in the spec:
|
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.
Lets put this in its own path, tests/default/indices/resolve.yaml
.
The path
in the test will have to be /_resolve/index/{name}
as in the spec and {name}
will come in parameters
. Pre-create the index in a prologues
and delete it in epilogues
. Also test that a response to an index that doesn't exist is a 404.
Signed-off-by: Archer <[email protected]>
Signed-off-by: Archer <[email protected]>
Signed-off-by: Archer <[email protected]>
Signed-off-by: Archer <[email protected]>
Spec Test Coverage Analysis
|
Description
Adds the test for the
_resolve
endpoint in the current index.yaml test.By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.