Updates for issue #477, wrap exceptions which may be thrown in sumTimes and rethrow

ericmaxwell2003 · ericmaxwell2003 · commit f9cad32eff00 · 2013-03-12T14:11:27.000-04:00
diff --git a/core/src/main/java/cucumber/runtime/formatter/JUnitFormatter.java b/core/src/main/java/cucumber/runtime/formatter/JUnitFormatter.java
@@ -137,9 +137,9 @@ private String sumTimes(NodeList testCaseNodes) {
                         Double.parseDouble(testCaseNodes.item(i).getAttributes().getNamedItem("time").getNodeValue());
                 totalDurationSecondsForAllTimes += testCaseTime;
             } catch ( NumberFormatException e ) {
-                e.printStackTrace();
+                throw new CucumberException(e);
             } catch ( NullPointerException e ) {
-                e.printStackTrace();
+                throw new CucumberException(e);
             }
         }
         DecimalFormat nfmt = (DecimalFormat) NumberFormat.getNumberInstance(Locale.US);

Original file line number	Diff line number	Diff line change
`@@ -137,9 +137,9 @@ private String sumTimes(NodeList testCaseNodes) {`
`137`	`137`	`Double.parseDouble(testCaseNodes.item(i).getAttributes().getNamedItem("time").getNodeValue());`
`138`	`138`	`totalDurationSecondsForAllTimes += testCaseTime;`
`139`	`139`	`} catch ( NumberFormatException e ) {`
`140`		`- e.printStackTrace();`
	`140`	`+ throw new CucumberException(e);`
`141`	`141`	`} catch ( NullPointerException e ) {`
`142`		`- e.printStackTrace();`
	`142`	`+ throw new CucumberException(e);`
`143`	`143`	`}`
`144`	`144`	`}`
`145`	`145`	`DecimalFormat nfmt = (DecimalFormat) NumberFormat.getNumberInstance(Locale.US);`