fix: use a static object as fallback of the global object (#1901)

* fix: use a static object as fallback of the global object

* test: add test for getGlobalObject

Author: Means88

packages/utils/src/misc.ts

@@ -20,14 +20,22 @@ export function isNodeEnv(): boolean {
   return Object.prototype.toString.call(typeof process !== 'undefined' ? process : 0) === '[object process]';
 }
 
+const fallbackGlobalObject = {};
+
 /**
  * Safely get global scope object
  *
  * @returns Global scope object
  */
 // tslint:disable:strict-type-predicates
 export function getGlobalObject(): Window | NodeJS.Global | {} {
-  return isNodeEnv() ? global : typeof window !== 'undefined' ? window : typeof self !== 'undefined' ? self : {};
+  return isNodeEnv()
+    ? global
+    : typeof window !== 'undefined'
+    ? window
+    : typeof self !== 'undefined'
+    ? self
+    : fallbackGlobalObject;
 }
 // tslint:enable:strict-type-predicates
 

packages/utils/test/misc.test.ts

@@ -1,4 +1,4 @@
-import { getEventDescription } from '../src/misc';
+import { getEventDescription, getGlobalObject } from '../src/misc';
 
 describe('getEventDescription()', () => {
   test('message event', () => {
@@ -108,3 +108,14 @@ describe('getEventDescription()', () => {
     ).toEqual('<unknown>');
   });
 });
+
+describe('getGlobalObject()', () => {
+  test('should return the same object', () => {
+    const backup = global.process;
+    delete global.process;
+    const first = getGlobalObject();
+    const second = getGlobalObject();
+    expect(first).toEqual(second);
+    global.process = backup;
+  });
+});