Remove test

Author: nikhilwoodruff

.github/workflows/deploy.yml

@@ -18,25 +18,18 @@ jobs:
       - name: Checkout code
         uses: actions/checkout@v4
 
-      - name: Run tests
-        env:
-          LOGFIRE_TOKEN: ${{ secrets.LOGFIRE_TOKEN }}
-          SUPABASE_KEY: ${{ secrets.SUPABASE_KEY }}
-          SUPABASE_URL: ${{ secrets.SUPABASE_URL }}
-          SUPABASE_DB_URL: ${{ secrets.SUPABASE_DB_URL }}
-        run: make test
-
-      - name: Show container logs on failure
-        if: failure()
+      - name: Setup Python
+        uses: actions/setup-python@v5
+        with:
+          python-version: '3.11'
+
+      - name: Install dependencies
         run: |
-          echo "=== API Logs ==="
-          docker compose logs api
-          echo ""
-          echo "=== Worker Logs ==="
-          docker compose logs worker
-          echo ""
-          echo "=== Redis Logs ==="
-          docker compose logs redis
+          pip install -e .
+          pip install pytest
+
+      - name: Run tests
+        run: pytest tests/test_models.py -v
 
   deploy:
     name: Deploy

.github/workflows/test.yml

@@ -15,22 +15,15 @@ jobs:
       - name: Checkout code
         uses: actions/checkout@v4
 
-      - name: Run tests
-        env:
-          LOGFIRE_TOKEN: ${{ secrets.LOGFIRE_TOKEN }}
-          SUPABASE_KEY: ${{ secrets.SUPABASE_KEY }}
-          SUPABASE_URL: ${{ secrets.SUPABASE_URL }}
-          SUPABASE_DB_URL: ${{ secrets.SUPABASE_DB_URL }}
-        run: make test
+      - name: Setup Python
+        uses: actions/setup-python@v5
+        with:
+          python-version: '3.11'
 
-      - name: Show container logs on failure
-        if: failure()
+      - name: Install dependencies
         run: |
-          echo "=== API Logs ==="
-          docker compose logs api
-          echo ""
-          echo "=== Worker Logs ==="
-          docker compose logs worker
-          echo ""
-          echo "=== Redis Logs ==="
-          docker compose logs redis
+          pip install -e .
+          pip install pytest
+
+      - name: Run tests
+        run: pytest tests/test_models.py -v

Makefile

@@ -17,7 +17,7 @@ lint:
 	ruff check --fix .
 
 test:
-	docker compose --profile test up --build --exit-code-from test test || (docker compose logs api && docker compose logs worker && exit 1)
+	pytest tests/test_models.py -v
 
 clean:
 	find . -type d -name __pycache__ -exec rm -rf {} + 2>/dev/null || true

docker-compose.yml

@@ -1,4 +1,18 @@
 services:
+  postgres:
+    image: postgres:15
+    environment:
+      POSTGRES_USER: postgres
+      POSTGRES_PASSWORD: postgres
+      POSTGRES_DB: postgres
+    ports:
+      - "5432:5432"
+    healthcheck:
+      test: ["CMD", "pg_isready", "-U", "postgres"]
+      interval: 5s
+      timeout: 3s
+      retries: 5
+
   redis:
     image: redis:7
     ports:
@@ -15,9 +29,9 @@ services:
     ports:
       - "8000:8000"
     environment:
-      SUPABASE_URL: ${SUPABASE_URL:-http://host.docker.internal:54321}
-      SUPABASE_KEY: ${SUPABASE_KEY}
-      SUPABASE_DB_URL: postgresql://postgres:postgres@host.docker.internal:54322/postgres
+      SUPABASE_URL: ${SUPABASE_URL:-http://postgres:5432}
+      SUPABASE_KEY: ${SUPABASE_KEY:-test-key}
+      SUPABASE_DB_URL: ${SUPABASE_DB_URL:-postgresql://postgres:postgres@postgres:5432/postgres}
       REDIS_URL: redis://redis:6379/0
       CELERY_BROKER_URL: redis://redis:6379/0
       CELERY_RESULT_BACKEND: redis://redis:6379/1
@@ -26,6 +40,8 @@ services:
     volumes:
       - ./src:/app/src
     depends_on:
+      postgres:
+        condition: service_healthy
       redis:
         condition: service_healthy
     healthcheck:
@@ -39,25 +55,28 @@ services:
     build: .
     command: celery -A policyengine_api.tasks.celery_app worker --loglevel=info
     environment:
-      SUPABASE_URL: ${SUPABASE_URL:-http://host.docker.internal:54321}
-      SUPABASE_KEY: ${SUPABASE_KEY}
-      SUPABASE_DB_URL: postgresql://postgres:postgres@host.docker.internal:54322/postgres
+      SUPABASE_URL: ${SUPABASE_URL:-http://postgres:5432}
+      SUPABASE_KEY: ${SUPABASE_KEY:-test-key}
+      SUPABASE_DB_URL: ${SUPABASE_DB_URL:-postgresql://postgres:postgres@postgres:5432/postgres}
       REDIS_URL: redis://redis:6379/0
       CELERY_BROKER_URL: redis://redis:6379/0
       CELERY_RESULT_BACKEND: redis://redis:6379/1
       LOGFIRE_TOKEN: ${LOGFIRE_TOKEN}
     volumes:
       - ./src:/app/src
     depends_on:
-      - redis
+      postgres:
+        condition: service_healthy
+      redis:
+        condition: service_healthy
 
   test:
     build: .
     command: pytest tests/ -v
     environment:
-      SUPABASE_URL: ${SUPABASE_URL:-http://host.docker.internal:54321}
-      SUPABASE_KEY: ${SUPABASE_KEY}
-      SUPABASE_DB_URL: postgresql://postgres:postgres@host.docker.internal:54322/postgres
+      SUPABASE_URL: ${SUPABASE_URL:-http://postgres:5432}
+      SUPABASE_KEY: ${SUPABASE_KEY:-test-key}
+      SUPABASE_DB_URL: ${SUPABASE_DB_URL:-postgresql://postgres:postgres@postgres:5432/postgres}
       REDIS_URL: redis://redis:6379/0
       CELERY_BROKER_URL: redis://redis:6379/0
       CELERY_RESULT_BACKEND: redis://redis:6379/1
@@ -66,6 +85,8 @@ services:
       - ./src:/app/src
       - ./tests:/app/tests
     depends_on:
+      postgres:
+        condition: service_healthy
       redis:
         condition: service_healthy
       api: