Closed
Bug 813779
Opened 12 years ago
Closed 12 years ago
Make Marionette's xunit output compatible with Jenkins
Categories
(Remote Protocol :: Marionette, defect)
Remote Protocol
Marionette
Tracking
(firefox18 fixed, firefox19 fixed, firefox20 fixed)
RESOLVED
FIXED
mozilla20
People
(Reporter: jgriffin, Assigned: jgriffin)
References
Details
Attachments
(1 file, 1 obsolete file)
6.28 KB,
patch
|
davehunt
:
review+
|
Details | Diff | Splinter Review |
An example file which does work with jenkins is:
http://qa-selenium.mv.mozilla.com:8080/job/mozilla.com.trunk/2912/artifact/results.xml
Mozmill code which generates this is:
https://github.com/whimboo/mozmill-automation/blob/master/mozmill_automation/reports.py#L98
Assignee | ||
Comment 1•12 years ago
|
||
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•12 years ago
|
Assignee: nobody → jgriffin
Updated•12 years ago
|
Attachment #684092 -
Flags: review?(dhunt) → review?(dave.hunt)
Comment 2•12 years ago
|
||
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•12 years ago
|
||
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•12 years ago
|
Attachment #684092 -
Attachment is obsolete: true
Comment 4•12 years ago
|
||
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•12 years ago
|
||
Whiteboard: [automation-needed-in-aurora][automation-needed-in-beta]
Target Milestone: --- → mozilla20
Comment 6•12 years ago
|
||
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Comment 7•12 years ago
|
||
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]
Updated•2 years ago
|
Product: Testing → Remote Protocol
You need to log in
before you can comment on or make changes to this bug.
Description
•