fix(watch-service): remove watcher when it gets closed due to directory removal

[VladyslavG]

Fixes #1186

On Windows deleting watched directory is firing event error which is caught by NodeWatchService.ensureWatcher:

      directoryWatcher.once("error", (e) => {
        this.onWatchError(e, path);
      });

this in turn causes 2 effects:

1. watcher.close(); in the onWatchError
2. debounced (by onPathEvent) this.fsWatchers.delete(path); in the emitEvent method.

If unwatchAllPaths was invoked between these 2 effects we would enter a state when unwatchAllPaths can't finish.
This happened due to

1. unwatchAllPaths assumes that all this.fsWatchers are not closed
2. unwatchAllPaths invokes close on each watcher and waits for the close event on all of watchers
   But this can't work because watcher.close(); was called in the onWatchError, so we are left with a method which can't end.

The proposed fix is to remove watcher from this.fsWatchers inside the onWatchError immediately after watcher is closed.

[AviVahl]

Changes look fine.

On a related note, this API (WatchService) got deprecated and will get removed in a future major. We should use fs.watch with recursive: true instead, with additional watchers for files we want immediate events for.