Skip to content

Commit 3e2c856

Browse files
idegtiarenkoywangd
authored andcommitted
Fix testOptimizerExpressionRuleShouldNotVisitExcludedNodes (elastic#127755)
1 parent db561c1 commit 3e2c856

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
@@ -435,9 +435,6 @@ tests:
435435
- class: org.elasticsearch.backwards.MixedClusterClientYamlTestSuiteIT
436436
method: test {p0=search/350_point_in_time/point-in-time with index filter}
437437
issue: https://github.com/elastic/elasticsearch/issues/127741
438-
- class: org.elasticsearch.xpack.esql.optimizer.OptimizerRulesTests
439-
method: testOptimizerExpressionRuleShouldNotVisitExcludedNodes
440-
issue: https://github.com/elastic/elasticsearch/issues/127754
441438
- class: org.elasticsearch.xpack.remotecluster.RemoteClusterSecurityFcActionAuthorizationIT
442439
method: testIndicesPrivilegesAreEnforcedForCcrRestoreSessionActions
443440
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)