Skip to content

test: add escapePOSIXShell util #55125

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
nodejs-github-bot merged 12 commits into from
Sep 29, 2024
Merged

test: add escapePOSIXShell util #55125

Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
b931733
test: add `escapePOSIXShell` util
aduh95 Jun 10, 2023
7c9f006
fixup! test: add `escapePOSIXShell` util
aduh95 Sep 25, 2024
354271c
fixup! test: add `escapePOSIXShell` util
aduh95 Sep 26, 2024
aaf5b0a
fixup! test: add `escapePOSIXShell` util
aduh95 Sep 27, 2024
779d566
fixup! test: add `escapePOSIXShell` util
aduh95 Sep 27, 2024
38868c2
fixup! test: add `escapePOSIXShell` util
aduh95 Sep 27, 2024
ad045ab
fixup! test: add `escapePOSIXShell` util
aduh95 Sep 28, 2024
4e0ede9
Apply suggestions from code review
aduh95 Sep 29, 2024
d07a1ac
Update test/parallel/test-permission-allow-child-process-cli.js
aduh95 Sep 29, 2024
b3663ac
docs
aduh95 Sep 29, 2024
7215a86
fixup! docs
aduh95 Sep 29, 2024
e0a22b5
fixup! docs
aduh95 Sep 29, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
fixup! test: add escapePOSIXShell util
  • Loading branch information
@aduh95
aduh95 committed Sep 27, 2024
commit 779d566c9e32d900edfbf3f69c182b2a0d0b7a97
14 changes: 7 additions & 7 deletions test/parallel/test-cli-eval.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -68,13 +68,13 @@ child.exec(...common.escapePOSIXShell`"${process.execPath}" --eval "console.erro

// Assert that module loading works.
{
child.exec(...common.escapePOSIXShell`"${process.execPath}" --eval "${`require(${JSON.stringify(__filename)})`}"`,
common.mustCall((err, stdout, stderr) => {
assert.strictEqual(err.code, 42);
assert.strictEqual(
stdout, 'Loaded as a module, exiting with status code 42.\n');
assert.strictEqual(stderr, '');
}));
common.spawnPromisified(process.execPath, ['--eval', `require(${JSON.stringify(__filename)})`])
.then(common.mustCall(({ stdout, stderr, code }) => {
assert.strictEqual(stderr, '');
assert.strictEqual(
stdout, 'Loaded as a module, exiting with status code 42.\n');
assert.strictEqual(code, 42);
}))
}

// Check that builtin modules are pre-defined.
Expand Down