Skip to content

Tests: Use shared fixtures, increase test coverage#341

Merged
vigith merged 7 commits intomainfrom
unit-tests
Mar 22, 2026
Merged

Tests: Use shared fixtures, increase test coverage#341
vigith merged 7 commits intomainfrom
unit-tests

Conversation

@BulkBeing
Copy link
Contributor

@BulkBeing BulkBeing commented Mar 21, 2026

Moves remaining tests from unittest to pytest. We were already using pytest as the test runner, but some tests were written with unittest module.

  • Upgrade pytest minimum version (we already use latest since we just have >=7.x)
  • Use newer features in pytest
  • Increase test coverage.

Signed-off-by: Sreekanth <prsreekanth920@gmail.com>
Signed-off-by: Sreekanth <prsreekanth920@gmail.com>
@codecov
Copy link

codecov bot commented Mar 21, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 92.67%. Comparing base (7a908f2) to head (965d955).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #341   +/-   ##
=======================================
  Coverage   92.67%   92.67%           
=======================================
  Files          67       67           
  Lines        3509     3509           
  Branches      229      229           
=======================================
  Hits         3252     3252           
  Misses        193      193           
  Partials       64       64           

☔ View full report in Codecov by Sentry.
📢 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.

Signed-off-by: Sreekanth <prsreekanth920@gmail.com>
Signed-off-by: Sreekanth <prsreekanth920@gmail.com>
Signed-off-by: Sreekanth <prsreekanth920@gmail.com>
Signed-off-by: Sreekanth <prsreekanth920@gmail.com>
@BulkBeing BulkBeing marked this pull request as ready for review March 21, 2026 05:54
Signed-off-by: Sreekanth <prsreekanth920@gmail.com>
@vigith vigith merged commit f894c01 into main Mar 22, 2026
11 checks passed
@vigith vigith deleted the unit-tests branch March 22, 2026 00:52
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.

2 participants