moved 'wait for web server' immediately after docker compose; removed…

… 'docker down'
aasimsyed · May 4, 2024 · edb87a4 · edb87a4
1 parent 62acabd
commit edb87a4
Showing 1 changed file with 5 additions and 9 deletions.
diff --git a/.github/workflows/pr_ci_playwright_e2e.yaml b/.github/workflows/pr_ci_playwright_e2e.yaml
@@ -29,6 +29,11 @@ jobs:
       - name: Run Docker Compose
         run: docker compose --env-file .env.dev up --build -d
 
+      - name: Wait for web server to be ready
+        working-directory: ./frontend
+        run: |
+          curl --retry 20 --retry-delay 5 --retry-all-errors http://localhost:3000
+
       - name: Setup Node environment
         uses: actions/setup-node@v4
         with:
@@ -44,11 +49,6 @@ jobs:
         run: |
           sudo yarn playwright install --with-deps
 
-      - name: Wait for web server to be ready
-        working-directory: ./frontend
-        run: |
-          curl --retry 15 --retry-delay 5 --retry-all-errors http://localhost:3000
-
       - name: Run Playwright tests
         working-directory: ./frontend
         run: |
@@ -61,7 +61,3 @@ jobs:
           name: playwright-report
           path: frontend/playwright-report/
           retention-days: 30
-
-      - name: Shutdown Docker Compose
-        if: always()
-        run: docker-compose down