test(multiple): clean up manual ngOnDestroy calls (#23423)

We used to have to call `ngOnDestroy` manually on providers, because `TestBed` wouldn't do it for us. Now that we're using the `teardown` flag, we don't have to do it anymore.

Author: crisbeto

src/cdk-experimental/dialog/dialog.spec.ts

@@ -42,7 +42,6 @@ describe('Dialog', () => {
   let testViewContainerRef: ViewContainerRef;
   let viewContainerFixture: ComponentFixture<ComponentWithChildViewContainer>;
   let mockLocation: SpyLocation;
-  let overlayContainer: OverlayContainer;
 
   beforeEach(fakeAsync(() => {
     TestBed.configureTestingModule({
@@ -59,7 +58,6 @@ describe('Dialog', () => {
       (d: Dialog, l: Location, o: OverlayContainer) => {
     dialog = d;
     mockLocation = l as SpyLocation;
-    overlayContainer = o;
     overlayContainerElement = o.getContainerElement();
   }));
 
@@ -70,8 +68,6 @@ describe('Dialog', () => {
     testViewContainerRef = viewContainerFixture.componentInstance.childViewContainer;
   });
 
-  afterEach(() => overlayContainer.ngOnDestroy());
-
   it('should open a dialog with a component', () => {
     let dialogRef = dialog.openFromComponent(PizzaMsg, {
       viewContainerRef: testViewContainerRef

src/cdk-experimental/popover-edit/popover-edit.spec.ts

@@ -4,10 +4,9 @@ import {CdkTableModule} from '@angular/cdk/table';
 import {dispatchKeyboardEvent} from '../../cdk/testing/private';
 import {CommonModule} from '@angular/common';
 import {Component, Directive, ElementRef, ViewChild} from '@angular/core';
-import {ComponentFixture, fakeAsync, flush, TestBed, tick, inject} from '@angular/core/testing';
+import {ComponentFixture, fakeAsync, flush, TestBed, tick} from '@angular/core/testing';
 import {FormsModule, NgForm} from '@angular/forms';
 import {BidiModule, Direction} from '@angular/cdk/bidi';
-import {OverlayContainer} from '@angular/cdk/overlay';
 import {BehaviorSubject} from 'rxjs';
 
 import {
@@ -371,29 +370,18 @@ describe('CDK Popover Edit', () => {
     describe(label, () => {
       let component: BaseTestComponent;
       let fixture: ComponentFixture<BaseTestComponent>;
-      let overlayContainer: OverlayContainer;
 
       beforeEach(fakeAsync(() => {
         TestBed.configureTestingModule({
           imports: [CdkTableModule, CdkPopoverEditModule, CommonModule, FormsModule, BidiModule],
           declarations: [componentClass],
         }).compileComponents();
-        inject([OverlayContainer], (oc: OverlayContainer) => {
-          overlayContainer = oc;
-        })();
         fixture = TestBed.createComponent<BaseTestComponent>(componentClass);
         component = fixture.componentInstance;
         fixture.detectChanges();
         tick(10);
       }));
 
-      afterEach(() => {
-        // The overlay container's `ngOnDestroy` won't be called between test runs so we need
-        // to call it ourselves, in order to avoid leaking containers between tests and potentially
-        // throwing off `querySelector` calls.
-        overlayContainer.ngOnDestroy();
-      });
-
       describe('row hover content', () => {
         it('makes the first and last rows focusable but invisible', fakeAsync(() => {
           const rows = component.getRows();

src/cdk/a11y/aria-describer/aria-describer.spec.ts

@@ -21,10 +21,6 @@ describe('AriaDescriber', () => {
     fixture.detectChanges();
   }
 
-  afterEach(() => {
-    ariaDescriber.ngOnDestroy();
-  });
-
   it('should initialize without the message container', () => {
     createFixture();
     expect(getMessagesContainer()).toBeNull();

src/cdk/a11y/focus-monitor/focus-monitor.spec.ts

@@ -247,15 +247,13 @@ describe('FocusMonitor', () => {
     tick();
 
     expect(buttonElement.classList.length)
-
       .withContext('button should have exactly 2 focus classes')
       .toBe(2);
 
     focusMonitor.stopMonitoring(buttonElement);
     fixture.detectChanges();
 
     expect(buttonElement.classList.length)
-
       .withContext('button should not have any focus classes')
       .toBe(0);
   }));
@@ -266,7 +264,6 @@ describe('FocusMonitor', () => {
     tick();
 
     expect(buttonElement.classList.length)
-
       .withContext('button should have exactly 2 focus classes')
       .toBe(2);
 
@@ -275,7 +272,6 @@ describe('FocusMonitor', () => {
     fixture.detectChanges();
 
     expect(buttonElement.classList.length)
-
       .withContext('button should not have any focus classes')
       .toBe(0);
   }));

src/cdk/a11y/input-modality/input-modality-detector.spec.ts

@@ -32,11 +32,6 @@ describe('InputModalityDetector', () => {
     detector = TestBed.inject(InputModalityDetector);
   }
 
-  afterEach(() => {
-    detector?.ngOnDestroy();
-    detector = undefined!;
-  });
-
   it('should do nothing on non-browser platforms', () => {
     setupTest(false);
     expect(detector.mostRecentModality).toBe(null);

src/cdk/a11y/live-announcer/live-announcer.spec.ts

@@ -28,12 +28,6 @@ describe('LiveAnnouncer', () => {
       fixture = TestBed.createComponent(TestApp);
     })));
 
-    afterEach(() => {
-      // In our tests we always remove the current live element, in
-      // order to avoid having multiple announcer elements in the DOM.
-      announcer.ngOnDestroy();
-    });
-
     it('should correctly update the announce text', fakeAsync(() => {
       let buttonElement = fixture.debugElement.query(By.css('button'))!.nativeElement;
       buttonElement.click();
@@ -116,7 +110,6 @@ describe('LiveAnnouncer', () => {
     }));
 
     it('should ensure that there is only one live element at a time', fakeAsync(() => {
-      announcer.ngOnDestroy();
       fixture.destroy();
 
       TestBed.resetTestingModule().configureTestingModule({
@@ -278,12 +271,6 @@ describe('CdkAriaLive', () => {
     flush();
   })));
 
-  afterEach(fakeAsync(() => {
-    // In our tests we always remove the current live element, in
-    // order to avoid having multiple announcer elements in the DOM.
-    announcer.ngOnDestroy();
-  }));
-
   it('should default politeness to polite', fakeAsync(() => {
     fixture.componentInstance.content = 'New content';
     fixture.detectChanges();

src/cdk/drag-drop/drag-drop-registry.spec.ts

@@ -28,10 +28,6 @@ describe('DragDropRegistry', () => {
     })();
   }));
 
-  afterEach(() => {
-    registry.ngOnDestroy();
-  });
-
   it('should be able to start dragging an item', () => {
     const item = new DragItem();
 

src/cdk/overlay/dispatchers/overlay-keyboard-dispatcher.spec.ts

@@ -2,7 +2,7 @@ import {TestBed, inject} from '@angular/core/testing';
 import {dispatchKeyboardEvent} from '../../testing/private';
 import {ESCAPE} from '@angular/cdk/keycodes';
 import {Component, NgModule} from '@angular/core';
-import {OverlayModule, OverlayContainer, Overlay} from '../index';
+import {OverlayModule, Overlay} from '../index';
 import {OverlayKeyboardDispatcher} from './overlay-keyboard-dispatcher';
 import {ComponentPortal} from '@angular/cdk/portal';
 
@@ -22,10 +22,6 @@ describe('OverlayKeyboardDispatcher', () => {
     })();
   });
 
-  afterEach(inject([OverlayContainer], (overlayContainer: OverlayContainer) => {
-    overlayContainer.ngOnDestroy();
-  }));
-
   it('should track overlays in order as they are attached and detached', () => {
     const overlayOne = overlay.create();
     const overlayTwo = overlay.create();

src/cdk/overlay/dispatchers/overlay-outside-click-dispatcher.spec.ts

@@ -1,7 +1,7 @@
 import {TestBed, inject, fakeAsync} from '@angular/core/testing';
 import {Component, NgModule} from '@angular/core';
 import {dispatchFakeEvent, dispatchMouseEvent} from '../../testing/private';
-import {OverlayModule, OverlayContainer, Overlay} from '../index';
+import {OverlayModule, Overlay} from '../index';
 import {OverlayOutsideClickDispatcher} from './overlay-outside-click-dispatcher';
 import {ComponentPortal} from '@angular/cdk/portal';
 
@@ -21,10 +21,6 @@ describe('OverlayOutsideClickDispatcher', () => {
     })();
   });
 
-  afterEach(inject([OverlayContainer], (overlayContainer: OverlayContainer) => {
-    overlayContainer.ngOnDestroy();
-  }));
-
   it('should track overlays in order as they are attached and detached', () => {
     const overlayOne = overlay.create();
     const overlayTwo = overlay.create();

src/cdk/overlay/fullscreen-overlay-container.spec.ts

@@ -6,7 +6,6 @@ import {Overlay, OverlayContainer, OverlayModule, FullscreenOverlayContainer} fr
 
 describe('FullscreenOverlayContainer', () => {
   let overlay: Overlay;
-  let overlayContainer: FullscreenOverlayContainer;
   let fullscreenListeners: Set<Function>;
   let fakeDocument: any;
 
@@ -58,13 +57,11 @@ describe('FullscreenOverlayContainer', () => {
     }).compileComponents();
   }));
 
-  beforeEach(inject([Overlay, OverlayContainer], (o: Overlay, oc: OverlayContainer) => {
+  beforeEach(inject([Overlay], (o: Overlay) => {
     overlay = o;
-    overlayContainer = oc as FullscreenOverlayContainer;
   }));
 
   afterEach(() => {
-    overlayContainer.ngOnDestroy();
     fakeDocument = null;
   });