Skip to content

Commit

Permalink
Fix sporadic failures in QueryStringQueryTests#testToQueryFuzzyQueryA…
Browse files Browse the repository at this point in the history
…utoFuziness (#43322)

This commit ensures that the test does not use reserved keyword (OR, AND, NOT)
when generating the random query strings.

Closes #43318
  • Loading branch information
jimczi authored Jun 18, 2019
1 parent a3a4bd7 commit c171f9a
Showing 1 changed file with 7 additions and 6 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -787,25 +787,27 @@ public void testEnabledPositionIncrements() throws Exception {

public void testToQueryFuzzyQueryAutoFuziness() throws Exception {
for (int i = 0; i < 3; i++) {
final int len;
final int expectedEdits;
String queryString;
switch (i) {
case 0:
queryString = randomAlphaOfLengthBetween(1, 2);
len = randomIntBetween(1, 2);
expectedEdits = 0;
break;

case 1:
queryString = randomAlphaOfLengthBetween(3, 5);
len = randomIntBetween(3, 5);
expectedEdits = 1;
break;

default:
queryString = randomAlphaOfLengthBetween(6, 20);
len = randomIntBetween(6, 20);
expectedEdits = 2;
break;
}

char[] bytes = new char[len];
Arrays.fill(bytes, 'a');
String queryString = new String(bytes);
for (int j = 0; j < 2; j++) {
Query query = queryStringQuery(queryString + (j == 0 ? "~" : "~auto"))
.defaultField(STRING_FIELD_NAME)
Expand All @@ -817,7 +819,6 @@ public void testToQueryFuzzyQueryAutoFuziness() throws Exception {
}
}
}

public void testFuzzyNumeric() throws Exception {
QueryStringQueryBuilder query = queryStringQuery("12~1.0").defaultField(INT_FIELD_NAME);
QueryShardContext context = createShardContext();
Expand Down

0 comments on commit c171f9a

Please sign in to comment.