fix(tests): fixes bundle spread test
Metadata
Current evaluation
Merged. Resolves issue #1262 and CRAFT-2052 by fixing the bundle spread test. Replaced a failing charm path check with Python-based YAML validation. Approved by two reviewers, passed CI, and successfully executed the large spread test.
Suggested action: —
No scores available.
Issue body
Use python to check yaml.
Remove charm path check that does not work.
Fixes: https://github.com/canonical/charmcraft/issues/1262
CRAFT-2052
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged. Resolves issue #1262 and CRAFT-2052 by fixing the bundle spread test. Replaced a failing charm path check with Python-based YAML validation. Approved by two reviewers, passed CI, and successfully executed the large spread test. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged fix for bundle spread test. Replaced YAML validation with Python and removed a broken charm path check. Resolves issue #1262 and CRAFT-2052 following successful spread test execution. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Resolved bundle spread test failure by replacing the broken charm path check with Python YAML validation. Addresses issue #1262 and CRAFT-2052. Large spread tests executed successfully. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1324 test(spread): test-deploy charms in large spread tests | charmcraft | merged | Merged to add test deployment for charms and bundles in large spread tests. Approved by two reviewers, modified six files with 85 additions and 26 deletions, and linked to a passing CI run. | |
| #2426 fix(test): fix the reactive charm spread test | charmcraft | merged | Merged a one-line fix for the reactive charm spread test. Approved by two reviewers and passed CI checks. Resolves CHARMCRAFT-641. | |
| #2697 test(spread): stabilize Ubuntu 26.04 charm spread uv/platforms tasks | charmcraft | merged | Merged fix stabilizing Ubuntu 26.04 spread tests by switching charmcraft pack to managed mode, adding a src/charm.py fixture, and restoring dispatch in expected artifacts. CI passed and reviewers approved. | |
| #1325 test(spread): consolidate basic init/build tests | charmcraft | merged | Merged after approval by two reviewers. Consolidates basic init and build spread tests across four files, reducing 80 lines and adding 28. Resolves CRAFT-2127. | |
| #1427 tests(spread): make dependencies restore work even if no charm is built | charmcraft | merged | Merged a one-line change ensuring spread test dependencies restore correctly when no charm is built. Approved by two reviewers and successfully integrated. | |
| #386 tests: update testing to use craft-test | fetch-service | merged | Merged. Replaced direct spread calls with snapcraft test in workflows and spread.yaml, requiring a core24 base update. Approved by two reviewers and modified 10 files (+56/-125 lines). | |
| #1556 refactor(tests/spread): test with explicit charmcraft.yaml files | charmcraft | merged | Merged a refactor replacing dynamic charmcraft.yaml manipulation during the prepare step with explicit configuration files. Approved by three reviewers, the change simplifies testing for upcoming variants across nine modified files. | |
| #1217 tests: add more charms and bundles to large spread suite | charmcraft | merged | Merged test expansion adding more charms and bundles to the large spread suite to increase breaking change sensitivity. Approved by two reviewers, passed CI, and addressed timeout configuration queries. | |
| #16 tests: add spread support | rockcraft | merged | Merged after approval and passing CI, adding spread support to tests per CRAFT-865. | |
| #1386 fix: final spread test fixes for craft-application | charmcraft | merged | Merged after approval by two reviewers. Resolves final spread test issues for craft-application across seven files, removing 127 lines and adding 104. Changes integrated into the main branch. |