aboutsummaryrefslogtreecommitdiffstats
path: root/src/test/java/org/sonarsource/scanner/cli
diff options
context:
space:
mode:
Diffstat (limited to 'src/test/java/org/sonarsource/scanner/cli')
-rw-r--r--src/test/java/org/sonarsource/scanner/cli/CliTest.java46
-rw-r--r--src/test/java/org/sonarsource/scanner/cli/MainTest.java21
2 files changed, 64 insertions, 3 deletions
diff --git a/src/test/java/org/sonarsource/scanner/cli/CliTest.java b/src/test/java/org/sonarsource/scanner/cli/CliTest.java
index 640ca8d..9fb5786 100644
--- a/src/test/java/org/sonarsource/scanner/cli/CliTest.java
+++ b/src/test/java/org/sonarsource/scanner/cli/CliTest.java
@@ -47,10 +47,19 @@ public class CliTest {
}
@Test
+ public void should_fail_on_missing_prop() {
+ logs = mock(Logs.class);
+ cli = new Cli(exit, logs);
+ cli.parse(new String[] {"-D"});
+ verify(logs).error("Missing argument for option -D/--define");
+ verify(exit).exit(Exit.ERROR);
+ }
+
+ @Test
public void should_not_fail_with_errors_option() {
cli.parse(new String[] {"-e"});
}
-
+
@Test
public void should_parse_optional_task() {
cli.parse(new String[] {"-D", "foo=bar"});
@@ -68,6 +77,25 @@ public class CliTest {
}
@Test
+ public void should_enable_debug_mode_full() {
+ cli.parse(new String[] {"--debug"});
+ assertThat(cli.isDebugEnabled()).isTrue();
+ assertThat(cli.properties().get("sonar.verbose")).isEqualTo("true");
+ }
+
+ @Test
+ public void should_show_version() {
+ cli.parse(new String[] {"-v"});
+ assertThat(cli.isDisplayVersionOnly()).isTrue();
+ }
+
+ @Test
+ public void should_show_version_full() {
+ cli.parse(new String[] {"--version"});
+ assertThat(cli.isDisplayVersionOnly()).isTrue();
+ }
+
+ @Test
public void should_enable_stacktrace_log() {
cli.parse(new String[] {"-e"});
assertThat(cli.isDebugEnabled()).isFalse();
@@ -75,6 +103,13 @@ public class CliTest {
}
@Test
+ public void should_enable_stacktrace_log_full() {
+ cli.parse(new String[] {"--errors"});
+ assertThat(cli.isDebugEnabled()).isFalse();
+ assertThat(cli.properties().get("sonar.verbose")).isNull();
+ }
+
+ @Test
public void should_disable_debug_mode_and_stacktrace_log_by_default() {
cli.parse(new String[0]);
assertThat(cli.isDebugEnabled()).isFalse();
@@ -91,6 +126,15 @@ public class CliTest {
}
@Test
+ public void should_show_usage_full() {
+ logs = mock(Logs.class);
+ cli = new Cli(exit, logs);
+ cli.parse(new String[] {"--help"});
+ verify(logs).info("usage: sonar-scanner [options]");
+ verify(exit).exit(Exit.SUCCESS);
+ }
+
+ @Test
public void should_show_usage_on_bad_syntax() {
logs = mock(Logs.class);
cli = new Cli(exit, logs);
diff --git a/src/test/java/org/sonarsource/scanner/cli/MainTest.java b/src/test/java/org/sonarsource/scanner/cli/MainTest.java
index 3484aef..b410cbd 100644
--- a/src/test/java/org/sonarsource/scanner/cli/MainTest.java
+++ b/src/test/java/org/sonarsource/scanner/cli/MainTest.java
@@ -95,7 +95,23 @@ public class MainTest {
}
@Test
- public void show_error() {
+ public void should_exit_on_error() {
+ EmbeddedScanner runner = mock(EmbeddedScanner.class);
+ Exception e = new NullPointerException("NPE");
+ e = new IllegalStateException("Error", e);
+ doThrow(e).when(runner).stop();
+ when(runnerFactory.create(any(Properties.class))).thenReturn(runner);
+
+ Main main = new Main(exit, cli, conf, runnerFactory, logs);
+ main.execute();
+
+ verify(runner).stop();
+ verify(exit).exit(Exit.ERROR);
+ verify(logs).error("Unable to properly stop the scanner", e);
+ }
+
+ @Test
+ public void show_error_with_stacktrace() {
Exception e = createException(false);
testException(e, false);
@@ -109,6 +125,7 @@ public class MainTest {
testException(e, false);
verify(logs).error("Error during SonarQube Scanner execution");
+ verify(logs).error("Caused by: NPE");
verify(logs).error("Re-run SonarQube Scanner using the -X switch to enable full debug logging.");
}
@@ -138,7 +155,7 @@ public class MainTest {
private Exception createException(boolean messageException) {
Exception e;
if (messageException) {
- e = new MessageException("my message");
+ e = new MessageException("my message", new NullPointerException("NPE"));
} else {
e = new IllegalStateException("Error", new NullPointerException("NPE"));
}