diff options
Diffstat (limited to 'components/pm-subscription-handler/tests/test_subscription_handler.py')
| -rw-r--r-- | components/pm-subscription-handler/tests/test_subscription_handler.py | 33 |
1 files changed, 13 insertions, 20 deletions
diff --git a/components/pm-subscription-handler/tests/test_subscription_handler.py b/components/pm-subscription-handler/tests/test_subscription_handler.py index 168d0366..d922b96f 100644 --- a/components/pm-subscription-handler/tests/test_subscription_handler.py +++ b/components/pm-subscription-handler/tests/test_subscription_handler.py @@ -30,17 +30,15 @@ class SubscriptionHandlerTest(TestCase): @patch('mod.create_app') @patch('mod.subscription.Subscription') @patch('mod.pmsh_utils._MrPub') - @patch('mod.config_handler.ConfigHandler') @patch('mod.pmsh_utils.AppConfig') @patch('mod.pmsh_utils.PeriodicTask') - def setUp(self, mock_aai_event_thread, mock_app_conf, mock_config_handler, mock_mr_pub, + def setUp(self, mock_aai_event_thread, mock_app_conf, mock_mr_pub, mock_sub, mock_app): with open(os.path.join(os.path.dirname(__file__), 'data/cbs_data_1.json'), 'r') as data: self.cbs_data_1 = json.load(data) self.mock_app = mock_app self.mock_sub = mock_sub self.mock_mr_pub = mock_mr_pub - self.mock_config_handler = mock_config_handler self.mock_app_conf = mock_app_conf self.mock_aai_event_thread = mock_aai_event_thread self.nf_1 = NetworkFunction(nf_name='pnf_1') @@ -51,25 +49,22 @@ class SubscriptionHandlerTest(TestCase): @patch('mod.aai_client.get_pmsh_subscription_data') def test_execute_no_change_of_state(self, mock_get_aai, mock_logger): mock_get_aai.return_value = self.mock_sub, self.nfs - self.mock_config_handler.get_config.return_value = self.cbs_data_1 - sub_handler = SubscriptionHandler(self.mock_config_handler, - AdministrativeState.UNLOCKED.value, self.mock_mr_pub, + sub_handler = SubscriptionHandler(AdministrativeState.UNLOCKED.value, self.mock_mr_pub, self.mock_app, self.mock_app_conf, self.mock_aai_event_thread) - sub_handler.execute() - + with patch('mod.pmsh_utils.ConfigHandler.get_pmsh_config', return_value=self.cbs_data_1): + sub_handler.execute() mock_logger.assert_called_with('Administrative State did not change in the Config') @patch('mod.aai_client.get_pmsh_subscription_data') def test_execute_change_of_state_unlocked(self, mock_get_aai): mock_get_aai.return_value = self.mock_sub, self.nfs self.mock_aai_event_thread.return_value.start.return_value = 'start_method' - self.mock_config_handler.get_config.return_value = self.cbs_data_1 - sub_handler = SubscriptionHandler(self.mock_config_handler, - AdministrativeState.LOCKED.value, self.mock_mr_pub, + sub_handler = SubscriptionHandler(AdministrativeState.LOCKED.value, self.mock_mr_pub, self.mock_app, self.mock_app_conf, self.mock_aai_event_thread.return_value) - sub_handler.execute() + with patch('mod.pmsh_utils.ConfigHandler.get_pmsh_config', return_value=self.cbs_data_1): + sub_handler.execute() self.assertEqual(AdministrativeState.UNLOCKED.value, sub_handler.administrative_state) self.mock_sub.process_subscription.assert_called_with(self.nfs, self.mock_mr_pub, @@ -82,12 +77,11 @@ class SubscriptionHandlerTest(TestCase): self.mock_aai_event_thread.return_value.cancel.return_value = 'cancel_method' self.cbs_data_1['policy']['subscription']['administrativeState'] = \ AdministrativeState.LOCKED.value - self.mock_config_handler.get_config.return_value = self.cbs_data_1 - sub_handler = SubscriptionHandler(self.mock_config_handler, - AdministrativeState.UNLOCKED.value, self.mock_mr_pub, + sub_handler = SubscriptionHandler(AdministrativeState.UNLOCKED.value, self.mock_mr_pub, self.mock_app, self.mock_app_conf, self.mock_aai_event_thread.return_value) - sub_handler.execute() + with patch('mod.pmsh_utils.ConfigHandler.get_pmsh_config', return_value=self.cbs_data_1): + sub_handler.execute() self.assertEqual(AdministrativeState.LOCKED.value, sub_handler.administrative_state) self.mock_sub.process_subscription.assert_called_with(self.nfs, self.mock_mr_pub, @@ -98,12 +92,11 @@ class SubscriptionHandlerTest(TestCase): @patch('mod.aai_client.get_pmsh_subscription_data') def test_execute_exception(self, mock_get_aai, mock_logger): mock_get_aai.return_value = self.mock_sub, self.nfs - self.mock_config_handler.get_config.return_value = self.cbs_data_1 self.mock_sub.process_subscription.side_effect = Exception - sub_handler = SubscriptionHandler(self.mock_config_handler, - AdministrativeState.LOCKED.value, self.mock_mr_pub, + sub_handler = SubscriptionHandler(AdministrativeState.LOCKED.value, self.mock_mr_pub, self.mock_app, self.mock_app_conf, self.mock_aai_event_thread) - sub_handler.execute() + with patch('mod.pmsh_utils.ConfigHandler.get_pmsh_config', return_value=self.cbs_data_1): + sub_handler.execute() mock_logger.assert_called_with('Error occurred during the activation/deactivation process ') |