[JENKINS-51645] Add support for claims plugin in xUnit

Use collection or pipepine is not able to construct the list

Author: nfalco79

src/main/java/org/jenkinsci/plugins/xunit/XUnitProcessor.java

@@ -30,7 +30,6 @@
 import java.io.InputStream;
 import java.util.Arrays;
 import java.util.Collection;
-import java.util.Set;
 import java.util.UUID;
 
 import javax.annotation.CheckForNull;
@@ -143,7 +142,7 @@ public XUnitProcessor(@Nonnull TestType[] tools,
         this.processorId = UUID.randomUUID().toString();
     }
 
-    public void process(Run<?, ?> build, FilePath workspace, TaskListener listener, Launcher launcher, @Nonnull Set<TestDataPublisher> testDataPublishers)
+    public void process(Run<?, ?> build, FilePath workspace, TaskListener listener, Launcher launcher, @Nonnull Collection<TestDataPublisher> testDataPublishers)
      throws IOException, InterruptedException {
         logger = new XUnitLog(listener);
         logger.info("Starting to record.");
@@ -331,7 +330,7 @@ private TestResult recordTestResult(Run<?, ?> build,
                                         FilePath workspace,
                                         TaskListener listener,
                                         Launcher launcher,
-                                        Set<TestDataPublisher> testDataPublishers) throws IOException, InterruptedException {
+                                        Collection<TestDataPublisher> testDataPublishers) throws IOException, InterruptedException {
         TestResultAction existingAction = build.getAction(TestResultAction.class);
         final long buildTime = build.getTimestamp().getTimeInMillis();
         final long nowMaster = System.currentTimeMillis();

src/main/java/org/jenkinsci/plugins/xunit/XUnitPublisher.java

@@ -26,9 +26,9 @@
 
 import java.io.IOException;
 import java.util.Arrays;
+import java.util.Collection;
 import java.util.Collections;
-import java.util.LinkedHashSet;
-import java.util.Set;
+import java.util.LinkedList;
 
 import javax.annotation.CheckForNull;
 import javax.annotation.Nonnull;
@@ -37,8 +37,6 @@
 import org.jenkinsci.lib.dtkit.descriptor.TestTypeDescriptor;
 import org.jenkinsci.lib.dtkit.type.TestType;
 import org.jenkinsci.plugins.xunit.service.TransformerException;
-import org.jenkinsci.plugins.xunit.threshold.FailedThreshold;
-import org.jenkinsci.plugins.xunit.threshold.SkippedThreshold;
 import org.jenkinsci.plugins.xunit.threshold.XUnitThreshold;
 import org.jenkinsci.plugins.xunit.threshold.XUnitThresholdDescriptor;
 import org.kohsuke.stapler.DataBoundConstructor;
@@ -78,7 +76,7 @@ public class XUnitPublisher extends Recorder implements SimpleBuildStep {
     private XUnitThreshold[] thresholds;
     private int thresholdMode;
     private ExtraConfiguration extraConfiguration;
-    private Set<TestDataPublisher> testDataPublishers;
+    private Collection<TestDataPublisher> testDataPublishers;
 
     @DataBoundConstructor
     public XUnitPublisher(@CheckForNull TestType[] tools, @CheckForNull XUnitThreshold[] thresholds, int thresholdMode, @CheckForNull String testTimeMargin) {
@@ -126,7 +124,7 @@ public ExtraConfiguration getExtraConfiguration() {
         return extraConfiguration;
     }
 
-    public @Nonnull Set<TestDataPublisher> getTestDataPublishers() {
+    public @Nonnull Collection<TestDataPublisher> getTestDataPublishers() {
         return testDataPublishers;
     }
 
@@ -139,10 +137,10 @@ public ExtraConfiguration getExtraConfiguration() {
      *            publisher
      */
     @DataBoundSetter
-    public void setTestDataPublishers(@CheckForNull Set<TestDataPublisher> testDataPublishers) {
+    public void setTestDataPublishers(@CheckForNull Collection<TestDataPublisher> testDataPublishers) {
         this.testDataPublishers = testDataPublishers != null
-                ? new LinkedHashSet<>(testDataPublishers)
-                : Collections.<TestDataPublisher> emptySet();
+                ? new LinkedList<>(testDataPublishers)
+                : Collections.<TestDataPublisher> emptyList();
     }
 
     @SuppressWarnings("deprecation")