X-Git-Url: http://git.argeo.org/?a=blobdiff_plain;f=runtime%2Forg.argeo.slc.support.activemq%2Fsrc%2Fmain%2Fjava%2Forg%2Fargeo%2Fslc%2Fjms%2FJmsTreeTestResultListener.java;h=2902b94d4d95de82fe6ebb55ec55f27957ce6c31;hb=856754d1c9381f2734f3583aba037efbbaaeab1d;hp=7807b4219655c7152939b2bd00858b97b27c631f;hpb=719f374a748e531ae706115252978e06ad72c308;p=gpl%2Fargeo-slc.git

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 7807b4219..2902b94d4 100644
--- 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
@@ -1,3 +1,19 @@
+/*
+ * Copyright (C) 2010 Mathieu Baudier <mbaudier@argeo.org>
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *         http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
 package org.argeo.slc.jms;
 
 import javax.jms.Destination;
@@ -17,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());
 
@@ -37,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);
@@ -67,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);
@@ -76,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);
@@ -94,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());
@@ -112,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;
 	}