Skip to content

Disable metadata generic attribute decoder test on MacCatalyst#130009

Merged
MichalStrehovsky merged 2 commits into
mainfrom
copilot/fix-custom-attribute-decoder-test
Jun 30, 2026
Merged

Disable metadata generic attribute decoder test on MacCatalyst#130009
MichalStrehovsky merged 2 commits into
mainfrom
copilot/fix-custom-attribute-decoder-test

Conversation

Copilot AI commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

TestCustomAttributeDecoderGenericUsingReflection is failing on MacCatalyst CoreCLR due to mobile-specific reflection enum type resolution behavior.

  • Change
    • Extends the existing ActiveIssue platform skip for this test from iOS/tvOS to include MacCatalyst.
[ActiveIssue("https://github.com/dotnet/runtime/issues/60579",
    TestPlatforms.iOS | TestPlatforms.tvOS | TestPlatforms.MacCatalyst)]

@dotnet-policy-service

Copy link
Copy Markdown
Contributor

Tagging subscribers to this area: @dotnet/area-system-reflection-metadata
See info in area-owners.md if you want to be subscribed.

Co-authored-by: MichalStrehovsky <13110571+MichalStrehovsky@users.noreply.github.com>
Copilot AI requested review from Copilot and removed request for Copilot June 30, 2026 00:47
Copilot AI changed the title [WIP] Fix CustomAttributeDecoderTests on maccatalyst Disable metadata generic attribute decoder test on MacCatalyst Jun 30, 2026
Copilot AI requested a review from MichalStrehovsky June 30, 2026 00:48
@jkotas jkotas marked this pull request as ready for review June 30, 2026 00:50
Copilot AI review requested due to automatic review settings June 30, 2026 00:50

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.

Pull request overview

This PR updates the System.Reflection.Metadata decoding test suite to skip TestCustomAttributeDecoderGenericUsingReflection on MacCatalyst, aligning the platform skip behavior with existing Apple mobile exclusions.

Changes:

  • Extend the existing ActiveIssue skip for issue #60579 to include TestPlatforms.MacCatalyst for TestCustomAttributeDecoderGenericUsingReflection.

@MichalStrehovsky

Copy link
Copy Markdown
Member

Uh, looks like marking the PR as ready for review is also considered a "Push" somehow?

image

@dotnet/ilc-contrib could someone else approve? Neither me nor Jan can approve :(

@MichalStrehovsky MichalStrehovsky merged commit df35baa into main Jun 30, 2026
89 of 92 checks passed
@MichalStrehovsky MichalStrehovsky deleted the copilot/fix-custom-attribute-decoder-test branch June 30, 2026 07: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.

5 participants