Skip to content

test: add tests to have a full coverage#207

Draft
mdevolde wants to merge 2 commits into
jxmorris12:masterfrom
mdevolde:test/full_coverage
Draft

test: add tests to have a full coverage#207
mdevolde wants to merge 2 commits into
jxmorris12:masterfrom
mdevolde:test/full_coverage

Conversation

@mdevolde

Copy link
Copy Markdown
Collaborator

test: add tests to have a full coverage

Why the pull request was made

To have a full coverage of the library, and easily detect bugs on future changes.

Summary of changes

  • Refactor tests structure in benchmark, integration, property, unit.
  • Add benchmarks tests.
  • Add pytest-benchmark and hypothesis as test dependencies.
  • [in progress] Add tests.

Screenshots (if appropriate):

Not applicable.

How has this been tested?

Applied tests.

Resources

Not applicable.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update (changes to documentation only)
  • Refactor / code style update (non-breaking change that improves code structure or readability)
  • Tests / CI improvement (adding or updating tests or CI configuration only)
  • Chore / maintenance (non-breaking change that does not affect functionality, such as updating dependencies or fixing typos)
  • Other (please describe):

Checklist

  • Followed the project's contributing guidelines.
  • Updated any relevant tests.
  • Updated any relevant documentation.
  • Added comments to your code where necessary.
  • Formatted your code, run the linters, checked types and tests.
  • Added your changes to the CHANGELOG file, if applicable.

@mdevolde mdevolde self-assigned this Jun 27, 2026
@codecov-commenter

codecov-commenter commented Jun 27, 2026

Copy link
Copy Markdown

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 88.77%. Comparing base (f3d5fc1) to head (42c9c4e).
❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #207      +/-   ##
==========================================
+ Coverage   80.70%   88.77%   +8.07%     
==========================================
  Files          13       13              
  Lines        1767     1755      -12     
==========================================
+ Hits         1426     1558     +132     
+ Misses        341      197     -144     

☔ 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.

@mdevolde mdevolde force-pushed the test/full_coverage branch from b27716e to d7812b1 Compare June 27, 2026 17:26
Comment thread tests/unit/test_internals_utils.py Fixed
Comment thread tests/unit/test_download_unit.py Fixed
@mdevolde mdevolde force-pushed the test/full_coverage branch from d7812b1 to 42c9c4e Compare June 27, 2026 17:33
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