GeezerVison

The World Wide Web, restored to 800 pixels of glory.

GeezerVison is showing github.com — test: verify the shell option works properly on execFile · nodejs/node@e55e08c

File tree

Expand file treeCollapse file tree

Open diff view settings

Filter options

test
- parallel
  - test-child-process-execfile.js
- sequential
  - test-child-process-execsync.js

Expand file treeCollapse file tree

Open diff view settings

Collapse file

`‎test/parallel/test-child-process-execfile.js‎`

Copy file name to clipboardExpand all lines: test/parallel/test-child-process-execfile.js

+8Lines changed: 8 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -6,6 +6,7 @@ const uv = process.binding('uv');`
`6`	`6`	`const fixtures = require('../common/fixtures');`
`7`	`7`
`8`	`8`	`const fixture = fixtures.path('exit.js');`
	`9`	`+const execOpts = { encoding: 'utf8', shell: true };`
`9`	`10`
`10`	`11`	`{`
`11`	`12`	`execFile(`
`@@ -38,3 +39,10 @@ const fixture = fixtures.path('exit.js');`
`38`	`39`	`child.kill();`
`39`	`40`	`child.emit('close', code, null);`
`40`	`41`	`}`
	`42`	`+`
	`43`	`+{`
	`44`	`+ // Verify the shell option works properly`
	`45`	`+ execFile(process.execPath, [fixture, 0], execOpts, common.mustCall((err) => {`
	`46`	`+ assert.ifError(err);`
	`47`	`+ }));`
	`48`	`+}`

Collapse file

`‎test/sequential/test-child-process-execsync.js‎`

Copy file name to clipboardExpand all lines: test/sequential/test-child-process-execsync.js

+6Lines changed: 6 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -29,6 +29,7 @@ const TIMER = 200;`
`29`	`29`	`const SLEEP = 2000;`
`30`	`30`
`31`	`31`	`const start = Date.now();`
	`32`	`+const execOpts = { encoding: 'utf8', shell: true };`
`32`	`33`	`let err;`
`33`	`34`	`let caught = false;`
`34`	`35`
@@ -124,3 +125,8 @@ assert.strictEqual(ret, `${msg}\n`);
`124`	`125`	`return true;`
`125`	`126`	`});`
`126`	`127`	`}`
	`128`	`+`
	`129`	`+// Verify the shell option works properly`
	`130`	`+assert.doesNotThrow(() => {`
	`131`	`+ execFileSync(process.execPath, [], execOpts);`
	`132`	`+});`

0 commit comments

Comments

0 (0)

You are visitor 01402520 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.