Move fixtures into conftest - #197
Conversation
a642f0f to
f9c63c1
Compare
f9c63c1 to
d2026c7
Compare
d2026c7 to
97bdb29
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #197 +/- ##
===========================================
+ Coverage 99.94% 100.00% +0.05%
===========================================
Files 302 300 -2
Lines 60040 59990 -50
===========================================
- Hits 60005 59990 -15
+ Misses 35 0 -35 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
@nazrulworld The missing coverage could probably be ignored with |
I agree with you, that could be reason. |
5d0b565 to
748143a
Compare
|
I removed the |
Fixtures can go directly into conftest.
See: https://docs.pytest.org/en/stable/reference/fixtures.html#conftest-py-sharing-fixtures-across-multiple-files
Follow-up to #193.
When running, at some point, I got an error with the old
conftest.pywhich defined fixtures viapytest_pluginswhich is why I had changed it to an import.