Skip to content
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 missing Index API Spec Tests, Part 1 #678

Merged
merged 23 commits into from
Nov 24, 2024
Merged
Show file tree
Hide file tree
Changes from 21 commits
Commits
Show all changes
23 commits
Select commit Hold shift + click to select a range
106fd00
replication added
Tokesh Aug 3, 2024
e912e13
fixing lint and trying test verbose
Tokesh Aug 3, 2024
bd877a8
changing docker compose
Tokesh Aug 3, 2024
6fd6292
Merge branch 'main' into replication-namespace
Tokesh Aug 3, 2024
addcd76
adding tests for replication namespace
Tokesh Aug 3, 2024
638d60e
fixing lint, validate specs ci
Tokesh Aug 3, 2024
a6e0a87
fixing docker-compose and validate specs ci
Tokesh Aug 3, 2024
b012e7b
fixing validate specs ci
Tokesh Aug 3, 2024
236fea7
trying to fix validate specs
Tokesh Aug 3, 2024
864ee68
trying to fix validate specs v2
Tokesh Aug 3, 2024
30fca51
Merge branch 'replication-namespace'
Tokesh Aug 14, 2024
1d051cd
Merge branch 'main' of https://github.com/Tokesh/opensearch-api-speci…
Tokesh Aug 14, 2024
9620b94
Merge branch 'opensearch-project:main' into main
Tokesh Aug 22, 2024
f847731
Merge remote-tracking branch 'refs/remotes/origin/main'
Tokesh Sep 29, 2024
42056a6
adding not covered index api tests
Tokesh Nov 17, 2024
7c7328c
Merge branch 'main' into index-tests
Tokesh Nov 23, 2024
94915e9
deleting not necessary files
Tokesh Nov 23, 2024
cf167fd
fixing lint
Tokesh Nov 23, 2024
93a58c9
fixing paths, deleting not necessary code, adding validate query api …
Tokesh Nov 23, 2024
7e5570b
deleting empty file
Tokesh Nov 23, 2024
1514af5
fixing minor bug in scenario
Tokesh Nov 23, 2024
37b40b1
adding special folder for validation tests in indices
Tokesh Nov 24, 2024
d925d04
fixing path to test story schema
Tokesh Nov 24, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion spec/namespaces/replication.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -298,4 +298,4 @@ components:
content:
application/json:
schema:
$ref: '../schemas/_common.yaml#/components/schemas/AcknowledgedResponseBase'
$ref: '../schemas/_common.yaml#/components/schemas/AcknowledgedResponseBase'
61 changes: 61 additions & 0 deletions tests/default/indices/mapping/field.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,61 @@
$schema: ../../../../json_schemas/test_story.schema.yaml

description: Test specific field mappings endpoints.
prologues:
- path: /{index}
method: PUT
parameters:
index: movies
request:
payload:
mappings:
properties:
director:
type: text
year:
type: integer
location:
type: ip
ignore_malformed: true
epilogues:
- path: /movies
method: DELETE
status: [200, 404]
chapters:
- synopsis: Add new fields to index mapping.
path: /{index}/_mapping
method: POST
parameters:
index: movies
request:
payload:
properties:
genre:
type: text
rating:
type: float
response:
status: 200
payload:
acknowledged: true
- synopsis: Get specific field mappings for an index.
path: /{index}/_mapping/field/{fields}
method: GET
parameters:
index: movies
fields: director,year
response:
status: 200
payload:
movies:
mappings:
year:
full_name: year
mapping:
year:
type: integer
director:
full_name: director
mapping:
director:
type: text
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
$schema: ../../../json_schemas/test_story.schema.yaml
$schema: ../../../../json_schemas/test_story.schema.yaml

description: Test mappings endpoints.
prologues:
Expand Down Expand Up @@ -145,4 +145,4 @@ chapters:
response:
status: 200
payload:
acknowledged: true
acknowledged: true
36 changes: 36 additions & 0 deletions tests/default/indices/msearch.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,36 @@
$schema: ../../../json_schemas/test_story.schema.yaml

description: Test endpoints relevant to the lifecycle of an index, including multi-get and multi-search operations.
prologues:
- path: /_bulk
method: POST
parameters:
refresh: true
request:
content_type: application/x-ndjson
payload:
- {create: {_index: books, _id: book1}}
- {author: Harper Lee, title: To Kill a Mockingbird, year: 60}
- {create: {_index: books, _id: book2}}
- {director: Bennett Miller, title: The Cruise, year: 1998}
- {create: {_index: books, _id: book3}}
- {director: Nicolas Winding Refn, title: Drive, year: 1960}
epilogues:
- path: /books
method: DELETE
status: [200, 404]
chapters:
- synopsis: Perform a `_msearch` request to execute multiple search queries in a single call.
path: /{index}/_msearch
method: POST
parameters:
index: books
request:
content_type: application/x-ndjson
payload:
- {index: books}
- {query: {match: {title: To Kill a Mockingbird}}}
- {index: books}
- {query: {match: {title: The Cruise}}}
response:
status: 200
2 changes: 1 addition & 1 deletion tests/default/indices/refresh.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -36,4 +36,4 @@ chapters:
status: 200
payload:
_shards:
failed: 0
failed: 0
49 changes: 49 additions & 0 deletions tests/default/indices/validate_query.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,49 @@
$schema: ../../../json_schemas/test_story.schema.yaml
Copy link
Member

Choose a reason for hiding this comment

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

This file should be validate/query.yaml.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

no worries! i'm really interested in working with OpenSearch in the form of tests or writing specs to understand it more deeply.


description: Test validating queries for a specific index using both GET and POST methods.
epilogues:
- path: /movies
method: DELETE
status: [200, 404]
prologues:
- path: /_bulk
method: POST
parameters:
refresh: true
request:
content_type: application/x-ndjson
payload:
- {create: {_index: movies, _id: movie1}}
- {director: Bennett Miller, title: The Cruise, year: 1998}
- {create: {_index: movies, _id: movie2}}
- {director: Nicolas Winding Refn, title: Drive, year: 1960}
chapters:
- synopsis: Validate a match query (GET).
path: /{index}/_validate/query
method: GET
parameters:
index: movies
request:
payload:
query:
match:
year: 1998
response:
status: 200
payload:
valid: true

- synopsis: Validate a match query (POST).
path: /{index}/_validate/query
method: POST
parameters:
index: movies
request:
payload:
query:
match:
title: Drive
response:
status: 200
payload:
valid: true
Loading