coverband 5.1.0 → 5.2.0

data/lib/coverband/adapters/null_store.rb

@@ -0,0 +1,42 @@
+# frozen_string_literal: true
+
+module Coverband
+  module Adapters
+    ###
+    # NullStore is for benchmarking the impacts of calculating
+    # and storing coverage data independent of Coverband/Coverage
+    #
+    # Usage:
+    # config.store = Coverband::Adapters::NullStore.new
+    ###
+    class NullStore < Base
+      def initialize(_opts = {})
+        super()
+      end
+
+      def clear!
+        # NOOP
+      end
+
+      def size
+        0
+      end
+
+      def migrate!
+        raise NotImplementedError, "NullStore doesn't support migrations"
+      end
+
+      def coverage(_local_type = nil)
+        {}
+      end
+
+      def save_report(report)
+        # NOOP
+      end
+
+      def raw_store
+        raise NotImplementedError, "NullStore doesn't support raw_store"
+      end
+    end
+  end
+end

data/lib/coverband/adapters/redis_store.rb

@@ -7,7 +7,7 @@ module Coverband
     ###
     class RedisStore < Base
       ###
-      # This key isn't related to the coverband version, but to the interal format
+      # This key isn't related to the coverband version, but to the internal format
       # used to store data to redis. It is changed only when breaking changes to our
       # redis format are required.
       ###

data/lib/coverband/collectors/coverage.rb

@@ -46,12 +46,30 @@ module Coverband
         @store.type = old_coverage_type
       end
 
+      def toggle_eager_loading
+        old_coverage_type = @store.type
+        eager_loading!
+        yield
+      ensure
+        @store.type = old_coverage_type
+      end
+
       def report_coverage
         @semaphore.synchronize do
           raise "no Coverband store set" unless @store
 
           files_with_line_usage = filtered_files(Delta.results)
-          @store.save_report(files_with_line_usage)
+          if @store.type == Coverband::EAGER_TYPE && Coverband.configuration.defer_eager_loading_data?
+            @deferred_eager_loading_data = files_with_line_usage
+          else
+            if @deferred_eager_loading_data && Coverband.configuration.defer_eager_loading_data?
+              toggle_eager_loading do
+                @store.save_report(@deferred_eager_loading_data)
+                @deferred_eager_loading_data = nil
+              end
+            end
+            @store.save_report(files_with_line_usage)
+          end
         end
       rescue => e
         @logger&.error "coverage failed to store"

data/lib/coverband/collectors/view_tracker.rb

@@ -1,5 +1,6 @@
 # frozen_string_literal: true
 
+require "set"
 require "singleton"
 
 module Coverband
@@ -12,7 +13,7 @@ module Coverband
     # https://github.com/livingsocial/flatfoot
     ###
     class ViewTracker
-      attr_accessor :target, :logged_views, :views_to_record
+      attr_accessor :target
       attr_reader :logger, :roots, :store, :ignore_patterns
 
       def initialize(options = {})
@@ -29,8 +30,16 @@ module Coverband
         @roots = @roots.split(",") if @roots.is_a?(String)
         @one_time_timestamp = false
 
-        @logged_views = []
-        @views_to_record = []
+        @logged_views = Set.new
+        @views_to_record = Set.new
+      end
+
+      def logged_views
+        @logged_views.to_a
+      end
+
+      def views_to_record
+        @views_to_record.to_a
       end
 
       ###
@@ -40,8 +49,8 @@ module Coverband
       def track_views(_name, _start, _finish, _id, payload)
         if (file = payload[:identifier])
           if newly_seen_file?(file)
-            logged_views << file
-            views_to_record << file if track_file?(file)
+            @logged_views << file
+            @views_to_record << file if track_file?(file)
           end
         end
 
@@ -54,8 +63,8 @@ module Coverband
         return unless (layout_file = payload[:layout])
         return unless newly_seen_file?(layout_file)
 
-        logged_views << layout_file
-        views_to_record << layout_file if track_file?(layout_file, layout: true)
+        @logged_views << layout_file
+        @views_to_record << layout_file if track_file?(layout_file, layout: true)
       end
 
       def used_views
@@ -81,16 +90,17 @@ module Coverband
         all_views.uniq
       end
 
-      def unused_views
-        recently_used_views = used_views.keys
-        unused_views = all_views.reject { |view| recently_used_views.include?(view) }
+      def unused_views(used_views = nil)
+        recently_used_views = (used_views || self.used_views).keys
+        unused_views = all_views - recently_used_views
         # since layouts don't include format we count them used if they match with ANY formats
         unused_views.reject { |view| view.match(/\/layouts\//) && recently_used_views.any? { |used_view| view.include?(used_view) } }
       end
 
       def as_json
+        used_views = self.used_views
         {
-          unused_views: unused_views,
+          unused_views: unused_views(used_views),
           used_views: used_views
         }.to_json
       end
@@ -113,17 +123,17 @@ module Coverband
 
         filename = "#{@project_directory}/#{filename}"
         redis_store.hdel(tracker_key, filename)
-        logged_views.delete(filename)
+        @logged_views.delete(filename)
       end
 
       def report_views_tracked
         redis_store.set(tracker_time_key, Time.now.to_i) unless @one_time_timestamp || tracker_time_key_exists?
         @one_time_timestamp = true
         reported_time = Time.now.to_i
-        views_to_record.each do |file|
+        @views_to_record.each do |file|
           redis_store.hset(tracker_key, file, reported_time)
         end
-        self.views_to_record = []
+        @views_to_record.clear
       rescue => e
         # we don't want to raise errors if Coverband can't reach redis.
         # This is a nice to have not a bring the system down
@@ -137,15 +147,12 @@ module Coverband
       protected
 
       def newly_seen_file?(file)
-        return false if logged_views.include?(file)
-
-        true
+        !@logged_views.include?(file)
       end
 
       def track_file?(file, options = {})
-        @ignore_patterns.none? do |pattern|
-          file.include?(pattern)
-        end && (file.start_with?(@project_directory) || options[:layout])
+        (file.start_with?(@project_directory) || options[:layout]) &&
+          @ignore_patterns.none? { |pattern| file.include?(pattern) }
       end
 
       private

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,7 +244,13 @@ 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?
+      return false unless service?
+
       (coverband_env == "test" && !Coverband.configuration.service_test_mode) ||
         (coverband_env == "development" && !Coverband.configuration.service_dev_mode)
     end

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/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/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'
+  VERSION = '5.2.0'
 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/forked/rails_full_stack_test.rb

@@ -58,7 +58,7 @@ class RailsFullStackTest < Minitest::Test
 
       # we don't want this to run during our standard test suite
       # as the below profiler changes the runtime
-      # and shold only be included for isolated processes
+      # and should only be included for isolated processes
       begin
         require "memory_profiler"
 

data/test/integration/full_stack_deferred_eager_test.rb

@@ -0,0 +1,51 @@
+# frozen_string_literal: true
+
+require File.expand_path("../test_helper", File.dirname(__FILE__))
+require "rack"
+
+class FullStackDeferredEagerTest < Minitest::Test
+  REDIS_STORAGE_FORMAT_VERSION = Coverband::Adapters::RedisStore::REDIS_STORAGE_FORMAT_VERSION
+  TEST_RACK_APP = "../fake_app/basic_rack.rb"
+
+  def setup
+    super
+    Coverband::Collectors::Coverage.instance.reset_instance
+    Coverband.configure do |config|
+      config.background_reporting_enabled = false
+      config.track_gems = true
+      config.defer_eager_loading_data = true
+    end
+    Coverband.start
+    Coverband::Collectors::Coverage.instance.eager_loading!
+    @rack_file = require_unique_file "fake_app/basic_rack.rb"
+    Coverband.report_coverage
+    Coverband::Collectors::Coverage.instance.runtime!
+  end
+
+  test "call app" do
+    # eager loaded class coverage starts empty
+    Coverband.eager_loading_coverage!
+    expected = {}
+    assert_equal expected, Coverband.configuration.store.coverage
+
+    Coverband::Collectors::Coverage.instance.runtime!
+    request = Rack::MockRequest.env_for("/anything.json")
+    middleware = Coverband::BackgroundMiddleware.new(fake_app_with_lines)
+    results = middleware.call(request)
+    assert_equal "Hello Rack!", results.last
+    Coverband.report_coverage
+    expected = [nil, nil, 0, nil, 0, 0, 1, nil, nil]
+    assert_equal expected, Coverband.configuration.store.coverage[@rack_file]["data"]
+
+    # eager loaded class coverage is saved at first normal coverage report
+    Coverband.eager_loading_coverage!
+    expected = [nil, nil, 1, nil, 1, 1, 0, nil, nil]
+    assert_equal expected, Coverband.configuration.store.coverage[@rack_file]["data"]
+  end
+
+  private
+
+  def fake_app_with_lines
+    @fake_app_with_lines ||= ::HelloWorld.new
+  end
+end