Skip to content

[tests] Disable the AppSizeTest on branches using an Xcode beta.#26037

Merged
rolfbjarne merged 1 commit into
mainfrom
dev/rolf/disable-appsizetest-on-beta-branches
Jul 10, 2026
Merged

[tests] Disable the AppSizeTest on branches using an Xcode beta.#26037
rolfbjarne merged 1 commit into
mainfrom
dev/rolf/disable-appsizetest-on-beta-branches

Conversation

@rolfbjarne

Copy link
Copy Markdown
Member

On branches that use beta versions of Xcode, we typically do a lot of binding
work, and that mean that frequent updates to the known failures for this test
are necessary. So frequent that it becomes frustrating, and also slows down
pace, because there will be a lot of merge conflicts between PRs that all
update the known failures for this test.

So just disable it when using a beta version of Xcode. Once the branch
switches to a stable version of Xcode, there will be much less binding work,
and we can resume this test.

On branches that use beta versions of Xcode, we typically do a lot of binding
work, and that mean that frequent updates to the known failures for this test
are necessary. So frequent that it becomes frustrating, and also slows down
pace, because there will be a lot of merge conflicts between PRs that all
update the known failures for this test.

So just disable it when using a beta version of Xcode. Once the branch
switches to a stable version of Xcode, there will be much less binding work,
and we can resume this test.
Copilot AI review requested due to automatic review settings July 10, 2026 10:42

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR disables AppSizeTest when the test suite is running against a non-stable (beta/preview) Xcode, to avoid frequent churn and merge conflicts in the app size “known failures/expected” data during periods of heavy binding work.

Changes:

  • Add an early Assert.Ignore gate in the shared Run(...) helper when Configuration.XcodeIsStable is false.
  • Ensure the ignore message clearly documents why the test is being skipped under beta Xcode.

@rolfbjarne rolfbjarne enabled auto-merge (squash) July 10, 2026 11:38
@vs-mobiletools-engineering-service2

Copy link
Copy Markdown
Collaborator

✅ API diff for current PR / commit

NET (empty diffs)

✅ API diff vs stable

NET (empty diffs)

ℹ️ Generator diff

Generator Diff: vsdrops (html) vsdrops (raw diff) gist (raw diff) - Please review changes)

Pipeline on Agent
Hash: a16bb43c6c8aafe2f84ba4c3388d23311e61ad1d [PR build]

@vs-mobiletools-engineering-service2

Copy link
Copy Markdown
Collaborator

🚀 [CI Build #a16bb43] Test results 🚀

Test results

✅ All tests passed on VSTS: test results.

🎉 All 199 tests passed 🎉

Tests counts

✅ assembly-processing: All 1 tests passed. Html Report (VSDrops) Download
✅ cecil: All 1 tests passed. Html Report (VSDrops) Download
✅ dotnettests (iOS): All 1 tests passed. Html Report (VSDrops) Download
✅ dotnettests (MacCatalyst): All 1 tests passed. Html Report (VSDrops) Download
✅ dotnettests (macOS): All 1 tests passed. Html Report (VSDrops) Download
✅ dotnettests (Multiple platforms): All 1 tests passed. Html Report (VSDrops) Download
✅ dotnettests (tvOS): All 1 tests passed. Html Report (VSDrops) Download
✅ framework: All 2 tests passed. Html Report (VSDrops) Download
✅ fsharp: All 4 tests passed. Html Report (VSDrops) Download
✅ generator: All 5 tests passed. Html Report (VSDrops) Download
✅ interdependent-binding-projects: All 4 tests passed. Html Report (VSDrops) Download
✅ introspection: All 4 tests passed. Html Report (VSDrops) Download
✅ linker (iOS): All 15 tests passed. Html Report (VSDrops) Download
✅ linker (MacCatalyst): All 15 tests passed. Html Report (VSDrops) Download
✅ linker (macOS): All 21 tests passed. Html Report (VSDrops) Download
✅ linker (tvOS): All 15 tests passed. Html Report (VSDrops) Download
✅ monotouch (iOS): All 18 tests passed. Html Report (VSDrops) Download
✅ monotouch (MacCatalyst): All 17 tests passed. Html Report (VSDrops) Download
✅ monotouch (macOS): All 18 tests passed. Html Report (VSDrops) Download
✅ monotouch (tvOS): All 18 tests passed. Html Report (VSDrops) Download
✅ msbuild: All 2 tests passed. Html Report (VSDrops) Download
✅ sharpie: All 1 tests passed. Html Report (VSDrops) Download
✅ windows: All 3 tests passed. Html Report (VSDrops) Download
✅ xcframework: All 4 tests passed. Html Report (VSDrops) Download
✅ xtro: All 1 tests passed. Html Report (VSDrops) Download

macOS tests

✅ Tests on macOS Monterey (12): All 5 tests passed. Html Report (VSDrops) Download
✅ Tests on macOS Ventura (13): All 5 tests passed. Html Report (VSDrops) Download
✅ Tests on macOS Sonoma (14): All 5 tests passed. Html Report (VSDrops) Download
✅ Tests on macOS Sequoia (15): All 5 tests passed. Html Report (VSDrops) Download
✅ Tests on macOS Tahoe (26): All 5 tests passed. Html Report (VSDrops) Download

Linux Build Verification

Linux build succeeded

Pipeline on Agent
Hash: a16bb43c6c8aafe2f84ba4c3388d23311e61ad1d [PR build]

@rolfbjarne rolfbjarne merged commit cd5a32f into main Jul 10, 2026
57 checks passed
@rolfbjarne rolfbjarne deleted the dev/rolf/disable-appsizetest-on-beta-branches branch July 10, 2026 14:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants