Support parsing '-nan' (#3329)

Due to usages of `sprintf(str, "%.15g", d);` when d is NaN. From @LiorKogan: "We should treat `-nan` as `nan`". References: - RedisGraph/RedisGraph#2930
redis · Mar 22, 2023 · 281003b · 281003b
1 parent fe96e93
commit 281003b
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 0 deletions.
diff --git a/src/main/java/redis/clients/jedis/util/DoublePrecision.java b/src/main/java/redis/clients/jedis/util/DoublePrecision.java
@@ -26,6 +26,7 @@ public static Double parseFloatingPointNumber(String str) {
           return Double.NEGATIVE_INFINITY;
 
         case "nan":
+        case "-nan": // for some module commands // TODO: remove
           return Double.NaN;
 
         default:

diff --git a/src/test/java/redis/clients/jedis/modules/graph/GraphValuesTest.java b/src/test/java/redis/clients/jedis/modules/graph/GraphValuesTest.java
@@ -38,4 +38,12 @@ public void parseNaN() {
     Record r = rs.iterator().next();
     assertEquals(Double.NaN, r.getValue(0), 0d);
   }
+
+  @Test
+  public void parseMinusNaN() {
+    ResultSet rs = client.graphQuery("db", "RETURN sqrt(-1)");
+    assertEquals(1, rs.size());
+    Record r = rs.iterator().next();
+    assertEquals(Double.NaN, r.getValue(0), 0d);
+  }
 }