Skip to content

Commit fe8c55f

Browse files
idegtiarenkoafoucret
authored andcommitted
Fix testOptimizerExpressionRuleShouldNotVisitExcludedNodes (elastic#127755)
1 parent 45f994f commit fe8c55f

File tree

2 files changed

+2
-5
lines changed

2 files changed

+2
-5
lines changed

muted-tests.yml

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -432,9 +432,6 @@ tests:
432432
- class: org.elasticsearch.backwards.MixedClusterClientYamlTestSuiteIT
433433
method: test {p0=search/350_point_in_time/point-in-time with index filter}
434434
issue: https://github.com/elastic/elasticsearch/issues/127741
435-
- class: org.elasticsearch.xpack.esql.optimizer.OptimizerRulesTests
436-
method: testOptimizerExpressionRuleShouldNotVisitExcludedNodes
437-
issue: https://github.com/elastic/elasticsearch/issues/127754
438435
- class: org.elasticsearch.xpack.remotecluster.RemoteClusterSecurityFcActionAuthorizationIT
439436
method: testIndicesPrivilegesAreEnforcedForCcrRestoreSessionActions
440437
issue: https://github.com/elastic/elasticsearch/issues/127782

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

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@
3232
import static org.elasticsearch.xpack.esql.core.type.DataType.BOOLEAN;
3333
import static org.elasticsearch.xpack.esql.core.util.TestUtils.getFieldAttribute;
3434
import static org.elasticsearch.xpack.esql.core.util.TestUtils.of;
35-
import static org.hamcrest.Matchers.contains;
35+
import static org.hamcrest.Matchers.containsInAnyOrder;
3636

3737
public class OptimizerRulesTests extends ESTestCase {
3838

@@ -138,6 +138,6 @@ protected Expression rule(Expression e, LogicalOptimizerContext ctx) {
138138
var alias = new Alias(new Source(1, 18, "x=f1+1"), "x", add);
139139

140140
// contains expressions only from EVAL
141-
assertThat(rule.appliedTo, contains(alias, add, attribute, literal));
141+
assertThat(rule.appliedTo, containsInAnyOrder(alias, add, attribute, literal));
142142
}
143143
}

0 commit comments

Comments
 (0)