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

Adds test for APIs related to security plugin and updates spec to add new APIs. #439

Merged
merged 57 commits into from
Aug 12, 2024
Merged
Show file tree
Hide file tree
Changes from 55 commits
Commits
Show all changes
57 commits
Select commit Hold shift + click to select a range
af53f9c
Updates the missing item types for security API related schemas
DarshitChanpura Jul 15, 2024
f5220c1
Adds tests for some APIs
DarshitChanpura Jul 18, 2024
cd4709b
Merge remote-tracking branch 'upstream/main' into security-tests
DarshitChanpura Jul 18, 2024
ec97b6b
Merge remote-tracking branch 'upstream/main' into correct-security-sc…
DarshitChanpura Jul 18, 2024
4c51de4
Fixes schema path and lint error in a file
DarshitChanpura Jul 18, 2024
a35cfda
Adds version check for account API test
DarshitChanpura Jul 18, 2024
488e774
Adds a CHANGELOG entry
DarshitChanpura Jul 18, 2024
0439eb2
Fixes tests to expect status as integer
DarshitChanpura Jul 18, 2024
f65a6cf
Adds base tests for all APIs
DarshitChanpura Jul 23, 2024
a25b14f
Fixes linter errors and complete payloads and request bodies
DarshitChanpura Jul 23, 2024
dce0965
Merge remote-tracking branch 'upstream/main' into security-tests
DarshitChanpura Jul 23, 2024
ab976d4
Adds API spec for the new certificates API and adds tests
DarshitChanpura Jul 23, 2024
463401a
Adds missing new lines at the end of files and removes any extra line…
DarshitChanpura Jul 23, 2024
ee3f3b4
Merge remote-tracking branch 'upstream/main' into security-tests
DarshitChanpura Jul 31, 2024
092a1d6
Fixes lint errors
DarshitChanpura Jul 31, 2024
757b0ed
Fixes test spec lint
DarshitChanpura Jul 31, 2024
70e9f4c
Fixes without api prefix tests
DarshitChanpura Aug 1, 2024
456bc51
Merge remote-tracking branch 'upstream/main' into security-tests
DarshitChanpura Aug 5, 2024
588ff10
Merge remote-tracking branch 'origin/correct-security-schemas' into s…
DarshitChanpura Aug 5, 2024
c86e8fa
Fixes accounts tests
DarshitChanpura Aug 7, 2024
892561b
Merge remote-tracking branch 'upstream/main' into security-tests
DarshitChanpura Aug 7, 2024
7b34505
Fixes action-groups tests
DarshitChanpura Aug 7, 2024
0eac313
Fixes account, allowlist and audit tests
DarshitChanpura Aug 8, 2024
f16a75a
Fixes certificates tests
DarshitChanpura Aug 8, 2024
41c2edf
Fixes internal users test
DarshitChanpura Aug 8, 2024
7622f10
Fixes nodesdn test
DarshitChanpura Aug 8, 2024
d73694e
Fixes roles test
DarshitChanpura Aug 8, 2024
85612c0
Fixes rolesmapping test
DarshitChanpura Aug 8, 2024
6537422
Fixes securityconfig test
DarshitChanpura Aug 8, 2024
c87fa98
Fixes ssl_certs tests
DarshitChanpura Aug 8, 2024
f12ee76
Fixes tenancyconfig tests
DarshitChanpura Aug 8, 2024
48cbc7f
Fixes tenants tests
DarshitChanpura Aug 8, 2024
cb5fe80
Fixes tokens and upgrade tests
DarshitChanpura Aug 8, 2024
269b416
Fixes user tests
DarshitChanpura Aug 8, 2024
44160ab
Fixes validate tests and run linter
DarshitChanpura Aug 8, 2024
aa3f2f3
Merge remote-tracking branch 'upstream/main' into security-tests
DarshitChanpura Aug 8, 2024
a92aa5b
Fixes spec
DarshitChanpura Aug 8, 2024
ef92ce6
Adds missing new lines to EOFs
DarshitChanpura Aug 9, 2024
2e090ee
Address changes in CHANGELOG
DarshitChanpura Aug 9, 2024
688e69f
Updates parameter descriptions
DarshitChanpura Aug 9, 2024
a91e52c
Re-organize folder structure
DarshitChanpura Aug 9, 2024
2b3494f
Moved security tests inside plugins folder
DarshitChanpura Aug 9, 2024
833c82e
Updates test spec workflow to run security tests
DarshitChanpura Aug 9, 2024
451b179
Adds security test specific docker compose file
DarshitChanpura Aug 9, 2024
cb0547e
Cleans commented code and updates test-spec to run for 2.16 only
DarshitChanpura Aug 9, 2024
54f4cfa
Refactor s boolean to be true boolean
DarshitChanpura Aug 9, 2024
536cf04
Move security tests to default folder and updates spec file
DarshitChanpura Aug 9, 2024
87640ec
Fixes 1.3 tests
DarshitChanpura Aug 9, 2024
c4205c5
Fixes 2.0 test failures
DarshitChanpura Aug 9, 2024
4227e00
Fixes TLS cipher versiion payload issue
DarshitChanpura Aug 9, 2024
b393538
Merge remote-tracking branch 'upstream/main' into security-tests
DarshitChanpura Aug 12, 2024
2f30633
Re-verifies auth token fix
DarshitChanpura Aug 12, 2024
cc10600
Checks for newer images and pulls them before running tests
DarshitChanpura Aug 12, 2024
355cace
Updates the docker compose command
DarshitChanpura Aug 12, 2024
172328a
Removes sha ref for staging branches
DarshitChanpura Aug 12, 2024
7e6bf0c
Updates shas
DarshitChanpura Aug 12, 2024
705e6cf
Renames everything to match the API path
DarshitChanpura Aug 12, 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
1 change: 1 addition & 0 deletions .cspell
Original file line number Diff line number Diff line change
Expand Up @@ -165,6 +165,7 @@ subqueries
subschemas
subword
syserr
tcnative
tdigest
tenantinfo
termvectors
Expand Down
4 changes: 1 addition & 3 deletions .github/workflows/test-spec.yml
Original file line number Diff line number Diff line change
Expand Up @@ -34,10 +34,8 @@ jobs:
tests: snapshot
- version: 2.17.0
hub: opensearchstaging
ref: '@sha256:6398c27d7560626ed6b0ba28b3d6b20b7f00c6d94abf45ad3a820f8eeb3d61a3'
dblock marked this conversation as resolved.
Show resolved Hide resolved
- version: 3.0.0
hub: opensearchstaging
ref: '@sha256:101681eea630393f8caf5987dd023a975a9656b63090a07bfdfe6ad2f73f0640'

