Skip to content

Re-enable Blazor AppsettingsTests disabled during .NET 11 transition#124217

Merged
maraf merged 2 commits intomainfrom
copilot/re-enable-tests-runtime-issue-122338
Feb 11, 2026
Merged

Re-enable Blazor AppsettingsTests disabled during .NET 11 transition#124217
maraf merged 2 commits intomainfrom
copilot/re-enable-tests-runtime-issue-122338

Conversation

Copy link
Contributor

Copilot AI commented Feb 10, 2026

Fixes #122338

Description

Wasm.Build.Tests.Blazor.AppsettingsTests.FileInVfs was disabled during the .NET 10→11 transition due to runtime pack path version mismatches. The repository is now on .NET 11.0.0, resolving the underlying issue.

Changes

  • Removed [ActiveIssue] attribute from AppsettingsTests.FileInVfs
  • Added Wasm.Build.Tests.Blazor.AppsettingsTests to BuildWasmAppsJobsList.txt for CI execution

Testing

Test logic unchanged. Original failure was environmental (version 10.0.0 vs expected 11.0.0), now resolved.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • aka.ms
    • Triggering command: /usr/bin/curl curl -I -sSL --retry 5 --retry-delay 2 --connect-timeout 15 REDACTED o/mini/.. /home/REDACTED/work/runtime/runtime/src/mono/brows-fignore-exceptions ttin�� -DDISABLE_CONTRACTS (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Original prompt

Re-enable tests from #122338


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

@github-actions github-actions bot added the needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners label Feb 10, 2026
Copilot AI changed the title [WIP] Re-enable tests from runtime issue 122338 Re-enable Blazor AppsettingsTests disabled during .NET 11 transition Feb 10, 2026
Copilot AI requested a review from maraf February 10, 2026 13:27
@maraf maraf added arch-wasm WebAssembly architecture area-Infrastructure-mono os-browser Browser variant of arch-wasm and removed needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners labels Feb 10, 2026
@maraf maraf added this to the 11.0.0 milestone Feb 10, 2026
@maraf maraf marked this pull request as ready for review February 10, 2026 16:44
@maraf maraf requested a review from ilonatommy as a code owner February 10, 2026 16:44
Copilot AI review requested due to automatic review settings February 10, 2026 16:44
@dotnet-policy-service
Copy link
Contributor

Tagging subscribers to this area: @akoeplinger, @matouskozak, @simonrozsival
See info in area-owners.md if you want to be subscribed.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR re-enables the Wasm.Build.Tests.Blazor.AppsettingsTests.FileInVfs test that was disabled during the .NET 10→11 transition. The test was failing due to runtime pack version mismatches (expected 11.0.0 but found 10.0.0). Now that the repository has completed the transition to .NET 11, the underlying issue is resolved and the test can be safely re-enabled.

Changes:

  • Removed [ActiveIssue] attribute blocking test execution
  • Added test class to CI execution list for automated testing on Helix

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
src/mono/wasm/Wasm.Build.Tests/Blazor/AppsettingsTests.cs Removed [ActiveIssue] attribute from FileInVfs test method to re-enable execution
eng/testing/scenarios/BuildWasmAppsJobsList.txt Added Wasm.Build.Tests.Blazor.AppsettingsTests to enable parallel test execution on Helix CI infrastructure

@maraf maraf merged commit 73f9225 into main Feb 11, 2026
45 of 46 checks passed
@akoeplinger akoeplinger deleted the copilot/re-enable-tests-runtime-issue-122338 branch February 11, 2026 11:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

arch-wasm WebAssembly architecture area-Infrastructure-mono os-browser Browser variant of arch-wasm

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[browser][WBT] Blazor.AppsettingsTests - disabled

4 participants