summaryrefslogtreecommitdiff
path: root/core/src/test/java/org/elasticsearch/index/query/BoolQueryBuilderTests.java
diff options
context:
space:
mode:
authorMartijn van Groningen <martijn.v.groningen@gmail.com>2017-06-29 23:25:05 +0200
committerMartijn van Groningen <martijn.v.groningen@gmail.com>2017-06-30 13:48:18 +0200
commitc8da7f84a20b836671048749245dba3f2c95dbeb (patch)
treef519b7573beb27feda954d6f299a9b2a53e8a760 /core/src/test/java/org/elasticsearch/index/query/BoolQueryBuilderTests.java
parent1fee1045b960ed1a582ff84ddd844e8b11bdfaa5 (diff)
WrapperQueryBuilder should also rewrite the parsed query.
Failing to do so can cause other errors later on during query execution. For example if `WrapperQueryBuilder` wraps a `GeoShapeQueryBuilder` that fetches the shape from an index then it will skip the shape fetching and fail later with the error that no shapes have been fetched.
Diffstat (limited to 'core/src/test/java/org/elasticsearch/index/query/BoolQueryBuilderTests.java')
-rw-r--r--core/src/test/java/org/elasticsearch/index/query/BoolQueryBuilderTests.java2
1 files changed, 1 insertions, 1 deletions
diff --git a/core/src/test/java/org/elasticsearch/index/query/BoolQueryBuilderTests.java b/core/src/test/java/org/elasticsearch/index/query/BoolQueryBuilderTests.java
index 68d2d369bb..17cb4d0c4f 100644
--- a/core/src/test/java/org/elasticsearch/index/query/BoolQueryBuilderTests.java
+++ b/core/src/test/java/org/elasticsearch/index/query/BoolQueryBuilderTests.java
@@ -409,7 +409,7 @@ public class BoolQueryBuilderTests extends AbstractQueryTestCase<BoolQueryBuilde
boolQueryBuilder.must(new WrapperQueryBuilder(new WrapperQueryBuilder(new MatchAllQueryBuilder().toString()).toString()));
QueryBuilder rewritten = boolQueryBuilder.rewrite(createShardContext());
BoolQueryBuilder expected = new BoolQueryBuilder();
- expected.must(new WrapperQueryBuilder(new MatchAllQueryBuilder().toString()));
+ expected.must(new MatchAllQueryBuilder());
assertEquals(expected, rewritten);
expected = new BoolQueryBuilder();