-
Notifications
You must be signed in to change notification settings - Fork 123
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
Distribution tree tests #1860
Distribution tree tests #1860
Conversation
Hello @pavelpicka! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found: There are currently no PEP 8 issues detected in this Pull Request. Cheers! 🍻 Comment last updated at 2020-10-20 15:45:56 UTC |
e595019
to
8188745
Compare
Attached issue: https://pulp.plan.io/issues/7571 |
615707c
to
bc5ee6e
Compare
@@ -1215,3 +1220,164 @@ def test_all(self): | |||
RPM_FIXTURE_SUMMARY, | |||
get_content_summary(repo.to_dict()) | |||
) | |||
|
|||
|
|||
class DistributionTreeTest(PulpTestCase): |
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.
Optional suggestion here but may want to break out these distribution tests into a separate distribution_tree.py file since test_sync.py and test_copy.py are getting incredibly long.
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.
I agree it is good idea, so moving test to test_distribution_tree.py
This is looking good 👍 |
Sync test: update main repo of DT update addon repo of DT update variant repo DT remove repository with DT, and orphan cleanup to remove DT Copy test: simple copy of DT copy DT by content specified [nocoverage] closes: #7571 https://pulp.plan.io/issues/7571
Sync test:
update main repo of DT
update addon repo of DT
update variant repo DT
remove repository with DT, and orphan cleanup to remove DT
Copy test:
simple copy of DT
copy DT by content specified
closes: #7571
https://pulp.plan.io/issues/7571