diff options
author | Simon Brandhof <simon.brandhof@sonarsource.com> | 2015-06-29 15:53:56 +0200 |
---|---|---|
committer | Simon Brandhof <simon.brandhof@sonarsource.com> | 2015-06-29 19:01:12 +0200 |
commit | 501289ca85e1c8dd21d5e0ba5f457b40ee62f5ef (patch) | |
tree | ba5b06c4357edb58ec3f248c6fa12324c1ec71ca /server/sonar-process-monitor/src | |
parent | fcf0d15e049e9c6e647fae628dc0e2bc17d078b6 (diff) | |
download | sonarqube-501289ca85e1c8dd21d5e0ba5f457b40ee62f5ef.tar.gz sonarqube-501289ca85e1c8dd21d5e0ba5f457b40ee62f5ef.zip |
Quality flaws
- do not declare multiple variables on same line
- fix imports
Diffstat (limited to 'server/sonar-process-monitor/src')
-rw-r--r-- | server/sonar-process-monitor/src/test/java/org/sonar/process/monitor/MonitorTest.java | 12 |
1 files changed, 8 insertions, 4 deletions
diff --git a/server/sonar-process-monitor/src/test/java/org/sonar/process/monitor/MonitorTest.java b/server/sonar-process-monitor/src/test/java/org/sonar/process/monitor/MonitorTest.java index 3edd35324a9..a144d7aad1d 100644 --- a/server/sonar-process-monitor/src/test/java/org/sonar/process/monitor/MonitorTest.java +++ b/server/sonar-process-monitor/src/test/java/org/sonar/process/monitor/MonitorTest.java @@ -142,7 +142,8 @@ public class MonitorTest { @Test public void start_then_stop_sequence_of_commands() throws Exception { monitor = newDefaultMonitor(); - HttpProcessClient p1 = new HttpProcessClient("p1"), p2 = new HttpProcessClient("p2"); + HttpProcessClient p1 = new HttpProcessClient("p1"); + HttpProcessClient p2 = new HttpProcessClient("p2"); monitor.start(Arrays.asList(p1.newCommand(), p2.newCommand())); // start p2 when p1 is fully started (ready) @@ -163,7 +164,8 @@ public class MonitorTest { @Test public void stop_all_processes_if_monitor_shutdowns() throws Exception { monitor = newDefaultMonitor(); - HttpProcessClient p1 = new HttpProcessClient("p1"), p2 = new HttpProcessClient("p2"); + HttpProcessClient p1 = new HttpProcessClient("p1"); + HttpProcessClient p2 = new HttpProcessClient("p2"); monitor.start(Arrays.asList(p1.newCommand(), p2.newCommand())); assertThat(p1.isReady()).isTrue(); assertThat(p2.isReady()).isTrue(); @@ -179,7 +181,8 @@ public class MonitorTest { @Test public void stop_all_processes_if_one_shutdowns() throws Exception { monitor = newDefaultMonitor(); - HttpProcessClient p1 = new HttpProcessClient("p1"), p2 = new HttpProcessClient("p2"); + HttpProcessClient p1 = new HttpProcessClient("p1"); + HttpProcessClient p2 = new HttpProcessClient("p2"); monitor.start(Arrays.asList(p1.newCommand(), p2.newCommand())); assertThat(p1.isReady()).isTrue(); assertThat(p2.isReady()).isTrue(); @@ -197,7 +200,8 @@ public class MonitorTest { @Test public void stop_all_processes_if_one_fails_to_start() throws Exception { monitor = newDefaultMonitor(); - HttpProcessClient p1 = new HttpProcessClient("p1"), p2 = new HttpProcessClient("p2", -1); + HttpProcessClient p1 = new HttpProcessClient("p1"); + HttpProcessClient p2 = new HttpProcessClient("p2", -1); try { monitor.start(Arrays.asList(p1.newCommand(), p2.newCommand())); fail(); |