Skip to content

Fix composeResources crash on android 7#3659

Merged
andreia-ferreira merged 17 commits intomasterfrom
andreia/3655/fix-asset-crash
Apr 10, 2026
Merged

Fix composeResources crash on android 7#3659
andreia-ferreira merged 17 commits intomasterfrom
andreia/3655/fix-asset-crash

Conversation

@andreia-ferreira
Copy link
Copy Markdown
Collaborator

@andreia-ferreira andreia-ferreira commented Apr 7, 2026

Fixes #3655

After moving the fonts to the shared :core:ui module, a crash would happen on android 7 java.lang.RuntimeException - Font asset not found composeResources. This is because androidResources.enable = true was missing from that module's build gradle.
This was apparently not required for newer devices because they are more tolerant of missing or differently packaged Android resources at runtime, but on older android versions resource handling is stricter.

This PR adds that missing build.gradle config and updates the CI to run the instrumented tests in parallel with a android 7 device, alongside the current device. This ensures that all new code is compatible with our min SDK without impacting overall runtime since the tests execute concurrently

@shobhitagarwal1612 PTAL?

@codecov
Copy link
Copy Markdown

codecov bot commented Apr 7, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 66.79%. Comparing base (84e146e) to head (aab85a9).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff            @@
##             master    #3659   +/-   ##
=========================================
  Coverage     66.79%   66.79%           
  Complexity     1607     1607           
=========================================
  Files           363      363           
  Lines          8980     8980           
  Branches       1159     1159           
=========================================
  Hits           5998     5998           
  Misses         2333     2333           
  Partials        649      649           
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@andreia-ferreira andreia-ferreira changed the title Fix composeResources crash Fix composeResources crash on android 7 Apr 8, 2026
@andreia-ferreira andreia-ferreira marked this pull request as ready for review April 8, 2026 10:37
@auto-assign auto-assign bot requested a review from gino-m April 8, 2026 10:37
@andreia-ferreira andreia-ferreira requested review from shobhitagarwal1612 and removed request for gino-m April 8, 2026 10:37
@andreia-ferreira andreia-ferreira merged commit 9868ea3 into master Apr 10, 2026
15 of 17 checks passed
@andreia-ferreira andreia-ferreira deleted the andreia/3655/fix-asset-crash branch April 10, 2026 08:36
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.

[CRASH] java.lang.RuntimeException: Font asset not found composeResources on Android 7

2 participants