Skip to content

Add Kotlin unit tests to CI#19

Merged
pgarr merged 2 commits into
masterfrom
add-kotlin-ci
May 25, 2026
Merged

Add Kotlin unit tests to CI#19
pgarr merged 2 commits into
masterfrom
add-kotlin-ci

Conversation

@pgarr
Copy link
Copy Markdown
Owner

@pgarr pgarr commented May 25, 2026

Summary

Adds a kotlin-tests job to .github/workflows/test.yml that runs ./gradlew :app:testDebugUnitTest in parallel with the existing JS test job.

Why it needs npm ci: The React Native Gradle plugin resolves paths by shelling out to node during Gradle's configuration phase (e.g. require('expo/scripts/resolveAppEntry')). Without node_modules present the build fails before any test code runs.

Why JDK 17: Required by the Android Gradle Plugin version used in this project.

Gradle cache: Caches ~/.gradle/caches and ~/.gradle/wrapper keyed on *.gradle* and gradle-wrapper.properties to avoid re-downloading on every run.

Test plan

  • Open a PR and confirm both test and kotlin-tests jobs appear and pass in GitHub Actions

🤖 Generated with Claude Code

pgarr and others added 2 commits May 25, 2026 18:46
Runs ./gradlew :app:testDebugUnitTest in parallel with the existing JS
test job. Needs Node/npm-ci to satisfy the React Native Gradle plugin's
node invocations during configuration, plus JDK 17 and a Gradle cache.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@pgarr pgarr merged commit 97c0bb3 into master May 25, 2026
3 checks passed
@pgarr pgarr deleted the add-kotlin-ci branch May 25, 2026 16:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant