-
Notifications
You must be signed in to change notification settings - Fork 40
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
Fix AssertionError when AUTO cases are requested outside a "test_" module #320
Conversation
The correct file-name pattern is "cases_", not "case_"
PR is good to go from my side. @smarie, feel free to add any suggestions |
Thanks @michele-riva ! I made a few minor comments |
Co-authored-by: Sylvain Marié <[email protected]>
Co-authored-by: Sylvain Marié <[email protected]>
Thanks for the suggestions, @smarie. I have just implemented them. |
All good, thanks @michele-riva ! |
This broke a use case where we defined our "tests" in It would be nice if this used the pytest config to determine where to look for AUTO cases |
I was not aware of this possibility @mroeschke ! And since I have been designing pytest-cases just for this (creating benchmarks) I really agree with fixing this. Let's move this to an explicit issue and discuss possible workarounds |
Fixes #309