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

fix(spec): fix #760, fakeAsyncTestSpec should handle microtask with additional args #762

Merged
merged 1 commit into from
May 2, 2017
Merged

fix(spec): fix #760, fakeAsyncTestSpec should handle microtask with additional args #762

merged 1 commit into from
May 2, 2017

Conversation

JiaLiPassion
Copy link
Collaborator

fix #760.

MicroTask callback may have additional arguments,
such as

process.nextTick(() => {...}, arg1, arg2);

fakeAsyncTestSpec should be able to send those args(arg1, arg2) to the original nextTick function
correctly.

@JiaLiPassion JiaLiPassion mentioned this pull request Apr 27, 2017
Closed
@mhevery mhevery merged commit f8d17ac into angular:master May 2, 2017
@JiaLiPassion JiaLiPassion deleted the issue-760 branch May 6, 2017 04:52
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@mhevery mhevery mhevery approved these changes

Assignees
No one assigned
Labels
cla: yes
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

fakeAsync in node throws TypeError: Cannot read property 'length' of undefined
3 participants
@JiaLiPassion @mhevery @googlebot