-
Notifications
You must be signed in to change notification settings - Fork 98
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
feat: poetry script for easier test case running #445
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.
Another script to add: running one specific test/test file (you just leave off the discover
)
I recommend updating the CI scripts to use poe as well.
I just remembered that I added Perhaps a script should run both sets of tests just for coverage purposes? I recommend using these settings:
|
Did not call |
|
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
High Level Overview of Change
Enabled developers to use
poetry run test FILE PATH
,poetry run test_unit
,poetry run test_integration
andpoetry run test_coverage
for testing nowContext of Change
Currently it’s hard to remember how to run tests, it basically requires looking up the instructions in the CONTRIBUTING.md
Instead, it’d be better to write some short poetry scripts that allow you to just do something like poetry run test and poetry run test_integration for unit and integration tests respectively.
Type of Change
Test Plan
Ran both commands