test: refactor tests

tests/e2e/lib/genericadminpage.ts

@@ -1,4 +1,4 @@
-import { Locator, Page } from '@playwright/test';
+import type { Locator, Page } from '@playwright/test';
 
 const selectors = {
 	menuBarMyAccount: '#wp-admin-bar-my-account',

tests/e2e/lib/loginpage.ts

@@ -1,4 +1,4 @@
-import { Locator, Page } from '@playwright/test';
+import type { Locator, Page } from '@playwright/test';
 
 const selectors = {
 	userField: '#user_login',
@@ -37,16 +37,8 @@ export class LoginPage {
 	}
 
 	public async login(username: string, password: string): Promise<void> {
-		await this.userFieldLocator.click();
-		// eslint-disable-next-line playwright/no-wait-for-timeout
-		await this.page.waitForTimeout(60);
 		await this.userFieldLocator.fill(username);
-		await this.passwordFieldLocator.click();
-		// eslint-disable-next-line playwright/no-wait-for-timeout
-		await this.page.waitForTimeout(60);
 		await this.passwordFieldLocator.fill(password);
-		// eslint-disable-next-line playwright/no-wait-for-timeout
-		await this.page.waitForTimeout(60);
 		await this.submitButtonLocator.click();
 		await this.page.waitForLoadState('domcontentloaded');
 	}

tests/e2e/lib/profilepage.ts

@@ -1,4 +1,4 @@
-import { Locator, Page, Response } from '@playwright/test';
+import type { Locator, Page, Response } from '@playwright/test';
 
 const selectors = {
 	twoFactorOptions: '#two-factor-options',

tests/e2e/lib/test-helpers.ts

@@ -1,4 +1,4 @@
-import { Page, CDPSession, expect } from '@playwright/test';
+import { type Page, type CDPSession, expect } from '@playwright/test';
 import { LoginPage } from './loginpage';
 import type { Protocol } from 'playwright-core/types/protocol';
 import { ProfilePage } from './profilepage';

tests/e2e/lib/webauthsettingspage.ts

@@ -1,4 +1,4 @@
-import { Locator, Page } from '@playwright/test';
+import type { Locator, Page } from '@playwright/test';
 
 const selectors = {
 	authenticatorAttachment: '#authenticator_attachment',

tests/e2e/specs/login.spec.ts

@@ -1,4 +1,4 @@
-import { test, expect, CDPSession } from '@playwright/test';
+import { test, expect, type CDPSession } from '@playwright/test';
 import { LoginPage } from '../lib/loginpage';
 import { ProfilePage } from '../lib/profilepage';
 import { GenericAdminPage } from '../lib/genericadminpage';

tests/e2e/specs/rename.spec.ts

@@ -1,4 +1,4 @@
-import { test, expect, CDPSession } from '@playwright/test';
+import { test, expect, type CDPSession } from '@playwright/test';
 import settings from '../e2e-settings';
 import { addVirtualAuthenticator, clearCredentials, removeVirtualAuthenticator } from '../lib/webauthn-helpers';
 import { login, registerKey } from '../lib/test-helpers';

tests/e2e/specs/revoke.spec.ts

@@ -1,4 +1,4 @@
-import { test, expect, CDPSession } from '@playwright/test';
+import { test, expect, type CDPSession } from '@playwright/test';
 import settings from '../e2e-settings';
 import { addVirtualAuthenticator, removeVirtualAuthenticator } from '../lib/webauthn-helpers';
 import { login, registerKey } from '../lib/test-helpers';