no-inline-assertions rule

[sindresorhus]

For readability reasons, I try to not use inline arrow functions when the function doesn't return, to make it clear it doesn't return. I would like to prevent assertions from being called in an inline arrow function test. This might seem like an edge-case, but I've seen it many times and have had to comment about it on multiple pull requests.

Fail

test('foo', t => t.true(fn()));

Pass

test('foo', t => {
    t.true(fn());
});

---

IssueHunt Summary

jlhwung has been rewarded.

Backers (Total: $60.00)

• issuehunt ($60.00)

Submitted pull Requests

• #262 WIP: Add no-inline-assertion rule (Fixes #152)

---

Tips

• Checkout the Issuehunt explorer to discover more funded issues.
• Need some help from other developers? Add your repositories on IssueHunt to raise funds.

---

IssueHunt has been backed by the following sponsors. Become a sponsor

[jfmengels]

Sounds good to me.

[IssueHuntBot]

@issuehunt has funded $60.00 to this issue.

---

• Submit pull request via IssueHunt to receive this reward.
• Want to contribute? Chip in to this issue via IssueHunt.
• Checkout the IssueHunt Issue Explorer to see more funded issues.
• Need help from developers? Add your repository on IssueHunt to raise funds.

[JLHwung]

I would like to work on it in couple of days.

[sindresorhus]

Go ahead :)

Make sure you carefully read https://github.com/avajs/eslint-plugin-ava/blob/master/docs/new-rule.md first and also look at some previous PRs to see how it's done.

[issuehunt-oss]

@sindresorhus has rewarded $54.00 to @JLHwung. See it on IssueHunt

• 💰 Total deposit: $60.00
• 🎉 Repository reward(0%): $0.00
• 🔧 Service fee(10%): $6.00