fix(apigw): fix describe api missing issue (#68)

* fix(apigw): fix describe api missing issue

* feat(apigw): update service type dependence

Author: LinLzis

src/modules/apigw/index.js

@@ -108,12 +108,12 @@ class Apigw {
     };
   }
 
-  marshalApiInput(endpoint, apiInputs, serviceType) {
+  marshalApiInput(endpoint, apiInputs) {
     if (endpoint.param) {
       apiInputs.requestParameters = endpoint.param;
     }
 
-    apiInputs.serviceType = serviceType;
+    const { serviceType } = apiInputs;
     endpoint.function = endpoint.function || {};
     // handle front-end API type of WEBSOCKET/HTTP
     if (endpoint.protocol === 'WEBSOCKET') {
@@ -241,7 +241,10 @@ class Apigw {
           }
         }
       }
-    } else {
+    }
+
+    // get API info after apiId confirmed
+    if (endpoint.apiId) {
       apiDetail = await this.request({
         Action: 'DescribeApi',
         serviceId: serviceId,
@@ -254,7 +257,7 @@ class Apigw {
     }
 
     if (!exist) {
-      this.marshalApiInput(endpoint, apiInputs, apiInputs.serviceType);
+      this.marshalApiInput(endpoint, apiInputs);
       const { ApiId } = await this.request({
         Action: 'CreateApi',
         ...apiInputs,
@@ -272,7 +275,7 @@ class Apigw {
       output.internalDomain = apiDetail.InternalDomain;
     } else {
       console.log(`Updating api with api id ${endpoint.apiId}.`);
-      this.marshalApiInput(endpoint, apiInputs, apiDetail.ServiceType);
+      this.marshalApiInput(endpoint, apiInputs);
       await this.request({
         Action: 'ModifyApi',
         apiId: endpoint.apiId,

src/modules/apigw/index.test.js

@@ -82,6 +82,19 @@ async function runTest() {
           method: 'GET',
         },
       },
+      {
+        path: '/wsf',
+        protocol: 'WEBSOCKET',
+        apiName: 'ws-scf',
+        method: 'GET',
+        serviceType: 'SCF',
+        function: {
+          functionNamespace: 'default',
+          functionQualifier: '$DEFAULT',
+          transportFunctionName: 'fullstack-api',
+          registerFunctionName: 'myRestAPI',
+        },
+      },
     ],
   };
   const apigw = new Apigw(credentials, inputs.region);