diff options
author | Martin Polden <mpolden@mpolden.no> | 2018-08-09 13:58:49 +0200 |
---|---|---|
committer | Martin Polden <mpolden@mpolden.no> | 2018-08-09 14:53:54 +0200 |
commit | f0ea42d7c2c905d6a68ba8307e4f90422907e07a (patch) | |
tree | f6d8358770641d703e0cf7a8b84addeec2946e25 /node-repository | |
parent | 495feef286d1b88c139fca71d62c3943b1068820 (diff) |
Clean up signatures
Diffstat (limited to 'node-repository')
3 files changed, 5 insertions, 5 deletions
diff --git a/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/maintenance/NodeFailerTest.java b/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/maintenance/NodeFailerTest.java index 63bc04ac671..c483615203d 100644 --- a/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/maintenance/NodeFailerTest.java +++ b/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/maintenance/NodeFailerTest.java @@ -8,8 +8,6 @@ import com.yahoo.vespa.applicationmodel.ServiceInstance; import com.yahoo.vespa.applicationmodel.ServiceStatus; import com.yahoo.vespa.hosted.provision.Node; import com.yahoo.vespa.hosted.provision.NodeRepository; -import com.yahoo.vespa.orchestrator.ApplicationIdNotFoundException; -import com.yahoo.vespa.orchestrator.ApplicationStateChangeDeniedException; import org.junit.Test; import java.time.Duration; @@ -39,7 +37,7 @@ import static org.mockito.Mockito.when; public class NodeFailerTest { @Test - public void nodes_for_suspended_applications_are_not_failed() throws ApplicationStateChangeDeniedException, ApplicationIdNotFoundException { + public void nodes_for_suspended_applications_are_not_failed() { NodeFailTester tester = NodeFailTester.withTwoApplications(); tester.suspend(NodeFailTester.app1); @@ -57,7 +55,7 @@ public class NodeFailerTest { } @Test - public void node_failing() throws InterruptedException { + public void node_failing() { NodeFailTester tester = NodeFailTester.withTwoApplications(); // For a day all nodes work so nothing happens diff --git a/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/maintenance/NodeRebooterTest.java b/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/maintenance/NodeRebooterTest.java index 7e1f4f60b65..f17995a5210 100644 --- a/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/maintenance/NodeRebooterTest.java +++ b/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/maintenance/NodeRebooterTest.java @@ -17,7 +17,7 @@ import static org.junit.Assert.assertEquals; public class NodeRebooterTest { @Test - public void testRebootScheduling() throws InterruptedException { + public void testRebootScheduling() { Duration rebootInterval = Duration.ofMinutes(250); MaintenanceTester tester = new MaintenanceTester(); tester.createReadyTenantNodes(15); diff --git a/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/maintenance/NodeRetirerTest.java b/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/maintenance/NodeRetirerTest.java index 13ac7b6f2d3..6eeeb3dfe08 100644 --- a/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/maintenance/NodeRetirerTest.java +++ b/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/maintenance/NodeRetirerTest.java @@ -32,6 +32,7 @@ import static org.mockito.Mockito.when; * @author freva */ public class NodeRetirerTest { + private NodeRetirerTester tester; private NodeRetirer retirer; private final RetirementPolicy policy = mock(RetirementPolicy.class); @@ -171,4 +172,5 @@ public class NodeRetirerTest { verify(retirer, never()).retireUnallocated(); verify(retirer, never()).retireAllocated(); } + } |