android / android/nowinandroid

[Bug]: JVM unit tests are not executed on CI

Open
#1,546 5 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?

Since the recent merge of this PR
- https://github.com/android/nowinandroid/pull/923

, the `:core:common` module is now a pure JVM module.

Unfortunately, the CI currently only runs Android tests:

https://github.com/android/nowinandroid/blob/50b13ecb21138b76e14ff0b2f4cf1dcfda9d0f43/.github/workflows/Build.yaml#L106

This also brought up this issue with missing test dependencies that were not visible on CI builds:
- #1544

### 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

Inspect .github/workflows/Build.yaml around line 106 and check how the :core:common module's tests are currently selected. Run the relevant CI workflow or test configuration, then verify that JVM tests execute and expose missing test dependencies such as those discussed in #1544.

Written by the indexing model from the issue text.

Assessment

Tech stack
github-actions, kotlin
Domain
ci-cd, testing
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.