com.palantir.baseline-idea should set up groovy test source dirs correctly

[iamdanfox]

What happened?

The following snippet is copied and pasted in lots of internal projects:

idea {
    module {
        testSourceDirs += file("src/test/groovy")
    }
}

What did you want to happen?

This should happen automatically if the groovy plugin is applied.

[robert3005]

shouldn't this be handled by gradle itself? All other jvm languages seem to be working ootb

[stale[bot]]

This issue has been automatically marked as stale because it has not been touched in the last 60 days. Please comment if you'd like to keep it open, otherwise it'll be closed in 7 days time.

[robert3005]

sigh, this had just came up on a pr and I looked at gradle and in fact groovy is not automatically handled in https://github.com/gradle/gradle/blob/master/subprojects/ide/src/main/java/org/gradle/plugins/ide/idea/IdeaPlugin.java#L132-L138

[robert3005]

Though https://github.com/gradle/gradle/blob/master/subprojects/plugins/src/main/java/org/gradle/api/plugins/GroovyBasePlugin.java#L112 looks like it should work