Trying another direction for screenshot grabbing

Author: shai-almog

.github/workflows/scripts-android.yml

@@ -37,3 +37,12 @@ jobs:
           target: google_apis
           script: |
             ./scripts/run-android-instrumentation-tests.sh "${{ steps.build-android-app.outputs.gradle_project_dir }}"
+      - name: Upload emulator screenshot
+        if: always()   # still collect it if tests fail
+        uses: actions/upload-artifact@v4
+        with:
+          name: emulator-screenshot
+          path: artifacts/*.png
+          if-no-files-found: warn
+          retention-days: 14
+          compression-level: 6

scripts/build-android-app.sh

@@ -325,12 +325,51 @@ package $PACKAGE_NAME;
 import android.content.Context;
 import android.test.InstrumentationTestCase;
 
+import android.app.Instrumentation;
+import android.os.ParcelFileDescriptor;
+import android.util.Base64;
+
+import java.io.FileInputStream;
+import java.io.IOException;
+import java.io.InputStream;
+
 public class HelloCodenameOneInstrumentedTest extends InstrumentationTestCase {
 
     public void testUseAppContext() {
         Context appContext = getInstrumentation().getTargetContext();
         assertEquals("$PACKAGE_NAME", appContext.getPackageName());
     }
+
+    private static final int CHUNK = 200_000;
+
+    public static void printPngToStdout(Instrumentation inst) {
+      try {
+        ParcelFileDescriptor pfd = inst.getUiAutomation().executeShellCommand("screencap -p");
+        byte[] png;
+        try (InputStream in = new FileInputStream(pfd.getFileDescriptor())) {
+            png = readAll(in);
+        }
+        String b64 = Base64.encodeToString(png, Base64.NO_WRAP);
+        System.out.println("<<CN1_SCREENSHOT_BEGIN>>");
+        for (int i = 0; i < b64.length(); i += CHUNK) {
+            int end = Math.min(i + CHUNK, b64.length());
+            System.out.println(b64.substring(i, end));
+        }
+        System.out.println("<<CN1_SCREENSHOT_END>>");
+        System.out.flush();
+      } catch (IOException err) {
+        err.printStackTrace();
+        throw new RuntimeException(err);
+      }
+    }
+
+    private static byte[] readAll(InputStream in) throws IOException {
+        byte[] buf = new byte[64 * 1024];
+        int n;
+        java.io.ByteArrayOutputStream out = new java.io.ByteArrayOutputStream();
+        while ((n = in.read(buf)) != -1) out.write(buf, 0, n);
+        return out.toByteArray();
+    }
 }
 EOF
 ba_log "Created instrumentation test at $TEST_CLASS"

scripts/run-android-instrumentation-tests.sh

@@ -9,6 +9,11 @@ if [ $# -lt 1 ]; then
   exit 1
 fi
 
+# near the top
+ARTIFACTS_DIR="${ARTIFACTS_DIR:-${GITHUB_WORKSPACE:-$REPO_ROOT}/artifacts}"
+mkdir -p "$ARTIFACTS_DIR"
+TEST_LOG="$ARTIFACTS_DIR/connectedAndroidTest.log"
+
 GRADLE_PROJECT_DIR="$1"
 
 SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
@@ -102,11 +107,58 @@ if [ ! -x "$GRADLE_PROJECT_DIR/gradlew" ]; then
   chmod +x "$GRADLE_PROJECT_DIR/gradlew"
 fi
 
-ra_log "Running instrumentation tests from $GRADLE_PROJECT_DIR"
-ORIGINAL_JAVA_HOME="${JAVA_HOME:-}"; export JAVA_HOME="$JAVA17_HOME"
+set -o pipefail
+
+ra_log "Running instrumentation tests (stdout -> $TEST_LOG; stderr -> terminal)"
+status=0
 (
   cd "$GRADLE_PROJECT_DIR"
-  ./gradlew --no-daemon connectedDebugAndroidTest
-)
-export JAVA_HOME="$ORIGINAL_JAVA_HOME"
+  # stdout goes to tee+file, stderr remains on the terminal
+  ./gradlew --no-daemon --console=plain connectedDebugAndroidTest | tee "$TEST_LOG"
+) || status=$?
+
+# Show the log now (helpful for review even on success)
+echo
+ra_log "==== Begin connectedAndroidTest.log (tail -n 200) ===="
+tail -n 200 "$TEST_LOG" || true
+ra_log "==== End connectedAndroidTest.log ===="
+echo
+
+# Where Gradle puts connected test XML reports
+RESULTS_GLOB="$GRADLE_PROJECT_DIR/app/build/outputs/androidTest-results/connected"
+
+# Pick the newest XML (handles single or multiple files)
+NEWEST_XML="$(find "$RESULTS_GLOB" -type f -name 'TEST-*.xml' -printf '%T@ %p\n' 2>/dev/null \
+  | sort -nr | awk 'NR==1{print $2}')"
+
+ARTIFACTS_DIR="${ARTIFACTS_DIR:-${GITHUB_WORKSPACE:-$REPO_ROOT}/artifacts}"
+mkdir -p "$ARTIFACTS_DIR"
+SCREENSHOT_PATH="$ARTIFACTS_DIR/emulator-screenshot.png"
+
+if [ -n "$NEWEST_XML" ] && [ -f "$NEWEST_XML" ]; then
+  ra_log "Extracting screenshot from $NEWEST_XML -> $SCREENSHOT_PATH"
+
+  # Pull everything between your begin/end markers from <system-out> (handles CDATA and line breaks)
+  if awk '
+    /<<CN1_SCREENSHOT_BEGIN>>/ {on=1; next}
+    /<<CN1_SCREENSHOT_END>>/   {on=0}
+    on
+  ' "$NEWEST_XML" \
+    | tr -d "\r\n" \
+    | base64 -d > "$SCREENSHOT_PATH" 2>/dev/null; then
+
+    if [ -s "$SCREENSHOT_PATH" ]; then
+      ra_log "Screenshot saved: $(ls -lh "$SCREENSHOT_PATH" | awk "{print \$5, \$9}")"
+    else
+      ra_log "Markers found but produced empty file – check that your test emitted a non-empty PNG payload."
+    fi
+  else
+    ra_log "Could not decode base64 from markers in $NEWEST_XML"
+  fi
+else
+  ra_log "No TEST-*.xml found under $RESULTS_GLOB (did the test run?)"
+fi
+
+ra_log "Latest connected test report: ${NEWEST_XML:-<none>}"
+
 ra_log "Instrumentation tests completed successfully"