SQSCANNER-67 - Warn on duplicate property definitions (#89)

src/main/java/org/sonarsource/scanner/cli/Cli.java

@@ -84,7 +84,8 @@ class Cli {
       displayVersionOnly = true;
 
     } else if (asList("-e", "--errors").contains(arg)) {
-      logger.info("Option -e/--errors is no longer supported and will be ignored");
+      logger
+        .info("Option -e/--errors is no longer supported and will be ignored");
 
     } else if (asList("-X", "--debug").contains(arg)) {
       props.setProperty("sonar.verbose", "true");
@@ -95,7 +96,8 @@ class Cli {
       return processProp(args, pos);
 
     } else if ("--embedded".equals(arg)) {
-      logger.info("Option --embedded is deprecated and will be removed in a future release.");
+      logger.info(
+        "Option --embedded is deprecated and will be removed in a future release.");
       embedded = true;
 
     } else if (arg.startsWith("--from")) {
@@ -130,7 +132,7 @@ class Cli {
     displayVersionOnly = false;
   }
 
-  private static void appendPropertyTo(String arg, Properties props) {
+  private void appendPropertyTo(String arg, Properties props) {
     final String key;
     final String value;
     int j = arg.indexOf('=');
@@ -141,7 +143,11 @@ class Cli {
       key = arg.substring(0, j);
       value = arg.substring(j + 1);
     }
-    props.setProperty(key, value);
+    Object oldValue = props.setProperty(key, value);
+    if (oldValue != null) {
+      logger.warn("Property '" + key + "' with value '" + oldValue + "' is "
+        + "overridden with value '" + value + "'");
+    }
   }
 
   private void printErrorAndExit(String message) {

src/test/java/org/sonarsource/scanner/cli/CliTest.java

@@ -47,6 +47,14 @@ public class CliTest {
     assertThat(cli.properties().get("boolean")).isEqualTo("true");
   }
 
+  @Test
+  public void should_warn_on_duplicate_properties() {
+    logs = mock(Logs.class);
+    cli = new Cli(exit, logs);
+    cli.parse(new String[] {"-D", "foo=bar", "--define", "foo=baz"});
+    verify(logs).warn("Property 'foo' with value 'bar' is overridden with value 'baz'");
+  }
+
   @Test
   public void should_fail_on_missing_prop() {
     logs = mock(Logs.class);