Skip to content
This repository was archived by the owner on Feb 26, 2024. It is now read-only.

fix(spec): fix flush() behavior in handling periodic timers#881

Merged
mhevery merged 1 commit intoangular:masterfrom
vikerman:fake-async-flush
Aug 18, 2017
Merged

fix(spec): fix flush() behavior in handling periodic timers#881
mhevery merged 1 commit intoangular:masterfrom
vikerman:fake-async-flush

Conversation

@vikerman
Copy link
Copy Markdown
Contributor

FakeAsyncTest Zone now just finds the last task currently in the timer queue
and tick-s till the endTime of that task.

Solves the issue when there is a short running setInterval in front of a
longer setInterval/setTimeout.

@vikerman vikerman requested a review from mhevery August 18, 2017 20:51
Copy link
Copy Markdown
Contributor

@mhevery mhevery left a comment

Choose a reason for hiding this comment

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

Needs failing test.

FakeAsyncTest Zone now just finds the last task currently in the timer queue
and tick-s till the endTime of that task.

Solves the issue when there is a short running setInterval in front of a
longer setInterval/setTimeout.
@mhevery mhevery merged commit eed776c into angular:master Aug 18, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants