Merge pull request #11 from allenxwang/master

Enable junit tests logging

build.gradle

@@ -29,6 +29,8 @@ subprojects {
         compile 'commons-collections:commons-collections:3.2.1'
         testCompile 'junit:junit:4.10'
         testCompile 'log4j:log4j:1.2.16'
+        testCompile 'org.slf4j:slf4j-log4j12:1.6.4'
+        
     }   
 }
 

ribbon-core/src/test/java/com/netflix/loadbalancer/SimpleRoundRobinLBTest.java

@@ -27,6 +27,9 @@ import java.util.Map;
 import java.util.Set;
 import java.util.concurrent.ConcurrentHashMap;
 
+import org.apache.log4j.Level;
+import org.apache.log4j.LogManager;
+import org.junit.AfterClass;
 import org.junit.BeforeClass;
 import org.junit.Test;
 
@@ -36,6 +39,7 @@ public class SimpleRoundRobinLBTest {
 	
 	@BeforeClass
 	public static void setup(){
+		LogManager.getRootLogger().setLevel((Level)Level.DEBUG);
 		isAliveMap.put("dummyservice0.netflix.com:8080", Boolean.TRUE);
 		isAliveMap.put("dummyservice1.netflix.com:8080", Boolean.TRUE);
 		isAliveMap.put("dummyservice2.netflix.com:8080", Boolean.TRUE);
@@ -62,6 +66,10 @@ public class SimpleRoundRobinLBTest {
 		System.out.println(lb.getServerList(true));
 	}
 	
+	@AfterClass
+	public static void cleanup() {
+		LogManager.getRootLogger().setLevel((Level)Level.INFO);
+	}
 	/**
 	 * Simulate a single user who should just round robin among the available servers
 	 */

ribbon-core/src/test/resources/log4j.properties

+log4j.rootCategory=INFO,stdout
+log4j.appender.stdout=org.apache.log4j.ConsoleAppender
+log4j.appender.stdout.layout=org.apache.log4j.PatternLayout
+log4j.appender.stdout.layout.ConversionPattern=%d %-5p %C:%L [%t] [%M] %m%n