X-Git-Url: http://git.argeo.org/?a=blobdiff_plain;f=runtime%2Forg.argeo.slc.server%2Fsrc%2Fmain%2Fjava%2Forg%2Fargeo%2Fslc%2Fservices%2Fimpl%2Ftest%2FTestManagerServiceImpl.java;h=59b7b37db734b8b1976b27895b5663b63919955a;hb=719f374a748e531ae706115252978e06ad72c308;hp=7ccc35d11e24c1e2e20e1f2c698ef0f156fd20a2;hpb=8328dc992e509741b6b9a5036cc7c10089c116f2;p=gpl%2Fargeo-slc.git

diff --git a/runtime/org.argeo.slc.server/src/main/java/org/argeo/slc/services/impl/test/TestManagerServiceImpl.java b/runtime/org.argeo.slc.server/src/main/java/org/argeo/slc/services/impl/test/TestManagerServiceImpl.java
index 7ccc35d11..59b7b37db 100644
--- a/runtime/org.argeo.slc.server/src/main/java/org/argeo/slc/services/impl/test/TestManagerServiceImpl.java
+++ b/runtime/org.argeo.slc.server/src/main/java/org/argeo/slc/services/impl/test/TestManagerServiceImpl.java
@@ -16,7 +16,9 @@ import org.argeo.slc.process.SlcExecution;
 import org.argeo.slc.services.test.TestManagerService;
 import org.argeo.slc.test.TestRunDescriptor;
 
-/** Implementation of complex operations impacting the underlying data. */
+/**
+ * Implementation of complex operations impacting the underlying data.
+ */
 public class TestManagerServiceImpl implements TestManagerService {
 	private Log log = LogFactory.getLog(getClass());
 
@@ -57,9 +59,9 @@ public class TestManagerServiceImpl implements TestManagerService {
 							.getTestResultUuid());
 				}
 			} else {
-				log
-						.trace("ResultUUID="
-								+ testRunDescriptor.getTestResultUuid());
+				if (log.isTraceEnabled())
+					log.trace("ResultUUID="
+							+ testRunDescriptor.getTestResultUuid());
 				addResultToCollection("default", testRunDescriptor
 						.getTestResultUuid());
 			}
@@ -67,6 +69,7 @@ public class TestManagerServiceImpl implements TestManagerService {
 	}
 
 	public void addResultToCollection(String collectionId, String resultUuid) {
+		// TODO: improve collections
 		TreeTestResultCollection ttrc = treeTestResultCollectionDao
 				.getTestResultCollection(collectionId);
 		if (ttrc == null) {
@@ -100,6 +103,11 @@ public class TestManagerServiceImpl implements TestManagerService {
 		treeTestResultDao.create(treeTestResult);
 
 		registerTestRunDescriptor(msg.getTestRunDescriptor());
+
+		// FIXME: temporary hack before better collection management is found
+		if (msg.getTestRunDescriptor() == null) {
+			addResultToCollection("default", treeTestResult.getUuid());
+		}
 	}
 
 	public void addResultPart(ResultPartRequest msg) {