From 815a65d91bb75131151526041efaef28b50a0b6b Mon Sep 17 00:00:00 2001 From: "stark, steven" Date: Tue, 5 Feb 2019 10:40:55 -0800 Subject: [PATCH] [VVP] revert nested resource section - R-40551 specifies nested templates MAY or MAY NOT contain a resource section - remove check for get_param in allowed_address_pairs, no requirement states get_param MUST be used Change-Id: If133b4ec4d39c25a95c77482b1743562e2ae74a8 Issue-ID: VVP-158 Signed-off-by: stark, steven --- ice_validator/tests/test_heat_template_structure.py | 6 +++--- ice_validator/tests/test_initial_configuration.py | 4 ++-- ice_validator/tests/test_neutron_port_addresses.py | 2 +- 3 files changed, 6 insertions(+), 6 deletions(-) diff --git a/ice_validator/tests/test_heat_template_structure.py b/ice_validator/tests/test_heat_template_structure.py index a988aea..192f45a 100644 --- a/ice_validator/tests/test_heat_template_structure.py +++ b/ice_validator/tests/test_heat_template_structure.py @@ -97,18 +97,18 @@ def test_heat_template_structure_contains_parameters(yaml_file): @validates("R-23664") -def test_heat_template_structure_contains_resources(yaml_file): +def test_heat_template_structure_contains_resources(heat_template): """ Check that all heat templates have the required sections """ required_key_values = ["resources"] - with open(yaml_file) as fh: + with open(heat_template) as fh: yml = yaml.load(fh) assert all( [k in yml for k in required_key_values] ), "{} doesn't contain the {} section, but it is required".format( - yaml_file, required_key_values[0] + heat_template, required_key_values[0] ) diff --git a/ice_validator/tests/test_initial_configuration.py b/ice_validator/tests/test_initial_configuration.py index 3ecfb59..83f887c 100644 --- a/ice_validator/tests/test_initial_configuration.py +++ b/ice_validator/tests/test_initial_configuration.py @@ -184,11 +184,11 @@ def test_05_all_get_param_have_defined_parameter(yaml_file): @validates("R-90152") @pytest.mark.base -def test_06_heat_template_resource_section_has_resources(yaml_file): +def test_06_heat_template_resource_section_has_resources(heat_template): found_resource = False - with open(yaml_file) as fh: + with open(heat_template) as fh: yml = yaml.load(fh) resources = yml.get("resources") diff --git a/ice_validator/tests/test_neutron_port_addresses.py b/ice_validator/tests/test_neutron_port_addresses.py index 05b0223..b5ce375 100644 --- a/ice_validator/tests/test_neutron_port_addresses.py +++ b/ice_validator/tests/test_neutron_port_addresses.py @@ -163,7 +163,7 @@ def validate_field(heat, allowed_address_pairs, field, v6=False): if field in allowed_address_pair: param = heat.nested_get(allowed_address_pair, field, "get_param") if param is None: - error = 'allowed_address_pair %s requires "get_param"' % field + # error = 'allowed_address_pair %s requires "get_param"' % field break else: # if v6 and testing v6, or inverse -- 2.16.6