From f42a826945809e0eab475edd2550f4514cb07a70 Mon Sep 17 00:00:00 2001 From: heinrich5991 Date: Mon, 31 Jan 2022 00:07:48 +0100 Subject: [PATCH] Fix Windows CI not properly erroring out on errors Previously, it would not error when tests failed to build or run. Powershell (at least on CI) apparently only cares about the last exit status, so separate running the server from running the tests. Fixes #3057. --- .github/workflows/build.yaml | 13 +++++++++++++ 1 file changed, 13 insertions(+) diff --git a/.github/workflows/build.yaml b/.github/workflows/build.yaml index 0f8cd62ba..1b5ac376a 100644 --- a/.github/workflows/build.yaml +++ b/.github/workflows/build.yaml @@ -98,6 +98,10 @@ jobs: run: | cd debug ${{ matrix.cmake-path }}cmake --build . --config Debug --target run_tests ${{ matrix.build-args }} + - name: Run debug server + env: ${{ matrix.env }} + run: | + cd debug ./DDNet-Server shutdown - name: Build in release mode @@ -112,6 +116,10 @@ jobs: run: | cd release ${{ matrix.cmake-path }}cmake --build . --config Release --target run_tests ${{ matrix.build-args }} + - name: Run release server + env: ${{ matrix.env }} + run: | + cd release ./DDNet-Server shutdown - name: Build headless client @@ -142,6 +150,11 @@ jobs: run: | cd fancy ${{ matrix.cmake-path }}cmake --build . --config RelWithDebInfo --target run_tests ${{ matrix.build-args }} + - name: Run fancy server + if: matrix.fancy + env: ${{ matrix.env }} + run: | + cd fancy ./DDNet-Server shutdown - name: Package