aboutsummaryrefslogtreecommitdiffstats
path: root/ice_validator/tests/test_heat_parameter_section.py
diff options
context:
space:
mode:
authorLovett, Trevor (tl2972) <tl2972@att.com>2018-08-22 11:13:45 -0500
committerLovett, Trevor (tl2972) <tl2972@att.com>2018-08-22 12:52:32 -0500
commitf5edc06be0d8bedeb0904b348ba5e3e67c74f186 (patch)
tree5fba01ca97aebe14f4b7004551bc58c17a858169 /ice_validator/tests/test_heat_parameter_section.py
parent1af0d577ab6d8c431ae1322657c50efd5e0a1a93 (diff)
[VVP] Added new three new reports
Resolves VVP-81 by introducing three new reports to the validation script output. Each report is written to the output directory under ice_validator. * mapping_errors.csv - each row represents an issue where a test is mapped to a requirement that does not exist in the requirement.json file * traceability.csv - Shows a mapping between all test cases and requirements. Rows will also be shown where a requirement does not have a tests, and where tests are not mapped to a requirement * report.{html|csv|xlsx} - Formatted error report that shows each failure that was detected and maps it to a the file name, requirement(s) violated if known, possible resolution steps, error message, and raw output from pytest. The report can be written in html, csv, or Excel format Other minor changes: * Replaced smart quotes with regular quotes to resolve issues with py3 * Updated copyright of modified files * Fixed Flake-8 errors Issue-ID: VVP-81 Change-Id: I659836cb20262ae44652c03639281a817dd051ae Signed-off-by: Lovett, Trevor (tl2972) <tl2972@att.com>
Diffstat (limited to 'ice_validator/tests/test_heat_parameter_section.py')
-rw-r--r--ice_validator/tests/test_heat_parameter_section.py38
1 files changed, 18 insertions, 20 deletions
diff --git a/ice_validator/tests/test_heat_parameter_section.py b/ice_validator/tests/test_heat_parameter_section.py
index 2aab7d3..dde4176 100644
--- a/ice_validator/tests/test_heat_parameter_section.py
+++ b/ice_validator/tests/test_heat_parameter_section.py
@@ -2,11 +2,11 @@
# ============LICENSE_START=======================================================
# org.onap.vvp/validation-scripts
# ===================================================================
-# Copyright © 2017 AT&T Intellectual Property. All rights reserved.
+# Copyright © 2018 AT&T Intellectual Property. All rights reserved.
# ===================================================================
#
# 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
#
@@ -43,16 +43,15 @@ import re
import yaml
# one or more (alphanumeric or underscore)
-RE_VALID_PARAMETER_NAME = re.compile(r'[\w_]+$')
+RE_VALID_PARAMETER_NAME = re.compile(r"[\w_]+$")
def test_parameter_valid_keys(yaml_file):
- '''
+ """
Make sure each parameter is a dict and only contain
valid keys
- '''
- key_values = ["type", "label", "description",
- "hidden", "constraints", "immutable"]
+ """
+ key_values = ["type", "label", "description", "hidden", "constraints", "immutable"]
with open(yaml_file) as fh:
yml = yaml.load(fh)
@@ -72,11 +71,11 @@ def test_parameter_valid_keys(yaml_file):
assert not set(invalid_params)
-@validates('R-90526')
+@validates("R-90526")
def test_default_values(yaml_file):
- '''
+ """
Make sure no default values are set for any parameter.
- '''
+ """
with open(yaml_file) as fh:
yml = yaml.load(fh)
@@ -88,19 +87,19 @@ def test_default_values(yaml_file):
for v1 in yml["parameters"].values():
if not isinstance(v1, dict):
continue
- if any(k == 'default' for k in v1):
+ if any(k == "default" for k in v1):
invalid_params.append(str(v1))
assert not set(invalid_params)
-@validates('R-25877')
+@validates("R-25877")
def test_parameter_names(yaml_file):
- '''
+ """
A VNF's Heat Orchestration Template's parameter name
(i.e., <param name>) **MUST** contain only alphanumeric
characters and underscores ('_').
- '''
+ """
with open(yaml_file) as fh:
yml = yaml.load(fh)
@@ -108,8 +107,7 @@ def test_parameter_names(yaml_file):
if "parameters" not in yml:
pytest.skip("No parameters specified in the heat template")
- for key in yml['parameters']:
- assert RE_VALID_PARAMETER_NAME.match(key), (
- '%s parameter "%s" not alphanumeric or underscore' % (
- yaml_file,
- key))
+ for key in yml["parameters"]:
+ assert RE_VALID_PARAMETER_NAME.match(
+ key
+ ), '%s parameter "%s" not alphanumeric or underscore' % (yaml_file, key)