gradle-integration-test-plugin icon indicating copy to clipboard operation
gradle-integration-test-plugin copied to clipboard

Idea support - integration srcset marked as test source

Open bgalek opened this issue 1 year ago • 2 comments

Since there is a known workaround for marking the integration directory as a test source with kotlin internal classes working (without explicitly using the idea plugin), maybe it's time to support this feature again? :)

https://github.com/sdkotlin/sd-kotlin-talks/commit/a8a6996a65b4da839ffc4cb270fe923174b0fc88

bgalek avatar Aug 30 '24 05:08 bgalek

Thanks for the link, sounds interesting. It requires migrating to test suites first https://github.com/coditory/gradle-integration-test-plugin/issues/126.

I will probably test it in the near future but cannot provide any timeline. Would you like to do it yourself?

pmendelski avatar Sep 01 '24 20:09 pmendelski

Sure thing, I'll be happy to try, but don't wait for me if you find time yourself ;)

bgalek avatar Sep 02 '24 07:09 bgalek

Hey @bgalek, take a look at release 2.0.0. New release integrates with Jvm Test Suites and Jacoco, but look out for some backward incompatibilities.

pmendelski avatar Oct 13 '24 17:10 pmendelski

@pmendelski Nice, but...

src/integration rename to src/integrationTest is going to cause a lot of havoc (at least at my org), since dependabot will update plugin version and build will be passing, just less tests would be run... That can be missed...

I don't feel it's necessary too.

I really don't want to end up with a fork, but migrating over 10k repos will be challenging for me.

bgalek avatar Oct 13 '24 17:10 bgalek

I changed it because having different source name integration and task name integrationTest is cumbersome in JVM test Suites.

The plugin could simply fail builds if src/integration is detected ...but for now I created a walk-around with some mapping so the change is less problematic. https://github.com/coditory/gradle-integration-test-plugin/pull/141

Let me know if that works for you

pmendelski avatar Oct 13 '24 18:10 pmendelski

Released as 2.0.1

pmendelski avatar Oct 13 '24 18:10 pmendelski

Thank you, I think it will be ok now :)

bgalek avatar Oct 13 '24 19:10 bgalek

Cool and you're welcome.

BTW, 10k repos using this plugin and only 61 stars submitted. That's outrageous 😜

pmendelski avatar Oct 13 '24 21:10 pmendelski

@pmendelski I'll take care of this on Wendsday on Hacktoberfest event <3

bgalek avatar Oct 14 '24 08:10 bgalek