actually stop the watcher (#370)

* actually stop the watcher.

* 60s timeout

* return a promise

* fixup

Author: mcollina

.taprc

@@ -3,3 +3,4 @@ jsx: false
 jobs: 1
 reporter: terse
 check-coverage: false
+timeout: 60

lib/watch/index.js

@@ -26,7 +26,7 @@ const watch = function (args, ignoreWatch, verboseWatch) {
     }
     if (watcher) {
       allStop = true
-      watcher.close()
+      return watcher.close()
     }
   }
 
@@ -109,6 +109,8 @@ const watch = function (args, ignoreWatch, verboseWatch) {
     stop(watcher)
   })
 
+  emitter.stop = stop.bind(null, watcher)
+
   return emitter
 }
 

package.json

@@ -93,4 +93,4 @@
   "tsd": {
     "directory": "test"
   }
-}
+}

test/start.test.js

@@ -621,8 +621,8 @@ test('should reload the env on restart when watching', async (t) => {
   const fastifyEmitter = await requireUncached('../start').start(argv)
 
   t.teardown(async () => {
-    await fastifyEmitter.emit('close')
-    await process.chdir(cwd)
+    await fastifyEmitter.stop()
+    process.chdir(cwd)
   })
 
   await once(fastifyEmitter, 'ready')