From c4e9c29266eb8656d0573a3aff8b6b2a713861d7 Mon Sep 17 00:00:00 2001 From: Mathieu Baudier Date: Sun, 29 Jun 2008 21:41:28 +0000 Subject: [PATCH] Don't close empty test results Add warnIfAlreadyClosed property git-svn-id: https://svn.argeo.org/slc/trunk@1344 4cfe0d0a-d680-48aa-b62c-e0a02a3f76cc --- .../slc/core/test/tree/TreeTestResult.java | 21 ++++++++++++++++--- 1 file changed, 18 insertions(+), 3 deletions(-) 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 418ab62b4..3e365afe0 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 @@ -35,7 +35,9 @@ public class TreeTestResult implements TestResult, StructureAware, private Date closeDate; - private boolean isClosed = false; + private Boolean isClosed = false; + + private Boolean warnIfAlreadyClosed = true; private String uuid; @@ -111,9 +113,17 @@ public class TreeTestResult implements TestResult, StructureAware, } public void close() { + if (resultParts.size() == 0) { + if (log.isTraceEnabled()) + log.trace("Test Result #" + getUuid() + + " contains no results, no need to close it."); + return; + } + if (isClosed) { - log.warn("Test Result #" + getUuid() - + " already closed. Doing nothing."); + if (warnIfAlreadyClosed) + log.warn("Test Result #" + getUuid() + + " already closed. Doing nothing."); return; } @@ -201,4 +211,9 @@ public class TreeTestResult implements TestResult, StructureAware, public void setAttributes(Map attributes) { this.attributes = attributes; } + + public void setWarnIfAlreadyClosed(Boolean warnIfAlreadyClosed) { + this.warnIfAlreadyClosed = warnIfAlreadyClosed; + } + } -- 2.39.5