fixup! fixup! fix(sandbox): normalize windows paths correctly

npm · Aug 25, 2021 · bc35e47 · bc35e47
1 parent 84ee370
commit bc35e47
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/test/fixtures/sandbox.js b/test/fixtures/sandbox.js
@@ -70,7 +70,6 @@ const redactedDefaults = [
 const vagueRedactedDefaults = [
   'editor',
   'shell',
-  'viewer',
 ]
 
 const normalize = (str) => str
@@ -152,6 +151,10 @@ class Sandbox extends EventEmitter {
       }
     }
 
+    const viewer = _process.platform === 'win32'
+      ? /"browser"[^:]/g
+      : /"man"/g
+
     // the global prefix is platform dependent
     const realGlobalPrefix = _process.platform === 'win32'
       ? dirname(_process.execPath)
@@ -163,6 +166,7 @@ class Sandbox extends EventEmitter {
 
     // and finally replace some paths we know could be present
     clean = clean
+      .replace(viewer, '"{VIEWER}"')
       .split(normalize(this[_proxy].execPath)).join('{EXECPATH}')
       .split(normalize(_process.execPath)).join('{REALEXECPATH}')
       .split(normalize(this.global)).join('{GLOBALPREFIX}')