IE 8 splice "bug" in TickScheduler.coffee

[vikikamath]

When running browserified tests in IE 8, IE 8's splice() "bug" on this line fails silently causing IE 8 to hang indefinitely.

Explicitly providing second argument to splice() on this line will make it support on IE 8 :
forNow = schedule[currentTick].splice(0, schedule[currentTick].length)

[raimohanska]

Downloading IE8 virtualbox image right now...

[raimohanska]

Hmm.. Fixing that isn't enough to make tests run on IE 8. Did you succeed? I get "Object doesn't support this property or method". I think Mocha isn't compatible with IE 8.

[vikikamath]

@raimohanska Ohh yeah thats true...I had to add polyfill for Object.keys described here to get this working.

I am using mocha version 1.21.4. Additionally I used jquery version 1.9

[raimohanska]

I'd be extra glad if you did a Pull Request with the polyfill and other possible fixes to make tests runnable in IE8!

[vikikamath]

Sure @raimohanska will do ...I forgot to mention earlier chai.js Expect matchers are not compatible with IE8. I had to replace tests with Assert API of chai.js using npm package assertive-chai

[raimohanska]

My IE8 vbox just won't work. Sorry, I don't have the time to advance this. Maybe we should convert all the tests to assertive anyways to make them runnable in older browsers?

[vikikamath]

I wasn't quite sure of how both Assertive and Expect API based tests would be hosted without being nightmare to maintain. I think it's a good idea to convert all tests to Assertive 👍 . I can make a PR.

Just in case if you were curious how tests looked on IE8, I have hosted my PR here. To view it in IE8, you can use any crossbrowser testing Services like "browserstack" (You require to register though!)