feat(tabs): add animation done event #5238

[amcdnl]

Addresses #5238 by adding an animationDone event that is triggered after the tab body transition has completed.

Closes #3032

[jelbourn]

Unit test? Even with NoopAnimationModule, it should still fire the event

[amcdnl]

It looks like the _onTranslateTabComplete function is getting called in tab-body.ts, however, the onCentered event that is captured in tab-group.ts in the _removeTabBodyWrapperHeight method is not getting triggered in the tests. Any thoughts?

[jelbourn]

What kind of flushing / waiting does the test use?

[amcdnl]

Here is the test I've written for this:

fit('should fire animation done event', async (() => {
  fixture.detectChanges();

  spyOn(fixture.componentInstance, 'animationDone');
  let tabLabel = fixture.debugElement.queryAll(By.css('.mat-tab-label'))[1];
  tabLabel.nativeElement.click();
  fixture.detectChanges();

  fixture.whenStable().then(() => {
    fixture.detectChanges();
    expect(fixture.componentInstance.animationDone).toHaveBeenCalled();
  });
}));

[amcdnl]

@jelbourn Test added :)

[jelbourn]

ERROR: /home/travis/build/angular/material2/src/lib/tabs/tab-group.spec.ts[209, 5]: Calls to 'fit' are not allowed.

[amcdnl]

@jelbourn - #facepalm - fixed.

[jelbourn]

LGTM

[tinayuangao]

@amcdnl Please rebase. Thanks!

[amcdnl]

@tinayuangao - Done :)

[angular-automatic-lock-bot]

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

_{This action has been performed automatically by a bot.}