Closed Bug 797711 Opened 8 years ago Closed 5 years ago

Test duration wrong for restart tests (only last module is taking into account)

Categories

(Testing Graveyard :: Mozmill, defect)

Product:
Testing Graveyard
Component:
Mozmill
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED WONTFIX

People

(Reporter: whimboo, Unassigned, Mentored)

References

Details

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.
Whiteboard: [mentor=whimboo][lang=js] → [mentor=whimboo][lang=js][good first bug]
Mentor: hskupin
Whiteboard: [mentor=whimboo][lang=js][good first bug] → [lang=js][good first bug]
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.
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → WONTFIX
Whiteboard: [lang=js][good first bug]
Product: Testing → Testing Graveyard
You need to log in before you can comment on or make changes to this bug.