Skip to content

Commit 0b3ae01

Browse files
authored
test: fix embedding test for Windows
PR-URL: #53659 Reviewed-By: Yagiz Nizipli <[email protected]> Reviewed-By: Rafael Gonzaga <[email protected]> Reviewed-By: Stefan Stojanovic <[email protected]> Reviewed-By: Chengzhong Wu <[email protected]> Reviewed-By: Joyee Cheung <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Luigi Pinca <[email protected]>
1 parent 342b501 commit 0b3ae01

File tree

1 file changed

+2
-7
lines changed

1 file changed

+2
-7
lines changed

Diff for: test/embedding/test-embedding.js

+2-7
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,7 @@ const {
1010
} = require('../common/child_process');
1111
const path = require('path');
1212
const fs = require('fs');
13+
const os = require('os');
1314

1415
tmpdir.refresh();
1516
common.allowGlobals(global.require);
@@ -152,12 +153,6 @@ for (const extraSnapshotArgs of [
152153
{ cwd: tmpdir.path });
153154
}
154155

155-
// Skipping rest of the test on Windows because it fails in the CI
156-
// TODO(StefanStojanovic): Reenable rest of the test after fixing it
157-
if (common.isWindows) {
158-
return;
159-
}
160-
161156
// Guarantee NODE_REPL_EXTERNAL_MODULE won't bypass kDisableNodeOptionsEnv
162157
{
163158
spawnSyncAndExit(
@@ -172,6 +167,6 @@ if (common.isWindows) {
172167
{
173168
status: 9,
174169
signal: null,
175-
stderr: `${binary}: NODE_REPL_EXTERNAL_MODULE can't be used with kDisableNodeOptionsEnv\n`,
170+
stderr: `${binary}: NODE_REPL_EXTERNAL_MODULE can't be used with kDisableNodeOptionsEnv${os.EOL}`,
176171
});
177172
}

0 commit comments

Comments
 (0)