From fcac8182e33971aef3fe08d49fd98c4e77b26f3b Mon Sep 17 00:00:00 2001 From: dosas Date: Wed, 21 Aug 2024 15:06:50 +0200 Subject: [PATCH] Fix cli host tests by applying pattern from similar test (#15984) --- tests/foreman/cli/test_host.py | 17 ++++++++++++----- 1 file changed, 12 insertions(+), 5 deletions(-) diff --git a/tests/foreman/cli/test_host.py b/tests/foreman/cli/test_host.py index 489f7386103..6eaa3d4309e 100644 --- a/tests/foreman/cli/test_host.py +++ b/tests/foreman/cli/test_host.py @@ -1221,7 +1221,9 @@ def test_positive_view_parameter_by_non_admin_user(target_sat, function_host, fu {'host-id': function_host['id'], 'name': param_name, 'value': param_value} ) host = target_sat.cli.Host.info({'id': function_host['id']}) - assert host['parameters'][param_name] == param_value + assert (param_name, param_value) in [ + (param['name'], param['value']) for param in host['parameters'] + ] role = target_sat.api.Role(name=gen_string('alphanumeric')).create() target_sat.cli_factory.add_role_permissions( role.id, @@ -1235,8 +1237,9 @@ def test_positive_view_parameter_by_non_admin_user(target_sat, function_host, fu host = target_sat.cli.Host.with_user( username=function_user['user'].login, password=function_user['password'] ).info({'id': host['id']}) - assert param_name in host['parameters'] - assert host['parameters'][param_name] == param_value + assert (param_name, param_value) in [ + (param['name'], param['value']) for param in host['parameters'] + ] @pytest.mark.cli_host_parameter @@ -1268,7 +1271,9 @@ def test_negative_edit_parameter_by_non_admin_user(target_sat, function_host, fu {'host-id': function_host['id'], 'name': param_name, 'value': param_value} ) host = target_sat.cli.Host.info({'id': function_host['id']}) - assert host['parameters'][param_name] == param_value + assert (param_name, param_value) in [ + (param['name'], param['value']) for param in host['parameters'] + ] role = target_sat.api.Role(name=gen_string('alphanumeric')).create() target_sat.cli_factory.add_role_permissions( role.id, @@ -1287,7 +1292,9 @@ def test_negative_edit_parameter_by_non_admin_user(target_sat, function_host, fu {'host-id': function_host['id'], 'name': param_name, 'value': param_new_value} ) host = target_sat.cli.Host.info({'id': function_host['id']}) - assert host['parameters'][param_name] == param_value + assert (param_name, param_value) in [ + (param['name'], param['value']) for param in host['parameters'] + ] @pytest.mark.cli_host_parameter