Re '@Test' without 'abstract': There's a corner case of a base abstract
class with all test cases. Then 2 or 3 inheriting classes with no
'@Test' annotations bc they just run the base class with different
configs, setups, envs, parametrized runs, etc I've seen some of those.
On 1/11/22 22:54, Mick Semb Wever wrote:
Aha wait a minute I got it :D Yeah. This is not what you invented,
it is already in, sorry.
So, you are saying we should just modify this to cover @Test /
abstract.
I dont know ... interesting.
Lets wait for others to answer.
This thread is rather painful to digest because of the top-posting :'(
I agree that all non-abstract classes that contain a @test annotation
must have a filename ending in "Test.java"
That leaves CI scripts untouched.
How we enforce that I have no opinion about. If it's not easy in
checkstyle, just do it with some bash in the cassandra-test.sh script.
It'll get caught pre-commit either way, and once the rename of
existing files is done the precedence is clearly in place.
