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

Improve Debugger Tests ART Integration #2766

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

alonalbert
Copy link
Contributor

Extract an interface that will allow Android Studio to inject a class that will handle ART VM debugger dependencies.

The interface is agnostic to ART. It should work for any remote VM.

}
}

private fun loadProvider(className: String, testCase: KotlinDescriptorTestCase): RemoteConnectionProvider {
Copy link
Contributor

Choose a reason for hiding this comment

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

Could you please elaborate a bit on why such flexibility is needed?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

We are working on a way to run these tests from our own repository.

As we fix bugs, we expect to have to change the way the ART VM is launched. For example, we may want to add a certain command line option to the VM in order to activate some experimental feature/fix.

Since we can't anticipate what changes we will need, we would like to be able to have our own version of the ArtAttacher so we can iterate faster. When we stabilize it, we will merge it back upstream.

@alonalbert alonalbert force-pushed the IJPL-150161 branch 2 times, most recently from 0ac9e95 to 27cd144 Compare May 23, 2024 15:15
Extract an interface that will allow Android Studio to inject a class that will handle ART VM debugger dependencies.

The interface is agnostic to ART. It should work for any remote VM.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants