diff options
author | Sébastien Lesaint <sebastien.lesaint@sonarsource.com> | 2019-09-05 12:29:59 +0200 |
---|---|---|
committer | SonarTech <sonartech@sonarsource.com> | 2019-09-06 20:21:05 +0200 |
commit | 306cc616fcf6a3e2d624ddfd19eb948351c76593 (patch) | |
tree | 3a9075059bc6e5ac239d0e102a89a7bbd185a0c3 /server/sonar-main/src | |
parent | 0a49bc18a94656ee6cd6e70e17bb60d49f3622a3 (diff) | |
download | sonarqube-306cc616fcf6a3e2d624ddfd19eb948351c76593.tar.gz sonarqube-306cc616fcf6a3e2d624ddfd19eb948351c76593.zip |
remove dead code in SchedulerImplTest (and fix imports)
and for the sake of clean logs, indicate CE is operational
Diffstat (limited to 'server/sonar-main/src')
-rw-r--r-- | server/sonar-main/src/test/java/org/sonar/application/SchedulerImplTest.java | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/server/sonar-main/src/test/java/org/sonar/application/SchedulerImplTest.java b/server/sonar-main/src/test/java/org/sonar/application/SchedulerImplTest.java index 533e9abc932..cf6f704916e 100644 --- a/server/sonar-main/src/test/java/org/sonar/application/SchedulerImplTest.java +++ b/server/sonar-main/src/test/java/org/sonar/application/SchedulerImplTest.java @@ -56,8 +56,8 @@ import static org.sonar.process.ProcessId.ELASTICSEARCH; import static org.sonar.process.ProcessId.WEB_SERVER; import static org.sonar.process.ProcessProperties.Property.CLUSTER_ENABLED; import static org.sonar.process.ProcessProperties.Property.CLUSTER_NODE_HOST; -import static org.sonar.process.ProcessProperties.Property.CLUSTER_NODE_NAME; import static org.sonar.process.ProcessProperties.Property.CLUSTER_NODE_HZ_PORT; +import static org.sonar.process.ProcessProperties.Property.CLUSTER_NODE_NAME; import static org.sonar.process.ProcessProperties.Property.CLUSTER_NODE_TYPE; public class SchedulerImplTest { @@ -119,6 +119,7 @@ public class SchedulerImplTest { web.operational = true; waitForAppStateOperational(appState, WEB_SERVER); TestManagedProcess ce = processLauncher.waitForProcess(COMPUTE_ENGINE); + ce.operational = true; assertThat(ce.isAlive()).isTrue(); assertThat(processLauncher.processes).hasSize(3); assertThat(processLauncher.commands).containsExactly(esScriptCommand, webLeaderCommand, ceCommand); @@ -157,6 +158,7 @@ public class SchedulerImplTest { web.operational = true; waitForAppStateOperational(appState, WEB_SERVER); TestManagedProcess ce = processLauncher.waitForProcess(COMPUTE_ENGINE); + ce.operational = true; assertThat(ce.isAlive()).isTrue(); assertThat(processLauncher.processes).hasSize(3); assertThat(processLauncher.commands).containsExactly(esScriptCommand, webLeaderCommand, ceCommand); @@ -440,7 +442,6 @@ public class SchedulerImplTest { private final ProcessId processId; private final CountDownLatch alive = new CountDownLatch(1); private boolean operational = false; - private boolean askedForStop = false; private boolean askedForRestart = false; private TestManagedProcess(ProcessId processId) { @@ -468,7 +469,6 @@ public class SchedulerImplTest { @Override public void askForStop() { - this.askedForStop = true; destroyForcibly(); } |