ci.yml

name: CI

on:
  push:
    branches: [ main, develop ]
  pull_request:
    branches: [ main, develop ]

jobs:
  lint-and-build:
    name: Lint and Build
    runs-on: ubuntu-latest

    steps:
      - name: Checkout code
        uses: actions/checkout@v4

      - name: Setup Node.js
        uses: actions/setup-node@v4
        with:
          node-version: '20'
          cache: 'npm'

      - name: Install dependencies
        run: |
          # Note: Deprecation warnings for inflight@1.0.6 and glob@7.2.3 are expected
          # These come from Jest's transitive dependencies and are harmless.
          # They will be resolved when Jest updates its dependencies.
          if [ -f "package-lock.json" ]; then
            npm ci
          else
            echo "package-lock.json not found, generating it with npm install"
            npm install
          fi

      - name: Type check
        run: npm run build

      - name: Check for linting errors
        run: |
          # Run build to catch TypeScript errors
          npm run build
          # If you add ESLint later, uncomment:
          # npm run lint

  test:
    name: Run Tests
    runs-on: ubuntu-latest
    needs: lint-and-build

    permissions:
      contents: read
      checks: write

    env:
      NODE_ENV: test
      DATABASE_URL: postgresql://postgres:postgres@localhost:5432/test
      REDIS_URL: redis://localhost:6379
      PROPERTY_TEST_RUNS: 100
      JWT_SECRET: test-secret-key-for-ci
      API_KEY_SALT: test-salt-for-ci

    steps:
      - name: Checkout code
        uses: actions/checkout@v4

      - name: Setup PostgreSQL
        run: |
          docker run -d \
            --name test-postgres \
            -e POSTGRES_DB=test \
            -e POSTGRES_USER=postgres \
            -e POSTGRES_PASSWORD=postgres \
            -p 5432:5432 \
            postgres:14-alpine
          
          # Wait for PostgreSQL to be ready
          for i in {1..30}; do
            if docker exec test-postgres pg_isready -U postgres > /dev/null 2>&1; then
              echo "PostgreSQL is ready!"
              break
            fi
            sleep 2
          done

      - name: Setup Redis
        run: |
          docker run -d \
            --name test-redis \
            -p 6379:6379 \
            redis:7-alpine
          
          # Wait for Redis to be ready
          for i in {1..30}; do
            if docker exec test-redis redis-cli ping | grep -q PONG; then
              echo "Redis is ready!"
              break
            fi
            sleep 2
          done

      - name: Setup Node.js
        uses: actions/setup-node@v4
        with:
          node-version: '20'
          cache: 'npm'

      - name: Install dependencies
        run: |
          # Note: Deprecation warnings for inflight@1.0.6 and glob@7.2.3 are expected
          # These come from Jest's transitive dependencies and are harmless.
          # They will be resolved when Jest updates its dependencies.
          if [ -f "package-lock.json" ]; then
            npm ci
          else
            echo "package-lock.json not found, generating it with npm install"
            npm install
          fi

      - name: Cache Jest cache
        uses: actions/cache@v4
        with:
          path: .jest-cache
          key: jest-cache-${{ runner.os }}-${{ hashFiles('package-lock.json') }}
          restore-keys: |
            jest-cache-${{ runner.os }}-

      - name: Build project
        run: npm run build

      - name: Run tests
        run: |
          # Run tests with optimized settings for CI
          # --maxWorkers=100% uses all available CPU cores
          # --bail stops on first failure (already set in config but explicit here)
          # --no-coverage skips coverage collection (faster, only collect when needed)
          npm run test:ci -- --maxWorkers=100% --no-coverage

      - name: Upload test results
        if: always()
        uses: actions/upload-artifact@v4
        with:
          name: test-results
          path: |
            junit.xml
          retention-days: 30

      - name: Publish test results
        if: always()
        uses: EnricoMi/publish-unit-test-result-action@v2
        with:
          files: junit.xml
          check_name: Test Results