test b4 worked for failed register, testing all at once now

Author: Aushveen

.github/workflows/e2e-tests.yml

@@ -57,4 +57,4 @@ jobs:
       - name: Run E2E tests
         env:
           NODE_OPTIONS: '--dns-result-order=ipv4first'
-        run: xvfb-run --auto-servernum --server-args="-screen 0 1920x1080x24" yarn test:e2e registration-failed.spec.ts
+        run: xvfb-run --auto-servernum --server-args="-screen 0 1920x1080x24" yarn test:e2e

apps/web/e2e/appSession.ts

@@ -95,6 +95,7 @@ export async function switchToBaseNetworkIfNeeded(page: Page): Promise<boolean>
     page.url(),
   );
   await page.waitForLoadState('networkidle');
+  await page.waitForTimeout(5000);
   // Prefer the explicit "Connect to Base" button if present
   const explicitSelector = 'button:has-text("Connect to Base")';
 

apps/web/e2e/basenameHelpers.ts

@@ -47,11 +47,13 @@ export async function navigateToBasenameRegistration(page: Page): Promise<void>
  * @param basename - The basename to search for
  */
 export async function searchForBasename(page: Page, basename: string): Promise<void> {
+  await page.waitForTimeout(2000);
   const searchInput = page.locator(SELECTORS.SEARCH_INPUT).first();
   await expect(searchInput).toBeVisible();
   await searchInput.clear();
   await searchInput.fill(basename);
   await page.waitForLoadState('networkidle');
+  await page.waitForTimeout(2000);
 }
 
 /**

apps/web/e2e/tests/connectWallet.spec.ts

@@ -1,6 +1,8 @@
 import { test } from '../testFixture';
 import { connectWallet } from '../appSession';
 
+test.describe.configure({ mode: 'parallel' });
+
 test.describe('Connect Wallet', () => {
   test('should connect to wallet', async ({ page, metamask }) => {
     if (!metamask) {

apps/web/e2e/tests/registration-failed.spec.ts

@@ -4,14 +4,14 @@ import { prepareBasenameFlow, handleTransaction } from '../appSession';
 import { initiateRegistration } from '../basenameHelpers';
 import { ActionApprovalType } from '@coinbase/onchaintestkit';
 
+test.describe.configure({ mode: 'parallel' });
+
 test.describe('Basename Registration', () => {
   test('should fail registration when wallet has insufficient funds', async ({
     page,
     metamask,
     node,
   }) => {
-    test.setTimeout(120000); // Increase timeout to 2 minutes
-
     // Validate prerequisites
     if (!metamask) {
       throw new Error('MetaMask is not defined');
@@ -26,7 +26,7 @@ test.describe('Basename Registration', () => {
 
     // Add a small delay to ensure everything is stable
     console.log('[test] Waiting for network to stabilize...');
-    await mainPage.waitForTimeout(4000);
+    await mainPage.waitForTimeout(2000);
 
     // Get the first default wallet address from Anvil (0xf39Fd6e51aad88F6F4ce6aB8827279cffFb92266)
     let walletAddress;

apps/web/e2e/tests/registration-rejected.spec.ts

@@ -4,6 +4,8 @@ import { prepareBasenameFlow, handleTransaction } from '../appSession';
 import { initiateRegistration } from '../basenameHelpers';
 import { ActionApprovalType } from '@coinbase/onchaintestkit';
 
+test.describe.configure({ mode: 'parallel' });
+
 test.describe('Basename Registration', () => {
   test('should fail registration when transaction is rejected', async ({ page, metamask }) => {
     // Validate prerequisites
@@ -14,7 +16,7 @@ test.describe('Basename Registration', () => {
     // Common preparation steps
     const { mainPage } = await prepareBasenameFlow(page, metamask);
 
-    await mainPage.waitForTimeout(4000);
+    await mainPage.waitForTimeout(2000);
 
     // Attempt registration and explicitly reject the transaction in MetaMask
     await initiateRegistration(mainPage);

apps/web/e2e/tests/registration-success.spec.ts

@@ -3,6 +3,8 @@ import { test } from '../testFixture';
 import { handleTransaction, prepareBasenameFlow } from '../appSession';
 import { initiateRegistration, SELECTORS } from '../basenameHelpers';
 
+test.describe.configure({ mode: 'parallel' });
+
 // Main test
 test.describe('Basename Registration', () => {
   test('should successfully register a basename', async ({ page, metamask }) => {
@@ -14,7 +16,7 @@ test.describe('Basename Registration', () => {
     // Common preparation steps (connect wallet, switch network, navigate & select basename)
     const { mainPage } = await prepareBasenameFlow(page, metamask);
 
-    await mainPage.waitForTimeout(4000);
+    await mainPage.waitForTimeout(2000);
 
     // Complete registration
     await initiateRegistration(mainPage);

apps/web/e2e/walletConfig/coinbaseWalletConfig.ts

@@ -23,7 +23,7 @@ const baseConfig = configure()
     chainId: base.id,
     symbol: 'ETH',
     // placeholder for the actual rpcUrl, which is auto injected by the node fixture
-    rpcUrl: 'http://127.0.0.1:8545',
+    rpcUrl: 'http://localhost:8545',
   });
 
 // Build the config

apps/web/playwright.config.ts

@@ -19,16 +19,16 @@ const baseURL = `http://localhost:${PORT}`;
  * See https://playwright.dev/docs/test-configuration.
  */
 export default defineConfig({
-  timeout: 5 * 60 * 1000,
+  timeout: 2 * 60 * 1000,
   testDir: './e2e/tests',
   /* Run tests in files in parallel */
-  fullyParallel: false,
+  fullyParallel: true,
   /* Fail the build on CI if you accidentally left test.only in the source code. */
   forbidOnly: !!process.env.CI,
   /* Retry on CI only */
-  retries: process.env.CI ? 5 : 0,
+  retries: 6,
   /* Opt out of parallel tests on CI. */
-  workers: 10,
+  workers: process.env.CI ? 4 : 8,
   maxFailures: 1,
   /* Reporter to use. See https://playwright.dev/docs/test-reporters */
   reporter: 'html',
@@ -51,7 +51,7 @@ export default defineConfig({
 
   /* Increase default expect timeout */
   expect: {
-    timeout: 120000,
+    timeout: 30000,
   },
 
   /* Configure projects for major browsers */