Improve logging

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

diff --git a/runtime/org.argeo.slc.core/src/main/java/org/argeo/slc/core/test/tree/TreeTestResult.java b/runtime/org.argeo.slc.core/src/main/java/org/argeo/slc/core/test/tree/TreeTestResult.java
index dc9de9a5114a3bb7adfecf3a98dd3d4955e43dd3..de27b55f1089f4c926a97f9acefdd809b48a822e 100644 (file)
--- a/runtime/org.argeo.slc.core/src/main/java/org/argeo/slc/core/test/tree/TreeTestResult.java
+++ b/runtime/org.argeo.slc.core/src/main/java/org/argeo/slc/core/test/tree/TreeTestResult.java
@@ -32,30 +32,30 @@ import org.argeo.slc.test.TestRunAware;
  */\r
 public class TreeTestResult implements TestResult, StructureAware<TreeSPath>,\r
                Comparable<TreeTestResult>, AttachmentsEnabled, Serializable {\r
-       private static final long serialVersionUID = 1L;\r
 \r
+       private static final long serialVersionUID = 1L;\r
        private final static Log log = LogFactory.getLog(TreeTestResult.class);\r
 \r
-       private List<TestResultListener<TreeTestResult>> listeners = new Vector<TestResultListener<TreeTestResult>>();\r
-\r
-       private TreeSPath currentPath;\r
-       private TestRun currentTestRun;\r
-\r
-       private Date closeDate;\r
-\r
-       private Boolean isClosed = false;\r
-\r
-       private Boolean warnIfAlreadyClosed = true;\r
-\r
+       // Persistence data\r
        private String uuid = UUID.randomUUID().toString();\r
+       private Date closeDate;\r
 \r
        private SortedMap<TreeSPath, PartSubList> resultParts = new TreeMap<TreeSPath, PartSubList>();\r
        private SortedMap<TreeSPath, StructureElement> elements = new TreeMap<TreeSPath, StructureElement>();\r
        private List<SimpleAttachment> attachments = new ArrayList<SimpleAttachment>();\r
 \r
+       // Headers. Used to accelerate request on a specific test result.\r
        private Map<String, String> attributes = new TreeMap<String, String>();\r
 \r
+       // Runtime Data\r
+       private TreeSPath currentPath;\r
+       private TestRun currentTestRun;\r
+       private Boolean warnIfAlreadyClosed = true;\r
        private Boolean strictChecks = false;\r
+       // TODO is it really necessary closeDate == null ?\r
+       private Boolean isClosed = false;\r
+\r
+       private List<TestResultListener<TreeTestResult>> listeners = new Vector<TestResultListener<TreeTestResult>>();\r
 \r
        /** Sets the list of listeners. */\r
        public void setListeners(List<TestResultListener<TreeTestResult>> listeners) {\r
@@ -69,7 +69,7 @@ public class TreeTestResult implements TestResult, StructureAware<TreeSPath>,
                                                        + " consider changing the scope of this test result:"\r
                                                        + " you are referencing the same stored data with each new call.",\r
                                        null);\r
-               \r
+\r
                if (currentPath == null)\r
                        throw new SlcException("No current path set.");\r
 \r
@@ -129,8 +129,11 @@ public class TreeTestResult implements TestResult, StructureAware<TreeSPath>,
                return resultParts;\r
        }\r
 \r
-       /** Used by ORM systems. */\r
-       void setResultParts(SortedMap<TreeSPath, PartSubList> resultParts) {\r
+       /**\r
+        * Used by ORM systems. \r
+        * Changed to public in order to enable jcr persistence\r
+        */\r
+       public void setResultParts(SortedMap<TreeSPath, PartSubList> resultParts) {\r
                this.resultParts = resultParts;\r
        }\r
 \r