Switched to androidx testing

Author: shai-almog

scripts/build-android-app.sh

@@ -201,6 +201,7 @@ xmlstarlet sel -N "$NS" -t -c "/mvn:project/mvn:build/mvn:plugins" -n "$ROOT_POM
 [ -f "$APP_DIR/build.sh" ] && chmod +x "$APP_DIR/build.sh"
 
 SETTINGS_FILE="$APP_DIR/common/codenameone_settings.properties"
+echo "codename1.arg.android.useAndroidX=true" >> "$SETTINGS_FILE"
 [ -f "$SETTINGS_FILE" ] || { ba_log "codenameone_settings.properties not found at $SETTINGS_FILE" >&2; exit 1; }
 
 # --- Read settings ---
@@ -266,112 +267,155 @@ if [ -z "$GRADLE_PROJECT_DIR" ]; then
 fi
 
 ba_log "Configuring instrumentation test sources in $GRADLE_PROJECT_DIR"
+
+# Make sure the generated project opts into AndroidX/Jetifier
+GRADLE_PROPS="$GRADLE_PROJECT_DIR/gradle.properties"
+mkdir -p "$(dirname "$GRADLE_PROPS")"
+grep -q '^android.useAndroidX=' "$GRADLE_PROPS" 2>/dev/null || echo 'android.useAndroidX=true' >> "$GRADLE_PROPS"
+grep -q '^android.enableJetifier=' "$GRADLE_PROPS" 2>/dev/null || echo 'android.enableJetifier=true' >> "$GRADLE_PROPS"
+
 APP_BUILD_GRADLE="$GRADLE_PROJECT_DIR/app/build.gradle"
-if [ -f "$APP_BUILD_GRADLE" ]; then
-  python3 - "$APP_BUILD_GRADLE" <<'PYTHON'
-import pathlib
-import re
-import sys
+ROOT_BUILD_GRADLE="$GRADLE_PROJECT_DIR/build.gradle"
+
+ensure_repos_block() {
+  local file="$1"
+  if [ -f "$file" ]; then
+    if ! grep -q 'google()' "$file"; then
+      # add google() to the first repositories { } block, or create one at bottom
+      if grep -q 'repositories[[:space:]]*{' "$file"; then
+        sed -E -i '0,/repositories[[:space:]]*\{/s//repositories {\n        google()\n        mavenCentral()/' "$file"
+      else
+        printf '\nrepositories {\n    google()\n    mavenCentral()\n}\n' >> "$file"
+      fi
+    fi
+    if ! grep -q 'mavenCentral()' "$file"; then
+      sed -E -i '0,/repositories[[:space:]]*\{/s//repositories {\n        google()\n        mavenCentral()/' "$file"
+    fi
+  fi
+}
 
-path = pathlib.Path(sys.argv[1])
-text = path.read_text()
+# Ensure repos in both root and app build.gradle
+ensure_repos_block "$ROOT_BUILD_GRADLE"
+ensure_repos_block "$APP_BUILD_GRADLE"
+
+if [ -f "$APP_BUILD_GRADLE" ]; then
+  python3 - "$APP_BUILD_GRADLE" <<'PY'
+import re, sys, pathlib
+p = pathlib.Path(sys.argv[1])
+t = p.read_text()
+orig = t
 modified = False
 
-if "android.test.InstrumentationTestRunner" not in text:
-    def add_runner(match):
-        prefix = match.group(0)
-        return prefix + "\n        testInstrumentationRunner \"android.test.InstrumentationTestRunner\""
+# Runner -> AndroidX
+if "androidx.test.runner.AndroidJUnitRunner" not in t:
+    t2, n = re.subn(r'testInstrumentationRunner\s*".*?"',
+                    'testInstrumentationRunner "androidx.test.runner.AndroidJUnitRunner"', t)
+    if n == 0:
+        t2, n = re.subn(r'(defaultConfig\s*\{)',
+                        r'\1\n        testInstrumentationRunner "androidx.test.runner.AndroidJUnitRunner"', t, count=1)
+    t = t2
+    modified = True
 
-    new_text, count = re.subn(r"(defaultConfig\s*\{)", add_runner, text, count=1, flags=re.MULTILINE)
-    if count:
-        text = new_text
-        modified = True
-    else:
-        raise SystemExit("defaultConfig block not found while adding instrumentation runner")
-
-libraries = [
-    "useLibrary 'android.test.base'",
-    "useLibrary 'android.test.mock'",
-    "useLibrary 'android.test.runner'",
-]
-
-missing_libraries = [lib for lib in libraries if lib not in text]
-if missing_libraries:
-    match = re.search(r"^(\s*android\s*\{)", text, flags=re.MULTILINE)
-    if not match:
-        raise SystemExit("android block not found while adding instrumentation libraries")
-    line = match.group(1)
-    indent = re.match(r"^(\s*)", line).group(1)
-    insertion = "".join(f"\n{indent}    {lib}" for lib in missing_libraries)
-    text = text[: match.end()] + insertion + text[match.end():]
+# Remove legacy useLibrary lines
+t2, n = re.subn(r'^\s*useLibrary\s+\'android\.test\.(base|mock|runner)\'\s*\n?', '', t, flags=re.M)
+if n:
+    t = t2
     modified = True
 
