test: fix Windows native test suites

StefanStojanovic · StefanStojanovic · commit 169b0f8a0076 · 2024-05-27T09:57:19.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
diff --git a/test/embedding/test-embedding.js b/test/embedding/test-embedding.js
@@ -152,6 +152,10 @@ for (const extraSnapshotArgs of [
     { cwd: tmpdir.path });
 }
 
+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,10 @@ for (const extraSnapshotArgs of [`
`152`	`152`	`{ cwd: tmpdir.path });`
`153`	`153`	`}`
`154`	`154`
	`155`	`+if (common.isWindows) {`
	`156`	`+ return;`
	`157`	`+}`
	`158`	`+`
`155`	`159`	`// Guarantee NODE_REPL_EXTERNAL_MODULE won't bypass kDisableNodeOptionsEnv`
`156`	`160`	`{`
`157`	`161`	`spawnSyncAndExit(`