Skip to content

Commit 253671d

Browse files
authoredJun 14, 2022
chore(tests): fix libnpmdiff tests for execPaths with spaces (#4988)
1 parent b46fed7 commit 253671d

File tree

1 file changed

+6
-5
lines changed

1 file changed

+6
-5
lines changed
 

‎workspaces/libnpmdiff/test/index.js

+6-5
Original file line numberDiff line numberDiff line change
@@ -5,11 +5,12 @@ const t = require('tap')
55
const diff = require('../lib/index.js')
66

77
const normalizePath = p => p
8-
.replace(/\\+/g, '/')
8+
.replace(/\\+(?!")/g, '/')
99
.replace(/\r\n/g, '\n')
1010

1111
t.cleanSnapshot = (str) => normalizePath(str)
1212
.replace(normalizePath(process.execPath), 'node')
13+
.replace(/\\"node\\"/g, 'node')
1314

1415
const json = (obj) => `${JSON.stringify(obj, null, 2)}\n`
1516

@@ -62,7 +63,7 @@ t.test('folder in node_modules', async t => {
6263
name: 'a',
6364
version: '1.0.0',
6465
scripts: {
65-
prepare: `${process.execPath} prepare.js`,
66+
prepare: `"${process.execPath}" prepare.js`,
6667
},
6768
}),
6869
'prepare.js': 'throw new Error("ERR")',
@@ -72,7 +73,7 @@ t.test('folder in node_modules', async t => {
7273
name: 'b',
7374
version: '2.0.0',
7475
scripts: {
75-
prepare: `${process.execPath} prepare.js`,
76+
prepare: `"${process.execPath}" prepare.js`,
7677
},
7778
}),
7879
'prepare.js': 'throw new Error("ERR")',
@@ -86,7 +87,7 @@ t.test('folder in node_modules', async t => {
8687
name: 'a',
8788
version: '1.0.1',
8889
scripts: {
89-
prepare: `${process.execPath} prepare.js`,
90+
prepare: `"${process.execPath}" prepare.js`,
9091
},
9192
}),
9293
'prepare.js': '',
@@ -96,7 +97,7 @@ t.test('folder in node_modules', async t => {
9697
name: 'b',
9798
version: '2.0.1',
9899
scripts: {
99-
prepare: `${process.execPath} prepare.js`,
100+
prepare: `"${process.execPath}" prepare.js`,
100101
},
101102
}),
102103
'prepare.js': '',

0 commit comments

Comments
 (0)
Please sign in to comment.