← Back to issue list

tests: skip classic confined tests on armhf

View original Github issue

Metadata

Project
snapcraft
Number
#1822
Type
pull request
State
merged
Author
sergiusens
Labels
—
Created
Updated
Closed

Current evaluation

Merged as commit 1c97b9653, skipping classic-confinement tests on armhf because Snap Store patchelf installation disabled them.

Resolution: closed resolved

Reason: PR snapcraft#1822 was approved and merged as commit 1c97b9653. Its test changes were needed after the snapcraft deb began requiring patchelf from the Snap Store.

Evaluated by: mr-cal-bot

No scores available.

Issue body

We recently added a requirement to the snapcraft deb to install patchelf from the snapstore which disabled a couple of tests. Signed-off-by: Sergio Schvezov <sergio.schvezov@canonical.com> - [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`? -----

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged as commit 1c97b9653, skipping classic-confinement tests on armhf because Snap Store patchelf installation disabled them.
stealth/space-bunny-alpha — closed resolved Merged PR #1822 as commit 1c97b9653, skipping classic-confinement tests on armhf because the deb now sources patchelf from the Snap Store.
stealth/space-bunny-alpha — closed resolved Merged #1822 as commit 1c97b9653, skipping classic-confinement tests on armhf after the Snapcraft deb required Patchelf from the Snap Store.
stealth/space-bunny-alpha — closed resolved Merged snapcraft#1822 as 1c97b9653, skipping classic-confinement tests on armhf after the snapcraft deb began requiring Patchelf from the Snap Store.
stealth/space-bunny-alpha — closed resolved Merged in snapcraft#1822 (commit 1c97b9653), skipping classic confinement tests on armhf after the Snapcraft deb began requiring patchelf from the Snap Store.
stealth/space-bunny-alpha — closed resolved Merged PR #1822 (commit 1c97b9653), skipping classic confinement tests on armhf because the new patchelf-from-Snap-Store requirement disabled them.
qwen/qwen3.6-35b-a3b — — Merged. Skips classic confined tests on armhf architecture because a snapcraft deb requirement to install patchelf from the snapstore disabled them. Approved by one reviewer and merged with a +14/-6 change across seven files.
qwen3.6-35b-a3b-mtp-q6 — — Merged changes to skip classic confined tests on armhf architecture. The update addresses test failures caused by a snapcraft deb requirement to install patchelf from the snapstore. All static and unit checks passed.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#990 tests: fix snaps tests in armhf snapcraft merged Merged PR #990 as commit f467e55bc, correcting armhf failures in snap tests across 11 files and closing Launchpad bug #1650458.
72%
#945 Fix unittests in armhf snapcraft merged Merged PR #945 as commit 073332b9b, making Snapcraft unit tests architecture-agnostic so they pass on armhf.
71%
#1650458 snaps tests fail in armhf snapcraft (launchpad) closed Armhf snap test builds were fixed by snapcore/snapcraft#990, merged as f467e55bc; installation testing remained deferred because runners used LXC.
70%
#1430 tests: enable the snaps installation in armhf snapcraft closed PR #1430 was closed unmerged as stale WIP; its attempt to enable snap-installation tests on armhf was abandoned after the change stopped working.
70%
#1595 tests: fix the skip of snapd integration tests in armhf snapcraft merged Merged #1595 (8db14753d), fixing armhf snapd integration-test skips by checking self.deb_arch only after the parent setUp initialized it.
70%