From f5502fa7206b7d552f80091ed39a31ff61219028 Mon Sep 17 00:00:00 2001 From: tomaszszopinski Date: Tue, 14 Oct 2025 13:55:32 +0200 Subject: [PATCH 01/15] adding another job approach --- .github/workflows/browser-tests.yml | 12 ++++++++++++ 1 file changed, 12 insertions(+) diff --git a/.github/workflows/browser-tests.yml b/.github/workflows/browser-tests.yml index 0fca81c..6a5ac1c 100644 --- a/.github/workflows/browser-tests.yml +++ b/.github/workflows/browser-tests.yml @@ -355,3 +355,15 @@ jobs: env: SLACK_WEBHOOK_URL: ${{ secrets.SLACK_WEBHOOK_URL }} SLACK_WEBHOOK_TYPE: INCOMING_WEBHOOK + upload-test-artifacts: + needs: browser-tests + if: failure() + runs-on: ubuntu-latest + steps: + - name: Upload screenshots from failed tests + uses: actions/upload-artifact@v4 + with: + name: failed-test-screenshots + path: ${{ github.workspace }}/build/project/behat-output/*.png + if-no-files-found: ignore + retention-days: 14 From daf6514247f77725b5f39efef5922ab65ea08e67 Mon Sep 17 00:00:00 2001 From: tomaszszopinski Date: Tue, 14 Oct 2025 15:09:49 +0200 Subject: [PATCH 02/15] Trying a new yaml file --- .github/workflows/browser-tests.yml | 12 ------------ 1 file changed, 12 deletions(-) diff --git a/.github/workflows/browser-tests.yml b/.github/workflows/browser-tests.yml index 6a5ac1c..0fca81c 100644 --- a/.github/workflows/browser-tests.yml +++ b/.github/workflows/browser-tests.yml @@ -355,15 +355,3 @@ jobs: env: SLACK_WEBHOOK_URL: ${{ secrets.SLACK_WEBHOOK_URL }} SLACK_WEBHOOK_TYPE: INCOMING_WEBHOOK - upload-test-artifacts: - needs: browser-tests - if: failure() - runs-on: ubuntu-latest - steps: - - name: Upload screenshots from failed tests - uses: actions/upload-artifact@v4 - with: - name: failed-test-screenshots - path: ${{ github.workspace }}/build/project/behat-output/*.png - if-no-files-found: ignore - retention-days: 14 From 9f125205072a7fbf3b3b16a5d7ae81776e9dea3f Mon Sep 17 00:00:00 2001 From: tomaszszopinski Date: Tue, 14 Oct 2025 15:23:43 +0200 Subject: [PATCH 03/15] add test-artifacts.yaml --- .github/workflows/test-artifacts.yaml | 27 +++++++++++++++++++++++++++ 1 file changed, 27 insertions(+) create mode 100644 .github/workflows/test-artifacts.yaml diff --git a/.github/workflows/test-artifacts.yaml b/.github/workflows/test-artifacts.yaml new file mode 100644 index 0000000..ae04bef --- /dev/null +++ b/.github/workflows/test-artifacts.yaml @@ -0,0 +1,27 @@ +name: Upload Test Artifacts + +on: + workflow_run: + workflows: ["Browser tests"] + types: + - completed + +jobs: + upload-test-artifacts: + runs-on: ubuntu-latest + if: ${{ github.event.workflow_run.conclusion == 'failure' }} + steps: + - name: Download artifact + uses: dawidd6/action-download-artifact@v3 + with: + workflow: browser-tests.yaml + run_id: ${{ github.event.workflow_run.id }} + path: artifacts + + - name: Upload screenshots from failed tests + uses: actions/upload-artifact@v4 + with: + name: failed-test-screenshots + path: artifacts/**/build/project/behat-output/*.png + if-no-files-found: ignore + retention-days: 14 From 95cc216af22d5894ed5584ebd83228e37097b67f Mon Sep 17 00:00:00 2001 From: tomaszszopinski Date: Tue, 14 Oct 2025 15:53:57 +0200 Subject: [PATCH 04/15] made test-artifacts on call --- .github/workflows/test-artifacts.yaml | 27 ------------------------- .github/workflows/test-artifacts.yml | 29 +++++++++++++++++++++++++++ 2 files changed, 29 insertions(+), 27 deletions(-) delete mode 100644 .github/workflows/test-artifacts.yaml create mode 100644 .github/workflows/test-artifacts.yml diff --git a/.github/workflows/test-artifacts.yaml b/.github/workflows/test-artifacts.yaml deleted file mode 100644 index ae04bef..0000000 --- a/.github/workflows/test-artifacts.yaml +++ /dev/null @@ -1,27 +0,0 @@ -name: Upload Test Artifacts - -on: - workflow_run: - workflows: ["Browser tests"] - types: - - completed - -jobs: - upload-test-artifacts: - runs-on: ubuntu-latest - if: ${{ github.event.workflow_run.conclusion == 'failure' }} - steps: - - name: Download artifact - uses: dawidd6/action-download-artifact@v3 - with: - workflow: browser-tests.yaml - run_id: ${{ github.event.workflow_run.id }} - path: artifacts - - - name: Upload screenshots from failed tests - uses: actions/upload-artifact@v4 - with: - name: failed-test-screenshots - path: artifacts/**/build/project/behat-output/*.png - if-no-files-found: ignore - retention-days: 14 diff --git a/.github/workflows/test-artifacts.yml b/.github/workflows/test-artifacts.yml new file mode 100644 index 0000000..c0314a3 --- /dev/null +++ b/.github/workflows/test-artifacts.yml @@ -0,0 +1,29 @@ +name: Upload Test Artifacts + +on: + workflow_call: + inputs: + artifact-name: + required: false + type: string + default: 'failed-test-screenshots' + artifact-path: + required: false + type: string + default: 'build/project/behat-output/*.png' + retention-days: + required: false + type: number + default: 14 + +jobs: + upload-test-artifacts: + runs-on: ubuntu-latest + steps: + - name: Upload screenshots from failed tests + uses: actions/upload-artifact@v4 + with: + name: ${{ inputs.artifact-name }} + path: ${{ inputs.artifact-path }} + if-no-files-found: ignore + retention-days: ${{ inputs.retention-days }} From 75f5ccf2137f2dd9f9b758df413d77c97225aa5e Mon Sep 17 00:00:00 2001 From: tomaszszopinski Date: Wed, 15 Oct 2025 12:05:16 +0200 Subject: [PATCH 05/15] changed default path --- .github/workflows/test-artifacts.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/test-artifacts.yml b/.github/workflows/test-artifacts.yml index c0314a3..ebd7aec 100644 --- a/.github/workflows/test-artifacts.yml +++ b/.github/workflows/test-artifacts.yml @@ -10,7 +10,7 @@ on: artifact-path: required: false type: string - default: 'build/project/behat-output/*.png' + default: 'var/www/behat-output/*.png' retention-days: required: false type: number From a95fb8509265c20ee40fdc50d6704c11893888fe Mon Sep 17 00:00:00 2001 From: tomaszszopinski Date: Thu, 16 Oct 2025 10:24:05 +0200 Subject: [PATCH 06/15] added file list debug --- .github/workflows/test-artifacts.yml | 3 +++ 1 file changed, 3 insertions(+) diff --git a/.github/workflows/test-artifacts.yml b/.github/workflows/test-artifacts.yml index ebd7aec..a20e29f 100644 --- a/.github/workflows/test-artifacts.yml +++ b/.github/workflows/test-artifacts.yml @@ -20,6 +20,9 @@ jobs: upload-test-artifacts: runs-on: ubuntu-latest steps: + - name: List screenshot files + run: ls -l build/project/behat-output/ + run: ls -l var/www/behat-output/ - name: Upload screenshots from failed tests uses: actions/upload-artifact@v4 with: From b1b1b187a35865c98319e86c0330cd3e65793431 Mon Sep 17 00:00:00 2001 From: tomaszszopinski Date: Thu, 16 Oct 2025 10:49:01 +0200 Subject: [PATCH 07/15] fixed list --- .github/workflows/test-artifacts.yml | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/.github/workflows/test-artifacts.yml b/.github/workflows/test-artifacts.yml index a20e29f..53d3fd4 100644 --- a/.github/workflows/test-artifacts.yml +++ b/.github/workflows/test-artifacts.yml @@ -21,8 +21,9 @@ jobs: runs-on: ubuntu-latest steps: - name: List screenshot files - run: ls -l build/project/behat-output/ - run: ls -l var/www/behat-output/ + run: | + ls -l build/project/behat-output/ + ls -l var/www/behat-output/ - name: Upload screenshots from failed tests uses: actions/upload-artifact@v4 with: From b8ec68299f97b842d819f6bb3e5482b3bd88a90b Mon Sep 17 00:00:00 2001 From: tomaszszopinski Date: Thu, 16 Oct 2025 11:04:32 +0200 Subject: [PATCH 08/15] fix for job to continue --- .github/workflows/test-artifacts.yml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/.github/workflows/test-artifacts.yml b/.github/workflows/test-artifacts.yml index 53d3fd4..6f02925 100644 --- a/.github/workflows/test-artifacts.yml +++ b/.github/workflows/test-artifacts.yml @@ -22,8 +22,8 @@ jobs: steps: - name: List screenshot files run: | - ls -l build/project/behat-output/ - ls -l var/www/behat-output/ + ls -l build/project/behat-output/ || true + ls -l var/www/behat-output/ || true - name: Upload screenshots from failed tests uses: actions/upload-artifact@v4 with: From b44346c92dad43ca0a03967daa56e4dadcd8412c Mon Sep 17 00:00:00 2001 From: tomaszszopinski Date: Thu, 16 Oct 2025 13:53:22 +0200 Subject: [PATCH 09/15] more debugging --- .github/workflows/test-artifacts.yml | 1 + 1 file changed, 1 insertion(+) diff --git a/.github/workflows/test-artifacts.yml b/.github/workflows/test-artifacts.yml index 6f02925..73599e2 100644 --- a/.github/workflows/test-artifacts.yml +++ b/.github/workflows/test-artifacts.yml @@ -24,6 +24,7 @@ jobs: run: | ls -l build/project/behat-output/ || true ls -l var/www/behat-output/ || true + ls -l - name: Upload screenshots from failed tests uses: actions/upload-artifact@v4 with: From 58be898b26196fdd60bd2e0388128e254e07ee0f Mon Sep 17 00:00:00 2001 From: tomaszszopinski Date: Thu, 16 Oct 2025 14:52:16 +0200 Subject: [PATCH 10/15] added more listing --- .github/workflows/test-artifacts.yml | 1 + 1 file changed, 1 insertion(+) diff --git a/.github/workflows/test-artifacts.yml b/.github/workflows/test-artifacts.yml index 73599e2..6872127 100644 --- a/.github/workflows/test-artifacts.yml +++ b/.github/workflows/test-artifacts.yml @@ -24,6 +24,7 @@ jobs: run: | ls -l build/project/behat-output/ || true ls -l var/www/behat-output/ || true + ls -l var/www/html/build/project/behat-output || true ls -l - name: Upload screenshots from failed tests uses: actions/upload-artifact@v4 From 526b15a07a5eabe0f3b58252a26fbef9d883f79a Mon Sep 17 00:00:00 2001 From: tomaszszopinski Date: Fri, 20 Feb 2026 11:45:23 +0100 Subject: [PATCH 11/15] more verbose yaml --- .github/workflows/test-artifacts.yml | 64 +++++++++++++++------------- 1 file changed, 34 insertions(+), 30 deletions(-) diff --git a/.github/workflows/test-artifacts.yml b/.github/workflows/test-artifacts.yml index 6872127..a6c6e89 100644 --- a/.github/workflows/test-artifacts.yml +++ b/.github/workflows/test-artifacts.yml @@ -1,35 +1,39 @@ name: Upload Test Artifacts on: - workflow_call: - inputs: - artifact-name: - required: false - type: string - default: 'failed-test-screenshots' - artifact-path: - required: false - type: string - default: 'var/www/behat-output/*.png' - retention-days: - required: false - type: number - default: 14 + workflow_call: + inputs: + artifact-name: + required: false + type: string + default: 'failed-test-screenshots' + artifact-path: + required: false + type: string + # More robust default: upload all PNGs found anywhere + default: '**/*.png' + retention-days: + required: false + type: number + default: 14 jobs: - upload-test-artifacts: - runs-on: ubuntu-latest - steps: - - name: List screenshot files - run: | - ls -l build/project/behat-output/ || true - ls -l var/www/behat-output/ || true - ls -l var/www/html/build/project/behat-output || true - ls -l - - name: Upload screenshots from failed tests - uses: actions/upload-artifact@v4 - with: - name: ${{ inputs.artifact-name }} - path: ${{ inputs.artifact-path }} - if-no-files-found: ignore - retention-days: ${{ inputs.retention-days }} + upload-test-artifacts: + runs-on: ubuntu-latest + steps: + - name: Print workspace location + run: pwd + + - name: List all files in workspace + run: ls -lR + + - name: List all PNG files recursively + run: find . -name '*.png' -print + + - name: Upload screenshots from failed tests + uses: actions/upload-artifact@v4 + with: + name: ${{ inputs.artifact-name }} + path: ${{ inputs.artifact-path }} + if-no-files-found: ignore + retention-days: ${{ inputs.retention-days }} \ No newline at end of file From 1e00c868c966a13c3a8fede843a2ad22b3af3767 Mon Sep 17 00:00:00 2001 From: tomaszszopinski Date: Mon, 23 Feb 2026 08:21:37 +0100 Subject: [PATCH 12/15] added harcoded path --- .github/workflows/test-artifacts.yml | 15 +++++++-------- 1 file changed, 7 insertions(+), 8 deletions(-) diff --git a/.github/workflows/test-artifacts.yml b/.github/workflows/test-artifacts.yml index a6c6e89..cdd8e58 100644 --- a/.github/workflows/test-artifacts.yml +++ b/.github/workflows/test-artifacts.yml @@ -24,16 +24,15 @@ jobs: - name: Print workspace location run: pwd - - name: List all files in workspace - run: ls -lR - - - name: List all PNG files recursively - run: find . -name '*.png' -print - - name: Upload screenshots from failed tests uses: actions/upload-artifact@v4 with: name: ${{ inputs.artifact-name }} - path: ${{ inputs.artifact-path }} + path: /tmp/behat-output if-no-files-found: ignore - retention-days: ${{ inputs.retention-days }} \ No newline at end of file + retention-days: ${{ inputs.retention-days }} + - name: List all files in workspace + run: ls -lR + + - name: List all PNG files recursively + run: find . -name '*.png' -print \ No newline at end of file From 0d7577c7cb60ff2483f4fb7cdbd04943eebd39ff Mon Sep 17 00:00:00 2001 From: tomaszszopinski Date: Mon, 23 Feb 2026 09:07:16 +0100 Subject: [PATCH 13/15] restored path --- .github/workflows/test-artifacts.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/test-artifacts.yml b/.github/workflows/test-artifacts.yml index cdd8e58..1dac81f 100644 --- a/.github/workflows/test-artifacts.yml +++ b/.github/workflows/test-artifacts.yml @@ -28,7 +28,7 @@ jobs: uses: actions/upload-artifact@v4 with: name: ${{ inputs.artifact-name }} - path: /tmp/behat-output + path: ${{ inputs.artifact-path }} if-no-files-found: ignore retention-days: ${{ inputs.retention-days }} - name: List all files in workspace From e1093a9b69afcf5ad692db55a7abd914dc2a77a6 Mon Sep 17 00:00:00 2001 From: tomaszszopinski Date: Mon, 23 Feb 2026 10:28:01 +0100 Subject: [PATCH 14/15] going back to hardcoded --- .github/workflows/test-artifacts.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/test-artifacts.yml b/.github/workflows/test-artifacts.yml index 1dac81f..b1d9fa0 100644 --- a/.github/workflows/test-artifacts.yml +++ b/.github/workflows/test-artifacts.yml @@ -28,7 +28,7 @@ jobs: uses: actions/upload-artifact@v4 with: name: ${{ inputs.artifact-name }} - path: ${{ inputs.artifact-path }} + path: tmp/behat-output if-no-files-found: ignore retention-days: ${{ inputs.retention-days }} - name: List all files in workspace From 4e02d1ee6afbc2bf6b6e95cc5ff79f6f967de971 Mon Sep 17 00:00:00 2001 From: tomaszszopinski Date: Mon, 23 Feb 2026 11:45:54 +0100 Subject: [PATCH 15/15] yet another approach --- .github/workflows/test-artifacts.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/test-artifacts.yml b/.github/workflows/test-artifacts.yml index b1d9fa0..1dac81f 100644 --- a/.github/workflows/test-artifacts.yml +++ b/.github/workflows/test-artifacts.yml @@ -28,7 +28,7 @@ jobs: uses: actions/upload-artifact@v4 with: name: ${{ inputs.artifact-name }} - path: tmp/behat-output + path: ${{ inputs.artifact-path }} if-no-files-found: ignore retention-days: ${{ inputs.retention-days }} - name: List all files in workspace