fixup! src: fix external module env and kDisableNodeOptionsEnv

RafaelGSS · RafaelGSS · commit ccfa477751f7 · 2024-05-17T11:20:09.000-03:00
diff --git a/src/node.cc b/src/node.cc
@@ -903,7 +903,7 @@ static ExitCode InitializeNodeWithArgsInternal(
         !node_repl_external_env.empty()) {
       errors->emplace_back("NODE_REPL_EXTERNAL_MODULE can't be used with "
                            "kDisableNodeOptionsEnv");
-      return ExitCode::kBootstrapFailure;
+      return ExitCode::kInvalidCommandLineArgument;
     }
   }
 #endif
diff --git a/test/embedding/test-embedding.js b/test/embedding/test-embedding.js
@@ -163,7 +163,7 @@ for (const extraSnapshotArgs of [
       },
     },
     {
-      status: 10,
+      status: 9,
       signal: null,
       stderr: `${binary}: NODE_REPL_EXTERNAL_MODULE can't be used with kDisableNodeOptionsEnv\n`,
     });

Original file line number	Diff line number	Diff line change
`@@ -903,7 +903,7 @@ static ExitCode InitializeNodeWithArgsInternal(`
`903`	`903`	`!node_repl_external_env.empty()) {`
`904`	`904`	`errors->emplace_back("NODE_REPL_EXTERNAL_MODULE can't be used with "`
`905`	`905`	`"kDisableNodeOptionsEnv");`
`906`		`- return ExitCode::kBootstrapFailure;`
	`906`	`+ return ExitCode::kInvalidCommandLineArgument;`
`907`	`907`	`}`
`908`	`908`	`}`
`909`	`909`	`#endif`
Original file line number	Diff line number	Diff line change
`@@ -163,7 +163,7 @@ for (const extraSnapshotArgs of [`
`163`	`163`	`},`
`164`	`164`	`},`
`165`	`165`	`{`
`166`		`- status: 10,`
	`166`	`+ status: 9,`
`167`	`167`	`signal: null,`
`168`	`168`	stderr: `${binary}: NODE_REPL_EXTERNAL_MODULE can't be used with kDisableNodeOptionsEnv\n`,
`169`	`169`	`});`