Use mockito-core instead of mockito-all which is uberjar

google / gwtmockito

Better GWT unit testing

https://google.github.io/gwtmockito

Apache License 2.0

157 stars 51 forks source link

Closed psiroky closed 8 years ago

psiroky commented 9 years ago

mockito-all is uberjar which bundles all transitive deps (hamcrest-core and objenesis) into single jar . This is bad, becuase it disrupts Maven based dependency management. Projects using gwtmockito may require different version of hamcrest-core than the one bundled in mockito-all. That will result in hamcrest-core classes being twice on classpath without any guarantee which one will be loaded first. That leads to very hard to debug issues.

ekuefler commented 8 years ago

Thanks! It's been a long time since I've considered these dependencies; this looks like a good change.

psiroky commented 8 years ago

Thanks for merging!