Make Marionette's xunit output compatible with Jenkins

RESOLVED FIXED in Firefox 18

Status

defect
RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: jgriffin, Assigned: jgriffin)

Tracking

Trunk
mozilla20
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox18 fixed, firefox19 fixed, firefox20 fixed)

Details

Attachments

(1 attachment, 1 obsolete attachment)

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+
https://hg.mozilla.org/integration/mozilla-inbound/rev/73fdd2c4f8f0
Whiteboard: [automation-needed-in-aurora][automation-needed-in-beta]
Target Milestone: --- → mozilla20
https://hg.mozilla.org/mozilla-central/rev/73fdd2c4f8f0
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.