tiny fix

Change-Id: Ib6281cb3dfdc917ecac0c9450f7919e3898f73db
apache · Feb 23, 2021 · 227e899 · 227e899
1 parent aa43d99
commit 227e899
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/hugegraph-api/src/main/java/com/baidu/hugegraph/auth/HugeAuthenticator.java b/hugegraph-api/src/main/java/com/baidu/hugegraph/auth/HugeAuthenticator.java
@@ -459,7 +459,8 @@ public static String roleFor(String owner, HugePermission perm) {
              * Construct required permission such as:
              *  $owner=graph1 $action=read
              *  (means required read permission of any one resource)
-             * maybe also support:
+             *
+             * In the future maybe also support:
              *  $owner=graph1 $action=vertex_read
              */
             return String.format("%s=%s %s=%s", KEY_OWNER, owner,

diff --git a/hugegraph-test/src/main/java/com/baidu/hugegraph/unit/core/RolePermissionTest.java b/hugegraph-test/src/main/java/com/baidu/hugegraph/unit/core/RolePermissionTest.java
@@ -143,12 +143,12 @@ public void testResourceType() {
         Assert.assertTrue(ResourceType.ROOT.match(ResourceType.EDGE));
         Assert.assertTrue(ResourceType.ROOT.match(ResourceType.VERTEX_LABEL));
         Assert.assertTrue(ResourceType.ROOT.match(ResourceType.META));
-        Assert.assertTrue(ResourceType.ALL.match(ResourceType.ALL));;
+        Assert.assertTrue(ResourceType.ROOT.match(ResourceType.ALL));
         Assert.assertTrue(ResourceType.ROOT.match(ResourceType.GRANT));
         Assert.assertTrue(ResourceType.ROOT.match(ResourceType.USER_GROUP));
         Assert.assertTrue(ResourceType.ROOT.match(ResourceType.TARGET));
         Assert.assertTrue(ResourceType.ROOT.match(ResourceType.METRICS));
-        Assert.assertFalse(ResourceType.ALL.match(ResourceType.ROOT));
+        Assert.assertTrue(ResourceType.ROOT.match(ResourceType.ROOT));
 
         // ALL
         Assert.assertTrue(ResourceType.ALL.match(ResourceType.NONE));