From 88b8838811cd4c64f6155cd91e319fd4718e8f75 Mon Sep 17 00:00:00 2001 From: sinejespersen Date: Thu, 11 Sep 2025 08:57:19 +0200 Subject: [PATCH 1/5] TEST: add paths to pull request/GA --- .github/workflows/playwright.yaml | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/.github/workflows/playwright.yaml b/.github/workflows/playwright.yaml index 5b78012d6..9dc688236 100644 --- a/.github/workflows/playwright.yaml +++ b/.github/workflows/playwright.yaml @@ -1,4 +1,7 @@ -on: pull_request +on: + pull_request: + paths: + - "assets/admin/**" name: Test From 15812fdb4410f549ecf919f1220a87b50802a21a Mon Sep 17 00:00:00 2001 From: sinejespersen Date: Thu, 11 Sep 2025 08:58:39 +0200 Subject: [PATCH 2/5] add console log to test if playright action is triggered --- assets/admin/components/slide/slide-manager.jsx | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/assets/admin/components/slide/slide-manager.jsx b/assets/admin/components/slide/slide-manager.jsx index b7a725930..23b665f53 100644 --- a/assets/admin/components/slide/slide-manager.jsx +++ b/assets/admin/components/slide/slide-manager.jsx @@ -44,7 +44,7 @@ function SlideManager({ const { t } = useTranslation("common", { keyPrefix: "slide-manager" }); const dispatch = useDispatch(); const navigate = useNavigate(); - + console.log("test"); // Context const context = useContext(UserContext); From d1342fb3f9ef54e32ff2b22c4893b7a0cf973a19 Mon Sep 17 00:00:00 2001 From: sinejespersen Date: Thu, 11 Sep 2025 09:21:43 +0200 Subject: [PATCH 3/5] two different GA files, one for admin one for template --- ...{playwright.yaml => playwright_admin.yaml} | 2 +- .github/workflows/playwright_template.yaml | 49 +++++++++++++++++++ 2 files changed, 50 insertions(+), 1 deletion(-) rename .github/workflows/{playwright.yaml => playwright_admin.yaml} (99%) create mode 100644 .github/workflows/playwright_template.yaml diff --git a/.github/workflows/playwright.yaml b/.github/workflows/playwright_admin.yaml similarity index 99% rename from .github/workflows/playwright.yaml rename to .github/workflows/playwright_admin.yaml index 9dc688236..73ec1f2fe 100644 --- a/.github/workflows/playwright.yaml +++ b/.github/workflows/playwright_admin.yaml @@ -39,7 +39,7 @@ jobs: CI: "true" run: | docker compose run --rm playwright npx playwright install --with-deps - docker compose run --rm playwright npx playwright test + docker compose run --rm playwright npx playwright test admin - uses: actions/upload-artifact@v4 if: always() diff --git a/.github/workflows/playwright_template.yaml b/.github/workflows/playwright_template.yaml new file mode 100644 index 000000000..6288f6a74 --- /dev/null +++ b/.github/workflows/playwright_template.yaml @@ -0,0 +1,49 @@ +on: + pull_request: + paths: + - "assets/template/**" + +name: Test + +env: + COMPOSE_USER: runner + +jobs: + frontend-build-and-test: + name: Playwright + runs-on: ubuntu-latest + strategy: + fail-fast: true + steps: + - name: Checkout + uses: actions/checkout@v4 + + - name: Setup network + run: docker network create frontend + + - name: Composer install + run: | + docker compose run --rm phpfpm composer install + + - name: Copy fixture assets to public/fixtures + run: | + docker compose run --rm phpfpm cp -r fixtures/public/fixtures public/fixtures + + - name: Build assets + run: | + docker compose run --rm node npm install + docker compose run --rm node npm run build + + - name: Run playwright + env: + CI: "true" + run: | + docker compose run --rm playwright npx playwright install --with-deps + docker compose run --rm playwright npx playwright test template + + - uses: actions/upload-artifact@v4 + if: always() + with: + name: playwright-report + path: playwright-report/ + retention-days: 30 From 18f22b64172f1b53a90a5632d92132d316bf64f0 Mon Sep 17 00:00:00 2001 From: sinejespersen Date: Thu, 11 Sep 2025 09:22:37 +0200 Subject: [PATCH 4/5] remove console.log test and hopefully trigger playwright/admin --- assets/admin/components/slide/slide-manager.jsx | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/assets/admin/components/slide/slide-manager.jsx b/assets/admin/components/slide/slide-manager.jsx index 23b665f53..b7a725930 100644 --- a/assets/admin/components/slide/slide-manager.jsx +++ b/assets/admin/components/slide/slide-manager.jsx @@ -44,7 +44,7 @@ function SlideManager({ const { t } = useTranslation("common", { keyPrefix: "slide-manager" }); const dispatch = useDispatch(); const navigate = useNavigate(); - console.log("test"); + // Context const context = useContext(UserContext); From f6c4b60368f83bdefad794f641c94d8fbc732835 Mon Sep 17 00:00:00 2001 From: sinejespersen Date: Thu, 11 Sep 2025 09:24:23 +0200 Subject: [PATCH 5/5] add test variable to trigger actions --- assets/admin/components/slide/slide-manager.jsx | 1 + 1 file changed, 1 insertion(+) diff --git a/assets/admin/components/slide/slide-manager.jsx b/assets/admin/components/slide/slide-manager.jsx index b7a725930..916cc7f70 100644 --- a/assets/admin/components/slide/slide-manager.jsx +++ b/assets/admin/components/slide/slide-manager.jsx @@ -45,6 +45,7 @@ function SlideManager({ const dispatch = useDispatch(); const navigate = useNavigate(); + const test = "test"; // Context const context = useContext(UserContext);