test: allow adding known-globals through ENV

refack · MylesBorins · commit 9f71570981f1 · 2017-10-17T00:28:16.000-04:00
PR-URL: #15187 Refs: https://youtrack.jetbrains.com/issue/WEB-27528 Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com> Reviewed-By: Timothy Gu <timothygu99@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
diff --git a/test/common/index.js b/test/common/index.js
@@ -362,6 +362,11 @@ if (global.Symbol) {
   knownGlobals.push(Symbol);
 }
 
+if (process.env.NODE_TEST_KNOWN_GLOBALS) {
+  const knownFromEnv = process.env.NODE_TEST_KNOWN_GLOBALS.split(',');
+  allowGlobals(...knownFromEnv);
+}
+
 function allowGlobals(...whitelist) {
   knownGlobals = knownGlobals.concat(whitelist);
 }

Original file line number	Diff line number	Diff line change
`@@ -362,6 +362,11 @@ if (global.Symbol) {`
`362`	`362`	`knownGlobals.push(Symbol);`
`363`	`363`	`}`
`364`	`364`
	`365`	`+if (process.env.NODE_TEST_KNOWN_GLOBALS) {`
	`366`	`+ const knownFromEnv = process.env.NODE_TEST_KNOWN_GLOBALS.split(',');`
	`367`	`+ allowGlobals(...knownFromEnv);`
	`368`	`+}`
	`369`	`+`
`365`	`370`	`function allowGlobals(...whitelist) {`
`366`	`371`	`knownGlobals = knownGlobals.concat(whitelist);`
`367`	`372`	`}`