[Net11]Skip request-interception tests that hit echo.free.beeceptor.com#36020
Merged
PureWeen merged 1 commit intoJun 19, 2026
Conversation
…e BlazorWebView interception tests (CustomDataReturned, Cancelled, and CaseInsensitiveHeadersRead) and one HybridWebView interception test (Cancelled) were running against the https://echo.free.beeceptor.com/ InlineData on the Windows and Android Helix queues, even though the same external service is already known to be flaky and is skipped in other sibling tests (see dotnet#33927 and dotnet#32400). Split the [Theory] attributes so that the iOS/MacCatalyst app:// scheme variants continue to run, while only the https:// variants that depend on the external echo service are skipped.
Contributor
|
🚀 Dogfood this PR with:
curl -fsSL https://raw.githubusercontent.com/dotnet/maui/main/eng/scripts/get-maui-pr.sh | bash -s -- 36020Or
iex "& { $(irm https://raw.githubusercontent.com/dotnet/maui/main/eng/scripts/get-maui-pr.ps1) } 36020" |
BagavathiPerumal
temporarily deployed
to
gh-aw-agents
June 19, 2026 12:45 — with 
GitHub Actions
Inactive
BagavathiPerumal
temporarily deployed
to
gh-aw-agents
June 19, 2026 12:48 — with
GitHub Actions
Inactive
BagavathiPerumal
temporarily deployed
to
gh-aw-agents
June 19, 2026 12:50 — with
GitHub Actions
Inactive
BagavathiPerumal
temporarily deployed
to
gh-aw-agents
June 19, 2026 12:50 — with
GitHub Actions
Inactive
vishnumenon2684
marked this pull request as ready for review
June 19, 2026 12:53
Collaborator
|
/azp run |
kubaflo
approved these changes
Jun 19, 2026
|
Azure Pipelines successfully started running 3 pipeline(s). |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
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 of Change
Several request-interception device tests were running an
[InlineData("https://echo.free.beeceptor.com/...")]row on Windows and Android Helix queues even though that external service is known to be flaky (tracked in #33927). The sibling tests in the same files already skip that scenario; these four were missed:BlazorWebViewTests.RequestsCanBeInterceptedAndCustomDataReturnedForDifferentHostsBlazorWebViewTests.RequestsCanBeInterceptedAndCancelledForDifferentHostsBlazorWebViewTests.RequestsCanBeInterceptedAndCaseInsensitiveHeadersReadHybridWebViewTests_Interception.RequestsCanBeInterceptedAndCancelledForDifferentHostsReflection on the built
Microsoft.Maui.Controls.DeviceTests.dll(net10.0-windows10.0.19041.0) confirmed each of these methods had aMicrosoft.Maui.TheoryAttributewith noSkipand thehttps://echo.free.beeceptor.com/InlineDataattached — they would attempt a real outbound TLS request from the Helix worker whenever interception did not fire.Fix
Split the single
[Theory]into two: one inside the#if !ANDROID && !WINDOWSblock (paired with theapp://echoservice/row) and one inside the#if !IOS && !MACCATALYSTblock carryingSkip = "Flaky due to external service dependency (echo.free.beeceptor.com). See https://github.com/dotnet/maui/issues/33927"(paired with thehttps://row).This matches the convention already used by the three other sibling methods (
CustomDataReturned,HeadersAdded,CaseInsensitiveHeadersReadinHybridWebViewTests_Interception.cs).Importantly, iOS / MacCatalyst coverage is preserved — only the
https://rows are skipped. Theapp://echoservice/rows are fully self-contained (the OS cannot resolve theapp://scheme, so the request never touches the network — it is satisfied entirely by theWebResourceRequestedhandler inside the test).Per-TFM safety check
The custom
Microsoft.Maui.TheoryAttributehasAllowMultiple = false, and the#ifguards make the two[Theory]attributes mutually exclusive across every TFM inMauiDeviceTestsPlatforms:[Theory](app://)[Theory(Skip)](https://)Issues Fixed
Related to #33927.
Note: Created a backport PR for the net11 branch based on the changes introduced in the main PR (#35661).