Skip to content

Skip flaky BlazorWebView header interception test#33928

Merged
rmarinho merged 1 commit intomainfrom
fix/skip-flaky-blazorwebview-header-test
Feb 9, 2026
Merged

Skip flaky BlazorWebView header interception test#33928
rmarinho merged 1 commit intomainfrom
fix/skip-flaky-blazorwebview-header-test

Conversation

@rmarinho
Copy link
Member

@rmarinho rmarinho commented Feb 6, 2026

Note

Are you waiting for the changes in this PR to be merged?
It would be very helpful if you could test the resulting artifacts from this PR and let us know in a comment if this change resolves your issue. Thank you!

Description

Skips the flaky RequestsCanBeInterceptedAndHeadersAddedForDifferentHosts BlazorWebView device test which depends on an external echo service (echo.free.beeceptor.com), making it unreliable in CI.

The test failed on CoreCLR MacCatalyst with Assert.NotNull() Failure: Value is null while passing on Mono MacCatalyst in the same build:

Issues Fixed

Related #33927

Copilot AI review requested due to automatic review settings February 6, 2026 15:36
@rmarinho rmarinho added the area-blazor Blazor Hybrid / Desktop, BlazorWebView label Feb 6, 2026
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

Skips a flaky BlazorWebView device test that relies on an external echo service, improving CI reliability.

Changes:

  • Marked RequestsCanBeInterceptedAndHeadersAddedForDifferentHosts as skipped via xUnit Theory(Skip=...).
  • Documented the skip reason and linked the tracking issue in the skip message.

@rmarinho
Copy link
Member Author

rmarinho commented Feb 6, 2026

/backport to net10.0

@github-actions
Copy link
Contributor

github-actions bot commented Feb 6, 2026

Started backporting to net10.0 (link to workflow run)

@rmarinho
Copy link
Member Author

rmarinho commented Feb 6, 2026

/backport to net11.0

@github-actions
Copy link
Contributor

github-actions bot commented Feb 6, 2026

Started backporting to net11.0 (link to workflow run)

@rmarinho
Copy link
Member Author

rmarinho commented Feb 6, 2026

/azp run maui-pr-devicetests

rmarinho added a commit that referenced this pull request Feb 6, 2026
Backport of #33928 to net11.0

/cc @rmarinho

Co-authored-by: Rui Marinho <me@ruimarinho.net>
rmarinho added a commit that referenced this pull request Feb 6, 2026
Backport of #33928 to net10.0

/cc @rmarinho

Co-authored-by: Rui Marinho <me@ruimarinho.net>
@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@rmarinho
Copy link
Member Author

rmarinho commented Feb 9, 2026

/rebase

The RequestsCanBeInterceptedAndHeadersAddedForDifferentHosts test
depends on an external echo service (echo.free.beeceptor.com) which
makes it unreliable in CI. It failed on CoreCLR MacCatalyst while
passing on Mono MacCatalyst.

Fixes #33927
@github-actions github-actions bot force-pushed the fix/skip-flaky-blazorwebview-header-test branch from d831bf1 to db42321 Compare February 9, 2026 08:25
@dotnet dotnet deleted a comment from azure-pipelines bot Feb 9, 2026
@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@rmarinho rmarinho merged commit f2c4612 into main Feb 9, 2026
33 of 42 checks passed
@rmarinho rmarinho deleted the fix/skip-flaky-blazorwebview-header-test branch February 9, 2026 10:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-blazor Blazor Hybrid / Desktop, BlazorWebView area-testing Unit tests, device tests testing-flakiness testing-green-ci

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants