fixup! feat(cdk/testing): add method to wait for async tasks to complete

Change name

Author: devversion

src/cdk/testing/component-harness.ts

@@ -177,7 +177,7 @@ export interface LocatorFactory {
    * Waits for all scheduled or running async tasks to complete. This allows harness
    * authors to wait for async tasks outside of the Angular zone.
    */
-  waitForAsyncTasksToComplete(): Promise<void>;
+  waitForTasksOutsideAngular(): Promise<void>;
 }
 
 /**
@@ -295,8 +295,8 @@ export abstract class ComponentHarness {
    * Waits for all scheduled or running async tasks to complete. This allows harness
    * authors to wait for async tasks outside of the Angular zone.
    */
-  protected async waitForAsyncTasksToComplete() {
-    return this.locatorFactory.waitForAsyncTasksToComplete();
+  protected async waitForTasksOutsideAngular() {
+    return this.locatorFactory.waitForTasksOutsideAngular();
   }
 }
 

src/cdk/testing/harness-environment.ts

@@ -131,7 +131,7 @@ export abstract class HarnessEnvironment<E> implements HarnessLoader, LocatorFac
   abstract forceStabilize(): Promise<void>;
 
   // Part of LocatorFactory interface, subclasses will implement.
-  abstract waitForAsyncTasksToComplete(): Promise<void>;
+  abstract waitForTasksOutsideAngular(): Promise<void>;
 
   /** Gets the root element for the document. */
   protected abstract getDocumentRoot(): E;

src/cdk/testing/protractor/protractor-harness-environment.ts

@@ -25,7 +25,7 @@ export class ProtractorHarnessEnvironment extends HarnessEnvironment<ElementFind
 
   async forceStabilize(): Promise<void> {}
 
-  async waitForAsyncTasksToComplete(): Promise<void> {
+  async waitForTasksOutsideAngular(): Promise<void> {
     // TODO: figure out how we can do this for the protractor environment.
     // https://github.com/angular/components/issues/17412
   }

src/cdk/testing/testbed/testbed-harness-environment.ts

@@ -64,7 +64,7 @@ export class TestbedHarnessEnvironment extends HarnessEnvironment<Element> {
     await this._fixture.whenStable();
   }
 
-  async waitForAsyncTasksToComplete(): Promise<void> {
+  async waitForTasksOutsideAngular(): Promise<void> {
     // If we run in the fake async zone, we run "flush" to run any scheduled tasks. This
     // ensures that the harnesses behave inside of the FakeAsyncTestZone similar to the
     // "AsyncTestZone" and the root zone (i.e. neither fakeAsync or async). Note that we

src/cdk/testing/tests/harnesses/main-component-harness.ts

@@ -104,7 +104,7 @@ export class MainComponentHarness extends ComponentHarness {
     await (await this.taskStateTestTrigger()).click();
     // Wait for async tasks to complete since the click caused a
     // timeout to be scheduled outside of the NgZone.
-    await this.waitForAsyncTasksToComplete();
+    await this.waitForTasksOutsideAngular();
     return (await this.taskStateTestResult()).text();
   }
 }

src/cdk/testing/tests/testbed.spec.ts

@@ -253,17 +253,17 @@ describe('TestbedHarnessEnvironment', () => {
       expect(subcomps.length).toBe(2);
     });
 
-    it('should respect tasks outside of NgZone when stabilizing within native async/await',
+    it('should be able to wait for tasks outside of Angular within native async/await',
         async () => {
       expect(await harness.getTaskStateResult()).toBe('result');
     });
 
-    it('should respect tasks outside of NgZone when stabilizing within async test zone',
+    it('should be able to wait for tasks outside of Angular within async test zone',
         async (() => {
       harness.getTaskStateResult().then(res => expect(res).toBe('result'));
     }));
 
-    it('should respect tasks outside of NgZone when stabilizing within fakeAsync test zone',
+    it('should be able to wait for tasks outside of Angular within fakeAsync test zone',
         fakeAsync(async () => {
       expect(await harness.getTaskStateResult()).toBe('result');
     }));