fix(ci): isolate e2e container name to prevent killing production

This commit is contained in:
Jarian Cottingham 2026-07-07 22:54:17 +00:00
parent 9c2ffabfbf
commit 1298f88df8

View File

@ -98,8 +98,8 @@ jobs:
- name: Start app container
run: |
docker rm -f newsarchiver 2>/dev/null || true
docker run -d --name newsarchiver \
docker rm -f newsarchiver-e2e 2>/dev/null || true
docker run -d --name newsarchiver-e2e \
--network newsarchiver-network \
-p 5000:5000 \
-e ADMIN_PASSWORD="" \
@ -111,7 +111,7 @@ jobs:
- name: Seed test data
run: |
sleep 3
docker exec newsarchiver python3 -c "
docker exec newsarchiver-e2e python3 -c "
import sqlite3, os
db = os.environ.get('ARCHIVE_DIR', '/app/archival_data') + '/cache.db'
conn = sqlite3.connect(db)
@ -152,14 +152,14 @@ jobs:
--network newsarchiver-network \
-v $GITHUB_WORKSPACE/tests/playwright:/tests \
-w /tests \
-e APP_URL=http://newsarchiver:5000 \
-e APP_URL=http://newsarchiver-e2e:5000 \
mcr.microsoft.com/playwright:v1.51.0-jammy \
npx playwright test
- name: Cleanup
if: always()
run: |
docker rm -f newsarchiver || true
docker rm -f newsarchiver-e2e || true
docker network rm newsarchiver-network 2>/dev/null || true
docker-build: