Skip to content

Keep Mac unit tests away from mounted drives - #29

Merged
mikecerisano merged 1 commit into
mainfrom
test/no-permission-prompts-20261017
Oct 17, 2026
Merged

mikecerisano merged 1 commit into
mainfrom
test/no-permission-prompts-20261017

Conversation

@mikecerisano

@mikecerisano mikecerisano commented Oct 17, 2026 •

Copy link
Copy Markdown
Collaborator

Mac app unit tests launch a BitMatch app host before their fixtures run. That host was starting live volume discovery and loading saved app state, which could touch mounted media and contribute to access prompts or stalled test startup.

In DEBUG XCTest hosts, startup now uses isolated in-memory state, no SFTP scheduler, and no live /Volumes scan or Disk Arbitration registration. The Release path and entitlements are unchanged. Tests assert the host detection and that the singleton never starts drive monitoring in the unit-test process.

Local validation: focused Mac host tests 4/4 passed; drive-access policy tests 3/3 passed. After this was combined with the other draft slices, the full Mac suite passed (267 XCTest cases, 12 intentional skips, plus 888 Swift Testing cases) and Mac/iOS Release builds passed. An earlier zero-case filter run was discarded. This does not prove every UI or signed-app test is prompt-free.

GitHub Actions started automatically when this draft was pushed. Observed Mac jobs stop before tests during an Xcode 26.3 compiler IR-generation crash in PreferencesWindow.swift; engine jobs stop before tests on Swift concurrency compile errors in ChecksumTruncationTests.swift and StressTests.swift. The same compile failures appear on the existing #28 CI branch. Evidence-corpus jobs pass. No manual CI rerun was requested; the local results above are the review evidence for this draft.

@mikecerisano
mikecerisano marked this pull request as ready for review October 17, 2026 06:38
@mikecerisano
mikecerisano merged commit dec90ab into main Oct 17, 2026
4 of 8 checks passed
@mikecerisano
mikecerisano deleted the test/no-permission-prompts-20261017 branch October 17, 2026 06:38
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.

1 participant