aboutsummaryrefslogtreecommitdiffstats
path: root/node-repository
diff options
context:
space:
mode:
authorMartin Polden <mpolden@mpolden.no>2018-08-09 13:58:49 +0200
committerMartin Polden <mpolden@mpolden.no>2018-08-09 14:53:54 +0200
commitf0ea42d7c2c905d6a68ba8307e4f90422907e07a (patch)
treef6d8358770641d703e0cf7a8b84addeec2946e25 /node-repository
parent495feef286d1b88c139fca71d62c3943b1068820 (diff)
Clean up signatures
Diffstat (limited to 'node-repository')
-rw-r--r--node-repository/src/test/java/com/yahoo/vespa/hosted/provision/maintenance/NodeFailerTest.java6
-rw-r--r--node-repository/src/test/java/com/yahoo/vespa/hosted/provision/maintenance/NodeRebooterTest.java2
-rw-r--r--node-repository/src/test/java/com/yahoo/vespa/hosted/provision/maintenance/NodeRetirerTest.java2
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();
}
+
}