Spun off from review feedback on #16986 by @Youssef1313 (#16986 (comment) and #16986 (comment)).
Context
PR #16986 renamed the Helix Sdk CreateXUnitV3WorkItems task / XUnitV3Project item to CreateMTPWorkItems / MTPProject and unconditionally produces an MTP command line that:
- Requires the test project to reference
Microsoft.Testing.Extensions.TrxReport (previously: xUnit XML reporter).
- Emits
--report-trx --report-trx-filename testResults.trx (previously: --report-xunit-xml --report-xunit-xml-filename testResults.xml or legacy -xml testResults.xml -noAutoReporters depending on UseMicrosoftTestingPlatformRunner).
This is intentional and the PR is flagged as breaking. The concern raised in review is that downstream repos consuming the Helix Sdk may still be on:
- xUnit v3 without MTP enabled (
UseMicrosoftTestingPlatformRunner previously gated this), or
- xUnit v3 with MTP enabled but producing xUnit-format reports instead of TRX.
Both groups will see a hard break the first time they pick up the new Helix Sdk.
Action items
cc @Youssef1313 @ilyas1974
Spun off from review feedback on #16986 by @Youssef1313 (#16986 (comment) and #16986 (comment)).
Context
PR #16986 renamed the Helix Sdk
CreateXUnitV3WorkItemstask /XUnitV3Projectitem toCreateMTPWorkItems/MTPProjectand unconditionally produces an MTP command line that:Microsoft.Testing.Extensions.TrxReport(previously: xUnit XML reporter).--report-trx --report-trx-filename testResults.trx(previously:--report-xunit-xml --report-xunit-xml-filename testResults.xmlor legacy-xml testResults.xml -noAutoReportersdepending onUseMicrosoftTestingPlatformRunner).This is intentional and the PR is flagged as breaking. The concern raised in review is that downstream repos consuming the Helix Sdk may still be on:
UseMicrosoftTestingPlatformRunnerpreviously gated this), orBoth groups will see a hard break the first time they pick up the new Helix Sdk.
Action items
<XUnitV3Project>(search acrossdotnet/*and known consumers).Microsoft.Testing.Extensions.TrxReport.CreateXUnitV3WorkItemstask is kept as a thin shim forwarding toCreateMTPWorkItemswith a warning.cc @Youssef1313 @ilyas1974