diff options
author | jonmv <venstad@gmail.com> | 2023-09-01 13:39:20 +0200 |
---|---|---|
committer | jonmv <venstad@gmail.com> | 2023-09-01 13:39:20 +0200 |
commit | 14ab02260c9d0aa0f74c259ce1efd05e875db673 (patch) | |
tree | 912017857ebcc952a6f93ab7b649bbf38386ca0c /container-search/src/test/java/com/yahoo | |
parent | 424b9147adcc164ca74ab41c6e6968147e76e8ec (diff) |
Ensure node search path index follows config order
Diffstat (limited to 'container-search/src/test/java/com/yahoo')
-rw-r--r-- | container-search/src/test/java/com/yahoo/search/dispatch/searchcluster/SearchClusterTest.java | 10 |
1 files changed, 8 insertions, 2 deletions
diff --git a/container-search/src/test/java/com/yahoo/search/dispatch/searchcluster/SearchClusterTest.java b/container-search/src/test/java/com/yahoo/search/dispatch/searchcluster/SearchClusterTest.java index 65b0261b1f8..f6a1ca5cae3 100644 --- a/container-search/src/test/java/com/yahoo/search/dispatch/searchcluster/SearchClusterTest.java +++ b/container-search/src/test/java/com/yahoo/search/dispatch/searchcluster/SearchClusterTest.java @@ -403,14 +403,20 @@ public class SearchClusterTest { SearchGroups newGroups = state.searchCluster.groupList(); assertEquals(Set.copyOf(updatedNodes), newGroups.nodes()); - Map<Node, Node> oldNodesByIdentity = newGroups.nodes().stream().collect(toMap(identity(), identity())); + Map<Node, Node> oldNodesByIdentity = oldGroups.nodes().stream().collect(toMap(identity(), identity())); Map<Node, Node> newNodesByIdentity = newGroups.nodes().stream().collect(toMap(identity(), identity())); assertSame(updatedNodes.get(0), newNodesByIdentity.get(updatedNodes.get(0))); assertSame(updatedNodes.get(1), newNodesByIdentity.get(updatedNodes.get(1))); assertSame(updatedNodes.get(2), newNodesByIdentity.get(updatedNodes.get(2))); - assertSame(oldNodesByIdentity.get(referenceNodes.get(3)), newNodesByIdentity.get(updatedNodes.get(3))); + assertSame(oldNodesByIdentity.get(updatedNodes.get(3)), newNodesByIdentity.get(updatedNodes.get(3))); assertSame(updatedNodes.get(4), newNodesByIdentity.get(updatedNodes.get(4))); assertSame(updatedNodes.get(5), newNodesByIdentity.get(updatedNodes.get(5))); + + // Also verify search-path index within group follows node order, as given by config. + int[] pathIndexWithinGroup = new int[3]; + for (Node node : updatedNodes) + assertEquals(pathIndexWithinGroup[node.group()]++, newNodesByIdentity.get(node).pathIndex(), + "search path index within group should follow updated node order for: " + node); } } |