diff options
author | Håkon Hallingstad <hakon@oath.com> | 2017-11-01 14:44:37 +0100 |
---|---|---|
committer | Håkon Hallingstad <hakon@oath.com> | 2017-11-01 14:44:37 +0100 |
commit | 93d628a79c7cc800a8ffd305b4a93ee720cb8ce3 (patch) | |
tree | 2cc48dca60ad25ae22fa175e5f0044fe6d695864 /orchestrator/src/test | |
parent | 451f936c4d19ae8dc08332682d06c6caa3a39000 (diff) |
Avoid changing API before all clients handle it
Diffstat (limited to 'orchestrator/src/test')
-rw-r--r-- | orchestrator/src/test/java/com/yahoo/vespa/orchestrator/resources/HostResourceTest.java | 17 |
1 files changed, 11 insertions, 6 deletions
diff --git a/orchestrator/src/test/java/com/yahoo/vespa/orchestrator/resources/HostResourceTest.java b/orchestrator/src/test/java/com/yahoo/vespa/orchestrator/resources/HostResourceTest.java index 65309440aee..ca0596d8926 100644 --- a/orchestrator/src/test/java/com/yahoo/vespa/orchestrator/resources/HostResourceTest.java +++ b/orchestrator/src/test/java/com/yahoo/vespa/orchestrator/resources/HostResourceTest.java @@ -48,6 +48,7 @@ import static com.yahoo.vespa.orchestrator.TestUtil.makeServiceClusterSet; import static org.fest.assertions.Assertions.assertThat; import static org.fest.assertions.Fail.fail; import static org.junit.Assert.assertEquals; +import static org.junit.Assert.assertNull; import static org.mockito.Matchers.any; import static org.mockito.Matchers.eq; import static org.mockito.Mockito.doThrow; @@ -357,13 +358,17 @@ public class HostResourceTest { Collections.singletonList(serviceInstance)); when(orchestrator.getHost(hostName)).thenReturn(host); GetHostResponse response = hostResource.getHost(hostName.s()); - assertEquals("https://foo.com/bar", response.applicationUrl()); assertEquals("hostname", response.hostname()); assertEquals("ALLOWED_TO_BE_DOWN", response.state()); - assertEquals(1, response.services().size()); - assertEquals("clusterId", response.services().get(0).clusterId); - assertEquals("configId", response.services().get(0).configId); - assertEquals("UP", response.services().get(0).serviceStatus); - assertEquals("serviceType", response.services().get(0).serviceType); + + // See TODO in HostResource: + // assertEquals("https://foo.com/bar", response.applicationUrl()); + // assertEquals(1, response.services().size()); + // assertEquals("clusterId", response.services().get(0).clusterId); + // assertEquals("configId", response.services().get(0).configId); + // assertEquals("UP", response.services().get(0).serviceStatus); + // assertEquals("serviceType", response.services().get(0).serviceType); + assertNull(response.applicationUrl()); + assertNull(response.services()); } } |