Feature request: check page and locator actions for missing-playwright-await rule

[benatshippabo]

Suggestion

We should add additional functions to check if they are being awaited on. Currently we only check expect and test matchers, but none of the page actions or locator actions:

eslint-plugin-playwright/src/rules/missing-playwright-await.ts

Lines 11 to 58 in f3cfc99

	const validTypes = new Set([
	'AwaitExpression',
	'ReturnStatement',
	'ArrowFunctionExpression',
	]);
	const expectPlaywrightMatchers = [
	'toBeChecked',
	'toBeDisabled',
	'toBeEnabled',
	'toEqualText', // deprecated
	'toEqualUrl',
	'toEqualValue',
	'toHaveFocus',
	'toHaveSelector',
	'toHaveSelectorCount',
	'toHaveText', // deprecated
	'toMatchAttribute',
	'toMatchComputedStyle',
	'toMatchText',
	'toMatchTitle',
	'toMatchURL',
	'toMatchValue',
	];
	const playwrightTestMatchers = [
	'toBeChecked',
	'toBeDisabled',
	'toBeEditable',
	'toBeEmpty',
	'toBeEnabled',
	'toBeFocused',
	'toBeHidden',
	'toBeVisible',
	'toContainText',
	'toHaveAttribute',
	'toHaveClass',
	'toHaveCount',
	'toHaveCSS',
	'toHaveId',
	'toHaveJSProperty',
	'toBeOK',
	'toHaveScreenshot',
	'toHaveText',
	'toHaveTitle',
	'toHaveURL',
	'toHaveValue',
	];

[revelt]

just to add 2p, for example, currently, this missing page await is not flagged up:

test('has title', async ({ page }) => {
  page.goto('https://playwright.dev/'); // await is missing!
// ^^^

  // Expect a title "to contain" a substring.
  await expect(page).toHaveTitle(/Playwright/);
});