Fix scores in tests

Author: carlosdelest

x-pack/plugin/esql/src/internalClusterTest/java/org/elasticsearch/xpack/esql/plugin/MatchFunctionIT.java

@@ -284,10 +284,9 @@ public void testDisjunctionScoring() {
 
             // Matches full text query and non pushable query
             assertThat((Double) values.get(0).get(1), greaterThan(1.0));
-            // Matches just non pushable query
             assertThat((Double) values.get(1).get(1), greaterThan(1.0));
-            // Matches just full text query
-            assertThat((Double) values.get(2).get(1), equalTo(0.0));
+            // Matches just non pushable query
+            assertThat((Double) values.get(2).get(1), equalTo(1.0));
         }
     }
 
@@ -308,12 +307,12 @@ public void testDisjunctionScoringMultipleNonPushableFunctions() {
             assertThat(values.get(0).get(0), equalTo(1));
             assertThat(values.get(1).get(0), equalTo(6));
 
-            // Matches the full text query and a non pushable query
-            assertThat((Double) values.get(0).get(1), greaterThan(1.0));
-            assertThat((Double) values.get(0).get(1), lessThan(2.0));
+            // Matches the full text query and a two pushable query
+            assertThat((Double) values.get(0).get(1), greaterThan(2.0));
+            assertThat((Double) values.get(0).get(1), lessThan(3.0));
             // Matches just the match function
-            assertThat((Double) values.get(1).get(1), lessThan(1.0));
-            assertThat((Double) values.get(1).get(1), greaterThan(0.0));
+            assertThat((Double) values.get(1).get(1), lessThan(2.0));
+            assertThat((Double) values.get(1).get(1), greaterThan(1.0));
         }
     }
 
@@ -335,10 +334,10 @@ public void testDisjunctionScoringWithNot() {
             assertThat(values.get(1).get(0), equalTo(4));
             assertThat(values.get(2).get(0), equalTo(5));
 
-            // Matches NOT and non pushable query gets score of 0.0
-            assertThat((Double) values.get(0).get(1), equalTo(0.0));
-            assertThat((Double) values.get(1).get(1), equalTo(0.0));
-            assertThat((Double) values.get(2).get(1), equalTo(0.0));
+            // Matches NOT gets 0.0 and default score is 1.0
+            assertThat((Double) values.get(0).get(1), equalTo(1.0));
+            assertThat((Double) values.get(1).get(1), equalTo(1.0));
+            assertThat((Double) values.get(2).get(1), equalTo(1.0));
         }
     }