Skip to content

chore: move codex-exec unit tests into sibling files#16581

Merged
bolinfest merged 1 commit intomainfrom
pr16581
Apr 2, 2026
Merged

chore: move codex-exec unit tests into sibling files#16581
bolinfest merged 1 commit intomainfrom
pr16581

Conversation

@bolinfest
Copy link
Copy Markdown
Collaborator

@bolinfest bolinfest commented Apr 2, 2026

Why

codex-rs/exec/src/lib.rs already keeps unit tests in a sibling lib_tests.rs module so the implementation stays top-heavy and easier to read. This applies that same layout to the rest of codex-rs/exec/src so each production file keeps its entry points and helpers ahead of test code.

What

  • Move inline unit tests out of cli.rs, main.rs, event_processor_with_human_output.rs, and event_processor_with_jsonl_output.rs into sibling *_tests.rs files.
  • Keep test modules wired through #[cfg(test)] plus #[path = "..."] mod tests;, matching the lib.rs pattern.
  • Preserve the existing test coverage and assertions while making this a source-layout-only refactor.

Verification

  • cargo test -p codex-exec

@bolinfest bolinfest changed the title chore: move unit tests in codex-exec into their own files chore: move codex-exec unit tests into sibling files Apr 2, 2026
@bolinfest bolinfest merged commit 95b0b5a into main Apr 2, 2026
22 checks passed
@bolinfest bolinfest deleted the pr16581 branch April 2, 2026 17:01
@github-actions github-actions bot locked and limited conversation to collaborators Apr 2, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant