Add test script_score error on negative score

7.5 and 7.6 had a regression that allowed for
script_score queries to have negative scores.
We have corrected this regression in #52478.
This is an addition to #52478 that adds a test for this.

Related to #53133

Author: mayya-sharipova

server/src/test/java/org/elasticsearch/search/query/ScriptScoreQueryTests.java

@@ -53,7 +53,7 @@
 import static org.mockito.Mockito.when;
 
 public class ScriptScoreQueryTests extends ESTestCase {
-    
+
     private Directory dir;
     private IndexWriter w;
     private DirectoryReader reader;
@@ -131,6 +131,14 @@ public void testExplainDefaultNoScore() throws IOException {
         assertThat(explanation.getValue(), equalTo(2.0f));
     }
 
+    public void testScriptScoreErrorOnNegativeScore() {
+        Script script = new Script("script that returns a negative score");
+        ScoreScript.LeafFactory factory = newFactory(script, false, explanation -> -1000.0);
+        ScriptScoreQuery query = new ScriptScoreQuery(Queries.newMatchAllQuery(), script, factory, null, "index", 0, Version.CURRENT);
+        IllegalArgumentException e = expectThrows(IllegalArgumentException.class, () -> searcher.search(query, 1));
+        assertTrue(e.getMessage().contains("Must be a non-negative score!"));
+    }
+
     private ScoreScript.LeafFactory newFactory(Script script, boolean needsScore,
                                                Function<ScoreScript.ExplanationHolder, Double> function) {
         SearchLookup lookup = mock(SearchLookup.class);