Merge pull request #1682 from private-octopus/revise-github-actions

Update github actions to show results

Author: huitema

.github/workflows/ci-tests-clang.yml

@@ -41,6 +41,7 @@ jobs:
         run: |
             ulimit -c unlimited -S
             cd build
-            make test && QUICRESULT=$?
-            if [[ ${QUICRESULT} == 0 ]]; then exit 0; fi;
+            ./picoquic_ct -S .. -n -r && QUICRESULT=$?
+            ./picohttp_ct -S .. -n -r -x http_corrupt && HTTPRESULT=$?
+            if [[ ${QUICRESULT} == 0 ]] && [[ ${HTTPRESULT} == 0 ]]; then exit 0; fi;
             exit 1

.github/workflows/ci-tests-mac-os.yml

@@ -37,6 +37,7 @@ jobs:
         run: |
             ulimit -c unlimited -S
             cd build
-            make test && QUICRESULT=$?
-            if [[ ${QUICRESULT} == 0 ]]; then exit 0; fi;
+            ./picoquic_ct -S .. -n -r && QUICRESULT=$?
+            ./picohttp_ct -S .. -n -r -x http_corrupt && HTTPRESULT=$?
+            if [[ ${QUICRESULT} == 0 ]] && [[ ${HTTPRESULT} == 0 ]]; then exit 0; fi;
             exit 1

.github/workflows/ci-tests-no-fusion.yml

@@ -36,10 +36,9 @@ jobs:
         run: |
             ulimit -c unlimited -S
             cd build
-            make test && QUICRESULT=$?
-            cd ..
-            if [[ ${QUICRESULT} == 0 ]]; then exit 0; fi;
-            cat /home/runner/work/picoquic/picoquic/build/Testing/Temporary/LastTest.log
+            ./picoquic_ct -S .. -n -r && QUICRESULT=$?
+            ./picohttp_ct -S .. -n -r -x http_corrupt && HTTPRESULT=$?
+            if [[ ${QUICRESULT} == 0 ]] && [[ ${HTTPRESULT} == 0 ]]; then exit 0; fi;
             exit 1
             
       #TODO: reset the test once openssl issue is understood.

.github/workflows/ci-tests.yml

@@ -36,12 +36,11 @@ jobs:
             ./picoquic_ct -S .. -n -r && QUICRESULT=$?
             ./picohttp_ct -S .. -n -r -x http_corrupt && HTTPRESULT=$?
             if [[ ${QUICRESULT} == 0 ]] && [[ ${HTTPRESULT} == 0 ]]; then exit 0; fi;
-            cat /home/runner/work/picoquic/picoquic/build/Testing/Temporary/LastTest.log
             exit 1
-            
-      - name: Run Valgrind
-        run: |
-            sudo apt-get update
-            sudo apt-get install -y valgrind
-            # Remove, until the picotls/openssl issue is resolved.
-            # valgrind -v --error-exitcode=1  --track-origins=yes build/picoquic_ct zero_rtt_many_losses
+      
+      # Remove, until the picotls/openssl issue is resolved. TODO: restore when understood.
+      # - name: Run Valgrind
+      #  run: |
+      #      sudo apt-get update
+      #      sudo apt-get install -y valgrind
+      #      valgrind -v --error-exitcode=1  --track-origins=yes build/picoquic_ct zero_rtt_many_losses