-
Notifications
You must be signed in to change notification settings - Fork 5.1k
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
Tidy up tests directory #3553
Tidy up tests directory #3553
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice work cleaning up the tests structure. Maybe worth doing these as well:
- Adding the
ext
,inline
,passport
folders in the tests so we can shift related test files in there. I believe they can have their ownconftest
s as well if that helps in better organizing it
I guess then it would make sense to mimic the whole directory structure of |
conftest
to tests/auxil
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Replacement for #3420