Skip to content

Ensure reporting is aware of the two test executions. #176

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
edburns merged 3 commits into from
May 8, 2026
+8 −5
Merged

Ensure reporting is aware of the two test executions. #176

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
a56a227
Ensure reporting is aware of the two test executions.
edburns May 8, 2026
0fbd0df
avoid builtin
edburns May 8, 2026
5df33ee
Remove dead code
edburns May 8, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 2 additions & 5 deletions .github/actions/test-report/action.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -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"
Comment thread
edburns marked this conversation as resolved.
jacoco-path:
description: "Path to the JaCoCo XML report"
required: false
Expand All @@ -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
Comment thread
edburns marked this conversation as resolved.
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}')
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/build-test.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down
5 changes: 5 additions & 0 deletions pom.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -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 -->
Expand Down