Skip to content
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

Benchmark #460

Closed
sindresorhus opened this issue Jan 20, 2016 · 7 comments
Closed

Benchmark #460

sindresorhus opened this issue Jan 20, 2016 · 7 comments

Comments

@sindresorhus
Copy link
Member

Re https://twitter.com/sindresorhus/status/689851060303872000

We should set up a benchmark with various types of test suites, like one with sync tests in one file, one with multiple files and async tests. This to make sure we don't regress in performance and ability to compare ourselves to other test runners.

@kentcdodds
Copy link
Contributor

Thanks for taking this so seriously! 😄 I'm currently working on a repo to compare mocha and ava with a lot of tests with a lot of files. I'm no benchmark expert, but maybe this will be helpful. Will comment when I've pushed it up. Thanks!

@jfmengels
Copy link
Contributor

I have created this repo for the same purpose: https://github.com/jfmengels/mocha-vs-ava-performance
It's pretty small, but it makes it obvious how big the difference is between Mocha's instant feedback in watch mode, and Ava's "watch" with Nodemon, or even single runs.

@jamestalmage
Copy link
Contributor

watch mode

AVA is at an extreme disadvantage in watch mode until we implement our own watch mechanism. That's certainly on the roadmap, but we've been punting on it in favor of other improvements. Not sure if it's time to tackle that yet or not.

@jamestalmage
Copy link
Contributor

I have confirmed this commit caused a major performance hit. I think I've identified the problem and I am working up a solution.

See twada/empower-core#7

@twada
Copy link
Contributor

twada commented Jan 21, 2016

Just released empower-core 0.5.0 🎉 (Thanks @jamestalmage !!)

@sindresorhus
Copy link
Member Author

@jamestalmage did a superb job adding a pretty benchmark in #496. It's easy to add additional things to benchmark, so suggestions welcome.

@sindresorhus
Copy link
Member Author

Instructions here: c22eabd

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

No branches or pull requests

5 participants