Skip to content

Conversation

@bearomorphism
Copy link
Collaborator

@bearomorphism bearomorphism commented Feb 3, 2026

As title. There are duplicated fixtures in test files.

What is the change

  • Removed duplicated fixtures
  • Made changelog_path and config_path of type Path
  • Replaced with open(some_path_obj) with with some_path_obj.open() in tests

@codecov
Copy link

codecov bot commented Feb 3, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.99%. Comparing base (0731f76) to head (db6d3ec).
⚠️ Report is 10 commits behind head on master.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1843      +/-   ##
==========================================
- Coverage   97.99%   97.99%   -0.01%     
==========================================
  Files          60       60              
  Lines        2696     2691       -5     
==========================================
- Hits         2642     2637       -5     
  Misses         54       54              

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

@bearomorphism bearomorphism marked this pull request as draft February 3, 2026 16:22
@bearomorphism bearomorphism changed the title test(conftest): remove duplicated fixtures test: use pathlib utilities in tests Feb 3, 2026
@bearomorphism bearomorphism marked this pull request as ready for review February 3, 2026 16:37
@bearomorphism bearomorphism merged commit 6ee9868 into master Feb 4, 2026
20 checks passed
@bearomorphism bearomorphism deleted the test-config-dup branch February 4, 2026 03:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants