-
Notifications
You must be signed in to change notification settings - Fork 301
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
Set pytest directory to speed up test collection #1755
Set pytest directory to speed up test collection #1755
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1755 +/- ##
==========================================
- Coverage 67.56% 67.55% -0.01%
==========================================
Files 146 146
Lines 18343 18340 -3
==========================================
- Hits 12393 12390 -3
Misses 5950 5950 |
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.
Cool
@jenshnielsen pinging because I added to restrict the file glob pattern. |
setup.cfg
Outdated
[tool:pytest] | ||
testpaths = "qcodes/tests" | ||
python_files = test_*.py | ||
python_functions = test_* |
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.
the python functions and python files entries - these are pytest defaults, right? do we really need to explicitly define them here?
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.
I wasn't 100% sure, how this was handled, but I read up on it: you are absolutely right. These are the defaults, which get overwritten individually when set. (only difference is python_function = test*, but we really don't care). So I will remove them again.
This seems to speed up my local test collection.