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) { } }