Skip to content

[Gradle] Add integration tests to KMP for External Android Target test execution and reporting - #7637

Open
Yermukhamed Shakhman (ermawqa-jetbrains) wants to merge 6 commits into
masterfrom
yermukhamed/kmp-external-android-target-integration-tests
Open

[Gradle] Add integration tests to KMP for External Android Target test execution and reporting#7637
Yermukhamed Shakhman (ermawqa-jetbrains) wants to merge 6 commits into
masterfrom
yermukhamed/kmp-external-android-target-integration-tests

Conversation

@ermawqa-jetbrains

Copy link
Copy Markdown
Contributor

Summary

  • Added integration test coverage for External Android Target host test execution, reporting, and device test wiring.
  • Extracted externalAndroidLibraryProject to testProjectsBuildScriptInjectionDsl.kt to share project setup across tests.

Follow-up

  • Migrating other external Android target test suites (AllTestsExternalAndroidTargetIT, AndroidJvmLibDependencyResolutionIT, ComposeCompilerPluginExternalAndroidTargetIT...) to use externalAndroidLibraryProject will be handled in a follow-up refactoring

^KQA-3438

@kodee-bot

kodee-bot Bot commented Aug 21, 2026

Copy link
Copy Markdown

Hi! It looks like there are no references to any YT issues in the commit messages.
If the change is non-functional, feel free to ignore the comment :)
Otherwise, please add them as requested in the guideline.

@kotlin-safemerge

kotlin-safemerge Bot commented Aug 21, 2026

Copy link
Copy Markdown

Code Owners

RuleOwnersApproval
/​libraries/​tools/​kotlin-​gradle-​plugin-​integration-​tests/​
kotlin-build-tools

Egorand 🔒
PR commands for maintainers
CommandDescriptionParameters
/safe-mergeRebase-merges with automatic fixup commit squashing--fixup Autosquash fixup commits (on by default)
/safe-squash-mergeSquash-merges with optional commit title/body override--title Title of the squashed commit
--message Body of the squashed commit
/dry-runRuns the test pipeline with changes rebased on latest master--retry Retry the CI run on failure
/test-publicTriggers the public test suite without rebasing on latest master
/test-privateTriggers the private test suite without rebasing on latest master
/codeownersTriggers code owners check and comment update
/fixupSquashes fixup commits and force pushes the branch
/cancel-coordinatorCancels the merge coordinator currently running for this branch

@Egorand Egor Andreevich (Egorand) left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/final

buildScriptInjection {
kotlinMultiplatform.apply {
targets.withType(KotlinMultiplatformAndroidLibraryTarget::class.java).configureEach { target ->
target.compileSdk = 34

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: let's eventually set this to the latest SDK (37)?

jdkVersion = jdkVersion,
namespace = "org.jetbrains.sample.devicetestwiring",
androidLibraryConfiguration = {
withHostTest {}

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: do we need to configure withHostTest {} in this test case?

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.

2 participants