Merge pull request #26539 from vespa-engine/balder/one-doc-is-also-ok

It is enough to have a single document to verify that the corpus is OK.

2 files changed, 14 insertions, 1 deletions

diff --git a/container-search/src/main/java/com/yahoo/search/result/Coverage.java b/container-search/src/main/java/com/yahoo/search/result/Coverage.java
index 390c6ec4435..cd0e5acc96a 100644
--- a/container-search/src/main/java/com/yahoo/search/result/Coverage.java
+++ b/container-search/src/main/java/com/yahoo/search/result/Coverage.java
@@ -10,7 +10,8 @@ package com.yahoo.search.result;
 public class Coverage extends com.yahoo.container.handler.Coverage {
 
     public Coverage(long docs, long active) {
-        this(docs, active, docs > 1 ? 1 : 0, docs > 1 ? 1: 0);
+        // TODO Wonder about this special handling.....
+        this(docs, active, (docs > 0) ? 1 : 0, (docs > 0) ? 1: 0);
     }
 
     public Coverage(long docs, long active, int nodes) {
diff --git a/container-search/src/test/java/com/yahoo/search/result/CoverageTestCase.java b/container-search/src/test/java/com/yahoo/search/result/CoverageTestCase.java
index 4d521e3e8b4..1bbca461216 100644
--- a/container-search/src/test/java/com/yahoo/search/result/CoverageTestCase.java
+++ b/container-search/src/test/java/com/yahoo/search/result/CoverageTestCase.java
@@ -90,4 +90,16 @@ public class CoverageTestCase {
         verifyCoverageConversion(new Coverage(6, 10).setDegradedReason(7).setTargetActive(12));
     }
 
+    @Test
+    void testCoverageWorksForCorpusAboveZero() {
+        Coverage zero = new Coverage(0, 0);
+        assertEquals(0, zero.getNodes()); // TODO Wonder about this special handling.....
+        assertEquals(0, zero.getFullResultSets());
+        for (int i =1; i < 10; i++) {
+            Coverage coverage = new Coverage(i, i);
+            assertEquals(1, coverage.getNodes());
+            verifyCoverageConversion(coverage);
+        }
+    }
+
 }