Skip to content
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.

Sign up for GitHub

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 steps, refresh uploaded manifest file #14195

Merged
merged 1 commit into from
Feb 28, 2024
Merged

add test steps, refresh uploaded manifest file #14195

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
16 changes: 14 additions & 2 deletions tests/foreman/api/test_repository.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1521,14 +1521,22 @@ def test_positive_sync_kickstart_check_os(
),
indirect=True,
)
def test_missing_content_id(self, repo):
def test_missing_content_id(self, repo, function_sca_manifest_org, target_sat):
"""Handle several cases of missing content ID correctly

:id: f507790a-933b-4b3f-ac93-cade6967fbd2

:parametrized: yes

:expectedresults: Repository URL can be set to something new and the repo can be deleted
:setup:
1. Create product and repo, sync repo

:steps:
1. Try to update repo URL
2. Attempt to delete repo
3. Refresh manifest file

:expectedresults: Repo URL can be updated, repo can be deleted and manifest refresh works after repo delete

:BZ:2032040
"""
Expand All @@ -1546,6 +1554,10 @@ def test_missing_content_id(self, repo):
repo.delete()
with pytest.raises(HTTPError):
repo.read()
output = target_sat.cli.Subscription.refresh_manifest(
{'organization-id': function_sca_manifest_org.id}
)
assert 'Candlepin job status: SUCCESS' in output, 'Failed to refresh manifest'


class TestDockerRepository:
Expand Down
Loading