refactor: remove setup-jest files

BREAKING CHANGE - Use `setupZoneTestEnv` as a replacement for zone test environment setup - Use `setupZonelessTestEnv` as a replacement for zoneless test environment setup
thymikee · Feb 28, 2025 · 6bf89a4 · 6bf89a4
1 parent 94dd3b1
commit 6bf89a4
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 108 deletions.
diff --git a/scripts/test-examples.js b/scripts/test-examples.js
@@ -34,13 +34,11 @@ const executeTest = (projectPath) => {
     logger.log('installing bundled version of jest-preset-angular');
     logger.log();
 
-    ['build', 'presets', 'jest-preset.js', 'package.json', 'setup-jest.js', 'setup-jest.mjs', 'setup-env'].forEach(
-        (asset) => {
-            const assetToReplace = join(projectPath, 'node_modules', 'jest-preset-angular', asset);
-            const assetToCopy = join(rootDir, asset);
-            copySync(assetToCopy, assetToReplace, {});
-        },
-    );
+    ['build', 'presets', 'jest-preset.js', 'package.json', 'setup-env'].forEach((asset) => {
+        const assetToReplace = join(projectPath, 'node_modules', 'jest-preset-angular', asset);
+        const assetToCopy = join(rootDir, asset);
+        copySync(assetToCopy, assetToReplace, {});
+    });
 
     // then we can run the tests
     const cmdLine = ['yarn', 'test'];

diff --git a/setup-jest.js b/setup-jest.js
diff --git a/setup-jest.mjs b/setup-jest.mjs
diff --git a/src/config/setup-jest.spec.ts → src/config/setup-env.spec.ts b/src/config/setup-jest.spec.ts → src/config/setup-env.spec.ts
@@ -47,7 +47,7 @@ jest.mock('@angular/core', () => {
     };
 });
 
-describe('setup-jest', () => {
+describe('Setup env utilities', () => {
     const assertOnInitTestEnv = (): void => {
         expect(mockGetTestBed).toHaveBeenCalled();
         expect(mockInitTestEnvironment.mock.calls[0][0][0]).toBeInstanceOf(BrowserDynamicTestingModuleStub);
@@ -64,33 +64,12 @@ describe('setup-jest', () => {
     };
 
     beforeEach(() => {
-        delete globalThis.ngJest;
         delete globalThis.TextEncoder;
         jest.clearAllMocks();
         jest.resetModules();
     });
 
-    describe('for CJS setup-jest, test environment initialization', () => {
-        it('should setup test environment with setup-jest file', async () => {
-            globalThis.ngJest = {
-                testEnvironmentOptions: {
-                    teardown: {
-                        destroyAfterEach: false,
-                        rethrowErrors: true,
-                    },
-                    errorOnUnknownElements: true,
-                    errorOnUnknownProperties: true,
-                },
-            };
-
-            await import('../../setup-jest.js');
-
-            expect(globalThis.TextEncoder).toBeDefined();
-            expect(mockZoneJs).toHaveBeenCalled();
-            expect(mockZoneJsTesting).toHaveBeenCalled();
-            assertOnInitTestEnv();
-        });
-
+    describe('for CJS, test environment initialization', () => {
         it('should setup test environment with setupZoneTestEnv()', async () => {
             const { setupZoneTestEnv } = await import('../../setup-env/zone/index.js');
 
@@ -129,27 +108,7 @@ describe('setup-jest', () => {
         });
     });
 
-    describe('for ESM setup-jest, test environment initialization', () => {
-        it('should setup test environment with setup-jest file', async () => {
-            globalThis.ngJest = {
-                testEnvironmentOptions: {
-                    teardown: {
-                        destroyAfterEach: false,
-                        rethrowErrors: true,
-                    },
-                    errorOnUnknownElements: true,
-                    errorOnUnknownProperties: true,
-                },
-            };
-
-            await import('../../setup-jest.mjs');
-
-            expect(globalThis.TextEncoder).toBeDefined();
-            expect(mockZoneJs).toHaveBeenCalled();
-            expect(mockZoneJsTesting).toHaveBeenCalled();
-            assertOnInitTestEnv();
-        });
-
+    describe('for ESM, test environment initialization', () => {
         it('should setup test environment with setupZoneTestEnv()', async () => {
             const { setupZoneTestEnv } = await import('../../setup-env/zone/index.mjs');