Fix

Author: luigidellaquila

x-pack/plugin/build.gradle

@@ -134,6 +134,7 @@ tasks.named("yamlRestCompatTestTransform").configure({ task ->
   task.skipTest("esql/90_non_indexed/fetch", "Temporary until backported")
   task.skipTest("esql/63_enrich_int_range/Invalid age as double", "TODO: require disable allow_partial_results")
   task.skipTest("esql/191_lookup_join_on_datastreams/data streams not supported in LOOKUP JOIN", "Added support for aliases in JOINs")
+  task.skipTest("esql/10_basic/Test wrong LIMIT parameter", "Different error message")
 })
 
 tasks.named('yamlRestCompatTest').configure {

x-pack/plugin/esql-core/src/test/java/org/elasticsearch/xpack/esql/core/util/TestUtils.java

@@ -7,6 +7,7 @@
 
 package org.elasticsearch.xpack.esql.core.util;
 
+import org.elasticsearch.common.lucene.BytesRefs;
 import org.elasticsearch.xpack.esql.core.expression.FieldAttribute;
 import org.elasticsearch.xpack.esql.core.expression.Literal;
 import org.elasticsearch.xpack.esql.core.tree.Source;
@@ -22,6 +23,8 @@
 import static org.elasticsearch.test.ESTestCase.randomFrom;
 import static org.elasticsearch.xpack.esql.core.tree.Source.EMPTY;
 import static org.elasticsearch.xpack.esql.core.type.DataType.INTEGER;
+import static org.elasticsearch.xpack.esql.core.type.DataType.KEYWORD;
+import static org.elasticsearch.xpack.esql.core.type.DataType.TEXT;
 
 public final class TestUtils {
     private TestUtils() {}
@@ -39,6 +42,10 @@ public static Literal of(Source source, Object value) {
         if (value instanceof Literal) {
             return (Literal) value;
         }
+        DataType type = DataType.fromJava(value);
+        if (value instanceof String && (type == TEXT || type == KEYWORD)) {
+            value = BytesRefs.toBytesRef(value);
+        }
         return new Literal(source, value, DataType.fromJava(value));
     }
 
@@ -58,12 +65,16 @@ public static FieldAttribute getFieldAttribute(String name, DataType dataType) {
         return new FieldAttribute(EMPTY, name, new EsField(name + "f", dataType, emptyMap(), true));
     }
 
-    /** Similar to {@link String#strip()}, but removes the WS throughout the entire string. */
+    /**
+     * Similar to {@link String#strip()}, but removes the WS throughout the entire string.
+     */
     public static String stripThrough(String input) {
         return WS_PATTERN.matcher(input).replaceAll(StringUtils.EMPTY);
     }
 
-    /** Returns the input string, but with parts of it having the letter casing changed. */
+    /**
+     * Returns the input string, but with parts of it having the letter casing changed.
+     */
     public static String randomCasing(String input) {
         StringBuilder sb = new StringBuilder(input.length());
         for (int i = 0, inputLen = input.length(), step = (int) Math.sqrt(inputLen); i < inputLen; i += step) {