[ci/test] exit after printing logs (#36)

* exit after printing logs * move ||true --------- Co-authored-by: grqx_wsl <[email protected]>
Brainicism · Sep 21, 2024 · fddd1eb · fddd1eb
1 parent a4c37f6
commit fddd1eb
Showing 1 changed file with 9 additions and 4 deletions.
diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -53,8 +53,11 @@ jobs:
         shell: bash
         run: |
           exit_code=0
-          script_response=$(./yt-dlp ${{ env.TEST_ARGS }} --extractor-args "youtube:getpot_bgutil_script=server/build/generate_once.js" dQw4w9WgXcQ 2>&1)
-          ret=$?
+
+          {
+            script_response=$(./yt-dlp ${{ env.TEST_ARGS }} --extractor-args "youtube:getpot_bgutil_script=server/build/generate_once.js" dQw4w9WgXcQ 2>&1)
+            ret=$?
+          } || true
 
           echo "::group::Logs from yt-dlp"
           printf "%s\n" "$script_response"
@@ -120,8 +123,10 @@ jobs:
           echo "$PING_RESP" | jq -C
           echo "::endgroup::"
 
-          script_response=$(./yt-dlp ${{ env.TEST_ARGS }} --extractor-args "youtube:getpot_bgutil_baseurl=http://127.0.0.1:${{ env.PORT }}" dQw4w9WgXcQ 2>&1)
-          ret=$?
+          {
+            script_response=$(./yt-dlp ${{ env.TEST_ARGS }} --extractor-args "youtube:getpot_bgutil_baseurl=http://127.0.0.1:${{ env.PORT }}" dQw4w9WgXcQ 2>&1)
+            ret=$?
+          } || true
 
           docker stop bgutil-provider