Skip to content

Refactor of conftest #353

@felix-e-h-p

Description

@felix-e-h-p

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

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions