← Back to issue list

tests: fix the recording tests to work in multiple architectures

View original Github issue

Metadata

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

Current evaluation

PR #1292 merged into master (commit 0b00c6ce0): recording tests fixed to work across architectures, after #1291 began failing; maintainer noted Travis only runs amd64 so arm coverage comes from nightly autopkg runs.

Resolution: closed resolved

Reason: The PR was approved and merged into master as commit 0b00c6ce0 ('tests: fix the recording tests to work in multiple architectures (#1292)'), a +4/-4 change to one test file that unblocked the beta-testing PR #1291 which was failing on the recording tests.

Evaluated by: mr-cal-bot

No scores available.

Issue body

No description provided.

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved PR #1292 merged into master (commit 0b00c6ce0): recording tests fixed to work across architectures, after #1291 began failing; maintainer noted Travis only runs amd64 so arm coverage comes from nightly autopkg runs.
stealth/space-bunny-alpha — closed resolved Merged PR #1292 into master, fixing recording tests across architectures after failures surfaced in #1291.
qwen/qwen3.6-35b-a3b — — Merged a four-line change that corrected recording tests for cross-architecture compatibility. Approved by one reviewer, the update caused zero coverage impact and was successfully integrated into the master branch.
qwen3.6-35b-a3b-mtp-q6 — — Merged fix for recording tests to support multiple architectures. Resolved integration test failures on non-amd64 platforms by adjusting test configurations, as Travis CI only executes amd64 builds. Code coverage remained unchanged.

Update history

No update history recorded yet.

Related work

  • Related To: canonical/snapcraft#1291 (confidence 90%)

    Maintainer comment states 'I caught this because #1291 started failing'; #1291 (beta-testing PR) was closed unmerged after inactivity.

Related issues

Issue Project State Summary Similarity
#971 tests: fix integration tests in armhf snapcraft merged Merged PR #971 as commit a725e69da, fixing armhf integration-test setup with architecture-specific test configuration.
71%
#1316 Run unit tests on Travis with mock armhf snapcraft merged Merged PR #1316 as commit b1e2a0e11, running Travis unit tests with platform.machine mocked as armhf to expose architecture-specific failures.
70%