From 64e118659925dccb8d3bf1641732377b0780ee7c Mon Sep 17 00:00:00 2001 From: Andy Cohen Date: Thu, 22 Jan 2026 09:43:22 -0600 Subject: [PATCH 1/3] Add overridable env var FAILURE_SCREENSHOTS_PATH --- system-tests/action.yml | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/system-tests/action.yml b/system-tests/action.yml index e60a1a6..07935d2 100644 --- a/system-tests/action.yml +++ b/system-tests/action.yml @@ -81,7 +81,9 @@ runs: uses: actions/upload-artifact@v4 with: name: rspec-screenshots - path: tmp/capybara + path: ${{ env.FAILURE_SCREENSHOTS_PATH }} + env: + FAILURE_SCREENSHOTS_PATH: tmp/capybara - name: Cleanup uses: RoleModel/actions/test-cleanup@v1 From e2171ec4be251121922470c553820d596079f031 Mon Sep 17 00:00:00 2001 From: Andy Cohen Date: Thu, 22 Jan 2026 09:58:29 -0600 Subject: [PATCH 2/3] move to input instead of ENV --- system-tests/action.yml | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/system-tests/action.yml b/system-tests/action.yml index 07935d2..cead257 100644 --- a/system-tests/action.yml +++ b/system-tests/action.yml @@ -1,5 +1,10 @@ name: 'Ruby System Tests' description: 'Runs Ruby system tests' +inputs: + failure-screenshot-dir: + description: 'tmp directory where your tests save failure screenshots to' + required: false + default: 'tmp/capybara' runs: using: "composite" @@ -81,9 +86,7 @@ runs: uses: actions/upload-artifact@v4 with: name: rspec-screenshots - path: ${{ env.FAILURE_SCREENSHOTS_PATH }} - env: - FAILURE_SCREENSHOTS_PATH: tmp/capybara + path: ${{ inputs.failure-screenshot-dir }} - name: Cleanup uses: RoleModel/actions/test-cleanup@v1 From 5a2c7fb002f7d4989459c300fe02e099337ce152 Mon Sep 17 00:00:00 2001 From: Andy Cohen Date: Thu, 22 Jan 2026 10:21:02 -0600 Subject: [PATCH 3/3] clean up --- system-tests/action.yml | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/system-tests/action.yml b/system-tests/action.yml index cead257..71b31f0 100644 --- a/system-tests/action.yml +++ b/system-tests/action.yml @@ -1,10 +1,10 @@ -name: 'Ruby System Tests' -description: 'Runs Ruby system tests' +name: Ruby System Tests +description: Runs Ruby system tests inputs: failure-screenshot-dir: - description: 'tmp directory where your tests save failure screenshots to' + description: tmp directory where your tests save failure screenshots to required: false - default: 'tmp/capybara' + default: tmp/capybara runs: using: "composite"