-if modified:
-    if not text.endswith("\n"):
-        text += "\n"
-    path.write_text(text)
-PYTHON
-  ba_log "Ensured instrumentation runner and libraries are declared"
+# Decide config name: if file uses modern 'implementation/api', use androidTestImplementation; else androidTestCompile
+uses_modern = re.search(r'(?m)^\s*(implementation|api|testImplementation|androidTestImplementation)\b', t) is not None
+conf = "androidTestImplementation" if uses_modern else "androidTestCompile"
+
+# Ensure dependencies block exists
+if not re.search(r'(?ms)^\s*dependencies\s*\{.*?\}', t):
+    t += "\n\ndependencies {\n}\n"
+
+# Inject deps if missing
+def add_dep(body, depcoord):
+    if depcoord in body:
+        return body
+    return re.sub(r'(?ms)(^\s*dependencies\s*\{)', r'\1\n    ' + conf + f' "{depcoord}"', body, count=1)
+
+for dep in [
+    "androidx.test:runner:1.5.2",
+    "androidx.test:core:1.5.0",
+    "androidx.test.services:storage:1.4.2",
+]:
+    t2 = add_dep(t, dep)
+    if t2 != t:
+        t = t2
+        modified = True
+
+if modified and t != orig:
+    if not t.endswith("\n"): t += "\n"
+    p.write_text(t)
+    print(f"Patched build.gradle (deps via {conf})")
+else:
+    print("No build.gradle changes needed")
+PY
+  ba_log "Ensured AndroidX test runner/deps and repositories are declared (legacy-safe)"
 else
   ba_log "Warning: Gradle build file not found at $APP_BUILD_GRADLE; skipping instrumentation dependency configuration" >&2
 fi
 
 TEST_SRC_DIR="$GRADLE_PROJECT_DIR/app/src/androidTest/java/${PACKAGE_PATH}"
 mkdir -p "$TEST_SRC_DIR"
 TEST_CLASS="$TEST_SRC_DIR/HelloCodenameOneInstrumentedTest.java"
-cat >"$TEST_CLASS" <<EOF
-package $PACKAGE_NAME;
-
-import android.content.Context;
-import android.test.InstrumentationTestCase;
+cat >"$TEST_CLASS" <<'EOF'
+package @PACKAGE@;
 
 import android.app.Instrumentation;
 import android.os.ParcelFileDescriptor;
-import android.util.Base64;
 
+import androidx.test.ext.junit.runners.AndroidJUnit4;
+import androidx.test.platform.app.InstrumentationRegistry;
+import androidx.test.services.storage.TestStorage;
+import androidx.test.core.app.ApplicationProvider;
+
+import org.junit.Test;
+import org.junit.runner.RunWith;
+
+import java.io.File;
 import java.io.FileInputStream;
-import java.io.IOException;
+import java.io.FileOutputStream;
 import java.io.InputStream;
+import java.io.ByteArrayOutputStream;
 
-public class HelloCodenameOneInstrumentedTest extends InstrumentationTestCase {
+import static org.junit.Assert.assertEquals;
 
-    public void testUseAppContext() {
-        Context appContext = getInstrumentation().getTargetContext();
-        assertEquals("$PACKAGE_NAME", appContext.getPackageName());
-    }
+@RunWith(AndroidJUnit4.class)
+public class HelloCodenameOneInstrumentedTest {
 
-    private static final int CHUNK = 200_000;
+    @Test
+    public void testUseAppContext_andSaveScreenshot() throws Exception {
+        // Keep your assertion
+        String expectedPkg = "@PACKAGE@";
+        String actualPkg = ApplicationProvider.getApplicationContext().getPackageName();
+        assertEquals(expectedPkg, actualPkg);
 
-    public static void printPngToStdout(Instrumentation inst) {
-      try {
+        // Capture a PNG from the emulator
+        Instrumentation inst = InstrumentationRegistry.getInstrumentation();
         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));
+        byte[] png = readAll(new FileInputStream(pfd.getFileDescriptor()));
+
+        // Write to AndroidX TestStorage (runner copies back to host)
+        File out = new File(TestStorage.getInstance().getOutputDir(), "emulator-screenshot.png");
+        try (FileOutputStream fos = new FileOutputStream(out)) {
+            fos.write(png);
         }
-        System.out.println("<<CN1_SCREENSHOT_END>>");
-        System.out.flush();
-      } catch (IOException err) {
-        err.printStackTrace();
-        throw new RuntimeException(err);
-      }
+        System.out.println("WROTE_TO_TEST_STORAGE:" + out.getAbsolutePath());
     }
 
-    private static byte[] readAll(InputStream in) throws IOException {
+    private static byte[] readAll(InputStream in) throws Exception {
         byte[] buf = new byte[64 * 1024];
         int n;
-        java.io.ByteArrayOutputStream out = new java.io.ByteArrayOutputStream();
+        ByteArrayOutputStream out = new ByteArrayOutputStream();
         while ((n = in.read(buf)) != -1) out.write(buf, 0, n);
         return out.toByteArray();
     }
 }
 EOF
+sed -i "s|@PACKAGE@|$PACKAGE_NAME|g" "$TEST_CLASS"
 ba_log "Created instrumentation test at $TEST_CLASS"
 
 DEFAULT_ANDROID_TEST="$GRADLE_PROJECT_DIR/app/src/androidTest/java/com/example/myapplication2/ExampleInstrumentedTest.java"