X-Git-Url: https://gerrit.onap.org/r/gitweb?a=blobdiff_plain;f=ice_validator%2Ftests%2Ftest_cloud_config_resource_id.py;h=c78fe711570636b58d499811c77c32f2a66ac0ee;hb=00c64458942602efe5a3b712540b98ac94fb60f4;hp=bd04e3776620fe1ea7f6405c03736e8560dc444a;hpb=1af0d577ab6d8c431ae1322657c50efd5e0a1a93;p=vvp%2Fvalidation-scripts.git diff --git a/ice_validator/tests/test_cloud_config_resource_id.py b/ice_validator/tests/test_cloud_config_resource_id.py index bd04e37..c78fe71 100644 --- a/ice_validator/tests/test_cloud_config_resource_id.py +++ b/ice_validator/tests/test_cloud_config_resource_id.py @@ -35,13 +35,12 @@ # # ============LICENSE_END============================================ # -# ECOMP is a trademark and service mark of AT&T Intellectual Property. # -''' +""" A VNF's Heat Orchestration Template's Resource ``OS::Heat::CloudConfig`` Resource ID **MUST** contain the ``{vm-type}``. -''' +""" import pytest @@ -49,24 +48,24 @@ from .structures import Heat from .helpers import validates from .utils import vm_types -VERSION = '1.0.0' +VERSION = "1.0.0" -@validates('R-04747') -def test_cloud_config(heat_template): - '''validate resource ids - ''' - h = Heat(filepath=heat_template) +@validates("R-04747") +def test_cloud_config(yaml_file): + """validate resource ids + """ + h = Heat(filepath=yaml_file) if not h.resources: - pytest.skip('No resources in this template') + pytest.skip("No resources in this template") cloud_configs = get_cloud_configs(h) if not cloud_configs: - pytest.skip('No CloudConfig resources in this template') + pytest.skip("No CloudConfig resources in this template") resource_vm_types = vm_types.get_vm_types(h.resources) if not resource_vm_types: - pytest.skip('No resources with {vm-type} in this template') + pytest.skip("No resources with {vm-type} in this template") bad = set() for rid in cloud_configs: @@ -75,13 +74,17 @@ def test_cloud_config(heat_template): break else: bad.add(rid) - assert not bad, 'CloudConfigs %s have {vm-type} not in %s' % ( - list(bad), - list(resource_vm_types)) + assert not bad, "CloudConfigs %s have {vm-type} not in %s" % ( + list(bad), + list(resource_vm_types), + ) def get_cloud_configs(heat): """Return list of resource_id whose type is OS::Heat::CloudConfig. """ - return [rid for rid, resource in heat.resources.items() - if heat.nested_get(resource, 'type') == 'OS::Heat::CloudConfig'] + return [ + rid + for rid, resource in heat.resources.items() + if heat.nested_get(resource, "type") == "OS::Heat::CloudConfig" + ]