Skip to content

Conversation

@nasubbia
Copy link
Member

@nasubbia nasubbia commented Oct 23, 2025

Description

Adds pytest-cov to generate test coverage report.

Fixes # (issue)

  • Fixes test runs. Adds venv path to flake8 exclusion list.

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)

How Has This Been Tested?

  • Run make test
  • After the tests are run, open index.html under htmlcov folder to see the test coverage report

Test Configuration:

  • Firmware version:
  • Hardware:
  • Toolchain:
  • SDK:

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas - N/A
  • I have made corresponding changes to the documentation - N/A test coverage is run as part of the current test command
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published in downstream modules - N/A

@oracle-contributor-agreement oracle-contributor-agreement bot added the OCA Verified All contributors have signed the Oracle Contributor Agreement. label Oct 23, 2025
@nasubbia nasubbia force-pushed the nasubbia/adds-test-coverage-report branch from e8a4e0d to 72590be Compare October 28, 2025 21:08
@cboffa13
Copy link
Member

Is there a way so set the coverage threshold to what it is now and then we can slowly add more tests til we reach the 80% mark?

@nasubbia nasubbia force-pushed the nasubbia/adds-test-coverage-report branch from 72590be to 54ab93a Compare October 30, 2025 20:38
@nasubbia
Copy link
Member Author

Is there a way so set the coverage threshold to what it is now and then we can slowly add more tests til we reach the 80% mark?

yes, currently we're at 72% coverage. I've added that as the current threshold

@nasubbia nasubbia force-pushed the nasubbia/adds-test-coverage-report branch from 54ab93a to 62f9d9d Compare November 4, 2025 19:46
@nasubbia nasubbia force-pushed the nasubbia/adds-test-coverage-report branch from 62f9d9d to d17e8c2 Compare November 5, 2025 00:25
@nasubbia nasubbia force-pushed the nasubbia/adds-test-coverage-report branch from d17e8c2 to 3aa6729 Compare November 5, 2025 00:52
@nasubbia nasubbia force-pushed the nasubbia/adds-test-coverage-report branch from 3aa6729 to 4328168 Compare November 5, 2025 18:32
AlaaShaker
AlaaShaker previously approved these changes Nov 5, 2025
@gebhardtr gebhardtr merged commit c36757f into oracle:main Nov 7, 2025
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

OCA Verified All contributors have signed the Oracle Contributor Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants