Skip to content

Disable GC/Coverage/271010 on NativeAOT - #132048

Open
jkoritzinsky with Copilot wants to merge 2 commits into
mainfrom
copilot/fix-nativeaot-gc-test-failure
Open

Disable GC/Coverage/271010 on NativeAOT#132048
jkoritzinsky with Copilot wants to merge 2 commits into
mainfrom
copilot/fix-nativeaot-gc-test-failure

Conversation

Copilot AI commented Aug 9, 2026

Copy link
Copy Markdown
Contributor

GC\Coverage\271010 trips a NativeAOT runtime assertion (ASSERT_UNCONDITIONALLY, EHHelpers.cpp:600) on the windows-x64 Checked NativeAOT_Pri1 - GC leg, reproducing across consecutive builds rather than flaking.

Adds a second [ActiveIssue] on the test entry point, conditioned on TestLibrary.Utilities.IsNativeAot, so the test is skipped under NativeAOT while the underlying assert is investigated. The existing CoreCLR/non-Browser skip is untouched.

[ActiveIssue("https://github.com/dotnet/runtime/issues/5933", typeof(Test_271010), nameof(IsCoreClrOnNonBrowser))]
[ActiveIssue("https://github.com/dotnet/runtime/issues/131565", typeof(TestLibrary.Utilities), nameof(TestLibrary.Utilities.IsNativeAot))]
[Fact]
public static int TestEntryPoint() {

XUnitWrapperGenerator handles the 3-arg (reason, conditionType, conditionMembers) form by negating the condition, and stacked [ActiveIssue] attributes are already used elsewhere (e.g. src/tests/reflection/Modifiers/modifiers.cs).

Note that this only suppresses the symptom. Per @MichalStrehovsky's comment, the likely cause is fallout from dropping RequiresProcessIsolation — another test leaving the process in a bad state — which remains unaddressed here.

Copilot AI lite review requested due to automatic review settings August 9, 2026 04:17

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
16 pipeline(s) were filtered out due to trigger conditions.
There may be pipelines that require an authorized user to comment /azp run to run.

Co-authored-by: jkoritzinsky <1571408+jkoritzinsky@users.noreply.github.com>
Copilot AI review requested due to automatic review settings August 9, 2026 04:21

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.

Copilot AI changed the title [WIP] Fix nativeaot runtime assertion in GC test Disable GC/Coverage/271010 on NativeAOT Aug 9, 2026
Copilot AI requested a review from jkoritzinsky August 9, 2026 04:22
@dotnet-policy-service

Copy link
Copy Markdown
Contributor

Tagging subscribers to this area: @agocke, @dotnet/ilc-contrib
See info in area-owners.md if you want to be subscribed.

@jkoritzinsky
jkoritzinsky marked this pull request as ready for review August 9, 2026 06:20
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 3 pipeline(s).
13 pipeline(s) were filtered out due to trigger conditions.
There may be pipelines that require an authorized user to comment /azp run to run.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

[ci-scan] Test failure: NativeAOT GC leg assert ASSERT_UNCONDITIONALLY in EHHelpers.cpp:600

3 participants