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/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');

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

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 }) => {

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 */