await expect - new false positive in 1.8.0

[dylang]

Hi, it looks like the work done in e283efc is not correctly supporting async matchers.

For example, this is now considered incorrect usage of await:

await expect(pageObject.tabTo('.first', '.third', { maxCount: 1 })).rejects.toThrow(
  'Could not reach the selector .third using the keyboard'
);

[mdmower]

I'm similarly seeing what I believe are false positives for expect.poll():

await expect.poll(() => getSlowStorageValue(page, 'key')).toBe('value');
// warning  Unnecessary await expression. This method does not return a Promise  playwright/no-useless-await

Documentation implies await is needed: https://playwright.dev/docs/test-assertions#expectpoll

[github-actions]

🎉 This issue has been resolved in version 1.8.1 🎉

The release is available on:

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

Your semantic-release bot 📦🚀