🚨 Fix review findings: JUnit reporter, remove migration script, playwright --with-deps

Author: mormubis

.gitlab-ci.yml

@@ -219,7 +219,7 @@ unit:
       junit: test-report/unit/*.xml
   script:
     - yarn
-    - yarn playwright install chromium
+    - yarn playwright install chromium --with-deps
     - yarn test:unit
   after_script:
     - node ./scripts/test/export-test-result.ts unit

scripts/migrate-to-vitest.ts

@@ -119,5 +119,7 @@ export default defineConfig({
     sequence: {
       shuffle: true,
     },
+
+    reporters: process.env.CI ? ['default', ['junit', { outputFile: 'test-report/unit-bs/results.xml' }]] : ['default'],
   },
 })

vitest.bs.config.ts

@@ -74,5 +74,7 @@ export default defineConfig({
 
     // Note: Karma used stopSpecOnExpectationFailure, but Vitest's bail stops the entire run.
     // Use --bail on CLI when needed for fast feedback during development.
+
+    reporters: process.env.CI ? ['default', ['junit', { outputFile: 'test-report/unit/results.xml' }]] : ['default'],
   },
 })