Update tests

Author: smalyshev

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

@@ -9,8 +9,10 @@
 
 import org.elasticsearch.ElasticsearchException;
 import org.elasticsearch.client.internal.Client;
+import org.elasticsearch.cluster.RemoteException;
 import org.elasticsearch.common.Strings;
 import org.elasticsearch.common.settings.Settings;
+import org.elasticsearch.index.IndexNotFoundException;
 import org.elasticsearch.index.query.QueryBuilder;
 import org.elasticsearch.index.query.RangeQueryBuilder;
 import org.elasticsearch.test.transport.MockTransportService;
@@ -275,13 +277,13 @@ public void testFilterWithMissingIndex() {
             );
             assertThat(e.getDetailedMessage(), containsString("Unknown index [missing]"));
             // Local index missing + wildcards
-            // FIXME: planner does not catch this now
-            // e = expectThrows(VerificationException.class, () -> runQuery("from missing,logs*", randomBoolean(), filter).close());
-            // assertThat(e.getDetailedMessage(), containsString("Unknown index [missing]"));
+            // FIXME: planner does not catch this now, it should be VerificationException but for now it's runtime IndexNotFoundException
+            var ie = expectThrows(IndexNotFoundException.class, () -> runQuery("from missing,logs*", randomBoolean(), filter).close());
+            assertThat(ie.getDetailedMessage(), containsString("no such index [missing]"));
             // Local index missing + existing index
-            // FIXME: planner does not catch this now
-            // e = expectThrows(VerificationException.class, () -> runQuery("from missing,logs-1", randomBoolean(), filter).close());
-            // assertThat(e.getDetailedMessage(), containsString("Unknown index [missing]"));
+            // FIXME: planner does not catch this now, it should be VerificationException but for now it's runtime IndexNotFoundException
+            ie = expectThrows(IndexNotFoundException.class, () -> runQuery("from missing,logs-1", randomBoolean(), filter).close());
+            assertThat(ie.getDetailedMessage(), containsString("no such index [missing]"));
             // Local index missing + existing remote
             e = expectThrows(VerificationException.class, () -> runQuery("from missing,cluster-a:logs-2", randomBoolean(), filter).close());
             assertThat(e.getDetailedMessage(), containsString("Unknown index [missing]"));
@@ -318,15 +320,19 @@ public void testFilterWithMissingRemoteIndex() {
             );
             assertThat(e.getDetailedMessage(), containsString("Unknown index [cluster-a:missing]"));
             // Local index missing + wildcards
-            // FIXME: planner does not catch this now
-            // e = expectThrows(VerificationException.class, () -> runQuery("from cluster-a:missing,cluster-a:logs*", randomBoolean(),
-            // filter).close());
-            // assertThat(e.getDetailedMessage(), containsString("Unknown index [cluster-a:missing]"));
+            // FIXME: planner does not catch this now, it should be VerificationException but for now it's runtime RemoteException
+            var ie = expectThrows(
+                RemoteException.class,
+                () -> runQuery("from cluster-a:missing,cluster-a:logs*", randomBoolean(), filter).close()
+            );
+            assertThat(ie.getDetailedMessage(), containsString("no such index [missing]"));
             // Local index missing + existing index
-            // FIXME: planner does not catch this now
-            // e = expectThrows(VerificationException.class, () -> runQuery("from cluster-a:missing,cluster-a:logs-2", randomBoolean(),
-            // filter).close());
-            // assertThat(e.getDetailedMessage(), containsString("Unknown index [cluster-a:missing]"));
+            // FIXME: planner does not catch this now, it should be VerificationException but for now it's runtime RemoteException
+            ie = expectThrows(
+                RemoteException.class,
+                () -> runQuery("from cluster-a:missing,cluster-a:logs-2", randomBoolean(), filter).close()
+            );
+            assertThat(ie.getDetailedMessage(), containsString("no such index [missing]"));
             // Local index + missing remote
             e = expectThrows(VerificationException.class, () -> runQuery("from logs-1,cluster-a:missing", randomBoolean(), filter).close());
             assertThat(e.getDetailedMessage(), containsString("Unknown index [cluster-a:missing]"));