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 <prad...@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) {
                }
        }

Reply via email to