-
Notifications
You must be signed in to change notification settings - Fork 794
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
test: Adds test-(slow|fast)
options
#3555
Conversation
Locally, this covers most tests that take 2-10 secs. In total, 354/1671 tests are skipped and this reduces runtime 40 secs -> 20 secs
test-(slow|fast)
test options
test-(slow|fast)
test optionstest-(slow|fast)
options
These were some drive-by fixes of typing issues, fixing them here in case this stays in review for a while |
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.
Really neat! :) I learned a lot while reading this. Also great to have more convenient decorators for skipping tests re pyarrow and vlc.
Thanks @binste! I've found when reading the |
While working on #3547, I've been running the test suite a bunch to identify bottlenecks.
This PR provides
Impact?
Important
The default behaviour is unchanged: