[PMSH] Read all subscriptions API

Issue-ID: DCAEGEN2-2847
Signed-off-by: raviteja.karumuri <raviteja.karumuri@est.tech>
Change-Id: I6a2cbc127e12f2f6b051ed5f58fd2fa584a71908

3 files changed, 91 insertions, 11 deletions

diff --git a/components/pm-subscription-handler/tests/base_setup.py b/components/pm-subscription-handler/tests/base_setup.py
index 4328f59c..be7d1b8f 100755
--- a/components/pm-subscription-handler/tests/base_setup.py
+++ b/components/pm-subscription-handler/tests/base_setup.py
@@ -29,11 +29,29 @@ from mod.pmsh_config import AppConfig as NewAppConfig
 
 
 def get_pmsh_config(file_path='data/cbs_data_1.json'):
+    """
+    Gets PMSH config from the JSON file
+
+    Args:
+        file_path (String): Name of the file with path
+
+    Returns
+        dict: Dictionary representation of the the service configuration
+    """
     with open(os.path.join(os.path.dirname(__file__), file_path), 'r') as data:
         return json.load(data)
 
 
-def subscription_data(subscription_name):
+def create_subscription_data(subscription_name):
+    """
+    Creates subscription model object
+
+    Args:
+        subscription_name (String): Name of the Subscription
+
+    Returns
+        SubscriptionModel: single subscription model object
+    """
     nf_filter = NetworkFunctionFilterModel(subscription_name, '{^pnf.*,^vnf.*}',
                                            '{}', '{}', '{}')
     mg_first = MeasurementGroupModel(subscription_name, 'MG1', 'UNLOCKED', 15, '/pm/pm.xml',
@@ -51,6 +69,22 @@ def subscription_data(subscription_name):
     return subscription_model
 
 
+def create_multiple_subscription_data(subscription_names):
+    """
+    Creates a list of subscription model objects
+
+    Args:
+        subscription_names (List): Name of the Subscriptions
+
+    Returns
+        list (SubscriptionModel): of subscription model objects
+    """
+    subscriptions = []
+    for subscription_name in subscription_names:
+        subscriptions.append(create_subscription_data(subscription_name))
+    return subscriptions
+
+
 class BaseClassSetup(TestCase):
     app = None
     app_context = None
diff --git a/components/pm-subscription-handler/tests/services/test_subscription_service.py b/components/pm-subscription-handler/tests/services/test_subscription_service.py
index a0e4d541..86bf2639 100644
--- a/components/pm-subscription-handler/tests/services/test_subscription_service.py
+++ b/components/pm-subscription-handler/tests/services/test_subscription_service.py
@@ -29,6 +29,7 @@ from mod.api.custom_exception import DuplicateDataException, InvalidDataExceptio
 from mod.pmsh_config import AppConfig
 from tests.base_setup import BaseClassSetup
 from mod.api.services import subscription_service, nf_service, measurement_group_service
+from tests.base_setup import create_multiple_subscription_data
 
 
 class SubscriptionServiceTestCase(BaseClassSetup):
@@ -357,3 +358,21 @@ class SubscriptionServiceTestCase(BaseClassSetup):
         db_string = '{}'
         db_array = convert_db_string_to_list(db_string)
         self.assertEqual(len(db_array), 0)
+
+    @patch('mod.api.services.subscription_service.query_all_subscriptions',
+           MagicMock(return_value=create_multiple_subscription_data(
+               ['sub_demo_one', 'sub_demo_two'])))
+    def test_get_subscriptions_list(self):
+        subs = subscription_service.get_subscriptions_list()
+        self.assertEqual(subs[0]['subscription']['subscriptionName'], 'sub_demo_one')
+        self.assertEqual(subs[1]['subscription']['subscriptionName'], 'sub_demo_two')
+        self.assertEqual(subs[1]['subscription']['measurementGroups'][0]['measurementGroup']
+                         ['measurementGroupName'], 'MG1')
+        self.assertEqual(len(subs[1]['subscription']['measurementGroups']), 2)
+        self.assertEqual(len(subs), 2)
+
+    @patch('mod.api.services.subscription_service.query_all_subscriptions',
+           MagicMock(return_value=[]))
+    def test_get_subscriptions_list_empty(self):
+        subs = subscription_service.get_subscriptions_list()
+        self.assertEqual(subs, [])
diff --git a/components/pm-subscription-handler/tests/test_controller.py b/components/pm-subscription-handler/tests/test_controller.py
index 7bd72a29..77ab889c 100755
--- a/components/pm-subscription-handler/tests/test_controller.py
+++ b/components/pm-subscription-handler/tests/test_controller.py
@@ -21,12 +21,13 @@ from unittest.mock import patch, MagicMock
 from http import HTTPStatus
 
 from mod import aai_client
-from mod.api.controller import status, post_subscription, get_subscription_by_name
+from mod.api.controller import status, post_subscription, get_subscription_by_name,\
+    get_subscriptions
 from tests.base_setup import BaseClassSetup
 from mod.api.db_models import SubscriptionModel, NfMeasureGroupRelationalModel
 from mod.subscription import SubNfState
 from mod.network_function import NetworkFunctionFilter
-from tests.base_setup import subscription_data
+from tests.base_setup import create_subscription_data, create_multiple_subscription_data
 
 
 class ControllerTestCase(BaseClassSetup):
@@ -111,11 +112,11 @@ class ControllerTestCase(BaseClassSetup):
         self.assertEqual(response[1], 400)
         self.assertEqual(response[0], 'No value provided in subscription name')
 
-    @patch('mod.api.services.subscription_service.get_subscription_by_name',
-           MagicMock(return_value=subscription_data('sub_demo')))
+    @patch('mod.api.services.subscription_service.query_subscription_by_name',
+           MagicMock(return_value=create_subscription_data('sub_demo')))
     def test_get_subscription_by_name_api(self):
         sub, status_code = get_subscription_by_name('sub_demo')
-        self.assertEqual(status_code, HTTPStatus.OK)
+        self.assertEqual(status_code, HTTPStatus.OK.value)
         self.assertEqual(sub['subscription']['subscriptionName'], 'sub_demo')
         self.assertEqual(sub['subscription']['nfFilter']['nfNames'],
                          ['^pnf.*', '^vnf.*'])
@@ -125,16 +126,42 @@ class ControllerTestCase(BaseClassSetup):
         self.assertEqual(sub['subscription']['operationalPolicyName'],
                          'pmsh_operational_policy')
 
-    @patch('mod.api.services.subscription_service.get_subscription_by_name',
+    @patch('mod.api.services.subscription_service.query_subscription_by_name',
            MagicMock(return_value=None))
-    def test_get_subscription_by_name_api_error(self):
+    def test_get_subscription_by_name_api_none(self):
         sub, status_code = get_subscription_by_name('sub_demo')
-        self.assertEqual(status_code, HTTPStatus.NOT_FOUND)
+        self.assertEqual(status_code, HTTPStatus.NOT_FOUND.value)
         self.assertEqual(sub['error'],
                          'Subscription was not defined with the name : sub_demo')
 
-    @patch('mod.api.services.subscription_service.get_subscription_by_name',
+    @patch('mod.api.services.subscription_service.query_subscription_by_name',
            MagicMock(side_effect=Exception('something failed')))
     def test_get_subscription_by_name_api_exception(self):
         sub, status_code = get_subscription_by_name('sub_demo')
-        self.assertEqual(status_code, HTTPStatus.INTERNAL_SERVER_ERROR)
+        self.assertEqual(status_code, HTTPStatus.INTERNAL_SERVER_ERROR.value)
+
+    @patch('mod.api.services.subscription_service.query_all_subscriptions',
+           MagicMock(return_value=create_multiple_subscription_data(
+               ['sub_demo_one', 'sub_demo_two'])))
+    def test_get_subscriptions_api(self):
+        subs, status_code = get_subscriptions()
+        self.assertEqual(status_code, HTTPStatus.OK.value)
+        self.assertEqual(subs[0]['subscription']['subscriptionName'], 'sub_demo_one')
+        self.assertEqual(subs[1]['subscription']['subscriptionName'], 'sub_demo_two')
+        self.assertEqual(subs[1]['subscription']['measurementGroups'][0]['measurementGroup']
+                         ['measurementGroupName'], 'MG1')
+        self.assertEqual(len(subs[1]['subscription']['measurementGroups']), 2)
+        self.assertEqual(len(subs), 2)
+
+    @patch('mod.api.services.subscription_service.query_all_subscriptions',
+           MagicMock(return_value=None))
+    def test_get_subscriptions_api_none(self):
+        subs, status_code = get_subscriptions()
+        self.assertEqual(status_code, HTTPStatus.OK.value)
+        self.assertEqual(subs, [])
+
+    @patch('mod.api.services.subscription_service.query_all_subscriptions',
+           MagicMock(side_effect=Exception('something failed')))
+    def test_get_subscriptions_api_exception(self):
+        subs, status_code = get_subscriptions()
+        self.assertEqual(status_code, HTTPStatus.INTERNAL_SERVER_ERROR.value)