test: fix Windows native test suites

StefanStojanovic · marco-ippolito · commit 3a1693421da4 · 2024-07-19T15:26:56.000+02:00
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>
diff --git a/test/embedding/test-embedding.js b/test/embedding/test-embedding.js
@@ -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(

Original file line number	Diff line number	Diff line change
`@@ -152,6 +152,12 @@ for (const extraSnapshotArgs of [`
`152`	`152`	`{ cwd: tmpdir.path });`
`153`	`153`	`}`
`154`	`154`
	`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`	`+`
`155`	`161`	`// Guarantee NODE_REPL_EXTERNAL_MODULE won't bypass kDisableNodeOptionsEnv`
`156`	`162`	`{`
`157`	`163`	`spawnSyncAndExit(`