diff options
author | Valerij Fredriksen <valerijf@oath.com> | 2018-10-26 15:46:59 +0200 |
---|---|---|
committer | Valerij Fredriksen <valerijf@oath.com> | 2018-10-26 15:46:59 +0200 |
commit | 2d677c94c9a727c565e879262dc99706d78fa0b3 (patch) | |
tree | 1db9ba9e659756d339fc97f4d3f9c2d5290a80f8 /node-admin/src/test/java/com | |
parent | bfa236c47b1d663b5b91b3bb35c8cf199ae85823 (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.java | 14 | ||||
-rw-r--r-- | node-admin/src/test/java/com/yahoo/vespa/hosted/node/admin/integrationTests/RebootTest.java | 4 | ||||
-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"); |