From e2b5191d127ab0e8c019112c2e75238d76a01897 Mon Sep 17 00:00:00 2001 From: Mamtha Date: Wed, 30 Oct 2019 23:51:35 +0530 Subject: Sonar fix for vvp 291 and 292 Issue-ID: VVP-291 Change-Id: I4d2eebbb9d4d96b17cc09a5de34e95e41cb9ec7d Signed-off-by: Mamtha --- .../tests/test_nova_servers_environment_context.py | 19 ++++--------------- 1 file changed, 4 insertions(+), 15 deletions(-) (limited to 'ice_validator/tests/test_nova_servers_environment_context.py') diff --git a/ice_validator/tests/test_nova_servers_environment_context.py b/ice_validator/tests/test_nova_servers_environment_context.py index 0da97f2..780733a 100644 --- a/ice_validator/tests/test_nova_servers_environment_context.py +++ b/ice_validator/tests/test_nova_servers_environment_context.py @@ -43,7 +43,8 @@ import pytest from tests import cached_yaml as yaml -from .helpers import validates +from .helpers import validates, get_param + VERSION = "1.0.0" @@ -89,17 +90,5 @@ def validate_metadata(metadata, parameters): Return error message string or None if no errors. """ for value in metadata.values(): - if isinstance(value, dict): - if "get_param" in value: - if value["get_param"] == "environment_context": - wc = parameters.get("environment_context", {}) - if wc.get("type") == "string": - break - else: - return ( - 'must have parameter "environment_context"' - ' of type "string"' - ) - break - else: - return None + if get_param(value) == "environment_context" and parameters.get("environment_context", {}).get("type", "") != "string": + return 'must have parameter "environment_context" of type "string"' -- cgit 1.2.3-korg