Add rule to require assertion count assertion

I always try to add a expect.assertions(<number of assertions>) call to the top of every test I write. It helps me keep track of the assertions I am using, and prevents accidental adding and removing of assertions.

I was thinking a rule that requires that assertion for every test would be nice. There is also expect.hasAssertions(), maybe that should be included in a separate rule? Or maybe not at all, I’ve never used it because it seems better to specify the count of assertions.

If you guys think this sounds good I would like to give a try at implementing it.

[giodamelio]

Go for it, I was going to take a crack at it but things got busy this weekend. I’ll be looking forward to it.

On Sat, Dec 30, 2017, 21:58 tushardhole [e-mail] wrote:

@SimenB https://github.com/simenb @giodamelio https://github.com/giodamelio As this is labeled as help wanted. If allowed, I would be happy to work on a PR for this 😄

Here is some https://github.com/tushardhole/eslint-plugin-jest/commit/b9cd260e483f8d9c343541fea8f3555393afe208 initial work I did.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/jest-community/eslint-plugin-jest/issues/40#issuecomment-354587058, or mute the thread https://github.com/notifications/unsubscribe-auth/AAa9LsrbDh9LcvHOisaDT6qVHJ0cu-B2ks5tFyKbgaJpZM4RM-HW .

[SimenB]

Docs released in 21.6.1