X-Git-Url: http://git.argeo.org/?a=blobdiff_plain;f=org.argeo.slc.core%2Fsrc%2Fmain%2Fjava%2Forg%2Fargeo%2Fslc%2Funit%2Ftest%2Ftree%2FUnitTestTreeUtil.java;h=1862055fa7607fa413fdbd3ead8e0e0e3c8a2431;hb=5978e7e4f44af177846149d60cf5ff6e2d7462bf;hp=05c56acb69394c8892157da8b094fd5e87561d95;hpb=e860e15db1fd6dc797205de434c75a22f6e5f9a4;p=gpl%2Fargeo-slc.git

diff --git a/org.argeo.slc.core/src/main/java/org/argeo/slc/unit/test/tree/UnitTestTreeUtil.java b/org.argeo.slc.core/src/main/java/org/argeo/slc/unit/test/tree/UnitTestTreeUtil.java
index 05c56acb6..1862055fa 100644
--- a/org.argeo.slc.core/src/main/java/org/argeo/slc/unit/test/tree/UnitTestTreeUtil.java
+++ b/org.argeo.slc.core/src/main/java/org/argeo/slc/unit/test/tree/UnitTestTreeUtil.java
@@ -1,9 +1,13 @@
 package org.argeo.slc.unit.test.tree;
 
+import org.apache.commons.logging.Log;
+import org.apache.commons.logging.LogFactory;
+
 import static junit.framework.Assert.assertEquals;
 import static junit.framework.Assert.assertNotNull;
 import static junit.framework.Assert.assertNull;
 import static junit.framework.Assert.fail;
+import static org.argeo.slc.unit.UnitUtils.assertDateSec;
 
 import org.argeo.slc.core.structure.SimpleSElement;
 import org.argeo.slc.core.structure.tree.TreeSPath;
@@ -14,10 +18,12 @@ import org.argeo.slc.core.test.tree.TreeTestResult;
 
 /** Utilities for unit tests. */
 public class UnitTestTreeUtil {
+	private final static Log log = LogFactory.getLog(UnitTestTreeUtil.class);
+
 	public static void assertTreeTestResult(TreeTestResult expected,
 			TreeTestResult reached) {
-		assertEquals(expected.getTestResultId(), reached.getTestResultId());
-		assertEquals(expected.getCloseDate(), reached.getCloseDate());
+		assertEquals(expected.getUuid(), reached.getUuid());
+		assertDateSec(expected.getCloseDate(), reached.getCloseDate());
 
 		assertEquals(expected.getResultParts().size(), reached.getResultParts()
 				.size());
@@ -60,20 +66,6 @@ public class UnitTestTreeUtil {
 
 	public static void assertPartSubList(PartSubList lstExpected,
 			PartSubList lstReached) {
-		if (lstExpected.getSlcExecutionUuid() == null) {
-			assertNull(lstReached.getSlcExecutionUuid());
-		} else {
-			assertEquals(lstExpected.getSlcExecutionUuid(), lstReached
-					.getSlcExecutionUuid());
-		}
-
-		if (lstExpected.getSlcExecutionStepUuid() == null) {
-			assertNull(lstReached.getSlcExecutionStepUuid());
-		} else {
-			assertEquals(lstExpected.getSlcExecutionStepUuid(), lstReached
-					.getSlcExecutionStepUuid());
-		}
-
 		assertEquals(lstExpected.getParts().size(), lstReached.getParts()
 				.size());
 		for (int i = 0; i < lstExpected.getParts().size(); i++) {
@@ -82,15 +74,10 @@ public class UnitTestTreeUtil {
 		}
 	}
 
-	/**
-	 * Assert one part of a tree test result.
-	 * 
-	 * @deprecated use {@link #assertPart(TestResultPart, TestResultPart)}
-	 *             instead
-	 */
+	/** Asserts one part of a tree test result */
 	public static void assertPart(TreeTestResult testResult, String pathStr,
 			int index, Integer status, String message) {
-		TreeSPath path = TreeSPath.parseToCreatePath(pathStr);
+		TreeSPath path = new TreeSPath(pathStr);
 		PartSubList list = testResult.getResultParts().get(path);
 		if (list == null) {
 			fail("No result for path " + path);
@@ -100,25 +87,58 @@ public class UnitTestTreeUtil {
 			fail("Not enough parts.");
 		}
 		SimpleResultPart part = (SimpleResultPart) list.getParts().get(index);
-		assertPart(part, status, message, null);
+		assertPart(part, status, message, null, part.getTestRunUuid(), true);
 	}
 
 	public static void assertPart(TestResultPart expected,
 			TestResultPart reached) {
+		String expectedTestRunUuid = null;
+		if (expected instanceof SimpleResultPart) {
+			expectedTestRunUuid = ((SimpleResultPart) expected)
+					.getTestRunUuid();
+		}
+
 		assertPart(reached, expected.getStatus(), expected.getMessage(),
-				expected.getException());
+				expected.getExceptionMessage(), expectedTestRunUuid, false);
 	}
 
 	/** Assert one part of a tree test result. */
 	private static void assertPart(TestResultPart part, Integer status,
-			String message, Exception exception) {
+			String message, String exceptionDescription,
+			String expectedTestRunUuid, boolean skipExceptionMessage) {
 		assertEquals(status, part.getStatus());
+
+		if (log.isTraceEnabled()) {
+			log.trace("Expected message:" + message);
+			log.trace("Reached message:" + part.getMessage());
+		}
 		assertEquals(message, part.getMessage());
-		if (exception == null) {
-			assertNull(part.getException());
+		if (!skipExceptionMessage) {
+			if (exceptionDescription == null) {
+				assertNull(part.getExceptionMessage());
+			} else {
+				if (log.isTraceEnabled()) {
+					log.trace("Expected exception message:"
+							+ exceptionDescription);
+					log.trace("Reached exception message:"
+							+ part.getExceptionMessage());
+				}
+
+				assertEquals(exceptionDescription, part.getExceptionMessage());
+			}
+		}
+
+		if (expectedTestRunUuid != null) {
+			SimpleResultPart reachedPart = (SimpleResultPart) part;
+			assertNotNull(reachedPart.getTestRunUuid());
+			assertEquals(expectedTestRunUuid, reachedPart.getTestRunUuid());
 		} else {
-			assertEquals(exception, part.getException());
+			if (part instanceof SimpleResultPart) {
+				assertNull(((SimpleResultPart) part).getTestRunUuid());
+			}
+
 		}
+
 	}
 
 	/** Makes sure this is a singleton */