This is already implemented in SimpleTest's test-run.sh (cf. #602332: XML output option for use with Hudson / JUnit) but it would be nice to have to same option using Drush's test-run.

CommentFileSizeAuthor
#12 junit.xml_.patch4.18 KBAnonymous (not verified)
#3 1109408-drush-test-xml.patch3.89 KBmsonnabaum

Comments

moshe weitzman’s picture

moshe weitzman commented
Status: Active » Closed (duplicate)
moshe weitzman’s picture

moshe weitzman commented
Status: Closed (duplicate) » Active

I see, You want command specific XML. Yeah, its inevitable. Reopening. I won't be working on this though.

msonnabaum’s picture

msonnabaum commented
Status: Active » Needs review
StatusFileSize
new 1109408-drush-test-xml.patch3.89 KB

Decided to take a stab at this tonight. New code is stolen from run-tests.sh almost verbatim, but seems to work fine.

kenorb’s picture

kenorb commented

sub

pol’s picture

pol
the/that
Location Brussels
commented

Hello,

Can you provide an example of the command line with XML output ?

Thanks !

msonnabaum’s picture

msonnabaum commented

I just added a --xml option, so you'd just add that to the existing drush test syntax.

pol’s picture

pol
the/that
Location Brussels
commented
Status: Needs review » Reviewed & tested by the community

Yes I saw that in the patch just after my comment ! Sorry to ask.

We've tested it here at work and it's working perfectly and as it should.

Thanks for the patch!

moshe weitzman’s picture

moshe weitzman commented
Assigned: Unassigned » msonnabaum

ok with me. mark can commit once he is satisfied.

Anonymous’s picture

Anonymous (not verified) commented

subscribe, testing now, will report back shortly.

Anonymous’s picture

Anonymous (not verified) commented
Status: Reviewed & tested by the community » Needs review
StatusFileSize
new junit.xml_.patch4.18 KB

updated patch fixes last db_query() call not wrapped in a version check.

moshe weitzman’s picture

moshe weitzman commented
Status: Needs review » Reviewed & tested by the community

Not a big deal, we have version independent db query and fetch functions. see drush_db_select() and friends.

Am OK if this goes in as is.

msonnabaum’s picture

msonnabaum commented
Status: Reviewed & tested by the community » Fixed

Tested this tonight with Jenkins and it worked great. Committed to 5.x and 4.x.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.