From: Mathieu Baudier <mbaudier@argeo.org>
Date: Sun, 29 Jun 2008 20:09:07 +0000 (+0000)
Subject: Make tree test result less restrictive (warn instead of throwing an exception)
X-Git-Tag: argeo-slc-2.1.7~2704
X-Git-Url: http://git.argeo.org/?a=commitdiff_plain;h=c61e5354d4adb565b5ea6c3d97909090a01344b4;p=gpl%2Fargeo-slc.git

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
---

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<TreeSPath>,
 	}
 
 	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<TreeSPath>,
 
 	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) {