Fix testValidFromPattern (elastic#121996)

(cherry picked from commit c92b7b5)

Author: idegtiarenko

muted-tests.yml

@@ -363,9 +363,6 @@ tests:
 - class: org.elasticsearch.xpack.migrate.action.ReindexDatastreamIndexTransportActionIT
   method: testFailIfMetadataBlockSet
   issue: https://github.com/elastic/elasticsearch/issues/121978
-- class: org.elasticsearch.xpack.esql.parser.StatementParserTests
-  method: testValidFromPattern
-  issue: https://github.com/elastic/elasticsearch/issues/121990
 
 # Examples:
 #

x-pack/plugin/esql/src/test/java/org/elasticsearch/xpack/esql/IdentifierGenerator.java

@@ -78,16 +78,17 @@ public static String randomIndexPattern(Feature... features) {
             index.insert(0, "-");
         }
 
-        var pattern = maybeQuote(index.toString());
+        var pattern = index.toString();
+        if (pattern.contains("|")) {
+            pattern = quote(pattern);
+        }
+        pattern = maybeQuote(pattern);
+
         if (canAdd(Features.CROSS_CLUSTER, features)) {
             var cluster = maybeQuote(randomIdentifier());
             pattern = maybeQuote(cluster + ":" + pattern);
         }
 
-        if (pattern.contains("|") && pattern.contains("\"") == false) {
-            pattern = quote(pattern);
-        }
-
         return pattern;
     }