eslint-plugin-playwright
eslint-plugin-playwright copied to clipboard
playwright-community
ESLint plugin for Playwright
I've been going through our fairly large suite of Playwright tests and I noticed a common pattern that I think would be great to correct. If I have some time,...
Right now we have a separate rule that checks for `count()` and recommends using `toHaveCount()` instead. Not sure if there is a reason for this differentiation or special treatment for...
Several of the "sub-rules" in `valid-title` have options to individually disable them. It would be great if you could add an option to disable `duplicatePrefix`. Thank you!
It's a feature request to add one more case to `prefer-to-have-count` rule. The following code should be considered invalid: ```js expect(await page.getByText("test").all()).toHaveLength(2); ``` The following code should be considered valid:...
If using nested `describe` blocks to organize multiple sets of tests, the `expect` calls within the `it` tests in the `describe` blocks after the first, are causing `no-standalone-expect` to flag...
Right now there are two `await` related rules One to make sure that `expect` awaits if locators are what is being asserted on, and one to make sure there are...
- Part of https://github.com/playwright-community/eslint-plugin-playwright/issues/159 This adds support for checking if `Page` methods are properly awaited, as part of the `missing-playwright-await` rule. I've intentionally skipped the `waitForEvent`, `waitForRequest`, and `waitForResponse` methods,...
Would make tests for page stuff easier (e.g. https://github.com/playwright-community/eslint-plugin-playwright/pull/310)
Pretty much an extension of [this issue](https://github.com/playwright-community/eslint-plugin-playwright/issues/154). We use a sort of scrappy version of the Screenplay pattern, basically just an abstract base class for common testing logic and child...
