X-Git-Url: https://gerrit.onap.org/r/gitweb?a=blobdiff_plain;f=ice_validator%2Ftests%2Ftest_volume_templates.py;h=634da5c58fa729e68e3a580aac98267857771b17;hb=0c4e64d87728b89aa9cd4d41d738f5bfe64ceee3;hp=6d2a44c053e319ce0be53dd3463c4095b85b55af;hpb=60d5ad7d00eadd6395eca186e6fa76a43df3c6cf;p=vvp%2Fvalidation-scripts.git

diff --git a/ice_validator/tests/test_volume_templates.py b/ice_validator/tests/test_volume_templates.py
index 6d2a44c..634da5c 100644
--- a/ice_validator/tests/test_volume_templates.py
+++ b/ice_validator/tests/test_volume_templates.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
@@ -35,33 +35,60 @@
 #
 # ============LICENSE_END============================================
 #
-# ECOMP is a trademark and service mark of AT&T Intellectual Property.
 #
-
 from tests import cached_yaml as yaml
+
+from .helpers import validates
+from .utils.nested_files import get_list_of_nested_files
+
+import os
 import pytest
 
 
-def test_volume_templates_only_contains_cinder(volume_template):
-    '''
+@validates("R-270358")
+def test_volume_templates_contains_cinder_or_resource_group(volume_template):
+    """
     Check that all templates marked as volume templates are
     in fact volume templates
-    '''
+    """
+    acceptable_resources = []
+    dirname = os.path.dirname(volume_template)
+    list_of_files = get_list_of_nested_files(volume_template, dirname)
+
+    list_of_files.append(volume_template)
+
+    for file in list_of_files:
+        with open(file) as fh:
+            yml = yaml.load(fh)
+        resources = yml.get("resources") or {}
+        for k, v in resources.items():
+            if not isinstance(v, dict):
+                continue
+            if "type" not in v:
+                continue
+            if v["type"] in ["OS::Cinder::Volume", "OS::Heat::ResourceGroup"]:
+                acceptable_resources.append(k)
+
+    assert acceptable_resources, (
+        "No OS::Cinder::Volume or OS::Heat::ResourceGroup resources "
+        "found in volume module"
+    )
+
+
+@validates("R-55306")
+def test_no_vf_module_index_in_cinder(volume_template):
+    """
+    vf_module_index is prohibited in volume templates
+    """
+
     with open(volume_template) as fh:
         yml = yaml.load(fh)
 
-    # skip if resources are not defined
-    if "resources" not in yml:
-        pytest.skip("No resources specified in the heat template")
-
-    non_volumes = []
-    for k, v in yml["resources"].items():
-        if not isinstance(v, dict):
-            continue
-        if "type" not in v:
-            continue
-        if v["type"] not in ["OS::Cinder::Volume",
-                             "OS::Heat::ResourceGroup"]:
-            non_volumes.append(k)
+    if "parameters" not in yml:
+        pytest.skip("No parameters specified in the heat template")
 
-    assert not set(non_volumes)
+    parameters = yml.get("parameters")
+    if parameters and isinstance(parameters, dict):
+        assert (
+            "vf_module_index" not in parameters
+        ), "{} must not use vf_module_index as a parameter".format(volume_template)