test: replace more uses of global with globalThis

PR-URL: #56712
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>

Author: jasnell

test/parallel/test-aborted-util.js

@@ -32,7 +32,7 @@ test('Aborted with gc cleanup', async () => {
   const { promise, resolve } = Promise.withResolvers();
 
   setImmediate(() => {
-    global.gc();
+    globalThis.gc();
     ac.abort();
     strictEqual(ac.signal.aborted, true);
     strictEqual(getEventListeners(ac.signal, 'abort').length, 0);

test/parallel/test-abortsignal-drop-settled-signals.mjs

@@ -16,7 +16,7 @@ function makeSubsequentCalls(limit, done, holdReferences = false) {
       // This setImmediate is necessary to ensure that in the last iteration the remaining signal is GCed (if not
       // retained)
       setImmediate(() => {
-        global.gc();
+        globalThis.gc();
         done(ac.signal, dependantSymbol);
       });
       return;
@@ -50,7 +50,7 @@ function runShortLivedSourceSignal(limit, done) {
 
   function run(iteration) {
     if (iteration > limit) {
-      global.gc();
+      globalThis.gc();
       done(signalRefs);
       return;
     }
@@ -74,9 +74,9 @@ function runWithOrphanListeners(limit, done) {
     const ac = new AbortController();
     if (iteration > limit) {
       setImmediate(() => {
-        global.gc();
+        globalThis.gc();
         setImmediate(() => {
-          global.gc();
+          globalThis.gc();
 
           done(composedSignalRefs);
         });
@@ -147,7 +147,7 @@ it('drops settled dependant signals when signal is composite', (t, done) => {
   t.assert.strictEqual(controllers[1].signal[kDependantSignals].size, 1);
 
   setImmediate(() => {
-    global.gc({ execution: 'async' }).then(async () => {
+    globalThis.gc({ execution: 'async' }).then(async () => {
       await gcUntil('all signals are GCed', () => {
         const totalDependantSignals = Math.max(
           controllers[0].signal[kDependantSignals].size,

test/parallel/test-assert-checktag.js

@@ -49,13 +49,13 @@ test('', { skip: !hasCrypto }, () => {
 
   {  // At the moment global has its own type tag
     const fakeGlobal = {};
-    Object.setPrototypeOf(fakeGlobal, Object.getPrototypeOf(global));
-    for (const prop of Object.keys(global)) {
+    Object.setPrototypeOf(fakeGlobal, Object.getPrototypeOf(globalThis));
+    for (const prop of Object.keys(globalThis)) {
       fakeGlobal[prop] = global[prop];
     }
-    assert.notDeepEqual(fakeGlobal, global);
+    assert.notDeepEqual(fakeGlobal, globalThis);
     // Message will be truncated anyway, don't validate
-    assert.throws(() => assert.deepStrictEqual(fakeGlobal, global),
+    assert.throws(() => assert.deepStrictEqual(fakeGlobal, globalThis),
                   assert.AssertionError);
   }
 

test/parallel/test-async-hooks-destroy-on-gc.js

@@ -22,6 +22,6 @@ let asyncId = null;
 }
 
 setImmediate(() => {
-  global.gc();
+  globalThis.gc();
   setImmediate(() => assert.ok(destroyedIds.has(asyncId)));
 });

test/parallel/test-async-hooks-disable-gc-tracking.js

@@ -14,7 +14,7 @@ const hook = async_hooks.createHook({
 new async_hooks.AsyncResource('foobar', { requireManualDestroy: true });
 
 setImmediate(() => {
-  global.gc();
+  globalThis.gc();
   setImmediate(() => {
     hook.disable();
   });

test/parallel/test-async-hooks-prevent-double-destroy.js

@@ -17,7 +17,7 @@ const hook = async_hooks.createHook({
 }
 
 setImmediate(() => {
-  global.gc();
+  globalThis.gc();
   setImmediate(() => {
     hook.disable();
   });

test/parallel/test-cli-eval.js

@@ -345,7 +345,7 @@ child.exec(
 // Regression test for https://github.com/nodejs/node/issues/45336
 child.execFile(process.execPath,
                ['-p',
-                'Object.defineProperty(global, "fs", { configurable: false });' +
+                'Object.defineProperty(globalThis, "fs", { configurable: false });' +
                 'fs === require("node:fs")'],
                common.mustSucceed((stdout) => {
                  assert.match(stdout, /^true/);

test/parallel/test-common-gc.js

@@ -5,10 +5,10 @@ const { onGC } = require('../common/gc');
 
 {
   onGC({}, { ongc: common.mustCall() });
-  global.gc();
+  globalThis.gc();
 }
 
 {
   onGC(process, { ongc: common.mustNotCall() });
-  global.gc();
+  globalThis.gc();
 }

test/parallel/test-console-assign-undefined.js

@@ -1,28 +1,28 @@
 'use strict';
 
-// Patch global.console before importing modules that may modify the console
+// Patch globalThis.console before importing modules that may modify the console
 // object.
 
-const tmp = global.console;
-global.console = 42;
+const tmp = globalThis.console;
+globalThis.console = 42;
 
 require('../common');
 const assert = require('assert');
 
 // Originally the console had a getter. Test twice to verify it had no side
 // effect.
-assert.strictEqual(global.console, 42);
-assert.strictEqual(global.console, 42);
+assert.strictEqual(globalThis.console, 42);
+assert.strictEqual(globalThis.console, 42);
 
 assert.throws(
   () => console.log('foo'),
   { name: 'TypeError' }
 );
 
-global.console = 1;
-assert.strictEqual(global.console, 1);
+globalThis.console = 1;
+assert.strictEqual(globalThis.console, 1);
 assert.strictEqual(console, 1);
 
 // Reset the console
-global.console = tmp;
+globalThis.console = tmp;
 console.log('foo');

test/parallel/test-console-instance.js

@@ -36,9 +36,9 @@ process.stdout.write = process.stderr.write = common.mustNotCall();
 // Make sure that the "Console" function exists.
 assert.strictEqual(typeof Console, 'function');
 
-assert.strictEqual(requiredConsole, global.console);
+assert.strictEqual(requiredConsole, globalThis.console);
 // Make sure the custom instanceof of Console works
-assert.ok(global.console instanceof Console);
+assert.ok(globalThis.console instanceof Console);
 assert.ok(!({} instanceof Console));
 
 // Make sure that the Console constructor throws