You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by rm...@apache.org on 2013/07/30 22:19:15 UTC

svn commit: r1508604 - in /lucene/dev/trunk: ./ lucene/ lucene/core/ lucene/core/src/test/org/apache/lucene/util/packed/TestPackedInts.java

Author: rmuir
Date: Tue Jul 30 20:19:15 2013
New Revision: 1508604

URL: http://svn.apache.org/r1508604
Log:
fix test not to generate nullreaders > 10, since it always asserts bulk reads from that position

Modified:
    lucene/dev/trunk/   (props changed)
    lucene/dev/trunk/lucene/   (props changed)
    lucene/dev/trunk/lucene/core/   (props changed)
    lucene/dev/trunk/lucene/core/src/test/org/apache/lucene/util/packed/TestPackedInts.java

Modified: lucene/dev/trunk/lucene/core/src/test/org/apache/lucene/util/packed/TestPackedInts.java
URL: http://svn.apache.org/viewvc/lucene/dev/trunk/lucene/core/src/test/org/apache/lucene/util/packed/TestPackedInts.java?rev=1508604&r1=1508603&r2=1508604&view=diff
==============================================================================
--- lucene/dev/trunk/lucene/core/src/test/org/apache/lucene/util/packed/TestPackedInts.java (original)
+++ lucene/dev/trunk/lucene/core/src/test/org/apache/lucene/util/packed/TestPackedInts.java Tue Jul 30 20:19:15 2013
@@ -541,7 +541,8 @@ public class TestPackedInts extends Luce
   }
 
   public void testPackedIntsNull() {
-    int size = _TestUtil.nextInt(random(), 0, 256);
+    // must be > 10 for the bulk reads below
+    int size = _TestUtil.nextInt(random(), 11, 256);
     Reader packedInts = new PackedInts.NullReader(size);
     assertEquals(0, packedInts.get(_TestUtil.nextInt(random(), 0, size - 1)));
     long[] arr = new long[size + 10];



Re: svn commit: r1508604 - in /lucene/dev/trunk: ./ lucene/ lucene/core/ lucene/core/src/test/org/apache/lucene/util/packed/TestPackedInts.java

Posted by Adrien Grand <jp...@gmail.com>.
On Tue, Jul 30, 2013 at 10:19 PM,  <rm...@apache.org> wrote:
> fix test not to generate nullreaders > 10, since it always asserts bulk reads from that position

Thanks Robert!

-- 
Adrien

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@lucene.apache.org
For additional commands, e-mail: dev-help@lucene.apache.org