Skip to content

Update MTPHelpSnapshotTests snapshot for --progress option changes#54950

Merged
Evangelink merged 1 commit into
mainfrom
michaelsimons/fix-mtp-snapshot
Jun 24, 2026
Merged

Update MTPHelpSnapshotTests snapshot for --progress option changes#54950
Evangelink merged 1 commit into
mainfrom
michaelsimons/fix-mtp-snapshot

Conversation

@MichaelSimons

@MichaelSimons MichaelSimons commented Jun 23, 2026

Copy link
Copy Markdown
Member

Summary

Regenerates the MTPHelpSnapshotTests.VerifyMTPHelpOutput Verify snapshot to match the current dotnet test --help output. This test is failing across all legs on the main CI build.

Changes to snapshot

  • --no-progress reference updated to --progress off in the --ansi platform option description
  • New --progress platform option added (control whether progress is reported to screen)
  • --ansi removed from Extension Options (it was duplicated from Platform Options)
  • {arch} (process architecture) placeholder added to --report-trx-filename description

Failing build

https://dev.azure.com/dnceng-public/public/_build/results?buildId=1477619

Fixes #54948

Regression introduced by #54765 (testfx dependency update: Microsoft.Testing.Platform 2.3.0-preview.26307.5 → 2.3.0-preview.26321.1)

The MTP help output was updated to include a new --progress option and
remove the duplicate --ansi from Extension Options. The snapshot file
was not regenerated, causing test failures across all legs.

Changes:
- --no-progress reference updated to --progress off
- New --progress platform option added
- --ansi removed from Extension Options (was duplicated from Platform Options)
- {arch} placeholder added to --report-trx-filename description

Fixes #54948

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings June 23, 2026 19:43
@MichaelSimons MichaelSimons requested a review from a team as a code owner June 23, 2026 19:43

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the Verify snapshot used by MTPHelpSnapshotTests.VerifyMTPHelpOutput so it matches the current dotnet test --help output for Microsoft.Testing.Platform, unblocking CI where the snapshot mismatch is failing on all legs.

Changes:

  • Updates --ansi help text to reference --progress off instead of --no-progress.
  • Adds the new --progress platform option block (including precedence over deprecated --no-progress).
  • Removes duplicated --ansi from Extension Options and updates --report-trx-filename placeholder list to include {arch}.

@MichaelSimons MichaelSimons requested a review from Evangelink June 23, 2026 19:52
@Youssef1313

Copy link
Copy Markdown
Member

/ba-g Failures look unrelated. @Evangelink Something broke with the MSTest upgrades, some Helix work items are failing with exit code 8 (zero tests run) when all tests skipped.

@Evangelink Evangelink merged commit bb45103 into main Jun 24, 2026
31 of 33 checks passed
@Evangelink Evangelink deleted the michaelsimons/fix-mtp-snapshot branch June 24, 2026 06:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

MTPHelpSnapshotTests.VerifyMTPHelpOutput snapshot mismatch (--progress option added)

5 participants