Skip to content

Commit f6a2c2c

Browse files
committed
update tests
1 parent 51adf1e commit f6a2c2c

File tree

2 files changed

+7
-5
lines changed

2 files changed

+7
-5
lines changed

metadata-io/src/test/java/com/linkedin/metadata/search/query/request/AutocompleteRequestHandlerTest.java

+3-2
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,7 @@
2929
import org.opensearch.index.query.MultiMatchQueryBuilder;
3030
import org.opensearch.index.query.QueryBuilder;
3131
import org.opensearch.index.query.QueryBuilders;
32+
import org.opensearch.index.query.TermQueryBuilder;
3233
import org.opensearch.index.query.functionscore.FunctionScoreQueryBuilder;
3334
import org.opensearch.index.query.functionscore.ScoreFunctionBuilders;
3435
import org.opensearch.search.builder.SearchSourceBuilder;
@@ -140,7 +141,7 @@ public void testDefaultAutocompleteRequest() {
140141
assertEquals("keyPart1.delimited", prefixQuery.fieldName());
141142

142143
assertEquals(wrapper.mustNot().size(), 1);
143-
MatchQueryBuilder removedFilter = (MatchQueryBuilder) wrapper.mustNot().get(0);
144+
TermQueryBuilder removedFilter = (TermQueryBuilder) wrapper.mustNot().get(0);
144145
assertEquals(removedFilter.fieldName(), "removed");
145146
assertEquals(removedFilter.value(), true);
146147
HighlightBuilder highlightBuilder = sourceBuilder.highlighter();
@@ -185,7 +186,7 @@ public void testAutocompleteRequestWithField() {
185186
(MatchPhrasePrefixQueryBuilder) query.should().get(1);
186187
assertEquals("field.delimited", prefixQuery.fieldName());
187188

188-
MatchQueryBuilder removedFilter = (MatchQueryBuilder) wrapper.mustNot().get(0);
189+
TermQueryBuilder removedFilter = (TermQueryBuilder) wrapper.mustNot().get(0);
189190
assertEquals(removedFilter.fieldName(), "removed");
190191
assertEquals(removedFilter.value(), true);
191192
HighlightBuilder highlightBuilder = sourceBuilder.highlighter();

metadata-io/src/test/java/com/linkedin/metadata/search/query/request/SearchRequestHandlerTest.java

+4-3
Original file line numberDiff line numberDiff line change
@@ -56,6 +56,7 @@
5656
import org.opensearch.index.query.BoolQueryBuilder;
5757
import org.opensearch.index.query.ExistsQueryBuilder;
5858
import org.opensearch.index.query.MatchQueryBuilder;
59+
import org.opensearch.index.query.TermQueryBuilder;
5960
import org.opensearch.index.query.TermsQueryBuilder;
6061
import org.opensearch.search.aggregations.AggregationBuilder;
6162
import org.opensearch.search.aggregations.AggregationBuilders;
@@ -433,7 +434,7 @@ private BoolQueryBuilder constructFilterQuery(
433434
}
434435

435436
private void testFilterQuery(BoolQueryBuilder testQuery) {
436-
Optional<MatchQueryBuilder> mustNotHaveRemovedCondition =
437+
Optional<TermQueryBuilder> mustNotHaveRemovedCondition =
437438
testQuery.filter().stream()
438439
.filter(or -> or instanceof BoolQueryBuilder)
439440
.map(or -> (BoolQueryBuilder) or)
@@ -442,8 +443,8 @@ private void testFilterQuery(BoolQueryBuilder testQuery) {
442443
System.out.println("processing: " + or.mustNot());
443444
return or.mustNot().stream();
444445
})
445-
.filter(and -> and instanceof MatchQueryBuilder)
446-
.map(and -> (MatchQueryBuilder) and)
446+
.filter(and -> and instanceof TermQueryBuilder)
447+
.map(and -> (TermQueryBuilder) and)
447448
.filter(match -> match.fieldName().equals("removed"))
448449
.findAny();
449450

0 commit comments

Comments
 (0)