Skip to content

Commit d94f484

Browse files
committed
Fix tests following change in Node's CLI parser
nodejs/node#24647
1 parent 1468066 commit d94f484

File tree

1 file changed

+26
-20
lines changed

1 file changed

+26
-20
lines changed

test/basic.js

+26-20
Original file line numberDiff line numberDiff line change
@@ -14,8 +14,14 @@ const WRAPPER = require.resolve('./fixtures/basic.wrapper.js')
1414

1515
var unwrap = sw.patchInternals({ wrapper: WRAPPER, mode: 'same-process' })
1616

17-
var expect = 'WRAP ["--","{{FIXTURE}}","xyz"]\n' +
18-
'["--"]\n' +
17+
const NODE_MAJOR_VERSION = parseInt(process.versions.node.split('.')[0])
18+
// https://github.com/nodejs/node/issues/24647
19+
const DOUBLE_DASH = NODE_MAJOR_VERSION >= 10 ? '"--"' : ''
20+
const DOUBLE_DASH_COMMA = NODE_MAJOR_VERSION >= 10 ? '"--",' : ''
21+
const COMMA_DOUBLE_DASH = NODE_MAJOR_VERSION >= 10 ? ',"--"' : ''
22+
23+
var expect = 'WRAP [' + DOUBLE_DASH_COMMA + '"{{FIXTURE}}","xyz"]\n' +
24+
'[' + DOUBLE_DASH + ']\n' +
1925
'["xyz"]\n' +
2026
'EXIT [0,null]\n'
2127

@@ -63,8 +69,8 @@ t.test('spawn execPath', function (t) {
6369
child.on('close', function (code, signal) {
6470
t.equal(code, 0)
6571
t.equal(signal, null)
66-
t.equal(out, 'WRAP ["--","{{FIXTURE}}","xyz"]\n' +
67-
'["--"]\n' +
72+
t.equal(out, 'WRAP [' + DOUBLE_DASH_COMMA + '"{{FIXTURE}}","xyz"]\n' +
73+
'[' + DOUBLE_DASH + ']\n' +
6874
'["xyz"]\n' +
6975
'SIGINT\n' +
7076
'EXIT [0,null]\n')
@@ -82,8 +88,8 @@ t.test('spawn execPath', function (t) {
8288
})
8389
child.on('close', function (code, signal) {
8490
t.equal(signal, 'SIGHUP')
85-
t.equal(out, 'WRAP ["--","{{FIXTURE}}","xyz"]\n' +
86-
'["--"]\n' +
91+
t.equal(out, 'WRAP [' + DOUBLE_DASH_COMMA + '"{{FIXTURE}}","xyz"]\n' +
92+
'[' + DOUBLE_DASH + ']\n' +
8793
'["xyz"]\n' +
8894
'SIGHUP\n' +
8995
'EXIT [null,"SIGHUP"]\n')
@@ -133,8 +139,8 @@ t.test('spawn node', function (t) {
133139
child.on('close', function (code, signal) {
134140
t.equal(code, 0)
135141
t.equal(signal, null)
136-
t.equal(out, 'WRAP ["--","{{FIXTURE}}","xyz"]\n' +
137-
'["--"]\n' +
142+
t.equal(out, 'WRAP [' + DOUBLE_DASH_COMMA + '"{{FIXTURE}}","xyz"]\n' +
143+
'[' + DOUBLE_DASH + ']\n' +
138144
'["xyz"]\n' +
139145
'SIGINT\n' +
140146
'EXIT [0,null]\n')
@@ -152,8 +158,8 @@ t.test('spawn node', function (t) {
152158
})
153159
child.on('close', function (code, signal) {
154160
t.equal(signal, 'SIGHUP')
155-
t.equal(out, 'WRAP ["--","{{FIXTURE}}","xyz"]\n' +
156-
'["--"]\n' +
161+
t.equal(out, 'WRAP [' + DOUBLE_DASH_COMMA + '"{{FIXTURE}}","xyz"]\n' +
162+
'[' + DOUBLE_DASH + ']\n' +
157163
'["xyz"]\n' +
158164
'SIGHUP\n' +
159165
'EXIT [null,"SIGHUP"]\n')
@@ -212,8 +218,8 @@ t.test('exec execPath', function (t) {
212218
child.on('close', function (code, signal) {
213219
t.equal(code, 0)
214220
t.equal(signal, null)
215-
t.equal(out, 'WRAP ["--","{{FIXTURE}}","xyz"]\n' +
216-
'["--"]\n' +
221+
t.equal(out, 'WRAP [' + DOUBLE_DASH_COMMA + '"{{FIXTURE}}","xyz"]\n' +
222+
'[' + DOUBLE_DASH + ']\n' +
217223
'["xyz"]\n' +
218224
'SIGINT\n' +
219225
'EXIT [0,null]\n')
@@ -231,8 +237,8 @@ t.test('exec execPath', function (t) {
231237
})
232238
child.on('close', function (code, signal) {
233239
t.equal(signal, 'SIGHUP')
234-
t.equal(out, 'WRAP ["--","{{FIXTURE}}","xyz"]\n' +
235-
'["--"]\n' +
240+
t.equal(out, 'WRAP [' + DOUBLE_DASH_COMMA + '"{{FIXTURE}}","xyz"]\n' +
241+
'[' + DOUBLE_DASH + ']\n' +
236242
'["xyz"]\n' +
237243
'SIGHUP\n' +
238244
'EXIT [null,"SIGHUP"]\n')
@@ -269,8 +275,8 @@ t.test('exec shebang', { skip: winNoShebang }, function (t) {
269275
})
270276
child.on('close', function (code, signal) {
271277
t.equal(signal, 'SIGHUP')
272-
t.equal(out, 'WRAP ["--","{{FIXTURE}}","xyz"]\n' +
273-
'["--"]\n' +
278+
t.equal(out, 'WRAP [' + DOUBLE_DASH_COMMA + '"{{FIXTURE}}","xyz"]\n' +
279+
'[' + DOUBLE_DASH + ']\n' +
274280
'["xyz"]\n' +
275281
'SIGHUP\n' +
276282
'EXIT [null,"SIGHUP"]\n')
@@ -292,8 +298,8 @@ t.test('exec shebang', { skip: winNoShebang }, function (t) {
292298
child.on('close', function (code, signal) {
293299
t.equal(code, 0)
294300
t.equal(signal, null)
295-
t.equal(out, 'WRAP ["--","{{FIXTURE}}","xyz"]\n' +
296-
'["--"]\n' +
301+
t.equal(out, 'WRAP [' + DOUBLE_DASH_COMMA + '"{{FIXTURE}}","xyz"]\n' +
302+
'[' + DOUBLE_DASH + ']\n' +
297303
'["xyz"]\n' +
298304
'SIGINT\n' +
299305
'EXIT [0,null]\n')
@@ -352,8 +358,8 @@ t.test('--harmony', function (t) {
352358
child.on('close', function (code, signal) {
353359
t.equal(code, 0)
354360
t.equal(signal, null)
355-
t.equal(out, 'WRAP ["--harmony","--","{{FIXTURE}}","xyz"]\n' +
356-
'["--harmony","--"]\n' +
361+
t.equal(out, 'WRAP ["--harmony",' + DOUBLE_DASH_COMMA + '"{{FIXTURE}}","xyz"]\n' +
362+
'["--harmony"' + COMMA_DOUBLE_DASH + ']\n' +
357363
'["xyz"]\n' +
358364
'EXIT [0,null]\n')
359365
t.end()

0 commit comments

Comments
 (0)