Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implement launcher for Instrumented Test on iOS #5198

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

ASalavei
Copy link
Collaborator

@ASalavei ASalavei commented Jan 13, 2025

The instrumented-test project allows to write instrumented tests for Compose Multiplatform and run corresponding XCTests with a host application on the iOS simulator. It includes the following modules and projects:
ui-xctest module contains code to convert a set of kotlin.test to XCTest.
ui-instrumented-test module contains test utilities such as test runner and touch simulation, as well as tests themeselve.
launcher project - wrapper around kotlin.test that runs the tests in the XCode environment.

Fixes https://youtrack.jetbrains.com/issue/CMP-5880/Implement-test-UITouch-API-for-internal-touch-testing

Release Notes

Features - iOS

  • Launcher for Instrumented Test on iOS

@ASalavei ASalavei requested review from kropp, Kpotko, mazunin-v-jb and igordmn and removed request for kropp January 13, 2025 18:33
@ASalavei
Copy link
Collaborator Author

ASalavei commented Jan 13, 2025

@igordmn , could you please also confirm that the copyright is correct.
Also, I located project inside the root directory, because I cannot find the better place - maybe, you know one.

@igordmn
Copy link
Collaborator

igordmn commented Jan 13, 2025

could you please also confirm that the copyright is correct.

Looks fine

I located project inside the rood directory,

Also looks fine

Copy link
Member

@MatkovIvan MatkovIvan left a comment

Choose a reason for hiding this comment

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

The code LGTM. Didn't test locally, please wait for another review

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.

4 participants