few updates on logs

[gpl/argeo-slc.git] / runtime / org.argeo.slc.support.activemq / src / main / java / org / argeo / slc / jms / JmsTreeTestResultListener.java

diff --git a/runtime/org.argeo.slc.support.activemq/src/main/java/org/argeo/slc/jms/JmsTreeTestResultListener.java b/runtime/org.argeo.slc.support.activemq/src/main/java/org/argeo/slc/jms/JmsTreeTestResultListener.java
index 6b58bb8a53c32895ef6ce8c201d583a27669ea36..2902b94d4d95de82fe6ebb55ec55f27957ce6c31 100644 (file)
--- a/runtime/org.argeo.slc.support.activemq/src/main/java/org/argeo/slc/jms/JmsTreeTestResultListener.java
+++ b/runtime/org.argeo.slc.support.activemq/src/main/java/org/argeo/slc/jms/JmsTreeTestResultListener.java
@@ -33,6 +33,7 @@ import org.argeo.slc.test.TestResultPart;
 import org.springframework.jms.JmsException;
 import org.springframework.jms.core.JmsTemplate;
 
+/** JMS based tree test result listener implementation. */
 public class JmsTreeTestResultListener implements TreeTestResultListener {
        private final Log log = LogFactory.getLog(getClass());
 
@@ -53,16 +54,16 @@ public class JmsTreeTestResultListener implements TreeTestResultListener {
                                CreateTreeTestResultRequest req = new CreateTreeTestResultRequest(
                                                testResult);
 
-                               if (log.isDebugEnabled())
-                                       log.debug("Send create result request for result "
+                               if (log.isTraceEnabled())
+                                       log.trace("Send create result request for result "
                                                        + testResult.getUuid());
 
                                jmsTemplate.convertAndSend(executionEventDestination, req);
                        } else {
                                ResultPartRequest req = new ResultPartRequest(testResult);
 
-                               if (log.isDebugEnabled())
-                                       log.debug("Send result parts for result "
+                               if (log.isTraceEnabled())
+                                       log.trace("Send result parts for result "
                                                        + testResult.getUuid());
 
                                jmsTemplate.convertAndSend(executionEventDestination, req);
@@ -83,8 +84,8 @@ public class JmsTreeTestResultListener implements TreeTestResultListener {
                                CreateTreeTestResultRequest req = new CreateTreeTestResultRequest(
                                                testResult);
 
-                               if (log.isDebugEnabled())
-                                       log.debug("Send create result request for result "
+                               if (log.isTraceEnabled())
+                                       log.trace("Send onClose create result request for result "
                                                        + testResult.getUuid());
 
                                jmsTemplate.convertAndSend(executionEventDestination, req);
@@ -92,8 +93,8 @@ public class JmsTreeTestResultListener implements TreeTestResultListener {
                                CloseTreeTestResultRequest req = new CloseTreeTestResultRequest(
                                                testResult);
 
-                               if (log.isDebugEnabled())
-                                       log.debug("Send close result request for result "
+                               if (log.isTraceEnabled())
+                                       log.trace("Send close result request for result "
                                                        + testResult.getUuid());
 
                                jmsTemplate.convertAndSend(executionEventDestination, req);
@@ -110,6 +111,9 @@ public class JmsTreeTestResultListener implements TreeTestResultListener {
        }
 
        public void addAttachment(TreeTestResult testResult, Attachment attachment) {
+               if (onlyOnClose)
+                       return;
+
                try {
                        AddTreeTestResultAttachmentRequest req = new AddTreeTestResultAttachmentRequest();
                        req.setResultUuid(testResult.getUuid());
@@ -128,6 +132,7 @@ public class JmsTreeTestResultListener implements TreeTestResultListener {
 
        }
 
+       /** Publishes the test result only when it gets closed. */
        public void setOnlyOnClose(Boolean onlyOnClose) {
                this.onlyOnClose = onlyOnClose;
        }