← Back to issue list

tests: add /snap/bin to PATH in autopkgtests

View original Github issue

Metadata

Project
snapcraft
Number
#1603
Type
pull request
State
merged
Author
come-maiz
Labels
—
Created
Updated
Closed

Current evaluation

Merged PR #1603 (commit 70a0c6fb1) into master, adding /snap/bin to PATH in the autopkgtest setup so integration tests could find snap binaries; it depended on #1595, also merged.

Resolution: closed resolved

Reason: The PR was approved by kyrofa and merged, landing as commit 70a0c6fb1 (a one-line change adding /snap/bin to PATH in the autopkgtest config). The author noted it fixed all tests failing to locate the binaries, resolving the issue directly.

Evaluated by: mr-cal-bot

No scores available.

Issue body

- [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/)? - [x] If this is a bugfix. Have you checked that there is a bug report open for the issue you are trying to fix on [bug reports](https://bugs.launchpad.net/snapcraft)? - [x] If this is a new feature. Have you discussed the design on the [forum](https://forum.snapcraft.io)? - [x] Have you successfully run `./runtests.sh static`? - [x] Have you successfully run `./runtests.sh unit`? ----- Requires #1595.

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged PR #1603 (commit 70a0c6fb1) into master, adding /snap/bin to PATH in the autopkgtest setup so integration tests could find snap binaries; it depended on #1595, also merged.
stealth/space-bunny-alpha — closed resolved Merged in #1603 as commit 70a0c6fb1, adding /snap/bin to autopkgtest PATH so tests could find snap binaries.
stealth/space-bunny-alpha — closed resolved Merged #1603 via commit 70a0c6fb1, adding /snap/bin to autopkgtests PATH so tests could locate binaries.
stealth/space-bunny-alpha — closed resolved Merged PR #1603, commit 70a0c6fb1, added /snap/bin to PATH in autopkgtests so tests could locate binaries.
qwen/qwen3.6-35b-a3b — — Merged to add /snap/bin to the autopkgtests PATH, fixing binary lookup failures. Approved, passed CI, and depends on PR #1595.
qwen3.6-35b-a3b-mtp-q6 — — Merged, adding /snap/bin to the autopkgtests PATH to fix binary lookup failures. The related ros repo key issue was resolved via test skipping. Requires #1595.

Update history

No update history recorded yet.

Related work

  • Blocked By: canonical/snapcraft#1595 (confidence 85%)

    PR body states 'Requires #1595', the armhf snapd integration-test skip fix, which was merged as 8db14753d

  • Likely Fixed By: canonical/snapcraft#1595 (confidence 60%)

    The follow-up comment mentions the remaining ROS repo key issue was fixed via skipping in the same follow-up work

Related issues

Issue Project State Summary Similarity
#2380 tests: use autopkgtest to leverage snap testing snapcraft merged Merged #2380 as commit 4c5b97a6b, updating 31 files to use autopkgtest for snap testing.
79%
#1200 tests: allow to run individual autopkgtest suites snapcraft merged Merged PR #1200 as commit 532b6d2ff, adding selectable autopkgtest suites so integration and snaps_tests can run independently; fixes LP #1673981.
76%
#1716 tests: split the integration autopkgtests snapcraft merged Merged in snapcraft#1716 via 57af155a7, splitting integration autopkgtests into separate runs to prevent timeout failures despite added setup overhead.
74%
#1542 tests: add integration tests for build snaps snapcraft merged PR #1542 merged as 8c9900f38, adding integration coverage for build snaps; two scenarios remained excluded due to Launchpad bugs #1716054 and #1716055.
73%
#1756 tests: add the missing quote on autopkgtest run snapcraft merged Merged PR snapcraft#1756 restored a missing quote in the local autopkgtest command, fixing a one-character syntax error in commit ab5e6e37b.
73%
#1849 tests: add snap not found tests snapcraft merged Merged missing-snap test coverage; snapcraft PR #1849 landed as commit 54cf2bd9e.
71%
#293 Add the unit tests to the autopkgtest suite. snapcraft merged Merged as commit 4057fe797 (merge commit af4d3bbf5) by elopio, adding Snapcraft's unit tests to the autopkgtest suite for LP #1541421; approved by @sergiusens.
70%