Introduce test result attributes instead of root tags

[gpl/argeo-slc.git] / org.argeo.slc.core / src / main / java / org / argeo / slc / core / test / tree / TreeTestResultLogger.java

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 e20614f905f59f3a1fd7e3f34c97eb159c377c47..535abe61b52ba0c38a600b55dbb898703557635c 100644 (file)
--- 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;\r
 import org.apache.commons.logging.LogFactory;\r
 \r
-import org.argeo.slc.core.test.SimpleResultPart;\r
-import org.argeo.slc.core.test.TestResult;\r
 import org.argeo.slc.core.test.TestResultListener;\r
 import org.argeo.slc.core.test.TestResultPart;\r
 import org.argeo.slc.core.test.TestStatus;\r
-import org.argeo.slc.core.test.tree.AsynchronousTreeTestResultListener.PartStruct;\r
 \r
 /**\r
  * Listener logging tree-based test results to the underlying logging system.\r
@@ -20,6 +17,8 @@ public class TreeTestResultLogger implements TestResultListener<TreeTestResult>
 \r
        private static Log log = LogFactory.getLog(TreeTestResultLogger.class);\r
 \r
+       private Boolean logExceptionMessages = false;\r
+\r
        public void resultPartAdded(TreeTestResult testResult,\r
                        TestResultPart testResultPart) {\r
                String msg = testResultPart + " - " + testResult.getUuid() + ":"\r
@@ -29,7 +28,14 @@ public class TreeTestResultLogger implements TestResultListener<TreeTestResult>
                } else if (testResultPart.getStatus().equals(TestStatus.FAILED)) {\r
                        log.warn(msg);\r
                } else if (testResultPart.getStatus().equals(TestStatus.ERROR)) {\r
-                       log.error(msg + "\n" + testResultPart.getExceptionMessage());\r
+                       if (logExceptionMessages)\r
+                               msg = msg + "\n" + testResultPart.getExceptionMessage();\r
+\r
+                       log.error(msg);\r
+\r
+                       if (!logExceptionMessages || log.isDebugEnabled())\r
+                               log.debug(testResultPart.getExceptionMessage());\r
+\r
                } else {\r
                        log.error("Unknow test status: " + msg);\r
                }\r
@@ -39,4 +45,8 @@ public class TreeTestResultLogger implements TestResultListener<TreeTestResult>
                log.info("Test result " + testResult.getUuid() + " closed.");\r
        }\r
 \r
+       public void setLogExceptionMessages(Boolean logExceptionMessages) {\r
+               this.logExceptionMessages = logExceptionMessages;\r
+       }\r
+\r
 }\r