Standardize test assertions a bit for DedupTest
Project: http://git-wip-us.apache.org/repos/asf/tinkerpop/repo Commit: http://git-wip-us.apache.org/repos/asf/tinkerpop/commit/f9311114 Tree: http://git-wip-us.apache.org/repos/asf/tinkerpop/tree/f9311114 Diff: http://git-wip-us.apache.org/repos/asf/tinkerpop/diff/f9311114 Branch: refs/heads/TINKERPOP-1437 Commit: f9311114108591d7e61a4b0560273c1c092aefef Parents: c621154 Author: Stephen Mallette <sp...@genoprime.com> Authored: Sat Sep 17 04:01:34 2016 -0400 Committer: Stephen Mallette <sp...@genoprime.com> Committed: Sat Sep 17 04:01:34 2016 -0400 ---------------------------------------------------------------------- .../step/filter/GroovyDedupTest.groovy | 4 +-- .../traversal/step/filter/DedupTest.java | 36 ++++++++------------ 2 files changed, 16 insertions(+), 24 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/f9311114/gremlin-groovy-test/src/main/groovy/org/apache/tinkerpop/gremlin/process/traversal/step/filter/GroovyDedupTest.groovy ---------------------------------------------------------------------- diff --git a/gremlin-groovy-test/src/main/groovy/org/apache/tinkerpop/gremlin/process/traversal/step/filter/GroovyDedupTest.groovy b/gremlin-groovy-test/src/main/groovy/org/apache/tinkerpop/gremlin/process/traversal/step/filter/GroovyDedupTest.groovy index cfad92b..d03f526 100644 --- a/gremlin-groovy-test/src/main/groovy/org/apache/tinkerpop/gremlin/process/traversal/step/filter/GroovyDedupTest.groovy +++ b/gremlin-groovy-test/src/main/groovy/org/apache/tinkerpop/gremlin/process/traversal/step/filter/GroovyDedupTest.groovy @@ -33,11 +33,11 @@ public abstract class GroovyDedupTest { public static class Traversals extends DedupTest { @Override public Traversal<Vertex, String> get_g_V_out_in_valuesXnameX_fold_dedupXlocalX_unfold() { - return new ScriptTraversal<>(g, "gremlin-groovy", "g.V.out.in.values('name').fold.dedup(Scope.local).unfold;"); + return new ScriptTraversal<>(g, "gremlin-groovy", "g.V.out.in.values('name').fold.dedup(Scope.local).unfold"); } @Override - public Traversal<Vertex, Map<String, String>> get_g_V_out_in_asXxX_in_asXyX_selectXx_yX_byXnameX_fold_dedupXlocal_x_yX_unfold() { + public Traversal<Vertex, Map<String, String>> get_g_V_out_asXxX_in_asXyX_selectXx_yX_byXnameX_fold_dedupXlocal_x_yX_unfold() { return new ScriptTraversal<>(g, "gremlin-groovy", "g.V.out.as('x').in.as('y').select('x','y').by('name').fold.dedup(Scope.local,'x','y').unfold"); } http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/f9311114/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/filter/DedupTest.java ---------------------------------------------------------------------- diff --git a/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/filter/DedupTest.java b/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/filter/DedupTest.java index a0cc6e9..fa55c8e 100644 --- a/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/filter/DedupTest.java +++ b/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/filter/DedupTest.java @@ -79,36 +79,28 @@ public abstract class DedupTest extends AbstractGremlinProcessTest { public abstract Traversal<Vertex, String> get_g_V_out_in_valuesXnameX_fold_dedupXlocalX_unfold(); - public abstract Traversal<Vertex, Map<String, String>> get_g_V_out_in_asXxX_in_asXyX_selectXx_yX_byXnameX_fold_dedupXlocal_x_yX_unfold(); + public abstract Traversal<Vertex, Map<String, String>> get_g_V_out_asXxX_in_asXyX_selectXx_yX_byXnameX_fold_dedupXlocal_x_yX_unfold(); @Test @LoadGraphWith(MODERN) public void g_V_out_in_valuesXnameX_fold_dedupXlocalX_unfold() { final Traversal<Vertex, String> traversal = get_g_V_out_in_valuesXnameX_fold_dedupXlocalX_unfold(); - final List<String> names = traversal.toList(); - assertEquals(3, names.size()); - assertThat(names, containsInAnyOrder("marko", "josh", "peter")); + printTraversalForm(traversal); + checkResults(Arrays.asList("marko", "josh", "peter"), traversal); } @Test @LoadGraphWith(MODERN) - public void g_V_out_in_asXxX_in_asXyX_selectXx_yX_byXnameX_fold_dedupXlocal_x_yX_unfold() { - final Traversal<Vertex, Map<String, String>> traversal = get_g_V_out_in_asXxX_in_asXyX_selectXx_yX_byXnameX_fold_dedupXlocal_x_yX_unfold(); - final List<Map<String,String>> names = traversal.toList(); - assertEquals(6, names.size()); - for (final Map<String, String> m : names) { - if (m.get("x").equals("lop")) { - assertThat(m.get("y"), isIn(Arrays.asList("marko", "josh", "peter"))); - } else if (m.get("x").equals("vadas")) { - assertEquals("marko", m.get("y")); - } else if (m.get("x").equals("josh")) { - assertEquals("marko", m.get("y")); - } else if (m.get("x").equals("ripple")) { - assertEquals("josh", m.get("y")); - } else { - fail("A value was returned that was not expected in the result: " + m.get("x")); - } - } + public void g_V_out_asXxX_in_asXyX_selectXx_yX_byXnameX_fold_dedupXlocal_x_yX_unfold() { + final Traversal<Vertex, Map<String, String>> traversal = get_g_V_out_asXxX_in_asXyX_selectXx_yX_byXnameX_fold_dedupXlocal_x_yX_unfold(); + printTraversalForm(traversal); + checkResults(makeMapList(2, + "x", "lop", "y", "marko", + "x", "lop", "y", "josh", + "x", "lop", "y", "peter", + "x", "vadas", "y", "marko", + "x", "josh", "y", "marko", + "x", "ripple", "y", "josh"), traversal); } @Test @@ -284,7 +276,7 @@ public abstract class DedupTest extends AbstractGremlinProcessTest { } @Override - public Traversal<Vertex, Map<String, String>> get_g_V_out_in_asXxX_in_asXyX_selectXx_yX_byXnameX_fold_dedupXlocal_x_yX_unfold() { + public Traversal<Vertex, Map<String, String>> get_g_V_out_asXxX_in_asXyX_selectXx_yX_byXnameX_fold_dedupXlocal_x_yX_unfold() { return g.V().out().as("x").in().as("y").select("x","y").by("name").fold().dedup(Scope.local,"x","y").unfold(); }