Skip to content

Switching to Mocha #319

Closed
tmcw opened this Issue Mar 8, 2013 · 5 comments

5 participants

@tmcw
tmcw commented Mar 8, 2013

Similar to #317 in 'massive scope and likeliness to get closed immediately:

Use mocha, sinon, and expect.js instead of Jasmine - as do most projects nowadays. Jasmine development is at a standstill with lots of unanswered PRs and Mocha is much better in terms of async tests and cross-platform runners.

@tschaub
OpenLayers member
tschaub commented Mar 8, 2013

I agree we should use Mocha (and suggested it originally). I'll see about putting together a pull request - but it won't be for at least another week, so anyone else should feel free.

@elemoine
OpenLayers member
elemoine commented Mar 8, 2013

FWIW Mocha sounds good to me. I already ported tests from jsunit to jasmine and, at this point, don't feel like doing it again for Mocha :) If someone else does it then great!

@twpayne
twpayne commented Mar 9, 2013

+1 from me for switching to Mocha.

@Turbo87
Turbo87 commented Mar 13, 2013

FYI I've started converting the tests to mocha, expect.js and sinon. Everything works fine so far, except for the async tests, that I haven't been able to convert yet.

@Turbo87 Turbo87 added a commit to Turbo87/ol3 that referenced this issue Mar 13, 2013
@Turbo87 Turbo87 Replaced jasmine testing framework by mocha, expect.js and sinon
as discussed in #319
89ab68c
@twpayne
twpayne commented Mar 14, 2013

Fixed thanks to @Turbo87's herculean efforts in #358.

@twpayne twpayne closed this Mar 14, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.