Updated the code after Seth's feedback on the PR

Author: neelkanth-kaushik

packages/consent/consent-tools-integration-tests/package.json

@@ -14,7 +14,9 @@
     "playwright-test": "playwright test",
     "playwright-test:headed": "playwright test --headed",
     "playwright-test:debug": "playwright test --debug",
-    "serve": "http-server public -p 4567"
+    "serve": "http-server public -p 4567",
+    "server": "http-server --port 5432",
+    "browser": "playwright test --debug"
   },
   "installConfig": {
     "hoistingLimits": "workspaces"

packages/consent/consent-tools-integration-tests/playwright.config.ts

@@ -1,24 +1,56 @@
 import type { PlaywrightTestConfig } from '@playwright/test'
+import { devices } from '@playwright/test'
+import path from 'path'
 
-const PORT = 4567
+/**
+ * See https://playwright.dev/docs/test-configuration.
+ */
 const config: PlaywrightTestConfig = {
-  testDir: './src/tests',
-  timeout: 30000,
-  retries: 1,
   webServer: {
-    command: 'yarn http-server public -p 4567',
-    port: 4567,
+    command: 'yarn run server',
+    url: 'http://127.0.0.1:5432',
     reuseExistingServer: !process.env.CI,
   },
+  testDir: './src/tests',
+  globalSetup: path.resolve(__dirname, 'playwright.global-setup.ts'),
+  /* Maximum time one test can run for. */
+  timeout: 30 * 1000,
+  expect: {
+    /**
+     * Maximum time expect() should wait for the condition to be met.
+     * For example in `await expect(locator).toHaveText();`
+     */
+    timeout: 5000,
+  },
+  /* Run tests in files in parallel */
+  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 ? 2 : 0,
+  /* Opt out of parallel tests on CI. */
+  workers: process.env.CI ? 1 : undefined,
+  /* Reporter to use. See https://playwright.dev/docs/test-reporters */
+  reporter: [['html', { open: 'never' }]],
+  /* Shared settings for all the projects below. See https://playwright.dev/docs/api/class-testoptions. */
   use: {
-    headless: true,
-    viewport: { width: 1280, height: 720 },
-    baseURL: `http://localhost:${PORT}`,
-    video: {
-      mode: 'off',
-      size: { width: 640, height: 480 },
+    /* Collect trace when retrying the failed test. See https://playwright.dev/docs/trace-viewer */
+    baseURL: `http://127.0.0.1:5432`,
+    trace: 'on',
+    launchOptions: {
+      args: ['--enable-precise-memory-info', '--js-flags=--expose-gc'],
     },
   },
+
+  /* Configure projects for major browsers */
+  projects: [
+    {
+      name: 'chromium',
+      use: {
+        ...devices['Desktop Chrome'],
+      },
+    },
+  ],
 }
 
 export default config

packages/consent/consent-tools-integration-tests/playwright.global-setup.ts

@@ -0,0 +1,8 @@
+import type { FullConfig } from '@playwright/test'
+import { execSync } from 'child_process'
+
+export default function globalSetup(_cfg: FullConfig) {
+  console.log('Executing global setup...')
+  execSync('yarn build', { stdio: 'inherit' })
+  console.log('Finished global setup.')
+}

packages/consent/consent-tools-integration-tests/src/page-objects/base-page.ts

@@ -104,7 +104,6 @@ export abstract class BasePage {
     await this.page.route('**/settings', async (route: Route) => {
       await route.fulfill({
         status: 200,
-        contentType: 'application/json',
         body: JSON.stringify(settings),
       })
     })