Skip to content

Split resource fixtures and factories #2353

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.

Sign up for GitHub

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

Jump to bottom
Open
nejch opened this issue Oct 31, 2022 · 1 comment
Open

Split resource fixtures and factories #2353

nejch opened this issue Oct 31, 2022 · 1 comment
Labels
test For issues or missing tests

Comments

@nejch
Copy link
Member

nejch commented Oct 31, 2022 

    Does it make sense to move this block into a fixture and re-use it in other functions?

Originally posted by @lmilbaum in #2350 (comment)
@nejch nejch mentioned this issue Oct 31, 2022
Merged
@nejch nejch added the test For issues or missing tests label Oct 31, 2022
@nejch
Copy link
Member Author

nejch commented Oct 31, 2022

This might be a little harder than I initially thought because we use quite a few module-level fixtures for heavy resources. We'd probably need pytest-dev/pytest#1681 to be able to use factories in these fixtures. 🤔

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
test For issues or missing tests
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@nejch