fix: fix tests on arm64
Metadata
Current evaluation
Merged to fix arm64 test failures. Approved by two reviewers and integrated after passing most CI checks, despite a failing OSV-scanner. Resolves #143 and #144 with a 32-line change across two files.
Suggested action: —
No scores available.
Issue body
Requires #143
Fixes #144
CRAFT-3801
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged to fix arm64 test failures. Approved by two reviewers and integrated after passing most CI checks, despite a failing OSV-scanner. Resolves #143 and #144 with a 32-line change across two files. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged to fix arm64 test failures. The update depends on PR #143 and closes issue #144, resolving CRAFT-3801. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #990 tests: fix snaps tests in armhf | snapcraft | merged | Merged into master to fix failing snap tests on the armhf architecture. Approved by a reviewer, the change updated 11 files and resolved CI failures without altering overall code coverage. | |
| #971 tests: fix integration tests in armhf | snapcraft | merged | Merged to master after approval and passing CI. The patch fixes integration tests on armhf architecture. Reviewers approved the change but recommended using project_options over platform.machine for future architectural consistency. Code coverage remained stable at 96.39%. | |
| #1174 fix: add `armv8l` support | craft-application | merged | Added armv8l architecture support, resolving issue #1173. Merged following approval by two reviewers and passing core CI checks. Changes modify five files with 31 additions and 5 deletions. | |
| #945 Fix unittests in armhf | snapcraft | merged | Merged fix for armhf unit tests. Updated 3 files with 33 additions and 16 deletions. Code coverage remained stable at 97.06% post-merge. | |
| #1985 tests: document arm testing setup | snapcraft | merged | Merged pull request documenting the ARM testing setup. The change adds 66 lines to a single file and was approved by two reviewers before integration. | |
| #1595 tests: fix the skip of snapd integration tests in armhf | snapcraft | merged | Merged a fix for snapd integration test skipping on armhf. Accessing self.deb_arch before the parent setUp caused failures. Correcting the initialization order resolved the issue. Approved by one reviewer and passed CI. | |
| #2173 fix(snap): fix build on 32-bit platforms | charmcraft | merged | Merged a one-line fix resolving snap build failures on 32-bit armhf platforms. Approved by three reviewers and validated via a successful Launchpad remote build. | |
| #1030 ci: run Python tests on arm64 | craft-application | merged | Merged. Enables Python unit and integration tests on ARM64 runners. Resolved ARM64 compatibility issues and marked complex fetch service tests as amd64-only, tracking remaining work in issue #1032. Approved and passes CI. Prerequisite for #1027 and CRAFT-4992. | |
| #6350 test: fix spread tests | snapcraft | merged | Merged after fixing non-store spread tests and the OSV scanner. Pinned black to 26.3.1 to resolve patchelf segfaults and silent failures. Approved by two reviewers and passed CI checks. | |
| #244 fix: add support for armv8l | craft-platforms | merged | Merged to resolve issue #243 by mapping the armv8l architecture to armhf and armv7l. The change was approved by two reviewers, passed all linting and test suites, and integrated into the main branch. |