Browse Source

ES|QL: Make some REST tests deterministic (#107200)

Adding an explicit SORT to two tests to make them more deterministic,
especially for scenarios where the execution is on multiple nodes.
Luigi Dell'Aquila 1 year ago
parent
commit
cf8fe17766

+ 1 - 1
x-pack/plugin/esql/qa/server/src/main/java/org/elasticsearch/xpack/esql/qa/rest/RestEnrichTestCase.java

@@ -167,7 +167,7 @@ public abstract class RestEnrichTestCase extends ESRestTestCase {
 
     public void testMatchField_ImplicitFieldsList() throws IOException {
         Map<String, Object> result = runEsql(
-            new RestEsqlTestCase.RequestObjectBuilder().query("from test | enrich countries | keep number")
+            new RestEsqlTestCase.RequestObjectBuilder().query("from test | enrich countries | keep number | sort number")
         );
         var columns = List.of(Map.of("name", "number", "type", "long"));
         var values = List.of(List.of(1000), List.of(1000), List.of(5000));

+ 1 - 1
x-pack/plugin/esql/qa/server/src/main/java/org/elasticsearch/xpack/esql/qa/rest/RestEsqlTestCase.java

@@ -322,7 +322,7 @@ public abstract class RestEsqlTestCase extends ESRestTestCase {
             matchesMap().entry("values", List.of(List.of(1))).entry("columns", List.of(Map.of("name", "min(value)", "type", "long")))
         );
 
-        builder = new RequestObjectBuilder().query(fromIndex() + " | stats min(value) by group");
+        builder = new RequestObjectBuilder().query(fromIndex() + " | stats min(value) by group | sort group, `min(value)`");
         result = runEsql(builder);
         assertMap(
             result,