Skip to content

Commit

Permalink
The gtest options thing puts in a newline which messes up the redirec…
Browse files Browse the repository at this point in the history
…tion after that. Attempting to fix.
  • Loading branch information
ygoldfeld committed Dec 15, 2023
1 parent 5213c35 commit 7500480
Showing 1 changed file with 8 additions and 2 deletions.
10 changes: 8 additions & 2 deletions .github/workflows/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -270,7 +270,7 @@ jobs:
Borrower_shm_pool_collection_test.Multiprocess:\
Shm_pool_collection_test.Interface:\
Shm_pool_collection_test.Multiprocess:\
Shm_session_data_test.Multithread_object_database\
Shm_session_data_test.Multithread_object_database
cmake-flags: # See the other *SAN; but for now I heed jkontrik's words to keep consistent with non-*SAN.
- id: relwithdebinfo
conan-build-type: RelWithDebInfo
Expand Down Expand Up @@ -606,7 +606,13 @@ jobs:
echo $TSAN_OPTIONS
cd $GITHUB_WORKSPACE/install/${{ matrix.build-type.conan-build-type }}/bin
mkdir -p logs/libipc_unit_test
./libipc_unit_test.exec ${{ matrix.build-type.gtest-unit-tests-params }} > logs/libipc_unit_test/console.log 2>&1
# Some newline issues with the possible additional args; so need to make a wrapper script
# and then redirect, as desired, its output.
cat <<'EOF' > $GITHUB_WORKSPACE/install/${{ matrix.build-type.conan-build-type }}/bin/run_unit_test.sh
./libipc_unit_test.exec ${{ matrix.build-type.gtest-unit-tests-params }}
EOF
/usr/bin/bash -e $GITHUB_WORKSPACE/install/${{ matrix.build-type.conan-build-type }}/bin/run_unit_test.sh \
> logs/libipc_unit_test/console.log 2>&1
# For tests below where on failure it can be very helpful to look at higher-verbosity logs,
# any failing step is repeated with logging hiked up. Note that formally (and practically)
Expand Down

0 comments on commit 7500480

Please sign in to comment.