Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Integrated optional junit xml output #141

Merged
merged 1 commit into from
Aug 6, 2013
Merged

Conversation

edspencer
Copy link

Addresses #87, follows the 7 steps laid out in that ticket.

Adds 3 new configurations to CLI and guard config:

  • junit - set to true to save output to xml files (false by default)
  • junit_consolidate - rolls nested describes up into their parent describe blocks (true by default)
  • junit_save_path - optional path to save the xml files to

Uses the junit reporter created by larrymyers from http://github.com/larrymyers/jasmine-reporters

Sample usage:

guard :jasmine, :junit => true, :junit_save_path => 'reports' do
  watch(%r{^spec/javascripts/.+$}) { 'spec/javascripts' }
  watch(%r{^spec/javascripts/fixtures/.+$}) { 'spec/javascripts' }
  watch(%r{^app/assets/javascripts/(.+?)\.(js\.coffee|js|coffee)(?:\.\w+)*$}) { 'spec/javascripts' }
end

We needed to do this to get Jasmine results properly integrated in our Jenkins CI server, which is now working like a charm :)

Adds 3 new configurations to CLI and guard config:
* junit - set to true to save output to xml files (false by default)
* junit_consolidate - rolls nested describes up into their parent describe blocks (true by default)
* junit_save_path - optional path to save the xml files to

Uses the junit reporter created by larrymyers from github/com/larrymyers/jasmine-reporters
@netzpirat
Copy link
Contributor

That looks perfect, thanks a lot! I'll merge and release a new version when I'm back from holiday in the first week of August, because I don't have my computer with me.

@netzpirat netzpirat merged commit f213c1d into guard:master Aug 6, 2013
@farias-r7
Copy link

Awesome! Pretty much exactly what we needed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants