fix: config file removed before watcher started (#316)

.github/workflows/ci.yml

@@ -38,5 +38,4 @@ jobs:
       - name: Test
         run: npm test
       - name: Integration Tests
-        if: matrix.os == 'ubuntu-latest'
         run: npm run test:integration

lib/launcher.js

@@ -48,13 +48,14 @@ export async function launchDaemon(resolver, hash) {
  */
 export async function stopDaemon(resolver, config) {
   try {
-    await platformAwareStopDaemon(config);
+    await Promise.all([
+      waitForConfig(resolver.base),
+      platformAwareStopDaemon(config)
+    ]);
   } catch (err) {
     console.error(`eslint_d: ${err} - removing config`);
     await removeConfig(resolver);
-    return;
   }
-  await waitForConfig(resolver.base);
 }
 
 /**

lib/launcher.test.js

@@ -245,10 +245,11 @@ describe('lib/launcher', () => {
         assert.calledOnceWith(fs_promises.unlink, `${resolver.base}/.eslint_d`);
       });
 
-      it('does not watch for the config file', () => {
-        stopDaemon(resolver, config);
+      it('does not wait for the config file removal', async () => {
+        const promise = stopDaemon(resolver, config);
 
-        refute.called(fs.watch);
+        assert.calledOnceWith(fs.watch, resolver.base);
+        await assert.resolves(promise, undefined);
       });
     });
   });
@@ -339,10 +340,11 @@ describe('lib/launcher', () => {
         assert.calledOnceWith(fs_promises.unlink, `${resolver.base}/.eslint_d`);
       });
 
-      it('does not watch for the config file', () => {
-        stopDaemon(resolver, config);
+      it('does not wait for the config file removal', async () => {
+        const promise = stopDaemon(resolver, config);
 
-        refute.called(fs.watch);
+        assert.calledOnceWith(fs.watch, resolver.base);
+        await assert.resolves(promise, undefined);
       });
     });
   });

package.json

@@ -21,7 +21,7 @@
   "scripts": {
     "lint": "eslint .",
     "test": "mocha '**/*.test.js'",
-    "test:integration": "mocha --bail --slow 1000 test/test.integration.js",
+    "test:integration": "mocha --bail --slow 1000 --timeout 5000 test/test.integration.js",
     "test:coverage": "mcr --filter \"{'**/node_modules/**':false,'**/**':true}\" -r v8,console-details mocha '**/*.test.js'",
     "watch": "chokidar '**/*.js' -c 'npm t' --initial --silent",
     "prepare": "husky && sh scripts/install-fixture-deps.sh",