Expand test coverage and add cross-platform CI matrix - #3
Merged
Conversation
added 2 commits
April 23, 2026 14:23
Adds unit, integration, and end-to-end tests, and rewrites the CI workflow so the full test suite runs on ubuntu-latest, macos-latest, and windows-latest with cargo caching. - End-to-end CLI workflow test covering add/list/copy/pop/pin/unpin against a shared queue file - Pin/unpin error-path tests (unknown id, short id, idempotent pin) - TUI render tests using ratatui TestBackend - TUI map_key unit tests for keyboard routing across panes - qcli-platform tests for config_path, images_dir, nested app-dir creation, and FakeClipboard overwrite/default behavior - Mutex-guarded env-var access in paths tests to avoid parallel-runner flakes - CI: lint (fmt + clippy) stays single-OS; test job fans out to the three release platforms with per-OS cargo caches
The initial test assumed `copy --next` skips pinned prompts, but in HEAD it returns the first prompt in queue order regardless of pinned state. On CI — which sees only committed code — this caused copy --next to return the pinned "sticky note" when the test expected "first prompt". Rewrite the scenario so copy --next is only invoked when no pinned prompt is at the head. The test still exercises every subcommand (add, list, copy, pop, pin, unpin), pop's skip-pinned behavior, and the stdin/arg add paths — just without assuming copy also skips.
alexanderkreidich
added a commit
that referenced
this pull request
Aug 9, 2026
Expand test coverage and add cross-platform CI matrix
alexanderkreidich
added a commit
that referenced
this pull request
Aug 9, 2026
Expand test coverage and add cross-platform CI matrix
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
What's new
What's not in this PR
Three test files had prior uncommitted content from another agent (`cli_copy.rs`, `cli_pop.rs`, `cli_help.rs`). To avoid stepping on that work, I left those files unstaged — the 7 additional tests I wrote for them stay in the local working tree until they can be reconciled.
Test plan