Now using loggers for all output to reduce test noise

src/main/java/se/citerus/dddsample/application/util/SampleDataGenerator.java

@@ -1,5 +1,7 @@
 package se.citerus.dddsample.application.util;
 
+import org.apache.commons.logging.Log;
+import org.apache.commons.logging.LogFactory;
 import org.hibernate.SessionFactory;
 import org.hibernate.Session;
 import org.springframework.beans.factory.BeanFactoryUtils;
@@ -33,6 +35,7 @@ import java.util.Date;
  * Provides sample data.
  */
 public class SampleDataGenerator implements ServletContextListener {
+  private static final Log logger = LogFactory.getLog(SampleDataGenerator.class);
 
   private static final Timestamp base; 
   static {
@@ -202,7 +205,7 @@ public class SampleDataGenerator implements ServletContextListener {
   }
 
   public static void loadHibernateData(TransactionTemplate tt, final SessionFactory sf, final HandlingEventFactory handlingEventFactory, final HandlingEventRepository handlingEventRepository) {
-    System.out.println("*** Loading Hibernate data ***");
+    logger.info("*** Loading Hibernate data ***");
     tt.execute(new TransactionCallbackWithoutResult() {
       @Override
       protected void doInTransactionWithoutResult(TransactionStatus status) {

src/test/java/se/citerus/dddsample/infrastructure/messaging/stub/SynchronousApplicationEventsStub.java

@@ -1,5 +1,7 @@
 package se.citerus.dddsample.infrastructure.messaging.stub;
 
+import org.apache.commons.logging.Log;
+import org.apache.commons.logging.LogFactory;
 import se.citerus.dddsample.application.ApplicationEvents;
 import se.citerus.dddsample.application.CargoInspectionService;
 import se.citerus.dddsample.domain.model.cargo.Cargo;
@@ -9,6 +11,7 @@ import se.citerus.dddsample.interfaces.handling.HandlingEventRegistrationAttempt
 public class SynchronousApplicationEventsStub implements ApplicationEvents {
 
   CargoInspectionService cargoInspectionService;
+  private final Log logger = LogFactory.getLog(getClass());
 
   public void setCargoInspectionService(CargoInspectionService cargoInspectionService) {
     this.cargoInspectionService = cargoInspectionService;
@@ -16,22 +19,22 @@ public class SynchronousApplicationEventsStub implements ApplicationEvents {
 
   @Override
   public void cargoWasHandled(HandlingEvent event) {
-    System.out.println("EVENT: cargo was handled: " + event);
+    logger.info("EVENT: cargo was handled: " + event);
     cargoInspectionService.inspectCargo(event.cargo().trackingId());
   }
 
   @Override
   public void cargoWasMisdirected(Cargo cargo) {
-    System.out.println("EVENT: cargo was misdirected");
+    logger.info("EVENT: cargo was misdirected");
   }
 
   @Override
   public void cargoHasArrived(Cargo cargo) {
-    System.out.println("EVENT: cargo has arrived: " + cargo.trackingId().idString());
+    logger.info("EVENT: cargo has arrived: " + cargo.trackingId().idString());
   }
 
   @Override
   public void receivedHandlingEventRegistrationAttempt(HandlingEventRegistrationAttempt attempt) {
-    System.out.println("EVENT: received handling event registration attempt");
+    logger.info("EVENT: received handling event registration attempt");
   }
 }

src/test/resources/log4j.properties

@@ -1,4 +1,4 @@
-log4j.rootLogger=info, stdout
+log4j.rootLogger=warn, stdout
 
 log4j.appender.stdout=org.apache.log4j.ConsoleAppender
 log4j.appender.stdout.layout=org.apache.log4j.PatternLayout
@@ -6,8 +6,8 @@ log4j.appender.stdout.layout.ConversionPattern=%d %p [%c] - %m%n
 
 log4j.logger.net.sf.ehcache.config.ConfigurationFactory=error
 
-log4j.logger.se.citerus.dddsample=debug
-log4j.logger.org.hibernate.SQL=debug
+#log4j.logger.se.citerus.dddsample=info
+#log4j.logger.org.hibernate.SQL=info
 #log4j.logger.org.hibernate.tool.hbm2ddl.SchemaExport=debug
 
 #log4j.logger.org.springframework.orm=debug