Correct Playwright Output Directory #7260

Author: tobiu

.github/ISSUE/ticket-correct-playwright-output-dir.md

@@ -0,0 +1,17 @@
+# Ticket: Correct Playwright Output Directory
+
+GH ticket id: #7260
+
+**Assignee:** Gemini
+**Status:** Done
+
+## Description
+
+The Playwright test runner was incorrectly creating a `test-results` directory at the project root, instead of within the `test/playwright` directory as intended.
+
+## Resolution
+
+1.  The `playwright.config.mjs` file was modified to use absolute paths for `testDir` and `outputDir`. This was achieved by using Node.js's `path` and `url` modules to ensure the paths are always resolved correctly, regardless of the directory from which the test command is run.
+2.  The `test` script in `package.json` was updated to explicitly specify the path to the configuration file using the `-c` flag (`playwright test -c test/playwright/playwright.config.mjs`).
+
+This combination ensures that all test artifacts are consistently and correctly placed in `test/playwright/test-results`.

test/playwright/playwright.config.mjs

@@ -1,11 +1,16 @@
 import { defineConfig } from '@playwright/test';
+import path from 'path';
+import { fileURLToPath } from 'url';
+
+const __filename = fileURLToPath(import.meta.url);
+const __dirname = path.dirname(__filename);
 
 /**
  * @see https://playwright.dev/docs/test-configuration
  */
 export default defineConfig({
-  testDir: './unit',
-  outputDir: 'test-results',
+  testDir: path.join(__dirname, 'unit'),
+  outputDir: path.join(__dirname, 'test-results'),
   /* Run tests in files in parallel */
   fullyParallel: true,
   /* Fail the build on CI if you accidentally left test.only in the source code. */
@@ -15,7 +20,7 @@ export default defineConfig({
   /* Opt out of parallel tests on CI. */
   workers: process.env.CI ? 1 : undefined,
   /* Reporter to use. See https://playwright.dev/docs/test-reporters */
-  reporter: [['json', { outputFile: 'test-results/test-results.json' }]],
+  reporter: [['json', { outputFile: path.join(__dirname, 'test-results/test-results.json') }]],
   /* Shared settings for all the projects below. See https://playwright.dev/docs/api/class-testoptions. */
   use: {
     /* Collect trace when retrying the failed test. See https://playwright.dev/docs/trace-viewer */