From 0b16fc0f564a55f925fa220929ca1b9aa2c54e66 Mon Sep 17 00:00:00 2001 From: Vladimir Sedmik Date: Tue, 11 Jun 2024 15:15:36 +0200 Subject: [PATCH] Update loop vars to be less confusing --- tests/foreman/api/test_repository.py | 4 ++-- tests/upgrades/test_repository.py | 6 +++--- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/tests/foreman/api/test_repository.py b/tests/foreman/api/test_repository.py index a4454aff104..4dde7744e40 100644 --- 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 index 7568df1a3f2..ddb1d85345a 100644 --- 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)