Unquarantine WebWorker E2E tests and add diagnostics by ilonatommy · Pull Request #66072 · dotnet/aspnetcore

ilonatommy · 2026-03-30T15:15:52Z

Summary

Removes [QuarantinedTest] from 4 WebWorker template E2E tests quarantined in #66035. Adds build-time diagnostic logging to help diagnose CI-only failures if they recur.

What happened

CI server-side request logs (build 1353846) showed:

_framework/blazor.webassembly.js (unfingerprinted) → 404, only 2 fallback candidates, both rejected
_framework/dotnet.f86z0mrdmk.js (fingerprinted) → 200 OK, 4 candidates including direct routes

What we don't know

We could not reproduce the failure locally — the fingerprint placeholder #[.{fingerprint}] resolves correctly on current HEAD in both Debug and Release. The 51 commits between the breaking PR (#65885) and HEAD contain no relevant changes to static web assets, dev server, or templates. The root cause of the CI-only fingerprint resolution failure remains unknown.

Why diagnostics

Since the failure is not reproducible locally, we need CI-side evidence to diagnose it if it recurs. The LogBuildDiagnostics() method logs three things after each build/publish, written to test output (visible in CI artifacts):

Processed HTML — the actual <script> tags from the build's processed index.html. Shows whether
blazor.webassembly.js is fingerprinted or not.
Endpoints manifest routes — the registered routes for blazor.webassembly from the
.staticwebassets.endpoints.json. Shows whether the dev server would be able to serve the URL.
Source HTML — the raw template index.html with the #[.{fingerprint}] placeholder, confirming the input is
correct.

Together these pinpoint which stage failed: template source → MSBuild placeholder resolution → endpoint registration → served HTML.

The diagnostics are read-only (only File.ReadLines and JsonDocument.Parse on existing build output) and do not affect build behavior or test execution.

Fixes #66035

It had No-merge label because I wanted to run it several times before merging to collect logs but all runs were green. Removing the label.

Remove [QuarantinedTest] from 4 WebWorker template E2E tests that were quarantined by dotnet#66035. The tests pass locally on current HEAD. Add LogBuildDiagnostics() to capture: - Processed HTML script tags (fingerprint resolution) - Endpoints manifest routes for blazor.webassembly - Source index.html placeholder state This will help diagnose any CI-only failures where the fingerprint placeholder resolves to empty, causing blazor.webassembly.js 404s. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot

Pull request overview

This PR re-enables previously quarantined Blazor WebWorker template E2E tests and adds extra build-time logging intended to diagnose CI-only failures related to static asset fingerprinting and static web assets endpoint routing.

Changes:

Removed [QuarantinedTest] from 4 WebWorker template E2E tests.
Added LogBuildDiagnostics(...) and invoked it after build/publish to print processed HTML references and static web assets endpoints manifest routes.

src/ProjectTemplates/test/Templates.Blazor.Tests/WebWorkerTemplateE2ETest.cs

- Use TemplateBuildDir/TemplatePublishDir instead of hardcoded path - Guard against missing obj directory - Only log HTML files that have matching blazor.webassembly lines - Use File.ReadLines instead of File.ReadAllText - Log full exception (ex.ToString()) instead of just ex.Message Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

oroztocil · 2026-04-02T08:33:01Z

src/ProjectTemplates/test/Templates.Blazor.Tests/WebWorkerTemplateE2ETest.cs

+        // Log the processed index.html to check if fingerprints resolved correctly.
+        // The #[.{fingerprint}] placeholder should be replaced with an actual hash.
+        // If it resolves to empty, the script tag will reference the unfingerprinted
+        // 'blazor.webassembly.js' which returns 404 from the dev server.


Are we checking/asserting that this happened correctly somewhere? E.g., that the fingerprint is not empty and is not "#[.{fingerprint}]" still?

…rs, CanDisposeWorker) These tests were recently unquarantined in #66072 (issue #66035) but have already failed again (build 1342501). Re-quarantining to avoid CI disruption while the underlying flakiness is investigated. Associated issue: #aw_webwrk1 Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