name: test-opensearch-spec (version=${{ matrix.entry.version }}, hub=${{ matrix.entry.hub || 'opensearchproject' }}, tests=${{ matrix.entry.tests || 'default' }})
runs-on: ubuntu-latest
Expand All @@ -63,7 +61,7 @@ jobs:

- name: Run OpenSearch Cluster
working-directory: tests/${{ matrix.entry.tests || 'default' }}
run: docker compose up -d
run: docker compose pull && docker compose up -d

- name: Get Container ID
id: container
Expand Down
3 changes: 3 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -68,6 +68,8 @@ Inspired from [Keep a Changelog](https://keepachangelog.com/en/1.0.0/)
- Added `observability` namespace API specifications ([#474](https://github.com/opensearch-project/opensearch-api-specification/pull/474))
- Added support for reusing output variables as keys in payload expectations ([#471](https://github.com/opensearch-project/opensearch-api-specification/pull/471))
- Added support for running tests against Amazon OpenSearch ([#476](https://github.com/opensearch-project/opensearch-api-specification/pull/476))
- Added API spec for security plugin ([#271](https://github.com/opensearch-project/opensearch-api-specification/pull/271))
- Added `/_plugins/_security/api/certificates/` to API spec ([#439](https://github.com/opensearch-project/opensearch-api-specification/pull/439))

### Changed

Expand Down Expand Up @@ -111,6 +113,7 @@ Inspired from [Keep a Changelog](https://keepachangelog.com/en/1.0.0/)
- Fixed `_update_by_query` with a simple term ([#451](https://github.com/opensearch-project/opensearch-api-specification/pull/451))
- Fixed `Duration` to allow for non-integers ([#479](https://github.com/opensearch-project/opensearch-api-specification/pull/479))
- Fixed accuracy of the index stats schemas ([#491](https://github.com/opensearch-project/opensearch-api-specification/pull/491))
- Fixed security spec to add support for 400 and 403s ([#439](https://github.com/opensearch-project/opensearch-api-specification/pull/439))

### Security

Expand Down
Loading
Loading