mwinter
/
unboundedpress
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity
You cannot select more than 25 topics Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.
main
legacy_v2
legacy_v1
Branches Tags
${ item.name }
Create tag ${ searchTerm }
Create branch ${ searchTerm }
from 'legacy_v1'
${ noResults }
unboundedpress/node_modules/babel-plugin-ava-throws-helper
History
Michael Winter 555e2c196e initial commit 5 years ago
..
index.js initial commit 5 years ago
license initial commit 5 years ago
package.json initial commit 5 years ago
readme.md initial commit 5 years ago

readme.md

babel-plugin-ava-throws-helper Build Status

internal AVA internal plugin for protecting against improper use of t.throws

Probably not useful except as an internal plugin for the AVA test runner.

Genesis of the idea: https://github.com/sindresorhus/eslint-plugin-ava/issues/75

Issue

I've seen a lot of incorrect use of the throws assertion in other test runner and even done the mistake myself sometimes. Now I'm beginning to see it with AVA too, so would be nice to be preemptive about it.

People don't realize they need to wrap their call in a function, so many end up doing t.throws(foo()) instead of t.throws(() => foo());. It's an easy mistake to make.

The difficulty is that t.throws(foo()) is allowed if foo() returns a promise or a function. There is no good way to differentiate between the two at runtime. So providing a good error message is going to take some AST transform magic.

The solution

See example-output.md for the transformation this performs.

The example output can be generated by calling:

$ WRITE_EXAMPLES=1 ava

Contributors should not commit new examples. They will be regenerated as part of the release process.

License

MIT © James Talmage