You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ofbiz.apache.org by ja...@apache.org on 2014/03/19 15:34:04 UTC

svn commit: r1579240 - /ofbiz/trunk/framework/testtools/src/org/ofbiz/testtools/TestRunContainer.java

Author: jacopoc
Date: Wed Mar 19 14:34:03 2014
New Revision: 1579240

URL: http://svn.apache.org/r1579240
Log:
Minor change to avoid adding to the logs the empty error/failure lists (in case of no errors/failures).

Modified:
    ofbiz/trunk/framework/testtools/src/org/ofbiz/testtools/TestRunContainer.java

Modified: ofbiz/trunk/framework/testtools/src/org/ofbiz/testtools/TestRunContainer.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/testtools/src/org/ofbiz/testtools/TestRunContainer.java?rev=1579240&r1=1579239&r2=1579240&view=diff
==============================================================================
--- ofbiz/trunk/framework/testtools/src/org/ofbiz/testtools/TestRunContainer.java (original)
+++ ofbiz/trunk/framework/testtools/src/org/ofbiz/testtools/TestRunContainer.java Wed Mar 19 14:34:03 2014
@@ -158,7 +158,7 @@ public class TestRunContainer implements
             Debug.logInfo("[JUNIT] Results for test suite: " + suite.getName(), module);
             Debug.logInfo("[JUNIT] Pass: " + results.wasSuccessful() + " | # Tests: " + results.runCount() + " | # Failed: " +
                     results.failureCount() + " # Errors: " + results.errorCount(), module);
-            if (Debug.importantOn()) {
+            if (Debug.importantOn() && !results.wasSuccessful()) {
                 Debug.logInfo("[JUNIT] ----------------------------- ERRORS ----------------------------- [JUNIT]", module);
                 Enumeration<?> err = results.errors();
                 if (!err.hasMoreElements()) {