Skip to content

Commit

Permalink
updating simplecontentaccess disable method
Browse files Browse the repository at this point in the history
  • Loading branch information
ColeHiggins2 committed Jan 10, 2024
1 parent b9c36c8 commit 9e0e98b
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
4 changes: 3 additions & 1 deletion robottelo/cli/simple_content_access.py
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,9 @@ class SimpleContentAccess(Base):
def disable(cls, options=None, timeout=None):
"""Disable simple content access for a manifest"""
cls.command_sub = 'disable'
return cls.execute(cls._construct_command(options), ignore_stderr=True, timeout=timeout)
return cls.execute(
cls._construct_command(options), return_raw_response=True, timeout=timeout
)

@classmethod
def enable(cls, options=None, timeout=None):
Expand Down
4 changes: 1 addition & 3 deletions tests/foreman/cli/test_subscription.py
Original file line number Diff line number Diff line change
Expand Up @@ -303,9 +303,7 @@ def test_positive_prepare_for_sca_only_hammer(function_org, target_sat):
'Simple Content Access will be required for all organizations in the next release'
in str(org_results.stderr)
)
sca_results = target_sat.execute(
f'hammer simple-content-access disable --organization-id {function_org.id}'
)
sca_results = target_sat.cli.SimpleContentAccess.disable({'organization-id': function_org.id})
assert (
'Simple Content Access will be required for all organizations in the next release'
in str(sca_results.stderr)
Expand Down

0 comments on commit 9e0e98b

Please sign in to comment.