chore: try to normalize echo between OSes

rollup · Jan 7, 2020 · 927475f · 927475f
1 parent 8df73bc
commit 927475f
Show file tree

Hide file tree

Showing 6 changed files with 6 additions and 28 deletions.
diff --git a/packages/auto-install/test/snapshots/yarn-bare.js.md b/packages/auto-install/test/snapshots/yarn-bare.js.md
diff --git a/packages/auto-install/test/snapshots/yarn-bare.js.snap b/packages/auto-install/test/snapshots/yarn-bare.js.snap
diff --git a/packages/auto-install/test/snapshots/yarn.js.md b/packages/auto-install/test/snapshots/yarn.js.md
diff --git a/packages/auto-install/test/snapshots/yarn.js.snap b/packages/auto-install/test/snapshots/yarn.js.snap
diff --git a/packages/auto-install/test/yarn-bare.js b/packages/auto-install/test/yarn-bare.js
@@ -23,12 +23,13 @@ test('yarn, bare', async (t) => {
     },
     plugins: [
       // mock the call to yarn here. yarn has had consistent issues in this test env
-      autoInstall({ manager: 'yarn', commands: { yarn: 'echo "yarn.bare" > yarn.lock' } }),
+      autoInstall({ manager: 'yarn', commands: { yarn: 'echo yarn.bare > yarn.lock' } }),
       resolve()
     ]
   });
   const lockFile = readFileSync('yarn.lock', 'utf-8');
-  t.snapshot(lockFile.replace(/\r\n/g, '\n'));
+  // snapshots for this are a nightmare cross-platform
+  t.truthy(/yarn\.bare\s+node-noop/.test(lockFile));
 });
 
 test.after(async () => {

diff --git a/packages/auto-install/test/yarn.js b/packages/auto-install/test/yarn.js
@@ -22,10 +22,11 @@ test('yarn', async (t) => {
       format: 'cjs'
     },
     // mock the call to yarn here. yarn has had consistent issues in this test env
-    plugins: [autoInstall({ commands: { yarn: 'echo "yarn" > yarn.lock' } }), resolve()]
+    plugins: [autoInstall({ commands: { yarn: 'echo yarn > yarn.lock' } }), resolve()]
   });
   const lockFile = readFileSync('yarn.lock', 'utf-8');
-  t.snapshot(lockFile.replace(/\r\n/g, '\n'));
+  // snapshots for this are a nightmare cross-platform
+  t.truthy(/yarn\s+node-noop/.test(lockFile));
 });
 
 test.after(async () => {