refactor: clean up test imports and formatting for consistency

- Updated import statements in multiple E2E test files to ensure consistent usage of the Playwright test library.
- Improved code readability by formatting the assertion in the notification web push test for better clarity.
- Enhanced maintainability of test files by aligning import structures across different test cases.

Author: mitchdowney

apps/api/src/test/notification-webpush.test.ts

@@ -395,7 +395,9 @@ describe('notification web push (API integration)', () => {
     }
     expect(firstArg).toHaveLength(2);
     expect(firstArg).toEqual(expect.arrayContaining([ownerId, otherUser.id]));
-    expect(vi.mocked(webPush.sendNotification)).toHaveBeenCalledTimes(ownerSubCount + otherSubCount);
+    expect(vi.mocked(webPush.sendNotification)).toHaveBeenCalledTimes(
+      ownerSubCount + otherSubCount
+    );
     await deleteAllWebPushSubscriptionsForUser(ownerId);
     await deleteAllWebPushSubscriptionsForUser(otherUser.id);
     listSpy.mockRestore();

apps/management-web/e2e/bucket-detail-url-state-contract.spec.ts

@@ -1,6 +1,7 @@
-import { expect, test } from '@playwright/test';
 import type { Page } from '@playwright/test';
 
+import { expect, test } from '@playwright/test';
+
 import { loginAsManagementSuperAdmin } from './helpers/advancedFixtures';
 import { actionAndCapture, capturePageLoad } from './helpers/stepScreenshots';
 import { setE2EUserContext } from './helpers/userContext';

apps/management-web/e2e/user-detail-limited-admin-users-read.spec.ts

@@ -2,8 +2,8 @@ import { expect, test } from '@playwright/test';
 
 import { loginAsLimitedAdmin } from './helpers/advancedFixtures';
 import { expectInvalidRouteShowsNotFound } from './helpers/flowHelpers';
-import { actionAndCapture, capturePageLoad } from './helpers/stepScreenshots';
 import { E2E_NONEXISTENT_ENTITY_ID } from './helpers/seedConstants';
+import { actionAndCapture, capturePageLoad } from './helpers/stepScreenshots';
 import { setE2EUserContext } from './helpers/userContext';
 
 const E2E_MAIN_USER_ID = '11111111-1111-4111-a111-111111111111';

apps/management-web/e2e/user-detail-super-admin-full-crud.spec.ts

@@ -2,8 +2,8 @@ import { expect, test } from '@playwright/test';
 
 import { loginAsManagementSuperAdmin } from './helpers/advancedFixtures';
 import { expectInvalidRouteShowsNotFound } from './helpers/flowHelpers';
-import { actionAndCapture, capturePageLoad } from './helpers/stepScreenshots';
 import { E2E_NONEXISTENT_ENTITY_ID } from './helpers/seedConstants';
+import { actionAndCapture, capturePageLoad } from './helpers/stepScreenshots';
 import { setE2EUserContext } from './helpers/userContext';
 
 const E2E_MAIN_USER_ID = '11111111-1111-4111-a111-111111111111';

apps/web/e2e/bucket-detail-url-state-contract.spec.ts

@@ -1,6 +1,7 @@
-import { expect, test } from '@playwright/test';
 import type { Page } from '@playwright/test';
 
+import { expect, test } from '@playwright/test';
+
 import { loginAsWebE2EUserAndExpectDashboard } from './helpers/advancedFixtures';
 import { actionAndCapture, capturePageLoad } from './helpers/stepScreenshots';
 import { setE2EUserContext } from './helpers/userContext';