From 3d8c03bb2c3cdd09a44e8b163d62db6d18773490 Mon Sep 17 00:00:00 2001 From: Daniel Kuppitz Date: Thu, 27 Jul 2017 18:07:25 +0200 Subject: [PATCH] Fixed test case that ultimately failed in `master/` (coerced `BigDecimal` to `Double`). --- .../step/sideEffect/GroovySackTest.groovy | 2 +- .../traversal/step/sideEffect/SackTest.java | 14 +++++++------- 2 files changed, 8 insertions(+), 8 deletions(-) diff --git a/gremlin-groovy-test/src/main/groovy/org/apache/tinkerpop/gremlin/process/traversal/step/sideEffect/GroovySackTest.groovy b/gremlin-groovy-test/src/main/groovy/org/apache/tinkerpop/gremlin/process/traversal/step/sideEffect/GroovySackTest.groovy index d1681cba38c..725e03e1c4b 100644 --- a/gremlin-groovy-test/src/main/groovy/org/apache/tinkerpop/gremlin/process/traversal/step/sideEffect/GroovySackTest.groovy +++ b/gremlin-groovy-test/src/main/groovy/org/apache/tinkerpop/gremlin/process/traversal/step/sideEffect/GroovySackTest.groovy @@ -76,7 +76,7 @@ public abstract class GroovySackTest { } @Override - Traversal get_g_withSackX2X_V_sackXdivX_byXconstantXBigDecimal_valueOfX3XXX_sack() { + Traversal get_g_withSackX2X_V_sackXdivX_byXconstantX3_0XX_sack() { TraversalScriptHelper.compute("g.withSack(2).V.sack(div).by(constant(3.0)).sack", g) } } diff --git a/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/sideEffect/SackTest.java b/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/sideEffect/SackTest.java index 63f3649ef08..7903b074cc8 100644 --- a/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/sideEffect/SackTest.java +++ b/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/sideEffect/SackTest.java @@ -69,7 +69,7 @@ public abstract class SackTest extends AbstractGremlinProcessTest { public abstract Traversal get_g_withSackXBigInteger_TEN_powX1000X_assignX_V_localXoutXknowsX_barrierXnormSackXX_inXknowsX_barrier_sack(); - public abstract Traversal get_g_withSackX2X_V_sackXdivX_byXconstantXBigDecimal_valueOfX3XXX_sack(); + public abstract Traversal get_g_withSackX2X_V_sackXdivX_byXconstantX3_0XX_sack(); @Test @LoadGraphWith(MODERN) @@ -161,13 +161,13 @@ public void g_withSackXBigInteger_TEN_powX1000X_assignX_V_localXoutXknowsX_barri @Test @LoadGraphWith(MODERN) - public void g_withSackX2X_V_sackXdivX_byXconstantXBigDecimal_valueOfX3XXX_sack() { - final Traversal traversal = get_g_withSackX2X_V_sackXdivX_byXconstantXBigDecimal_valueOfX3XXX_sack(); + public void g_withSackX2X_V_sackXdivX_byXconstantX3_0XX_sack() { + final Traversal traversal = get_g_withSackX2X_V_sackXdivX_byXconstantX3_0XX_sack(); printTraversalForm(traversal); final double expected = 2.0 / 3.0; for (int i = 0; i < 6; i++) { assertTrue(traversal.hasNext()); - assertEquals(expected, traversal.next().doubleValue(), 0.0001); + assertEquals(expected, ((Number) traversal.next()).doubleValue(), 0.0001); } assertFalse(traversal.hasNext()); } @@ -223,8 +223,8 @@ public Traversal get_g_withSackXBigInteger_TEN_powX1000X_ass } @Override - public Traversal get_g_withSackX2X_V_sackXdivX_byXconstantXBigDecimal_valueOfX3XXX_sack() { - return g.withSack(2).V().sack(Operator.div).by(constant(BigDecimal.valueOf(3))).sack(); + public Traversal get_g_withSackX2X_V_sackXdivX_byXconstantX3_0XX_sack() { + return g.withSack(2).V().sack(Operator.div).by(constant(3.0)).sack(); } } -} \ No newline at end of file +}