diff options
author | Mamtha <mamtha.sabesan@in.ibm.com> | 2019-10-30 23:51:35 +0530 |
---|---|---|
committer | Mamtha <mamtha.sabesan@in.ibm.com> | 2019-10-30 23:55:18 +0530 |
commit | e2b5191d127ab0e8c019112c2e75238d76a01897 (patch) | |
tree | ce5e8c5fd8a527baf5d7dbd56cc3b8c469ed5ada /ice_validator/tests/test_nova_servers_environment_context.py | |
parent | d87106650286493af38cecd423b58bdcead15db4 (diff) |
Sonar fix for vvp 291 and 292
Issue-ID: VVP-291
Change-Id: I4d2eebbb9d4d96b17cc09a5de34e95e41cb9ec7d
Signed-off-by: Mamtha <mamtha.sabesan@in.ibm.com>
Diffstat (limited to 'ice_validator/tests/test_nova_servers_environment_context.py')
-rw-r--r-- | ice_validator/tests/test_nova_servers_environment_context.py | 19 |
1 files changed, 4 insertions, 15 deletions
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"' |