Details
-
Bug
-
Status: Closed
-
Major
-
Resolution: Fixed
-
None
-
None
-
None
Description
When running tests in pre-commit CI, the test parser will only identify test classes start with 'Test'
But when running using `mvn test`, it will also parse classes such as '*Test.java':
so for:
it will also be included in the test, for example:
This is because that class TrustDomainAuthenticationTest is actually a parent class that does not have the parameters for init. The actual tests are its children classes like the rest in https://builds.apache.org/view/H-L/view/Hive/job/Hive-linux-ARM-trunk/23/testReport/junit/org.apache.hive.service.auth/
they will pass actuall parameters for init: https://github.com/apache/hive/blob/d2163cbfb8bacf859fa8572e24c8533bb2dcb0f3/itests/hive-unit/src/test/java/org/apache/hive/service/auth/TestTrustDomainAuthenticationBinary.java#L26
we can make this class abstract so that it won't be included when running 'mvn test', like https://github.com/apache/hive/blob/d2163cbfb8bacf859fa8572e24c8533bb2dcb0f3/itests/hive-unit/src/test/java/org/apache/hive/jdbc/AbstractJdbcTriggersTest.java#L54