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:31:27 UTC

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

Repository: lucene-solr
Updated Branches:
  refs/heads/branch_7x e260f923b -> c821c1131


[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/c821c113
Tree: http://git-wip-us.apache.org/repos/asf/lucene-solr/tree/c821c113
Diff: http://git-wip-us.apache.org/repos/asf/lucene-solr/diff/c821c113

Branch: refs/heads/branch_7x
Commit: c821c113137eb9c546d2812323ede3c0546ce0c2
Parents: e260f92
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:31:12 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/c821c113/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());