javascript/ionic-framework
1
0
Fork 0
mirror of https://github.com/ionic-team/ionic-framework.git synced 2025-08-20 12:29:55 +08:00
Code Issues Packages Projects Releases Wiki Activity

fix(overlay): ensure lifecycles fire properly (#19579)

Browse Source
This commit is contained in:
Liam DeBeasi
2019-10-08 10:41:52 -04:00
committed by GitHub
parent 02c1724978
commit a7b964279b
2 changed files with 13 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
core/src/components/modal/test/test.utils.ts
View File

@ -14,8 +14,16 @@ export const testModal = async (
});
const screenshotCompares = [];
const ionModalWillPresent = await page.spyOnEvent('ionModalWillPresent');
const ionModalDidPresent = await page.spyOnEvent('ionModalDidPresent');
const ionModalWillDismiss = await page.spyOnEvent('ionModalWillDismiss');
const ionModalDidDismiss = await page.spyOnEvent('ionModalDidDismiss');
await page.click(selector);
await ionModalWillPresent.next();
await ionModalDidPresent.next();
await page.waitForSelector(selector);
let modal = await page.find('ion-modal');
@ -25,6 +33,10 @@ export const testModal = async (
screenshotCompares.push(await page.compareScreenshot());
await modal.callMethod('dismiss');
await ionModalWillDismiss.next();
await ionModalDidDismiss.next();
await modal.waitForNotVisible();
screenshotCompares.push(await page.compareScreenshot('dismiss'));

2
core/src/utils/overlays.ts
View File

@ -221,7 +221,7 @@ const overlayAnimation = async (
/**
* TODO: Remove AnimationBuilder
*/
const hasCompleted = (typeof animationResult as any === 'boolean') ? animationResult : (animation as any).hasCompleted;
const hasCompleted = (typeof animationResult as any === 'undefined') ? true : (animation as any).hasCompleted;
if (isAnimationBuilder) {
animation.destroy();
}