-
-
Notifications
You must be signed in to change notification settings - Fork 109
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
Parallelize mocha tests #112
Comments
I usually use AVA. It was built with parralism and es6 in mind |
Initially, this project was using Ava but I had to switch to mocha because I couldn't get chaijs to work with mocha |
I also like the cleaner output of mocha. |
But if we can get these things with Ava, lets migrate to it. |
Ok. That's cool. Can you be more specific sbot what you can't do with ava? |
It's helpful to have the tests output in an ordered 'list', like they do in mocha. |
I played around with Jest for a few day. So far so good, it uses expect as it assert library so no change on tests file. And with Jest 14, they released a killer feature: Snapshot testing with React http://facebook.github.io/jest/blog/2016/07/27/jest-14.html I suggest you give it a try |
This should be a planned feature but at the moment i dont think its very urgent. When this project is stable i think we should migrate add screenshot tests to all the component tests. |
Agreed. Visual testing is slow and error prone. I forgot to mention, Jest can run test in parallel by default |
@onbjerg Can I have nested tests like this:
|
Not really. I agree with sindre that it is sort of an anti-pattern and should be discouraged. The most you can do is: import test from 'ava'
test('something', (t) => {
t.is(1, 1, '1 is 1')
}) Regardless, the thing you're asking was/is being discussed in avajs/ava#222. |
Can chai be used with ava? |
Actually, scratch that. There's a package for that(tm): https://github.com/sheerun/ava-spec.
Yes. But you might want to read a bit more about it here, as there are some trade-offs: Custom assertions. Also, read avajs/ava#25 (which is linked in that section of the README), since the trade off might not be a deal-breaker overall. |
I see. All our tests are already written in mocha. It would be worth he migration to ava if it supported running tests on multiple threads. At the moment tho there are a bunch of architectural concerns with the project that need attention right now. Are you interested in contributing? 😁 |
It runs tests in parallel per default. I might be up for contributing 🍇 |
Yesss! 🔥🎉😁 What areas are you experienced in? Node? React? Redux? etc |
Been doing Node for a few years, been doing React/Redux for a year or so 😬 I'm experienced in a lot of other things, too, but I don't think they apply to this project. |
That's great! Also I invited you to the gitter. I just commented an introduction to the project. Would recommend checking that out. |
@onbjerg just looked into jest and it looks very promising. What do you think about PR'ing for this? 😀 |
No description provided.
The text was updated successfully, but these errors were encountered: