summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
-rw-r--r--utils/test/result_collection_api/opnfv_testapi/resources/result_models.py17
-rw-r--r--utils/test/result_collection_api/opnfv_testapi/tests/unit/test_result.py18
-rw-r--r--utils/test/result_collection_api/update/templates/changes_in_mongodb.py39
3 files changed, 47 insertions, 27 deletions
diff --git a/utils/test/result_collection_api/opnfv_testapi/resources/result_models.py b/utils/test/result_collection_api/opnfv_testapi/resources/result_models.py
index dd1e3dc53..f73f5c612 100644
--- a/utils/test/result_collection_api/opnfv_testapi/resources/result_models.py
+++ b/utils/test/result_collection_api/opnfv_testapi/resources/result_models.py
@@ -55,15 +55,14 @@ class TI(object):
@staticmethod
def from_dict(a_dict):
- if a_dict is None:
- return None
t = TI()
- t.current = a_dict.get('current')
- if 'histories' in a_dict.keys():
- for history in a_dict.get('histories', None):
- t.histories.append(TIHistory.from_dict(history))
- else:
- t.histories = []
+ if a_dict:
+ t.current = a_dict.get('current')
+ if 'histories' in a_dict.keys():
+ for history in a_dict.get('histories', None):
+ t.histories.append(TIHistory.from_dict(history))
+ else:
+ t.histories = []
return t
@@ -97,7 +96,7 @@ class ResultCreateRequest(object):
self.build_tag = build_tag
self.scenario = scenario
self.criteria = criteria
- self.trust_indicator = trust_indicator
+ self.trust_indicator = trust_indicator if trust_indicator else TI(0)
def format(self):
return {
diff --git a/utils/test/result_collection_api/opnfv_testapi/tests/unit/test_result.py b/utils/test/result_collection_api/opnfv_testapi/tests/unit/test_result.py
index 98ef7c08c..eee06c6e5 100644
--- a/utils/test/result_collection_api/opnfv_testapi/tests/unit/test_result.py
+++ b/utils/test/result_collection_api/opnfv_testapi/tests/unit/test_result.py
@@ -182,6 +182,24 @@ class TestResultCreate(TestResultBase):
self.assertEqual(code, HTTP_OK)
self.assert_href(body)
+ def test_no_ti(self):
+ req = ResultCreateRequest(pod_name=self.pod,
+ project_name=self.project,
+ case_name=self.case,
+ installer=self.installer,
+ version=self.version,
+ start_date=self.start_date,
+ stop_date=self.stop_date,
+ details=self.details.format(),
+ build_tag=self.build_tag,
+ scenario=self.scenario,
+ criteria=self.criteria)
+ (code, res) = self.create(req)
+ _id = res.href.split('/')[-1]
+ self.assertEqual(code, HTTP_OK)
+ code, body = self.get(_id)
+ self.assert_res(code, body, req)
+
class TestResultGet(TestResultBase):
def test_getOne(self):
diff --git a/utils/test/result_collection_api/update/templates/changes_in_mongodb.py b/utils/test/result_collection_api/update/templates/changes_in_mongodb.py
index b3e23803d..9744dd9fe 100644
--- a/utils/test/result_collection_api/update/templates/changes_in_mongodb.py
+++ b/utils/test/result_collection_api/update/templates/changes_in_mongodb.py
@@ -24,24 +24,27 @@ docs_old2New = {
# ({'criteria': 'failed'}, {'criteria': 'FAILED'}),
# ({'criteria': 'passed'}, {'criteria': 'PASS'})
# ]
- 'testcases': [
- ({'name': 'vPing'}, {'name': 'vping_ssh'}),
- ({'name': 'Tempest'}, {'name': 'tempest_smoke_serial'}),
- ({'name': 'Rally'}, {'name': 'rally_sanity'}),
- ({'name': 'ODL'}, {'name': 'odl'}),
- ({'name': 'vIMS'}, {'name': 'vims'}),
- ({'name': 'ONOS'}, {'name': 'onos'}),
- ({'name': 'vPing_userdata'}, {'name': 'vping_userdata'}),
- ({'name': 'ovno'}, {'name': 'ocl'})
- ],
+ # 'testcases': [
+ # ({'name': 'vPing'}, {'name': 'vping_ssh'}),
+ # ({'name': 'Tempest'}, {'name': 'tempest_smoke_serial'}),
+ # ({'name': 'Rally'}, {'name': 'rally_sanity'}),
+ # ({'name': 'ODL'}, {'name': 'odl'}),
+ # ({'name': 'vIMS'}, {'name': 'vims'}),
+ # ({'name': 'ONOS'}, {'name': 'onos'}),
+ # ({'name': 'vPing_userdata'}, {'name': 'vping_userdata'}),
+ # ({'name': 'ovno'}, {'name': 'ocl'})
+ # ],
+ # 'results': [
+ # ({'case_name': 'vPing'}, {'case_name': 'vping_ssh'}),
+ # ({'case_name': 'Tempest'}, {'case_name': 'tempest_smoke_serial'}),
+ # ({'case_name': 'Rally'}, {'case_name': 'rally_sanity'}),
+ # ({'case_name': 'ODL'}, {'case_name': 'odl'}),
+ # ({'case_name': 'vIMS'}, {'case_name': 'vims'}),
+ # ({'case_name': 'ONOS'}, {'case_name': 'onos'}),
+ # ({'case_name': 'vPing_userdata'}, {'case_name': 'vping_userdata'}),
+ # ({'case_name': 'ovno'}, {'case_name': 'ocl'})
+ # ]
'results': [
- ({'case_name': 'vPing'}, {'case_name': 'vping_ssh'}),
- ({'case_name': 'Tempest'}, {'case_name': 'tempest_smoke_serial'}),
- ({'case_name': 'Rally'}, {'case_name': 'rally_sanity'}),
- ({'case_name': 'ODL'}, {'case_name': 'odl'}),
- ({'case_name': 'vIMS'}, {'case_name': 'vims'}),
- ({'case_name': 'ONOS'}, {'case_name': 'onos'}),
- ({'case_name': 'vPing_userdata'}, {'case_name': 'vping_userdata'}),
- ({'case_name': 'ovno'}, {'case_name': 'ocl'})
+ ({'trust_indicator': 0}, {'trust_indicator': {'current': 0, 'histories': []}})
]
}