From: hongyuzhao <zhao.hongyu@zte.com.cn>
Date: Fri, 14 Jun 2019 08:32:56 +0000 (+0800)
Subject: Fix bug for miss check callbackurl in subscription post request.
X-Git-Tag: 1.3.4~31
X-Git-Url: https://gerrit.onap.org/r/gitweb?a=commitdiff_plain;h=refs%2Fchanges%2F56%2F89956%2F1;p=vfc%2Fgvnfm%2Fvnflcm.git

Fix bug for miss check callbackurl in subscription post request.

Issue-ID: VFC-1418
Signed-off-by: hongyuzhao <zhao.hongyu@zte.com.cn>
Change-Id: I2612cf6edd88b5a68860cc157932f03756fc06b6
---

diff --git a/lcm/lcm/nf/biz/create_subscription.py b/lcm/lcm/nf/biz/create_subscription.py
index 8c6d4528..164bf6c8 100644
--- a/lcm/lcm/nf/biz/create_subscription.py
+++ b/lcm/lcm/nf/biz/create_subscription.py
@@ -62,7 +62,7 @@ class CreateSubscription:
 
     def do_biz(self):
         self.subscription_id = str(uuid.uuid4())
-        # self.check_callbackuri_connection()
+        self.check_callbackuri_connection()
         self.check_valid_auth_info()
         self.check_filter_types()
         self.check_valid()
diff --git a/lcm/lcm/nf/tests/test_subscribe_notification.py b/lcm/lcm/nf/tests/test_subscribe_notification.py
index 94c2ff7c..70a309b8 100644
--- a/lcm/lcm/nf/tests/test_subscribe_notification.py
+++ b/lcm/lcm/nf/tests/test_subscribe_notification.py
@@ -174,6 +174,3 @@ class TestSubscription(TestCase):
         response = self.client.post("/api/vnflcm/v1/subscriptions", data=miss_callbackUri_subscription, format='json')
         self.assertEqual(400, response.status_code)
         self.assertEqual({'callbackUri': ['This field is required.']}, response.data['detail'])
-        # self.assertEqual(temp_uuid, response.data["id"])
-        # response = self.client.post("/api/vnflcm/v1/subscriptions", data=dummy_subscription, format='json')
-        # self.assertEqual(303, response.status_code)