X-Git-Url: https://gerrit.onap.org/r/gitweb?a=blobdiff_plain;f=ice_validator%2Ftests%2Ftest_network_format.py;h=b291c535a3059f3af1b9dc066cf60180be64d673;hb=5706da0c7f430c01a0cebb534f1a965ee9b5cca7;hp=7fb343a2f442e892f4a56a47ed5449b38e32f4b1;hpb=60d5ad7d00eadd6395eca186e6fa76a43df3c6cf;p=vvp%2Fvalidation-scripts.git

diff --git a/ice_validator/tests/test_network_format.py b/ice_validator/tests/test_network_format.py
index 7fb343a..b291c53 100644
--- a/ice_validator/tests/test_network_format.py
+++ b/ice_validator/tests/test_network_format.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,41 +35,78 @@
 #
 # ============LICENSE_END============================================
 #
-# ECOMP is a trademark and service mark of AT&T Intellectual Property.
 #
+from itertools import chain
+
+import re
 
-import pytest
 from tests import cached_yaml as yaml
+from tests.structures import Heat
+
+from tests.helpers import validates
+
+RE_INTERNAL_NETWORK_RID = re.compile(r"int_(?P<network_role>.+)_network$")
+NETWORK_RESOURCE_TYPES = ["OS::Neutron::Net", "OS::ContrailV2::VirtualNetwork"]
 
-from .helpers import validates
-from .utils.network_roles import get_network_role_from_port, \
-    property_uses_get_resource
 
+@validates("R-16968")
+def test_network_resource_id_format(yaml_file):
+    heat = Heat(yaml_file)
+    network_ids = chain.from_iterable(
+        heat.get_resource_by_type(t) for t in NETWORK_RESOURCE_TYPES
+    )
+    invalid_networks = {
+        r_id for r_id in network_ids if not RE_INTERNAL_NETWORK_RID.match(r_id)
+    }
+    assert not invalid_networks, (
+        "Heat templates must only create internal networks "
+        "and their resource IDs must follow the format "
+        "int_{{network-role}}_network. The following network's resource IDs "
+        "have invalid resource ID formats: "
+        "{}".format(", ".join(invalid_networks))
+    )
 
-@validates('R-62983', 'R-86182')
-def test_network_format(heat_template):
-    '''
-    Make sure all network properties use the allowed naming
-    conventions
-    '''
-    with open(heat_template) as fh:
+
+@validates("R-16241")
+def test_network_has_subnet(yaml_file):
+    """
+    if creating internal network, make sure there is a
+    corresponding subnet that references it
+    """
+
+    with open(yaml_file) 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")
+    networks = []
 
-    invalid_ports = []
     for k, v in yml["resources"].items():
-        if not isinstance(v, dict):
-            continue
-        if "properties" not in v:
+        if not has_properties(v) or v.get("type") not in ["OS::Neutron::Net"]:
             continue
-        if property_uses_get_resource(v, "network"):
-            continue
-        if v.get("type") != "OS::Neutron::Port":
+        # need to check if contrail networks also require subnet
+        # and it is defined the same as neutron networks
+        # if v.get("type") not in NETWORK_RESOURCE_TYPES:
+        networks.append(k)
+
+    for k, v in yml["resources"].items():
+        network_prop = v.get("properties", {}).get("network", {}).get("get_resource")
+        if (
+            not has_properties(v)
+            and v.get("type") != "OS::Neutron::Subnet"
+            and not network_prop
+        ):
             continue
-        if not get_network_role_from_port(v):
-            invalid_ports.append(k)
+        x = 0
+        for network in networks:
+            if network == network_prop:
+                networks.pop(x)
+                break
+            x += 1
+
+    assert not networks, "Networks detected without subnet {}".format(networks)
+
 
-    assert not set(invalid_ports)
+def has_properties(resource):
+    """
+    checks resource is a Neutron Subnet
+    """
+    return isinstance(resource, dict) and "properties" in resource