LuciferYang opened a new pull request, #37:
URL: https://github.com/apache/geronimo-xbean/pull/37
Run `mvn clean install`, `JarArchiveTest#testXBEAN337` there will be test
failed as follows:
```
[ERROR] Tests run: 8, Failures: 0, Errors: 1, Skipped: 0, Time elapsed:
0.039 s <<< FAILURE! - in org.apache.xbean.finder.archive.JarArchiveTest
[ERROR] testXBEAN337(org.apache.xbean.finder.archive.JarArchiveTest) Time
elapsed: 0.005 s <<< ERROR!
java.lang.IllegalArgumentException: Please provide 'file:/...' or
'jar:file:/...!/' URL instead of
'http:ftp:jar:file:/tmp/junit8712324922078383147/!JarArchiveTest!-temp!/path
with spaces4573946630928385333.jar!/'
at
org.apache.xbean.finder.archive.JarArchiveTest.testXBEAN337(JarArchiveTest.java:159)
[ERROR] Errors:
[ERROR] JarArchiveTest.testXBEAN337:159 IllegalArgument Please provide
'file:/...' o...
[INFO]
[ERROR] Tests run: 111, Failures: 0, Errors: 1, Skipped: 2
```
https://github.com/apache/geronimo-xbean/blob/e40b760ffe7e36a50c5a524395e84fffc46f07aa/xbean-finder/src/test/java/org/apache/xbean/finder/archive/JarArchiveTest.java#L163
should catch `IllegalArgumentException` thrown by
https://github.com/apache/geronimo-xbean/blob/e40b760ffe7e36a50c5a524395e84fffc46f07aa/xbean-finder/src/main/java/org/apache/xbean/finder/archive/JarArchive.java#L94-L96
instead of `UnsupportedOperationException`.
After this change `mvn clean install` can run successfully
--
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: dev-unsubscr...@geronimo.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
