summaryrefslogtreecommitdiffstats
path: root/node-admin
diff options
context:
space:
mode:
authorJon Bratseth <bratseth@verizonmedia.com>2019-05-06 22:16:11 +0200
committerJon Bratseth <bratseth@verizonmedia.com>2019-05-06 22:16:11 +0200
commitb436b5431b3a4e8d0af68871478b110129693b15 (patch)
treec93abac47b46666f338f0b733fc4def03a6221e7 /node-admin
parentcff03667109b5539a0f71487fa695601760d498c (diff)
Expect the correct capacity values
Diffstat (limited to 'node-admin')
-rw-r--r--node-admin/src/test/java/com/yahoo/vespa/hosted/node/admin/configserver/noderepository/RealNodeRepositoryTest.java19
1 files changed, 10 insertions, 9 deletions
diff --git a/node-admin/src/test/java/com/yahoo/vespa/hosted/node/admin/configserver/noderepository/RealNodeRepositoryTest.java b/node-admin/src/test/java/com/yahoo/vespa/hosted/node/admin/configserver/noderepository/RealNodeRepositoryTest.java
index 048008d72ac..fb443ed14c4 100644
--- a/node-admin/src/test/java/com/yahoo/vespa/hosted/node/admin/configserver/noderepository/RealNodeRepositoryTest.java
+++ b/node-admin/src/test/java/com/yahoo/vespa/hosted/node/admin/configserver/noderepository/RealNodeRepositoryTest.java
@@ -37,10 +37,11 @@ import static org.junit.Assert.fail;
* @author dybdahl
*/
public class RealNodeRepositoryTest {
+
+ private static final double delta = 0.00000001;
private JDisc container;
private NodeRepository nodeRepositoryApi;
-
private int findRandomOpenPort() throws IOException {
try (ServerSocket socket = new ServerSocket(0)) {
socket.setReuseAddress(true);
@@ -63,7 +64,7 @@ public class RealNodeRepositoryTest {
// a few times before giving up
for (int i = 0; i < 3; i++) {
try {
- final int port = findRandomOpenPort();
+ int port = findRandomOpenPort();
container = JDisc.fromServicesXml(ContainerConfig.servicesXmlV2(port), Networking.enable);
ConfigServerApi configServerApi = ConfigServerApiImpl.createForTesting(
Collections.singletonList(URI.create("http://127.0.0.1:" + port)));
@@ -101,25 +102,25 @@ public class RealNodeRepositoryTest {
public void testGetContainersToRunApi() {
String dockerHostHostname = "dockerhost1.yahoo.com";
- final List<NodeSpec> containersToRun = nodeRepositoryApi.getNodes(dockerHostHostname);
+ List<NodeSpec> containersToRun = nodeRepositoryApi.getNodes(dockerHostHostname);
assertThat(containersToRun.size(), is(1));
- final NodeSpec node = containersToRun.get(0);
+ NodeSpec node = containersToRun.get(0);
assertThat(node.getHostname(), is("host4.yahoo.com"));
assertThat(node.getWantedDockerImage().get(), is(DockerImage.fromString("docker-registry.domain.tld:8080/dist/vespa:6.42.0")));
assertThat(node.getState(), is(NodeState.active));
assertThat(node.getWantedRestartGeneration().get(), is(0L));
assertThat(node.getCurrentRestartGeneration().get(), is(0L));
- assertThat(node.getMinCpuCores(), is(0.2));
- assertThat(node.getMinMainMemoryAvailableGb(), is(0.5));
- assertThat(node.getMinDiskAvailableGb(), is(100.0));
+ assertEquals(1, node.getMinCpuCores(), delta);
+ assertEquals(1, node.getMinMainMemoryAvailableGb(), delta);
+ assertEquals(100, node.getMinDiskAvailableGb(), delta);
}
@Test
public void testGetContainer() {
String hostname = "host4.yahoo.com";
Optional<NodeSpec> node = nodeRepositoryApi.getOptionalNode(hostname);
- assertThat(node.isPresent(), is(true));
- assertThat(node.get().getHostname(), is(hostname));
+ assertTrue(node.isPresent());
+ assertEquals(hostname, node.get().getHostname());
}
@Test