diff options
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': []}}) ] } |