Uploaded image for project: 'ngrinder'
  1. ngrinder
  2. NGRINDER-344

Mean Test time graph is wrongly calculated

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 3.0.1
    • Fix Version/s: 3.0.2
    • Component/s: controller
    • Labels:
      None

      Description

      Currently values on the graph is just sum of average mean test time for all tests.

      Actually this is wrong. Mean Test Time in total Statistics is calculated well by grinder using following calculation.

      • sum timedTest / count timedTest

      So If we compare two values they shows different trends.
      Check the attached screenshot

        Activity

        Hide
        junoyoon JunHo Yoon added a comment -

        It happens there are multiple tests in a script.

        Show
        junoyoon JunHo Yoon added a comment - It happens there are multiple tests in a script.
        Hide
        junoyoon JunHo Yoon added a comment -

        Fixed. We need long validation for this fix.

        NGRINDER-344 Fix the wrong calculation.

        • It uses last statistics for the calculation without any calc.
        • Previously, we calc the sum from the each test model but not it comes
          form intervalStatistics directly.
        • We lost the HTTP plugin statistics eventually. CSV will only show
          basic statistics.
        Show
        junoyoon JunHo Yoon added a comment - Fixed. We need long validation for this fix. NGRINDER-344 Fix the wrong calculation. It uses last statistics for the calculation without any calc. Previously, we calc the sum from the each test model but not it comes form intervalStatistics directly. We lost the HTTP plugin statistics eventually. CSV will only show basic statistics.
        Hide
        junoyoon JunHo Yoon added a comment -

        Already done

        Show
        junoyoon JunHo Yoon added a comment - Already done

          People

          • Assignee:
            junoyoon JunHo Yoon
            Reporter:
            junoyoon JunHo Yoon
            CC (Referrer):
            duban, Mavlarn Tuohuti
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: