+ Switching Comparison from String to JSON (apache#15364)

317brian · Nov 14, 2023 · 5edeac2 · 5edeac2
1 parent a134cc3
commit 5edeac2
Showing 1 changed file with 5 additions and 6 deletions.
diff --git a/...java/org/apache/druid/query/aggregation/histogram/ApproximateHistogramAggregatorTest.java b/...java/org/apache/druid/query/aggregation/histogram/ApproximateHistogramAggregatorTest.java
@@ -19,6 +19,7 @@
 
 package org.apache.druid.query.aggregation.histogram;
 
+import com.fasterxml.jackson.databind.JsonNode;
 import org.apache.druid.jackson.DefaultObjectMapper;
 import org.apache.druid.java.util.common.granularity.Granularities;
 import org.apache.druid.query.Druids;
@@ -106,14 +107,12 @@ public void testFinalize() throws Exception
 
     ApproximateHistogramAggregator agg = new ApproximateHistogramAggregator(selector, resolution, 0, 100);
     agg.aggregate();
-
     Object finalizedObjectHumanReadable = humanReadableFactory.finalizeComputation(agg.get());
     String finalStringHumanReadable = objectMapper.writeValueAsString(finalizedObjectHumanReadable);
-    Assert.assertEquals(
-        "{\"breaks\":[23.0,23.0,23.0,23.0,23.0,23.0],\"counts\":[0.0,0.0,0.0,0.0,0.0]}",
-        finalStringHumanReadable
-    );
-
+    JsonNode expectedJson = objectMapper.readTree(
+            "{\"breaks\":[23.0,23.0,23.0,23.0,23.0,23.0],\"counts\":[0.0,0.0,0.0,0.0,0.0]}");
+    JsonNode actualJson = objectMapper.readTree(finalStringHumanReadable);
+    Assert.assertEquals(expectedJson, actualJson);
     Object finalizedObjectBinary = binaryFactory.finalizeComputation(agg.get());
     String finalStringBinary = objectMapper.writeValueAsString(finalizedObjectBinary);
     Assert.assertEquals(