Skip to content

chore: make capture tests portable - #130

Merged
abrichr merged 1 commit into
mainfrom
codex/capture-postmerge-windows
Aug 31, 2026
Merged

chore: make capture tests portable#130
abrichr merged 1 commit into
mainfrom
codex/capture-postmerge-windows

Conversation

@abrichr

@abrichr abrichr commented Aug 31, 2026

Copy link
Copy Markdown
Member

Why

The larger Windows test matrix on main found two test assumptions after #126 merged. POSIX mode bits do not have the same meaning on Windows. A live monotonic-clock calculation can also cross an exact floating-point boundary. The product behavior passed; only these assertions failed in the main build.

What changed

  • Keep the empty authentication-manifest content check on every platform. Check owner-only mode bits on POSIX.
  • Give the macOS accessibility-timeout test a fixed monotonic clock.

This leaves capture and authentication behavior unchanged.

Verification

  • pytest for both affected tests: 2 passed
  • ruff check --no-cache for both affected files
  • git diff --check

@abrichr
abrichr merged commit e514cee into main Aug 31, 2026
12 checks passed
@abrichr
abrichr deleted the codex/capture-postmerge-windows branch August 31, 2026 15:30
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