aboutsummaryrefslogtreecommitdiffstats
path: root/node-admin/src/test/java/com
diff options
context:
space:
mode:
authorValerij Fredriksen <valerijf@oath.com>2018-10-26 15:46:59 +0200
committerValerij Fredriksen <valerijf@oath.com>2018-10-26 15:46:59 +0200
commit2d677c94c9a727c565e879262dc99706d78fa0b3 (patch)
tree1db9ba9e659756d339fc97f4d3f9c2d5290a80f8 /node-admin/src/test/java/com
parentbfa236c47b1d663b5b91b3bb35c8cf199ae85823 (diff)
Rename to NASU
Diffstat (limited to 'node-admin/src/test/java/com')
-rw-r--r--node-admin/src/test/java/com/yahoo/vespa/hosted/node/admin/integrationTests/DockerTester.java14
-rw-r--r--node-admin/src/test/java/com/yahoo/vespa/hosted/node/admin/integrationTests/RebootTest.java4
-rw-r--r--node-admin/src/test/java/com/yahoo/vespa/hosted/node/admin/nodeadmin/NodeAdminStateUpdaterTest.java (renamed from node-admin/src/test/java/com/yahoo/vespa/hosted/node/admin/nodeadmin/NodeAdminStateUpdaterImplTest.java)14
3 files changed, 16 insertions, 16 deletions
diff --git a/node-admin/src/test/java/com/yahoo/vespa/hosted/node/admin/integrationTests/DockerTester.java b/node-admin/src/test/java/com/yahoo/vespa/hosted/node/admin/integrationTests/DockerTester.java
index c2932b939df..57835a3a759 100644
--- a/node-admin/src/test/java/com/yahoo/vespa/hosted/node/admin/integrationTests/DockerTester.java
+++ b/node-admin/src/test/java/com/yahoo/vespa/hosted/node/admin/integrationTests/DockerTester.java
@@ -14,7 +14,7 @@ import com.yahoo.vespa.hosted.node.admin.docker.DockerOperations;
import com.yahoo.vespa.hosted.node.admin.docker.DockerOperationsImpl;
import com.yahoo.vespa.hosted.node.admin.maintenance.StorageMaintainer;
import com.yahoo.vespa.hosted.node.admin.nodeadmin.NodeAdminImpl;
-import com.yahoo.vespa.hosted.node.admin.nodeadmin.NodeAdminStateUpdaterImpl;
+import com.yahoo.vespa.hosted.node.admin.nodeadmin.NodeAdminStateUpdater;
import com.yahoo.vespa.hosted.node.admin.nodeagent.NodeAgent;
import com.yahoo.vespa.hosted.node.admin.nodeagent.NodeAgentContextImpl;
import com.yahoo.vespa.hosted.node.admin.nodeagent.NodeAgentImpl;
@@ -60,11 +60,11 @@ public class DockerTester implements AutoCloseable {
final StorageMaintainer storageMaintainer = mock(StorageMaintainer.class);
final InOrder inOrder = Mockito.inOrder(docker, nodeRepository, orchestrator, storageMaintainer);
- final NodeAdminStateUpdaterImpl nodeAdminStateUpdaterImpl;
+ final NodeAdminStateUpdater nodeAdminStateUpdater;
final NodeAdminImpl nodeAdmin;
private boolean terminated = false;
- private volatile NodeAdminStateUpdaterImpl.State wantedState = NodeAdminStateUpdaterImpl.State.RESUMED;
+ private volatile NodeAdminStateUpdater.State wantedState = NodeAdminStateUpdater.State.RESUMED;
DockerTester() {
@@ -97,15 +97,15 @@ public class DockerTester implements AutoCloseable {
new NodeAgentContextImpl.Builder(hostName).fileSystem(fileSystem).build(), nodeRepository,
orchestrator, dockerOperations, storageMaintainer, clock, INTERVAL, Optional.empty(), Optional.empty());
nodeAdmin = new NodeAdminImpl(nodeAgentFactory, Optional.empty(), mr, Clock.systemUTC());
- nodeAdminStateUpdaterImpl = new NodeAdminStateUpdaterImpl(nodeRepository, orchestrator,
+ nodeAdminStateUpdater = new NodeAdminStateUpdater(nodeRepository, orchestrator,
nodeAdmin, HOST_HOSTNAME);
this.loopThread = new Thread(() -> {
- nodeAdminStateUpdaterImpl.start();
+ nodeAdminStateUpdater.start();
while (! terminated) {
try {
- nodeAdminStateUpdaterImpl.converge(wantedState);
+ nodeAdminStateUpdater.converge(wantedState);
} catch (RuntimeException e) {
log.info(e.getMessage());
}
@@ -126,7 +126,7 @@ public class DockerTester implements AutoCloseable {
.build());
}
- void setWantedState(NodeAdminStateUpdaterImpl.State wantedState) {
+ void setWantedState(NodeAdminStateUpdater.State wantedState) {
this.wantedState = wantedState;
}
diff --git a/node-admin/src/test/java/com/yahoo/vespa/hosted/node/admin/integrationTests/RebootTest.java b/node-admin/src/test/java/com/yahoo/vespa/hosted/node/admin/integrationTests/RebootTest.java
index b0de5d67eb2..7e17e2df530 100644
--- a/node-admin/src/test/java/com/yahoo/vespa/hosted/node/admin/integrationTests/RebootTest.java
+++ b/node-admin/src/test/java/com/yahoo/vespa/hosted/node/admin/integrationTests/RebootTest.java
@@ -6,7 +6,7 @@ import com.yahoo.vespa.hosted.dockerapi.ContainerName;
import com.yahoo.vespa.hosted.dockerapi.ContainerResources;
import com.yahoo.vespa.hosted.dockerapi.DockerImage;
import com.yahoo.vespa.hosted.node.admin.configserver.noderepository.NodeSpec;
-import com.yahoo.vespa.hosted.node.admin.nodeadmin.NodeAdminStateUpdaterImpl;
+import com.yahoo.vespa.hosted.node.admin.nodeadmin.NodeAdminStateUpdater;
import com.yahoo.vespa.hosted.provision.Node;
import org.junit.Test;
@@ -37,7 +37,7 @@ public class RebootTest {
eq(dockerImage), eq(ContainerResources.from(0, 0)), eq(new ContainerName("host1")), eq(hostname));
try {
- tester.setWantedState(NodeAdminStateUpdaterImpl.State.SUSPENDED);
+ tester.setWantedState(NodeAdminStateUpdater.State.SUSPENDED);
} catch (RuntimeException ignored) { }
tester.inOrder(tester.orchestrator).suspend(
diff --git a/node-admin/src/test/java/com/yahoo/vespa/hosted/node/admin/nodeadmin/NodeAdminStateUpdaterImplTest.java b/node-admin/src/test/java/com/yahoo/vespa/hosted/node/admin/nodeadmin/NodeAdminStateUpdaterTest.java
index 2abb096a098..437195ca6d5 100644
--- a/node-admin/src/test/java/com/yahoo/vespa/hosted/node/admin/nodeadmin/NodeAdminStateUpdaterImplTest.java
+++ b/node-admin/src/test/java/com/yahoo/vespa/hosted/node/admin/nodeadmin/NodeAdminStateUpdaterTest.java
@@ -15,9 +15,9 @@ import java.util.List;
import java.util.stream.Collectors;
import java.util.stream.IntStream;
-import static com.yahoo.vespa.hosted.node.admin.nodeadmin.NodeAdminStateUpdaterImpl.State.RESUMED;
-import static com.yahoo.vespa.hosted.node.admin.nodeadmin.NodeAdminStateUpdaterImpl.State.SUSPENDED;
-import static com.yahoo.vespa.hosted.node.admin.nodeadmin.NodeAdminStateUpdaterImpl.State.SUSPENDED_NODE_ADMIN;
+import static com.yahoo.vespa.hosted.node.admin.nodeadmin.NodeAdminStateUpdater.State.RESUMED;
+import static com.yahoo.vespa.hosted.node.admin.nodeadmin.NodeAdminStateUpdater.State.SUSPENDED;
+import static com.yahoo.vespa.hosted.node.admin.nodeadmin.NodeAdminStateUpdater.State.SUSPENDED_NODE_ADMIN;
import static org.junit.Assert.assertEquals;
import static org.junit.Assert.fail;
import static org.mockito.ArgumentMatchers.any;
@@ -32,17 +32,17 @@ import static org.mockito.Mockito.verify;
import static org.mockito.Mockito.when;
/**
- * Basic test of NodeAdminStateUpdaterImpl
+ * Basic test of NodeAdminStateUpdater
*
* @author freva
*/
-public class NodeAdminStateUpdaterImplTest {
+public class NodeAdminStateUpdaterTest {
private final NodeRepository nodeRepository = mock(NodeRepository.class);
private final Orchestrator orchestrator = mock(Orchestrator.class);
private final NodeAdmin nodeAdmin = mock(NodeAdmin.class);
private final HostName hostHostname = HostName.from("basehost1.test.yahoo.com");
- private final NodeAdminStateUpdaterImpl refresher = spy(new NodeAdminStateUpdaterImpl(
+ private final NodeAdminStateUpdater refresher = spy(new NodeAdminStateUpdater(
nodeRepository, orchestrator, nodeAdmin, hostHostname));
@@ -160,7 +160,7 @@ public class NodeAdminStateUpdaterImplTest {
verify(orchestrator, times(1)).suspend(eq(hostHostname.value()), eq(activeHostnames));
}
- private void assertResumeStateError(NodeAdminStateUpdaterImpl.State targetState, String reason) {
+ private void assertResumeStateError(NodeAdminStateUpdater.State targetState, String reason) {
try {
refresher.converge(targetState);
fail("Expected set resume state to fail with \"" + reason + "\", but it succeeded without error");