Fix signed zero equality behavior for doubles to match specification

PiperOrigin-RevId: 809130214

Author: l46kok

common/src/main/java/dev/cel/common/internal/ComparisonFunctions.java

@@ -81,7 +81,8 @@ public static int compareUintInt(UnsignedLong ul, long l) {
   public static boolean numericEquals(Number x, Number y) {
     if (x instanceof Double) {
       if (y instanceof Double) {
-        return !(Double.isNaN((Double) x) || Double.isNaN((Double) y)) && x.equals(y);
+        return !(Double.isNaN((Double) x) || Double.isNaN((Double) y))
+            && x.doubleValue() == y.doubleValue();
       }
       if (y instanceof Long) {
         return compareDoubleInt((Double) x, (Long) y) == 0;
@@ -115,10 +116,7 @@ public static boolean numericEquals(Number x, Number y) {
     return false;
   }
 
-
-  /**
-   * Compare two numeric values of any type (double, int, uint).
-   */
+  /** Compare two numeric values of any type (double, int, uint). */
   public static int numericCompare(Number x, Number y) {
     if (x instanceof Double) {
       if (y instanceof Double) {

runtime/src/test/resources/arithmDouble.baseline

@@ -1,3 +1,8 @@
+Source: 0.0 == -0.0
+=====>
+bindings: {}
+result:   true
+
 Source: 1.9 < 2.0 && 1.1 <= 1.1 && 2.0 > 1.9 && 1.1 >= 1.1 && 1.1 == 1.1 && 2.0 != 1.9
 =====>
 bindings: {}

testing/src/main/java/dev/cel/testing/BaseInterpreterTest.java

@@ -306,6 +306,9 @@ public void arithmUInt64_error() {
 
   @Test
   public void arithmDouble() {
+    source = "0.0 == -0.0";
+    runTest();
+
     source = "1.9 < 2.0 && 1.1 <= 1.1 && 2.0 > 1.9 && 1.1 >= 1.1 && 1.1 == 1.1 && 2.0 != 1.9";
     runTest();