Update loop vars to be less confusing

SatelliteQE · Jun 11, 2024 · 0b16fc0 · 0b16fc0
1 parent 2ae351d
commit 0b16fc0
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/tests/foreman/api/test_repository.py b/tests/foreman/api/test_repository.py
@@ -1850,11 +1850,11 @@ def test_positive_synchronize_docker_repo_with_included_tags(self, repo_options,
             [
                 {
                     'content_type': 'docker',
-                    'docker_upstream_name': repo['upstream_name'],
+                    'docker_upstream_name': item['upstream_name'],
                     'name': gen_string('alpha'),
                     'url': constants.PULP_CONTAINER_REGISTRY_HUB,
                 }
-                for repo in LABELLED_REPOS
+                for item in LABELLED_REPOS
             ]
         ),
         indirect=True,

diff --git a/tests/upgrades/test_repository.py b/tests/upgrades/test_repository.py
@@ -412,17 +412,17 @@ def test_pre_container_repo_sync(
         :expectedresults: Container repositories are synced and ready for upgrade.
         """
         repos = dict()
-        for model in LABELLED_REPOS:
+        for item in LABELLED_REPOS:
             repo = target_sat.api.Repository(
                 content_type='docker',
-                docker_upstream_name=model['upstream_name'],
+                docker_upstream_name=item['upstream_name'],
                 product=module_product,
                 url=PULP_CONTAINER_REGISTRY_HUB,
             ).create()
             repo.sync()
             repo = repo.read()
             assert repo.content_counts['docker_manifest'] > 0
-            repos[model['upstream_name']] = repo.id
+            repos[item['upstream_name']] = repo.id
         save_test_data(repos)
 
     @pytest.mark.post_upgrade(depend_on=test_pre_container_repo_sync)