Skip to content

Add tools and fixture to support Code Coverage#280

Merged
cedelavergne-ledger merged 1 commit into
masterfrom
cev/coverage
Jun 12, 2026
Merged

Add tools and fixture to support Code Coverage#280
cedelavergne-ledger merged 1 commit into
masterfrom
cev/coverage

Conversation

@cedelavergne-ledger

Copy link
Copy Markdown
Contributor

Add the following parameters

  • "--coverage": Allows to activate the trace for lcov to generate the coverage.info output file
  • "--coverage_output": Output lcov file (default: 'coverage.info')

An html report is automatically generated in genhtml is available (by default in the current path where tests are executed)

It displays the summary at the end of the tests, for example

$ pytest --device=stax test_trusted_name.py --coverage
=================== test session starts ====================
...
=================== firmware coverage ====================
  stax: 1255/7408 lines (16.9%), 135 files -> /work/apps/app-ethereum/tests/functional/coverage.info
        HTML: /work/apps/app-ethereum/tests/functional/coverage_html/index.html

NOTE: the elf file needs to contains the .debug* sections, so here we need to use the binary compiled with DEBUG=1

@cedelavergne-ledger cedelavergne-ledger requested a review from a team as a code owner June 11, 2026 14:23
@cedelavergne-ledger cedelavergne-ledger requested review from sportron-ledger and removed request for a team June 11, 2026 14:23
@codecov-commenter

codecov-commenter commented Jun 11, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 7.10900% with 196 lines in your changes missing coverage. Please review.
✅ Project coverage is 70.27%. Comparing base (3689345) to head (9435fcd).

Files with missing lines Patch % Lines
src/ragger/utils/coverage.py 0.00% 168 Missing ⚠️
src/ragger/conftest/base_conftest.py 29.72% 26 Missing ⚠️
src/ragger/backend/speculos.py 66.66% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #280      +/-   ##
==========================================
- Coverage   75.88%   70.27%   -5.61%     
==========================================
  Files          38       39       +1     
  Lines        2289     2493     +204     
==========================================
+ Hits         1737     1752      +15     
- Misses        552      741     +189     
Flag Coverage Δ
apex_p-py3.10 70.19% <7.10%> (-5.61%) ⬇️
apex_p-py3.11 70.19% <7.10%> (-5.61%) ⬇️
apex_p-py3.12 70.19% <7.10%> (-5.61%) ⬇️
apex_p-py3.13 70.19% <7.10%> (-5.61%) ⬇️
flex-py3.10 70.19% <7.10%> (-5.61%) ⬇️
flex-py3.11 70.19% <7.10%> (-5.61%) ⬇️
flex-py3.12 70.19% <7.10%> (-5.61%) ⬇️
flex-py3.13 70.19% <7.10%> (-5.61%) ⬇️
nanos-py3.10 68.75% <7.10%> (-5.48%) ⬇️
nanos-py3.11 68.75% <7.10%> (-5.48%) ⬇️
nanos-py3.12 68.75% <7.10%> (-5.48%) ⬇️
nanos-py3.13 68.75% <7.10%> (-5.48%) ⬇️
nanosp-py3.10 68.79% <7.10%> (-5.48%) ⬇️
nanosp-py3.11 68.79% <7.10%> (-5.48%) ⬇️
nanosp-py3.12 68.79% <7.10%> (-5.48%) ⬇️
nanosp-py3.13 68.79% <7.10%> (-5.48%) ⬇️
nanox-py3.10 68.75% <7.10%> (-5.48%) ⬇️
nanox-py3.11 68.75% <7.10%> (-5.48%) ⬇️
nanox-py3.12 68.75% <7.10%> (-5.48%) ⬇️
nanox-py3.13 68.75% <7.10%> (-5.48%) ⬇️
stax-py3.10 70.19% <7.10%> (-5.61%) ⬇️
stax-py3.11 70.19% <7.10%> (-5.61%) ⬇️
stax-py3.12 70.19% <7.10%> (-5.61%) ⬇️
stax-py3.13 70.19% <7.10%> (-5.61%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@cedelavergne-ledger cedelavergne-ledger force-pushed the cev/coverage branch 2 times, most recently from e8abe99 to 0b9b4d7 Compare June 11, 2026 14:37
@cedelavergne-ledger cedelavergne-ledger force-pushed the cev/coverage branch 3 times, most recently from 4f9b791 to 6dfec4a Compare June 12, 2026 07:57
@sonarqubecloud

Copy link
Copy Markdown

@cedelavergne-ledger cedelavergne-ledger removed the request for review from sportron-ledger June 12, 2026 08:49
@cedelavergne-ledger cedelavergne-ledger merged commit 4c4d3ab into master Jun 12, 2026
59 checks passed
@cedelavergne-ledger cedelavergne-ledger deleted the cev/coverage branch June 12, 2026 09:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants