imbajin commented on code in PR #3159:
URL: https://github.com/apache/hugegraph/pull/3159#discussion_r3804326237


##########
hugegraph-server/hugegraph-api/src/main/java/org/apache/hugegraph/auth/HugeGraphAuthProxy.java:
##########
@@ -2461,4 +2518,36 @@ public String toString() {
             return this.origin.toString();
         }
     }
+
+    private static Set<HugePermission> traversalPermissions(
+                                       Traversal.Admin<?, ?> traversal) {
+        Set<HugePermission> permissions = EnumSet.noneOf(HugePermission.class);
+        collectTraversalPermissions(traversal, permissions);
+        return permissions;
+    }
+
+    private static void collectTraversalPermissions(
+                        Traversal.Admin<?, ?> traversal,
+                        Set<HugePermission> permissions) {
+        for (Step<?, ?> step : traversal.getSteps()) {
+            if (step instanceof AddVertexStartStep ||

Review Comment:
   Final fixture-packaging hardening in 2ed446b3: moved the exact-package 
TinkerPop 3.7 compatibility fixtures from `src/main/java` to `src/test/java`, 
added `target/test-classes` only to the unit-test execution classpath, and made 
the main-source regression test construct fixtures reflectively. The four 
vertex/edge create/onMatch strategy tests, exact-package child recursion test, 
and external same-simple-name negative test remain active. Clean focused run: 
28 tests passed; targeted API/test Checkstyle passed. After a clean build, 
exact merge fixtures exist only in `target/test-classes`; exact 
`org/apache/tinkerpop/.../Merge(Vertex|Edge)Step.class` entries are absent from 
`target/classes` and the main JAR input/archive.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to