The role to indicate the contents of TestFailure was given to it.

diff --git a/src/ccunit/CCUnitTestRunner.c b/src/ccunit/CCUnitTestRunner.c
index a389574..d99a1b6 100644 (file)
--- a/src/ccunit/CCUnitTestRunner.c
+++ b/src/ccunit/CCUnitTestRunner.c
@@ -112,15 +112,18 @@ static void printFailures (CCUnitTestRunner* runner, CCUnitTestResult* result)
   itor = ccunit_failures (result);
   while ((f = ccunit_nextListIterator (itor)) != NULL)
     {
-      const char* str = f->toString (f);
-      fprintf (runner->ofp, "%s", str);
-      safe_free ((char*)str);
+      fprintf (runner->ofp, "%s:%u: %s:\n\t%s\n",
+              f->file, f->line, f->testCase->desc, f->condstr);
+      if (f->expect || f->actual)
+       fprintf (runner->ofp, "\texpect: %s\n\tactual: %s\n", f->expect, f->actual);
     }
   ccunit_deleteListIterator (itor);
 }
 
 /**
  * Prints failures.
+ * @param runner test runner
+ * @param result test result
  */
 static inline void print (CCUnitTestRunner* runner, CCUnitTestResult* result)
 {
@@ -130,6 +133,9 @@ static inline void print (CCUnitTestRunner* runner, CCUnitTestResult* result)
 
 /**
  * Runs a single test and collects its results.
+ * @param runner test runner
+ * @param suite test suite
+ * @return 0 if succeeded. -1 if error occurd.
  */
 static int doRun (CCUnitTestRunner* runner, CCUnitTestSuite* suite)
 {
@@ -144,7 +150,7 @@ static int doRun (CCUnitTestRunner* runner, CCUnitTestSuite* suite)
   timerclear (&endTime);
   timerclear (&runTime);
   gettimeofday (&startTime, NULL);
-  suite->test.run (&suite->test, runner->result);
+  ccunit_runTestSuite (suite, runner->result);
   gettimeofday (&endTime, NULL);
   timersub (&endTime, &startTime, &runTime);
   fprintf (runner->ofp,
@@ -177,4 +183,9 @@ void ccunit_deleteTestRunner (CCUnitTestRunner* runner)
   free (runner);
 }
 
+inline int ccunit_runTestRunner (CCUnitTestRunner* runner, CCUnitTestSuite* suite)
+{
+  return runner->run (runner, suite);
+}
+
 /** @} */