Skip to content

Conversation

@ctaepper
Copy link
Contributor

@ctaepper ctaepper commented Feb 6, 2018

using this library results into loss of angular-sync in protractor tests due to pending macro tasks, see screenshot:

bildschirmfoto 2018-02-06 um 14 17 22

according to the protractor docs, one should run those long running tasks outside of ngzone.

this PR accomplishes that, our protractor tests don't run into timeouts

however, it seems that this repo doesnt provide any tests for the lib, so I am unsure if this change breaks something...

@ctaepper
Copy link
Contributor Author

ctaepper commented Mar 8, 2018

@manfredsteyer any input here?

@mderoche
Copy link

Would be really nice to have this pulled in... this library breaks e2e testing for any applications that use it. Is this project dead or ready for a fork?

@ctaepper
Copy link
Contributor Author

presumably not dead

@manfredsteyer
Copy link
Owner

;-) Yes, I'm still alive ;-)

Thanks for this!

@manfredsteyer manfredsteyer merged commit 62cb609 into manfredsteyer:master May 9, 2018
@manfredsteyer
Copy link
Owner

Thx

@evanrooijen
Copy link

Please also merge this fix back to version 3.x

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.

4 participants