Merge pull request #176 from edburns/edburns/adjust-reporting-for-two-executions

Ensure reporting is aware of the two test executions.

Author: edburns

.github/actions/test-report/action.yml

@@ -4,7 +4,7 @@ inputs:
   report-path:
     description: "Path to the test report XML files (glob pattern)"
     required: false
-    default: "target/surefire-reports/TEST-*.xml"
+    default: "target/surefire-reports*/TEST-*.xml"
   jacoco-path:
     description: "Path to the JaCoCo XML report"
     required: false
@@ -22,10 +22,7 @@ runs:
         echo "## 🧪 Copilot Java SDK :: Test Results" >> $GITHUB_STEP_SUMMARY
         echo "" >> $GITHUB_STEP_SUMMARY
 
-        REPORT_DIR=$(dirname "${{ inputs.report-path }}")
-        REPORT_PATTERN=$(basename "${{ inputs.report-path }}")
-
-        if [ -d "$REPORT_DIR" ]; then
+        if ls ${{ inputs.report-path }} 1>/dev/null 2>&1; then
           TESTS_RUN=$(grep -h "tests=" ${{ inputs.report-path }} 2>/dev/null | sed 's/.*tests="\([0-9]*\)".*/\1/' | awk '{s+=$1} END {print s}')
           FAILURES=$(grep -h "failures=" ${{ inputs.report-path }} 2>/dev/null | sed 's/.*failures="\([0-9]*\)".*/\1/' | awk '{s+=$1} END {print s}')
           ERRORS=$(grep -h "errors=" ${{ inputs.report-path }} 2>/dev/null | sed 's/.*errors="\([0-9]*\)".*/\1/' | awk '{s+=$1} END {print s}')

.github/workflows/build-test.yml

@@ -130,6 +130,7 @@ jobs:
           path: |
             target/jacoco-test-results/sdk-tests.exec
             target/surefire-reports/
+            target/surefire-reports-isolated/
           retention-days: 1
 
       - name: Generate JaCoCo badge

pom.xml

@@ -359,6 +359,11 @@
                         </goals>
                         <configuration>
                             <groups>isolated-resume</groups>
+                            <!-- Use a separate report directory so these results
+                                 don't overwrite the default-test XML reports for
+                                 the same test classes (CopilotSessionTest,
+                                 StreamingFidelityTest). -->
+                            <reportsDirectory>${project.build.directory}/surefire-reports-isolated</reportsDirectory>
                         </configuration>
                     </execution>
                     <!-- Exclude the isolated resume tests from the main run -->