Tibor17 commented on pull request #431:
URL: https://github.com/apache/maven-surefire/pull/431#issuecomment-1022333756


   @slawekjaranowski
   I found that there are issues on both sides.
   First, our test used [wrong artifact 
parrent](https://github.com/apache/maven-surefire/blob/master/maven-surefire-common/src/test/java/org/apache/maven/plugin/surefire/util/DependenciesScannerTest.java#L106),
 it should be like this:
   `groupId[:artifactId[:type[:classifier][:version]]]`
   Second, the library has two issues. It uses the comparison via `!=` on 
`char[]`, see more specifically `tokens[3] != ANY` and it expects wildcard on 
classifier, see the comment `we only accept 5 tokens if the classifier = '*'`.


-- 
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]


Reply via email to