← Back to issue list

spread: add build packages to package repository tests

View original Github issue

Metadata

Project
snapcraft
Number
#3806
Type
pull request
State
merged
Author
cmatsuoka
Labels
—
Created
Updated
Closed

Current evaluation

Merged pull request adding spread tests to verify build packages install via snapcraft.yaml package repositories in destructive and build instance modes. Approved and merged with no coverage impact.

Suggested action: —

No scores available.

Issue body

Ensure build packages are also installable using package repositories specified in snapcraft.yaml, in both destructive mode and using a build instance. - [x] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)? - [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [ ] Have you successfully run `make lint`? - [ ] Have you successfully run `pytest tests/unit`? -----

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b — — Merged pull request adding spread tests to verify build packages install via snapcraft.yaml package repositories in destructive and build instance modes. Approved and merged with no coverage impact.
qwen3.6-35b-a3b-mtp-q6 — — Merged into main. Adds tests verifying build packages install via package repositories in destructive mode and build instances. Code coverage unchanged.
qwen3.6-35b-a3b-mtp-q6 — — Merged into main. Added spread tests verifying build packages install via package repositories in snapcraft.yaml for both destructive mode and build instances. Codecov reported no coverage impact.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#5845 test(spread): test building base and snapd snaps snapcraft merged Merged spread tests for building base and snapd snaps. Despite initial CI failures linked to snapcraft issue #5843, the changes were approved and integrated to maintain build compatibility with mainline snapcraft.
77%
#1272 tests: initial setup for the snapcraft snap tests with spread snapcraft merged Merged to master following single reviewer approval. Introduced initial spread test infrastructure for snapcraft across four files (+39 lines). Code coverage remained stable at 95.33%.
77%
#2793 tests: enable SNAPCRAFT_BUILD_INFO for spread snapcraft merged Merged following approval and passing CI. Enabled SNAPCRAFT_BUILD_INFO for spread tests to cover manifest changes. Included a fix for the nil-basic spread test. Minimal diff across three files.
76%
#4105 tests: check for submodules before running spread snapcraft merged Merged. Adds a pre-run check for the snapd-testing-tools submodule in spread tests to provide a clear error if missing. Approved by two reviewers, passed CI, and introduced no coverage changes.
73%
#1084 tests: check for submodules before running spread charmcraft merged Merged changes verifying the snapd-testing-tools submodule before running Spread tests. Adds a clear error if missing. Approved by a reviewer, passed CI, and merged with minimal modifications.
73%
#240 tests: check for submodules before running spread rockcraft merged Merged. Adds a pre-check for the snapd-testing-tools submodule before running spread tests, providing a clear error if missing. Approved by two reviewers, passed CI, and merged successfully.
73%
#2321 tests: add spread suite for plainbox plugin (#2317) snapcraft merged Merged after approval and passing CI. The pull request adds a Spread test suite for the plainbox plugin, pins the checkbox-ng package, and removes a misplaced test from the snapcraft test suite.
72%
#5389 test(spread): trace logs in spread tests snapcraft merged Merged after approval. Added trace logging to spread tests. Despite CI failures from extra snapcraft output, the change was approved and merged following a related feature branch integration to prevent conflicts.
72%
#4937 [DRAFT] publish test snap for spread tests snapcraft closed Abandoned without merging. The draft debugging PR successfully published two revisions of the test-snapcraft-assertions snap to the staging store for spread testing, fulfilling its purpose.
72%
#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).
71%