Improvement: Report RegExp with global flag saved in a variable

[Belco90]

What rule do you want to change?

no-global-regexp-flag-in-query

Does this change cause the rule to produce more or fewer warnings?

More warnings

How will the change be implemented?

Right now, we are only checking the RegExp passed directly to Testing Library queries. If a variable is found, we should check what's the value held in it to report global flags there also.

Example code

it('should render a basic demo', () => {
  const countRegExp = /count/gi
  render(<App />)
  expect(screen.getByText(countRegExp)).toBeInTheDocument()
})

How does the current rule affect the code?

screen.getByText(countRegExp) should be reported.

How will the new rule affect the code?

N/A

Anything else?

No response

Do you want to submit a pull request to change the rule?

No

[github-actions]

🎉 This issue has been resolved in version 5.8.0 🎉

The release is available on:

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

Your semantic-release bot 📦🚀

[github-actions]

🎉 This issue has been resolved in version 6.0.0-alpha.14 🎉

The release is available on:

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

Your semantic-release bot 📦🚀