From 795432337a7ee8703f87b24ec87f6a7bc44de69c Mon Sep 17 00:00:00 2001
From: vsedmik <46570670+vsedmik@users.noreply.github.com>
Date: Mon, 6 Jan 2025 22:03:52 +0100
Subject: [PATCH] Fix legacy errata upgrade scenario tests (#17183)

Fix errata upgrade scenario tests

(cherry picked from commit caff4dd5b45db3bf6a135184053253634eebbd60)
---
 robottelo/host_helpers/satellite_mixins.py | 5 ++++-
 tests/upgrades/test_errata.py              | 4 +++-
 tests/upgrades/test_subscription.py        | 4 +++-
 3 files changed, 10 insertions(+), 3 deletions(-)

diff --git a/robottelo/host_helpers/satellite_mixins.py b/robottelo/host_helpers/satellite_mixins.py
index 6d4ebdda0dc..0f8d25e88ee 100644
--- a/robottelo/host_helpers/satellite_mixins.py
+++ b/robottelo/host_helpers/satellite_mixins.py
@@ -5,6 +5,7 @@
 import random
 import re
 
+from fauxfactory import gen_string
 import requests
 from wait_for import TimedOutError, wait_for
 
@@ -181,15 +182,17 @@ def is_sca_mode_enabled(self, org_id):
         """
         return self.api.Organization(id=org_id).read().simple_content_access
 
-    def publish_content_view(self, org, repo_list, name):
+    def publish_content_view(self, org, repo_list, name=None):
         """This method publishes the content view for a given organization and repository list.
 
         :param str org: The name of the organization to which the content view belongs
         :param list or str repo_list:  A list of repositories or a single repository
+        :param str name: Name of the Content View to create. Defaults to random string.
 
         :return: A dictionary containing the details of the published content view.
         """
         repo = repo_list if isinstance(repo_list, list) else [repo_list]
+        name = name or gen_string('alpha')
         content_view = self.api.ContentView(organization=org, repository=repo, name=name).create()
         content_view.publish()
         return content_view.read()
diff --git a/tests/upgrades/test_errata.py b/tests/upgrades/test_errata.py
index 1425d341158..d7138ab1949 100644
--- a/tests/upgrades/test_errata.py
+++ b/tests/upgrades/test_errata.py
@@ -186,7 +186,9 @@ def test_pre_scenario_generate_errata_for_client(
             }
         )
 
-    @pytest.mark.parametrize('pre_upgrade_data', ['rhel7', 'rhel8', 'rhel9'], indirect=True)
+    @pytest.mark.parametrize(
+        'pre_upgrade_data', ['rhel7-ipv4', 'rhel8-ipv4', 'rhel9-ipv4'], indirect=True
+    )
     @pytest.mark.post_upgrade(depend_on=test_pre_scenario_generate_errata_for_client)
     def test_post_scenario_errata_count_installation(self, target_sat, pre_upgrade_data):
         """Post-upgrade scenario that applies errata on the RHEL client that was set up
diff --git a/tests/upgrades/test_subscription.py b/tests/upgrades/test_subscription.py
index cbf34dd05a6..b89876ac465 100644
--- a/tests/upgrades/test_subscription.py
+++ b/tests/upgrades/test_subscription.py
@@ -143,7 +143,9 @@ def test_pre_subscription_scenario_auto_attach(
             }
         )
 
-    @pytest.mark.parametrize('pre_upgrade_data', ['rhel7', 'rhel8', 'rhel9'], indirect=True)
+    @pytest.mark.parametrize(
+        'pre_upgrade_data', ['rhel7-ipv4', 'rhel8-ipv4', 'rhel9-ipv4'], indirect=True
+    )
     @pytest.mark.post_upgrade(depend_on=test_pre_subscription_scenario_auto_attach)
     def test_post_subscription_scenario_auto_attach(self, request, target_sat, pre_upgrade_data):
         """Run subscription auto-attach on pre-upgrade content host registered