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

Fix unit test in IE7 #360

Closed
wants to merge 1 commit into from
Closed

Conversation

vojtajina
Copy link
Contributor

This commit fixes 2 tests in IE7, so now all tests pass in IE7, IE8 on my machine.

Unfortunately there is still one test, that is even not executed in IE - there are 810 tests executed in Chrome / FF / Opera, but only 809 tests in IE7 / IE8.
That's problem of JSTD, it silently ignore tests with syntax errors:
http://code.google.com/p/js-test-driver/issues/detail?id=21

@vojtajina
Copy link
Contributor Author

Just for future info, the one test which is not executed on IE is not an issue, as there is a condition to run the test only if not IE browser:
https://github.com/vojtajina/angular.js/blob/master/test/AngularSpec.js#L576

@IgorMinar IgorMinar closed this Jun 2, 2011
@IgorMinar
Copy link
Contributor

angular.js: master Vojta Jina * aa64d37 (3 files in 2 dirs): Fix unit test in IE7 - http://bit.ly/jCfSH5

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants