Skip to content

Commit cc6316f

Browse files
authored
ESQL: Fix test (#116096)
I missed this in #116086. The test needs to have a good `toString` for it to pass. Closes #116094
1 parent 82d21f3 commit cc6316f

File tree

2 files changed

+12
-4
lines changed

2 files changed

+12
-4
lines changed

muted-tests.yml

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -225,9 +225,6 @@ tests:
225225
issue: https://github.com/elastic/elasticsearch/issues/116050
226226
- class: org.elasticsearch.xpack.application.connector.ConnectorIndexServiceTests
227227
issue: https://github.com/elastic/elasticsearch/issues/116087
228-
- class: org.elasticsearch.compute.operator.FilterOperatorTests
229-
method: testSimpleDescription
230-
issue: https://github.com/elastic/elasticsearch/issues/116094
231228
- class: org.elasticsearch.xpack.searchbusinessrules.PinnedQueryBuilderIT
232229
method: testPinnedPromotions
233230
issue: https://github.com/elastic/elasticsearch/issues/116097

x-pack/plugin/esql/compute/src/test/java/org/elasticsearch/compute/operator/FilterOperatorTests.java

Lines changed: 12 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,18 @@ public void close() {}
5353

5454
@Override
5555
protected Operator.OperatorFactory simple() {
56-
return new FilterOperator.FilterOperatorFactory(dvrCtx -> new SameLastDigit(dvrCtx, 0, 1));
56+
return new FilterOperator.FilterOperatorFactory(new EvalOperator.ExpressionEvaluator.Factory() {
57+
58+
@Override
59+
public EvalOperator.ExpressionEvaluator get(DriverContext context) {
60+
return new SameLastDigit(context, 0, 1);
61+
}
62+
63+
@Override
64+
public String toString() {
65+
return "SameLastDigit[lhs=0, rhs=1]";
66+
}
67+
});
5768
}
5869

5970
@Override

0 commit comments

Comments
 (0)