fix(MongoInstance): add call to "instanceFailed" to "closeHandler"

- add an call to "instanceFailed" into "closeHandler" to ensure the "launch" promise resolves
- add an timeout to "kill_internal"
- always call "kill_internal" if the childprocess's are defined

fixes #385

Author: hasezoey

packages/mongodb-memory-server-core/src/util/MongoInstance.ts

@@ -150,21 +150,26 @@ export default class MongoInstance {
      */
     async function kill_internal(process: ChildProcess, name: string, debug: DebugFn) {
       await new Promise((resolve) => {
+        const timeout = setTimeout(() => {
+          debug('kill_internal timeout triggered');
+          resolve();
+        }, 1000 * 10);
         process.once(`exit`, (code, signal) => {
           debug(`- ${name}: got exit signal, Code: ${code}, Signal: ${signal}`);
+          clearTimeout(timeout);
           resolve();
         });
         debug(`- ${name}: send "SIGINT"`);
         process.kill('SIGINT');
       });
     }
 
-    if (this.childProcess && !this.childProcess.killed) {
+    if (!isNullOrUndefined(this.childProcess)) {
       await kill_internal(this.childProcess, 'childProcess', this.debug);
     } else {
       this.debug('- childProcess: nothing to shutdown, skipping.');
     }
-    if (this.killerProcess && !this.killerProcess.killed) {
+    if (!isNullOrUndefined(this.killerProcess)) {
       await kill_internal(this.killerProcess, 'killerProcess', this.debug);
     } else {
       this.debug('- killerProcess: nothing to shutdown, skipping.');
@@ -265,6 +270,7 @@ export default class MongoInstance {
       this.debug('Mongod instance closed with an non-0 code!');
     }
     this.debug(`CLOSE: ${code}`);
+    this.instanceFailed(`Mongod instance closed with code "${code}"`);
   }
 
   /**