Skip to content

Commit 771fdc7

Browse files
author
Christoph Büscher
committed
[Test] Prevent errors on occasionally invalid range query
Tests failed on rare occasions where from/to on integer range fields were just one step apart and both includeLower and includeUpper was set to false. This could lead to invalid ranges being rejected.
1 parent 05a6b91 commit 771fdc7

File tree

1 file changed

+4
-1
lines changed

1 file changed

+4
-1
lines changed

core/src/test/java/org/elasticsearch/index/query/RangeQueryBuilderTests.java

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,10 @@ protected RangeQueryBuilder doCreateTestQueryBuilder() {
6666
case 0:
6767
// use mapped integer field for numeric range queries
6868
query = new RangeQueryBuilder(randomBoolean() ? INT_FIELD_NAME : INT_RANGE_FIELD_NAME);
69-
query.from(randomIntBetween(1, 100));
69+
query.from(randomIntBetween(0, 99));
70+
// we need a gap of 1 between from/to for range fields, otherwise if
71+
// from=100, to=101 and both includeLower=false and includeUpper=false, ranges
72+
// can be min > max (101 > 100) which throws an error
7073
query.to(randomIntBetween(101, 200));
7174
break;
7275
case 1:

0 commit comments

Comments
 (0)