Merge pull request #1201 from getlipa/fix/use-bash-for-monitor-tests

Stop using tee and just cat test.json
getlipa · Sep 26, 2024 · af8cac6 · af8cac6
2 parents 6b37f3a + 7089824
commit af8cac6
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/.github/workflows/monitoring-tests.yml b/.github/workflows/monitoring-tests.yml
@@ -47,6 +47,8 @@ jobs:
         with:
           path: .3l_local_test
           key: monitor-local-test-cache
+      - name: Cat test results
+        run: cat test.json
       - name: Check test result
         run: |
           if [ ${{ steps.monitortest.outcome }} == "failure" ]; then

diff --git a/Makefile b/Makefile
@@ -37,8 +37,7 @@ testregisternode:
 
 .PHONY: monitortest
 monitortest:
-	set -o pipefail
-	cargo +nightly test --test monitoring_test -- --test-threads=1 --ignored -Z unstable-options --report-time --format json | tee test.json
+	cargo +nightly test --test monitoring_test -- --test-threads=1 --ignored -Z unstable-options --report-time --format json > test.json
 
 .PHONY: testall
 testall: test integrationtests