X-Git-Url: http://git.argeo.org/?a=blobdiff_plain;f=org.argeo.slc.core%2Fsrc%2Fmain%2Fjava%2Forg%2Fargeo%2Fslc%2Fcore%2Ftest%2Ftree%2FTreeTestResultLogger.java;h=535abe61b52ba0c38a600b55dbb898703557635c;hb=1a4b784250a43f6c7421ab5d8567c3bfbf51e990;hp=667e2e5a911f6612d9f7328c97b40d37e439a3a4;hpb=e705b3804cd22f56dec2ca69db3de83a53ab010a;p=gpl%2Fargeo-slc.git

diff --git a/org.argeo.slc.core/src/main/java/org/argeo/slc/core/test/tree/TreeTestResultLogger.java b/org.argeo.slc.core/src/main/java/org/argeo/slc/core/test/tree/TreeTestResultLogger.java
index 667e2e5a9..535abe61b 100644
--- a/org.argeo.slc.core/src/main/java/org/argeo/slc/core/test/tree/TreeTestResultLogger.java
+++ b/org.argeo.slc.core/src/main/java/org/argeo/slc/core/test/tree/TreeTestResultLogger.java
@@ -3,12 +3,9 @@ package org.argeo.slc.core.test.tree;
 import org.apache.commons.logging.Log;
 import org.apache.commons.logging.LogFactory;
 
-import org.argeo.slc.core.test.SimpleResultPart;
-import org.argeo.slc.core.test.TestResult;
 import org.argeo.slc.core.test.TestResultListener;
 import org.argeo.slc.core.test.TestResultPart;
 import org.argeo.slc.core.test.TestStatus;
-import org.argeo.slc.core.test.tree.AsynchronousTreeTestResultListener.PartStruct;
 
 /**
  * Listener logging tree-based test results to the underlying logging system.
@@ -20,6 +17,8 @@ public class TreeTestResultLogger implements TestResultListener<TreeTestResult>
 
 	private static Log log = LogFactory.getLog(TreeTestResultLogger.class);
 
+	private Boolean logExceptionMessages = false;
+
 	public void resultPartAdded(TreeTestResult testResult,
 			TestResultPart testResultPart) {
 		String msg = testResultPart + " - " + testResult.getUuid() + ":"
@@ -29,13 +28,25 @@ public class TreeTestResultLogger implements TestResultListener<TreeTestResult>
 		} else if (testResultPart.getStatus().equals(TestStatus.FAILED)) {
 			log.warn(msg);
 		} else if (testResultPart.getStatus().equals(TestStatus.ERROR)) {
-			log.error(msg + "\n" + testResultPart.getExceptionMessage());
+			if (logExceptionMessages)
+				msg = msg + "\n" + testResultPart.getExceptionMessage();
+
+			log.error(msg);
+
+			if (!logExceptionMessages || log.isDebugEnabled())
+				log.debug(testResultPart.getExceptionMessage());
+
 		} else {
 			log.error("Unknow test status: " + msg);
 		}
 	}
 
 	public void close(TreeTestResult testResult) {
+		log.info("Test result " + testResult.getUuid() + " closed.");
+	}
+
+	public void setLogExceptionMessages(Boolean logExceptionMessages) {
+		this.logExceptionMessages = logExceptionMessages;
 	}
 
 }