UmeshPatil-1 commented on code in PR #701:
URL: https://github.com/apache/atlas/pull/701#discussion_r3719891858
##########
repository/pom.xml:
##########
@@ -301,6 +310,12 @@
</exclusions>
</dependency>
+ <dependency>
+ <groupId>org.mockito</groupId>
Review Comment:
mockito-core is required for glossary unit tests.In this PR I moved it to
<scope>test</scope> (it was previously declared without scope, placing it on
the compile classpath unnecessarily). The dependency is retained for tests, it
is no longer bundled into the production artifact. All 68 repository glossary
tests pass with this configuration.
--
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]
