-
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
Fix syncs from file:// repos and add a test #2034
Conversation
Attached issue: https://pulp.plan.io/issues/9021 Attached issue: https://pulp.plan.io/issues/7350 |
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.
Can we add a "Required PR:" for pulp/pulpcore#1463 as well?
@@ -128,6 +128,29 @@ def test_sync(self): | |||
self.assertEqual(latest_version_href, repo.latest_version_href) | |||
self.assertDictEqual(get_content_summary(repo.to_dict()), RPM_FIXTURE_SUMMARY) | |||
|
|||
def test_sync_local(self): |
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.
👍
nm, that PR has been merged so we should be good. |
|
||
remote = self.remote_api.create(gen_rpm_remote(url="file:///tmp/rpm-unsigned/")) | ||
|
||
self.do_test(remote=remote, mirror=True) |
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.
Missing cleanup - see
pulp_rpm/pulp_rpm/tests/functional/api/test_sync.py
Lines 116 to 119 in d87b377
repo, remote = self.do_test() | |
self.addCleanup(self.repo_api.delete, repo.pulp_href) | |
self.addCleanup(self.remote_api.delete, remote.pulp_href) |
closes: #9021 https://pulp.plan.io/issues/9021 closes: #7350 https://pulp.plan.io/issues/7350
closes: #9021
https://pulp.plan.io/issues/9021
closes: #7350
https://pulp.plan.io/issues/7350