Make Marionette's xunit output compatible with Jenkins

RESOLVED FIXED in Firefox 18

Status

RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: jgriffin, Assigned: jgriffin)

Tracking

Trunk
mozilla20
Points:
---

Firefox Tracking Flags

(firefox18 fixed, firefox19 fixed, firefox20 fixed)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Comment 1

6 years ago
Created attachment 684092 [details] [diff] [review]
Make Marionette's xunit output compatible with Jenkins,

This makes output consistent with http://qa-selenium.mv.mozilla.com:8080/job/mozilla.com.trunk/2912/artifact/results.xml
Attachment #684092 - Flags: review?(dhunt)
(Assignee)

Updated

6 years ago
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-
(Assignee)

Comment 3

6 years ago
Created attachment 685375 [details] [diff] [review]
Make Marionette's xunit output compatible with Jenkins,

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)
(Assignee)

Updated

6 years ago
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+
(Assignee)

Comment 5

6 years ago
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
Last Resolved: 6 years ago
Resolution: --- → FIXED
https://hg.mozilla.org/releases/mozilla-aurora/rev/5a057c06e172
https://hg.mozilla.org/releases/mozilla-beta/rev/c9d0b3dabf0a
status-firefox18: --- → fixed
status-firefox19: --- → fixed
status-firefox20: --- → fixed
Whiteboard: [automation-needed-in-aurora][automation-needed-in-beta]
You need to log in before you can comment on or make changes to this bug.