test: allow adding known-globals through ENV

refack · refack · commit b8d532c4bdfe · 2017-09-13T17:07:00.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
@@ -412,6 +412,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
`@@ -412,6 +412,11 @@ if (global.Symbol) {`
`412`	`412`	`knownGlobals.push(Symbol);`
`413`	`413`	`}`
`414`	`414`
	`415`	`+if (process.env.NODE_TEST_KNOWN_GLOBALS) {`
	`416`	`+ const knownFromEnv = process.env.NODE_TEST_KNOWN_GLOBALS.split(',');`
	`417`	`+ allowGlobals(...knownFromEnv);`
	`418`	`+}`
	`419`	`+`
`415`	`420`	`function allowGlobals(...whitelist) {`
`416`	`421`	`knownGlobals = knownGlobals.concat(whitelist);`
`417`	`422`	`}`