Skip to content

Commit cf3087b

Browse files
committed
chore: pre-create a directory for reports to make it exists even when integration tests weren't run
When `docker compose up` has failed because a container couldn't startup, the integration tests step is skipped and directory hurl-reports wasn't created. In this case, "Save application logs" step also has been failing with error: tee: ../hurl-reports/application-logs.txt: No such file or directory Part of #13
1 parent 465d6fd commit cf3087b

File tree

1 file changed

+6
-3
lines changed

1 file changed

+6
-3
lines changed

.github/workflows/integration-tests.yml

+6-3
Original file line numberDiff line numberDiff line change
@@ -63,6 +63,9 @@ jobs:
6363
- name: Show docker compose version
6464
run: docker compose version
6565

66+
- name: Create directory for reports
67+
run: mkdir tests-reports
68+
6669
- name: Start containers
6770
working-directory: docker
6871
run: >-
@@ -83,7 +86,7 @@ jobs:
8386
run: >-
8487
hurl \
8588
--error-format long \
86-
--report-html hurl-reports \
89+
--report-html tests-reports \
8790
--variable SERVER_URL=http://127.0.0.1:${{ matrix.application-port }} \
8891
--test \
8992
tests/crud.hurl
@@ -95,7 +98,7 @@ jobs:
9598
docker compose logs \
9699
--no-log-prefix \
97100
--timestamps \
98-
${{ matrix.docker-service-name }} | tee ../hurl-reports/application-logs.txt
101+
${{ matrix.docker-service-name }} | tee ../tests-reports/application-logs.txt
99102
100103
- name: Stop containers
101104
if: always()
@@ -111,4 +114,4 @@ jobs:
111114
uses: actions/[email protected] # https://github.com/actions/upload-artifact
112115
with:
113116
name: ${{ matrix.docker-service-name }}-report-and-logs
114-
path: hurl-reports/
117+
path: tests-reports/

0 commit comments

Comments
 (0)