Improve SystemCall

[gpl/argeo-slc.git] / runtime / org.argeo.slc.server / src / main / java / org / argeo / slc / services / impl / test / TestManagerServiceImpl.java

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 7ccc35d11e24c1e2e20e1f2c698ef0f156fd20a2..59b7b37db734b8b1976b27895b5663b63919955a 100644 (file)
--- 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;\r
 import org.argeo.slc.test.TestRunDescriptor;\r
 \r
-/** Implementation of complex operations impacting the underlying data. */\r
+/**\r
+ * Implementation of complex operations impacting the underlying data.\r
+ */\r
 public class TestManagerServiceImpl implements TestManagerService {\r
        private Log log = LogFactory.getLog(getClass());\r
 \r
@@ -57,9 +59,9 @@ public class TestManagerServiceImpl implements TestManagerService {
                                                        .getTestResultUuid());\r
                                }\r
                        } else {\r
-                               log\r
-                                               .trace("ResultUUID="\r
-                                                               + testRunDescriptor.getTestResultUuid());\r
+                               if (log.isTraceEnabled())\r
+                                       log.trace("ResultUUID="\r
+                                                       + testRunDescriptor.getTestResultUuid());\r
                                addResultToCollection("default", testRunDescriptor\r
                                                .getTestResultUuid());\r
                        }\r
@@ -67,6 +69,7 @@ public class TestManagerServiceImpl implements TestManagerService {
        }\r
 \r
        public void addResultToCollection(String collectionId, String resultUuid) {\r
+               // TODO: improve collections\r
                TreeTestResultCollection ttrc = treeTestResultCollectionDao\r
                                .getTestResultCollection(collectionId);\r
                if (ttrc == null) {\r
@@ -100,6 +103,11 @@ public class TestManagerServiceImpl implements TestManagerService {
                treeTestResultDao.create(treeTestResult);\r
 \r
                registerTestRunDescriptor(msg.getTestRunDescriptor());\r
+\r
+               // FIXME: temporary hack before better collection management is found\r
+               if (msg.getTestRunDescriptor() == null) {\r
+                       addResultToCollection("default", treeTestResult.getUuid());\r
+               }\r
        }\r
 \r
        public void addResultPart(ResultPartRequest msg) {\r