Skip to content

[main] Update dependencies from devdiv/DevDiv/vs-code-coverage - #12016

Merged
Amaury Levé (Evangelink) merged 1 commit into
mainfrom
darc-main-1b249d58-e688-4e81-8b5e-35bd318d3858
Oct 19, 2026
Merged

Amaury Levé (Evangelink) merged 1 commit into
mainfrom
darc-main-1b249d58-e688-4e81-8b5e-35bd318d3858

Conversation

@dotnet-maestro

Copy link
Copy Markdown
Contributor

This pull request updates the following dependencies

From https://dev.azure.com/devdiv/DevDiv/_git/vs-code-coverage

…code-coverage build 20261016.5

On relative base path root
Microsoft.Testing.Extensions.AffectedTests , Microsoft.Testing.Extensions.CodeCoverage From Version 18.13.0-preview.26515.3 -> To Version 18.13.0-preview.26516.5
Copilot AI balanced review requested due to automatic review settings October 19, 2026 07:59
@github-actions github-actions Bot added the state/needs-review Awaiting review from the team. label Oct 19, 2026
@github-actions
github-actions Bot enabled auto-merge (squash) October 19, 2026 07:59

@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.

🔵 Needs a closer look

Package compatibility remains unverified despite consistent dependency metadata.

No open issues 🎉

What changed in this PR

Updates the CodeCoverage and AffectedTests dependencies from the upstream vs-code-coverage build.

Changes:

  • Bumps both packages to 18.13.0-preview.26516.5.
  • Synchronizes dependency tracking with upstream commit abd830dbe8726ef298073037c45e4b2f6d5e1db8.

Test-value evidence: The supplied versions and commit match the PR description; compatibility results for consumers of these packages were unavailable.

Confidence at a glance

🟡 Dependency update Metadata is consistent. Upstream behavior and package compatibility remain unverified.
File Description
eng/​Versions.props Updates both package version properties.
eng/​Version.Details.xml Updates tracked package versions and upstream commit hashes.

Review effort: Balanced

@github-actions

Copy link
Copy Markdown
Contributor

Final test-triage resolution — Azure Pipelines build 1639917

This resolution supersedes any earlier preliminary comment for this build.

Build outcome: The Windows Release leg failed. Linux Release and Windows Debug legs are not reported as failed in the timeline. The only failing pipeline step was "Test affected changes" (exit code 2), caused by two product-area test assertion failures targeting net472:

  • HangDumpMutationTests.cs:719 — NamedPipeServer_DisposeAfterConnection_WaitsForTheReadLoopToExit: Assert.IsTrue(disposalThread.Join(TimeSpan.FromSeconds(30))) failed — the dispose thread did not finish within 30s after the read loop task completed.
  • HangDumpTests.cs:315 — QueryOnceAndDumpTree_StartsEveryDumpBeforeAwaitingCompletion: Assert.AreSame(allDumpsStarted.Task, completed) failed — a dump completion was observed before the "all dumps started" signal.

Cross-leg correlation: Both tests passed in every other matrix leg captured in this build's evidence, including the matching Windows Debug net472 run (NamedPipeServer...: 2342ms passed; QueryOnceAndDumpTree...: 10995ms passed) and all Linux/net8.0–net11.0 runs. This is consistent with timing-sensitive coordination asserts (30s join, task-identity race) rather than a deterministic logic regression, and the PR itself only touches eng/Version.Details.xml/eng/Versions.props (a Microsoft.Testing.Extensions.AffectedTests/CodeCoverage dependency bump), which does not touch hang-dump code.

Historical check: history.json is explicitly incomplete (artifact-download-failed, download-budget-exhausted), so prior occurrences of these two specific tests could not be confirmed or ruled out across the last 7 captured builds. No existing GitHub issue references these test names or their assertion messages.

Other candidates: The remaining 6 failed/retried records (WinUIMtpPackagedApp.RetryFailsFirstAttempt, PackagedWinUI.RetryTestMethod_PassesOnSecondAppContainerHost, NativeTests.Outcome) are intentional retry-probe fixtures that deliberately fail-then-pass by design; they recovered on retry as expected and are not product defects.

Conclusion: One-off, test-area-isolated failure, not reproduced on any other leg/TFM in this build, unrelated to the PR's diff, and not corroborated by history (incomplete). This does not meet the durable-issue threshold (two independent builds, or deterministic regression evidence). No issue is being created. If this pair of tests fails again on a future net472 run, please flag it so a flaky-test issue can be opened with corroborating recurrence evidence.

🤖 Automated content by GitHub Copilot. Generated by the Pipeline Test Triage workflow. · copilot · auto · 109.7 AIC · ⌖ 4.49 AIC · ⊞ 9.7K

@Evangelink
Amaury Levé (Evangelink) merged commit d25ab5d into main Oct 19, 2026
34 of 38 checks passed
@Evangelink
Amaury Levé (Evangelink) deleted the darc-main-1b249d58-e688-4e81-8b5e-35bd318d3858 branch October 19, 2026 18:12
@github-actions github-actions Bot removed the state/needs-review Awaiting review from the team. label Oct 19, 2026
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