← Back to issue list

Run unit tests on Travis with mock armhf

View original Github issue

Metadata

Project
snapcraft
Number
#1316
Type
pull request
State
merged
Author
kalikiana
Labels
Created
Updated
Closed

Current evaluation

Merged to enable Travis armhf unit tests by globally mocking platform.machine. This catches cross-architecture failures previously only visible in autopkgtest. Reviewers approved the testing enhancement despite minor implementation feedback.

Suggested action:

No scores available.

Issue body

Since **Travis** doesn't run tests on _architectures_ other than **amd64** certain failures such as [bug 1690294](https://bugs.launchpad.net/snapcraft/+bug/1690294) only show up in autopkgtest runs. By mocking **platform.machine** globally they can be caught.

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged to enable Travis armhf unit tests by globally mocking platform.machine. This catches cross-architecture failures previously only visible in autopkgtest. Reviewers approved the testing enhancement despite minor implementation feedback.
qwen3.6-35b-a3b-mtp-q6 Merged to add mocked armhf tests to Travis CI, catching architecture-specific failures. Incorporated feedback to mock _get_platform_machine instead of globally patching platform.machine, and fixed failing tests before integration.
qwen3.6-35b-a3b-mtp-q6 Merged after maintainers requested mocking _get_platform_machine instead of globally patching platform.machine. Travis simulates armhf architecture during unit tests, catching architecture-specific bugs missed by amd64 runs.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#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%.
72%