From 44bd6b4d4c1c90fb31906f2158c9ad0feed9c64f Mon Sep 17 00:00:00 2001
From: Arjan Molenaar <gaphor@gmail.com>
Date: Sat, 21 Sep 2013 09:29:13 +0200
Subject: [PATCH] Remove e.printStackTrace from fitnesse.reporting

---
 src/fitnesse/reporting/BaseFormatter.java                  | 7 +++++--
 src/fitnesse/reporting/XmlFormatter.java                   | 5 +++--
 .../reporting/history/MostRecentPageHistoryReader.java     | 6 ++++--
 3 files changed, 12 insertions(+), 6 deletions(-)

diff --git a/src/fitnesse/reporting/BaseFormatter.java b/src/fitnesse/reporting/BaseFormatter.java
index 67fc85c40f..f7295723f2 100644
--- a/src/fitnesse/reporting/BaseFormatter.java
+++ b/src/fitnesse/reporting/BaseFormatter.java
@@ -13,8 +13,11 @@
 
 import java.io.Closeable;
 import java.io.IOException;
+import java.util.logging.Level;
+import java.util.logging.Logger;
 
 public abstract class BaseFormatter implements TestSystemListener<WikiTestPage>, Closeable {
+  protected static final Logger LOG = Logger.getLogger("Report");
 
   protected WikiPage page = null;
   protected FitNesseContext context;
@@ -41,8 +44,8 @@ protected WikiPage getPage() {
   public void errorOccurred(Throwable cause) {
     try {
       close();
-    } catch (Exception e) {
-      e.printStackTrace();
+    } catch (IOException e) {
+      LOG.log(Level.WARNING, "Unable to close formatter after error occurred", e);
     }
   }
 
diff --git a/src/fitnesse/reporting/XmlFormatter.java b/src/fitnesse/reporting/XmlFormatter.java
index 065f596b36..fda64eee04 100755
--- a/src/fitnesse/reporting/XmlFormatter.java
+++ b/src/fitnesse/reporting/XmlFormatter.java
@@ -24,6 +24,7 @@
 import java.io.Writer;
 import java.util.ArrayList;
 import java.util.List;
+import java.util.logging.Level;
 
 public class XmlFormatter extends BaseFormatter {
   public interface WriterFactory {
@@ -89,7 +90,7 @@ public void testAssertionVerified(Assertion assertion, TestResult testResult) {
         expectationResult.row = Integer.toString(cell.getRow());
       }
     } catch (Throwable e) {
-      e.printStackTrace();
+      LOG.log(Level.WARNING, "Unable to process assertion " + assertion + " with test result " + testResult, e);
     }
   }
 
@@ -111,7 +112,7 @@ public void testExceptionOccurred(Assertion assertion, ExceptionResult exception
       expectationResult.evaluationMessage = exceptionResult.getMessage();
       expectationResult.status = exceptionResult.getExecutionResult().toString();
     } catch (Throwable e) {
-      e.printStackTrace();
+      LOG.log(Level.WARNING, "Unable to process assertion " + assertion + " with exception result " + exceptionResult, e);
     }
   }
 
diff --git a/src/fitnesse/reporting/history/MostRecentPageHistoryReader.java b/src/fitnesse/reporting/history/MostRecentPageHistoryReader.java
index 393faa2428..7d7cb2ede9 100644
--- a/src/fitnesse/reporting/history/MostRecentPageHistoryReader.java
+++ b/src/fitnesse/reporting/history/MostRecentPageHistoryReader.java
@@ -2,10 +2,13 @@
 
 import java.io.File;
 import java.text.ParseException;
+import java.util.logging.Level;
+import java.util.logging.Logger;
 
 import fitnesse.reporting.TestResultRecord;
 
 public class MostRecentPageHistoryReader extends PageHistoryReader {
+  private final static Logger LOG = Logger.getLogger("Report");
 
   TestResultRecord mostRecentRecord = null;
   File directory = null;
@@ -18,8 +21,7 @@ public TestResultRecord findMostRecentTestRun() {
     try {
       readHistoryFromPageDirectory(directory);
     } catch (ParseException e) {
-      // TODO Auto-generated catch block
-      e.printStackTrace();
+      LOG.log(Level.WARNING, "Unable to read history from page directory", e);
     }
     return mostRecentRecord;
   }