test: adjust timeouts and set exit flag (#230)

Author: JustinBeckwith

.mocharc.json

@@ -1,6 +1,7 @@
 {
   "check-leaks": true,
-  "timeout": 60000,
+  "timeout": 2000,
   "throw-deprecation": true,
-  "enable-source-maps": true
+  "enable-source-maps": true,
+  "exit": true
 }

src/queue.ts

@@ -40,7 +40,7 @@ export class Queue {
         return;
       }
       const timer = setInterval(async () => {
-        if (this.activeTimers === 0 && this.q.size === 0) {
+        if (this.activeTimers === 0) {
           await this.q.onIdle();
           clearInterval(timer);
           resolve();

test/test.cli.ts

@@ -6,8 +6,10 @@ import * as util from 'util';
 import enableDestroy = require('server-destroy');
 import {LinkResult, LinkState} from '../src/index';
 
-describe('cli', () => {
+// eslint-disable-next-line prefer-arrow-callback
+describe('cli', function () {
   let server: http.Server;
+  this.timeout(30_000);
 
   if (process.env.LINKINATOR_SKIP_CLI_TESTS) {
     return;
@@ -142,17 +144,6 @@ describe('cli', () => {
     assert.match(res.stderr, /FORMAT must be/);
   });
 
-  it('should throw on invalid format', async () => {
-    const res = await execa(
-      'npx',
-      ['linkinator', './README.md', '--format', 'LOL'],
-      {
-        reject: false,
-      }
-    );
-    assert.match(res.stderr, /FORMAT must be/);
-  });
-
   it('should throw on invalid verbosity', async () => {
     const res = await execa(
       'npx',