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

@edspencer edspencer commented Jul 26, 2013

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

@netzpirat netzpirat commented Jul 27, 2013

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
1 check failed
1 check failed
@netzpirat
default The Travis CI build failed
Details
@farias-r7
Copy link

@farias-r7 farias-r7 commented Aug 14, 2013

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
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants