sijie commented on a change in pull request #1135: Specify repo in 
MavenClassLoader
URL: https://github.com/apache/bookkeeper/pull/1135#discussion_r167381007
 
 

 ##########
 File path: 
tests/integration-tests-utils/src/test/java/org/apache/bookkeeper/tests/MavenClassLoaderTest.java
 ##########
 @@ -39,4 +39,9 @@ public void testNoZooKeeperInterference() throws Exception {
                                    
org.apache.zookeeper.KeeperException.NoNodeException.class));
     }
 
+    @Test
+    public void testLoadSpecifyingMavenRepo() throws Exception {
 
 Review comment:
   I am not sure what is the purpose of this test? It seems that we are testing 
the functionality that provides by maven resolver. didn't see much value to do 
this though.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to