Please check contains "test" for `skipTestGroups`, not starts with

[Björn Kautler]

Currently, the skipTestGroups only checks for configurations starting with androidTest or test.
But this does not match things like integTest..., systemTest..., functionalTest..., ...
I think for the default recognition it would be more appropriate to check for toLowerCase().contains("test") instead.

To do something like that manually, you need some discouraged work-arounds like in the comments of #22 that reach into the models of all other projects to get all configuration names and then be able to filter on them, or list them all in the root project and hope none get added in subprojects in the future.

[Jeremy Long]

Sorry about the delay, I've had a ton going on outside of my OSS work. Can you take a look at #392 .

Does that work for you?

[Björn Kautler]

Sure, regex check is of course fine too.
I'd probably have done

boolean isTestConfiguration = configuration.name =~ /((^|[a-z0-9_])T|(^|_)t)est([A-Z0-9_]|$)/

instead though. :-)

[Jeremy Long]

My groovy code looks so much like Java... Fixed: be04e39

[Björn Kautler]

Well, I wouldn't use Groovy for a public plugin anyway. :-D

[Jeremy Long]

In hindsight I wouldn't either.