JENKINS-13377 duration on history page bugfix

core/src/main/java/hudson/tasks/junit/TestResult.java

@@ -185,6 +185,7 @@ public final class TestResult extends MetaTabulatedResult {
     }
 
     private void add(SuiteResult sr) {
+    	duration += sr.getDuration();
         for (SuiteResult s : suites) {
             // JENKINS-12457: If a testsuite is distributed over multiple files, merge it into a single SuiteResult:
             if(s.getName().equals(sr.getName())  && nullSafeEq(s.getId(),sr.getId())) {
@@ -204,7 +205,6 @@ public final class TestResult extends MetaTabulatedResult {
             }
         }
         suites.add(sr);
-        duration += sr.getDuration();
     }
     
     private boolean strictEq(Object lhs, Object rhs) {

core/src/test/java/hudson/tasks/junit/TestResultTest.java

@@ -125,6 +125,9 @@ public class TestResultTest extends TestCase {
         
         assertEquals("Wrong number of testsuites", 1, testResult.getSuites().size());
         assertEquals("Wrong number of test cases", 2, testResult.getTotalCount());
+        
+        // check duration: 157.980 (TestSuite_a1.xml) and 15.000 (TestSuite_a2.xml) = 172.98 
+        assertEquals("Wrong duration for test result", 172.98, testResult.getDuration(), 0.1);
     }
     
     /**