The Case B re-quarantine logic was counting failures from before the unquarantine commit was merged. This led to tests being unnecessarily re-quarantined based on stale failure data (e.g., PR #66148 re-quarantined tests based on a failure from March 18, but the unquarantine PR #66072 wasn't merged until March 30). Updated to explicitly require that only failures occurring after the unquarantine merge date are considered. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

…66174) * Fix re-quarantine criteria to only count post-unquarantine failures The Case B re-quarantine logic was counting failures from before the unquarantine commit was merged. This led to tests being unnecessarily re-quarantined based on stale failure data (e.g., PR #66148 re-quarantined tests based on a failure from March 18, but the unquarantine PR #66072 wasn't merged until March 30). Updated to explicitly require that only failures occurring after the unquarantine merge date are considered. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Update .github/workflows/test-quarantine.md Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> --------- Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

…rantine ShutdownTests.OutOfProcessToInProcessHostingModelSwitchWorks - Re-quarantine WebWorkerTemplateE2ETest at class level (was unquarantined in #66072, failing again in build 1373142) - Quarantine BlazorWasmTemplateTest at class level (NU1603 NuGet restore failures in builds 1342501 and 1373142) - Unquarantine ShutdownTests.OutOfProcessToInProcessHostingModelSwitchWorks (100% pass rate in quarantine pipeline for 30+ days) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

… WebWorkerTemplateE2ETest; unquarantine ShutdownTests (#66294) * Quarantine BlazorWasmTemplateTest and WebWorkerTemplateE2ETest; unquarantine ShutdownTests.OutOfProcessToInProcessHostingModelSwitchWorks - Re-quarantine WebWorkerTemplateE2ETest at class level (was unquarantined in #66072, failing again in build 1373142) - Quarantine BlazorWasmTemplateTest at class level (NU1603 NuGet restore failures in builds 1342501 and 1373142) - Unquarantine ShutdownTests.OutOfProcessToInProcessHostingModelSwitchWorks (100% pass rate in quarantine pipeline for 30+ days) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Update WebWorkerTemplateE2ETest.cs * Update BlazorWasmTemplateTest.cs --------- Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Co-authored-by: William Godbe <wigodbe@microsoft.com>

Copilot AI review requested due to automatic review settings March 30, 2026 15:15

github-actions bot added the area-blazor Includes: Blazor, Razor Components label Mar 30, 2026

Copilot started reviewing on behalf of ilonatommy March 30, 2026 15:16 View session

ilonatommy self-assigned this Mar 30, 2026

ilonatommy added the * NO MERGE * Do not merge this PR as long as this label is present. label Mar 30, 2026

Copilot AI reviewed Mar 30, 2026

View reviewed changes

ilonatommy and others added 3 commits March 30, 2026 17:33

Merge branch 'main' into itomkowicz/unquarantine-webworker-tests

c085ef2

Merge branch 'main' into itomkowicz/unquarantine-webworker-tests

02c4aff

ilonatommy removed the * NO MERGE * Do not merge this PR as long as this label is present. label Apr 1, 2026

ilonatommy requested review from javiercn, oroztocil and wtgodbe April 1, 2026 10:28

ilonatommy enabled auto-merge (squash) April 1, 2026 10:40

oroztocil approved these changes Apr 2, 2026

View reviewed changes

ilonatommy merged commit c6843cc into dotnet:main Apr 2, 2026
25 checks passed

dotnet-policy-service bot added this to the 11.0-preview3 milestone Apr 2, 2026

dotnet-maestro bot mentioned this pull request Apr 3, 2026

[main] Source code updates from dotnet/aspnetcore dotnet/dotnet#5888

Merged

wtgodbe modified the milestones: 11.0-preview3, 11.0-preview4 Apr 4, 2026

github-actions bot mentioned this pull request Apr 4, 2026

Quarantine Re-quarantine WebWorkerTemplateE2ETest tests (CanInvokeMethods, HandlesErrors, CanDisposeWorker) #66138

Closed

github-actions bot mentioned this pull request Apr 4, 2026

[test-quarantine] Re-quarantine WebWorkerTemplateE2ETest (CanInvokeMethods, HandlesErrors, CanDisposeWorker) #66148

Closed

wtgodbe mentioned this pull request Apr 6, 2026

Fix re-quarantine criteria to only count post-unquarantine failures #66174

Merged

DeagleGross mentioned this pull request Apr 12, 2026

[test-quarantine] Re-quarantine WebWorkerTemplateE2ETest methods DeagleGross/aspnetcore#275

Open

github-actions bot mentioned this pull request Apr 13, 2026

Quarantine WebWorkerTemplateE2ETest (re-quarantine after #66072) #66292

Closed

github-actions bot mentioned this pull request Apr 13, 2026

[test-quarantine] Quarantine BlazorWasmTemplateTest and re-quarantine WebWorkerTemplateE2ETest; unquarantine ShutdownTests #66294

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Unquarantine WebWorker E2E tests and add diagnostics#66072

Unquarantine WebWorker E2E tests and add diagnostics#66072
ilonatommy merged 4 commits intodotnet:mainfrom
ilonatommy:itomkowicz/unquarantine-webworker-tests

ilonatommy commented Mar 30, 2026 •

edited

Loading

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

oroztocil Apr 2, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Conversation

ilonatommy commented Mar 30, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

What happened

What we don't know

Why diagnostics

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

oroztocil Apr 2, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

ilonatommy commented Mar 30, 2026 •

edited

Loading