Uh oh!
There was an error while loading. Please reload this page.
[ios-clr] Classify System.ComponentModel.Composition.Tests as not supported on Apple mobile - #127057
Conversation
There was a problem hiding this comment.
Pull request overview
Enables System.ComponentModel.Composition (MEF) library tests to run on Apple mobile CoreCLR configurations that use aggressive trimming by rooting required assemblies via an ILLink descriptor, and by removing the prior whole-project exclusion while keeping a small set of known-failing tests disabled via ActiveIssue.
Changes:
- Add an
ILLink.Descriptors.xmland wire it into the MEF test project for Apple mobile + aggressive trimming (non-NativeAOT). - Remove the MEF test project exclusion from
src/libraries/tests.proj. - Add
ActiveIssueattributes to keep a handful of tests disabled on iOS/tvOS/MacCatalyst.
Reviewed changes
Copilot reviewed 7 out of 7 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| src/libraries/tests.proj | Removes the MEF test project exclusion so it can run on Apple mobile. |
| src/libraries/System.ComponentModel.Composition/tests/System.ComponentModel.Composition.Tests.csproj | Adds conditional TrimmerRootDescriptor inclusion for aggressive-trimming Apple mobile builds. |
| src/libraries/System.ComponentModel.Composition/tests/ILLink.Descriptors.xml | New ILLink descriptor intended to preserve MEF-related assemblies under trimming. |
| src/libraries/System.ComponentModel.Composition/tests/System/ComponentModel/Composition/Hosting/AssemblyCatalogTests.cs | Disables a non-statically-referenced assembly-loading test on Apple mobile via ActiveIssue. |
| src/libraries/System.ComponentModel.Composition/tests/System/ComponentModel/Composition/Hosting/DirectoryCatalogTests.cs | Disables a non-statically-referenced assembly-loading test on Apple mobile via ActiveIssue. |
| src/libraries/System.ComponentModel.Composition/tests/System/ComponentModel/Composition/Hosting/DirectoryCatalogDebuggerProxyTests.cs | Disables a test on Apple mobile via ActiveIssue. |
| src/libraries/System.ComponentModel.Composition/tests/System/ComponentModel/Composition/ReflectionModel/ReflectionMemberExportDefinitionTests.cs | Disables two tests on Apple mobile via ActiveIssue. |
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
…ported on Apple mobile MEF (System.ComponentModel.Composition) targets plugin/extension discovery from loose assemblies on disk, which is not a meaningful scenario on Apple mobile (no dynamic assembly delivery, App Store restrictions). The test project has been disabled on Mono everywhere since mono/mono#16417 and on Browser; treat Apple mobile CoreCLR the same way. Move the existing exclusion out of the dotnet#124344 block into the "Not supported on Apple mobile" group to reflect the policy. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
d8910d2 to
71eddd0CompareTagging subscribers to 'os-ios': @vitek-karas, @kotlarmilos, @steveisok, @akoeplinger |
Description
Move
System.ComponentModel.Composition.Testsintests.projfrom the#124344group toNot supported on Apple mobile, matching mono and browser. Additional trimming changes are needed and MEF extension scenarios aren't different on Apple mobile compared to desktop.