[TESTS] Make sure test end with ..Tests

[brwe]

We currently filter test classes by suffix Test or Tests (https://github.com/elasticsearch/elasticsearch/blob/master/pom.xml#L495), but there is no automation in place to guarantee that test classes actually end with this suffix. As a result, some tests did not run on CI without anyone noticing (see for example 600cb88). Would be great if we had some automation that checks that classes that derive from Test classes like ElasticsearchIntegrationTest have the right suffix.

[javanna]

+1 I think we should rather take ElasticsearchTestCase as the base class though.

[martijnvg]

Or if a test class has '@test' then the class name must end with *Test

[javanna]

@martijnvg I'm afraid we don't use the @Test annotation consistently though... we better look also for method names that start with test if we take that approach.

[martijnvg]

Yes, we take this approach we should also look for method names starting with test. I think we either should the test method name consistently or the test annotation, but that is a different discussion.

[s1monw]

thanks so much @brwe for finding this... das glück des tüchtigen...