diff options
| -rw-r--r-- | src/main/java/org/sonar/runner/Runner.java | 9 | ||||
| -rw-r--r-- | src/test/java/org/sonar/runner/RunnerTest.java | 2 |
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")); } } |