Show an errror when using arrow functions with hooks

Hi,

Today I’ve tried to use this.skip() inside before() it() but it didn’t work because I was using arrow functions instead of normal functions and so this was undefined.

It took me some time before being able to make this.skip() works, and I think creating a rule can save time from other people too.

This rule would be recommended and will throw an error when using arrow functions with :

• describe, context
• it, it.skip, it.only, … specify
• before/beforeEach
• after/afterEach

What do you think? I can open a PR for this rule (I’ve never write an ESLint rule but it does not seems really complicated)

Thanks!

[chrisbreiding]

Instead of erroring always when you use an arrow function with describe, it, etc, I would rather it error only if you use an arrow function and then use this inside the function body.

Bonus points if it could auto-fix by converting the arrow function into a regular function in that case.

[brettz9]

Btw, FWIW, some other nice Mocha plugins:

1. https://github.com/lo1tuma/eslint-plugin-mocha/
2. https://github.com/onechiporenko/eslint-plugin-mocha-cleanup

And chai ones:

1. https://github.com/gavinaiken/eslint-plugin-chai-expect-keywords/
2. https://github.com/fintechstudios/eslint-plugin-chai-as-promised/
3. https://github.com/ihordiachenko/eslint-plugin-chai-friendly/ (mentioned in the README)
4. https://github.com/Turbo87/eslint-plugin-chai-expect