• Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Minor Minor
    • junit-plugin
    • 1.617

      On the history chart of test results, the duration displayed on the Y axis is not correct, as you can see in the below image:

      In function getDisplayGraph() in History.java file line 114, removing divide by 1000 solves the problem.

      Replacing line
      data.add(((double) o.getDuration()) / (1000), "", new ChartLabel(o) {
      with:
      data.add(((double) o.getDuration()), "", new ChartLabel(o) {
      results in this chart:

        1. Capture_after.PNG
          Capture_after.PNG
          54 kB
        2. Capture_before.PNG
          Capture_before.PNG
          50 kB
        3. testresults.png
          testresults.png
          77 kB

          [JENKINS-29736] History chart duration is wrong

          Stefan Drissen added a comment - - edited

          I was about to report this and then found this issue... here's my screenshot. 18 minutes = 1080 seconds.

          Stefan Drissen added a comment - - edited I was about to report this and then found this issue... here's my screenshot. 18 minutes = 1080 seconds.

          Iosif Gut added a comment -

          The fix mentioned in the description was submitted via pull request: https://github.com/jenkinsci/junit-plugin/pull/61

          Iosif Gut added a comment - The fix mentioned in the description was submitted via pull request: https://github.com/jenkinsci/junit-plugin/pull/61

          Code changed in jenkins
          User: Iosif Gut
          Path:
          src/main/java/hudson/tasks/junit/History.java
          http://jenkins-ci.org/commit/junit-plugin/a88daae579a603c24aba4546b1704462eae32ce1
          Log:
          [FIXED JENKINS-29736] removed devide by 1000

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Iosif Gut Path: src/main/java/hudson/tasks/junit/History.java http://jenkins-ci.org/commit/junit-plugin/a88daae579a603c24aba4546b1704462eae32ce1 Log: [FIXED JENKINS-29736] removed devide by 1000

          Iosif Gut added a comment -

          Verified!

          Iosif Gut added a comment - Verified!

            igut Iosif Gut
            igut Iosif Gut
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

              Created:
              Updated:
              Resolved: