diff --git a/tacker/sol_refactored/common/lcm_op_occ_utils.py b/tacker/sol_refactored/common/lcm_op_occ_utils.py index e2a47e202..8c1845596 100644 --- a/tacker/sol_refactored/common/lcm_op_occ_utils.py +++ b/tacker/sol_refactored/common/lcm_op_occ_utils.py @@ -32,7 +32,7 @@ LOG = logging.getLogger(__name__) # not used at the moment def get_lcmocc(context, lcmocc_id): lcmocc = objects.VnfLcmOpOccV2.get_by_id(context, lcmocc_id) if lcmocc is None: - raise sol_ex.VnfLcmOpOccNotFound(lcmocc_id=id) + raise sol_ex.VnfLcmOpOccNotFound(lcmocc_id=lcmocc_id) return lcmocc diff --git a/tacker/tests/unit/sol_refactored/common/test_lcm_op_occ_utils.py b/tacker/tests/unit/sol_refactored/common/test_lcm_op_occ_utils.py index 4928e68cd..1770a5568 100644 --- a/tacker/tests/unit/sol_refactored/common/test_lcm_op_occ_utils.py +++ b/tacker/tests/unit/sol_refactored/common/test_lcm_op_occ_utils.py @@ -2358,11 +2358,15 @@ class TestLcmOpOccUtils(base.BaseTestCase): self.assertEqual(expected_result.operation, result.operation) @mock.patch.object(objects.base.TackerPersistentObject, 'get_by_id') - def test_get_lcmocc_error(self, mock_lcmocc): + def test_get_lcmocc_not_exist_lcmocc_id(self, mock_lcmocc): mock_lcmocc.return_value = None - self.assertRaises( + lcmocc_id = 'f5aa2eb2-6805-439d-b93d-bb0db8d96e3b' + ex = self.assertRaises( sol_ex.VnfLcmOpOccNotFound, - lcmocc_utils.get_lcmocc, context, 'lcmocc_id') + lcmocc_utils.get_lcmocc, context, lcmocc_id) + + expected_detail = f'VnfLcmOpOcc {lcmocc_id} not found.' + self.assertEqual(expected_detail, ex.detail) @mock.patch.object(objects.base.TackerPersistentObject, 'get_all') def test_get_lcmocc_all(self, mock_lcmocc):