-
Notifications
You must be signed in to change notification settings - Fork 5.3k
[clr-ios] Run full test suite on Apple mobile platforms #123033
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Changes from all commits
4344f90
17008b7
0d941eb
352054f
05157ad
6bfcef1
2144772
f8bc27d
903f537
f725ac8
126d31d
1b6620e
fd7161e
a74456e
cf08f82
c87844b
0e4bccc
26356ad
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -6,6 +6,8 @@ | |
| and '$(UseNativeAOTRuntime)' != 'true' | ||
| and '$(TargetOS)' != 'browser' | ||
| and '$(TargetOS)' != 'wasi' | ||
| and '$(TargetOS)' != 'android' | ||
| and '$(TargetsAppleMobile)' != 'true' | ||
| and '$(RuntimeFlavor)' != 'Mono' | ||
| and '$(UseRuntimeAsync)' != 'false'"> | ||
|
Comment on lines
8
to
12
|
||
| <EnablePreviewFeatures>true</EnablePreviewFeatures> | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -702,7 +702,14 @@ | |
| </ItemGroup> | ||
|
|
||
| <ItemGroup Condition="'$(ArchiveTests)' == 'true' and '$(TargetOS)' == 'ios' and '$(RunAOTCompilation)' == 'true'"> | ||
| <!-- Only System.Runtime tests on iOS for now --> | ||
| <ProjectReference Include="$(MSBuildThisFileDirectory)System.Runtime\tests\System.Runtime.Tests\System.Runtime.Tests.csproj" /> | ||
|
|
||
| <ProjectReference Include="$(RepoRoot)\src\tests\FunctionalTests\iOS\Device\**\*.Test.csproj" | ||
| Exclude="@(ProjectExclusions)" | ||
| BuildInParallel="false" /> | ||
| </ItemGroup> | ||
|
|
||
| <ItemGroup Condition="'$(ArchiveTests)' == 'true' and '$(TargetOS)' == 'ios' and '$(RuntimeFlavor)' == 'CoreCLR' and '$(UseNativeAOTRuntime)' != 'true'"> | ||
| <ProjectReference Include="$(MSBuildThisFileDirectory)System.Runtime\tests\System.Runtime.Tests\System.Runtime.Tests.csproj" /> | ||
|
Comment on lines
704
to
713
|
||
|
|
||
| <ProjectReference Include="$(RepoRoot)\src\tests\FunctionalTests\iOS\Device\**\*.Test.csproj" | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In this MacCatalyst CoreCLR libraries job, the buildArgs no longer pass /p:BuildTestsOnHelix=true. For mobile targets (maccatalyst is TargetsMobile=true), ArchiveTests only runs when BuildTestsOnHelix=true (see eng/testing/tests.targets:92-95), so this job is likely to skip producing the test archives needed by the subsequent helix.yml step. Add /p:BuildTestsOnHelix=true to the buildArgs (or otherwise ensure BuildTestsOnHelix is set during the build) so archiving and Helix submission work consistently.