mthmulders commented on PR #757: URL: https://github.com/apache/maven/pull/757#issuecomment-1173382409
Thanks for the IT. I've learned about `mvn <pluginPrefix>:<goal>@<id>` today :-) Since there's no PR for the IT, let me ask one question here: I noticed the name of the test methods all start with `test`, but there's no `@Test` annotation. I thought we were on JUnit 4, with JUnit 5 [underway](https://github.com/apache/maven-integration-testing/pull/176). Wouldn't it make more sense to write the tests in JUnit 4 style? Also, if we were on JUnit 5 already, you could even turn it into a parameterised test, but I think that's not possible at the moment. -- 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
