Skip to content

Commit

Permalink
[6.15.z] CV already published + single LCE (#14181)
Browse files Browse the repository at this point in the history
Fix the test. CV is already published and promoted by setup_org_for_a_rh_repo. Also, don't use 2 LCEs needlessly.
  • Loading branch information
lhellebr authored Mar 4, 2024
1 parent 78d3648 commit 8489712
Showing 1 changed file with 22 additions and 18 deletions.
40 changes: 22 additions & 18 deletions tests/foreman/cli/test_host.py
Original file line number Diff line number Diff line change
Expand Up @@ -2012,22 +2012,31 @@ def test_negative_without_attach(
assert "This system has no repositories available through subscriptions." in repo_list.stdout


@pytest.mark.parametrize('sca_enabled', [True, False], ids=['with_sca', 'without_sca'])
@pytest.mark.cli_host_subscription
@pytest.mark.tier3
def test_negative_without_attach_with_lce(
target_sat, host_subscription_client, function_org, function_lce
target_sat,
host_subscription_client,
function_org,
function_lce,
sca_enabled,
):
"""Attempt to enable a repository of a subscription that was not
attached to a host
attached to a host.
This test is not using the host_subscription entities except
subscription_name and repository_id
:id: fc469e70-a7cb-4fca-b0ea-3c9e3dfff849
:expectedresults: repository not enabled on host
:expectedresults: Repository enabled with SCA. Failure without SCA.
:parametrized: yes
"""
if sca_enabled:
function_org.sca_enable()
else:
function_org.sca_disable()
content_view = target_sat.api.ContentView(organization=function_org).create()
ak = target_sat.api.ActivationKey(
environment=function_lce,
Expand All @@ -2036,8 +2045,8 @@ def test_negative_without_attach_with_lce(
target_sat.cli_factory.setup_org_for_a_rh_repo(
{
'product': PRDS['rhel'],
'repository-set': REPOSET['rhst7'],
'repository': REPOS['rhst7']['name'],
'repository-set': REPOSET['rhsclient7'],
'repository': REPOS['rhsclient7']['name'],
'organization-id': function_org.id,
'content-view-id': content_view.id,
'lifecycle-environment-id': function_lce.id,
Expand All @@ -2046,27 +2055,22 @@ def test_negative_without_attach_with_lce(
},
force_use_cdn=True,
)
host_lce = target_sat.api.LifecycleEnvironment(organization=function_org).create()
# refresh content view data
content_view.publish()
content_view.read().version[-1].promote(data={'environment_ids': host_lce.id, 'force': False})

# register client
host_subscription_client.register_contenthost(
function_org.name,
lce=f'{host_lce.name}/{content_view.name}',
lce=f'{function_lce.name}/{content_view.name}',
auto_attach=False,
)

# get list of available subscriptions which are matched with default subscription
subscriptions = host_subscription_client.run(
f'subscription-manager list --available --matches "{DEFAULT_SUBSCRIPTION_NAME}" --pool-only'
)
pool_id = subscriptions.stdout.strip()
# attach to plain RHEL subsctiption
host_subscription_client.subscription_manager_attach_pool([pool_id])
assert host_subscription_client.subscribed
host_subscription_client.enable_repo(REPOS['rhst7']['id'])
res = host_subscription_client.enable_repo(REPOS['rhsclient7']['id'])
if sca_enabled:
assert res.status == 0
assert f"Repository '{REPOS['rhsclient7']['id']}' is enabled for this system." in res.stdout
else:
assert res.status == 1
assert 'This system has no repositories available through subscriptions' in res.stdout


@pytest.mark.e2e
Expand Down

0 comments on commit 8489712

Please sign in to comment.