Skip to content

Add simple PolicyParser unit test#17690

Merged
andrross merged 1 commit intoopensearch-project:mainfrom
andrross:policy-parser-unit-test
Mar 25, 2025
Merged

Add simple PolicyParser unit test#17690
andrross merged 1 commit intoopensearch-project:mainfrom
andrross:policy-parser-unit-test

Conversation

@andrross
Copy link
Copy Markdown
Member

Follow on to #17663. Adding a basic unit test of the policy parsing logic.

Check List

  • Functionality includes testing.

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.

Signed-off-by: Andrew Ross <andrross@amazon.com>
Copy link
Copy Markdown
Contributor

@reta reta left a comment

Choose a reason for hiding this comment

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

Thanks a mill , @andrross

@github-actions
Copy link
Copy Markdown
Contributor

✅ Gradle check result for 63f6804: SUCCESS

@codecov
Copy link
Copy Markdown

codecov bot commented Mar 25, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 72.17%. Comparing base (6d53f9d) to head (63f6804).
Report is 12 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff              @@
##               main   #17690      +/-   ##
============================================
- Coverage     72.40%   72.17%   -0.23%     
- Complexity    65828    65928     +100     
============================================
  Files          5316     5330      +14     
  Lines        305294   306922    +1628     
  Branches      44289    44651     +362     
============================================
+ Hits         221033   221520     +487     
- Misses        66187    67222    +1035     
- Partials      18074    18180     +106     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@andrross andrross merged commit 3fb09c7 into opensearch-project:main Mar 25, 2025
57 of 58 checks passed
@andrross andrross deleted the policy-parser-unit-test branch March 25, 2025 23:46
andrross added a commit to andrross/OpenSearch that referenced this pull request Mar 26, 2025
andrross added a commit to andrross/OpenSearch that referenced this pull request Mar 26, 2025
This reverts commit 3fb09c7.

Signed-off-by: Andrew Ross <andrross@amazon.com>
andrross added a commit to andrross/OpenSearch that referenced this pull request Mar 27, 2025
This reverts commit 3fb09c7.

Signed-off-by: Andrew Ross <andrross@amazon.com>
reta pushed a commit that referenced this pull request Mar 28, 2025
* Revert "Add simple PolicyParser unit test (#17690)"

This reverts commit 3fb09c7.

Signed-off-by: Andrew Ross <andrross@amazon.com>

* Revert "Add support of Java policies (#17663)"

This reverts commit 17289b7.

Signed-off-by: Andrew Ross <andrross@amazon.com>

---------

Signed-off-by: Andrew Ross <andrross@amazon.com>
kumargu pushed a commit to kumargu/OpenSearch that referenced this pull request Apr 1, 2025
Add simple PolicyParser unit test (opensearch-project#17690)

Signed-off-by: Andrew Ross <andrross@amazon.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants