coverband 5.1.0.rcmailer.1 → 5.2.0.rc.1

data/lib/coverband/collectors/view_tracker_service.rb

@@ -8,8 +8,8 @@ module Coverband
     class ViewTrackerService < ViewTracker
       def report_views_tracked
         reported_time = Time.now.to_i
-        if views_to_record.any?
-          relative_views = views_to_record.map! do |view|
+        if @views_to_record.any?
+          relative_views = @views_to_record.map! do |view|
             roots.each do |root|
               view = view.gsub(/#{root}/, "")
             end
@@ -17,7 +17,7 @@ module Coverband
           end
           save_tracked_views(views: relative_views, reported_time: reported_time)
         end
-        self.views_to_record = []
+        @views_to_record = []
       rescue => e
         # we don't want to raise errors if Coverband can't reach the service
         logger&.error "Coverband: view_tracker failed to store, error #{e.class.name}" if Coverband.configuration.verbose || Coverband.configuration.service_dev_mode

data/lib/coverband/configuration.rb

@@ -8,7 +8,7 @@ module Coverband
       :background_reporting_enabled,
       :test_env, :web_enable_clear, :gem_details, :web_debug, :report_on_exit,
       :simulate_oneshot_lines_coverage,
-      :view_tracker
+      :view_tracker, :defer_eager_loading_data
     attr_writer :logger, :s3_region, :s3_bucket, :s3_access_key_id,
       :s3_secret_access_key, :password, :api_key, :service_url, :coverband_timeout, :service_dev_mode,
       :service_test_mode, :process_type, :track_views, :redis_url,
@@ -62,6 +62,7 @@ module Coverband
       @store = nil
       @background_reporting_enabled = true
       @background_reporting_sleep_seconds = nil
+      @defer_eager_loading_data = false
       @test_env = nil
       @web_enable_clear = false
       @track_views = true
@@ -147,7 +148,9 @@ module Coverband
     end
 
     def track_views
-      @track_views ||= service_disabled_dev_test_env? ? false : true
+      return false if service_disabled_dev_test_env?
+
+      @track_views
     end
 
     ###
@@ -241,6 +244,10 @@ module Coverband
       Coverband.coverband_service? || !api_key.nil?
     end
 
+    def defer_eager_loading_data?
+      @defer_eager_loading_data
+    end
+
     def service_disabled_dev_test_env?
       (coverband_env == "test" && !Coverband.configuration.service_test_mode) ||
         (coverband_env == "development" && !Coverband.configuration.service_dev_mode)

data/lib/coverband/integrations/background.rb

@@ -31,15 +31,18 @@ module Coverband
         sleep_seconds = Coverband.configuration.background_reporting_sleep_seconds.to_i
         @thread = Thread.new {
           loop do
-            Coverband.report_coverage
-            Coverband.configuration.view_tracker&.report_views_tracked
             if Coverband.configuration.reporting_wiggle
               sleep_seconds = Coverband.configuration.background_reporting_sleep_seconds.to_i + rand(Coverband.configuration.reporting_wiggle.to_i)
             end
+            # NOTE: Normally as processes first start we immediately report, this causes a redis spike on deploys
+            # if deferred is set also sleep frst to spread load
+            sleep(sleep_seconds.to_i) if Coverband.configuration.defer_eager_loading_data?
+            Coverband.report_coverage
+            Coverband.configuration.view_tracker&.report_views_tracked
             if Coverband.configuration.verbose
               logger.debug("Coverband: background reporting coverage (#{Coverband.configuration.store.type}). Sleeping #{sleep_seconds}s")
             end
-            sleep(sleep_seconds.to_i)
+            sleep(sleep_seconds.to_i) unless Coverband.configuration.defer_eager_loading_data?
           end
         }
       end

data/lib/coverband/reporters/base.rb

@@ -15,7 +15,7 @@ module Coverband
           scov_style_report = get_current_scov_data_imp(store, all_roots)
 
           # These are extremelhy verbose but useful during coverband development, not generally for users
-          # Only avaiable by uncommenting this mode is never released
+          # Only available by uncommenting this mode is never released
           # if Coverband.configuration.verbose
           #   # msg = "report:\n #{scov_style_report.inspect}"
           #   # Coverband.configuration.logger.debug msg

data/lib/coverband/reporters/web.rb

@@ -124,7 +124,7 @@ module Coverband
           Coverband.configuration.store.clear!
           notice = "coverband coverage cleared"
         else
-          notice = "web_enable_clear isnt enabled in your configuration"
+          notice = "web_enable_clear isn't enabled in your configuration"
         end
         [301, {"Location" => "#{base_path}?notice=#{notice}"}, []]
       end
@@ -135,7 +135,7 @@ module Coverband
           Coverband.configuration.store.clear_file!(filename)
           notice = "coverage for file #{filename} cleared"
         else
-          notice = "web_enable_clear isnt enabled in your configuration"
+          notice = "web_enable_clear isn't enabled in your configuration"
         end
         [301, {"Location" => "#{base_path}?notice=#{notice}"}, []]
       end
@@ -146,7 +146,7 @@ module Coverband
           tracker.reset_recordings
           notice = "view tracking reset"
         else
-          notice = "web_enable_clear isnt enabled in your configuration"
+          notice = "web_enable_clear isn't enabled in your configuration"
         end
         [301, {"Location" => "#{base_path}/view_tracker?notice=#{notice}"}, []]
       end
@@ -158,7 +158,7 @@ module Coverband
           tracker.clear_file!(filename)
           notice = "coverage for file #{filename} cleared"
         else
-          notice = "web_enable_clear isnt enabled in your configuration"
+          notice = "web_enable_clear isn't enabled in your configuration"
         end
         [301, {"Location" => "#{base_path}/view_tracker?notice=#{notice}"}, []]
       end

data/lib/coverband/utils/dead_methods.rb

@@ -0,0 +1,63 @@
+# frozen_string_literal: ntrue
+
+require "coverband/utils/method_definition_scanner"
+
+module Coverband
+  module Utils
+    module ArrayToTableInConsole
+      refine Array do
+        def to_table
+          column_sizes =
+            reduce([]) { |lengths, row|
+              row.each_with_index.map do |iterand, index|
+                [lengths[index] || 0, iterand.to_s.length].max
+              end
+            }
+          puts head =
+                 "-" * (column_sizes.inject(&:+) + (3 * column_sizes.count) + 1)
+          each do |row|
+            row = row.fill(nil, row.size..(column_sizes.size - 1))
+            row =
+              row.each_with_index.map { |v, i|
+                v.to_s + " " * (column_sizes[i] - v.to_s.length)
+              }
+            puts "| " + row.join(" | ") + " |"
+          end
+          puts head
+        end
+      end
+    end
+
+    class DeadMethods
+      using ArrayToTableInConsole
+      def self.scan(file_path:, coverage:)
+        MethodDefinitionScanner.scan(file_path).reject do |method_definition|
+          method_definition.body.coverage?(coverage)
+        end
+      end
+
+      def self.scan_all
+        coverage = Coverband.configuration.store.coverage
+        coverage.flat_map do |file_path, coverage|
+          scan(file_path: file_path, coverage: coverage["data"])
+        end
+      end
+
+      def self.output_all
+        rows =
+          scan_all.each_with_object(
+            [%w[file class method line_number]]
+          ) { |dead_method, rows|
+            rows <<
+              [
+                dead_method.file_path,
+                dead_method.class_name,
+                dead_method.name,
+                dead_method.first_line_number
+              ]
+          }
+        rows.to_table
+      end
+    end
+  end
+end

data/lib/coverband/utils/method_definition_scanner.rb

@@ -0,0 +1,96 @@
+# frozen_string_literal: true
+
+if defined?(RubyVM::AbstractSyntaxTree)
+  module Coverband
+    module Utils
+      class MethodDefinitionScanner
+        attr_reader :path
+
+        def initialize(path)
+          @path = path
+        end
+
+        def scan
+          scan_node(RubyVM::AbstractSyntaxTree.parse_file(path), nil)
+        end
+
+        def self.scan(path)
+          new(path).scan
+        end
+
+        class MethodBody
+          def initialize(method_definition)
+            @method_definition = method_definition
+          end
+
+          def coverage?(file_coverage)
+            body_coverage =
+              file_coverage[(first_line_number - 1)..(last_line_number - 1)]
+            body_coverage.map(&:to_i).any?(&:positive?)
+          end
+
+          private
+
+          def first_line_number
+            @method_definition.first_line_number + 1
+          end
+
+          def last_line_number
+            @method_definition.last_line_number - 1
+          end
+        end
+
+        class MethodDefinition
+          attr_reader :last_line_number,
+            :first_line_number,
+            :name,
+            :class_name,
+            :file_path
+
+          def initialize(
+            first_line_number:,
+            last_line_number:,
+            name:,
+            class_name:,
+            file_path:
+          )
+            @first_line_number = first_line_number
+            @last_line_number = last_line_number
+            @name = name
+            @class_name = class_name
+            @file_path = file_path
+          end
+
+          def body
+            MethodBody.new(self)
+          end
+        end
+
+        private
+
+        def scan_node(node, class_name)
+          definitions = []
+          return definitions unless node.is_a?(RubyVM::AbstractSyntaxTree::Node)
+          current_class = node.type == :CLASS ? node.children.first.children.last : class_name
+          if node.type == :DEFN
+            definitions <<
+              MethodDefinition.new(
+                first_line_number: node.first_lineno,
+                last_line_number: node.last_lineno,
+                name: node.children.first,
+                class_name: current_class,
+                file_path: path
+              )
+          end
+          definitions + scan_children(node, current_class)
+        end
+
+        def scan_children(node, current_class)
+          node.children.flatten.compact.map { |child|
+            scan_node(child, current_class)
+          }.flatten
+        end
+      end
+    end
+  end
+end

data/lib/coverband/utils/railtie.rb

@@ -21,7 +21,7 @@ module Coverband
 
     config.after_initialize do
       unless Coverband.tasks_to_ignore?
-        Coverband.configure
+        Coverband.configure unless Coverband.configured?
         Coverband.eager_loading_coverage!
         Coverband.report_coverage
         Coverband.runtime_coverage!
@@ -37,7 +37,7 @@ module Coverband
 
           Coverband.configuration.view_tracker = COVERBAND_VIEW_TRACKER
 
-          ActiveSupport::Notifications.subscribe(/render_partial.action_view|render_template.action_view/) do |name, start, finish, id, payload|
+          ActiveSupport::Notifications.subscribe(/render_(template|partial|collection).action_view/) do |name, start, finish, id, payload|
             COVERBAND_VIEW_TRACKER.track_views(name, start, finish, id, payload) unless name.include?("!")
           end
         end
@@ -50,6 +50,7 @@ module Coverband
     config.before_configuration do
       unless ENV["COVERBAND_DISABLE_AUTO_START"]
         begin
+          Coverband.configure unless Coverband.configured?
           Coverband.start
         rescue Redis::CannotConnectError => error
           Coverband.configuration.logger.info "Redis is not available (#{error}), Coverband not configured"

data/lib/coverband/utils/result.rb

@@ -55,7 +55,7 @@ module Coverband
           Dir[tracked_files].each do |file|
             absolute = File.expand_path(file)
             result[absolute] ||= {
-              "data" => Array.new(File.foreach(absolute).count) { 0 },
+              "data" => [],
               "never_loaded" => true
             }
           end

data/lib/coverband/utils/source_file.rb

@@ -101,7 +101,7 @@ module Coverband
           @last_updated_at = Time.at(file_data["last_updated_at"]) if file_data["last_updated_at"]
           @never_loaded = file_data["never_loaded"] || false
         else
-          # TODO: Deprecate this code path this was backwards compatability from 3-4
+          # TODO: Deprecate this code path this was backwards compatibility from 3-4
           @coverage = file_data
           @first_updated_at = NOT_AVAILABLE
           @last_updated_at = NOT_AVAILABLE
@@ -139,7 +139,7 @@ module Coverband
         coverage_exceeding_source_warn if coverage.size > src.size
 
         lines = src.map.with_index(1) { |src, i|
-          Coverband::Utils::SourceFile::Line.new(src, i, coverage[i - 1])
+          Coverband::Utils::SourceFile::Line.new(src, i, never_loaded ? 0 : coverage[i - 1])
         }
 
         process_skipped_lines(lines)

data/lib/coverband/utils/tasks.rb

@@ -9,11 +9,25 @@ namespace :coverband do
     Coverband::Reporters::ConsoleReport.report(Coverband.configuration.store)
   end
 
+  if defined?(RubyVM::AbstractSyntaxTree)
+    require "coverband/utils/dead_methods"
+
+    desc "Output all dead methods"
+    task :dead_methods do
+      Coverband::Utils::DeadMethods.output_all
+    end
+  end
+
   desc "report runtime Coverband code coverage"
   task :coverage_server do
-    Rake.application["environment"].invoke if Rake::Task.task_defined?("environment")
-    Coverband.configuration.store.merge_mode = true if Coverband.configuration.store.is_a?(Coverband::Adapters::FileStore)
-    Rack::Server.start app: Coverband::Reporters::Web.new, Port: ENV.fetch("COVERBAND_COVERAGE_PORT", 9022).to_i
+    if Rake::Task.task_defined?("environment")
+      Rake.application["environment"].invoke
+    end
+    if Coverband.configuration.store.is_a?(Coverband::Adapters::FileStore)
+      Coverband.configuration.store.merge_mode = true
+    end
+    Rack::Server.start app: Coverband::Reporters::Web.new,
+                       Port: ENV.fetch("COVERBAND_COVERAGE_PORT", 9022).to_i
   end
 
   ###

data/lib/coverband/version.rb

@@ -2,8 +2,8 @@
 
 ###
 # ensure we properly do release candidate versioning; https://github.com/danmayer/coverband/issues/288
-# use format '4.2.1.rc.1' ~> 4.2.1.rc to prerelease versions like v4.2.1.rc.2 and v4.2.1.rc.3
+# use format "4.2.1.rc.1" ~> 4.2.1.rc to prerelease versions like v4.2.1.rc.2 and v4.2.1.rc.3
 ###
 module Coverband
-  VERSION = "5.1.0.rcmailer.1"
+  VERSION = '5.2.0.rc.1'
 end

data/lib/coverband.rb

@@ -13,6 +13,7 @@ require "coverband/adapters/redis_store"
 require "coverband/adapters/hash_redis_store"
 require "coverband/adapters/file_store"
 require "coverband/adapters/stdout_store"
+require "coverband/adapters/null_store"
 require "coverband/utils/file_hasher"
 require "coverband/collectors/coverage"
 require "coverband/collectors/view_tracker"

data/public/dependencies.js

@@ -103,7 +103,7 @@ var hljs=new function(){function l(o){return o.replace(/&/gm,"&amp;").replace(/<
                 return this.rel;
             },
             href: function() {
-                // using this.href would give the absolute url, when the href may have been inteded as a selector (e.g. '#container')
+                // using this.href would give the absolute url, when the href may have been intedned as a selector (e.g. '#container')
                 return $(this).attr('href');
             },
             title: function() {
@@ -397,7 +397,7 @@ var hljs=new function(){function l(o){return o.replace(/&/gm,"&amp;").replace(/<
                 loadedHeight = $loaded.outerHeight(true);
                 loadedWidth = $loaded.outerWidth(true);
 
-                // Opens inital empty Colorbox prior to content being loaded.
+                // Opens initial empty Colorbox prior to content being loaded.
                 var initialWidth = setSize(settings.get('initialWidth'), 'x');
                 var initialHeight = setSize(settings.get('initialHeight'), 'y');
                 var maxWidth = settings.get('maxWidth');
@@ -983,7 +983,7 @@ var hljs=new function(){function l(o){return o.replace(/&/gm,"&amp;").replace(/<
                         return;
                     }
 
-                    // A small pause because some browsers will occassionaly report a
+                    // A small pause because some browsers will occasionally report a
                     // img.width and img.height of zero immediately after the img.onload fires
                     setTimeout(function(){
                         var percent;
@@ -1236,7 +1236,7 @@ u;this.oApi._fnAddColumn=y;this.oApi._fnColumnOptions=C;this.oApi._fnAddData=w;t
 Q;this.oApi._fnFilterCreateSearch=fa;this.oApi._fnDataToSearch=ga;this.oApi._fnSort=O;this.oApi._fnSortAttachListener=$;this.oApi._fnSortingClasses=W;this.oApi._fnFeatureHtmlPaginate=wa;this.oApi._fnPageChange=Ba;this.oApi._fnFeatureHtmlInfo=va;this.oApi._fnUpdateInfo=Ca;this.oApi._fnFeatureHtmlLength=ra;this.oApi._fnFeatureHtmlProcessing=ta;this.oApi._fnProcessingDisplay=K;this.oApi._fnVisibleToColumnIndex=da;this.oApi._fnColumnIndexToVisible=N;this.oApi._fnNodeToDataIndex=R;this.oApi._fnVisbleColumns=
 T;this.oApi._fnCalculateEnd=F;this.oApi._fnConvertToWidth=Da;this.oApi._fnCalculateColumnWidths=ea;this.oApi._fnScrollingWidthAdjust=ia;this.oApi._fnGetWidestNode=Ea;this.oApi._fnGetMaxLenString=Fa;this.oApi._fnStringToCss=v;this.oApi._fnArrayCmp=La;this.oApi._fnDetectType=Z;this.oApi._fnSettingsFromNode=B;this.oApi._fnGetDataMaster=V;this.oApi._fnGetTrNodes=S;this.oApi._fnGetTdNodes=X;this.oApi._fnEscapeRegex=ha;this.oApi._fnDeleteIndex=ja;this.oApi._fnReOrderIndex=qa;this.oApi._fnColumnOrdering=
 aa;this.oApi._fnLog=J;this.oApi._fnClearTable=ba;this.oApi._fnSaveState=Ga;this.oApi._fnLoadState=Ia;this.oApi._fnCreateCookie=Ha;this.oApi._fnReadCookie=ka;this.oApi._fnGetUniqueThs=ca;this.oApi._fnScrollBarWidth=Ja;this.oApi._fnApplyToChildren=M;this.oApi._fnMap=n;var Ka=this;return this.each(function(){var a=0,b,c,d,f;a=0;for(b=E.length;a<b;a++){if(E[a].nTable==this)if(typeof g=="undefined"||typeof g.bRetrieve!="undefined"&&g.bRetrieve===true)return E[a].oInstance;else if(typeof g.bDestroy!="undefined"&&
-g.bDestroy===true){E[a].oInstance.fnDestroy();break}else{J(E[a],0,"Cannot reinitialise DataTable.\n\nTo retrieve the DataTables object for this table, please pass either no arguments to the dataTable() function, or set bRetrieve to true. Alternatively, to destory the old table and create a new one, set bDestroy to true (note that a lot of changes to the configuration can be made through the API which is usually much faster).");return}if(E[a].sTableId!==""&&E[a].sTableId==this.getAttribute("id")){E.splice(a,
+g.bDestroy===true){E[a].oInstance.fnDestroy();break}else{J(E[a],0,"Cannot reinitialise DataTable.\n\nTo retrieve the DataTables object for this table, please pass either no arguments to the dataTable() function, or set bRetrieve to true. Alternatively, to destroy the old table and create a new one, set bDestroy to true (note that a lot of changes to the configuration can be made through the API which is usually much faster).");return}if(E[a].sTableId!==""&&E[a].sTableId==this.getAttribute("id")){E.splice(a,
 1);break}}var e=new l;E.push(e);var i=false,h=false;a=this.getAttribute("id");if(a!==null){e.sTableId=a;e.sInstance=a}else e.sInstance=m._oExternConfig.iNextUnique++;e.oInstance=Ka;e.nTable=this;e.oApi=Ka.oApi;if(typeof g!="undefined"&&g!==null){e.oInit=g;n(e.oFeatures,g,"bPaginate");n(e.oFeatures,g,"bLengthChange");n(e.oFeatures,g,"bFilter");n(e.oFeatures,g,"bSort");n(e.oFeatures,g,"bInfo");n(e.oFeatures,g,"bProcessing");n(e.oFeatures,g,"bAutoWidth");n(e.oFeatures,g,"bSortClasses");n(e.oFeatures,
 g,"bServerSide");n(e.oScroll,g,"sScrollX","sX");n(e.oScroll,g,"sScrollXInner","sXInner");n(e.oScroll,g,"sScrollY","sY");n(e.oScroll,g,"bScrollCollapse","bCollapse");n(e,g,"asStripClasses");n(e,g,"fnRowCallback");n(e,g,"fnHeaderCallback");n(e,g,"fnFooterCallback");n(e,g,"fnInitComplete");n(e,g,"fnServerData");n(e,g,"fnFormatNumber");n(e,g,"aaSorting");n(e,g,"aaSortingFixed");n(e,g,"aLengthMenu");n(e,g,"sPaginationType");n(e,g,"sAjaxSource");n(e,g,"iCookieDuration");n(e,g,"sCookiePrefix");n(e,g,"sDom");
 n(e,g,"oSearch","oPreviousSearch");n(e,g,"aoSearchCols","aoPreSearchCols");n(e,g,"iDisplayLength","_iDisplayLength");n(e,g,"bJQueryUI","bJUI");typeof g.fnDrawCallback=="function"&&e.aoDrawCallback.push({fn:g.fnDrawCallback,sName:"user"});e.oFeatures.bServerSide&&e.oFeatures.bSort&&e.oFeatures.bSortClasses&&e.aoDrawCallback.push({fn:W,sName:"server_side_sort_classes"});if(typeof g.bJQueryUI!="undefined"&&g.bJQueryUI){e.oClasses=m.oJUIClasses;if(typeof g.sDom=="undefined")e.sDom='<"H"lfr>t<"F"ip>'}if(e.oScroll.sX!==

data/roadmap.md

@@ -0,0 +1,56 @@
+# Future Roadmap
+
+### Research Alternative Redis formats
+
+- Look at alternative storage formats for Redis
+  - [redis bitmaps](http://blog.getspool.com/2011/11/29/fast-easy-realtime-metrics-using-redis-bitmaps/)
+  - [redis bitfield](https://stackoverflow.com/questions/47100606/optimal-way-to-store-array-of-integers-in-redis-database)
+- Add support for [zadd](http://redis.io/topics/data-types-intro) so one could determine single call versus multiple calls on a line, letting us determine the most executed code in production.
+
+### Coverband Future...
+
+Will be the fully modern release that drops maintenance legacy support in favor of increased performance, ease of use, and maintainability.
+
+- Release will be aimed as significantly simplifying ease of use
+  - near zero config setup for Rails apps
+  - add built-in support for easy loading via Railties
+  - built in support for activejob, sidekiq, and other common frameworks
+  - reduced configuration options
+- support oneshot
+- drop middleware figure out a way to kick off background without middelware
+- options on reporting
+  - background reporting
+  - or middleware reporting
+- Support for file versions
+  - md5 or release tags
+  - add coverage timerange support
+- Drop Simplecov dependency
+- improved web reporting
+  - lists current config options
+  - eventually allow updating remote config
+  - full theming
+  - list redis data dump for debugging
+- additional adapters: Memcache, S3, and ActiveRecord
+- add articles / podcasts like prontos readme https://github.com/prontolabs/pronto
+- Add detailed Gem usage report, if we collect and send gem usage we can give percentage of gem code used, which should help application developers know when to remove gem dependencies (0%) or perhaps inline single methods for little usage (using <= 5%) for example.
+- add meta data information first seen last recorded to the coverage report views (probably need to drop simplecov for that).
+  - more details in this issue: https://github.com/danmayer/coverband/issues/118
+- Make good video on setup, install, usage
+- See if we can add support for views / templates
+  - using this technique https://github.com/ioquatix/covered
+- Better default grouping (could use groups features for gems for rails controllers, models, lib, etc)
+- Improved logging for easier debugging and development
+  - drop the verbose mode and better support standard logger levels
+- Possibly setup a build assets system
+  - my JS rules expanded the compressed JS at the top of application.js, basically we want to stitch together JS
+  - I guess we could also load multiple JS files as most of the JS is just default compressed JS and a tiny amount of actual app JS.
+- lazy load for Coverband results
+- view layer file coverage
+- move all code to work with relative paths leaving only stdlib Coverage working on full paths
+- add gem_safe_lists to track only some gems
+- add gem_details_safe list to report on details on some gems
+- - display gems that are in loaded with 0 coverage, thanks @kbaum
+
+# Alpha / Beta / Release Candidates
+
+### Coverband 5.?.?

data/test/coverband/adapters/null_store_test.rb

@@ -0,0 +1,39 @@
+# frozen_string_literal: true
+
+require File.expand_path("../../test_helper", File.dirname(__FILE__))
+
+class AdaptersNullStoreTest < Minitest::Test
+  def test_covered_lines_when_no_file
+    @store = Coverband::Adapters::NullStore.new("")
+    expected = {}
+    assert_equal expected, @store.coverage
+  end
+
+  describe "Coverband::Adapters::NullStore" do
+    def setup
+      super
+      @store = Coverband::Adapters::NullStore.new(@test_file_path)
+    end
+
+    def test_coverage
+      assert_equal @store.coverage, {}
+    end
+
+    def test_covered_lines_when_null
+      assert_nil @store.coverage["none.rb"]
+    end
+
+    def test_covered_files
+      assert_equal @store.covered_files.include?("dog.rb"), false
+    end
+
+    def test_clear
+      assert_nil @store.clear!
+    end
+
+    def test_save_report
+      @store.send(:save_report, "cat.rb" => [0, 1])
+      assert_equal @store.coverage, {}
+    end
+  end
+end

data/test/coverband/configuration_test.rb

@@ -10,7 +10,7 @@ class BaseTest < Minitest::Test
     Coverband.configure do |config|
       config.root = Dir.pwd
       config.root_paths = ["/app_path/"]
-      config.ignore = ["config/envionments"]
+      config.ignore = ["config/environments"]
       config.reporter = "std_out"
       config.store = Coverband::Adapters::RedisStore.new(Coverband::Test.redis, redis_namespace: "coverband_test")
     end
@@ -18,7 +18,7 @@ class BaseTest < Minitest::Test
 
   test "ignore works with equal" do
     Coverband::Collectors::Coverage.instance.reset_instance
-    expected = ["vendor/", ".erb$", ".slim$", "/tmp", "internal:prelude", "db/schema.rb", "config/envionments"]
+    expected = ["vendor/", ".erb$", ".slim$", "/tmp", "internal:prelude", "db/schema.rb", "config/environments"]
     assert_equal expected, Coverband.configuration.ignore
   end
 
@@ -33,7 +33,7 @@ class BaseTest < Minitest::Test
       "/tmp",
       "internal:prelude",
       "db/schema.rb",
-      "config/envionments",
+      "config/environments",
       "config/initializers"]
     assert_equal expected, Coverband.configuration.ignore
   end
@@ -47,7 +47,7 @@ class BaseTest < Minitest::Test
     Coverband::Collectors::Coverage.instance.reset_instance
     current_paths = Coverband.configuration.root_paths.dup
     # verify previous bug fix
-    # it would extend the root_paths instance variable on each invokation
+    # it would extend the root_paths instance variable on each invocation
     Coverband.configuration.all_root_paths
     Coverband.configuration.all_root_paths
     assert_equal current_paths, Coverband.configuration.root_paths
@@ -103,7 +103,7 @@ class BaseTest < Minitest::Test
     end
   end
 
-  test "store doesnt raises when api key and redis_url" do
+  test "store doesn't raises when api key and redis_url" do
     Coverband::Collectors::Coverage.instance.reset_instance
     Coverband.configuration.reset
     Coverband.configure do |config|

data/test/coverband/utils/dead_methods_test.rb

@@ -0,0 +1,53 @@
+# frozen_string_literal: true
+
+require File.expand_path("../../test_helper", File.dirname(__FILE__))
+
+if defined?(RubyVM::AbstractSyntaxTree)
+  require "coverband/utils/dead_methods"
+  module Coverband
+    module Utils
+      class DeadMethodsTest < Minitest::Test
+        attr_accessor :coverband
+
+        def setup
+          super
+          @coverband = Coverband::Collectors::Coverage.instance
+        end
+
+        def test_dog_dead_methods
+          file_path = require_unique_file
+          coverage = [nil, nil, 1, 1, 0, nil, nil]
+          dead_methods =
+            DeadMethods.scan(file_path: file_path, coverage: coverage)
+          assert_equal(1, dead_methods.length)
+          dead_method = dead_methods.first
+          assert_equal(4, dead_method.first_line_number)
+          assert_equal(6, dead_method.last_line_number)
+          assert_equal(file_path, dead_method.file_path)
+        end
+
+        def test_all_dead_methods
+          require_unique_file
+          @coverband.report_coverage
+          dead_methods = DeadMethods.scan_all
+          dead_method = dead_methods.find { |method| method.class_name == :Dog }
+          assert(dead_method)
+          assert_equal(4, dead_method.first_line_number)
+          assert_equal(6, dead_method.last_line_number)
+        end
+
+        def test_output_all
+          require_unique_file
+          @coverband.report_coverage
+          DeadMethods.output_all
+        end
+
+        def test_dog_methods_not_dead
+          file = require_unique_file
+          coverage = [nil, nil, 1, 1, 1, nil, nil]
+          assert_empty(DeadMethods.scan(file_path: file, coverage: coverage))
+        end
+      end
+    end
+  end
+end