Skip to content

Commit

Permalink
Apply suggestions from code review
Browse files Browse the repository at this point in the history
Co-authored-by: Martin "kokesak" Litwora <mlitwora@redhat.com>
  • Loading branch information
r0x0d and kokesak authored May 22, 2024
1 parent 195a9dd commit 7968237
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,11 @@ summary: |
tag+:
- non-destructive
- satellite

enabled: false
adjust+:
enabled: true
when: distro == centos-7
because: currently we have satellite key only for CentOS7
/enabled_repositories_after_analysis:
description: |
This test will perform the following operations:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,7 @@ def collect_enabled_repositories(shell):


@pytest.mark.test_enabled_repositories_after_analysis
@pytest.mark.parametrize("satellite_registration", ["RHEL7_AND_CENTOS7_SAT_REG"], indirect=True)
def test_enabled_repositories_after_analysis(shell, convert2rhel, satellite_registration):
"""Test analysis systems not connected to the Internet but requiring sub-mgr (e.g. managed by Satellite).
Expand All @@ -33,6 +34,8 @@ def test_enabled_repositories_after_analysis(shell, convert2rhel, satellite_regi
enabled_repositories_prior_analysis = collect_enabled_repositories(shell)

with convert2rhel("analyze -y --debug") as c2r:
c2r.expect("Enabling RHEL repositories:")
c2r.expect("rhel-7-server-rpms")
c2r.expect("Rollback: Enabling RHSM repositories")

assert c2r.exitstatus == 0
Expand Down

0 comments on commit 7968237

Please sign in to comment.