Skip to content

Commit

Permalink
Merge branch 'master' into vmware_compute_profile
Browse files Browse the repository at this point in the history
  • Loading branch information
shubhamsg199 authored Feb 28, 2024
2 parents eff6aa1 + b83e4f4 commit 429c14c
Show file tree
Hide file tree
Showing 2 changed files with 16 additions and 3 deletions.
3 changes: 2 additions & 1 deletion robottelo/cli/hammer.py
Original file line number Diff line number Diff line change
Expand Up @@ -49,8 +49,9 @@ def parse_csv(output):
# ignore warning about puppet and ostree deprecation
output.replace('Puppet and OSTree will no longer be supported in Katello 3.16\n', '')
is_rex = True if 'Job invocation' in output else False
is_pkg_list = True if 'Nvra' in output else False
# Validate if the output is eligible for CSV conversions else return as it is
if not is_csv(output) and not is_rex:
if not is_csv(output) and not is_rex and not is_pkg_list:
return output
output = output.splitlines()[0:2] if is_rex else output.splitlines()
reader = csv.reader(output)
Expand Down
16 changes: 14 additions & 2 deletions tests/foreman/api/test_repository.py
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

0 comments on commit 429c14c

Please sign in to comment.