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
Add test failure with descriptive error message when running tests without a {project} placeholder #1336
Conversation
…thout a {project} placeholder
3072bad
to
e4cbe81
Compare
@henryiii any chance of a review on this one too? :) |
Ahh, I thought this was a |
# pytest is run from there | ||
|
||
|
||
def 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.
This assumes pytest. Would it make more sense to write it in unittest, which pytest will also pick up and run? I'm assuming this still won't get other testing frameworks (like ward), but this might be better than just assuming pytest?
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.
good idea! I'll try that
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, minus one comment about the testing frameworks covered.
Also I assume if someone has testpaths=
in their pyproject.toml or elsewhere, that won't get picked up (since it's in the wrong directory) so the "error message" test file will still get run. 👍
Travis is having a slow day today 😴 |
Fixes #621