Refactor package.json and GitHub Actions workflow for improved testing setup. Change dependency versions to fixed values, update test commands to use web-test-runner configuration, and streamline installation steps in the CI pipeline.

Author: alexpmule

.github/workflows/deployment.yml

@@ -25,16 +25,16 @@ jobs:
         with:
           node-version: 18
           cache: 'npm'
-      - name: Install dependencies
+      - name: Clean install
         run: |
+          rm -rf node_modules package-lock.json
           npm install -g [email protected]
           npm ci
-      - name: Install test dependencies
+      - name: Install Playwright browsers
         run: |
-          npm install -D [email protected] @playwright/[email protected] @web/[email protected] @web/[email protected]
           npx playwright install --with-deps chromium firefox webkit
       - name: Run tests
-        run: npx web-test-runner --config web-test-runner.config.cjs
+        run: npm test
   test_win:
     name: "Windows"
     runs-on: windows-latest
@@ -44,16 +44,16 @@ jobs:
         with:
           node-version: 18
           cache: 'npm'
-      - name: Install dependencies
+      - name: Clean install
         run: |
+          rm -rf node_modules package-lock.json
           npm install -g [email protected]
           npm ci
-      - name: Install test dependencies
+      - name: Install Playwright browsers
         run: |
-          npm install -D [email protected] @playwright/[email protected] @web/[email protected] @web/[email protected]
           npx playwright install --with-deps chromium firefox webkit
       - name: Run tests
-        run: npx web-test-runner --config web-test-runner.config.cjs
+        run: npm test
   tag:
     name: "Publishing release"
     if: github.ref == 'refs/heads/main' || github.ref == 'refs/heads/master'

package.json

@@ -43,16 +43,16 @@
     "@commitlint/config-conventional": "^18.4.3",
     "@open-wc/eslint-config": "^12.0.0",
     "@open-wc/testing": "^3.2.0",
-    "@playwright/test": "^1.40.1",
+    "@playwright/test": "1.40.1",
     "@types/dompurify": "^3.0.5",
     "@web/dev-server": "^0.4.1",
-    "@web/test-runner": "^0.18.0",
-    "@web/test-runner-playwright": "^0.11.0",
+    "@web/test-runner": "0.18.0",
+    "@web/test-runner-playwright": "0.11.0",
     "eslint": "^8.55.0",
     "eslint-config-prettier": "^9.1.0",
     "husky": "^7.0.2",
     "lint-staged": "^11.1.2",
-    "playwright": "^1.40.1",
+    "playwright": "1.40.1",
     "rimraf": "^5.0.5",
     "sinon": "^11.1.2",
     "typescript": "^4.4.3",
@@ -65,8 +65,8 @@
     "lint:types": "tsc",
     "lint": "npm run lint:eslint",
     "format": "npm run format:eslint",
-    "test": "web-test-runner test/**/*.test.js --coverage --node-resolve --playwright --browsers chromium firefox webkit",
-    "test:watch": "web-test-runner test/**/*.test.js --node-resolve --watch --playwright --browsers chromium",
+    "test": "web-test-runner --config web-test-runner.config.cjs",
+    "test:watch": "web-test-runner --config web-test-runner.config.cjs --watch",
     "gen:wc": "wca analyze \"*.js\" --outFile custom-elements.json",
     "prepare": "node demo/model.js"
   },