Do not allow to change ProjectDefinition#properties

2 files changed, 11 insertions, 13 deletions

diff --git a/sonar-plugin-api/src/main/java/org/sonar/api/batch/bootstrap/ProjectDefinition.java b/sonar-plugin-api/src/main/java/org/sonar/api/batch/bootstrap/ProjectDefinition.java
index cec0930c949..df7acb38b3a 100644
--- a/sonar-plugin-api/src/main/java/org/sonar/api/batch/bootstrap/ProjectDefinition.java
+++ b/sonar-plugin-api/src/main/java/org/sonar/api/batch/bootstrap/ProjectDefinition.java
@@ -54,20 +54,16 @@ public final class ProjectDefinition implements BatchComponent {
   private List<ProjectDefinition> subProjects = Lists.newArrayList();
   private List<Object> containerExtensions = Lists.newArrayList();
 
-  private ProjectDefinition() {
-  }
-
-  public static ProjectDefinition create() {
-    return new ProjectDefinition();
+  private ProjectDefinition(Properties p) {
+    this.properties = p;
   }
 
-  public ProjectDefinition setProperties(Properties p) {
-    this.properties = p;
-    return this;
+  public static ProjectDefinition create(Properties properties) {
+    return new ProjectDefinition(properties);
   }
 
-  public File getBaseDir() {
-    return baseDir;
+  public static ProjectDefinition create() {
+    return new ProjectDefinition(new Properties());
   }
 
   public ProjectDefinition setBaseDir(File baseDir) {
@@ -75,6 +71,9 @@ public final class ProjectDefinition implements BatchComponent {
     return this;
   }
 
+  public File getBaseDir() {
+    return baseDir;
+  }
   public ProjectDefinition setWorkDir(File workDir) {
     this.workDir = workDir;
     return this;
diff --git a/sonar-plugin-api/src/test/java/org/sonar/api/batch/bootstrap/ProjectDefinitionTest.java b/sonar-plugin-api/src/test/java/org/sonar/api/batch/bootstrap/ProjectDefinitionTest.java
index 93b9bcdfefa..581e9ff3361 100644
--- a/sonar-plugin-api/src/test/java/org/sonar/api/batch/bootstrap/ProjectDefinitionTest.java
+++ b/sonar-plugin-api/src/test/java/org/sonar/api/batch/bootstrap/ProjectDefinitionTest.java
@@ -30,7 +30,6 @@ import java.util.Properties;
 import static org.hamcrest.CoreMatchers.nullValue;
 import static org.hamcrest.Matchers.is;
 import static org.junit.Assert.assertThat;
-import static org.junit.matchers.JUnitMatchers.hasItem;
 
 public class ProjectDefinitionTest {
 
@@ -56,12 +55,12 @@ public class ProjectDefinitionTest {
     def.setKey("myKey");
     assertThat(def.getName(), is("Unnamed - myKey"));
   }
+
   @Test
   public void shouldGetKeyFromProperties() {
     Properties props = new Properties();
     props.setProperty(CoreProperties.PROJECT_KEY_PROPERTY, "foo");
-    ProjectDefinition def = ProjectDefinition.create();
-    def.setProperties(props);
+    ProjectDefinition def = ProjectDefinition.create(props);
     assertThat(def.getKey(), is("foo"));
   }