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

Support timeout mocks on ie8 #54

Merged
merged 1 commit into from
Oct 31, 2014
Merged

Conversation

SevInf
Copy link
Contributor

@SevInf SevInf commented Oct 31, 2014

See http://www.adequatelygood.com/Replacing-setTimeout-Globally.html
for description of the problems and solutions.
Jasmine's boot.js already has this hack.

Fix #41

@SevInf
Copy link
Contributor Author

SevInf commented Oct 31, 2014

@vojtajina can you please look at it?

@maksimr
Copy link
Contributor

maksimr commented Oct 31, 2014

@SevInf thanks!

Can you change commit message following commit message style guide.

Soon we change on peer dependency.

See http://www.adequatelygood.com/Replacing-setTimeout-Globally.html
for description of the problems and solutions.
Jasmine's boot.js already has this hack.

Closes karma-runner#41
@SevInf
Copy link
Contributor Author

SevInf commented Oct 31, 2014

@maksimr, done.
As for peer dependency, as I can see, karma-jasmine will still have its own boot.js, so this fix is still relevant.

maksimr added a commit that referenced this pull request Oct 31, 2014
@maksimr maksimr merged commit ca379d2 into karma-runner:master Oct 31, 2014
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.

Mocking the JavaScript Timeout Functions in IE 8 with Jasmine 2.0.0
2 participants