Add required_capability: fn_contains to trigger CONTAINS test cases replacing skip logic.

Author: mjmbischoff

x-pack/plugin/esql/qa/testFixtures/src/main/resources/string.csv-spec

@@ -389,7 +389,8 @@ emp_no:integer | name:keyword
 ;
 
 
-contains#[skip:-9.2.99,reason:new string function added in 9.3]
+contains
+required_capability: fn_contains
 // tag::contains[]
 ROW a = "hello"
 | EVAL has_ll = CONTAINS(a, "ll")
@@ -402,28 +403,32 @@ hello     | true
 // end::contains-result[]
 ;
 
-containsFail#[skip:-9.2.99,reason:new string function added in 9.3]
+containsFail
+required_capability: fn_contains
 row a = "hello" | eval a_ll = contains(a, "int");
 
 a:keyword | a_ll:boolean
 hello | false
 ;
 
-containsLongerSubstr#[skip:-9.2.99,reason:new string function added in 9.3]
+containsLongerSubstr
+required_capability: fn_contains
 row a = "hello" | eval a_ll = contains(a, "farewell");
 
 a:keyword | a_ll:boolean
 hello | false
 ;
 
-containsSame#[skip:-9.2.99,reason:new string function added in 9.3]
+containsSame
+required_capability: fn_contains
 row a = "hello" | eval a_ll = contains(a, "hello");
 
 a:keyword | a_ll:boolean
 hello | true
 ;
 
-containsWithSubstring#[skip:-9.2.99,reason:new string function added in 9.3]
+containsWithSubstring
+required_capability: fn_contains
 from employees | where emp_no <= 10010 | eval f_s = substring(last_name, 2) | eval f_l = contains(last_name, f_s) | keep emp_no, last_name, f_s, f_l;
 ignoreOrder:true
 
@@ -440,28 +445,32 @@ emp_no:integer | last_name:keyword | f_s:keyword | f_l:boolean
 10010 | Piveteau  | iveteau | true
 ;
 
-containsUtf16Emoji#[skip:-9.2.99,reason:new string function added in 9.3]
+containsUtf16Emoji
+required_capability: fn_contains
 row a = "🐱Meow!🐶Woof!" | eval f_s = substring(a, 2) | eval f_l = contains(a, f_s);
 
 a:keyword | f_s:keyword | f_l:boolean
 🐱Meow!🐶Woof! | Meow!🐶Woof! | true
 ;
 
-containsNestedCase#[skip:-9.2.99,reason:new string function added in 9.3]
+containsNestedCase
+required_capability: fn_contains
 row a = "hello" | eval a_ll = CASE(contains(a, "ll"), "success","fail");
 
 a:keyword | a_ll:keyword
 hello | success
 ;
 
-containsNestSubstring#[skip:-9.2.99,reason:new string function added in 9.3]
+containsNestSubstring
+required_capability: fn_contains
 row a = "hello" | eval a_ll = contains(substring(a, 2), "ll");
 
 a:keyword | a_ll:boolean
 hello | true
 ;
 
-containsWarnings#[skip:-9.2.99,reason:new string function added in 9.3]
+containsWarnings
+required_capability: fn_contains
 
 from hosts | where host=="epsilon" | eval l1 = contains(host_group, "ate"), l2 = contains(description, "ate") | keep l1, l2;
 ignoreOrder:true

x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/action/EsqlCapabilities.java

@@ -190,6 +190,11 @@ public enum Cap {
          */
         FN_REVERSE_GRAPHEME_CLUSTERS,
 
+        /**
+         * Support for function {@code CONTAINS}. Done in <a href="https://github.com/elastic/elasticsearch/pull/133016">#133016.</a>
+         */
+        FN_CONTAINS,
+
         /**
          * Support for function {@code CBRT}. Done in #108574.
          */

x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/expression/function/scalar/string/Contains.java

@@ -103,8 +103,7 @@ static boolean process(BytesRef str, BytesRef substr) {
         if (str == null || substr == null || str.length < substr.length) {
             return false;
         }
-        String utf8ToString = str.utf8ToString();
-        return utf8ToString.contains(substr.utf8ToString());
+        return str.utf8ToString().contains(substr.utf8ToString());
     }
 
     @Override