summaryrefslogtreecommitdiffstats
path: root/orchestrator/src/test
diff options
context:
space:
mode:
authorHåkon Hallingstad <hakon@oath.com>2017-11-01 14:44:37 +0100
committerHåkon Hallingstad <hakon@oath.com>2017-11-01 14:44:37 +0100
commit93d628a79c7cc800a8ffd305b4a93ee720cb8ce3 (patch)
tree2cc48dca60ad25ae22fa175e5f0044fe6d695864 /orchestrator/src/test
parent451f936c4d19ae8dc08332682d06c6caa3a39000 (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.java17
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());
}
}