Tibor17 commented on a change in pull request #487:
URL: https://github.com/apache/maven-surefire/pull/487#discussion_r824589850
##########
File path:
surefire-api/src/main/java/org/apache/maven/surefire/api/testset/TestListResolver.java
##########
@@ -195,13 +195,13 @@ public boolean shouldRun( Class<?> testClass, String
methodName )
/**
* Returns {@code true} if satisfies {@code testClassFile} and {@code
methodName} filter.
*
- * @param testClassFile format must be e.g. "my/package/MyTest.class"
including class extension; or null
- * @param methodName real test-method name; or null
+ * @param containerName format must be e.g. "my/package/MyTest.class"
including class extension; or null
Review comment:
Don't change this class, it has nothing to do with what you are aiming
for. Rather do not make any additional refactoring because it is very hard to
separate reasonable and non-reasonable changes. I guess you wanted to make some
specific improvement in JUnit5 provider. Do not press CTRL+L because there is
again unnecessary changes.
--
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: issues-unsubscr...@maven.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
