fix(postgresql): wait for deleting success logical

yugasun · yugasun · commit 79a483d64d39 · 2020-08-18T14:54:43.000+08:00
diff --git a/package.json b/package.json
@@ -71,7 +71,7 @@
   },
   "dependencies": {
     "@tencent-sdk/capi": "^0.2.17",
-    "@ygkit/request": "^0.0.6",
+    "@ygkit/request": "^0.1.1",
     "moment": "^2.25.3",
     "tencent-cloud-sdk": "^1.0.3"
   }
diff --git a/src/modules/postgresql/utils.js b/src/modules/postgresql/utils.js
@@ -32,11 +32,10 @@ async function getDbInstanceDetail(capi, dBInstanceName) {
       } = res;
       return dbDetail;
     }
-    return null;
   } catch (e) {
     console.log(e);
-    return null;
   }
+  return undefined;
 }
 
 /**
@@ -138,7 +137,6 @@ async function deleteDbInstance(capi, dBInstanceName) {
   const detail = await waitResponse({
     callback: async () => getDbInstanceDetail(capi, dBInstanceName),
     targetResponse: undefined,
-    targetProp: 'DBInstanceStatus',
     timeout: TIMEOUT,
   });
   console.log(`Removed postgres instance ${dBInstanceName}.`);

Original file line number	Diff line number	Diff line change
`@@ -32,11 +32,10 @@ async function getDbInstanceDetail(capi, dBInstanceName) {`
`32`	`32`	`} = res;`
`33`	`33`	`return dbDetail;`
`34`	`34`	`}`
`35`		`- return null;`
`36`	`35`	`} catch (e) {`
`37`	`36`	`console.log(e);`
`38`		`- return null;`
`39`	`37`	`}`
	`38`	`+ return undefined;`
`40`	`39`	`}`
`41`	`40`
`42`	`41`	`/**`
`@@ -138,7 +137,6 @@ async function deleteDbInstance(capi, dBInstanceName) {`
`138`	`137`	`const detail = await waitResponse({`
`139`	`138`	`callback: async () => getDbInstanceDetail(capi, dBInstanceName),`
`140`	`139`	`targetResponse: undefined,`
`141`		`- targetProp: 'DBInstanceStatus',`
`142`	`140`	`timeout: TIMEOUT,`
`143`	`141`	`});`
`144`	`142`	console.log(`Removed postgres instance ${dBInstanceName}.`);