diff options
author | Ryan Ernst <ryan@iernst.net> | 2016-07-10 09:11:32 -0700 |
---|---|---|
committer | Ryan Ernst <ryan@iernst.net> | 2016-07-10 09:11:32 -0700 |
commit | 25ed93dd289b80a757bcc09e7e8daf75abc8c7fb (patch) | |
tree | 18074dae1c8884258387fdbe56cf3daac1345b5f /core/src/test/java/org/elasticsearch/common/bytes | |
parent | ab8b577aea10b9cf5cff9d0513b4567776aea99c (diff) |
Fix test edge case for random bytes reference iter.
Getting an offset to the last byte means we can only stream one byte and
then we are done, we can't get another offset after it.
Diffstat (limited to 'core/src/test/java/org/elasticsearch/common/bytes')
-rw-r--r-- | core/src/test/java/org/elasticsearch/common/bytes/AbstractBytesReferenceTestCase.java | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/core/src/test/java/org/elasticsearch/common/bytes/AbstractBytesReferenceTestCase.java b/core/src/test/java/org/elasticsearch/common/bytes/AbstractBytesReferenceTestCase.java index c521314f92..103e263ffa 100644 --- a/core/src/test/java/org/elasticsearch/common/bytes/AbstractBytesReferenceTestCase.java +++ b/core/src/test/java/org/elasticsearch/common/bytes/AbstractBytesReferenceTestCase.java @@ -271,6 +271,9 @@ public abstract class AbstractBytesReferenceTestCase extends ESTestCase { final int offset = randomIntBetween(0, length-1); assertEquals(offset, input.skip(offset)); assertEquals(pbr.get(offset), input.readByte()); + if (offset == length - 1) { + continue; // no more bytes to retrieve! + } final int nextOffset = randomIntBetween(offset, length-2); assertEquals(nextOffset - offset, input.skip(nextOffset - offset)); assertEquals(pbr.get(nextOffset+1), input.readByte()); // +1 for the one byte we read above |