Skip to content

Commit 2a1f18a

Browse files
committed
Fix more tests
Signed-off-by: Craig Perkins <[email protected]>
1 parent ad45784 commit 2a1f18a

File tree

2 files changed

+4
-2
lines changed

2 files changed

+4
-2
lines changed

src/main/java/org/opensearch/security/resources/ResourcePluginInfo.java

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -217,6 +217,9 @@ public ResourceProvider getResourceProvider(String type) {
217217
public String indexByType(String type) {
218218
lock.readLock().lock();
219219
try {
220+
if (!typeToProvider.containsKey(type)) {
221+
return null;
222+
}
220223
return typeToProvider.get(type).resourceIndexName();
221224
} finally {
222225
lock.readLock().unlock();

src/test/java/org/opensearch/security/resources/ResourceAccessHandlerTest.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -150,8 +150,7 @@ public void testHasPermission_noAccessLevelsDenied() {
150150
when(adminDNs.isAdmin(user)).thenReturn(false);
151151

152152
ResourceSharing doc = mock(ResourceSharing.class);
153-
when(doc.isCreatedBy("charlie")).thenReturn(false);
154-
when(doc.fetchAccessLevels(any(), any())).thenReturn(Collections.emptySet());
153+
when(doc.getAccessLevelsForUser(user)).thenReturn(Collections.emptySet());
155154

156155
doAnswer(inv -> {
157156
ActionListener<ResourceSharing> l = inv.getArgument(2);

0 commit comments

Comments
 (0)