Good use of `t.plan(n)` rule about eslint-plugin-ava HOT 4 OPEN

Good idea, but it should align with https://github.com/sindresorhus/ava/blob/master/docs/recipes/when-to-use-plan.md. There's usually no reason to use t.plan() with sync tests, unless with branching.

This rule should also ensure you use t.plan() in callback tests.

from eslint-plugin-ava.

Man I should really take a deeper look at the recipes. Will do that soon(-ish) and come back afterwards :d

from eslint-plugin-ava.

Check that the number of assertions equals n

This breaks a case, when you run assertions in a loop:

t.plan(4);

arr.forEach(item => {
  t.true(item);
});

Linter will count it as one assertion, thus not matching t.plan().

from eslint-plugin-ava.

Yeah, thought of that afterwards, don't think it is really possible...

from eslint-plugin-ava.