[tests] Add an AppSizeTest variation for CoreCLR, both with and without R2R.#24390
[tests] Add an AppSizeTest variation for CoreCLR, both with and without R2R.#24390rolfbjarne merged 1 commit intonet11.0from
Conversation
This comment was marked as outdated.
This comment was marked as outdated.
There was a problem hiding this comment.
Pull request overview
This PR adds CoreCLR runtime test variations to the AppSizeTest suite for iOS, alongside updates to existing test configurations to be more explicit about runtime selection.
Key changes:
- Adds two new test methods for CoreCLR:
CoreCLR_Interpreter(enabled) andCoreCLR_R2R(ignored, pending implementation) - Updates existing runtime tests to explicitly set the
UseMonoRuntimeMSBuild property for clarity - Adds baseline expected output files for CoreCLR-Interpreter configuration showing significantly larger app size (~221 MB vs ~3.6 MB for NativeAOT)
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| tests/dotnet/UnitTests/AppSizeTest.cs | Adds CoreCLR test methods and makes runtime selection explicit in all test configurations |
| tests/dotnet/UnitTests/expected/iOS-CoreCLR-Interpreter-size.txt | Baseline app size and file listing for CoreCLR interpreter mode |
| tests/dotnet/UnitTests/expected/iOS-NativeAOT-size.txt | Updated baseline reflecting reduced NativeAOT app size |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
e761f40 to
575c716
Compare
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
✅ [CI Build #575c716] Build passed (Build packages) ✅Pipeline on Agent |
✅ [PR Build #575c716] Build passed (Detect API changes) ✅Pipeline on Agent |
✅ [CI Build #575c716] Build passed (Build macOS tests) ✅Pipeline on Agent |
✅ API diff for current PR / commitNET (empty diffs)✅ API diff vs stableNET (empty diffs)ℹ️ Generator diffGenerator Diff: vsdrops (html) vsdrops (raw diff) gist (raw diff) - Please review changes) Pipeline on Agent |
💻 [CI Build #575c716] Tests on macOS X64 - Mac Sonoma (14) passed 💻✅ All tests on macOS X64 - Mac Sonoma (14) passed. Pipeline on Agent |
💻 [CI Build #575c716] Tests on macOS arm64 - Mac Sequoia (15) passed 💻✅ All tests on macOS arm64 - Mac Sequoia (15) passed. Pipeline on Agent |
💻 [CI Build #575c716] Tests on macOS M1 - Mac Ventura (13) passed 💻✅ All tests on macOS M1 - Mac Ventura (13) passed. Pipeline on Agent |
💻 [CI Build #575c716] Tests on macOS M1 - Mac Monterey (12) passed 💻✅ All tests on macOS M1 - Mac Monterey (12) passed. Pipeline on Agent |
This comment has been minimized.
This comment has been minimized.
💻 [CI Build #575c716] Tests on macOS arm64 - Mac Tahoe (26) passed 💻✅ All tests on macOS arm64 - Mac Tahoe (26) passed. Pipeline on Agent |
🚀 [CI Build #575c716] Test results 🚀Test results✅ All tests passed on VSTS: test results. 🎉 All 133 tests passed 🎉 Tests counts✅ cecil: All 1 tests passed. Html Report (VSDrops) Download Pipeline on Agent |
No description provided.