INT-3396: mark test file as zonefull

Author: michalnieruchalski-tiugo

tinymce-angular-component/src/test/ts/alien/InitTestEnvironment.ts

@@ -1,4 +1,15 @@
 import 'core-js/features/reflect';
+
+// zone.js is imported here because our test suite needs to cover both zoneless and
+// zone.js-based Angular applications. As a component library, our users may run
+// either mode, so we must ensure compatibility with both. Since Angular 21, zoneless
+// is the default, but zone.js remains supported. Once Angular drops zone.js support
+// entirely, this import, ng-zone specific tests and the zone.js devDependency can be removed.
+//
+// Note: importing zone.js patches native browser APIs (addEventListener, setTimeout,
+// setInterval, etc.), but Angular does not use these patches for change detection by
+// default. Change detection only relies on zone.js in tests that explicitly configure
+// `provideZoneChangeDetection`.
 import 'zone.js';
 import 'zone.js/plugins/fake-async-test';
 

tinymce-angular-component/src/test/ts/browser/NgZoneTest.ts

@@ -1,22 +1,29 @@
 import '../alien/InitTestEnvironment';
 
-import { NgZone } from '@angular/core';
+import { NgZone, provideZoneChangeDetection } from '@angular/core';
 import { Assertions } from '@ephox/agar';
-import { describe, it } from '@ephox/bedrock-client';
+import { beforeEach, describe, it } from '@ephox/bedrock-client';
 
 import { EditorComponent } from '../../../main/ts/editor/editor.component';
-import { eachVersionContext, fixtureHook } from '../alien/TestHooks';
+import { eachVersionContext } from '../alien/TestHooks';
 import { first } from 'rxjs';
 import { throwTimeout } from '../alien/TestHelpers';
+import { TestBed } from '@angular/core/testing';
 
-describe.skip('NgZoneTest', () => {
+describe('NgZoneTest', () => {
   eachVersionContext([ '4', '5', '6', '7', '8' ], () => {
-    const createFixture = fixtureHook(EditorComponent, { imports: [ EditorComponent ] });
+    beforeEach(async () => {
+      await TestBed.configureTestingModule({
+        imports: [ EditorComponent ],
+        providers: [ provideZoneChangeDetection() ]
+      }).compileComponents();
+    });
 
     it('Subscribers to events should run within NgZone', async () => {
-      const fixture = createFixture();
+      const fixture = TestBed.createComponent(EditorComponent);
       const editor = fixture.componentInstance;
       fixture.detectChanges();
+
       await new Promise<void>((resolve) => {
         editor.onInit.pipe(first(), throwTimeout(10000, 'Timed out waiting for init event')).subscribe(() => {
           Assertions.assertEq('Subscribers to onInit should run within NgZone', true, NgZone.isInAngularZone());
@@ -27,9 +34,10 @@ describe.skip('NgZoneTest', () => {
 
     // Lets just test one EventEmitter, if one works all should work
     it('Subscribers to onKeyUp should run within NgZone', async () => {
-      const fixture = createFixture();
+      const fixture = TestBed.createComponent(EditorComponent);
       const editor = fixture.componentInstance;
       fixture.detectChanges();
+
       await new Promise<void>((resolve) => {
         editor.onKeyUp.pipe(first(), throwTimeout(10000, 'Timed out waiting for key up event')).subscribe(() => {
           Assertions.assertEq('Subscribers to onKeyUp should run within NgZone', true, NgZone.isInAngularZone());