From 30ecb6e88ac4ad77cafa0737f908b8931e4220d8 Mon Sep 17 00:00:00 2001 From: Tobias Bosch Date: Wed, 18 Oct 2017 09:57:56 -0700 Subject: [PATCH] Revert "test(animations): ensure :enter callbacks fire on container insertion (#19674)" This reverts commit 41f57affb6ae0688ed4b6e4704bc8cf2201bc01a. As it was synched together with 5a9ed2de272912ac74ed56bfec4bdf4551f7b38e which broke an internal test. --- .../animation/animation_integration_spec.ts | 36 ------------------- 1 file changed, 36 deletions(-) diff --git a/packages/core/test/animation/animation_integration_spec.ts b/packages/core/test/animation/animation_integration_spec.ts index 9b27d99efd..86e76e40d0 100644 --- a/packages/core/test/animation/animation_integration_spec.ts +++ b/packages/core/test/animation/animation_integration_spec.ts @@ -340,42 +340,6 @@ export function main() { expect(completed).toBe(true); })); - it('should always fire inner callbacks even if no animation is fired when a view is inserted', - fakeAsync(() => { - @Component({ - selector: 'if-cmp', - template: ` -
-
-
- `, - animations: [ - trigger('myAnimation', []), - ] - }) - class Cmp { - exp: any = false; - log: string[] = []; - track(event: any) { this.log.push(`${event.triggerName}-${event.phaseName}`); } - } - - TestBed.configureTestingModule({declarations: [Cmp]}); - - const engine = TestBed.get(ɵAnimationEngine); - const fixture = TestBed.createComponent(Cmp); - const cmp = fixture.componentInstance; - fixture.detectChanges(); - flushMicrotasks(); - - expect(cmp.log).toEqual([]); - - cmp.exp = true; - fixture.detectChanges(); - flushMicrotasks(); - - expect(cmp.log).toEqual(['myAnimation-start', 'myAnimation-done']); - })); - it('should only turn a view removal as into `void` state transition', () => { @Component({ selector: 'if-cmp',