Epic 2 pbi 2.1 (improvement)

.eslintrc.json

@@ -1,13 +1,19 @@
 {
-  "env": {
-    "browser": true,
-    "es2021": true,
-    "node": true
-  },
-  "extends": "eslint:recommended",
+  "root": true,
+  "parser": "@typescript-eslint/parser",
+  "plugins": [
+    "@typescript-eslint"
+  ],
+  "extends": [
+    "eslint:recommended",
+    "plugin:@typescript-eslint/recommended"
+  ],
   "parserOptions": {
     "ecmaVersion": "latest",
-    "sourceType": "module"
+    "sourceType": "module",
+    "project": "./tsconfig.json" // Make sure this path is correct if you have a tsconfig.json
   },
-  "rules": {}
+  "rules": {
+    // Your custom rules here
+  }
 }

package.json

@@ -20,6 +20,8 @@
   "homepage": "https://github.com/DigitalProductInnovationAndDevelopment/Code-Reviews-of-GUI-Tests#readme",
   "devDependencies": {
     "@playwright/test": "^1.52.0",
-    "eslint": "^8.0.0"
+    "eslint": "^8.0.0",
+    "@typescript-eslint/eslint-plugin": "^7.0.0",
+    "@typescript-eslint/parser": "^7.0.0"
   }
 }

playwright.config.js

@@ -1,16 +1,37 @@
-const { defineConfig } = require('@playwright/test');
+// playwright.config.js
+import { defineConfig, devices } from '@playwright/test';
 
-module.exports = defineConfig({
-  testDir: './tests',
+export default defineConfig({
+  testDir: "./tests",
+  fullyParallel: true,
+  forbidOnly: !!process.env.CI,
+  retries: process.env.CI ? 2 : 0,
+  workers: process.env.CI ? 1 : undefined,
+  reporter: [
+    // Corrected to output HTML report to a subfolder within published-screenshots
+    // This should fix the 'published-screenshots/html/' not found warning.
+    ["html", { open: "never", outputFolder: "published-screenshots/html" }],
+    // Ensure JSON report is also in published-screenshots
+    ["json", { outputFile: "published-screenshots/results.json" }]
+  ],
   use: {
-    headless: true,
-    screenshot: 'on',
-    trace: 'on',
-    video: 'off',
-    ignoreHTTPSErrors: true,
+    trace: "on-first-retry",
+    headless: true, // Playwright runs headless by default in CI
+    screenshot: "on", // Takes screenshots on failure or specific steps
   },
-  reporter: [
-    ['json', { outputFile: 'playwright-metrics.json' }],
-    ['html', { outputFile: 'report.html', open: 'never' }],
+  projects: [
+    {
+      name: "chromium",
+      use: { ...devices["Desktop Chrome"] },
+    },
+    // You can add other projects for different browsers here if needed, e.
+    // {
+    //  name: "firefox",
+    //  use: { ...devices["Desktop Firefox"] },
+    // },
+    // {
+    //  name: "webkit",
+    //  use: { ...devices["Desktop Safari"] },
+    // },
   ],
-});
+});

tsconfig.json

@@ -0,0 +1,18 @@
+{
+  "compilerOptions": {
+    "target": "ES2020",
+    "module": "CommonJS",
+    "strict": true,
+    "esModuleInterop": true,
+    "skipLibCheck": true,
+    "forceConsistentCasingInFileNames": true,
+    // If your tests use JSX (e.g., for React components), uncomment the line below:
+    // "jsx": "react-jsx",
+    "resolveJsonModule": true,
+    "isolatedModules": true,
+    "allowJs": true,
+    "checkJs": true
+  },
+  "include": ["tests/**/*.ts", "tests/**/*.js"], // Changed this line
+  "exclude": ["node_modules"] // Changed this line
+}