single-line maestro as test runner executes scripts as individual lines

Author: markushi

.github/workflows/integration-tests-ui-critical.yml

@@ -138,19 +138,14 @@ jobs:
             adb uninstall io.sentry.uitest.android.critical || echo "Already uninstalled (or not found)"
             adb install -r -d "${{env.APK_NAME}}"
 
-            adb shell screenrecord --size 720x1280 --bit-rate 2000000 /sdcard/recording.mp4 &
-            SCREENRECORD_PID=$!
-
-            set +e
-            maestro test "${{env.BASE_PATH}}/maestro" --debug-output "${{env.BASE_PATH}}/maestro-logs"
-            MAESTRO_EXIT_CODE=$?
-            set -e
-
-            kill $SCREENRECORD_PID 2>/dev/null || true
-            wait $SCREENRECORD_PID 2>/dev/null || true
-            adb pull /sdcard/recording.mp4 "${{env.BASE_PATH}}/maestro-logs/recording.mp4"
-
-            exit $MAESTRO_EXIT_CODE
+            adb shell screenrecord --size 720x1280 --bit-rate 2000000 /sdcard/recording.mp4 & \
+              SCREENRECORD_PID=$!; \
+              maestro test "${{env.BASE_PATH}}/maestro" --debug-output "${{env.BASE_PATH}}/maestro-logs" || MAESTRO_EXIT_CODE=$?; \
+              kill $SCREENRECORD_PID 2>/dev/null || true; \
+              wait $SCREENRECORD_PID 2>/dev/null || true; \
+              adb pull /sdcard/recording.mp4 "${{env.BASE_PATH}}/maestro-logs/recording.mp4" || true; \
+              adb shell rm /sdcard/recording.mp4 || true; \
+              exit ${MAESTRO_EXIT_CODE:-0}
 
       - name: Upload Maestro test results
         if: failure()