Skip to content

Commit

Permalink
Add coveralls coverage reports
Browse files Browse the repository at this point in the history
  • Loading branch information
@tooky
tooky committed Jul 8, 2013
1 parent 925ceaa commit 60cf8af
Show file tree
Hide file tree
Showing 5 changed files with 14 additions and 1 deletion.
1 change: 1 addition & 0 deletions .coveralls.yml
View file
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
service_name: travis-ci
1 change: 1 addition & 0 deletions .gitignore
View file
Original file line number Diff line number Diff line change
Expand Up @@ -2,3 +2,4 @@
/tags
/Gemfile.lock
/tmp/
/coverage
2 changes: 1 addition & 1 deletion Rakefile
View file
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ $:.unshift File.expand_path("../lib", __FILE__)

require "rspec/core/rake_task"
RSpec::Core::RakeTask.new(:spec) do |t|
t.ruby_opts = %w[-w -r./spec/capture_warnings]
t.ruby_opts = %w[-w -r./spec/capture_warnings -r./spec/coveralls]
t.rspec_opts = %w[--color]
end

Expand Down
3 changes: 3 additions & 0 deletions cucumber-core.gemspec
View file
Original file line number Diff line number Diff line change
Expand Up @@ -26,6 +26,9 @@ Gem::Specification.new do |s|
s.add_development_dependency 'kramdown', '~> 0.14'
s.add_development_dependency 'yard', '~> 0.8.0'

# For coverage reports
s.add_development_dependency 'coveralls', '~> 0.6.7'

s.rubygems_version = ">= 1.6.1"
s.files = `git ls-files`.split("\n").reject {|path| path =~ /\.gitignore$/ }
s.test_files = `git ls-files -- {spec,features}/*`.split("\n")
Expand Down
8 changes: 8 additions & 0 deletions spec/coveralls.rb
View file
Original file line number Diff line number Diff line change
@@ -0,0 +1,8 @@
require 'simplecov'
require 'coveralls'

SimpleCov.formatter = SimpleCov::Formatter::MultiFormatter[
SimpleCov::Formatter::HTMLFormatter,
Coveralls::SimpleCov::Formatter
]
SimpleCov.start

1 comment on commit 60cf8af

@mattwynne
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good idea! Clearly reveals all the slop in the AST that we probably don't need.

We should leave that code in there until we've finished integrating with the front-end, then we can blitz it out. Except for the Ast::Table - that should get gutted ASAP. :)

Please sign in to comment.