🐛 End browser process pipes during cleanup

This is needed specifically for Windows
percy · Feb 23, 2021 · cea5f24 · cea5f24
1 parent 8fcf269
commit cea5f24
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/packages/core/src/discovery/browser.js b/packages/core/src/discovery/browser.js
@@ -140,6 +140,11 @@ export default class Browser extends EventEmitter {
 
     // after closing, attempt to clean up the profile directory
     await closed.then(() => new Promise(resolve => {
+      // needed to end hanging pipes on windows
+      this.process?.stdin.end();
+      this.process?.stdout.end();
+      this.process?.stderr.end();
+
       /* istanbul ignore else: sanity */
       if (this.profile) {
         rimraf(this.profile, error => {