Skip to content

Commit

Permalink
test: fix Windows native test suites
Browse files Browse the repository at this point in the history
2 PRs that landed independently caused this issue which makes every
native suites run in CI fail on Windows. This is just a quick patch to
unblock the CI.

Refs: #52905
Refs: #52646
PR-URL: #53173
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
  • Loading branch information
StefanStojanovic authored and marco-ippolito committed Jul 19, 2024
1 parent 9f8e343 commit 3a16934
Showing 1 changed file with 6 additions and 0 deletions.
6 changes: 6 additions & 0 deletions test/embedding/test-embedding.js
Original file line number Diff line number Diff line change
Expand Up @@ -152,6 +152,12 @@ for (const extraSnapshotArgs of [
{ cwd: tmpdir.path });
}

// Skipping rest of the test on Windows because it fails in the CI
// TODO(StefanStojanovic): Reenable rest of the test after fixing it
if (common.isWindows) {
return;
}

// Guarantee NODE_REPL_EXTERNAL_MODULE won't bypass kDisableNodeOptionsEnv
{
spawnSyncAndExit(
Expand Down

0 comments on commit 3a16934

Please sign in to comment.