X-Git-Url: http://git.argeo.org/?a=blobdiff_plain;f=org.argeo.slc.core%2Fsrc%2Fmain%2Fjava%2Forg%2Fargeo%2Fslc%2Fcore%2Ftest%2Ftree%2FTreeTestResult.java;h=781c6eee2a1cc29124f4cf4eee44d82d29133633;hb=3e9377296723ae3d8bb8a6efcdaf9828ff2e4139;hp=6d39d210e9c27f17e00c22cd7f8ce76d5532be2d;hpb=0a985ad4d014b575429384ca8f5247d6c7ad7d40;p=gpl%2Fargeo-slc.git 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 6d39d210e..781c6eee2 100644 --- 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 @@ -2,43 +2,36 @@ package org.argeo.slc.core.test.tree; import java.util.Date; import java.util.List; +import java.util.Map; import java.util.SortedMap; import java.util.TreeMap; import java.util.Vector; import org.apache.commons.logging.Log; import org.apache.commons.logging.LogFactory; - import org.argeo.slc.core.SlcException; -import org.argeo.slc.core.process.SlcExecution; -import org.argeo.slc.core.process.SlcExecutionAware; -import org.argeo.slc.core.process.SlcExecutionStep; import org.argeo.slc.core.structure.StructureAware; import org.argeo.slc.core.structure.StructureElement; import org.argeo.slc.core.structure.StructureRegistry; import org.argeo.slc.core.structure.tree.TreeSPath; -import org.argeo.slc.core.test.NumericTRId; import org.argeo.slc.core.test.TestResult; -import org.argeo.slc.core.test.TestResultId; import org.argeo.slc.core.test.TestResultListener; import org.argeo.slc.core.test.TestResultPart; +import org.argeo.slc.core.test.TestRun; +import org.argeo.slc.core.test.TestRunAware; /** * Complex implementation of a test result compatible with a tree based * structure. */ public class TreeTestResult implements TestResult, StructureAware, - SlcExecutionAware { + Comparable { private Log log = LogFactory.getLog(TreeTestResult.class); - /** For ORM */ - private Long tid; - //private NumericTRId testResultId; - private List listeners = new Vector(); + private List> listeners = new Vector>(); private TreeSPath currentPath; - private String currentSlcExecutionUuid; - private String currentSlcExecutionStepUuid; + private TestRun currentTestRun; private Date closeDate; @@ -49,27 +42,10 @@ public class TreeTestResult implements TestResult, StructureAware, private SortedMap resultParts = new TreeMap(); private SortedMap elements = new TreeMap(); - private StructureRegistry registry; - -// public TestResultId getTestResultId() { -// return testResultId; -// } - - /** - * Use of a NumericTRId is required by Hibernate. It may - * change in the future. - */ -// public NumericTRId getNumericResultId() { -// return testResultId; -// } - - /** Sets the test result id as a numeric test result id. */ -// public void setNumericResultId(NumericTRId testResultId) { -// this.testResultId = testResultId; -// } + private Map attributes = new TreeMap(); /** Sets the list of listeners. */ - public void setListeners(List listeners) { + public void setListeners(List> listeners) { this.listeners = listeners; } @@ -80,15 +56,16 @@ public class TreeTestResult implements TestResult, StructureAware, PartSubList subList = resultParts.get(currentPath); if (subList == null) { subList = new PartSubList(); - subList.setSlcExecutionUuid(currentSlcExecutionUuid); - subList.setSlcExecutionStepUuid(currentSlcExecutionStepUuid); resultParts.put(currentPath, subList); } + if (part instanceof TestRunAware && currentTestRun != null) { + ((TestRunAware) part).notifyTestRun(currentTestRun); + } subList.getParts().add(part); // notify listeners synchronized (listeners) { - for (TestResultListener listener : listeners) { + for (TestResultListener listener : listeners) { listener.resultPartAdded(this, part); } } @@ -101,18 +78,18 @@ public class TreeTestResult implements TestResult, StructureAware, if (!elements.containsKey(p)) { StructureElement elem = registry.getElement(p); if (elem != null) { - // elements.put(p, elem.getLabel()); elements.put(p, elem); - } else { - log.warn("An element is already registered for path " - + p); } + } else { + if (log.isTraceEnabled()) + log.trace("An element is already registered for path " + + p + " and was not updated"); } + } } - currentPath = (TreeSPath) path; - this.registry = registry; + currentPath = path; } /** Gets the current path. */ @@ -133,38 +110,22 @@ public class TreeTestResult implements TestResult, StructureAware, public void close() { if (isClosed) { throw new SlcException("Test Result #" + getUuid() - + " alredy closed."); + + " already closed."); } closeDate = new Date(); synchronized (listeners) { - for (TestResultListener listener : listeners) { + for (TestResultListener listener : listeners) { listener.close(this); } listeners.clear(); } isClosed = true; - log.info("Test Result #" + getUuid() + " closed."); + if (log.isTraceEnabled()) + log.trace("Test Result " + getUuid() + " closed."); } - Long getTid() { - return tid; - } - - void setTid(Long tid) { - this.tid = tid; - } - - /** Gets the related registry (can be null). */ - public StructureRegistry getRegistry() { - return registry; - } - - /** Sets the related registry. */ - // public void setRegistry(StructureRegistry registry) { - // this.registry = registry; - // } public Date getCloseDate() { return closeDate; } @@ -174,12 +135,8 @@ public class TreeTestResult implements TestResult, StructureAware, this.closeDate = closeDate; } - public void notifySlcExecution(SlcExecution slcExecution) { - currentSlcExecutionUuid = slcExecution.getUuid(); - SlcExecutionStep step = slcExecution.currentStep(); - if (step != null) { - currentSlcExecutionStepUuid = step.getUuid(); - } + public void notifyTestRun(TestRun testRun) { + currentTestRun = testRun; } public SortedMap getElements() { @@ -198,4 +155,45 @@ public class TreeTestResult implements TestResult, StructureAware, this.uuid = uuid; } + public SortedMap getRelatedElements( + TreeSPath path) { + if (path == null) + throw new SlcException( + "Cannot retrieve element for a null path in result #" + + uuid); + + SortedMap relatedElements = new TreeMap(); + List hierarchy = path.getHierarchyAsList(); + for (TreeSPath currPath : elements.keySet()) { + if (hierarchy.contains(currPath)) { + relatedElements.put(currPath, elements.get(currPath)); + } + } + return relatedElements; + } + + public TestRun getCurrentTestRun() { + return currentTestRun; + } + + public int compareTo(TreeTestResult ttr2) { + TreeTestResult ttr1 = this; + if (ttr1.getCloseDate() != null && ttr2.getCloseDate() != null) { + return -ttr1.getCloseDate().compareTo(ttr2.getCloseDate()); + } else if (ttr1.getCloseDate() != null && ttr2.getCloseDate() == null) { + return 1; + } else if (ttr1.getCloseDate() == null && ttr2.getCloseDate() != null) { + return -1; + } else { + return ttr1.getUuid().compareTo(ttr2.getUuid()); + } + } + + public Map getAttributes() { + return attributes; + } + + public void setAttributes(Map attributes) { + this.attributes = attributes; + } }