Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Reorganising pytest fixtures #222

Closed
sfmig opened this issue Jun 14, 2024 · 0 comments · Fixed by #380
Closed

Reorganising pytest fixtures #222

sfmig opened this issue Jun 14, 2024 · 0 comments · Fixed by #380
Assignees
Labels
enhancement New optional feature

Comments

@sfmig
Copy link
Contributor

sfmig commented Jun 14, 2024

Is your feature request related to a problem? Please describe.
We generally prefer fixtures to be close to the test they refer to, but sometimes the files become a bit bloated. It'd be nice for contributors if the tests are easy to read and navigate. For this shorter files are generally nicer.

The goal of this issue is to discuss refactoring options for the fixtures.

Describe the solution you'd like
Some options for refactoring the fixtures are:

  • using a conftest.py file(s)
  • using pytest_plugins variables like this (we do that in 🦀 s).

Describe alternatives you've considered
\

Additional context
\

@sfmig sfmig added the enhancement New optional feature label Jun 14, 2024
@sfmig sfmig moved this from 🤔 Triage to 📝 Todo in movement progress tracker Aug 27, 2024
@lochhh lochhh self-assigned this Jan 6, 2025
@niksirbi niksirbi linked a pull request Jan 21, 2025 that will close this issue
7 tasks
@niksirbi niksirbi moved this from 📝 Todo to 🚧 In Progress in movement progress tracker Jan 21, 2025
@lochhh lochhh mentioned this issue Jan 24, 2025
7 tasks
@github-project-automation github-project-automation bot moved this from 🚧 In Progress to ✅ Done in movement progress tracker Jan 31, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New optional feature
Projects
Development

Successfully merging a pull request may close this issue.

2 participants