Skip to content
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

Make executePendingJobs public #32

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

Conversation

Matthiee
Copy link

Fixes #31

@ekibun
Copy link
Owner

ekibun commented Nov 21, 2022

Can you provide some example code for unit test?

Execute pending job cannot work for marco tasks such as setTimeout and resolving dart objects. I think only creating a new instance for each Zone(or using threadpool to reuse the instance) can meet your requirements.

@Matthiee
Copy link
Author

Hi @ekibun currently I'm using the branch in this PR and all I can say is that the async/await code is now executed in the correct Zone if we are able to call executePendingJobs from inside that zone.

Creating a new instance for each zone is not an option for us at the moment.

I'll try to provide some sample code for unit tests soon.

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.

Give more control over jsExecutePendingJob / _executePendingJob
2 participants