Last Comment Bug 797711 - Test duration wrong for restart tests (only last module is taking into account)
: Test duration wrong for restart tests (only last module is taking into account)
Status: RESOLVED WONTFIX
:
Product: Testing Graveyard
Classification: Graveyard
Component: Mozmill (show other bugs)
: unspecified
: All All
-- normal
: ---
Assigned To: Nobody; OK to take it and work on it
:
:
Mentors: Henrik Skupin (:whimboo) [away 02/18 - 02/27]
Depends on: 767315
Blocks:
  Show dependency treegraph
 
Reported: 2012-10-04 01:04 PDT by Henrik Skupin (:whimboo) [away 02/18 - 02/27]
Modified: 2016-08-24 09:02 PDT (History)
3 users (show)
See Also:
QA Whiteboard:
Iteration: ---
Points: ---


Attachments

Description User image Henrik Skupin (:whimboo) [away 02/18 - 02/27] 2012-10-04 01:04:07 PDT
As bug 767315 has been shown we don't call endModule() for each of the test modules except the last one. Reason is that we call restart before we actually would reach the teardownModule call.

https://github.com/mozilla/mozmill/blob/master/mozmill/mozmill/extension/resource/modules/frame.js#L716

As the comment states we don't want to do that but in this case we have to figure out a better solution. That means we should take the first modules start time and the last modules end time to calculate the duration of the test.
Comment 1 User image Henrik Skupin (:whimboo) [away 02/18 - 02/27] 2015-01-20 05:45:19 PST
Mozmill will reach its end of life soon. We are currently working on getting all the tests for Firefox ported to Marionette. For status updates please see bug 1080766.

Note You need to log in before you can comment on or make changes to this bug.