X-Git-Url: https://gerrit.onap.org/r/gitweb?a=blobdiff_plain;f=ice_validator%2Ftests%2Ftest_required_parameters_no_constraints.py;h=7070178477460bcbd0e547a315b007bc2bb83f13;hb=dd0d480978570b4222206bc04edd271c39b88b44;hp=247eab67aaa0143689da0a2e46b07746936ca42c;hpb=0feaec6790274f87c2d35a039676a3b1c2702a4d;p=vvp%2Fvalidation-scripts.git

diff --git a/ice_validator/tests/test_required_parameters_no_constraints.py b/ice_validator/tests/test_required_parameters_no_constraints.py
index 247eab6..7070178 100644
--- a/ice_validator/tests/test_required_parameters_no_constraints.py
+++ b/ice_validator/tests/test_required_parameters_no_constraints.py
@@ -6,7 +6,7 @@
 # ===================================================================
 #
 # Unless otherwise specified, all software contained herein is licensed
-# under the Apache License, Version 2.0 (the “License”);
+# under the Apache License, Version 2.0 (the "License");
 # you may not use this software except in compliance with the License.
 # You may obtain a copy of the License at
 #
@@ -21,7 +21,7 @@
 #
 #
 # Unless otherwise specified, all documentation contained herein is licensed
-# under the Creative Commons License, Attribution 4.0 Intl. (the “License”);
+# under the Creative Commons License, Attribution 4.0 Intl. (the "License");
 # you may not use this documentation except in compliance with the License.
 # You may obtain a copy of the License at
 #
@@ -35,32 +35,63 @@
 #
 # ============LICENSE_END============================================
 #
-# ECOMP is a trademark and service mark of AT&T Intellectual Property.
 #
-from .helpers import validates
 
 import pytest
-import yaml
+from tests import cached_yaml as yaml
+
+from .helpers import validates
 
 
-@validates('R-55218', 'R-98374', 'R-44318')
-def test_required_parameters_no_constraints(yaml_file):
-    '''
-    Make sure all required parameters are specified without any
-    constraints in the heat template.
-    '''
-    required_parameters = ["vnf_id", "vf_module_id", "vnf_name"]
+def check_parameters_no_constraints(yaml_file, parameter):
 
     with open(yaml_file) as fh:
         yml = yaml.load(fh)
 
-    # skip if parameters are not defined
-    if "parameters" not in yml:
-        pytest.skip("No parameters specified in the heat template")
+    param = yml.get("parameters", {}).get(parameter)
+    if not param:
+        pytest.skip("Parameter {} not defined in parameters section".format(parameter))
+
+    assert (
+        "constraints" not in param
+    ), "Found constraints defined for parameter: {}".format(parameter)
+
+
+@validates("R-55218")
+def test_vnf_id_parameter_has_no_constraints(yaml_file):
+    check_parameters_no_constraints(yaml_file, "vnf_id")
+
+
+@validates("R-98374")
+def test_vf_module_id_parameter_has_no_constraints(yaml_file):
+    check_parameters_no_constraints(yaml_file, "vf_module_id")
+
+
+@validates("R-44318")
+def test_vnf_name_parameter_has_no_constraints(yaml_file):
+    check_parameters_no_constraints(yaml_file, "vnf_name")
+
+
+@validates("R-34055")
+def test_workload_context_parameter_has_no_constraints(yaml_file):
+    check_parameters_no_constraints(yaml_file, "workload_context")
+
+
+@validates("R-56183")
+def test_environment_context_parameter_has_no_constraints(yaml_file):
+    check_parameters_no_constraints(yaml_file, "environment_context")
+
+
+@validates("R-15480")
+def test_vf_module_name_parameter_has_no_constraints(yaml_file):
+    check_parameters_no_constraints(yaml_file, "vf_module_name")
+
+
+@validates("R-67597")
+def test_vm_role_parameter_has_no_constraints(yaml_file):
+    check_parameters_no_constraints(yaml_file, "vm_role")
 
-    invalid_params = []
-    for k1, v1 in yml["parameters"].items():
-        if k1 in required_parameters and "constraints" in v1:
-            invalid_params.append(k1)
 
-    assert not set(invalid_params)
+@validates("R-09811")
+def test_vf_module_index_parameter_has_no_constraints(yaml_file):
+    check_parameters_no_constraints(yaml_file, "vf_module_index")