generated from openclimatefix/ocf-template
-
-
Notifications
You must be signed in to change notification settings - Fork 37
Open
Description
Following on from merging of: #350
Current conftest contains a mix of repo-level and module-specific fixtures - functionally OK as is but could potentially increase a risk of hidden coupling across modules for example, with any further test updates, and I feel is genuinely not as easy to follow as could potentially be.
Proposing a slight refactor that would just be:
- Keep only shared/global fixtures in main conftest e.g seed setting, test assets etc.
- Move module specific fixtures into own conftest.py under relevant test directories i.e. tests/numpy_sample/conftest.py.
- Review of overall fixture scopes to match related usage.
Happy to put together a draft PR, cheers.
Metadata
Metadata
Assignees
Labels
No labels