Fix more tests

Author: JonasKunz

x-pack/plugin/esql/compute/src/main/java/org/elasticsearch/compute/data/ExponentialHistogramArrayBlock.java

@@ -246,6 +246,12 @@ public double min() {
             return Double.NaN;
         }
 
+        @Override
+        public double max() {
+            // TODO: implement
+            return Double.NaN;
+        }
+
         @Override
         public long ramBytesUsed() {
             return 0;

x-pack/plugin/esql/compute/src/test/java/org/elasticsearch/compute/operator/topn/TopNOperatorTests.java

@@ -603,7 +603,7 @@ public void testCollectAllValues_RandomMultiValues() {
 
         for (int type = 0; type < blocksCount; type++) {
             ElementType e = randomFrom(ElementType.values());
-            if (e == ElementType.UNKNOWN || e == COMPOSITE || e == AGGREGATE_METRIC_DOUBLE) {
+            if (e == ElementType.UNKNOWN || e == COMPOSITE || e == AGGREGATE_METRIC_DOUBLE || e == EXPONENTIAL_HISTOGRAM) {
                 continue;
             }
             elementTypes.add(e);

x-pack/plugin/esql/qa/testFixtures/src/main/java/org/elasticsearch/xpack/esql/EsqlTestUtils.java

@@ -893,13 +893,19 @@ public static Literal randomLiteral(DataType type) {
                 }
             }
             case DENSE_VECTOR -> Arrays.asList(randomArray(10, 10, i -> new Float[10], ESTestCase::randomFloat));
+            case EXPONENTIAL_HISTOGRAM -> new WriteableExponentialHistogram(randomExponentialHistogram());
             // TODO: implement
-            case UNSUPPORTED, OBJECT, DOC_DATA_TYPE, TSID_DATA_TYPE, PARTIAL_AGG, EXPONENTIAL_HISTOGRAM -> throw new IllegalArgumentException(
+            case UNSUPPORTED, OBJECT, DOC_DATA_TYPE, TSID_DATA_TYPE, PARTIAL_AGG -> throw new IllegalArgumentException(
                 "can't make random values for [" + type.typeName() + "]"
             );
         }, type);
     }
 
+    private static ExponentialHistogram randomExponentialHistogram() {
+        //TODO: implement properly with an actual histogram from random data
+        return ExponentialHistogram.empty();
+    }
+
     static Version randomVersion() {
         // TODO degenerate versions and stuff
         return switch (between(0, 2)) {

x-pack/plugin/esql/qa/testFixtures/src/main/java/org/elasticsearch/xpack/esql/JsonBackedExponentialHistogram.java

@@ -80,6 +80,12 @@ public double min() {
         return Double.NaN;
     }
 
+    @Override
+    public double max() {
+        // TODO: implement
+        return Double.NaN;
+    }
+
     @Override
     public ZeroBucket zeroBucket() {
         //TODO: implement

x-pack/plugin/esql/compute/test/src/main/java/org/elasticsearch/compute/test/WriteableExponentialHistogram.java renamed to x-pack/plugin/esql/qa/testFixtures/src/main/java/org/elasticsearch/xpack/esql/WriteableExponentialHistogram.java

@@ -5,7 +5,7 @@
  * 2.0.
  */
 
-package org.elasticsearch.compute.test;
+package org.elasticsearch.xpack.esql;
 
 import org.elasticsearch.TransportVersion;
 import org.elasticsearch.common.io.stream.GenericNamedWriteable;
@@ -27,7 +27,7 @@
  * so that it can be used e.g. in {@link org.elasticsearch.xpack.esql.core.expression.Literal}s.
  * Only intended for testing purposes.
  */
-class WriteableExponentialHistogram extends AbstractExponentialHistogram implements GenericNamedWriteable {
+public class WriteableExponentialHistogram extends AbstractExponentialHistogram implements GenericNamedWriteable {
 
     private static final String WRITEABLE_NAME = "test_exponential_histogram";
 

x-pack/plugin/esql/src/test/java/org/elasticsearch/xpack/esql/SerializationTestUtils.java

@@ -123,6 +123,7 @@ public static NamedWriteableRegistry writableRegistry() {
                 AggregateMetricDoubleBlockBuilder.AggregateMetricDoubleLiteral::new
             )
         );
+        entries.add(WriteableExponentialHistogram.ENTRY);
         return new NamedWriteableRegistry(entries);
     }
 }

x-pack/plugin/esql/src/test/java/org/elasticsearch/xpack/esql/plan/physical/AbstractPhysicalPlanSerializationTests.java

@@ -11,6 +11,7 @@
 import org.elasticsearch.common.settings.Settings;
 import org.elasticsearch.compute.data.AggregateMetricDoubleBlockBuilder;
 import org.elasticsearch.search.SearchModule;
+import org.elasticsearch.xpack.esql.WriteableExponentialHistogram;
 import org.elasticsearch.xpack.esql.core.tree.Node;
 import org.elasticsearch.xpack.esql.expression.ExpressionWritables;
 import org.elasticsearch.xpack.esql.expression.predicate.operator.arithmetic.Add;
@@ -53,6 +54,7 @@ protected final NamedWriteableRegistry getNamedWriteableRegistry() {
         entries.addAll(new SearchModule(Settings.EMPTY, List.of()).getNamedWriteables()); // Query builders
         entries.add(Add.ENTRY); // Used by the eval tests
         entries.add(AggregateMetricDoubleBlockBuilder.AggregateMetricDoubleLiteral.ENTRY);
+        entries.add(WriteableExponentialHistogram.ENTRY);
         return new NamedWriteableRegistry(entries);
     }