fix: check fail when request throw

.eslintignore

@@ -1,3 +1,4 @@
 node_modules
 target
 coverage
+example

lib/check.js

@@ -21,7 +21,12 @@ module.exports = async args => {
 
   const url = 'http://127.0.0.1:' + checkPort + checkPath;
   console.info('Request %s for health check', url);
-  const res = await urllib.request(url);
+  let res;
+  try {
+    res = await urllib.request(url);
+  } catch (_) {
+    return false;
+  }
   const body = res.data.toString() || '';
   // 匹配返回的 status
   if (res.status !== expectedStatus) return false;

test/check.test.js

@@ -21,7 +21,11 @@ describe('test/check.test.js', () => {
   beforeEach(() => mkdirp(releaseDir));
   afterEach(() => rimraf(homeDir));
   afterEach(done => {
-    server.close(done);
+    if (!server) done();
+    server.close(() => {
+      server = null;
+      done();
+    });
   });
 
   describe('check status', () => {
@@ -79,6 +83,24 @@ describe('test/check.test.js', () => {
         .expect('code', 1)
         .end();
     });
+
+  });
+
+  describe('check without listen', () => {
+    beforeEach(async () => {
+      await fs.symlink(
+        path.join(__dirname, 'fixtures/check-unlisten'),
+        path.join(homeDir, 'release/run')
+      );
+    });
+
+    it('should fail', async () => {
+      await coffee.fork(checkBin)
+        .debug()
+        .expect('stdout', /Request http:\/\/127.0.0.1:7002\/healthy for health check/)
+        .expect('code', 1)
+        .end();
+    });
   });
 
 });

test/fixtures/check-unlisten/.nodestack

@@ -0,0 +1,6 @@
+health_check:
+  enable: true
+  port: 7002
+  path: /healthy
+  status: 200
+  body: healthy