-
Notifications
You must be signed in to change notification settings - Fork 281
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
Forward port integration tests #3512
Forward port integration tests #3512
Conversation
Test framework that allows for declaring within the test class all related users, roles, and other security primitives. Emphasis on hamcrest Matchers for straight forward to read, author, and troubleshoot test cases. Signed-off-by: Peter Nied <petern@amazon.com> Signed-off-by: Peter Nied <peternied@hotmail.com> Signed-off-by: Peter Nied <petern@amazon.com>
Signed-off-by: Peter Nied <petern@amazon.com>
…requests (opensearch-project#3418) (opensearch-project#3496) Backport f7c47af from opensearch-project#3495 --------- Signed-off-by: Peter Nied <petern@amazon.com> Co-authored-by: Peter Nied <petern@amazon.com>
Signed-off-by: Peter Nied <petern@amazon.com>
Codecov Report
@@ Coverage Diff @@
## main #3512 +/- ##
============================================
+ Coverage 64.71% 64.73% +0.02%
- Complexity 3616 3620 +4
============================================
Files 281 281
Lines 20570 20585 +15
Branches 3400 3400
============================================
+ Hits 13312 13326 +14
Misses 5557 5557
- Partials 1701 1702 +1
|
@peternied The version increment PR in this repo is currently blocked because the PR to add the extension points for the header verifier has not been merged into 2.x yet: opensearch-project/OpenSearch#10442 |
Signed-off-by: Peter Nied <petern@amazon.com>
Signed-off-by: Peter Nied <petern@amazon.com>
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.
This change looks good. I have request changes related to HTTPClient 4 as the main is already on hc 5.
src/integrationTest/java/org/opensearch/security/http/BasicAuthWithoutChallengeTests.java
Show resolved
Hide resolved
Signed-off-by: Peter Nied <petern@amazon.com>
Description
Forward ports integration test changes from 2.11 and 2.x branches
Issues Resolved
Check List
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.