Closed Bug 813779 Opened 12 years ago Closed 12 years ago

Make Marionette's xunit output compatible with Jenkins

Categories

(Remote Protocol :: Marionette, defect)

defect
Not set
normal

Tracking

(firefox18 fixed, firefox19 fixed, firefox20 fixed)

RESOLVED FIXED
mozilla20
Tracking Status
firefox18 --- fixed
firefox19 --- fixed
firefox20 --- fixed

People

(Reporter: jgriffin, Assigned: jgriffin)

References

Details

Attachments

(1 file, 1 obsolete file)

Assignee: nobody → jgriffin
Attachment #684092 - Flags: review?(dhunt) → review?(dave.hunt)
Comment on attachment 684092 [details] [diff] [review] Make Marionette's xunit output compatible with Jenkins, Review of attachment 684092 [details] [diff] [review]: ----------------------------------------------------------------- ::: testing/marionette/client/marionette/runtests.py @@ +492,4 @@ > > + if result in ['failure', 'error', 'skipped']: > + f = doc.createElement(result) > + f.setAttribute('message', 'test %s' % result) Is it possible to include the elapsed time for the individual tests? If not, can we file a followup bug to add this? @@ +499,5 @@ > doc = dom.Document() > > + testsuite = doc.createElement('testsuite') > + testsuite.setAttribute('name', 'Marionette') > + testsuite.setAttribute('time', str(self.elapsedtime)) Elapsed time needs to be in milliseconds. The current format is: 0:01:14.989096
Attachment #684092 - Flags: review?(dave.hunt) → review-
This puts the time in ms. Let's file a separate bug for adding times of all the individual tests; that's a bit more work.
Attachment #685375 - Flags: review?(dave.hunt)
Attachment #684092 - Attachment is obsolete: true
Comment on attachment 685375 [details] [diff] [review] Make Marionette's xunit output compatible with Jenkins, Review of attachment 685375 [details] [diff] [review]: ----------------------------------------------------------------- Looks great, thanks!
Attachment #685375 - Flags: review?(dave.hunt) → review+
Whiteboard: [automation-needed-in-aurora][automation-needed-in-beta]
Target Milestone: --- → mozilla20
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Product: Testing → Remote Protocol
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: