Skip to content

Fix CI APK build and Robolectric test failures - #74

Merged
anchildress1 merged 1 commit into
dependabot/gradle/pkg-e2c21cacf6from
copilot/fix-failing-github-actions
Aug 13, 2026
Merged

Fix CI APK build and Robolectric test failures#74
anchildress1 merged 1 commit into
dependabot/gradle/pkg-e2c21cacf6from
copilot/fix-failing-github-actions

Conversation

Copilot AI commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

The required CI run failed during release-classpath verification because R8 could not resolve a protobuf annotation class. The test job also failed because Robolectric selected unsupported SDK 37.

  • R8: Suppress the missing compile-time protobuf annotation warning.
  • Robolectric: Default app and storage JVM tests to supported SDK 34.
-dontwarn com.google.protobuf.Internal$ProtoMethodMayReturnNull

Copilot AI changed the title Fix failing CI APK build and Robolectric tests Fix CI APK build and Robolectric test failures Aug 4, 2026
Copilot AI requested a review from anchildress1 August 4, 2026 12:30
@anchildress1
anchildress1 marked this pull request as ready for review August 6, 2026 22:01
Copilot AI lite review requested due to automatic review settings August 6, 2026 22:01

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Fixes CI failures by aligning R8 and Robolectric configuration with current dependency/tooling expectations: R8 is instructed to ignore a missing compile-time-only protobuf annotation, and Robolectric is pinned to an SDK level it supports for JVM tests.

Changes:

  • Add an R8 -dontwarn entry for com.google.protobuf.Internal$ProtoMethodMayReturnNull to unblock release-classpath verification.
  • Pin Robolectric SDK to 34 for app JVM tests via robolectric.properties.
  • Pin Robolectric SDK to 34 for core-storage JVM tests via robolectric.properties.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
core-storage/src/test/resources/robolectric.properties Pins Robolectric test SDK to 34 for storage JVM tests.
app/src/test/resources/robolectric.properties Pins Robolectric test SDK to 34 for app JVM tests.
app/proguard-rules.pro Suppresses R8 warning for a compile-time-only protobuf annotation class.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Co-authored-by: anchildress1 <6563688+anchildress1@users.noreply.github.com>
@anchildress1
anchildress1 force-pushed the dependabot/gradle/pkg-e2c21cacf6 branch from c1c0b5b to 3788c92 Compare August 11, 2026 01:34
@anchildress1
anchildress1 force-pushed the copilot/fix-failing-github-actions branch from 41f5854 to 1de8e7c Compare August 11, 2026 01:34
@sonarqubecloud

Copy link
Copy Markdown

@anchildress1
anchildress1 merged commit a95f946 into dependabot/gradle/pkg-e2c21cacf6 Aug 13, 2026
8 checks passed
@anchildress1
anchildress1 deleted the copilot/fix-failing-github-actions branch August 13, 2026 03:19
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.

3 participants