feat: add 'test' command
Metadata
Current evaluation
Merged. Added the 'test' CLI command and spread tests for rockcraft. Approved by two reviewers, passed CI checks, and introduced 276 lines across seven files to validate the command using hardcoded fixtures.
Suggested action: —
No scores available.
Issue body
This adds the cli command and a spread test to check 'rockcraft test'. The test uses hardcoded files while we don't have init profiles with test support.
- [ ] Have you followed the guidelines for contributing?
- [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [ ] Have you successfully run `make lint && make test`?
---
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged. Added the 'test' CLI command and spread tests for rockcraft. Approved by two reviewers, passed CI checks, and introduced 276 lines across seven files to validate the command using hardcoded fixtures. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged the 'test' CLI command and associated spread test for rockcraft. The implementation temporarily uses hardcoded files until init profiles support testing. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged addition of the 'test' CLI command and spread test for rockcraft. Implementation uses hardcoded files pending init profile support. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #939 feat: add entrypoint-command | rockcraft | merged | Merged PR adding an entrypoint-command field to rockcraft.yaml to replace the default pebble enter entrypoint. Includes tests and documentation. Approved by reviewers and passed CI checks before merge. | |
| #1660 feat: add test command and templates | charmcraft | merged | Merged addition of the charmcraft test command and templates. The feature invokes spread to run test suites locally and remotely using a multipass backend. Approved by two reviewers and rebased on main before integration. | |
| #5366 feat: add 'test' command | snapcraft | merged | Merged pull request introducing a new 'test' command. Approved by two reviewers, the change modified eight files with 170 additions and was successfully integrated into the repository. | |
| #100 tests: print rockcraft log files on spread failure | rockcraft | merged | Merged: adds 11 lines to print rockcraft log files during spread test failures, improving debugging visibility. Approved by one reviewer and passed CI checks. | |
| #40 feat(rockcraft-pack): support rockcraft test | craft-actions | merged | Merged to integrate rockcraft test into the build workflow. The change handles implicit pack execution to avoid OCI Factory duplication. Approved by two reviewers, passed CI, and merged with 218 additions and 12 deletions. | |
| #93 feat: support "--destructive-mode". | craft-application | merged | Merged. Implements --destructive-mode across all lifecycle commands, fixing issue #57 and a disabled rockcraft spread test. Approved by two reviewers, passed CI with 100% diff coverage, and changed three files. | |
| #105 docs: add commands | rockcraft | merged | Merged documentation adding rockcraft commands, resolving issue #101. After initial review requested changes, the maintainer coordinated with reviewers and merged the first draft. | |
| #173 tests: check before handling rockcraft instances | rockcraft | closed | Merged after two approvals and passing CI. Updates spread test cleanup to conditionally stop and delete rockcraft lxc instances, preventing errors when tests do not create them. (+5/-2 lines) | |
| #400 fix(docs): make sure we can build the example rockcraft.yaml | rockcraft | merged | Merged to resolve issue #399. Added a spread test validating the example rockcraft.yaml build process for documentation. Approved by three reviewers and passed all CI checks. | |
| #174 tests: check before handling rockcraft instances | rockcraft | merged | Merged a fix wrapping rockcraft LXC instance cleanup in an if condition to prevent errors when spread tests do not create instances. Approved by two reviewers, passed CI, and modified one file. |