Skip to content

Align MSTest/testfx package versions and document the sync requirement#55316

Open
Evangelink wants to merge 1 commit into
mainfrom
dev/amauryleve/align-mstest-versions
Open

Align MSTest/testfx package versions and document the sync requirement#55316
Evangelink wants to merge 1 commit into
mainfrom
dev/amauryleve/align-mstest-versions

Conversation

@Evangelink

Copy link
Copy Markdown
Member

What

Aligns the microsoft/testfx dependency group to a single coherent build and documents that these versions must stay in sync.

Bumps to the latest coherent testfx build (4.4.0-preview.26366.2, MTP 2.4.0-preview.26366.2, testfx commit c1e6453) across the three places that must agree:

  • global.json msbuild-sdksMSTest.Sdk (the source generator)
  • eng/Version.Details.propsMSTestPackageVersion + MicrosoftTestingPlatformPackageVersion
  • eng/Version.Details.xmlMSTest, MSTest.Sdk, Microsoft.Testing.Platform (version + Sha)

Why

The MSTest.Sdk source generator emits test registrations that compile and run against the MSTest.TestFramework runtime, so both must come from the same testfx build. A skew breaks the NativeAOT CLI test build:

  • generator lagging the framework → Zero tests ran (MTP exit code 8)
  • generator leading the framework → CS0234 on DynamicDataSourceResolver (a type added in a newer testfx build)

MSTest.TestAdapter 26366.2 also transitively requires Microsoft.Testing.Platform 26366.2, so that package moves with the group.

Adds KEEP IN SYNC comments above each version so future manual edits keep them together. Follow-up to #55302 (which made MSTest.Sdk darc-managed) and to a companion PR that stops dependabot from bumping MSTest.Sdk independently.

Bumps the microsoft/testfx dependency group to the latest coherent build
(4.4.0-preview.26366.2 / MTP 2.4.0-preview.26366.2, testfx c1e6453) across all
three places that must agree:

- global.json msbuild-sdks -> MSTest.Sdk (the source generator)
- eng/Version.Details.props -> MSTestPackageVersion + MicrosoftTestingPlatformPackageVersion
- eng/Version.Details.xml -> MSTest, MSTest.Sdk, Microsoft.Testing.Platform (+ Sha)

The MSTest.Sdk source generator emits test registrations against the
MSTest.TestFramework runtime, so these must come from the same testfx build.
A skew breaks the NativeAOT CLI test build ("Zero tests ran" when the generator
lags, CS0234 on DynamicDataSourceResolver when it leads). Adds KEEP-IN-SYNC
comments over each version so future manual edits keep them together. See #55302.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 1b315003-e0fc-4a09-865e-c8f4750fe3a5
Copilot AI review requested due to automatic review settings July 16, 2026 10:18
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 1 pipeline(s).
2 pipeline(s) were filtered out due to trigger conditions.
There may be pipelines that require an authorized user to comment /azp run to run.

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

Aligns the microsoft/testfx dependency versions across the SDK’s three “must match” update points so the MSTest source generator (MSTest.Sdk) and runtime/framework (MSTest.* + Microsoft.Testing.Platform) remain on the same coherent build, preventing NativeAOT test discovery/build breaks.

Changes:

  • Bump global.json msbuild-sdks MSTest.Sdk to 4.4.0-preview.26366.2.
  • Update eng/Version.Details.xml testfx dependency versions + Sha to 4.4.0-preview.26366.2 / 2.4.0-preview.26366.2 with matching commit c1e6453….
  • Update eng/Version.Details.props package version properties and add “KEEP IN SYNC” documentation notes.
Show a summary per file
File Description
global.json Updates the pinned MSBuild SDK version for MSTest.Sdk to the coherent testfx build.
eng/Version.Details.xml Updates darc-tracked testfx dependencies (versions + Sha) and documents the required synchronization.
eng/Version.Details.props Updates the corresponding version properties and adds guidance to keep the group coherent.

Copilot's findings

  • Files reviewed: 3/3 changed files
  • Comments generated: 0

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.

3 participants