fix: fix process exit not working on Windows (#132)

* feat: set NODE_ENV to development by default

* fix: fix process exit not working on windows

* test: add testcases to signal handle

* test: ignore win32 test

packages/svrx/__tests__/spec/svrx.base.js

@@ -324,3 +324,36 @@ describe('Public API', () => {
     });
   });
 });
+
+describe('Signal handling', () => {
+  after(() => {
+    process.removeAllListeners('SIGTERM');
+    process.removeAllListeners('SIGINT');
+  });
+
+  describe('Signal handling', () => {
+    ['SIGINT', 'SIGTERM'].forEach((SIGNAL) => {
+      describe(`${SIGNAL}`, () => {
+        let sandbox;
+        let exitStub;
+
+        beforeEach(() => {
+          sandbox = sinon.createSandbox({ useFakeTimers: true });
+          exitStub = sandbox.stub(process, 'exit');
+        });
+
+        afterEach(() => {
+          sandbox.restore();
+        });
+
+        it(`should call 'process.exit()' when receiving a ${SIGNAL}`, (done) => {
+          process.once(SIGNAL, () => {
+            sinon.assert.calledOnce(exitStub);
+            done();
+          });
+          process.kill(process.pid, SIGNAL);
+        });
+      });
+    });
+  });
+});

packages/svrx/index.js

@@ -32,6 +32,29 @@ function create(options) {
   return mapSvrxToExports(new Svrx(options));
 }
 
+process.env.NODE_ENV = process.env.NODE_ENV || 'development';
+
+/* istanbul ignore if */
+if (process.platform === 'win32') {
+  const rl = require('readline').createInterface({ // eslint-disable-line
+    input: process.stdin,
+    output: process.stdout,
+  });
+  rl.on('SIGINT', () => {
+    process.emit('SIGINT');
+  });
+  rl.on('SIGTERM', () => {
+    process.emit('SIGTERM');
+  });
+}
+
+process.on('SIGINT', () => {
+  process.exit();
+});
+process.on('SIGTERM', () => {
+  process.exit();
+});
+
 module.exports = create;
 
 create.create = create;