Also handle the case of empty selection string which evidently matches everything.

1 files changed, 8 insertions, 1 deletions

diff --git a/container-search/src/test/java/com/yahoo/vespa/streamingvisitors/VdsVisitorTestCase.java b/container-search/src/test/java/com/yahoo/vespa/streamingvisitors/VdsVisitorTestCase.java
index c84088a19bb..11862d84414 100644
--- a/container-search/src/test/java/com/yahoo/vespa/streamingvisitors/VdsVisitorTestCase.java
+++ b/container-search/src/test/java/com/yahoo/vespa/streamingvisitors/VdsVisitorTestCase.java
@@ -232,7 +232,11 @@ public class VdsVisitorTestCase {
         } else if (qa.groupName != null) {
             assertEquals(docType + " and id.group==\""+qa.groupName+"\"", params.getDocumentSelection());
         } else if (qa.selection != null) {
-            assertEquals(docType + " and " +qa.selection, params.getDocumentSelection());
+            if (qa.selection.isEmpty()) {
+                assertEquals(docType, params.getDocumentSelection());
+            } else {
+                assertEquals(docType + " and " + qa.selection, params.getDocumentSelection());
+            }
         } else {
             assertEquals("", params.getDocumentSelection());
         }
@@ -383,6 +387,9 @@ public class VdsVisitorTestCase {
         qa.maxBucketsPerVisitor = 0;
         qa.loadTypeName = null; // default loadTypeName, non-default priority
         verifyVisitorOk(factory, qa, route, searchCluster);
+        qa.selection = "";
+
+        verifyVisitorOk(factory, qa, route, searchCluster);
 
         // Userdoc and lots of non-default parameters
         qa.setNonDefaults();