You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@hugegraph.apache.org by GitBox <gi...@apache.org> on 2022/04/26 06:33:08 UTC

[GitHub] [incubator-hugegraph] JackyYangPassion commented on a diff in pull request #1842: fix hugegraph-test checkstyle issue and move java class to test/java

JackyYangPassion commented on code in PR #1842:
URL: https://github.com/apache/incubator-hugegraph/pull/1842#discussion_r858327935


##########
hugegraph-test/src/main/java/com/baidu/hugegraph/core/AuthTest.java:
##########
@@ -110,9 +110,9 @@ public void testCreateUser() {
         Assert.assertThrows(IllegalArgumentException.class, () -> {
             authManager.createUser(makeUser("tom", "pass1"));
         }, e -> {
-            Assert.assertContains("Can't save user", e.getMessage());
-            Assert.assertContains("that already exists", e.getMessage());
-        });
+                Assert.assertContains("Can't save user", e.getMessage());

Review Comment:
   means keeping 12 indents, not 16 ?



-- 
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: dev-unsubscribe@hugegraph.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org