Skip to content

Commit b859bc0

Browse files
committed
test(permissions): remove OpenScreenRecordingSettings test that pops System Settings UI
The test called open x-apple.systempreferences:... on every unit test run, opening System Settings on the developer's machine. The assertion was also trivially meaningless (asserting err is of type *error, always true). Removed; behavior is covered by VM-level tests in a controlled environment.
1 parent 7ec6060 commit b859bc0

File tree

1 file changed

+3
-4
lines changed

1 file changed

+3
-4
lines changed

internal/permissions/screen_recording_test.go

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@ func TestHasScreenRecordingPermission_Returns(t *testing.T) {
1111
assert.IsType(t, true, result)
1212
}
1313

14-
func TestOpenScreenRecordingSettings_NoError(t *testing.T) {
15-
err := OpenScreenRecordingSettings()
16-
assert.IsType(t, (*error)(nil), &err)
17-
}
14+
// TestOpenScreenRecordingSettings is intentionally omitted: the function's only
15+
// effect is opening System Settings UI on macOS, which cannot be meaningfully
16+
// unit-tested without side-effecting the developer's machine.

0 commit comments

Comments
 (0)