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

See how we can cover git-machete plugin tests with ide-probe #19

Open
lukaszwawrzyk opened this issue Jul 31, 2020 · 0 comments
Open

See how we can cover git-machete plugin tests with ide-probe #19

lukaszwawrzyk opened this issue Jul 31, 2020 · 0 comments

Comments

@lukaszwawrzyk
Copy link
Contributor

Git machete intellij plugin has some tests here: https://github.com/VirtusLab/git-machete-intellij-plugin/tree/develop/uiTests that use remote robot directly. This is nice set of scenarios we didn't cover so far

I think the ultimate goal here is to replace robot tests with ide-probe tests and after this change probe tests should be easier to write, maintain, setup etc.

Generally probe will help with accessing intellij api through javascript put inside strings, need for reflection due to different classloader, reimplementing waiting for background tasks, opening projects and so on.

If we want to invoke internal intellij methods and have tests, we need to setup 4 modules (api, intellij plugin, driver, tests) vs. just tests module needed for robot. We have a way to do this with sbt (which is not perfect, we have hacks for sbt-intellij-plugin to be able to build more than one plugin in one project, yet it still is imperfect and can cause problems), also most of intellij plugins will use gradle rather than sbt. Perhaps we may want to prepare a drop in seed project with the 4 modules I mentioned with sbt and CI (similar what was done in pants) or better, figure out how to setup such layout with gradle.

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

No branches or pull requests

1 participant