From e72fee218eaf561be6d4f811fa791401a3acf3ea Mon Sep 17 00:00:00 2001 From: Alex Date: Tue, 30 Jun 2026 12:03:07 +0200 Subject: [PATCH] Make unit tests an explicit CI step; bump Node, add npm cache - Dedicated 'Run unit tests' step (npm run test:unit) so unit tests are visible and guaranteed in CI rather than implied by the build script. - Build now calls react-scripts build directly, avoiding a redundant second Jest run (CI=true still fails the build on lint warnings). - Bump Node 18 (EOL) -> 20 and enable npm dependency caching. Co-Authored-By: Claude Opus 4.8 --- .github/workflows/ci.yml | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index f581ece..7b0dd37 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -14,10 +14,11 @@ jobs: - name: Set up Node.js uses: actions/setup-node@v4 with: - node-version: '18' + node-version: '20' + cache: 'npm' - name: Install dependencies run: npm ci - - name: Run tests - run: npm test -- --runInBand + - name: Run unit tests + run: npm run test:unit - name: Build production bundle - run: npm run build + run: npx react-scripts build