Skip to content

[main] Update dependencies from microsoft/testfx - #11058

Open
dotnet-maestro[bot] wants to merge 1 commit into
mainfrom
darc-main-b0053d66-2b82-4b48-aa13-0918b8ed9929
Open

[main] Update dependencies from microsoft/testfx#11058
dotnet-maestro[bot] wants to merge 1 commit into
mainfrom
darc-main-b0053d66-2b82-4b48-aa13-0918b8ed9929

Conversation

@dotnet-maestro

@dotnet-maestro dotnet-maestro Bot commented Sep 6, 2026

Copy link
Copy Markdown
Contributor

This pull request updates the following dependencies

From https://github.com/microsoft/testfx

…260905.2

On relative base path root
Microsoft.Testing.Platform From Version 2.5.0-preview.26454.15 -> To Version 2.5.0-preview.26455.2
MSTest From Version 4.5.0-preview.26454.15 -> To Version 4.5.0-preview.26455.2
Copilot AI balanced review requested due to automatic review settings September 6, 2026 02:02
@github-actions
github-actions Bot enabled auto-merge (squash) September 6, 2026 02:02

@microsoft-github-policy-service microsoft-github-policy-service Bot 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.

Auto-approve

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.

🟢 Approval recommended

The version and dependency metadata updates are complete and internally consistent.

Pull request overview

Updates MSTest and Microsoft.Testing.Platform dependencies to build 20260905.2.

Changes:

  • Bumps MSTest to 4.5.0-preview.26455.2.
  • Bumps Microsoft.Testing.Platform to 2.5.0-preview.26455.2.
  • Synchronizes dependency metadata with commit a179d7cba3.
File summaries
File Description
eng/Versions.props Updates package version properties.
eng/Version.Details.xml Updates dependency versions and commit SHAs.
Review details
  • Files reviewed: 2/2 changed files
  • Comments generated: 0
  • Review effort level: Balanced

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@github-actions

github-actions Bot commented Sep 6, 2026

Copy link
Copy Markdown
Contributor

**Final analysis — Azure Pipelines build [1584715]((dev.azure.com/redacted)

This is the completed (full) analysis for build 1584715, superseding any preliminary comment.

Failure signature: Dispose_FromANotificationHandler_DoesNotSelfWaitOnTheReadLoop (Microsoft.Testing.Platform.ServerMode.Client.Sources.UnitTests, net8.0) failed on the Windows Release leg only:

Assertion failed. Expected value to be less than the upper bound.
Dispose from a notification handler took 4803 ms; it must not self-wait on the read loop.
upper bound: 00:00:04, actual: 00:00:04.8028935

Correlation:

  • Only 1 failing/retried test candidate in this build; all other legs (Windows Debug, Linux Release, macOS Release) passed for this test.
  • No CTRF retry attempts recorded — this ran once and failed once (no fail-then-pass evidence, so it does not meet the flaky-test bar).
  • History (12 completed builds over the last 30 days, not flagged incomplete) shows this exact test passing on every sampled leg with durations of 9 ms–364 ms, well under the 4-second bound. There is no prior occurrence of this test approaching, let alone crossing, the threshold.

Assessment: This looks like a one-off timing-margin failure — the actual elapsed time (4.80 s) exceeded a hard-coded 4-second assertion bound by ~800 ms, most likely due to a momentarily loaded/slow Windows Release CI agent rather than a code or product regression (historical runs complete in well under 1 s). No open or recently closed issue matches this signature (checked issues #11039, #10898 — unrelated). Per the escalation thresholds, a single non-recurring, non-flaky, non-regression failure tied only to this PR's build does not warrant a new issue.

Next step if this recurs: if Dispose_FromANotificationHandler_DoesNotSelfWaitOnTheReadLoop fails again on Windows Release in a subsequent build (same or another PR), that would establish recurrence and justify either loosening the timing margin in MtpServerClientInProcessTests.cs:410 or filing a flaky/regression issue at that point.

No further action taken; re-run the Windows Release leg if needed.

🤖 Automated content by GitHub Copilot. Generated by the Pipeline Test Triage workflow. · auto · 47.3 AIC · ⌖ 1.77 AIC · ⊞ 9.2K · [◷]( · )

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.

2 participants