fix: spread tests and tutorials for extentions with default base bare
Metadata
Current evaluation
Merged. Updated spread tests and tutorials to align with extensions now using bare as the default base. Approved by three reviewers and passed CI after resolving initial failures. Modified 17 files.
Suggested action: —
No scores available.
Issue body
<!-- Describe your changes -->
Fix tutorials and spread tests for extensions that now have `bare` as default base
Follow up #1086
---
- [x] I've followed the [contribution guidelines](https://github.com/canonical/rockcraft/blob/main/CONTRIBUTING.md).
- [x] I've signed the [CLA](http://www.ubuntu.com/legal/contributors/).
- [ ] I've successfully run `make lint && make test`.
- [x] I've added or updated any relevant documentation.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged. Updated spread tests and tutorials to align with extensions now using bare as the default base. Approved by three reviewers and passed CI after resolving initial failures. Modified 17 files. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged following review, this update fixes spread tests and tutorials for extensions defaulting to bare. Resolves CI test failures flagged by reviewers and follows up on #1086. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged to update spread tests and tutorials for extensions adopting the new default bare base. Changes resolved failing Django, FastAPI, and Flask extension tests and aligned documentation with the base configuration introduced in #1086. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #3020 spread tests: default base for local plugin tests | snapcraft | merged | Merged updates to spread tests establishing a default base for local plugin tests. The change follows a prior commit pattern, adding two lines across two files. Approved by one reviewer and passed CI checks. | |
| #127 Fix spread tests | imagecraft | merged | Merged after receiving approval from two reviewers and passing all CI checks. The changes fix spread tests across 11 files (+49/-38 lines). | |
| #3662 tests: update spread url | snapcraft | merged | Merged update to spread tarball URLs in test files. Approved by a reviewer, passed CI checks, modified two files with minimal changes, and maintained existing code coverage. | |
| #1287 tests: minor cleanups on the spread tests | snapcraft | merged | Merged minor spread test cleanups into master. Modified four files with 16 additions and 6 deletions. Approved by one reviewer with zero unresolved comments, maintaining 95.27% coverage upon integration. | |
| #299 refactor(spread): prototype spread extension | fetch-service | merged | Merged after two approvals. Refactored test suites to use the spread extension mechanism, dynamically generating spread.yaml before execution. Resolves #291. Modified 14 files, +325/-317 lines. | |
| #3663 tests: update spread url | snapcraft | merged | Merged following reviewer approval and passing CI. Updated spread tarball URLs in test configuration files, modifying two files with minimal changes. | |
| #934 fix(templates): typo in spread .extension | rockcraft | merged | Merged a single-line correction for a typo in the spread .extension template. The fix passed all CI checks and received approval from two reviewers before integration. | |
| #13 test(spread): add more spread tests | snapcraft-rocks | merged | Successfully merged spread tests covering architectures, chisel slices, craftctl, package-repositories, and patchelf. Approved by two reviewers across 15 files (+206/-5 lines). | |
| #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. | |
| #5378 test: fix extension tests | snapcraft | merged | Merged unit test updates for expand_extensions(). No code changes were needed as existing logic already handles core22 and core24 correctly. Approved by two reviewers and passed CI checks prior to merge. |