Add create NS swagger generate logic 91/28991/3
authorfujinhua <fu.jinhua@zte.com.cn>
Wed, 24 Jan 2018 07:12:45 +0000 (15:12 +0800)
committerfujinhua <fu.jinhua@zte.com.cn>
Wed, 24 Jan 2018 07:39:42 +0000 (15:39 +0800)
Change-Id: Ibb0b0334826b7d0949392a9ac63034b9024dd2e4
Issue-ID: VFC-680
Signed-off-by: fujinhua <fu.jinhua@zte.com.cn>
lcm/ns/serializers.py
lcm/ns/tests/test_ns_create.py
lcm/ns/views.py

index e9d85e5..debacf9 100644 (file)
@@ -21,7 +21,7 @@ class ContextSerializer(serializers.Serializer):
 
 
 class CreateNsReqSerializer(serializers.Serializer):
-    csarId = serializers.CharField(help_text="Package ID of NS", required=True)
+    csarId = serializers.CharField(help_text="Package ID of NS", required=False, allow_null=True)
     nsName = serializers.CharField(help_text="Name of NS", required=False, allow_null=True)
     description = serializers.CharField(help_text="Description of NS", required=False, allow_null=True)
     context = ContextSerializer(help_text="Context of NS", required=False)
index e462e1c..61d8791 100644 (file)
@@ -70,8 +70,8 @@ class TestNsInstantiate(TestCase):
         mock_do_biz.side_effect = NSLCMException("nsd not exists.")
         new_nsd_id = '1'
         data = {
-            'nsdid': new_nsd_id,
-            'nsname': 'ns',
+            'csarId': new_nsd_id,
+            'nsName': 'ns',
             'description': 'description'
         }
         response = self.client.post("/api/nslcm/v1/ns", data=data)
index 76c4adb..954f648 100644 (file)
@@ -33,6 +33,7 @@ from lcm.pub.utils.jobutil import JobUtil, JOB_TYPE
 from lcm.pub.utils.restcall import req_by_msb
 from lcm.pub.utils.values import ignore_case_get
 from lcm.ns.serializers import CreateNsReqSerializer, CreateNsRespSerializer
+from lcm.pub.exceptions import NSLCMException
 
 logger = logging.getLogger(__name__)
 
@@ -58,20 +59,28 @@ class CreateNSView(APIView):
     )
     def post(self, request):
         logger.debug("Enter CreateNS: %s", request.data)
-        if ignore_case_get(request.data, 'test') == "test":
-            return Response(data={'nsInstanceId': "test"}, status=status.HTTP_201_CREATED)
-        csar_id = ignore_case_get(request.data, 'csarId')
-        ns_name = ignore_case_get(request.data, 'nsName')
-        description = ignore_case_get(request.data, 'description')
-        context = ignore_case_get(request.data, 'context')
         try:
+            req_serializer = CreateNsReqSerializer(data=request.data)
+            if not req_serializer.is_valid():
+                raise NSLCMException(req_serializer.errors)
+
+            if ignore_case_get(request.data, 'test') == "test":
+                return Response(data={'nsInstanceId': "test"}, status=status.HTTP_201_CREATED)
+            csar_id = ignore_case_get(request.data, 'csarId')
+            ns_name = ignore_case_get(request.data, 'nsName')
+            description = ignore_case_get(request.data, 'description')
+            context = ignore_case_get(request.data, 'context')
             ns_inst_id = CreateNSService(csar_id, ns_name, description, context).do_biz()
+
+            logger.debug("CreateNSView::post::ret={'nsInstanceId':%s}", ns_inst_id)
+            resp_serializer = CreateNsRespSerializer(data={'nsInstanceId': ns_inst_id})
+            if not resp_serializer.is_valid():
+                raise NSLCMException(resp_serializer.errors)
+            return Response(data=resp_serializer.data, status=status.HTTP_201_CREATED)
         except Exception as e:
             logger.error(traceback.format_exc())
             logger.error("Exception in CreateNS: %s", e.message)
             return Response(data={'error': e.message}, status=status.HTTP_500_INTERNAL_SERVER_ERROR)
-        logger.debug("CreateNSView::post::ret={'nsInstanceId':%s}", ns_inst_id)
-        return Response(data={'nsInstanceId': ns_inst_id}, status=status.HTTP_201_CREATED)
 
 
 class NSInstView(APIView):