From edb87a453594ec44c1512318bebbf5f1eb97bc3d Mon Sep 17 00:00:00 2001 From: aasim Date: Sat, 4 May 2024 13:14:18 -0400 Subject: [PATCH] moved 'wait for web server' immediately after docker compose; removed 'docker down' --- .github/workflows/pr_ci_playwright_e2e.yaml | 14 +++++--------- 1 file changed, 5 insertions(+), 9 deletions(-) diff --git a/.github/workflows/pr_ci_playwright_e2e.yaml b/.github/workflows/pr_ci_playwright_e2e.yaml index 1c71c2626..e191b31c2 100644 --- 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