← Back to issue list

tests: remove regression test for command help

View original Github issue

Metadata

Project
charmcraft
Number
#1591
Type
pull request
State
merged
Author
lengau
Labels
—
Created
Updated
Closed

Current evaluation

Merged PR #1591 as commit 4501ea01, dropping the 93-line command-help regression test since the craft-application base is now stable.

Resolution: closed resolved

Reason: The PR was approved by two reviewers and merged into main as commit 4501ea01, deleting the redundant help regression test (+0/-93 lines). No code fix was needed: the test existed only to guard an upstream craft-application issue that is now resolved.

Evaluated by: mr-cal-bot

No scores available.

Issue body

This is no longer necessary since the craft-application base is stable.

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged PR #1591 as commit 4501ea01, dropping the 93-line command-help regression test since the craft-application base is now stable.
stealth/space-bunny-alpha — closed resolved Merged PR #1591 as commit 4501ea01, removing the 93-line command-help regression test because the craft-application base is now stable.
qwen/qwen3.6-35b-a3b — — Merged removal of a regression test for command help, as the craft-application base is now stable. Approved by two reviewers, the change deletes 93 lines across one file.
qwen3.6-35b-a3b-mtp-q6 — — Merged removal of a command help regression test. The maintainer noted the test is obsolete now that the craft-application base is stable.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#5375 test: fix command tests snapcraft merged Test-only PR by maintainer mr-cal merged as commit 28cb1b828 ("test: fix command tests (#5375)"), updating fixtures for failing command unit tests (CRAFT-4414); +17/-9 across 2 files, approved by 2 reviewers.
73%
#1571 refactor: remove unused clean command implementation charmcraft merged Merged PR #1571 as commit 053d5a2d, removing the obsolete Charmcraft clean command and regression tests after craft-application took over clean handling.
71%