Configure Android memory (#248)

* Configure Android memory

* Configure max heap for forked JVMs during tests

* Add comment to gradle.properties to make it explicit

* Revert to Xmx2048m

Author: markmur

.github/workflows/ci.yml

@@ -68,7 +68,10 @@ jobs:
       - name: Setup
         uses: ./.github/actions/setup
 
-      - run: yarn test --coverage --testPathPatterns="modules/@shopify/checkout-sheet-kit/tests" --coverageReporters=json-summary
+      - run:
+          yarn test --coverage
+          --testPathPatterns="modules/@shopify/checkout-sheet-kit/tests"
+          --coverageReporters=json-summary
 
       - name: Coverage comment
         if: github.event_name == 'pull_request'
@@ -98,7 +101,7 @@ jobs:
 
       - name: Run Android tests
         env:
-          GRADLE_OPTS: -Xmx6g
+          GRADLE_OPTS: -Xmx2g -XX:MaxMetaspaceSize=512m
         run: |
           echo "STOREFRONT_DOMAIN=\"myshopify.com\"" > sample/.env
           yarn module build

sample/android/build.gradle

@@ -29,3 +29,11 @@ def loadProperties() {
 }
 
 apply plugin: "com.facebook.react.rootproject"
+
+// Configure memory for forked test JVMs to avoid OOMs in CI
+subprojects {
+    tasks.withType(Test).configureEach {
+        maxHeapSize = "1536m"
+        jvmArgs "-XX:MaxMetaspaceSize=512m"
+    }
+}

sample/android/gradle.properties

@@ -7,8 +7,10 @@
 # For more details on how to configure your build environment visit
 # http://www.gradle.org/docs/current/userguide/build_environment.html
 
-# Specifies the JVM arguments used for the daemon process.
-# The setting is particularly useful for tweaking memory settings.
+# IMPORTANT: This config applies ONLY to the Gradle daemon (local development).
+# CI runs tests with `--no-daemon`, so this setting is IGNORED in CI.
+# CI memory is configured in `.github/workflows/ci.yml` (GRADLE_OPTS) and
+# forked test JVM memory is configured in `sample/android/build.gradle` (Test tasks).
 org.gradle.jvmargs=-Xmx2048m -XX:MaxMetaspaceSize=512m