diff options
author | stark, steven <steven.stark@att.com> | 2018-12-17 12:43:02 -0800 |
---|---|---|
committer | stark, steven <steven.stark@att.com> | 2018-12-17 13:04:00 -0800 |
commit | 1f4df7c7ad27b23773ad9cdbe4db1632ce388cf1 (patch) | |
tree | 8092104f8be23051ff81c9f71ee34116df4d33ba /ice_validator/tests/test_nova_servers_vm_types_use_get_param.py | |
parent | ca9085f0f77d442d3741a8c754e65cc45b6a318d (diff) |
[VVP] updating validation scripts in dublin
- adding backlog of new validation scripts for dublin
- updating existing tests
- removing outdated tests
Issue-ID: VVP-123
Change-Id: Ib8260889ac957c1dd28d8ede450fc8edc6fb0ec0
Signed-off-by: stark, steven <steven.stark@att.com>
Diffstat (limited to 'ice_validator/tests/test_nova_servers_vm_types_use_get_param.py')
-rw-r--r-- | ice_validator/tests/test_nova_servers_vm_types_use_get_param.py | 22 |
1 files changed, 14 insertions, 8 deletions
diff --git a/ice_validator/tests/test_nova_servers_vm_types_use_get_param.py b/ice_validator/tests/test_nova_servers_vm_types_use_get_param.py index 6b1a668..2f69b52 100644 --- a/ice_validator/tests/test_nova_servers_vm_types_use_get_param.py +++ b/ice_validator/tests/test_nova_servers_vm_types_use_get_param.py @@ -2,7 +2,7 @@ # ============LICENSE_START======================================================= # org.onap.vvp/validation-scripts # =================================================================== -# Copyright © 2018 AT&T Intellectual Property. All rights reserved. +# Copyright © 2017 AT&T Intellectual Property. All rights reserved. # =================================================================== # # Unless otherwise specified, all software contained herein is licensed @@ -41,11 +41,14 @@ import pytest from tests import cached_yaml as yaml +from .helpers import validates + +@validates("R-901331", "R-481670", "R-663631") def test_vm_type_assignments_on_nova_servers_only_use_get_param(heat_template): - ''' + """ Make sure all nova servers only use get_param for their properties - ''' + """ with open(heat_template) as fh: yml = yaml.load(fh) @@ -54,7 +57,7 @@ def test_vm_type_assignments_on_nova_servers_only_use_get_param(heat_template): pytest.skip("No resources specified in the heat template") key_values = ["name", "flavor", "image"] - invalid_nova_servers = [] + invalid_nova_servers = set() for k, v in yml["resources"].items(): if not isinstance(v, dict): @@ -69,8 +72,11 @@ def test_vm_type_assignments_on_nova_servers_only_use_get_param(heat_template): for k2, v2 in v["properties"].items(): if k2 in key_values: if not isinstance(v2, dict): - invalid_nova_servers.append(k) + invalid_nova_servers.add(k) elif "get_param" not in v2: - invalid_nova_servers.append(k) - - assert not set(invalid_nova_servers) + invalid_nova_servers.add(k) + msg = ( + "These OS::Nova::Server resources do not derive one or more of " + + "their {} properties via get_param: {}" + ).format(", ".join(key_values), ", ".join(invalid_nova_servers)) + assert not invalid_nova_servers, msg |