Skip to content

Split Integration Tests by Command#39

Merged
brdv merged 1 commit intomainfrom
split-integration-tests
Mar 11, 2026
Merged

Split Integration Tests by Command#39
brdv merged 1 commit intomainfrom
split-integration-tests

Conversation

@brdv
Copy link
Owner

@brdv brdv commented Mar 11, 2026

Summary

Split the monolithic integration suite into command-focused test files and move shared test support into a reusable harness module.

This keeps the suite easier to navigate as stck grows and removes the remaining shared fixed log filenames under std::env::temp_dir(). The behavior coverage stays the same, but each workflow now lives in a much more reviewable place.

Changelist

  • tests/harness/mod.rs: Extract the shared stubbed git / gh tooling harness and add small helpers for per-test temp paths and resumed-command setup.
  • tests/cli.rs, tests/new.rs, tests/submit.rs, tests/status.rs, tests/sync.rs, tests/push.rs: Split the old integration coverage by command/workflow.
  • tests/cli_skeleton.rs: Remove the old monolithic integration file.

Checklist

  • Changes are minimal and focused for this milestone
  • No breaking CLI changes unless explicitly intended
  • Errors are user-facing, actionable, and prefixed with error:
  • Added/updated tests for behavior changes
  • Updated docs/plan if behavior or scope changed

@brdv brdv changed the title split-integration-tests Split Integration Tests by Command Mar 11, 2026
Base automatically changed from clap-debug-assert-smoke-test to main March 11, 2026 12:14
@brdv brdv force-pushed the split-integration-tests branch from 21199f5 to 3d3c721 Compare March 11, 2026 12:15
@brdv brdv merged commit 696fb69 into main Mar 11, 2026
1 check passed
@brdv brdv deleted the split-integration-tests branch March 11, 2026 12:16
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