DATAMONGO-1645 - Polishing.

Clean up appender and log level after test run. Suppress log output during tests.

Original pull request: #450.

spring-data-mongodb/src/test/java/org/springframework/data/mongodb/core/mapping/event/LoggingEventListenerTests.java

@@ -22,35 +22,58 @@
 import ch.qos.logback.classic.spi.ILoggingEvent;
 import ch.qos.logback.core.read.ListAppender;
 
+import org.junit.After;
 import org.junit.Before;
 import org.junit.Test;
 import org.slf4j.LoggerFactory;
 
 import com.mongodb.BasicDBObject;
 
 /**
+ * Tests for {@link LoggingEventListener}.
+ *
  * @author Christoph Strobl
+ * @author Mark Paluch
  */
 public class LoggingEventListenerTests {
 
-	LoggingEventListener listener;
 	ListAppender<ILoggingEvent> appender;
+	ch.qos.logback.classic.Logger logger;
+	LoggingEventListener listener;
 
 	@Before
 	public void setUp() {
 
+		appender = new ListAppender<ILoggingEvent>();
+
 		// set log level for LoggingEventListener to "info" and set up an appender capturing events.
-		ch.qos.logback.classic.Logger logger = (ch.qos.logback.classic.Logger) LoggerFactory
-				.getLogger(LoggingEventListener.class);
-		logger.setLevel(Level.toLevel("info"));
+		logger = (ch.qos.logback.classic.Logger) LoggerFactory.getLogger(LoggingEventListener.class);
 
-		appender = new ListAppender();
+		logger.setAdditive(false);
+		logger.setLevel(Level.INFO);
 		logger.addAppender(appender);
+
 		appender.start();
 
 		listener = new LoggingEventListener();
 	}
 
+	@After
+	public void tearDown() {
+
+		// cleanup
+		if (logger != null) {
+
+			logger.detachAppender(appender);
+			logger.setAdditive(true);
+			logger.setLevel(null);
+		}
+
+		if (appender != null) {
+			appender.stop();
+		}
+	}
+
 	@Test // DATAMONGO-1645
 	public void shouldSerializeAfterConvertEventCorrectly() {
 
@@ -98,5 +121,4 @@ public void shouldSerializeAfterDeleteEventEventCorrectly() {
 	static class Foo {
 
 	}
-
 }