fix(server): fix auth test (#64)

* fix(server): Delete the redundant storage during the test

* fix(server): fix auth test

Author: Tsukilc

hugegraph-server/hugegraph-core/src/main/java/org/apache/hugegraph/auth/HugeBelong.java

@@ -40,6 +40,7 @@ public class HugeBelong extends Relationship {
     public static final String GR = "gr";
     public static final String ALL = "*";
     private static final long serialVersionUID = -7242751631755533423L;
+    public static final String DEFAULT_GRAPH_SPACE = "DEFAULT";
 
     private String graphSpace;
     private Id user;
@@ -49,7 +50,7 @@ public class HugeBelong extends Relationship {
     private String link;
 
     public HugeBelong(Id user, Id group) {
-        this("*", user, group, null, UG);
+        this(DEFAULT_GRAPH_SPACE, user, group, null, UG);
     }
 
     public HugeBelong(String graphSpace, Id user, Id role) {

hugegraph-server/hugegraph-test/pom.xml

@@ -312,26 +312,6 @@
                 <serializer>text</serializer>
             </properties>
         </profile>
-        <profile>
-            <id>cassandra</id>
-            <activation>
-                <activeByDefault>false</activeByDefault>
-            </activation>
-            <properties>
-                <backend>cassandra</backend>
-                <serializer>cassandra</serializer>
-            </properties>
-        </profile>
-        <profile>
-            <id>scylladb</id>
-            <activation>
-                <activeByDefault>false</activeByDefault>
-            </activation>
-            <properties>
-                <backend>scylladb</backend>
-                <serializer>scylladb</serializer>
-            </properties>
-        </profile>
         <profile>
             <id>rocksdb</id>
             <activation>
@@ -342,26 +322,6 @@
                 <serializer>binary</serializer>
             </properties>
         </profile>
-        <profile>
-            <id>mysql</id>
-            <activation>
-                <activeByDefault>false</activeByDefault>
-            </activation>
-            <properties>
-                <backend>mysql</backend>
-                <serializer>mysql</serializer>
-            </properties>
-        </profile>
-        <profile>
-            <id>palo</id>
-            <activation>
-                <activeByDefault>false</activeByDefault>
-            </activation>
-            <properties>
-                <backend>palo</backend>
-                <serializer>palo</serializer>
-            </properties>
-        </profile>
         <profile>
             <id>hbase</id>
             <activation>
@@ -372,16 +332,6 @@
                 <serializer>hbase</serializer>
             </properties>
         </profile>
-        <profile>
-            <id>postgresql</id>
-            <activation>
-                <activeByDefault>false</activeByDefault>
-            </activation>
-            <properties>
-                <backend>postgresql</backend>
-                <serializer>postgresql</serializer>
-            </properties>
-        </profile>
         <profile>
             <id>hstore</id>
             <activation>

hugegraph-server/hugegraph-test/src/main/java/org/apache/hugegraph/core/AuthTest.java

@@ -131,6 +131,9 @@ public void clearAll() {
         for (HugeTarget target : authManager.listAllTargets(-1)) {
             authManager.deleteTarget(target.id());
         }
+        for (HugeBelong belong : authManager.listAllBelong(-1)) {
+            authManager.deleteBelong(belong.id());
+        }
 
         //FIXME: support project in hstore
         boolean isHstore = Objects.equals("hstore", System.getProperty("backend")) ||
@@ -710,12 +713,11 @@ public void testCreateBelong() {
         Map<String, Object> expected = new HashMap<>();
         expected.putAll(ImmutableMap.of("id", belong.id(),
                                         "user", user,
-                                        "graphspace", "*",
+                                        "graphspace", "DEFAULT",
                                         "group", group1));
         expected.putAll(ImmutableMap.of("belong_creator", "admin",
                                         "belong_create", belong.create(),
                                         "belong_update", belong.update()));
-        expected.put("role", null);
         expected.put("link", "ug");
         Assert.assertEquals(expected, belong.asMap());
 
@@ -728,12 +730,11 @@ public void testCreateBelong() {
         expected = new HashMap<>();
         expected.putAll(ImmutableMap.of("id", belong.id(),
                                         "user", user,
-                                        "graphspace", "*",
+                                        "graphspace", "DEFAULT",
                                         "group", group2));
         expected.putAll(ImmutableMap.of("belong_creator", "admin",
                                         "belong_create", belong.create(),
                                         "belong_update", belong.update()));
-        expected.put("role", null);
         expected.put("link", "ug");
         Assert.assertEquals(expected, belong.asMap());
 
@@ -760,13 +761,12 @@ public void testCreateBelong() {
         expected = new HashMap<>();
         expected.putAll(ImmutableMap.of("id", belong.id(),
                                         "user", user1,
-                                        "graphspace", "*",
+                                        "graphspace", "DEFAULT",
                                         "group", group1));
         expected.putAll(ImmutableMap.of("belong_description", "something2",
                                         "belong_creator", "admin",
                                         "belong_create", belong.create(),
                                         "belong_update", belong.update()));
-        expected.put("role", null);
         expected.put("link", "ug");
         Assert.assertEquals(expected, belong.asMap());
 

hugegraph-server/hugegraph-test/src/main/java/org/apache/hugegraph/core/CoreTestSuite.java

@@ -20,6 +20,7 @@
 import org.apache.hugegraph.HugeGraph;
 import org.apache.hugegraph.dist.RegisterUtil;
 import org.apache.hugegraph.masterelection.GlobalMasterInfo;
+import org.apache.hugegraph.meta.MetaManager;
 import org.apache.hugegraph.testutil.Utils;
 import org.apache.hugegraph.util.Log;
 import org.junit.AfterClass;
@@ -74,6 +75,21 @@ public static void init() {
         graph.clearBackend();
         graph.initBackend();
         graph.serverStarted(GlobalMasterInfo.master("server-test"));
+
+        // Initialize DEFAULT graphspace for V2 tests
+        try {
+            MetaManager metaManager =
+                    MetaManager.instance();
+            if (metaManager.isReady()) {
+                metaManager.initDefaultGraphSpace();
+            }
+        } catch (Exception e) {
+            // MetaManager may not be initialized for non-hstore backends
+            LOG.debug(
+                    "Failed to initialize default graphspace (expected for non-hstore backends): " +
+                    "{}",
+                    e.getMessage());
+        }
     }
 
     @AfterClass