diff options
author | Harald Musum <musum@verizonmedia.com> | 2023-02-24 10:39:50 +0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2023-02-24 10:39:50 +0100 |
commit | 36c0d79e750e1c7b32dcbc6e294a462a8d691bae (patch) | |
tree | f33799ee9f9e54207846185b7344fdee79690ae9 /configserver/src/test/java/com/yahoo/vespa/config/server/host/HostRegistryTest.java | |
parent | cdde57098b5806f0bf4c3d50d53e1b19465071d8 (diff) | |
parent | 5179fc9ac037d3bcc3cdc1a044caef8d5ee5a84a (diff) |
Merge pull request #26170 from vespa-engine/hmusum/configserver-cleanup-1b
Cleanup HostRegistry, no functional changes
Diffstat (limited to 'configserver/src/test/java/com/yahoo/vespa/config/server/host/HostRegistryTest.java')
-rw-r--r-- | configserver/src/test/java/com/yahoo/vespa/config/server/host/HostRegistryTest.java | 18 |
1 files changed, 9 insertions, 9 deletions
diff --git a/configserver/src/test/java/com/yahoo/vespa/config/server/host/HostRegistryTest.java b/configserver/src/test/java/com/yahoo/vespa/config/server/host/HostRegistryTest.java index df00d28134f..646017a498e 100644 --- a/configserver/src/test/java/com/yahoo/vespa/config/server/host/HostRegistryTest.java +++ b/configserver/src/test/java/com/yahoo/vespa/config/server/host/HostRegistryTest.java @@ -24,23 +24,23 @@ public class HostRegistryTest { @Test public void old_hosts_are_removed() { HostRegistry reg = new HostRegistry(); - assertNull(reg.getKeyForHost("foo.com")); + assertNull(reg.getApplicationId("foo.com")); reg.update(foo, List.of("foo.com", "bar.com", "baz.com")); assertGetKey(reg, "foo.com", foo); assertGetKey(reg, "bar.com", foo); assertGetKey(reg, "baz.com", foo); assertEquals(3, reg.getAllHosts().size()); reg.update(foo, List.of("bar.com", "baz.com")); - assertNull(reg.getKeyForHost("foo.com")); + assertNull(reg.getApplicationId("foo.com")); assertGetKey(reg, "bar.com", foo); assertGetKey(reg, "baz.com", foo); assertEquals(2, reg.getAllHosts().size()); assertTrue(reg.getAllHosts().containsAll(List.of("bar.com", "baz.com"))); - reg.removeHostsForKey(foo); + reg.removeHosts(foo); assertTrue(reg.getAllHosts().isEmpty()); - assertNull(reg.getKeyForHost("foo.com")); - assertNull(reg.getKeyForHost("bar.com")); + assertNull(reg.getApplicationId("foo.com")); + assertNull(reg.getApplicationId("bar.com")); } @Test @@ -74,9 +74,9 @@ public class HostRegistryTest { HostRegistry reg = new HostRegistry(); List<String> hosts = new ArrayList<>(List.of("foo.com", "bar.com", "baz.com")); reg.update(foo, hosts); - assertEquals(3, reg.getHostsForKey(foo).size()); + assertEquals(3, reg.getHosts(foo).size()); hosts.remove(2); - assertEquals(3, reg.getHostsForKey(foo).size()); + assertEquals(3, reg.getHosts(foo).size()); } @Test @@ -90,8 +90,8 @@ public class HostRegistryTest { } private void assertGetKey(HostRegistry reg, String host, ApplicationId expectedKey) { - assertNotNull(reg.getKeyForHost(host)); - assertEquals(expectedKey, reg.getKeyForHost(host)); + assertNotNull(reg.getApplicationId(host)); + assertEquals(expectedKey, reg.getApplicationId(host)); } } |