Skip to content

Fix unit test result publishing in PR pipeline#5618

Merged
mikaelweave merged 7 commits into
mainfrom
mikaelweave/fix-pr-test-results-publish
Jun 18, 2026
Merged

Fix unit test result publishing in PR pipeline#5618
mikaelweave merged 7 commits into
mainfrom
mikaelweave/fix-pr-test-results-publish

Conversation

@mikaelweave

@mikaelweave mikaelweave commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

Summary

  • add TRX generation (--report-trx) to the unit test coverage step in build/jobs/build.yml
  • disable built-in DotNetCoreCLI test result publishing for that step (publishTestResults: false)
  • add explicit PublishTestResults@2 with succeededOrFailed() against $(Agent.TempDirectory)/coverage/**/*.trx

Why

  • in build 49903, unit test failures were not reliably surfacing in the Azure DevOps Tests tab when the coverage test step failed
  • this aligns unit coverage behavior with the existing explicit publish pattern already used by integration/e2e jobs
CleanShot 2026-06-17 at 15 13 01

Tracking

Add TRX output and explicit PublishTestResults for coverage-based unit test runs so failed tests are surfaced reliably in Azure DevOps Tests tab.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@mikaelweave mikaelweave requested a review from a team as a code owner June 17, 2026 01:39
@codecov-commenter

codecov-commenter commented Jun 17, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 76.90%. Comparing base (abca618) to head (52ee55d).

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #5618   +/-   ##
=======================================
  Coverage   76.90%   76.90%           
=======================================
  Files         997      997           
  Lines       36616    36616           
  Branches     5529     5529           
=======================================
  Hits        28160    28160           
+ Misses       7104     7102    -2     
- Partials     1352     1354    +2     

see 6 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@mikaelweave mikaelweave added this to the FY26\Q4\2Wk\2Wk26 milestone Jun 17, 2026
@mikaelweave mikaelweave added Bug Bug bug bug. Open source This change is only relevant to the OSS code or release. No-ADR ADR not needed No-PaaS-breaking-change labels Jun 17, 2026
mikaelweave and others added 5 commits June 17, 2026 06:41
Inject an intentional failure to validate PR pipeline test reporting behavior before reverting.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Force a SQL E2E chaining test assertion mismatch so Azure DevOps publishes a real failed test result through the explicit PublishTestResults path.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Revert the temporary E2E assertion change and add an intentional failure in a unit test so validation targets unit-test publication.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@mikaelweave

Copy link
Copy Markdown
Contributor Author

/arp run

@mikaelweave mikaelweave merged commit 109e69b into main Jun 18, 2026
50 of 51 checks passed
@mikaelweave mikaelweave deleted the mikaelweave/fix-pr-test-results-publish branch June 18, 2026 03:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bug Bug bug bug. No-ADR ADR not needed No-PaaS-breaking-change Open source This change is only relevant to the OSS code or release.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants