Simplify TreeSPath

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

diff --git a/org.argeo.slc.core/src/main/java/org/argeo/slc/core/test/tree/TreeTestResult.java b/org.argeo.slc.core/src/main/java/org/argeo/slc/core/test/tree/TreeTestResult.java
index 36233e9942a2d321cb73b39d5d86842638adb275..8da3e7d5a2dde3bd932a5175b592cbac6d3c77a8 100644 (file)
--- a/org.argeo.slc.core/src/main/java/org/argeo/slc/core/test/tree/TreeTestResult.java
+++ b/org.argeo.slc.core/src/main/java/org/argeo/slc/core/test/tree/TreeTestResult.java
@@ -17,9 +17,7 @@ import org.argeo.slc.core.structure.StructureAware;
 import org.argeo.slc.core.structure.StructureElement;\r
 import org.argeo.slc.core.structure.StructureRegistry;\r
 import org.argeo.slc.core.structure.tree.TreeSPath;\r
-import org.argeo.slc.core.test.NumericTRId;\r
 import org.argeo.slc.core.test.TestResult;\r
-import org.argeo.slc.core.test.TestResultId;\r
 import org.argeo.slc.core.test.TestResultListener;\r
 import org.argeo.slc.core.test.TestResultPart;\r
 \r
@@ -30,10 +28,7 @@ import org.argeo.slc.core.test.TestResultPart;
 public class TreeTestResult implements TestResult, StructureAware<TreeSPath>,\r
                SlcExecutionAware {\r
        private Log log = LogFactory.getLog(TreeTestResult.class);\r
-       /** For ORM */\r
-       private Long tid;\r
 \r
-       private NumericTRId testResultId;\r
        private List<TestResultListener> listeners = new Vector<TestResultListener>();\r
 \r
        private TreeSPath currentPath;\r
@@ -44,27 +39,10 @@ public class TreeTestResult implements TestResult, StructureAware<TreeSPath>,
 \r
        private boolean isClosed = false;\r
 \r
-       private SortedMap<TreeSPath, PartSubList> resultParts = new TreeMap<TreeSPath, PartSubList>();\r
-       private SortedMap<TreeSPath, String> pathNames = new TreeMap<TreeSPath, String>();\r
-\r
-       private StructureRegistry<TreeSPath> registry;\r
-\r
-       public TestResultId getTestResultId() {\r
-               return testResultId;\r
-       }\r
-\r
-       /**\r
-        * Use of a <code>NumericTRId</code> is required by Hibernate. <b>It may\r
-        * change in the future.</b>\r
-        */\r
-       public NumericTRId getNumericResultId() {\r
-               return testResultId;\r
-       }\r
+       private String uuid;\r
 \r
-       /** Sets the test result id as a numeric test result id. */\r
-       public void setNumericResultId(NumericTRId testResultId) {\r
-               this.testResultId = testResultId;\r
-       }\r
+       private SortedMap<TreeSPath, PartSubList> resultParts = new TreeMap<TreeSPath, PartSubList>();\r
+       private SortedMap<TreeSPath, StructureElement> elements = new TreeMap<TreeSPath, StructureElement>();\r
 \r
        /** Sets the list of listeners. */\r
        public void setListeners(List<TestResultListener> listeners) {\r
@@ -96,17 +74,21 @@ public class TreeTestResult implements TestResult, StructureAware<TreeSPath>,
                        TreeSPath path) {\r
                if (registry != null) {\r
                        for (TreeSPath p : path.getHierarchyAsList()) {\r
-                               if (!pathNames.containsKey(p)) {\r
+                               if (!elements.containsKey(p)) {\r
                                        StructureElement elem = registry.getElement(p);\r
                                        if (elem != null) {\r
-                                               pathNames.put(p, elem.getLabel());\r
+                                               elements.put(p, elem);\r
                                        }\r
+                               } else {\r
+                                       if (log.isTraceEnabled())\r
+                                               log.trace("An element is already registered for path "\r
+                                                               + p + " and was not updated");\r
                                }\r
+\r
                        }\r
                }\r
 \r
                currentPath = (TreeSPath) path;\r
-               this.registry = registry;\r
        }\r
 \r
        /** Gets the current path. */\r
@@ -126,7 +108,7 @@ public class TreeTestResult implements TestResult, StructureAware<TreeSPath>,
 \r
        public void close() {\r
                if (isClosed) {\r
-                       throw new SlcException("Test Result #" + getTestResultId()\r
+                       throw new SlcException("Test Result #" + getUuid()\r
                                        + " alredy closed.");\r
                }\r
                closeDate = new Date();\r
@@ -139,26 +121,9 @@ public class TreeTestResult implements TestResult, StructureAware<TreeSPath>,
                }\r
                isClosed = true;\r
 \r
-               log.info("Test Result #" + getTestResultId() + " closed.");\r
+               log.info("Test Result #" + getUuid() + " closed.");\r
        }\r
 \r
-       Long getTid() {\r
-               return tid;\r
-       }\r
-\r
-       void setTid(Long tid) {\r
-               this.tid = tid;\r
-       }\r
-\r
-       /** Gets the related registry (can be null). */\r
-       public StructureRegistry<TreeSPath> getRegistry() {\r
-               return registry;\r
-       }\r
-\r
-       /** Sets the related registry. */\r
-       // public void setRegistry(StructureRegistry<TreeSPath> registry) {\r
-       // this.registry = registry;\r
-       // }\r
        public Date getCloseDate() {\r
                return closeDate;\r
        }\r
@@ -176,12 +141,20 @@ public class TreeTestResult implements TestResult, StructureAware<TreeSPath>,
                }\r
        }\r
 \r
-       public SortedMap<TreeSPath, String> getPathNames() {\r
-               return pathNames;\r
+       public SortedMap<TreeSPath, StructureElement> getElements() {\r
+               return elements;\r
+       }\r
+\r
+       public void setElements(SortedMap<TreeSPath, StructureElement> pathNames) {\r
+               this.elements = pathNames;\r
+       }\r
+\r
+       public String getUuid() {\r
+               return uuid;\r
        }\r
 \r
-       public void setPathNames(SortedMap<TreeSPath, String> pathNames) {\r
-               this.pathNames = pathNames;\r
+       public void setUuid(String uuid) {\r
+               this.uuid = uuid;\r
        }\r
 \r
 }\r