Add scenario and criteria field in the Test result object of the test API

scenario will be used for OPNFV scenario (currently using version field)
criteria will be passed or failed to give an immediate feedback on the test result status

JIRA: FUNCTEST-151

Change-Id: Iff5eb66bbf0cfbaafec0c3a6211031f6eb83c34f
Signed-off-by: Morgan Richomme <morgan.richomme@orange.com>

diff --git a/utils/test/result_collection_api/resources/handlers.py b/utils/test/result_collection_api/resources/handlers.py
index 1f4d0bb..1eda3b0 100644 (file)
--- a/utils/test/result_collection_api/resources/handlers.py
+++ b/utils/test/result_collection_api/resources/handlers.py
@@ -510,6 +510,8 @@ class TestResultsHandler(GenericApiHandler):
          - installer (fuel, ...)
          - build_tag : Jenkins build tag name
          - period : x (x last days)
+         - scenario : the test scenario (previously version)
+         - criteria : the global criteria status passed or failed
 
 
         :param result_id: Get a result by ID
@@ -526,6 +528,8 @@ class TestResultsHandler(GenericApiHandler):
         version_arg = self.get_query_argument("version", None)
         installer_arg = self.get_query_argument("installer", None)
         build_tag_arg = self.get_query_argument("build_tag", None)
+        scenario_arg = self.get_query_argument("scenario", None)
+        criteria_arg = self.get_query_argument("criteria", None)
         period_arg = self.get_query_argument("period", None)
 
         # prepare request
@@ -549,6 +553,12 @@ class TestResultsHandler(GenericApiHandler):
             if build_tag_arg is not None:
                 get_request["build_tag"] = build_tag_arg
 
+            if scenario_arg is not None:
+                get_request["scenario"] = scenario_arg
+
+            if criteria_arg is not None:
+                get_request["criteria_tag"] = criteria_arg
+
             if period_arg is not None:
                 try:
                     period_arg = int(period_arg)

diff --git a/utils/test/result_collection_api/resources/models.py b/utils/test/result_collection_api/resources/models.py
index 6f93862..35b6af1 100644 (file)
--- a/utils/test/result_collection_api/resources/models.py
+++ b/utils/test/result_collection_api/resources/models.py
@@ -151,6 +151,8 @@ class TestResult:
         self.creation_date = None\r
         self.details = None\r
         self.build_tag = None\r
+        self.scenario = None\r
+        self.criteria = None\r
 \r
     @staticmethod\r
     def test_result_from_dict(test_result_dict):\r
@@ -169,6 +171,8 @@ class TestResult:
         t.version = test_result_dict.get('version')\r
         t.installer = test_result_dict.get('installer')\r
         t.build_tag = test_result_dict.get('build_tag')\r
+        t.scenario = test_result_dict.get('scenario')\r
+        t.criteria = test_result_dict.get('criteria')\r
 \r
         return t\r
 \r
@@ -182,7 +186,9 @@ class TestResult:
             "version": self.version,\r
             "installer": self.installer,\r
             "details": self.details,\r
-            "build_tag": self.build_tag\r
+            "build_tag": self.build_tag,\r
+            "scenario": self.scenario,\r
+            "criteria": self.criteria\r
         }\r
 \r
     def format_http(self):\r
@@ -196,6 +202,8 @@ class TestResult:
             "version": self.version,\r
             "installer": self.installer,\r
             "details": self.details,\r
-            "build_tag": self.build_tag\r
+            "build_tag": self.build_tag,\r
+            "scenario": self.scenario,\r
+            "criteria": self.criteria\r
         }\r
 \r