Fix on parallel branches bug

Author: imonteroperez

src/main/java/org/jenkinsci/plugins/JiraTestResultReporter/JiraTestData.java

@@ -15,19 +15,24 @@
  */
 package org.jenkinsci.plugins.JiraTestResultReporter;
 
+import java.util.Collections;
+import java.util.List;
+import java.util.logging.Level;
+import java.util.logging.Logger;
+
 import hudson.EnvVars;
 import hudson.tasks.junit.CaseResult;
 import hudson.tasks.junit.TestAction;
 import hudson.tasks.junit.TestObject;
 import hudson.tasks.junit.TestResultAction;
 
-import java.util.Collections;
-import java.util.List;
-
 /**
  * Created by tuicu.
  */
 public class JiraTestData extends TestResultAction.Data {
+    
+    private static final Logger LOGGER = Logger.getLogger("JiraIssuePlugin.log");
+    
     private EnvVars envVars;
 
     /**
@@ -47,6 +52,10 @@ public EnvVars getEnvVars() {
     }
 
 
+    public static void log(String message) {
+        LOGGER.log(Level.INFO, message);
+    }
+    
     /**
      * Method for creating test actions associated with tests
      * @param testObject

src/main/java/org/jenkinsci/plugins/JiraTestResultReporter/JiraTestDataPublisher.java

@@ -210,9 +210,23 @@ public TestResultAction.Data contributeTestData(Run<?, ?> run, @Nonnull FilePath
             hasTestData |= unlinkIssuesForPassedTests(listener, project, job, envVars, getTestCaseResults(testResult));
         }
 
-        return hasTestData? new JiraTestData(envVars) : null;
+        if (hasTestData) {
+            // Workaround to make feasible to use the publisher in parallel executions
+            if (notReportedTestDataBefore(envVars)) {
+                JiraTestDataRegistry.getInstance().put(envVars);
+                return JiraTestDataRegistry.getInstance().get(envVars);
+            } else {
+                return null;
+            }
+        } else {
+            return null;
+        }
 	}
 
+    private boolean notReportedTestDataBefore(EnvVars envVars) {
+        return JiraTestDataRegistry.getInstance().get(envVars) == null;
+    }
+
     private boolean unlinkIssuesForPassedTests(TaskListener listener, Job project, Job job, EnvVars envVars, List<CaseResult> testCaseResults) {
         boolean unlinked = false;
         for(CaseResult test : testCaseResults) {

src/main/java/org/jenkinsci/plugins/JiraTestResultReporter/JiraTestDataRegistry.java

@@ -0,0 +1,37 @@
+package org.jenkinsci.plugins.JiraTestResultReporter;
+
+import java.util.HashMap;
+import java.util.Map;
+
+import hudson.EnvVars;
+
+public class JiraTestDataRegistry {
+    
+    private static final String BUILD_URL = "BUILD_URL";
+
+    private static JiraTestDataRegistry instance = new JiraTestDataRegistry();
+    
+    private Map<String, JiraTestData> jiraTestDataByUrl;
+    
+    private JiraTestDataRegistry() {
+        jiraTestDataByUrl = new HashMap<String, JiraTestData>(); 
+    }
+    
+    public static JiraTestDataRegistry getInstance() {
+        return instance;
+    }
+
+    public JiraTestData get(EnvVars envVars) {
+        String key = envVars.get(BUILD_URL);
+        synchronized (key) {
+            return jiraTestDataByUrl.get(key);
+        }
+    }
+
+    public void put(EnvVars envVars) {
+        String key = envVars.get(BUILD_URL);
+        synchronized (key) {
+            jiraTestDataByUrl.put(key, new JiraTestData(envVars));
+        }
+    }
+}