Skip to content

[blazor] E2E test coreCLR WASM#66331

Open
pavelsavara wants to merge 7 commits intodotnet:mainfrom
pavelsavara:blazor_wasm_coreCLR
Open

[blazor] E2E test coreCLR WASM#66331
pavelsavara wants to merge 7 commits intodotnet:mainfrom
pavelsavara:blazor_wasm_coreCLR

Conversation

@pavelsavara
Copy link
Copy Markdown
Member

@pavelsavara pavelsavara commented Apr 15, 2026

Reorder CoreCLR E2E job before Mono job for faster feedback.
Temporarily limit CoreCLR tests to StatePersistenceTest to debug
failures without exhausting CI agent disk space.
@pavelsavara pavelsavara marked this pull request as ready for review April 27, 2026 18:06
@pavelsavara pavelsavara requested a review from a team as a code owner April 27, 2026 18:06
Copilot AI review requested due to automatic review settings April 27, 2026 18:06
@pavelsavara pavelsavara requested review from a team and wtgodbe as code owners April 27, 2026 18:06
@pavelsavara pavelsavara changed the title test coreCLR WASM [blazor] E2E test coreCLR WASM Apr 27, 2026
Copy link
Copy Markdown
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

Adds initial CI coverage for running Components E2E tests against the CoreCLR WASM runtime (instead of Mono), while excluding Mono-only threading test assets.

Changes:

  • Update the Components E2E test project to exclude ThreadingApp tests and project references when UseMonoRuntime=false.
  • Add a new Azure Pipelines job that builds/runs a (temporarily narrowed) CoreCLR E2E test run, and rename the existing job/results to explicitly indicate Mono.

Reviewed changes

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

File Description
src/Components/test/E2ETest/Microsoft.AspNetCore.Components.E2ETests.csproj Conditionally removes threading tests and ThreadingApp references when building with CoreCLR (UseMonoRuntime=false).
.azure/pipelines/components-e2e-tests.yml Introduces a CoreCLR E2E job and adjusts display/test run titles to distinguish CoreCLR vs Mono runs.

displayName: NPM install
- script: npm run build
displayName: Build JS
- script: .dotnet/dotnet build ./src/Components/test/E2ETest -c $(BuildConfiguration) /p:UseMonoRuntime=false
# TEMPORARY: Run only StatePersistenceTest to debug CoreCLR WASM failures without exhausting disk.
.dotnet/dotnet test ./src/Components/test/E2ETest \
-c $(BuildConfiguration) \
--no-build \
.dotnet/dotnet test ./src/Components/test/E2ETest \
-c $(BuildConfiguration) \
--no-build \
--filter 'Quarantined!=true&FullyQualifiedName~StatePersistenceTest' \
Comment on lines 49 to +53
- template: /eng/common/templates/variables/pool-providers.yml

jobs:
# CoreCLR runtime E2E tests (UseMonoRuntime=false) — runs first for faster feedback
# TEMPORARY: narrowed to StatePersistenceTest only to debug disk-space issues
@wtgodbe
Copy link
Copy Markdown
Member

wtgodbe commented Apr 27, 2026

@pavelsavara is this ready for review?

- script: |
set -eo pipefail

# TEMPORARY: Run only StatePersistenceTest to debug CoreCLR WASM failures without exhausting disk.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Is this temporary for the PR, or to be merged?

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants