SONARPLUGINS-1466 Display all missing mandatory properties

2 files changed, 9 insertions, 2 deletions

diff --git a/src/main/java/org/sonar/runner/Runner.java b/src/main/java/org/sonar/runner/Runner.java
index a4d02b8..65fdaf3 100644
--- a/src/main/java/org/sonar/runner/Runner.java
+++ b/src/main/java/org/sonar/runner/Runner.java
@@ -79,11 +79,18 @@ public final class Runner {
   }
 
   void checkMandatoryProperties() {
+    StringBuilder missing = new StringBuilder();
     for (String mandatoryProperty : MANDATORY_PROPERTIES) {
       if (!properties.containsKey(mandatoryProperty)) {
-        throw new RunnerException("You must define mandatory property: " + mandatoryProperty);
+        if (missing.length() > 0) {
+          missing.append(", ");
+        }
+        missing.append(mandatoryProperty);
       }
     }
+    if (missing.length() != 0) {
+      throw new RunnerException("You must define mandatory properties: " + missing);
+    }
   }
 
   public String getServerURL() {
diff --git a/src/test/java/org/sonar/runner/RunnerTest.java b/src/test/java/org/sonar/runner/RunnerTest.java
index 2026269..b4ed378 100644
--- a/src/test/java/org/sonar/runner/RunnerTest.java
+++ b/src/test/java/org/sonar/runner/RunnerTest.java
@@ -40,7 +40,7 @@ public class RunnerTest {
       Runner.create(new Properties()).checkMandatoryProperties();
       fail();
     } catch (RunnerException e) {
-      assertThat(e.getMessage(), is("You must define mandatory property: sonar.projectKey"));
+      assertThat(e.getMessage(), is("You must define mandatory properties: sonar.projectKey, sonar.projectName, sonar.projectVersion, sources"));
     }
   }