android / android/nowinandroid

Should be divide testOptions from compose plugin.

Open
#1,443 0 comments 0 reactions 0 assignees View on GitHub
bug
Dominant language
Kotlin
Stars
21.8k
Forks
4.6k
Avg merge
19h 20m
Merged PRs (30d)
2

Description

### Is there an existing issue for this?

- [X] I have searched the existing issues

### Is there a StackOverflow question about this issue?

- [X] I have searched StackOverflow

### What happened?

I don't think the testOptions is the parts of Compose plugin, but it is Robolectric, and some test which needs access android resource.

https://github.com/android/nowinandroid/blob/b85cf7d4766ca84a424700f86a96f5eab0a3cfbd/build-logic/convention/src/main/kotlin/com/google/samples/apps/nowinandroid/AndroidCompose.kt#L48-L53

### Relevant logcat output

_No response_

### Code of Conduct

- [X] I agree to follow this project's Code of Conduct

Contributor guide

Open the contributing guide

Research direction

Start in build-logic/convention/src/main/kotlin/com/google/samples/apps/nowinandroid/AndroidCompose.kt at lines 48-53. Read how testOptions is configured and how the Compose plugin is organized, then inspect nearby convention plugins to determine the intended separation. Done means testOptions is no longer part of the Compose plugin while tests requiring Android resources continue to work.

Written by the indexing model from the issue text.

Assessment

Tech stack
kotlin
Domain
build-system
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.