test: improve unit tests

Author: shijistar

jest.config.js

@@ -12,7 +12,7 @@ module.exports = {
   preset: 'ts-jest',
   testEnvironment: 'node',
   testMatch: ['<rootDir>/test/**/*.{spec,test}.{ts,tsx}'],
-  setupFiles: ['./test/jest.setup.ts'],
+  setupFilesAfterEnv: ['<rootDir>/test/jest.setup.ts'],
   collectCoverage: true,
   coverageThreshold: {
     global: {

src/enum-item.ts

@@ -66,23 +66,7 @@ export class EnumItemClass<
     }
     return content;
   }
-  private _isTesting() {
-    let jestWorkerId: string | undefined = undefined;
-    try {
-      jestWorkerId = process.env.JEST_WORKER_ID;
-    } catch (e) {
-      // safely ignore
-    }
-    /* istanbul ignore next */
-    if (jestWorkerId != null) {
-      return !!jestWorkerId;
-    } else {
-      /* istanbul ignore next */
-      return false;
-    }
-  }
   private _readonlyPropMsg(name: string) {
-    /* istanbul ignore next */
     return `Cannot modify property "${name}" on EnumItem. EnumItem instances are readonly and should not be mutated.`;
   }
   private _localizedProxy = new Proxy(this, {
@@ -95,31 +79,19 @@ export class EnumItemClass<
     },
     // Not allowed to edit
     set: (_, prop) => {
-      /* istanbul ignore if */
-      if (!this._isTesting()) {
-        console.warn(this._readonlyPropMsg(String(prop)));
-      }
+      console.warn(this._readonlyPropMsg(String(prop)));
       return true;
     },
     defineProperty: (_, prop) => {
-      /* istanbul ignore if */
-      if (!this._isTesting()) {
-        console.warn(this._readonlyPropMsg(String(prop)));
-      }
+      console.warn(this._readonlyPropMsg(String(prop)));
       return true;
     },
     deleteProperty: (_, prop) => {
-      /* istanbul ignore if */
-      if (!this._isTesting()) {
-        console.warn(this._readonlyPropMsg(String(prop)));
-      }
+      console.warn(this._readonlyPropMsg(String(prop)));
       return true;
     },
     setPrototypeOf: () => {
-      /* istanbul ignore if */
-      if (!this._isTesting()) {
-        console.warn('Cannot change prototype of EnumItem. EnumItem instances are immutable.');
-      }
+      console.warn('Cannot change prototype of EnumItem. EnumItem instances are immutable.');
       return true;
     },
   });

test/jest.setup.ts

@@ -1,4 +1,13 @@
 import { defaultLocalize, Enum } from '@enum-plus';
 import { getLocales, setLang } from './data/week-config';
 
-setLang('en-US', Enum, getLocales, defaultLocalize);
+beforeAll(() => {
+  setLang('en-US', Enum, getLocales, defaultLocalize);
+  jest.spyOn(console, 'warn').mockImplementation(() => {
+    // Mock console.warn to suppress warnings during tests
+  });
+});
+
+afterAll(() => {
+  jest.clearAllMocks();
+});