Organize tyk2 test data #135
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #135 +/- ##
==========================================
+ Coverage 86.86% 87.30% +0.44%
==========================================
Files 11 14 +3
Lines 1439 1552 +113
==========================================
+ Hits 1250 1355 +105
- Misses 189 197 +8 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
mikemhenry
left a comment
mikemhenry
left a comment
There was a problem hiding this comment.
LGTM! Just one question about pytest_plugins = ["feflow.tests.fixtures.tyk2_fixtures"] since I haven't seen it before
|
|
||
| from feflow.tests.conftest import solvent_comp | ||
| # required plugins/fixtures | ||
| pytest_plugins = ["feflow.tests.fixtures.tyk2_fixtures"] |
There was a problem hiding this comment.
It basically just adds the fixtures/objects in that module to the current scope. I'm using it instead of having all the fixtures in the conftest.py file or something like that, which was the previous case and it was getting a little too big and hard to handle that way.
There was a problem hiding this comment.
I guess it would be equivalent to importing all from that module
This set of changes aim to organize the test data for tyk2 which was in two different places and now it's in the same subdirectory, avoiding redundant data for the main PDB protein file, and hopefully achieving a more organized way. A fixtures modules for the tyk2 data is also implemented to help organization.