summaryrefslogtreecommitdiffstats
path: root/orchestrator/src/test
diff options
context:
space:
mode:
authorJon Marius Venstad <venstad@gmail.com>2019-02-11 12:08:49 +0100
committerJon Marius Venstad <venstad@gmail.com>2019-02-11 12:08:49 +0100
commit11396b2c05a8ee029443481ff4f0f606a101c389 (patch)
tree98325904c52d2dad31a56c82c89285cf1a754537 /orchestrator/src/test
parent884d2897fa24c5642a0bf8b53f9665f9d23f5c85 (diff)
Put accessors back inside MutableStatusRegistry
Diffstat (limited to 'orchestrator/src/test')
-rw-r--r--orchestrator/src/test/java/com/yahoo/vespa/orchestrator/model/ModelTestUtils.java2
-rw-r--r--orchestrator/src/test/java/com/yahoo/vespa/orchestrator/resources/HostResourceTest.java6
-rw-r--r--orchestrator/src/test/java/com/yahoo/vespa/orchestrator/status/ZookeeperStatusServiceTest.java4
3 files changed, 5 insertions, 7 deletions
diff --git a/orchestrator/src/test/java/com/yahoo/vespa/orchestrator/model/ModelTestUtils.java b/orchestrator/src/test/java/com/yahoo/vespa/orchestrator/model/ModelTestUtils.java
index f9a6dc5b237..78f50dbfc3f 100644
--- a/orchestrator/src/test/java/com/yahoo/vespa/orchestrator/model/ModelTestUtils.java
+++ b/orchestrator/src/test/java/com/yahoo/vespa/orchestrator/model/ModelTestUtils.java
@@ -74,7 +74,7 @@ class ModelTestUtils {
MutableStatusRegistry registry = statusService.lockApplicationInstance_forCurrentThreadOnly(
OrchestratorContext.createContextForSingleAppOp(Clock.systemUTC()),
applicationInstance.reference());
- return new ApplicationApiImpl(nodeGroup, registry, statusService, clusterControllerClientFactory);
+ return new ApplicationApiImpl(nodeGroup, registry, clusterControllerClientFactory);
}
ApplicationInstance createApplicationInstance(
diff --git a/orchestrator/src/test/java/com/yahoo/vespa/orchestrator/resources/HostResourceTest.java b/orchestrator/src/test/java/com/yahoo/vespa/orchestrator/resources/HostResourceTest.java
index 2a8fdc8290c..55ac65c036c 100644
--- a/orchestrator/src/test/java/com/yahoo/vespa/orchestrator/resources/HostResourceTest.java
+++ b/orchestrator/src/test/java/com/yahoo/vespa/orchestrator/resources/HostResourceTest.java
@@ -119,8 +119,7 @@ public class HostResourceTest {
public void releaseSuspensionGrant(
OrchestratorContext context, ApplicationInstance applicationInstance,
HostName hostName,
- MutableStatusRegistry hostStatusRegistry,
- StatusService statusService) {
+ MutableStatusRegistry hostStatusRegistry) {
}
}
@@ -221,8 +220,7 @@ public class HostResourceTest {
public void releaseSuspensionGrant(
OrchestratorContext context, ApplicationInstance applicationInstance,
HostName hostName,
- MutableStatusRegistry hostStatusRegistry,
- StatusService statusService) throws HostStateChangeDeniedException {
+ MutableStatusRegistry hostStatusRegistry) throws HostStateChangeDeniedException {
doThrow();
}
diff --git a/orchestrator/src/test/java/com/yahoo/vespa/orchestrator/status/ZookeeperStatusServiceTest.java b/orchestrator/src/test/java/com/yahoo/vespa/orchestrator/status/ZookeeperStatusServiceTest.java
index 809ca7e5c97..9b1be12121d 100644
--- a/orchestrator/src/test/java/com/yahoo/vespa/orchestrator/status/ZookeeperStatusServiceTest.java
+++ b/orchestrator/src/test/java/com/yahoo/vespa/orchestrator/status/ZookeeperStatusServiceTest.java
@@ -91,7 +91,7 @@ public class ZookeeperStatusServiceTest {
TestIds.HOST_NAME1,
hostStatus);
- assertThat(zookeeperStatusService.getHostStatus(TestIds.APPLICATION_INSTANCE_REFERENCE, TestIds.HOST_NAME1),
+ assertThat(statusRegistry.getHostStatus(TestIds.HOST_NAME1),
is(hostStatus));
}
}
@@ -141,7 +141,7 @@ public class ZookeeperStatusServiceTest {
killSession(curator.framework(), testingServer);
//Throws SessionFailedException if the SessionFailRetryLoop has not been closed.
- zookeeperStatusService2.getHostStatus(TestIds.APPLICATION_INSTANCE_REFERENCE, TestIds.HOST_NAME1);
+ statusRegistry.getHostStatus(TestIds.HOST_NAME1);
});
assertThat(resultOfZkOperationAfterLockFailure, notHoldsException());