From: Simon Brandhof <simon.brandhof@gmail.com>
Date: Thu, 11 Apr 2013 16:13:49 +0000 (+0200)
Subject: SONAR-3755 fix component key and rename Issuable.IssueBuilder#create()
X-Git-Tag: 3.6~714
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=86033bd3c9ba856afdd42aaf06948169205a4522;p=sonarqube.git

SONAR-3755 fix component key and rename Issuable.IssueBuilder#create()
---

diff --git a/sonar-batch/src/main/java/org/sonar/batch/index/DefaultIndex.java b/sonar-batch/src/main/java/org/sonar/batch/index/DefaultIndex.java
index d7d4458c7d4..44055904fdf 100644
--- a/sonar-batch/src/main/java/org/sonar/batch/index/DefaultIndex.java
+++ b/sonar-batch/src/main/java/org/sonar/batch/index/DefaultIndex.java
@@ -366,6 +366,7 @@ public class DefaultIndex extends SonarIndex {
       return;
     }
 
+    violation.setResource(bucket.getResource());
     deprecatedViolations.add(violation);
     addViolation(violation, bucket, force);
   }
diff --git a/sonar-batch/src/main/java/org/sonar/batch/issue/IssueCache.java b/sonar-batch/src/main/java/org/sonar/batch/issue/IssueCache.java
index 2115546dcb6..bb9d98e5f2a 100644
--- a/sonar-batch/src/main/java/org/sonar/batch/issue/IssueCache.java
+++ b/sonar-batch/src/main/java/org/sonar/batch/issue/IssueCache.java
@@ -43,7 +43,6 @@ public class IssueCache implements BatchComponent {
   }
 
   public IssueCache add(Issue issue) {
-    System.out.println("Cache issue = " + issue.key() + " on " + issue.componentKey());
     componentIssues.put(issue.componentKey(), issue);
     return this;
   }
diff --git a/sonar-core/src/main/java/org/sonar/core/issue/DefaultIssueBuilder.java b/sonar-core/src/main/java/org/sonar/core/issue/DefaultIssueBuilder.java
index 435c0e036eb..917d19a58fc 100644
--- a/sonar-core/src/main/java/org/sonar/core/issue/DefaultIssueBuilder.java
+++ b/sonar-core/src/main/java/org/sonar/core/issue/DefaultIssueBuilder.java
@@ -88,7 +88,7 @@ public class DefaultIssueBuilder implements Issuable.IssueBuilder {
   }
 
   @Override
-  public Issue create() {
+  public Issue done() {
     Preconditions.checkNotNull(componentKey, "Component key must be set");
     Preconditions.checkNotNull(ruleRepository, "Rule repository must be set");
     Preconditions.checkNotNull(ruleKey, "Rule key must be set");
diff --git a/sonar-core/src/test/java/org/sonar/core/issue/DefaultIssueBuilderTest.java b/sonar-core/src/test/java/org/sonar/core/issue/DefaultIssueBuilderTest.java
index a113f2b810f..6e0bb3ea7ef 100644
--- a/sonar-core/src/test/java/org/sonar/core/issue/DefaultIssueBuilderTest.java
+++ b/sonar-core/src/test/java/org/sonar/core/issue/DefaultIssueBuilderTest.java
@@ -32,17 +32,19 @@ public class DefaultIssueBuilderTest {
 
   @Test
   public void should_create_issue() throws Exception {
-    DefaultIssue issue = (DefaultIssue) new DefaultIssueBuilder(callback, "org.apache.struts")
+    String componentKey = "org.apache.struts:struts-core:Action.java";
+    DefaultIssue issue = (DefaultIssue) new DefaultIssueBuilder(callback, componentKey)
       .message("msg")
       .line(123)
       .cost(10000.0)
       .rule("squid", "NullDereference")
       .severity(Issue.SEVERITY_CRITICAL)
-      .create();
+      .done();
 
     assertThat(issue).isNotNull();
     assertThat(issue.key()).isNotEmpty();
     assertThat(issue.cost()).isEqualTo(10000.0);
+    assertThat(issue.componentKey()).isEqualTo(componentKey);
     assertThat(issue.message()).isEqualTo("msg");
     assertThat(issue.line()).isEqualTo(123);
     assertThat(issue.ruleRepositoryKey()).isEqualTo("squid");
diff --git a/sonar-plugin-api/src/main/java/org/sonar/api/issue/Issuable.java b/sonar-plugin-api/src/main/java/org/sonar/api/issue/Issuable.java
index ceae0a99a08..ecc8dc4cfeb 100644
--- a/sonar-plugin-api/src/main/java/org/sonar/api/issue/Issuable.java
+++ b/sonar-plugin-api/src/main/java/org/sonar/api/issue/Issuable.java
@@ -45,7 +45,7 @@ public interface Issuable extends Perspective {
 
     IssueBuilder manual(boolean b);
 
-    Issue create();
+    Issue done();
   }
 
   IssueBuilder newIssue();