You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ranger.apache.org by pr...@apache.org on 2022/11/29 10:45:52 UTC
[ranger] branch ranger-2.4 updated: RANGER-3977: Fix Ranger TagRest API deleteTagResourceMapByGuid
This is an automated email from the ASF dual-hosted git repository.
pradeep pushed a commit to branch ranger-2.4
in repository https://gitbox.apache.org/repos/asf/ranger.git
The following commit(s) were added to refs/heads/ranger-2.4 by this push:
new c72e2e377 RANGER-3977: Fix Ranger TagRest API deleteTagResourceMapByGuid
c72e2e377 is described below
commit c72e2e3778847ea9988f958e6cf8eb49e1388a3f
Author: pradeep <pr...@apache.org>
AuthorDate: Thu Nov 24 00:10:53 2022 +0530
RANGER-3977: Fix Ranger TagRest API deleteTagResourceMapByGuid
---
security-admin/src/main/java/org/apache/ranger/rest/TagREST.java | 2 +-
security-admin/src/test/java/org/apache/ranger/rest/TestTagREST.java | 4 ++--
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/security-admin/src/main/java/org/apache/ranger/rest/TagREST.java b/security-admin/src/main/java/org/apache/ranger/rest/TagREST.java
index 36c6a4bc9..f8898b75b 100644
--- a/security-admin/src/main/java/org/apache/ranger/rest/TagREST.java
+++ b/security-admin/src/main/java/org/apache/ranger/rest/TagREST.java
@@ -934,7 +934,7 @@ public class TagREST {
try {
RangerTagResourceMap exist = validator.preDeleteTagResourceMapByGuid(guid);
- tagStore.deleteServiceResource(exist.getId());
+ tagStore.deleteTagResourceMap(exist.getId());
} catch(Exception excp) {
LOG.error("deleteTagResourceMapByGuid(" + guid + ") failed", excp);
diff --git a/security-admin/src/test/java/org/apache/ranger/rest/TestTagREST.java b/security-admin/src/test/java/org/apache/ranger/rest/TestTagREST.java
index 7fba152a9..5986d5182 100644
--- a/security-admin/src/test/java/org/apache/ranger/rest/TestTagREST.java
+++ b/security-admin/src/test/java/org/apache/ranger/rest/TestTagREST.java
@@ -1184,7 +1184,7 @@ public class TestTagREST {
} catch (Exception e) {
}
try {
- Mockito.doNothing().when(tagStore).deleteServiceResource(oldTagResourceMap.getId());
+ Mockito.doNothing().when(tagStore).deleteTagResourceMap(oldTagResourceMap.getId());
} catch (Exception e) {
}
@@ -1197,7 +1197,7 @@ public class TestTagREST {
} catch (Exception e) {
}
try {
- Mockito.verify(tagStore).deleteServiceResource(oldTagResourceMap.getId());
+ Mockito.verify(tagStore).deleteTagResourceMap(oldTagResourceMap.getId());
} catch (Exception e) {
}
}