Proposal: Test coverage report for the project about csv HOT 5 CLOSED

I did the first step of the plan. According to SimpleCov the project has 98.86% of coverage:

Some missing coverage is related to exceptions, and some other like Table#push has apparently no test at all:

This is the patch needed:

diff --git a/csv.gemspec b/csv.gemspec
index 6a3435f..5fc79e7 100644
--- a/csv.gemspec
+++ b/csv.gemspec
@@ -25,4 +25,5 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency "bundler"
   spec.add_development_dependency "rake"
   spec.add_development_dependency "benchmark-ips"
+  spec.add_development_dependency "simplecov"
 end
diff --git a/test/lib/test/unit.rb b/test/lib/test/unit.rb
index b83b56a..e600b38 100644
--- a/test/lib/test/unit.rb
+++ b/test/lib/test/unit.rb
@@ -9,6 +9,9 @@ require_relative '../envutil'
 require 'test/unit/testcase'
 require 'optparse'

+require 'simplecov'
+SimpleCov.start
+

Obviously would need bundle & rake in order to have your report.

On the optional part I also setup the CodeClimate for my fork (https://codeclimate.com/github/esparta/csv), and currently looks like this:

from csv.

OK.

from csv.

@hsbt Can you enable this repository on Code Climate or Codecov? (I don't have any opinion for code coverage report service.)

from csv.

I enabled codeclimate with ruby/csv.

@esparta What should I do next step?

from csv.

The next step should be just a simple PR with all the changes, we can use Github's suggestions to change the CodeClimate Test ID and the URL for the Badge.

from csv.