Skip to content

[Backport 2.x] fix cluster perm classification for msearch template#2914

Merged
cwperks merged 1 commit into2.xfrom
backport/backport-2892-to-2.x
Jun 29, 2023
Merged

[Backport 2.x] fix cluster perm classification for msearch template#2914
cwperks merged 1 commit into2.xfrom
backport/backport-2892-to-2.x

Conversation

@opensearch-trigger-bot
Copy link
Contributor

Backport 50ce79c from #2892

* fix cluster perm classification for msearch template

Signed-off-by: Derek Ho <dxho@amazon.com>

* move test to unit test file

Signed-off-by: Derek Ho <dxho@amazon.com>

* fully revert integration test file

Signed-off-by: Derek Ho <dxho@amazon.com>

* Update src/test/java/org/opensearch/security/privileges/PrivilegesEvaluatorUnitTest.java

Signed-off-by: Stephen Crawford <65832608+scrawfor99@users.noreply.github.com>

* spotless

Signed-off-by: Derek Ho <dxho@amazon.com>

---------

Signed-off-by: Derek Ho <dxho@amazon.com>
Signed-off-by: Stephen Crawford <65832608+scrawfor99@users.noreply.github.com>
Co-authored-by: Stephen Crawford <65832608+scrawfor99@users.noreply.github.com>
(cherry picked from commit 50ce79c)
@codecov
Copy link

codecov bot commented Jun 28, 2023

Codecov Report

Merging #2914 (19df14d) into 2.x (b81522b) will increase coverage by 0.02%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##                2.x    #2914      +/-   ##
============================================
+ Coverage     62.13%   62.16%   +0.02%     
- Complexity     3373     3376       +3     
============================================
  Files           264      264              
  Lines         19441    19441              
  Branches       3315     3315              
============================================
+ Hits          12080    12085       +5     
+ Misses         5766     5760       -6     
- Partials       1595     1596       +1     
Impacted Files Coverage Δ
...earch/security/privileges/PrivilegesEvaluator.java 73.12% <100.00%> (ø)

... and 4 files with indirect coverage changes

@cwperks cwperks merged commit 101f58c into 2.x Jun 29, 2023
@cwperks cwperks deleted the backport/backport-2892-to-2.x branch June 29, 2023 13:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants