Skip to content
This repository has been archived by the owner. It is now read-only.

Provide xunit test-reporter #19

Closed
mcbain opened this issue Dec 5, 2014 · 11 comments

Comments

@mcbain
Copy link

@mcbain mcbain commented Dec 5, 2014

Is it possible to generate a test-report xml parsable for teamcity or jenkins?

@robot-lavaina

This comment has been minimized.

Copy link

@robot-lavaina robot-lavaina commented Dec 5, 2014

Yes, it is, meteor's test-in-console supports it, but it needs some heavy
lifting, due to the way test-in-console supports it. I'll put it on the
roadmap.

@meonkeys

This comment has been minimized.

Copy link
Contributor

@meonkeys meonkeys commented Jan 2, 2015

+1. Great idea!

@rbabayoff

This comment has been minimized.

Copy link
Member

@rbabayoff rbabayoff commented Jan 4, 2015

Hey, will one of you be willing to work on a pr for this? I can guide you through it.

@meonkeys

This comment has been minimized.

Copy link
Contributor

@meonkeys meonkeys commented Jan 12, 2015

@rbabayoff I think so! Will you describe what needs to be done? Specs can probably be entered right in this issue. I'm also in the meteor-talk google group and on the #meteor channel on Freenode (my nick is meonkeys) if you'd like to chat/discuss elsewhere.

@rbabayoff

This comment has been minimized.

Copy link
Member

@rbabayoff rbabayoff commented Jan 15, 2015

@meonkeys that's great. Let me go over code to see exactly what needs to be done.

@meonkeys

This comment has been minimized.

Copy link
Contributor

@meonkeys meonkeys commented Mar 19, 2015

I see in Meteor's packages/test-in-console/driver.js how we can set platform to xunit to get the XML output we desire. I also see local mentioned in src/phantomjs-test-in-console.coffee, but I don't really get how/where to allow platform to be changed with a command-line argument to spacejam.

@rbabayoff

This comment has been minimized.

Copy link
Member

@rbabayoff rbabayoff commented Mar 26, 2015

Hey, sorry, dropped the ball on this. I can get on a skype / google hangout to run you trough it. Just email me.

meonkeys added a commit to meonkeys/spacejam that referenced this issue Mar 27, 2015
@rbabayoff - this is a start, just needs your bit to change 'local' to
'xunit' using a command line argument.

Seems clunky to use another shell script, but I'm still not quite sure
how to work this into the main spacejam code.

Oh, here's an idea... in src/phantomjs-test-in-console.coffee log lines
to test-output.xml if they match /^##_meteor_magic##xunit: /

Thoughts?

I'll work on an alternate PR to do that.
meonkeys added a commit to meonkeys/spacejam that referenced this issue Mar 27, 2015
Seems clunky to use another shell script, but I'm still not quite sure how to work this into the main spacejam code.

Oh, here's an idea... in src/phantomjs-test-in-console.coffee log lines to test-output.xml if they match /^##_meteor_magic##xunit: / ??

... hrm, I'm unable to write to a file in page.onConsoleMessage (in src/phantomjs-test-in-console.coffee). Maybe I'd need to use the HTML5 FileSystem API? Not sure.
@meonkeys

This comment has been minimized.

Copy link
Contributor

@meonkeys meonkeys commented Mar 27, 2015

See PR #28. I wish I could write to a file write from phantomjs onConsoleMessage, but I'm not sure how to do that. Oh, and I already had a fork, so I created the PR in my own fork. I'm happy to move it if it matters to you.

@rbabayoff

This comment has been minimized.

Copy link
Member

@rbabayoff rbabayoff commented Mar 27, 2015

The best way would be to do it on the spacejam side, filter the phantomjs child process output and save to file there.

@meonkeys

This comment has been minimized.

Copy link
Contributor

@meonkeys meonkeys commented Mar 28, 2015

Ok, I'll take another look.

@meonkeys

This comment has been minimized.

Copy link
Contributor

@meonkeys meonkeys commented Mar 29, 2015

See PR #29.

@rbabayoff rbabayoff closed this in 56840f4 Apr 19, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
4 participants
You can’t perform that action at this time.