fix: keep eventMap item not removed for Jest

Author: wwayne

examples/express_jest/case2.test.js

@@ -20,13 +20,13 @@ describe('api testing', () => {
       });
   })
 
-  // test('should able to delete', (done) => {
-  //   request
-  //     .delete('/projects/uuid-2')
-  //     .expect(204)
-  //     .end(function(err, res) {
-  //       if (err) throw err;
-  //       done()
-  //     });
-  // })
+  test('should able to delete', (done) => {
+    request
+      .delete('/projects/uuid-2')
+      .expect(204)
+      .end(function(err, res) {
+        if (err) throw err;
+        done()
+      });
+  })
 })

examples/package.json

@@ -6,7 +6,7 @@
   "scripts": {
     "express-mocha:test": "mocha -b --timeout 10000 --reporter spec --exit express_mocha/*.test.js",
     "express-mocha": "outdoc npm run express-mocha:test",
-    "express-jest:test": "jest --forceExit --detectOpenHandles --bail --no-cache express_jest/case2.test.js",
+    "express-jest:test": "jest --forceExit --detectOpenHandles --bail express_jest/*.test.js",
     "express-jest": "outdoc npm run express-jest:test"
   },
   "dependencies": {

src/RequestHook.ts

@@ -43,6 +43,5 @@ export default class RequestHook {
     const triggerAsyncId = res.triggerAsyncId;
     const responseBodyData = this.eventMap[triggerAsyncId];
     this.apiCollector.addAPIItem(res.data, responseBodyData);
-    delete this.eventMap[triggerAsyncId];
   }
 }

src/index.ts

@@ -34,7 +34,7 @@ type serverResType = {
 export class OutDoc {
   public static init (): void {
     // Prevent pollute running normal testing
-    if (process.env.IS_OUTDOC !== 'true') return
+    if (process.env.IS_OUTDOC !== 'true') return;
 
     const asyncHook = async_hooks.createHook({
       init: (
@@ -102,6 +102,7 @@ export class OutDoc {
                 }
               };
             }
+
             process.stderr.write(PREFIX_SERVER_RESPONSE + JSON.stringify(res) + "\n");
           }
         }

src/runner.ts

@@ -52,7 +52,7 @@ export async function runner (
 
   const childProcess = spawn(args[0], args.slice(1), {
     detached: true,
-    stdio: ["inherit", "pipe", "pipe"],
+    stdio: ["inherit", "inherit", "pipe"],
     env: {
       ...process.env,
       NODE_ENV: 'test',
@@ -75,7 +75,7 @@ export async function runner (
             requestHook.handleResponseBodyData(res);
           } catch (err) {
             if (err instanceof Error) {
-              process.stderr.write(err.message)
+              process.stderr.write(err.message);
             }
           }
           return;
@@ -98,7 +98,7 @@ export async function runner (
         }
 
         process.stderr.write(dataStr + "\n");
-      })
+      });
   });
 
   childProcess.on('close', async (code) => {
@@ -117,11 +117,11 @@ export async function runner (
             email: options.email
           }
         );
-        console.log('Generate API document success');
+        console.log('✅ Success generating API document');
       } catch (err) {
         let message = "";
         if (err instanceof Error) message = err.message;
-        console.log('Generate API document failed: ', message);
+        console.log('💔 Failed generating API document failed: ', message);
       }
     }
   });