New rule suggestion: Place all hooks at the top of the block.

[garyking]

I would like to suggest a new rule, to place all hook blocks before any other blocks.

So, this is good:

describe('module', () => {
  let a
  
  beforeEach(() => {
    a = new A();
  })
  
  test('function', () => {
    expect(a).not.toBeUndefined();
  })
})

And this is bad:

describe('module', () => {
  let a
  
  test('function', () => {
    expect(a).not.toBeUndefined();
  })
  
  beforeEach(() => {
    a = new A();
  })
})

[G-Rath]

Sounds like it could be useful - I'd think a good target goal would be something like "all hooks must be above the first test case in a block".

PRs welcome 😄

[SimenB]

🎉 This issue has been resolved in version 22.18.0 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