diff options
author | Jon Marius Venstad <venstad@gmail.com> | 2019-02-11 12:08:49 +0100 |
---|---|---|
committer | Jon Marius Venstad <venstad@gmail.com> | 2019-02-11 12:08:49 +0100 |
commit | 11396b2c05a8ee029443481ff4f0f606a101c389 (patch) | |
tree | 98325904c52d2dad31a56c82c89285cf1a754537 /orchestrator/src/test | |
parent | 884d2897fa24c5642a0bf8b53f9665f9d23f5c85 (diff) |
Put accessors back inside MutableStatusRegistry
Diffstat (limited to 'orchestrator/src/test')
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()); |