zstan commented on code in PR #13602:
URL: https://github.com/apache/ignite/pull/13602#discussion_r4104194544
##########
modules/tools/src/main/java/org/apache/ignite/tools/surefire/testsuites/CheckAllTestsInSuites.java:
##########
@@ -113,6 +113,13 @@ private void processSuite(Description suite, Set<String>
suitedClasses,
}
}
+ /** Calcite module tests inherited from legacy junit4 related classes. */
+ private static final Set<String> CALCITE_TESTS = Set.of(
Review Comment:
Yes this is a stub, after common junit5 refactoring - it need to be removed
without an issue cause it exploits junit4 classes which are not compatible with
junit5 and further implementor will be forced to fix it
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
