← Back to issue list

tests: add fake snapd fixture

View original Github issue

Metadata

Project
craft-parts
Number
#46
Type
pull request
State
merged
Author
cmatsuoka
Labels
—
Created
Updated
Closed

Current evaluation

Merged PR #46 as 64fdf574 (feature commit 9db65074), adding a 212-line fake snapd fixture for testing snap package handling in craft-parts.packages.

Resolution: closed resolved

Reason: PR #46 was merged after approval from two reviewers. Merge commit 64fdf574 records feature commit 9db65074, which added the fake snapd test fixture.

Evaluated by: mr-cal-bot

No scores available.

Issue body

The fake snapd fixture will be used to test snap package handling in `craft-parts.packages`. Packages/repo is planned to be moved into a separate project, where it will be further refactored. Current code follows the snapcraft implementation. Signed-off-by: Claudio Matsuoka <claudio.matsuoka@canonical.com> - [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? -----

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged PR #46 as 64fdf574 (feature commit 9db65074), adding a 212-line fake snapd fixture for testing snap package handling in craft-parts.packages.
stealth/space-bunny-alpha — closed resolved Merged PR #46 added a 212-line fake snapd fixture for testing snap package handling; merge commit 64fdf574.
stealth/space-bunny-alpha — closed resolved Merged PR #46 added a fake snapd fixture to test snap package handling in craft-parts.packages, adding 212 lines; Windows compatibility was deferred.
qwen/qwen3.6-35b-a3b — — Merged after two approvals and passing CI. Introduces a fake snapd fixture to test snap package handling in craft-parts.packages. Adds 212 lines across two files. Windows compatibility limitations were noted for separate follow-up.
qwen3.6-35b-a3b-mtp-q6 — — Merged. Added a fake snapd fixture for testing snap package handling in craft-parts.packages. Windows compatibility limitations were noted, with plans to address them separately.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#1569 tests: refactor the fake snapd to not hardcode values snapcraft merged Merged PR #1569 as commit cf2a0bf0a, refactoring fake snapd tests to remove hardcoded values; it was a required follow-up to #1567.
78%
#1849 tests: add snap not found tests snapcraft merged Merged missing-snap test coverage; snapcraft PR #1849 landed as commit 54cf2bd9e.
76%
#1600273 Add missing FakeParts snapcraft (launchpad) closed Missing FakeParts fixtures implemented via snapcore/snapcraft PR #640, merged as commit a119759b8 "Setup the FakeParts store for all unit tests"; maintainer sergiusens closed the bug pointing to that PR.
74%
#73 tests: add stage snaps integration test (CRAFT-68) craft-parts merged Merged PR #73 via commit a5dbc53e (merge 978aeac1), adding an integration test that verifies stage snaps are unpacked into the part install directory.
74%
#4662 fix(test): do not create 2 snapcraft.yaml and confuse location snapcraft merged Merged PR #4662 as commit a50b0617a, fixing test fixture paths so tests do not create duplicate root and /snap manifests that cause ambiguous hook lookup.
73%
#521 test(spread): add an improved snap inspector test fetch-service merged Merged in PR #521 (commit 06cd210), adding a more realistic snapd setup to the spread snap inspector test and exposing additional snapd network requests.
72%
#1799 tests: update the registered snap fake snapcraft merged Merged #1799, a test-only update to the registered snap fake changing four lines across two files; it received one approval with no unresolved comments.
72%
#1652 tests: fork skip into snaps_tests snapcraft merged PR #1652 (merged as commit c951a3679) forked a skipped test into snaps_tests to avoid overly broad snapcraft.tests imports that broke autopackage tests.
72%
#1724 tests: use the common base handler on the fake snapd server snapcraft merged Merged PR #1724 in commit af78689eb, switching the fake snapd server to the common base handler to prevent debug messages from leaking into stdout assertions.
72%
#2295 tests: use SNAPCRAFT_PACKAGE_TYPE everywhere snapcraft merged PR #2295 merged as commit ffbad69ea: unit tests now use SNAPCRAFT_PACKAGE_TYPE instead of legacy SNAPCRAFT_FROM_* env vars, and unused container tests were removed to ease migration to spread. (LP: #1794379)
72%