From a4289711e20d2f411a00c64f786b431596487779 Mon Sep 17 00:00:00 2001
From: Khalid Shakir <kshakir@broadinstitute.org>
Date: Sat, 1 Feb 2014 19:58:49 +0800
Subject: [PATCH] Distinct failsafe summary reports, just like invoker report
 directories.

---
 pom.xml                      | 5 +++++
 public/package-tests/pom.xml | 2 ++
 2 files changed, 7 insertions(+)

diff --git a/pom.xml b/pom.xml
index 0f2885b50..60b19f1b2 100644
--- a/pom.xml
+++ b/pom.xml
@@ -549,6 +549,8 @@
                         it overwrites the results of the previous invocation always with a zero exit status.
 
                         Mixing in the test name into the reportsDirectory also avoids collisions, when different maven jobs run tests in parallel.
+
+                        Similarly generating unique failsafe summary reports to avoid collisions.
                         -->
                     </configuration>
                     <executions>
@@ -584,6 +586,7 @@
                                 <properties>
                                     <integrationtests.profile.enabled>true</integrationtests.profile.enabled>
                                     <sting.packageintegrationtests.skipped>${sting.packageintegrationtests.skipped}</sting.packageintegrationtests.skipped>
+                                    <failsafe.summaryFile>${project.build.directory}/failsafe-reports/integration/failsafe-summary-${it.test}.xml</failsafe.summaryFile>
                                 </properties>
                             </configuration>
                         </execution>
@@ -602,6 +605,7 @@
                                 <properties>
                                     <integrationtests.profile.enabled>true</integrationtests.profile.enabled>
                                     <sting.packagepipelinetests.skipped>${sting.packagepipelinetests.skipped}</sting.packagepipelinetests.skipped>
+                                    <failsafe.summaryFile>${project.build.directory}/failsafe-reports/pipeline/failsafe-summary-${it.test}.xml</failsafe.summaryFile>
                                 </properties>
                             </configuration>
                         </execution>
@@ -620,6 +624,7 @@
                                 <properties>
                                     <integrationtests.profile.enabled>true</integrationtests.profile.enabled>
                                     <sting.packagelargescaletests.skipped>${sting.packagelargescaletests.skipped}</sting.packagelargescaletests.skipped>
+                                    <failsafe.summaryFile>${project.build.directory}/failsafe-reports/largescale/failsafe-summary-${it.test}.xml</failsafe.summaryFile>
                                 </properties>
                             </configuration>
                         </execution>
diff --git a/public/package-tests/pom.xml b/public/package-tests/pom.xml
index 5e3269d0f..62a1d570b 100644
--- a/public/package-tests/pom.xml
+++ b/public/package-tests/pom.xml
@@ -23,6 +23,7 @@
         <sting.packageintegrationtests.skipped>true</sting.packageintegrationtests.skipped>
         <sting.packagepipelinetests.skipped>true</sting.packagepipelinetests.skipped>
         <sting.packagelargescaletests.skipped>true</sting.packagelargescaletests.skipped>
+        <failsafe.summaryFile>${project.build.directory}/failsafe-reports/failsafe-summary.xml</failsafe.summaryFile>
     </properties>
 
     <!-- Dependency configuration (versions, etc.) -->
@@ -135,6 +136,7 @@
                             <workingDirectory>${sting.packagetests.basedir}</workingDirectory>
                             <classesDirectory>${project.build.outputDirectory}/ignored_by_package_test</classesDirectory>
                             <testClassesDirectory>${sting.packagetests.testClasses}</testClassesDirectory>
+                            <summaryFile>${failsafe.summaryFile}</summaryFile>
                             <dependenciesToScan>
                                 <scan>org.broadinstitute.sting:.*</scan>
                             </dependenciesToScan>