#2131 PR instrumentation tests can fail silently - return error codes -update nightly to have latest emulator runner and screenshots on failure

.github/workflows/nightly.yml

@@ -21,11 +21,18 @@ jobs:
           fetch-depth: 1
 
       - name: run instrumentation tests
-        uses: reactivecircus/android-emulator-runner@v2.0.0
+        uses: ReactiveCircus/android-emulator-runner@v2.8.0
         with:
           api-level: ${{ matrix.api-level }}
           arch: x86_64
-          script: ./gradlew connectedDebugAndroidTest
+          script: bash contrib/instrumentation_nightly.sh
+
+      - name: Upload screenshot result
+        uses: actions/upload-artifact@v1
+        if: failure()
+        with:
+          name: ${{ matrix.api-level }}
+          path: screencap.png
 
   unit_test_and_release:
     runs-on: ubuntu-latest

contrib/instrumentation.sh

@@ -2,6 +2,6 @@
 
 adb logcat -c
 adb logcat *:E -v color &
-./gradlew jacocoInstrumentationTestReport
+./gradlew jacocoInstrumentationTestReport; gradlew_return_code=$?
 adb exec-out screencap -p > screencap.png
-find screencap.png
+gradlew_return_code

contrib/instrumentation_nightly.sh

@@ -0,0 +1,7 @@
+#!/usr/bin/env bash
+
+adb logcat -c
+adb logcat *:E -v color &
+./gradlew connectedDebugAndroidTest; gradlew_return_code=$?
+adb exec-out screencap -p >screencap.png
+gradlew_return_code