Skip to content

Re-enable .NET 12 framework assembly checks - #69974

Merged
wtgodbe merged 3 commits into
dotnet:mainfrom
wtgodbe:wtgodbe/reenable-net12-framework-tests
Oct 23, 2026
Merged

wtgodbe merged 3 commits into
dotnet:mainfrom
wtgodbe:wtgodbe/reenable-net12-framework-tests

Conversation

@wtgodbe

@wtgodbe wtgodbe commented Oct 22, 2026 •

Copy link
Copy Markdown
Member

Restore framework identity checks after the .NET 12 dependency rollover.

Description

The temporary bootstrap assembly-version pin was already removed in #69836. This PR re-enables the three framework tests without weakening their assertions:

  • SharedFxTests.SharedFrameworkAssembliesHaveExpectedAssemblyVersions
  • TargetingPackTests.RefAssembliesHaveExpectedAssemblyVersions
  • TargetingPackTests.AssembliesAreReferenceAssemblies

Components.AI retains its existing ASP.NET Core 11 Web package dependency. Its dependency migration is intentionally excluded from this PR.

Validation

Validated locally on macOS arm64 with the repository-pinned .NET 12 SDK and DotNetUseShippingVersions=true:

  • The three restored framework tests changed from three skipped to three passed. A run covering these tests and three adjacent controls passed all six tests.
  • All five runtime-provided DLLs in the built Microsoft.AspNetCore.App.Ref NuGet package have ReferenceAssemblyAttribute and assembly version 12.0.0.0: System.Security.Cryptography.Xml, System.Security.Cryptography.Pkcs, System.Formats.Cbor, System.Diagnostics.EventLog, and System.Threading.RateLimiting.

Windows and Linux validation is left to CI.

Related to #69595.

wtgodbe and others added 2 commits October 22, 2026 09:46
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot AI balanced review requested due to automatic review settings October 22, 2026 16:46
@wtgodbe
wtgodbe requested a review from a team as a code owner October 22, 2026 16:46
@github-actions github-actions Bot added area-blazor Includes: Blazor, Razor Components area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework labels Oct 22, 2026
@dotnet-policy-service

Copy link
Copy Markdown
Contributor

Hey @dotnet/aspnet-build, looks like this PR is something you want to take a look at.

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

Human review should confirm Windows/Linux CI and the generated package dependency graph, which were not independently validated here.

No open issues 🎉

What changed in this PR

Completes the .NET 12 rollover follow-up by restoring framework identity checks and aligning Components.AI with the in-tree Components.Web build.

Changes:

  • Re-enables three framework tests without changing their assertions.
  • Replaces Components.AI’s ASP.NET Core 11 Web package reference with an in-tree project reference, retaining Web as a package dependency when packed.
  • Removes the unused central Components.Web package version pin.
File Description
src/​Framework/​test/​TargetingPackTests.cs Restores assembly-version and reference-assembly checks.
src/​Framework/​test/​SharedFxTests.cs Restores shared-framework assembly-version checks.
src/​Components/​AI/​src/​Microsoft.AspNetCore.Components.AI.csproj References the in-tree Components.Web project.
Directory.Packages.props Removes the obsolete Components.Web 11 package pin.

Review effort: Balanced

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@wtgodbe wtgodbe changed the title Re-enable framework checks and align Components.AI with .NET 12 Re-enable .NET 12 framework assembly checks Oct 22, 2026
@wtgodbe
wtgodbe merged commit 559db70 into dotnet:main Oct 23, 2026
30 checks passed
@wtgodbe
wtgodbe temporarily deployed to copilot-pat-pool October 23, 2026 05:01 — with GitHub Actions Inactive
@wtgodbe
wtgodbe temporarily deployed to copilot-pat-pool October 23, 2026 05:01 — with GitHub Actions Inactive
@wtgodbe
wtgodbe temporarily deployed to copilot-pat-pool October 23, 2026 05:02 — with GitHub Actions Inactive
@wtgodbe
wtgodbe temporarily deployed to copilot-pat-pool October 23, 2026 05:04 — with GitHub Actions Inactive
@wtgodbe
wtgodbe temporarily deployed to copilot-pat-pool October 23, 2026 05:05 — with GitHub Actions Inactive
@wtgodbe
wtgodbe temporarily deployed to copilot-pat-pool October 23, 2026 05:06 — with GitHub Actions Inactive
@github-actions

Copy link
Copy Markdown
Contributor

✅ No documentation PR was created.

Confidence that documentation is needed: 2%

No docs needed: the PR only re-enables internal framework assembly tests.

Documentation surfaces considered

  • Conceptual article: Not required — Only framework test files changed; no user-visible behavior.
  • Migration guidance: Not required — No upgrade action for users.
  • Breaking change: Not required — No compatibility impact; tests re-enabled only.

This branch was previously deployed

1 inactive deployment
copilot-pat-pool — 512fe60c Deployed Oct 23, 2026 by wtgodbe via conclusion #89
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-blazor Includes: Blazor, Razor Components area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants