test: improve DX (#1129)

Author: Belco90

.github/workflows/ci.yml

@@ -12,8 +12,6 @@ jobs:
   verifications:
     name: Verifications
     uses: ./.github/workflows/verifications.yml
-    secrets:
-      CODECOV_TOKEN: ${{ secrets.CODECOV_TOKEN }}
 
   required-checks:
     name: Require CI status checks

.github/workflows/main-coverage.yml

@@ -22,9 +22,14 @@ permissions:
   pull-requests: write # to be able to comment on released pull requests
 
 jobs:
+  verifications:
+    name: Verifications
+    uses: ./.github/workflows/verifications.yml
+
   publish:
     name: Publish package
     runs-on: ubuntu-latest
+    needs: [verifications]
     # Avoid publishing in forks
     if: github.repository == 'testing-library/eslint-plugin-testing-library'
     steps:

.github/workflows/release.yml

@@ -2,9 +2,9 @@ name: Verifications
 
 on:
   workflow_call:
-    secrets:
-      CODECOV_TOKEN:
-        required: true
+
+permissions:
+  id-token: write # OIDC auth for Codecov
 
 jobs:
   code-validation:
@@ -65,7 +65,16 @@ jobs:
       - name: Run tests
         run: pnpm run test:ci
 
-      - name: Upload coverage reports to Codecov
+      - name: Upload tests coverage report
         uses: codecov/codecov-action@v5
-        env:
-          CODECOV_TOKEN: ${{ secrets.CODECOV_TOKEN }}
+        if: ${{ !cancelled() && github.event.action != 'closed' }}
+        with:
+          use_oidc: true
+          report_type: 'coverage'
+
+      - name: Upload tests result report
+        uses: codecov/codecov-action@v5
+        if: ${{ !cancelled() && github.event.action != 'closed' }}
+        with:
+          use_oidc: true
+          report_type: 'test_results'

.github/workflows/verifications.yml

@@ -16,6 +16,7 @@ pids
 
 # Directory for instrumented libs generated by jscoverage/JSCover
 lib-cov
+test-report.junit.xml
 
 # Coverage directory used by tools like istanbul
 coverage
@@ -64,6 +65,7 @@ typings/
 yarn-error.log
 .pnp/
 .pnp.js
+
 # Yarn Integrity file
 .yarn-integrity
 

.gitignore

@@ -49,6 +49,7 @@
 		"semantic-release": "semantic-release",
 		"test": "vitest run",
 		"test:ci": "vitest run --coverage",
+		"test:ui": "vitest --ui",
 		"test:watch": "vitest",
 		"type-check": "tsc --noEmit"
 	},
@@ -66,6 +67,7 @@
 		"@typescript-eslint/rule-tester": "^8.15.0",
 		"@vitest/coverage-v8": "^3.2.4",
 		"@vitest/eslint-plugin": "^1.5.2",
+		"@vitest/ui": "3.2.4",
 		"del-cli": "^6.0.0",
 		"eslint": "^9.35.0",
 		"eslint-config-prettier": "^10.1.8",

package.json

@@ -1,9 +1,10 @@
-import { defineConfig } from 'vitest/config';
+import { defineConfig, configDefaults } from 'vitest/config';
 
 export default defineConfig({
 	test: {
 		include: ['**/tests/**/*.test.ts'],
 		setupFiles: ['./vitest.setup.mts'],
+		clearMocks: true,
 		coverage: {
 			include: ['lib/**'],
 			thresholds: {
@@ -13,5 +14,10 @@ export default defineConfig({
 				statements: 90,
 			},
 		},
+		// Enable JUnit reporter in CI environment
+		reporters: process.env.CI ? ['default', 'junit'] : configDefaults.reporters,
+		outputFile: {
+			junit: 'test-report.junit.xml',
+		},
 	},
 });