Fix query subscription bugs 61/85861/3
authorfujinhua <fu.jinhua@zte.com.cn>
Sat, 20 Apr 2019 03:14:59 +0000 (11:14 +0800)
committerfujinhua <fu.jinhua@zte.com.cn>
Sat, 20 Apr 2019 03:59:46 +0000 (11:59 +0800)
Change-Id: I35564a4eb5683d8b5654051d93cc17a690f52027
Issue-ID: VFC-1306
Signed-off-by: fujinhua <fu.jinhua@zte.com.cn>
lcm/ns/biz/query_subscription.py
lcm/ns/tests/test_query_subscriptions.py

index 4b36aa7..237c87a 100644 (file)
@@ -17,7 +17,6 @@ import json
 import logging
 
 from lcm.pub.database.models import SubscriptionModel
-from lcm.pub.exceptions import NSLCMException
 
 logger = logging.getLogger(__name__)
 ROOT_FILTERS = {
@@ -55,8 +54,6 @@ class QuerySubscription:
             subscriptions = SubscriptionModel.objects.filter(**query_data)
         else:
             subscriptions = SubscriptionModel.objects.all()
-        if not subscriptions.exists():
-            raise NSLCMException('Subscriptions do not exist')
         return [self.fill_resp_data(subscription) for subscription in subscriptions]
 
     def fill_resp_data(self, subscription):
index 87e98b8..ec70bda 100644 (file)
@@ -125,7 +125,8 @@ class TestQuerySubscriptions(TestCase):
                           links=json.dumps(links),
                           ns_instance_filter=json.dumps(ns_instance_filter)).save()
         response = self.client.get("/api/nslcm/v1/subscriptions?nsInstanceId=dummy", format='json')
-        self.assertEqual(response.status_code, status.HTTP_500_INTERNAL_SERVER_ERROR)
+        self.assertEqual(response.status_code, status.HTTP_200_OK)
+        self.assertEqual([], response.data)
 
     def test_get_subscriptions_with_invalid_filter(self):
         ns_instance_filter = {