ESQL: Fix LogicalPlanOptimizerTests testPlanSanityCheckWithBinaryPlans (

elastic#118672)
carlosdelest · Dec 13, 2024 · 54e839b · 54e839b
1 parent 950db57
commit 54e839b
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 5 deletions.
diff --git a/muted-tests.yml b/muted-tests.yml
@@ -317,9 +317,6 @@ tests:
 - class: org.elasticsearch.reservedstate.service.FileSettingsServiceTests
   method: testInvalidJSON
   issue: https://github.com/elastic/elasticsearch/issues/116521
-- class: org.elasticsearch.xpack.esql.optimizer.LogicalPlanOptimizerTests
-  method: testPlanSanityCheckWithBinaryPlans
-  issue: https://github.com/elastic/elasticsearch/issues/118656
 
 # Examples:
 #

diff --git a/.../esql/src/test/java/org/elasticsearch/xpack/esql/optimizer/LogicalPlanOptimizerTests.java b/.../esql/src/test/java/org/elasticsearch/xpack/esql/optimizer/LogicalPlanOptimizerTests.java
@@ -4911,8 +4911,7 @@ public void testPlanSanityCheckWithBinaryPlans() throws Exception {
             """);
 
         var project = as(plan, Project.class);
-        var limit = as(project.child(), Limit.class);
-        var join = as(limit.child(), Join.class);
+        var join = as(project.child(), Join.class);
 
         var joinWithInvalidLeftPlan = join.replaceChildren(join.right(), join.right());
         IllegalStateException e = expectThrows(IllegalStateException.class, () -> logicalOptimizer.optimize(joinWithInvalidLeftPlan));