Skip to content

Conversation

@whyoleg
Copy link
Collaborator

@whyoleg whyoleg commented Sep 16, 2025

This is not required, but because of this, even commonMain in a multiplatform project that has the android plugin applied will have android.jar in the classpath, and so it could theoretically resolve to declarations from there.
(spotted while writing a test for #4258)

@adam-enko, do you have any suggestions on how to test this?

@whyoleg whyoleg requested a review from adam-enko September 16, 2025 14:51
@adam-enko
Copy link
Member

@adam-enko, do you have any suggestions on how to test this?

We want to check that DokkaSourceSet.classpath passed to Dokka generator contains android.jar. This could change based on the AGP version, so it should be an integration test.

So, add an assertion in the AGP integration tests, e.g. dokka-integration-tests/gradle/src/testTemplateProjectAndroid/kotlin/Android0GradleIntegrationTest.kt, that checks the Dokka configuration contains android.jar.

Does that sound doable?

@whyoleg whyoleg self-assigned this Sep 18, 2025
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