Updated Branches: refs/heads/master [created] 8dbde6de8
[SUREFIRE-909] Wrong elapsed time calculation Fixed with unit test Project: http://git-wip-us.apache.org/repos/asf/maven-surefire/repo Commit: http://git-wip-us.apache.org/repos/asf/maven-surefire/commit/8dbde6de Tree: http://git-wip-us.apache.org/repos/asf/maven-surefire/tree/8dbde6de Diff: http://git-wip-us.apache.org/repos/asf/maven-surefire/diff/8dbde6de Branch: refs/heads/master Commit: 8dbde6de81cddd7335a5768dc22d9c414db6ef86 Parents: 32ed35b Author: Kristian Rosenvold <kristian.rosenv...@gmail.com> Authored: Tue Sep 18 20:34:15 2012 +0200 Committer: Kristian Rosenvold <kristian.rosenv...@gmail.com> Committed: Tue Sep 18 20:35:37 2012 +0200 ---------------------------------------------------------------------- .../plugin/surefire/report/WrappedReportEntry.java | 2 +- .../surefire/report/WrappedReportEntryTest.java | 10 ++++++++++ 2 files changed, 11 insertions(+), 1 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/maven-surefire/blob/8dbde6de/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/report/WrappedReportEntry.java ---------------------------------------------------------------------- diff --git a/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/report/WrappedReportEntry.java b/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/report/WrappedReportEntry.java index cbf1da8..c93c1e1 100644 --- a/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/report/WrappedReportEntry.java +++ b/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/report/WrappedReportEntry.java @@ -147,7 +147,7 @@ public class WrappedReportEntry implements ReportEntry StringBuilder reportContent = new StringBuilder(); reportContent.append( getName() ); reportContent.append( " Time elapsed: " ); - reportContent.append( getElapsed() ); + reportContent.append( elapsedTimeAsString() ); reportContent.append( " sec" ); return reportContent.toString(); http://git-wip-us.apache.org/repos/asf/maven-surefire/blob/8dbde6de/maven-surefire-common/src/test/java/org/apache/maven/plugin/surefire/report/WrappedReportEntryTest.java ---------------------------------------------------------------------- diff --git a/maven-surefire-common/src/test/java/org/apache/maven/plugin/surefire/report/WrappedReportEntryTest.java b/maven-surefire-common/src/test/java/org/apache/maven/plugin/surefire/report/WrappedReportEntryTest.java index 4585937..1229546 100644 --- a/maven-surefire-common/src/test/java/org/apache/maven/plugin/surefire/report/WrappedReportEntryTest.java +++ b/maven-surefire-common/src/test/java/org/apache/maven/plugin/surefire/report/WrappedReportEntryTest.java @@ -57,5 +57,15 @@ public class WrappedReportEntryTest assertEquals( "[0] 1, 2, 3 (testSum)", reportName ); } + public void testElapsed() + throws Exception + { + String category = "[0] 1\u002C 2\u002C 3 (testSum)(surefire.testcase.JunitParamsTest)"; + ReportEntry reportEntry = new SimpleReportEntry( "fud", category ); + WrappedReportEntry wr = new WrappedReportEntry( reportEntry, null, 12, null, null ); + String elapsedTimeSummary = wr.getElapsedTimeSummary(); + assertEquals( "[0] 1, 2, 3 (testSum)(surefire.testcase.JunitParamsTest) Time elapsed: 0.012 sec", elapsedTimeSummary ); + } + }