Skip to content

Commit b5e3678

Browse files
Merge pull request #8 from fleetingbytes/develop
test(fix): use variables and output redirection
2 parents e02b80c + d6dd42f commit b5e3678

File tree

1 file changed

+9
-4
lines changed

1 file changed

+9
-4
lines changed

.github/workflows/test-e2e.yml

Lines changed: 9 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -30,6 +30,11 @@ jobs:
3030
test-e2e:
3131
name: End-to-End Test
3232
runs-on: ubuntu-latest
33+
env:
34+
APP_CONSOLE_LOG: "e2e-test-app-stderr.log"
35+
APP_FILE_LOG: "e2e-test-logfile.log"
36+
APP_FILE_LOG_ROOT: "e2e-test-logfile_root.log"
37+
APP_EXPECTED_LOG_LINES: 3
3338
steps:
3439
- !step
3540
name: Setup | Checkout Repository
@@ -71,10 +76,10 @@ jobs:
7176
run: ${{ inputs.run-test-command }}
7277

7378
- name: Test | Run End-to-End App Test
74-
run: |
75-
[ $(./tests/e2e_test_app.py 2>&1 | wc -l) -eq 3 ]
79+
run: ./tests/e2e_test_app.py 2>"$APP_CONSOLE_LOG"
7680

7781
- name: Test | Check End-to-End Test Results
7882
run: |
79-
[ "$(wc -l < e2e-test-logfile.log)" -eq 3 ] && \
80-
[ ! -s "e2e-test-logfile_root.log" ]
83+
[ "$(wc -l < $APP_CONSOLE_LOG)" -eq $APP_EXPECTED_LOG_LINES ] && \
84+
[ "$(wc -l < $APP_FILE_LOG)" -eq $APP_EXPECTED_LOG_LINES ] && \
85+
[ ! -s "$APP_FILE_LOG_ROOT" ]

0 commit comments

Comments
 (0)