-
-
Notifications
You must be signed in to change notification settings - Fork 921
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
fix: prevent pytest from collecting tests on testing.TestClient #2148
Conversation
Codecov Report
@@ Coverage Diff @@
## master #2148 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 62 62
Lines 6789 6791 +2
Branches 1095 1095
=========================================
+ Hits 6789 6791 +2
|
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.
Hey there, thanks for this PR, it looked good bar some extraneous whitespace that made the flake8
and blue
checks fail.
However, in your latest commit you changed virtually every line of the whole codebase, we'll need to revert this one 😅
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.
Hi again, still the same problem. Your PR proposes to change 68706 lines, basically the whole codebase, while I expect this fix to consist of 2-3 lines plus maybe a newsfragment. You need to revert unrelated changes before we can proceed with this.
I'm actually unsure on how to revert it. Maybe a new PR? or are there any commands to execute locally? |
sure, feel free to close this and reopen a new PR. An alternative may be to force push on this branch applying your original change to master |
Looks like a classic CRLF (Windows) vs LF (Unix) issue If you're using vscode look at the bottom right. In this image it says Most repos use Unix convention of See: https://stackoverflow.com/questions/1967370/git-replacing-lf-with-crlf |
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.
Thanks for bringing this up @aryaniyaps! 👍
I took the liberty of cleaning whitespace and line endings, as well as adding a newsfragment.
And thanks for some good tips @rite2hhh! FWIW, I have now added |
Summary of Changes
prevent pytest from collecting tests on testing.TestClient
Related Issues
fixes #2147
Pull Request Checklist
This is just a reminder about the most common mistakes. Please make sure that you tick all appropriate boxes. But please read our contribution guide at least once; it will save you a few review cycles!
If an item doesn't apply to your pull request, check it anyway to make it apparent that there's nothing to do.
docs/
.docs/
.versionadded
,versionchanged
, ordeprecated
directives.docs/_newsfragments/
, with the file name format{issue_number}.{fragment_type}.rst
. (Runtowncrier --draft
to ensure it renders correctly.)If you have any questions to any of the points above, just submit and ask! This checklist is here to help you, not to deter you from contributing!
PR template inspired by the attrs project.