Skip to content

build: apply instrumented test dependencies conditionally#4698

Merged
jamesarich merged 1 commit intomainfrom
chore/test-sources
Mar 3, 2026
Merged

build: apply instrumented test dependencies conditionally#4698
jamesarich merged 1 commit intomainfrom
chore/test-sources

Conversation

@jamesarich
Copy link
Collaborator

Check for the existence of src/androidTest before adding test-related dependencies in convention plugins. This prevents unnecessary dependency resolution for modules without instrumented tests. Additionally, removed unused test dependencies from the firmware feature module.

Check for the existence of `src/androidTest` before adding test-related dependencies in convention plugins. This prevents unnecessary dependency resolution for modules without instrumented tests. Additionally, removed unused test dependencies from the firmware feature module.

Signed-off-by: James Rich <2199651+jamesarich@users.noreply.github.com>
@jamesarich jamesarich merged commit 17dcbed into main Mar 3, 2026
4 checks passed
@jamesarich jamesarich deleted the chore/test-sources branch March 3, 2026 16:29
@codecov
Copy link

codecov bot commented Mar 3, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 7.94%. Comparing base (c234ace) to head (d91315f).
⚠️ Report is 3 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff            @@
##            main   #4698      +/-   ##
========================================
- Coverage   8.12%   7.94%   -0.19%     
========================================
  Files         64      64              
  Lines       2499    2657     +158     
  Branches     289     309      +20     
========================================
+ Hits         203     211       +8     
- Misses      2264    2414     +150     
  Partials      32      32              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

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