[apache#6235] fix(authz): fix PathBasedMetadataObject toString method (…

…apache#6252) ### What changes were proposed in this pull request? fix `PathBasedMetadataObject#toString` method. ### Why are the changes needed? Fix: apache#6235 ### Does this PR introduce _any_ user-facing change? No. ### How was this patch tested? local ut test. --------- Co-authored-by: yangyang zhong <[email protected]> Co-authored-by: Xiaojian Sun <[email protected]> Co-authored-by: roryqi <[email protected]> Co-authored-by: TengYao Chi <[email protected]> Co-authored-by: Qi Yu <[email protected]> Co-authored-by: Chun-Hao Liu <[email protected]>
Abyss-lord · Jan 20, 2025 · 184726c · 184726c
1 parent 71cf1be
commit 184726c
Show file tree

Hide file tree

Showing 2 changed files with 35 additions and 3 deletions.
diff --git a/...mmon/src/main/java/org/apache/gravitino/authorization/common/PathBasedMetadataObject.java b/...mmon/src/main/java/org/apache/gravitino/authorization/common/PathBasedMetadataObject.java
@@ -122,8 +122,13 @@ public int hashCode() {
 
   @Override
   public String toString() {
-    return "MetadataObject: [fullName=" + fullName() + "],  [path=" + path == null
-        ? "null"
-        : path + "], [type=" + type + "]";
+    String strPath = path == null ? "null" : path;
+    return "MetadataObject: [fullName="
+        + fullName()
+        + "],  [path="
+        + strPath
+        + "], [type="
+        + type
+        + "]";
   }
 }
diff --git a/.../src/test/java/org/apache/gravitino/authorization/common/TestPathBasedMetadataObject.java b/.../src/test/java/org/apache/gravitino/authorization/common/TestPathBasedMetadataObject.java
@@ -47,4 +47,31 @@ public void PathBasedMetadataObjectNotEquals() {
 
     Assertions.assertNotEquals(pathBasedMetadataObject1, pathBasedMetadataObject2);
   }
+
+  @Test
+  void testToString() {
+    PathBasedMetadataObject pathBasedMetadataObject1 =
+        new PathBasedMetadataObject("parent", "name", "path", PathBasedMetadataObject.Type.PATH);
+    Assertions.assertEquals(
+        "MetadataObject: [fullName=parent.name],  [path=path], [type=PATH]",
+        pathBasedMetadataObject1.toString());
+
+    PathBasedMetadataObject pathBasedMetadataObject2 =
+        new PathBasedMetadataObject("parent", "name", null, PathBasedMetadataObject.Type.PATH);
+    Assertions.assertEquals(
+        "MetadataObject: [fullName=parent.name],  [path=null], [type=PATH]",
+        pathBasedMetadataObject2.toString());
+
+    PathBasedMetadataObject pathBasedMetadataObject3 =
+        new PathBasedMetadataObject(null, "name", null, PathBasedMetadataObject.Type.PATH);
+    Assertions.assertEquals(
+        "MetadataObject: [fullName=name],  [path=null], [type=PATH]",
+        pathBasedMetadataObject3.toString());
+
+    PathBasedMetadataObject pathBasedMetadataObject4 =
+        new PathBasedMetadataObject(null, "name", "path", PathBasedMetadataObject.Type.PATH);
+    Assertions.assertEquals(
+        "MetadataObject: [fullName=name],  [path=path], [type=PATH]",
+        pathBasedMetadataObject4.toString());
+  }
 }