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 tests from pulpcore #685
Conversation
Hello @bmbouter! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:
Comment last updated at 2022-04-18 20:24:44 UTC |
Required PR: pulp/pulp_file#685 [noissue]
Required PR: pulp/pulp_file#685 [noissue]
f5347ec
to
abbb8d6
Compare
Required PR: pulp/pulp_file#685 [noissue]
abbb8d6
to
ac26126
Compare
Required PR: pulp/pulp_file#685 [noissue]
497b9d5
to
d27e3d5
Compare
Required PR: pulp/pulpcore#2535 [noissue]
d27e3d5
to
737fbdb
Compare
Required PR: pulp/pulp_file#685 [noissue]
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.
Let me understand the directory name from_pulpcore
. I think it describes the history of that files. But that is no useful information here. Do you intend to move the tests out of this directory while transforming them to using pytest? Or should we name it with_pulpcore
or simply pulpcore
to signify that we test pulpcore
instead of pulp_file
in that directory?
@@ -7,4 +7,4 @@ include COPYRIGHT | |||
include functest_requirements.txt | |||
include test_requirements.txt | |||
include unittest_requirements.txt | |||
recursive-exclude pulp_file/tests/fixtures/ * | |||
recursive-exclude pulp_file/tests/functional * |
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.
Yay! Finally.
Required PR: pulp/pulp_file#685 [noissue]
Required PR: pulp/pulpcore#2535
[noissue]