You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by si...@apache.org on 2018/05/09 20:30:30 UTC

lucene-solr:master: [TEST]: Account for docs that randomly never got added

Repository: lucene-solr
Updated Branches:
  refs/heads/master 1067cabc1 -> 99d1ccefc


[TEST]: Account for docs that randomly never got added


Project: http://git-wip-us.apache.org/repos/asf/lucene-solr/repo
Commit: http://git-wip-us.apache.org/repos/asf/lucene-solr/commit/99d1ccef
Tree: http://git-wip-us.apache.org/repos/asf/lucene-solr/tree/99d1ccef
Diff: http://git-wip-us.apache.org/repos/asf/lucene-solr/diff/99d1ccef

Branch: refs/heads/master
Commit: 99d1ccefcd7dddf35ca6e891fe8062458924c047
Parents: 1067cab
Author: Simon Willnauer <si...@apache.org>
Authored: Wed May 9 22:29:58 2018 +0200
Committer: Simon Willnauer <si...@apache.org>
Committed: Wed May 9 22:30:22 2018 +0200

----------------------------------------------------------------------
 .../apache/lucene/index/TestMixedDocValuesUpdates.java    | 10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/lucene-solr/blob/99d1ccef/lucene/core/src/test/org/apache/lucene/index/TestMixedDocValuesUpdates.java
----------------------------------------------------------------------
diff --git a/lucene/core/src/test/org/apache/lucene/index/TestMixedDocValuesUpdates.java b/lucene/core/src/test/org/apache/lucene/index/TestMixedDocValuesUpdates.java
index 0927dcd..4315af2 100644
--- a/lucene/core/src/test/org/apache/lucene/index/TestMixedDocValuesUpdates.java
+++ b/lucene/core/src/test/org/apache/lucene/index/TestMixedDocValuesUpdates.java
@@ -595,7 +595,7 @@ public class TestMixedDocValuesUpdates extends LuceneTestCase {
     IndexWriter writer = new IndexWriter(dir, conf);
     int numDocs = 10 + random().nextInt(50);
     int currentSeqId = 0;
-    int[] seqId = new int[5];
+    int[] seqId = new int[] {-1, -1, -1, -1, -1};
     for (int i = 0; i < numDocs; i++) {
       Document doc = new Document();
       int id = random().nextInt(5);
@@ -615,11 +615,17 @@ public class TestMixedDocValuesUpdates extends LuceneTestCase {
     if (random().nextBoolean()) {
       writer.commit();
     }
+    int numHits = 0; // check if every doc has been selected at least once
+    for (int i : seqId) {
+      if (i > -1) {
+        numHits++;
+      }
+    }
     try(DirectoryReader reader = writer.getReader()) {
       IndexSearcher searcher = new IndexSearcher(reader);
 
       TopDocs is_live = searcher.search(new DocValuesFieldExistsQuery("is_live"), 5);
-      assertEquals(5, is_live.totalHits);
+      assertEquals(numHits, is_live.totalHits);
       for (ScoreDoc doc : is_live.scoreDocs) {
         int id = Integer.parseInt(reader.document(doc.doc).get("id"));
         int i = ReaderUtil.subIndex(doc.doc, reader.leaves());