busterjs / buster

Abandoned - A powerful suite of automated test tools for JavaScript.
http://docs.busterjs.org
Other
448 stars 37 forks source link

Fix buster/build #437

Closed GCheung55 closed 9 years ago

GCheung55 commented 9 years ago

There will likely be more necessary changes here, like removing commented lines of code.

buster/compat-0.x.js basically loads Sinon.JS's IE compatible code, but it actually causes issues in IE8. I've sent a pull request to fix this by wrapping the offending code in it's own closure: https://github.com/cjohansen/Sinon.JS/pull/628.

I spun up an http server and the tests were successful at http://localhost:8081/doc/samples/strftime.html in the following browsers I've tested: