modifty kout request param order (#112)

apache · Dec 15, 2020 · c0fc36d · c0fc36d
1 parent d38fd7c
commit c0fc36d
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 3 deletions.
diff --git a/src/main/java/com/baidu/hugegraph/api/traverser/KoutAPI.java b/src/main/java/com/baidu/hugegraph/api/traverser/KoutAPI.java
@@ -43,7 +43,7 @@ protected String type() {
 
     public List<Object> get(Object sourceId, Direction direction,
                             String label, int depth, boolean nearest,
-                            long degree, long limit, long capacity) {
+                            long degree, long capacity, long limit) {
         String source = GraphAPI.formatVertexId(sourceId, false);
 
         checkPositive(depth, "Depth of k-out");

diff --git a/src/test/java/com/baidu/hugegraph/api/traverser/KoutApiTest.java b/src/test/java/com/baidu/hugegraph/api/traverser/KoutApiTest.java
@@ -110,14 +110,27 @@ public void testKoutGetBothAllWithCapacity() {
 
         Assert.assertThrows(ServerException.class, () -> {
             koutAPI.get(markoId, Direction.BOTH, null,
-                        2, false, -1L, -1L, 1L);
+                        2, false, -1L, 1L, 2L);
         }, e -> {
             String expect = "Capacity can't be less than limit, " +
-                    "but got capacity '1' and limit '-1'";
+                            "but got capacity '1' and limit '2'";
             Assert.assertContains(expect, e.getMessage());
         });
     }
 
+    @Test
+    public void testKoutGetBothAllWithCapacityNoLimit() {
+        Object markoId = getVertexId("person", "name", "marko");
+
+        Assert.assertThrows(ServerException.class, () -> {
+            koutAPI.get(markoId, Direction.BOTH, null,
+                    2, false, -1L, 1L, -1L);
+        }, e -> {
+            String expect = "Capacity can't be less than limit, " +
+                            "but got capacity '1' and limit '-1'";
+            Assert.assertContains(expect, e.getMessage());
+        });
+    }
 
     @Test
     public void testKoutPost() {