Proposal on ja-heavy-testing

[rachelcarmena]

Goal

This is a pull request on ja-heavy-testing:

• To use kotlin-compile-testing in idea-plugin
• To avoid testing-plugin proposal #63

Details

I think idea-plugin doesn't need to use testing-plugin but kotlin-compile-testing:

[i-walker]

We definitely can use kotlin-compile-testing this PR has the same problem as #63.

Do you know how we can fix the Internal Error that we're getting?
if you run clean :idea-plugin:test you can see it.
It is similar to this: https://intellij-support.jetbrains.com/hc/en-us/community/posts/115000712004-How-to-integrate-KotlinPlugin-s-PSI-classes?page=1#community_comment_115000609590
But the solution is undocumented.

[i-walker]

Thank you @rachelcarmena.
We can keep this as a draft until we find a solution for the internal_error. This doesn't have to happen this week.

[jansorg]

As long as it‘s possible to generate .class files from a code snippet, I‘m all for replacing testing-plugin.

[rachelcarmena]

As long as it‘s possible to generate .class files from a code snippet, I‘m all for replacing testing-plugin.

Great @jansorg! Yes, it's possible to generate .class files from a code snippet.

testing-plugin is using kotlin-compiler-testing to get them.

[ahinchman1]

Hey @rachelcarmena! I was hoping for some clarification - would it be necessary to use the testing-plugin if we wanted to write tests for the IDE or is the plan to separate the testing setup into their respective projects?

[rachelcarmena]

Hey @rachelcarmena! I was hoping for some clarification - would it be necessary to use the testing-plugin if we wanted to write tests for the IDE or is the plan to separate the testing setup into their respective projects?

It seems the tests for the IDE just need the compiled files and that's already provided by kotlin-compile-testing.

testing-plugin is not ready for that need and adds more things that tests for the IDE wouldn't use.

That's the reason why it makes sense to use kotlin-compile-testing for IDE tests.

[jansorg]

@rachelcarmena I've merged your commits into ja-heavy-testing (#57). Thank you for the proposal, this definitely simplifies the compiler integration!

[rachelcarmena]

Thanks to you @jansorg !! Good to know it 👍