ci: run python unit tests with btc-only FW (nightly)
What changed, and why it matters
This commit only changes the project's automated testing configuration. It adds a nightly CI job that runs Python unit tests against a Bitcoin-only firmware build in addition to the existing universal build. There is no change to the actual firmware code, no bug fix, and no security-relevant behavior change.
No security action needed. This is a routine CI improvement to expand test coverage.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The commit modifies .github/workflows/core.yml to add a matrix dimension ‘coins’ to the core_unit_python_test job. On scheduled (nightly) runs, the matrix expands to include both ‘universal’ and ‘btconly’ firmware variants; on other triggers only ‘universal’ runs. It sets BITCOIN_ONLY=1 for the btconly variant. This is purely a CI/test coverage change.
Changed components
.github/workflows/core.ymlInspect captured patch +3 / −1
diff --git a/.github/workflows/core.yml b/.github/workflows/core.yml
index 6af34378..5ff9f9dc 100644
--- a/.github/workflows/core.yml
+++ b/.github/workflows/core.yml
@@ -212,16 +212,18 @@ jobs:
retention-days: 2
core_unit_python_test:
- name: Python unit tests (${{ matrix.model }}, ${{ matrix.asan }})
+ name: Python unit tests (${{ matrix.model }}, ${{ matrix.coins }}, ${{ matrix.asan }})
runs-on: ubuntu-latest
needs: param
strategy:
fail-fast: false
matrix:
model: [T2T1, T3B1, T3T1, T3W1]
+ coins: ${{ fromJSON(github.event_name == 'schedule' && '["universal", "btconly"]' || '["universal"]') }}
asan: ${{ fromJSON(needs.param.outputs.asan) }}
env:
TREZOR_MODEL: ${{ matrix.model }}
+ BITCOIN_ONLY: ${{ matrix.coins == 'universal' && '0' || '1' }}
ADDRESS_SANITIZER: ${{ matrix.asan == 'asan' && '1' || '0' }}
LSAN_OPTIONS: "suppressions=../../asan_suppressions.txt"
DISABLE_TROPIC: 0
Why this scored 15/100
Community notes
Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.
The AI analysis stands alone for now. Submit a note if you can add evidence or important context.