Fixed test case that ultimately failed in `master/` (coerced `BigDecimal` to 
`Double`).


Project: http://git-wip-us.apache.org/repos/asf/tinkerpop/repo
Commit: http://git-wip-us.apache.org/repos/asf/tinkerpop/commit/86edf56e
Tree: http://git-wip-us.apache.org/repos/asf/tinkerpop/tree/86edf56e
Diff: http://git-wip-us.apache.org/repos/asf/tinkerpop/diff/86edf56e

Branch: refs/heads/TINKERPOP-1682
Commit: 86edf56e98742a357891e8557c822a75e91921be
Parents: 9bc2d46
Author: Daniel Kuppitz <daniel_kupp...@hotmail.com>
Authored: Thu Jul 27 18:07:25 2017 +0200
Committer: Daniel Kuppitz <daniel_kupp...@hotmail.com>
Committed: Mon Jul 31 18:00:48 2017 +0200

----------------------------------------------------------------------
 .../process/traversal/step/sideEffect/SackTest.java     | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/86edf56e/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/sideEffect/SackTest.java
----------------------------------------------------------------------
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 f2abc11..bef19ee 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
@@ -67,7 +67,7 @@ public abstract class SackTest extends 
AbstractGremlinProcessTest {
 
     public abstract Traversal<Vertex, BigDecimal> 
get_g_withSackXBigInteger_TEN_powX1000X_assignX_V_localXoutXknowsX_barrierXnormSackXX_inXknowsX_barrier_sack();
 
-    public abstract Traversal<Vertex, BigDecimal> 
get_g_withSackX2X_V_sackXdivX_byXconstantXBigDecimal_valueOfX3XXX_sack();
+    public abstract Traversal<Vertex, Double> 
get_g_withSackX2X_V_sackXdivX_byXconstantX3_0XX_sack();
 
     @Test
     @LoadGraphWith(MODERN)
@@ -148,13 +148,13 @@ public abstract class SackTest extends 
AbstractGremlinProcessTest {
 
     @Test
     @LoadGraphWith(MODERN)
-    public void 
g_withSackX2X_V_sackXdivX_byXconstantXBigDecimal_valueOfX3XXX_sack() {
-        final Traversal<Vertex, BigDecimal> traversal = 
get_g_withSackX2X_V_sackXdivX_byXconstantXBigDecimal_valueOfX3XXX_sack();
+    public void g_withSackX2X_V_sackXdivX_byXconstantX3_0XX_sack() {
+        final Traversal<Vertex, Double> 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());
     }
@@ -205,8 +205,8 @@ public abstract class SackTest extends 
AbstractGremlinProcessTest {
         }
 
         @Override
-        public Traversal<Vertex, BigDecimal> 
get_g_withSackX2X_V_sackXdivX_byXconstantXBigDecimal_valueOfX3XXX_sack() {
-            return 
g.withSack(2).V().sack(Operator.div).by(constant(BigDecimal.valueOf(3))).sack();
+        public Traversal<Vertex, Double> 
get_g_withSackX2X_V_sackXdivX_byXconstantX3_0XX_sack() {
+            return 
g.withSack(2).V().sack(Operator.div).by(constant(3.0)).sack();
         }
     }
 }

Reply via email to