Skip to content

Disable System.Net.Http.Functional.Tests on Android CoreCLR#125171

Merged
matouskozak merged 1 commit intodotnet:mainfrom
matouskozak:disable-system-net-tests
Mar 4, 2026
Merged

Disable System.Net.Http.Functional.Tests on Android CoreCLR#125171
matouskozak merged 1 commit intodotnet:mainfrom
matouskozak:disable-system-net-tests

Conversation

@matouskozak
Copy link
Member

@matouskozak matouskozak commented Mar 4, 2026

For context: #124526

Disabling the test suit until full investigation is done.

@matouskozak matouskozak self-assigned this Mar 4, 2026
Copilot AI review requested due to automatic review settings March 4, 2026 09:17
@dotnet-policy-service
Copy link
Contributor

Tagging subscribers to this area: @karelz, @dotnet/ncl
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

Disables the System.Net.Http functional test assembly on Android when running under CoreCLR by adding an assembly-level ActiveIssue gate tied to #124526.

Changes:

  • Add [assembly: ActiveIssue(..., TestPlatforms.Android, ..., TestRuntimes.CoreCLR)] to skip System.Net.Http.Functional.Tests on Android CoreCLR.

You can also share your feedback on Copilot code review. Take the survey.

@matouskozak matouskozak marked this pull request as ready for review March 4, 2026 11:24
@matouskozak matouskozak requested review from EgorBo and wfurt March 4, 2026 11:25
Copy link
Member

@wfurt wfurt left a comment

Choose a reason for hiding this comment

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

LGTM.

@matouskozak matouskozak enabled auto-merge (squash) March 4, 2026 13:04
@matouskozak matouskozak merged commit 39fda2c into dotnet:main Mar 4, 2026
102 checks passed
@matouskozak matouskozak deleted the disable-system-net-tests branch March 4, 2026 14:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants