coverband 1.5.0 → 1.5.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 4e4f7c810900bee83c3516d394e950fde6cb5186
-  data.tar.gz: ba60c4f523b5d50e145cf179113dd6431fd6be4c
+  metadata.gz: cf69ef556cbc606786ff1d0b49b74728cedbf0f5
+  data.tar.gz: 28d03bf504a6a51b112a5db554b8393032f9678f
 SHA512:
-  metadata.gz: ca67d3de1ce546e6d5d3050bde171f70ef258b0792a2e871d8d4c4b93c5609f71e85afcb7d39b9a08d582381e6476fe13c791c45d764726eccaada3170dfd25a
-  data.tar.gz: a1543e402bcc3ef6a0c68c478235d34d1794f695ea9c3099eb31f7c4b54b56015622f1fb920804e0c20614118516fabcf12364e5c97a220108e820e53c29683b
+  metadata.gz: ff7da9c529350fb7dc8b1a7b6725bd3c50c5941934a8704fe17b2db9da8f8921269e8aa520ab9c17bae0a227fd269807823ae751149a7852523089aadc5f9260
+  data.tar.gz: 40fee47fee66304cb0610050c09beb9e08ce686e00b6012791e9dcc48c5efd3970174c3fa9d93eaf1efd28a7b61ebe5dd891be52beedb88275fc8dd1fc7a0120

data/README.md

@@ -67,7 +67,7 @@ That gives you the gem, but to get up and running then follow:
   * Insert middleware in stack  
 * run `bundle exec rake coverband:baseline` ([what is baseline?](https://github.com/danmayer/coverband#coverband-baseline))
 * run `bundle exec rake coverband:coverage` this will show app initialization coverage
-* run app and hit a controller (hit at least +1 time over your `config.startup_delay` setting default is 2)
+* run app and hit a controller (hit at least +1 time over your `config.startup_delay` setting default is 0)
 * run `bundle exec rake coverband:coverage` and you should see coverage increasing for the endpoints you hit.
 
 
@@ -118,7 +118,7 @@ Coverband.configure do |config|
   # Since rails and other frameworks lazy load code. I have found it is bad to allow
   # initial requests to record with coverband. This ignores first 15 requests
   # NOTE: If you are using a threaded webserver (example: Puma) this will ignore requests for each thread
-  config.startup_delay     = Rails.env.production? ? 15 : 2
+  config.startup_delay     = Rails.env.production? ? 5 : 0
   # Percentage of requests recorded
   config.percentage        = Rails.env.production? ? 30.0 : 100.0
 

data/coverband.gemspec

@@ -27,7 +27,7 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency 'sinatra'
   spec.add_development_dependency 'classifier-reborn'
   spec.add_development_dependency 'aws-sdk', '~> 2'
-  spec.add_runtime_dependency "simplecov"
+  spec.add_runtime_dependency "simplecov", '~> 0.11' 
   spec.add_runtime_dependency "json"
   # TODO make redis optional dependancy as we add additional adapters
   spec.add_runtime_dependency "redis"

data/lib/coverband/base.rb

@@ -21,8 +21,9 @@ module Coverband
     def sample
       configure_sampling
       record_coverage
-      yield
+      result = yield
       report_coverage
+      result
     end
 
     def save
@@ -38,6 +39,7 @@ module Coverband
     def reset_instance
       @project_directory = File.expand_path(Coverband.configuration.root)
       @enabled = false
+      @disable_on_failure_for = Coverband.configuration.disable_on_failure_for
       @tracer_set = false
       @file_line_usage = {}
       @ignored_files = Set.new
@@ -65,13 +67,14 @@ module Coverband
     end
 
     def record_coverage
-      if @enabled
+      if @enabled && !failed_recently?
         set_tracer
       else
         unset_tracer
       end
       @stats.increment "coverband.request.recorded.#{@enabled}" if @stats
     rescue RuntimeError => err
+      failed!
       if @verbose
         @logger.info "error stating recording coverage"
         @logger.info "error: #{err.inspect} #{err.message}"
@@ -86,6 +89,11 @@ module Coverband
 
       unset_tracer
 
+      if failed_recently?
+        @logger.info "coverage reporting sanding-by because of recent failure" if @verbose
+        return
+      end
+
       if @verbose
         @logger.info "coverband file usage: #{file_usage.inspect}"
         if @verbose=="debug"
@@ -109,6 +117,7 @@ module Coverband
         @logger.info @file_line_usage.inspect
       end
     rescue RuntimeError => err
+      failed!
       if @verbose
         @logger.info "coverage missing"
         @logger.info "error: #{err.inspect} #{err.message}"
@@ -144,6 +153,27 @@ module Coverband
 
     private
 
+    def failed_at_thread_key
+      "__#{self.class.name}__failed_at"
+    end
+
+    def failed_at
+      Thread.current[failed_at_thread_key]
+    end
+
+    def failed_at=(time)
+      Thread.current[failed_at_thread_key] = time
+    end
+
+    def failed!
+      self.failed_at = Time.now
+    end
+
+    def failed_recently?
+      return false unless @disable_on_failure_for && failed_at
+      failed_at + @disable_on_failure_for > Time.now
+    end
+
     def file_usage
       hash = {}
       @file_line_usage.each do |file, lines|

data/lib/coverband/configuration.rb

@@ -4,7 +4,8 @@ module Coverband
     attr_accessor :redis, :root_paths, :root,
                   :ignore, :percentage, :verbose, :reporter, :stats,
                   :logger, :startup_delay, :trace_point_events,
-                  :include_gems, :memory_caching, :s3_bucket, :coverage_file, :store
+                  :include_gems, :memory_caching, :s3_bucket, :coverage_file, :store,
+                  :disable_on_failure_for
 
     # deprecated, but leaving to allow old configs to 'just work'
     # remove for 2.0
@@ -26,6 +27,7 @@ module Coverband
       @memory_caching = false
       @coverage_file = nil
       @store = nil
+      @disable_on_failure_for = nil
     end
 
     def logger

data/lib/coverband/version.rb

@@ -1,3 +1,3 @@
 module Coverband
-  VERSION = "1.5.0"
+  VERSION = "1.5.1"
 end

data/test/unit/base_test.rb

@@ -93,4 +93,8 @@ class BaseTest < Test::Unit::TestCase
     coverband.save
   end
 
+  test "sample should return the result of the block" do
+    coverband = Coverband::Base.instance.reset_instance
+    assert_equal 2, coverband.sample { 1 + 1 }
+  end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: coverband
 version: !ruby/object:Gem::Version
-  version: 1.5.0
+  version: 1.5.1
 platform: ruby
 authors:
 - Dan Mayer
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-01-05 00:00:00.000000000 Z
+date: 2017-04-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -140,16 +140,16 @@ dependencies:
   name: simplecov
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '0.11'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '0.11'
 - !ruby/object:Gem::Dependency
   name: json
   requirement: !ruby/object:Gem::Requirement