From c61e5354d4adb565b5ea6c3d97909090a01344b4 Mon Sep 17 00:00:00 2001 From: Mathieu Baudier Date: Sun, 29 Jun 2008 20:09:07 +0000 Subject: [PATCH] Make tree test result less restrictive (warn instead of throwing an exception) git-svn-id: https://svn.argeo.org/slc/trunk@1338 4cfe0d0a-d680-48aa-b62c-e0a02a3f76cc --- .../argeo/slc/core/test/tree/TreeTestResult.java | 13 +++++++++---- 1 file changed, 9 insertions(+), 4 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 781c6eee2..418ab62b4 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 @@ -50,9 +50,12 @@ public class TreeTestResult implements TestResult, StructureAware, } public void addResultPart(TestResultPart part) { - if (currentPath == null) { + if (isClosed) + throw new SlcException("Cannot result parts to a closed result"); + + if (currentPath == null) throw new SlcException("No current path set."); - } + PartSubList subList = resultParts.get(currentPath); if (subList == null) { subList = new PartSubList(); @@ -109,9 +112,11 @@ public class TreeTestResult implements TestResult, StructureAware, public void close() { if (isClosed) { - throw new SlcException("Test Result #" + getUuid() - + " already closed."); + log.warn("Test Result #" + getUuid() + + " already closed. Doing nothing."); + return; } + closeDate = new Date(); synchronized (listeners) { -- 2.39.5