diff options
author | Harald Musum <musum@verizonmedia.com> | 2019-09-03 08:16:35 +0200 |
---|---|---|
committer | Harald Musum <musum@verizonmedia.com> | 2019-09-03 08:16:35 +0200 |
commit | e1356fadd445fbf7d58d2c3f0574dea092964c08 (patch) | |
tree | 2ba233aad5996abc0600cf45223bab878da0cb26 /clustercontroller-core/src/test/java/com/yahoo/vespa/clustercontroller/core/ClusterStatsAggregatorTest.java | |
parent | 7e26976e385c864da3b8bebaf63d4648bee694b2 (diff) |
Cleanup tests, no functional changes
Diffstat (limited to 'clustercontroller-core/src/test/java/com/yahoo/vespa/clustercontroller/core/ClusterStatsAggregatorTest.java')
-rw-r--r-- | clustercontroller-core/src/test/java/com/yahoo/vespa/clustercontroller/core/ClusterStatsAggregatorTest.java | 11 |
1 files changed, 5 insertions, 6 deletions
diff --git a/clustercontroller-core/src/test/java/com/yahoo/vespa/clustercontroller/core/ClusterStatsAggregatorTest.java b/clustercontroller-core/src/test/java/com/yahoo/vespa/clustercontroller/core/ClusterStatsAggregatorTest.java index 7f313b31a6c..9a295e95fff 100644 --- a/clustercontroller-core/src/test/java/com/yahoo/vespa/clustercontroller/core/ClusterStatsAggregatorTest.java +++ b/clustercontroller-core/src/test/java/com/yahoo/vespa/clustercontroller/core/ClusterStatsAggregatorTest.java @@ -4,7 +4,7 @@ package com.yahoo.vespa.clustercontroller.core; import com.google.common.collect.Sets; import org.junit.Test; -import java.util.*; +import java.util.Set; import static org.junit.Assert.assertEquals; import static org.junit.Assert.assertFalse; @@ -12,19 +12,18 @@ import static org.junit.Assert.assertTrue; /** * @author hakonhall - * @since 5.34 */ public class ClusterStatsAggregatorTest { private static class Fixture { private ClusterStatsAggregator aggregator; - public Fixture(Set<Integer> distributorNodes, + Fixture(Set<Integer> distributorNodes, Set<Integer> contentNodes) { aggregator = new ClusterStatsAggregator(distributorNodes, contentNodes); } - public void update(int distributorIndex, ContentClusterStatsBuilder clusterStats) { + void update(int distributorIndex, ContentClusterStatsBuilder clusterStats) { aggregator.updateForDistributor(distributorIndex, clusterStats.build()); } @@ -36,14 +35,14 @@ public class ClusterStatsAggregatorTest { assertEquals(expectedStats.build(), aggregator.getAggregatedStatsForDistributor(distributorIndex)); } - public boolean hasUpdatesFromAllDistributors() { + boolean hasUpdatesFromAllDistributors() { return aggregator.getAggregatedStats().hasUpdatesFromAllDistributors(); } } private static class FourNodesFixture extends Fixture { - public FourNodesFixture() { + FourNodesFixture() { super(distributorNodes(1, 2), contentNodes(3, 4)); update(1, new ContentClusterStatsBuilder() |