diff options
author | Jon Bratseth <bratseth@gmail.com> | 2021-05-10 17:05:20 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2021-05-10 17:05:20 +0200 |
commit | c05117e4ff37ede9550a7ddc1bfc330795268194 (patch) | |
tree | 5342e2e7f4ecdd0611993ddf9f51803042db9de2 /container-search/src/test/java/com/yahoo/search | |
parent | 22b4ff81bd1cb4ce254fcf0150088e628d281688 (diff) |
Revert "Don't consider number of working nodes in coverage"
Diffstat (limited to 'container-search/src/test/java/com/yahoo/search')
-rw-r--r-- | container-search/src/test/java/com/yahoo/search/dispatch/searchcluster/SearchClusterCoverageTest.java | 18 |
1 files changed, 2 insertions, 16 deletions
diff --git a/container-search/src/test/java/com/yahoo/search/dispatch/searchcluster/SearchClusterCoverageTest.java b/container-search/src/test/java/com/yahoo/search/dispatch/searchcluster/SearchClusterCoverageTest.java index 65e7173c4ee..6338107d4b6 100644 --- a/container-search/src/test/java/com/yahoo/search/dispatch/searchcluster/SearchClusterCoverageTest.java +++ b/container-search/src/test/java/com/yahoo/search/dispatch/searchcluster/SearchClusterCoverageTest.java @@ -74,21 +74,7 @@ public class SearchClusterCoverageTest { @Test public void three_groups_one_has_a_node_down() { - var tester = new SearchClusterTester(3, 3); - - tester.setDocsPerNode(100, 0); - tester.setDocsPerNode(100, 1); - tester.setDocsPerNode(100, 2); - tester.setWorking(1, 1, false); - tester.pingIterationCompleted(); - assertTrue(tester.group(0).hasSufficientCoverage()); - assertFalse(tester.group(1).hasSufficientCoverage()); - assertTrue(tester.group(2).hasSufficientCoverage()); - } - - @Test - public void three_groups_one_has_a_node_down_but_remaining_has_enough_docs() { - var tester = new SearchClusterTester(3, 3); + var tester = new SearchClusterTester(3, 3); tester.setDocsPerNode(100, 0); tester.setDocsPerNode(150, 1); @@ -96,7 +82,7 @@ public class SearchClusterCoverageTest { tester.setWorking(1, 1, false); tester.pingIterationCompleted(); assertTrue(tester.group(0).hasSufficientCoverage()); - assertTrue("Sufficient documents on remaining two nodes", tester.group(1).hasSufficientCoverage()); + assertFalse(tester.group(1).hasSufficientCoverage()); assertTrue(tester.group(2).hasSufficientCoverage()); } |