Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-29736

History chart duration is wrong

XMLWordPrintable

    • 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. testresults.png
          testresults.png
          77 kB
        2. Capture_after.PNG
          Capture_after.PNG
          54 kB
        3. Capture_before.PNG
          Capture_before.PNG
          50 kB

          [JENKINS-29736] History chart duration is wrong

          Iosif Gut created issue -
          Iosif Gut made changes -
          Description Original: 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:

          !Capture_before.PNG|thumbnail!

          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:

          !Capture_after.PNG|thumbnail!
          		 New: 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:

          !Capture_before.PNG|thumbnail!

          In function getDisplayGraph() in [History.java|https://github.com/jenkinsci/junit-plugin/blob/master/src/main/java/hudson/tasks/junit/History.java#L114] 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:

          !Capture_after.PNG|thumbnail!

          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.
          Stefan Drissen made changes -
          Attachment New: testresults.png [ 30656 ]
          R. Tyler Croy made changes -
          Workflow Original: JNJira [ 164811 ] New: JNJira + In-Review [ 181697 ]
          Iosif Gut made changes -
          Assignee New: Iosif Gut [ igut ]
          Iosif Gut made changes -
          Status Original: Open [ 1 ] New: In Progress [ 3 ]

          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
          Iosif Gut made changes -
          Resolution New: Fixed [ 1 ]
          Status Original: In Progress [ 3 ] New: Resolved [ 5 ]

          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

          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

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

              Created:
              Updated:
              Resolved: