tests: take into account the new current link.
Metadata
Current evaluation
Merged in PR #1187 (commit 57055d1e4), updating tests to account for snapd's new ~/snap/<name>/current symlink.
Resolution: closed resolved
Reason: PR #1187 was approved by @come-maiz and merged as commit 57055d1e4, updating the affected tests to exclude the new current symlink when searching snap directories. The maintainer noted that remaining failures came from separate snapd bug #1672495.
Evaluated by: mr-cal-bot
No scores available.
Issue body
snapd in zesty now creates a symlink from `~/snap/<name>/current`
to `~/snap/<name>/<revision>`. We need to take this into account
when searching for files in the snaps directories.
LP: #1672463
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged in PR #1187 (commit 57055d1e4), updating tests to account for snapd's new ~/snap/<name>/current symlink. | |
| stealth/space-bunny-alpha | — | closed resolved | Updated tests for snapd's ~/snap/<name>/current symlink; merged in canonical/snapcraft#1187 via commit 57055d1e4. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged a fix to handle snapd's new symlink behavior in zesty, updating file searches in snap directories to account for the `current` link pointing to revisions. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged. Updates tests to handle snapd's new current symlink pointing to revision directories. Resolves LP #1672463. Approved by maintainer after confirming unrelated test failures. |
Update history
No update history recorded yet.
Related work
-
Likely Fixed By:
snapcraft (launchpad)#1672463
(confidence 100%)
Launchpad bug explicitly linked in the PR body and recorded as fixed by merged PR #1187.
-
Related To:
snapd#1672495
(confidence 95%)
The maintainer identified this separate snapd bug as the cause of the only remaining failures.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1672463 snaps tests fail because of the new current link | snapcraft (launchpad) | closed | Updated snap tests for snapd's new current-revision symlink, fixing BusyBox and Mosquitto failures in snapcore/snapcraft#1187 (57055d1e4). | |
| #2352 tests: use valid snap names in unit tests | snapcraft | merged | Merged PR #2352 (commit b8c0a4fbf) changed snap names used in unit tests so they pass snap pack --check-skeleton; author merged directly as a minor test-only change. | |
| #2351 tests: use valid snap names in unit tests | snapcraft | closed | Closed unmerged; the same test-only change was merged instead via PR #2352 (commit b8c0a4fbf), updating unit-test snap names to pass `snap pack --check-skeleton`. | |
| #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. |