Thanks to visit codestin.com
Credit goes to github.com

Skip to content

refactor(zone.js): Add internal implementation for auto ticking fakeA… #62135

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

atscott
Copy link
Contributor

@atscott atscott commented Jun 18, 2025

…sync

Benefits of auto-ticking mock clocks have been described in other PRs, such as jasmine/jasmine#2042 and sinonjs/fake-timers#509. In short, fakeAsync cannot work when some tasks are required to be truly async, such as XHRs or observers like ResizeObserver. In addition, auto ticking mock clocks can be applied to tests without the tests then needing to update everything to manually flush timers.

…sync

Benefits of auto-ticking mock clocks have been described in other PRs,
such as jasmine/jasmine#2042 and
sinonjs/fake-timers#509. In short, `fakeAsync`
cannot work when some tasks are required to be truly async, such as XHRs
or observers like ResizeObserver. In addition, auto ticking mock clocks
can be applied to tests without the tests then needing to update
everything to manually flush timers.
@angular-robot angular-robot bot added the area: zones Issues related to zone.js label Jun 18, 2025
@ngbot ngbot bot added this to the Backlog milestone Jun 18, 2025
@atscott atscott added the target: patch This PR is targeted for the next patch release label Jun 18, 2025
@atscott atscott requested a review from alxhub June 20, 2025 18:43
@atscott atscott marked this pull request as ready for review June 20, 2025 18:43
@pullapprove pullapprove bot requested a review from JiaLiPassion June 20, 2025 18:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: zones Issues related to zone.js target: patch This PR is targeted for the next patch release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant