test: avoid redundant html validation (#1083)

Co-authored-by: Daniel Roe <daniel@roe.dev>

Author: essenmitsosse

.github/workflows/ci.yml

@@ -150,7 +150,9 @@ jobs:
         run: pnpm install
 
       - name: 🏗️ Build project
-        run: pnpm build:playwright
+        run: pnpm build:test
+        env:
+          VALIDATE_HTML: true
 
       - name: 🖥️ Test project (browser)
         run: pnpm test:browser:prebuilt
@@ -178,7 +180,7 @@ jobs:
         run: pnpm install
 
       - name: 🏗️ Build project
-        run: NODE_ENV=test pnpm build
+        run: pnpm build:test
 
       - name: ♿ Accessibility audit (Lighthouse - ${{ matrix.mode }} mode)
         run: ./scripts/lighthouse-a11y.sh

nuxt.config.ts

@@ -1,5 +1,6 @@
 import process from 'node:process'
 import { currentLocales } from './config/i18n'
+import { isCI, provider } from 'std-env'
 
 export default defineNuxtConfig({
   modules: [
@@ -195,6 +196,7 @@ export default defineNuxtConfig({
   },
 
   htmlValidator: {
+    enabled: !isCI || (provider !== 'vercel' && !!process.env.VALIDATE_HTML),
     failOnError: true,
   },
 

package.json

@@ -12,7 +12,7 @@
   "scripts": {
     "build": "nuxt build",
     "build:lunaria": "node ./lunaria/lunaria.ts",
-    "build:playwright": "NODE_ENV=test pnpm build",
+    "build:test": "NODE_ENV=test pnpm build",
     "dev": "nuxt dev",
     "dev:docs": "pnpm run --filter npmx-docs dev --port=3001",
     "i18n:check": "node scripts/compare-translations.ts",
@@ -30,10 +30,10 @@
     "generate:fixtures": "node scripts/generate-fixtures.ts",
     "generate:lexicons": "lex build --lexicons lexicons --out shared/types/lexicons --clear",
     "test": "vite test",
-    "test:browser": "pnpm build:playwright && pnpm test:browser:prebuilt",
+    "test:browser": "pnpm build:test && pnpm test:browser:prebuilt",
     "test:browser:prebuilt": "playwright test",
-    "test:browser:ui": "pnpm build:playwright && pnpm test:browser:prebuilt --ui",
-    "test:browser:update": "pnpm build:playwright && pnpm test:browser:prebuilt --update-snapshots",
+    "test:browser:ui": "pnpm build:test && pnpm test:browser:prebuilt --ui",
+    "test:browser:update": "pnpm build:test && pnpm test:browser:prebuilt --update-snapshots",
     "test:nuxt": "vite test --project nuxt",
     "test:types": "pnpm generate:lexicons && nuxt prepare && vue-tsc -b --noEmit && pnpm --filter npmx-connector test:types",
     "test:unit": "vite test --project unit",