fix: rename TEST_COMMAND_PREFIX
Metadata
Current evaluation
Merged PR #181, renaming the test command prefix to namespace discovered environment variables; all CI checks passed.
Resolution: closed resolved
Reason: canonical/starflow#181 was merged, updating six references in one file. All reported CI checks passed.
Evaluated by: mr-cal-bot
No scores available.
Issue body
Some tests (eg in craft-parts) have expectations on the environment variables that are set on the test, and use the "TEST_" prefix to discover them.
Maybe we should start 'namespacing' these env vars regardless?
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #181, renaming the test command prefix to namespace discovered environment variables; all CI checks passed. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged canonical/starflow#181, renaming TEST_COMMAND_PREFIX to namespace test-discovered environment variables; all CI checks passed. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged after passing all CI checks. Renames TEST_COMMAND_PREFIX to namespace environment variables for test discovery. | |
| qwen/qwen3.6-35b-a3b |
Impact:
30
Quick Win:
27
Staleness:
0
Complexity:
10
Confidence:
85
|
needs review | Renames TEST_COMMAND_PREFIX environment variable to namespace test-related env vars and reduce potential conflicts with external variables. All CI checks passing. |
Update history
| Date | Change |
|---|---|
| closed | |
| created |
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #182 ci(fix): restore TEST_COMMAND_PREFIX behavior | starflow | closed | Merged PR canonical/starflow#182 restored TEST_COMMAND_PREFIX handling to fix CI workflow failures; all reported checks passed. | |
| #1133 tests: fix the test that was modifying the environment variables | snapcraft | merged | Merged PR #1133 (commit c0b0f9752), fixing a test that leaked PATH/environment-variable changes and guarding against an empty PATH. |