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

Added test for POST /{index}/_mapping #745

Merged
merged 4 commits into from
Dec 18, 2024

Conversation

Tokesh
Copy link
Collaborator

@Tokesh Tokesh commented Dec 17, 2024

Description

POST /{index}/_mapping

Issues Resolved

[#663]

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.

@Tokesh Tokesh changed the title adding POST role mapping test Added test for POST /{index}/_mapping Dec 17, 2024
@Tokesh Tokesh added the skip-changelog No need to update CHANGELOG. label Dec 17, 2024
Copy link
Contributor

github-actions bot commented Dec 17, 2024

Changes Analysis

Commit SHA: 5f7e41f
Comparing To SHA: 1b6935f

API Changes

Summary

NO CHANGES

Report

The full API changes report is available at: https://github.com/opensearch-project/opensearch-api-specification/actions/runs/12397988594/artifacts/2338971786

API Coverage

Before After Δ
Covered (%) 606 (59.35 %) 606 (59.35 %) 0 (0 %)
Uncovered (%) 415 (40.65 %) 415 (40.65 %) 0 (0 %)
Unknown 43 43 0

Copy link
Contributor

github-actions bot commented Dec 17, 2024

Spec Test Coverage Analysis

Total Tested
536 537 (100.19 %)

@Tokesh
Copy link
Collaborator Author

Tokesh commented Dec 17, 2024

Dont merge this PR please! :)

@Tokesh Tokesh marked this pull request as ready for review December 18, 2024 16:51
Copy link
Member

@dblock dblock left a comment

Choose a reason for hiding this comment

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

Small ask.

acknowledged: true
- synopsis: Add or update mapping using POST method.
Copy link
Member

@dblock dblock Dec 18, 2024

Choose a reason for hiding this comment

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

This is a bit redundant with what's above. Change the first PUT /{index}/_mapping to having two verbs, PUT and POST instead.

@dblock
Copy link
Member

dblock commented Dec 18, 2024

Test coverage looks suspicious :)

Screenshot 2024-12-18 at 12 01 34 PM

Signed-off-by: Tokesh <[email protected]>
@dblock dblock merged commit 4a7439b into opensearch-project:main Dec 18, 2024
28 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
skip-changelog No need to update CHANGELOG.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants