coverband 0.0.5 → 0.0.7

data/README.md

@@ -20,6 +20,18 @@ Or install it yourself as:
 
 TODO: Write usage instructions here
 
+## TODO
+
+* improve the configuration flow (only one time redis setup etc)
+* fix performance by logging to files that purge later
+
+## Completed
+
+* fix issue if a file can't be found for reporting
+* add support for file matching ignore for example we need to ignore '/app/vendor/'
+  * fix issue on heroku where it logs non app files
+* Allow more configs to be passed in like percentage
+
 ## Contributing
 
 1. Fork it

data/lib/coverband.rb

@@ -1,3 +1,5 @@
+require 'redis'
+
 require "coverband/version"
 require "coverband/reporter"
 require "coverband/middleware"

data/lib/coverband/middleware.rb

@@ -9,6 +9,7 @@ module Coverband
       @function_set = false
       @files = {}
 
+      @ignore_patterns = settings[:ignore] || []
       @sample_percentage = settings[:percentage] || 100.0
       @reporter = settings[:reporter]
     end
@@ -48,7 +49,7 @@ module Coverband
     end
     
     def add_file(file, line)
-      unless !file.match(@project_directory) 
+      if file.match(@project_directory) && !@ignore_patterns.any?{|pattern| file.match(/#{pattern}/) } 
         if @files.include?(file)
           @files[file] << line
           @files[file].uniq!

data/lib/coverband/reporter.rb

@@ -28,7 +28,7 @@ module Coverband
 
     def self.report_rcov(redis, existing_coverage)
       rcov_style_report = {}
-      redis.smembers('coverband').each{|key| rcov_style_report.merge!(line_hash(redis, key)) }
+      redis.smembers('coverband').each{|key| line_data = line_hash(redis, key); rcov_style_report.merge!(line_data) if line_data }
       rcov_style_report = merge_existing_coverage(rcov_style_report, existing_coverage)
       puts "report:"
       puts rcov_style_report.inspect
@@ -65,16 +65,19 @@ module Coverband
     # {"/Users/danmayer/projects/hearno/script/tester.rb"=>[1, nil, 1, 1, nil, nil, nil]}
     def self.line_hash(redis, key)
       filename = key.gsub('.','/').gsub('/rb','.rb').gsub('/erb','.erb')
-
-      lines_hit = redis.smembers("coverband#{key}")
-      count = File.foreach(filename).inject(0) {|c, line| c+1}
-      if filename.match(/\.erb/)
-        line_array = Array.new(count, nil)
+      if File.exists?(filename)
+        lines_hit = redis.smembers("coverband#{key}")
+        count = File.foreach(filename).inject(0) {|c, line| c+1}
+        if filename.match(/\.erb/)
+          line_array = Array.new(count, nil)
+        else
+          line_array = Array.new(count, 0)
+        end
+        line_array.each_with_index{|line,index| line_array[index]=1 if lines_hit.include?((index+1).to_s) }
+        {filename => line_array}
       else
-        line_array = Array.new(count, 0)
+        puts "file #{filename} not found in project"
       end
-      line_array.each_with_index{|line,index| line_array[index]=1 if lines_hit.include?((index+1).to_s) }
-      {filename => line_array}
     end
 
   end

data/lib/coverband/version.rb

@@ -1,3 +1,3 @@
 module Coverband
-  VERSION = "0.0.5"
+  VERSION = "0.0.7"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: coverband
 version: !ruby/object:Gem::Version
-  version: 0.0.5
+  version: 0.0.7
   prerelease: 
 platform: ruby
 authors:
@@ -108,7 +108,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 1757980899486783468
+      hash: 1958954660094686763
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
@@ -117,7 +117,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 1757980899486783468
+      hash: 1958954660094686763
 requirements: []
 rubyforge_project: 
 rubygems_version: 1.8.25