Skip to content

[ios-clr] Exclude Reflection.Emit and Dynamic.Runtime tests on Apple mobile CoreCLR simulators#127065

Open
kotlarmilos wants to merge 1 commit intodotnet:mainfrom
kotlarmilos:ios-clr/pr-n-exclude-refemit
Open

[ios-clr] Exclude Reflection.Emit and Dynamic.Runtime tests on Apple mobile CoreCLR simulators#127065
kotlarmilos wants to merge 1 commit intodotnet:mainfrom
kotlarmilos:ios-clr/pr-n-exclude-refemit

Conversation

@kotlarmilos
Copy link
Copy Markdown
Member

@kotlarmilos kotlarmilos commented Apr 17, 2026 •
edited
Loading

Description

Exclude three test projects that fail on Apple mobile CoreCLR simulators from the library test run. Tracking issue: #124344.

@kotlarmilos
[ios-clr] Exclude Reflection.Emit and Dynamic.Runtime tests on Apple …
c4e025a 
…mobile CoreCLR

Reflection.Emit, Reflection.Emit.ILGeneration, and Dynamic.Runtime test
projects currently fail on Apple mobile CoreCLR. Exclude them from the
simulator CoreCLR test run pending investigation (dotnet#124344).

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings April 17, 2026 09:15
Copilot AI reviewed Apr 17, 2026
View reviewed changes
Copy link
Copy Markdown
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

Excludes failing Reflection.Emit and Dynamic.Runtime test projects from the Apple mobile CoreCLR simulator library test run, as part of splitting the larger Apple-mobile test-fix effort.

Changes:

  • Add System.Reflection.Emit.Tests and System.Reflection.Emit.ILGeneration.Tests to Apple-mobile CoreCLR simulator ProjectExclusions.
  • Add System.Dynamic.Runtime.Tests to the same exclusion list.

Comment thread src/libraries/tests.proj
@kotlarmilos kotlarmilos added the os-ios Apple iOS label Apr 17, 2026
@kotlarmilos kotlarmilos added this to the 11.0.0 milestone Apr 17, 2026
@dotnet-policy-service
Copy link
Copy Markdown
Contributor

dotnet-policy-service bot commented Apr 17, 2026

Tagging subscribers to 'os-ios': @vitek-karas, @kotlarmilos, @steveisok, @akoeplinger
See info in area-owners.md if you want to be subscribed.

@kotlarmilos kotlarmilos changed the title [ios-clr] Exclude Reflection.Emit and Dynamic.Runtime tests on Apple mobile CoreCLR [ios-clr] Exclude Reflection.Emit and Dynamic.Runtime tests on Apple mobile CoreCLR simulators Apr 17, 2026
@vitek-karas
Copy link
Copy Markdown
Member

vitek-karas commented Apr 17, 2026

I would add a line to the description of #124344 - that it now tracks these specific tests as well - so that it's easier to see what needs to be done in that issue.

@kotlarmilos
Copy link
Copy Markdown
Member Author

kotlarmilos commented Apr 17, 2026

I would add a line to the description of #124344 - that it now tracks these specific tests as well - so that it's easier to see what needs to be done in that issue.

It is already added here:

<!-- https://github.com/dotnet/runtime/issues/124044 -->
  <ItemGroup Condition="('$(TargetOS)' == 'iossimulator' or '$(TargetOS)' == 'tvossimulator' or '$(TargetOS)' == 'maccatalyst') and '$(RuntimeFlavor)' == 'CoreCLR' and '$(UseNativeAOTRuntime)' != 'true'">
...

@kotlarmilos
Copy link
Copy Markdown
Member Author

kotlarmilos commented Apr 17, 2026

/azp run runtime-ioslikesimulator,runtime-maccataylst

@azure-pipelines
Copy link
Copy Markdown

azure-pipelines bot commented Apr 17, 2026

Azure Pipelines successfully started running 1 pipeline(s).

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

Reviewers

Copilot code review Copilot Copilot left review comments

@jkotas jkotas jkotas approved these changes

@vitek-karas vitek-karas Awaiting requested review from vitek-karas

@matouskozak matouskozak Awaiting requested review from matouskozak

Assignees

@kotlarmilos kotlarmilos

Labels

area-Infrastructure-coreclr os-ios Apple iOS

Projects

Runtime Infra
Status: No status

Milestone

11.0.0

Development

Successfully merging this pull request may close these issues.

4 participants

@kotlarmilos @vitek-karas @jkotas