Skip to content

Conversation

edison1105
Copy link
Member

@edison1105 edison1105 commented Jun 17, 2021

fix #3963

Copy link
Member

@posva posva left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you add a test?

@edison1105
Copy link
Member Author

Can you add a test?

yes, too busy. I will do it later.

@edison1105 edison1105 requested a review from posva June 17, 2021 14:27
@edison1105
Copy link
Member Author

@posva
the test case has done. feel free to review.
time to go to bed.

await click('#toggleBtn')
await timeout(500)
expect(await html('#container')).toBe('<div class="">Loading...</div>')
await transitionFinish(1500)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Avoid using hard coded test durations like this.

See 740bcd1

@yyx990803 yyx990803 merged commit 43e2a72 into vuejs:master Jun 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Suspense emits fallback but does not displays the slot when using a transition

3 participants