[devops] Run macOS tests for Mac Catalyst-only builds - #26462
Conversation
Treat Mac Catalyst as a desktop platform when deciding whether to package and run macOS tests. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: 1cb26ff2-5aa5-43fa-946e-01286627b9d9
The Pester tests for configure decisions are bitrotted and are not maintained. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: 1cb26ff2-5aa5-43fa-946e-01286627b9d9
Keep the desktop-platform condition readable in the decision script. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: 1cb26ff2-5aa5-43fa-946e-01286627b9d9
There was a problem hiding this comment.
Pull request overview
Updates the DevOps configuration decision logic so that Mac Catalyst-only builds are treated as “desktop” builds when deciding whether to run packaged macOS tests, preventing the macOS test stage from being skipped in that scenario.
Changes:
- Extend the
RUN_MAC_TESTSdecision to also enable whenINCLUDE_DOTNET_MACCATALYSTis set (in addition toINCLUDE_DOTNET_MACOS).
💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.
| elif [[ "${CONFIGURE_PLATFORMS_INCLUDE_DOTNET_MACOS:-}" != "" ]] || | ||
| [[ "${CONFIGURE_PLATFORMS_INCLUDE_DOTNET_MACCATALYST:-}" != "" ]]; then | ||
| # Run mac tests if a .NET desktop platform is enabled | ||
| RUN_MAC_TESTS=true |
✅ 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 #0518f3a] Test results 🚀Test results✅ All tests passed on VSTS: test results. 🎉 All 205 tests passed 🎉 Tests counts✅ assembly-processing: All 1 tests passed. Html Report (VSDrops) Download macOS tests✅ Tests on macOS Monterey (12): All 5 tests passed. Html Report (VSDrops) Download Linux Build VerificationPipeline on Agent |
Treat Mac Catalyst as a desktop platform when deciding whether to package and run macOS tests. This prevents macOS test stages from being skipped when a branch enables only Mac Catalyst.
🤖 Pull request created by Copilot