Skip to content

Commit fa39aee

Browse files
committed
fix: standardize output variable names in e2e workflow
1 parent 563470e commit fa39aee

1 file changed

Lines changed: 10 additions & 10 deletions

File tree

.github/workflows/e2e.yml

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ jobs:
2323
runs-on: ubuntu-latest
2424
name: "Clean test resources"
2525
outputs:
26-
test-suites: ${{ steps.plan.outputs.test-suites }}
26+
testsuites: ${{ steps.plan.outputs.testsuites }}
2727
steps:
2828
- name: Checkout
2929
uses: actions/checkout@v4
@@ -47,20 +47,20 @@ jobs:
4747
- name: Plan
4848
id: plan
4949
run: |
50-
# Set test-suites output based on event type
50+
# Set testsuites output based on event type
5151
if [[ "${{ github.event_name }}" == "schedule" ]]; then
52-
echo 'test-suites=["--workflow-dispatch-only"]' >> "$GITHUB_OUTPUT"
52+
echo 'testsuites=["--workflow-dispatch-only"]' >> "$GITHUB_OUTPUT"
5353
else
54-
echo 'test-suites=["--issue-triggered-only", "--command-triggered-only", "--workflow-dispatch-only"]' >> "$GITHUB_OUTPUT"
54+
echo 'testsuites=["--issue-triggered-only", "--command-triggered-only", "--workflow-dispatch-only"]' >> "$GITHUB_OUTPUT"
5555
fi
5656
e2e:
5757
runs-on: ubuntu-latest
5858
needs: clean
5959
strategy:
6060
matrix:
61-
test-suite: ${{ fromJSON(needs.clean.outputs.test-suites) }}
61+
testsuite: ${{ fromJSON(needs.clean.outputs.testsuites) }}
6262
fail-fast: false
63-
name: "E2E Tests: ${{ matrix.test-suite.description }}"
63+
name: "E2E Tests: ${{ matrix.testsuite }}"
6464
steps:
6565
- name: Checkout
6666
uses: actions/checkout@v4
@@ -90,16 +90,16 @@ jobs:
9090
9191
# Run the e2e script, tee output to both stdout and log, capture exit code
9292
set +e
93-
./e2e.sh ${{ matrix.test-suite.option }} 2>&1 | tee e2e-output.log
93+
./e2e.sh ${{ matrix.testsuite }} 2>&1 | tee e2e-output.log
9494
rc=${PIPESTATUS[0]}
9595
set -e
9696
97-
echo "\n## E2E test run output (${{ matrix.test-suite.description }})" >> "$GITHUB_STEP_SUMMARY"
97+
echo "\n## E2E test run output (${{ matrix.testsuite }})" >> "$GITHUB_STEP_SUMMARY"
9898
echo '``````bash' >> "$GITHUB_STEP_SUMMARY"
9999
sed -n '1,50000p' e2e-output.log >> "$GITHUB_STEP_SUMMARY" || true
100100
echo '``````' >> "$GITHUB_STEP_SUMMARY"
101101
102-
echo "Detailed log attached as artifact: e2e-output-${{ matrix.test-suite.name }}.log"
102+
echo "Detailed log attached as artifact: e2e-output-${{ matrix.testsuite }}.log"
103103
104104
# Fail the job if the script exited non-zero so GH shows failure status
105105
if [[ $rc -ne 0 ]]; then
@@ -110,5 +110,5 @@ jobs:
110110
if: always()
111111
uses: actions/upload-artifact@v4
112112
with:
113-
name: e2e-output-${{ matrix.test-suite.name }}-log
113+
name: e2e-output-${{ matrix.testsuite.name }}-log
114114
path: e2e-output.log

0 commit comments

Comments
 (0)