coverband 6.1.2 → 6.1.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f23312d78597d5a4ccd34a6bd5284f384430ec78a4030ea4cd14e91a54d0226c
-  data.tar.gz: d52ca89becf12d1f2078ce951feda318415494fc2f1fe18a67b3b2514929b91a
+  metadata.gz: a3ffdf0c9a4cefbcbc728967f36ed20a90755074864a2fe858859b739e6912bb
+  data.tar.gz: e853e687820d45bbf414f51efd3a51f996b05f5eff0dcecac95351b076b3300d
 SHA512:
-  metadata.gz: 2cdc25f77daf7adaf94b8010626b4292492dc8660bdd7ab062237cc1fa2eda9f145baaa91e20f18c4e64255285ecc0c471cb6a9d02e073184bcb4857ed342ec2
-  data.tar.gz: 8f7ea9062243e7e7d4fc304ade85e7fe21f4dc93c55140a0640242970af2638374626121f78aa4b07ae0f7ce47af8659d936a0a76b7029d8685d35a5effff21d
+  metadata.gz: 3f2f69c9de574808771f002af9fc4903fed8ec9af0d52288457759ff21b1c8a3c0e42111a6ea59c8830893fc57acf05e300f8520903dda920a894419710ad386
+  data.tar.gz: 20073041f076062cefeeeb8bcec74d703086d70c86c6f7b0b2511474f5c0cd2b9cb61510245cca49104dff53a8979249a2b4dae2bfad85b09682c9dd56ce5628

data/.github/workflows/main.yml

@@ -35,3 +35,4 @@ jobs:
           bundler-cache: true # runs 'bundle install' and caches installed gems automatically
       - run: bundle exec standardrb
       - run: bundle exec rake
+      - run: "RUBYOPT='--enable=frozen-string-literal --debug=frozen-string-literal' bundle exec rake"

data/.rubocop.yml

@@ -0,0 +1,5 @@
+require: standard
+
+inherit_gem:
+  standard:
+    - config/ruby-2.7.yml

data/README.md

@@ -204,10 +204,10 @@ end
 
 Do you use figaro, mc-settings, dotenv or something else to inject environment variables into your app? If so ensure you have that done BEFORE Coverband is required.
 
-For example if you use dotenv, you need to do this, see https://github.com/bkeepers/dotenv#note-on-load-order
+For example if you use dotenv, you need to do this, see https://github.com/bkeepers/dotenv#load-order
 
 ```
-gem 'dotenv-rails', require: 'dotenv/load'
+gem 'dotenv', require: 'dotenv/load'
 gem 'coverband'
 gem 'other-gem-that-requires-env-variables'
 ```
@@ -244,7 +244,7 @@ Coverband provides a view of all of its current settings. Sometimes you might wa
 such as when sharing coverband data with a large number of developers of varying trust levels.
 You can disable the settings view like so:
 
-`config.hide_settings = false`
+`config.hide_settings = true`
 
 ### Fixing Coverage Only Shows Loading Hits
 
@@ -299,11 +299,6 @@ Coverband on very high volume sites with many server processes reporting can hav
 
 See more discussion [here](https://github.com/danmayer/coverband/issues/384).
 
-Please note that with the Redis Hash Store, everytime you load the full report, Coverband will execute `HGETALL` queries in your Redis server twice for every file in the project (once for runtime coverage and once for eager loading coverage). This shouldn't have a big impact in small to medium projects, but can be quite a hassle if your project has a few thousand files.
-To help reduce the extra redis load when getting the coverage report, you can enable `get_coverage_cache` (but note that when doing that, you will always get a previous version of the report, while a cache is re-populated with a newer version).
-
-- Use Hash Redis Store with _get coverage cache_: `config.store = Coverband::Adapters::HashRedisStore.new(redis, get_coverage_cache: true)`
-
 ### Clear Coverage
 
 Now that Coverband uses MD5 hashes there should be no reason to manually clear coverage unless one is testing, changing versions, or possibly debugging Coverband itself.
@@ -312,13 +307,10 @@ Now that Coverband uses MD5 hashes there should be no reason to manually clear c
 
 This can also be done through the web if `config.web_enable_clear` is enabled.
 
-### Coverage Data Migration
-
-Between the release of 4.0 and 4.1 our data format changed. This resets all your coverage data. If you want to restore your previous coverage data, feel free to migrate.
-
-`rake coverband:migrate`
+**NOTE**: The previous task does not clear the trackers data (views, routes, translations, etc).
+To clear trackers data, run
 
-- We will be working to support migrations going forward, when possible
+`rake coverband:clear_tracker`
 
 ### Adding Rake Tasks outside of Rails
 

data/changes.md

@@ -1,3 +1,24 @@
+### Coverband 6.1.4
+
+* fix on ignoring to many view files thx @fatkodima
+* improve docs on clear
+
+### Coverband 6.1.3
+
+* memory optimizations thx @fatkodima
+* drop support for Ruby less than 2.7 thx @fatkodima
+* improved 0 missed styles thx @fatkodima
+* improved sidekiq swarm support thx @fatkodima
+* default include memcached adapter thx @colemanja91
+* improved long filename web admin views thx @fatkodima
+* reduce logging noise thx @jamiecobbett
+* improved readme thx @hotoolong
+* fix for ignoring views @thijsnado 
+* improved documentation @jjb
+* add back static output summary @bessey
+
+other cleanup, small fixes, and updated mostly basic maintence, by me.
+
 ### Coverband 6.1.2
 
 * Fix for paging that would pull empty pages after getting all the data.

data/coverband.gemspec

@@ -38,22 +38,23 @@ Gem::Specification.new do |spec|
   # note: we are also adding 'spy' as mocha doesn't want us to spy on redis calls...
   spec.add_development_dependency "spy"
   # ^^^ probably need a large test cleanup refactor
-  spec.add_development_dependency "minitest", "= 5.18.1"
+  spec.add_development_dependency "minitest"
   spec.add_development_dependency "minitest-fork_executor"
   spec.add_development_dependency "minitest-stub-const"
-  spec.add_development_dependency "mocha", "~> 1.7.0"
+  spec.add_development_dependency "mocha"
   spec.add_development_dependency "rack"
   spec.add_development_dependency "rack-test"
   spec.add_development_dependency "rake"
   spec.add_development_dependency "resque"
-  spec.add_development_dependency "standard", "= 1.34.0"
+  spec.add_development_dependency "standard", ">= 1.35.1"
   # breaking changes in various rubocop versions
-  spec.add_development_dependency "rubocop", "= 1.60.0"
+  spec.add_development_dependency "rubocop"
 
   spec.add_development_dependency "coveralls"
   # minitest-profile is not compatible with Rails 7.1.0 setup... dropping it for now
   # spec.add_development_dependency "minitest-profile"
   spec.add_development_dependency "webmock"
+  spec.add_development_dependency "dalli" # Default memcached adapter
 
   # TODO: Remove when other production adapters exist
   # because the default configuration of redis store, we really do require

data/lib/coverband/adapters/base.rb

@@ -23,10 +23,6 @@ module Coverband
         raise ABSTRACT_KEY
       end
 
-      def migrate!
-        raise ABSTRACT_KEY
-      end
-
       def size
         raise ABSTRACT_KEY
       end
@@ -69,21 +65,8 @@ module Coverband
 
       def split_coverage(types, coverage_cache, options = {})
         types.reduce({}) do |data, type|
-          if type == Coverband::RUNTIME_TYPE && Coverband.configuration.simulate_oneshot_lines_coverage
-            data.update(type => coverage_cache[type] ||= simulated_runtime_coverage)
-          else
-            data.update(type => coverage_cache[type] ||= coverage(type, options))
-          end
-        end
-      end
-
-      def simulated_runtime_coverage
-        runtime_data = coverage(Coverband::RUNTIME_TYPE)
-        eager_data = coverage(Coverband::EAGER_TYPE)
-        eager_data.values do |vals|
-          vals["data"].map! { |line_coverage| line_coverage ? (0 - line_coverage) : line_coverage }
+          data.update(type => coverage_cache[type] ||= coverage(type, options))
         end
-        merge_reports(runtime_data, eager_data, skip_expansion: true)
       end
 
       def merged_coverage(types, coverage_cache)
@@ -142,12 +125,12 @@ module Coverband
         }
       end
 
-      # TODO: This should only be 2 cases get our dup / not dups aligned
+      # TODO: This should have cases reduced
       def array_add(latest, original)
-        if Coverband.configuration.use_oneshot_lines_coverage
+        if latest.empty? && original.empty?
+          []
+        elsif Coverband.configuration.use_oneshot_lines_coverage
           latest.map!.with_index { |v, i| ((v + original[i] >= 1) ? 1 : 0) if v && original[i] }
-        elsif Coverband.configuration.simulate_oneshot_lines_coverage
-          latest.map.with_index { |v, i| ((v + original[i] >= 1) ? 1 : 0) if v && original[i] }
         else
           latest.map.with_index { |v, i| (v && original[i]) ? v + original[i] : nil }
         end

data/lib/coverband/adapters/file_store.rb

@@ -48,10 +48,6 @@ module Coverband
         File.size?(path).to_i
       end
 
-      def migrate!
-        raise NotImplementedError, "FileStore doesn't support migrations"
-      end
-
       def coverage(_local_type = nil, opts = {})
         if merge_mode
           data = {}

data/lib/coverband/adapters/hash_redis_store.rb

@@ -5,82 +5,6 @@ require "securerandom"
 module Coverband
   module Adapters
     class HashRedisStore < Base
-      class GetCoverageNullCacheStore
-        def self.clear!(*_local_types)
-        end
-
-        def self.fetch(_local_type)
-          yield(0)
-        end
-      end
-
-      class GetCoverageRedisCacheStore
-        LOCK_LIMIT = 60 * 30 # 30 minutes
-
-        def initialize(redis, key_prefix)
-          @redis = redis
-          @key_prefix = [key_prefix, "get-coverage"].join(".")
-        end
-
-        def fetch(local_type)
-          cached_result = get(local_type)
-
-          # if no cache available, block the call and populate the cache
-          # if cache is available, return it and start re-populating it (with a lock)
-          if cached_result.nil?
-            value = yield(0)
-            result = set(local_type, JSON.generate(value))
-            value
-          else
-            if lock!(local_type)
-              Thread.new do
-                result = yield(deferred_time)
-                set(local_type, JSON.generate(result))
-              ensure
-                unlock!(local_type)
-              end
-            end
-            JSON.parse(cached_result)
-          end
-        end
-
-        def clear!(local_types = Coverband::TYPES)
-          Array(local_types).each do |local_type|
-            del(local_type)
-            unlock!(local_type)
-          end
-        end
-
-        private
-
-        # sleep in between to avoid holding other redis commands..
-        # with a small random offset so runtime and eager types can be processed "at the same time"
-        def deferred_time
-          rand(2.0..3.0)
-        end
-
-        def del(local_type)
-          @redis.del("#{@key_prefix}.cache.#{local_type}")
-        end
-
-        def get(local_type)
-          @redis.get("#{@key_prefix}.cache.#{local_type}")
-        end
-
-        def set(local_type, value)
-          @redis.set("#{@key_prefix}.cache.#{local_type}", value)
-        end
-
-        # lock for at most 60 minutes
-        def lock!(local_type)
-          @redis.set("#{@key_prefix}.lock.#{local_type}", "1", nx: true, ex: LOCK_LIMIT)
-        end
-
-        def unlock!(local_type)
-          @redis.del("#{@key_prefix}.lock.#{local_type}")
-        end
-      end
-
       FILE_KEY = "file"
       FILE_LENGTH_KEY = "file_length"
       META_DATA_KEYS = [DATA_KEY, FIRST_UPDATED_KEY, LAST_UPDATED_KEY, FILE_HASH].freeze
@@ -93,7 +17,7 @@ module Coverband
 
       JSON_PAYLOAD_EXPIRATION = 5 * 60
 
-      attr_reader :redis_namespace, :get_coverage_cache
+      attr_reader :redis_namespace
 
       def initialize(redis, opts = {})
         super()
@@ -105,13 +29,6 @@ module Coverband
 
         @ttl = opts[:ttl]
         @relative_file_converter = opts[:relative_file_converter] || Utils::RelativeFileConverter
-
-        @get_coverage_cache = if opts[:get_coverage_cache]
-          key_prefix = [REDIS_STORAGE_FORMAT_VERSION, @redis_namespace].compact.join(".")
-          GetCoverageRedisCacheStore.new(redis, key_prefix)
-        else
-          GetCoverageNullCacheStore
-        end
       end
 
       def supported?
@@ -129,7 +46,6 @@ module Coverband
           @redis.del(*file_keys) if file_keys.any?
           @redis.del(files_key)
           @redis.del(files_key(type))
-          @get_coverage_cache.clear!(type)
         end
         self.type = old_type
       end
@@ -139,7 +55,6 @@ module Coverband
         relative_path_file = @relative_file_converter.convert(file)
         Coverband::TYPES.each do |type|
           @redis.del(key(relative_path_file, type, file_hash: file_hash))
-          @get_coverage_cache.clear!(type)
         end
         @redis.srem(files_key, relative_path_file)
       end
@@ -176,30 +91,29 @@ module Coverband
       # When paging code should use coverage_for_types and pull eager and runtime together as matched pairs
       def coverage(local_type = nil, opts = {})
         page_size = opts[:page_size] || 250
-        cached_results = @get_coverage_cache.fetch(local_type || type) do |sleep_time|
-          files_set = if opts[:page]
-            raise "call coverage_for_types with paging"
-          elsif opts[:filename]
-            type_key_prefix = key_prefix(local_type)
-            # NOTE: a better way to extract filename from key would be better
-            files_set(local_type).select do |cache_key|
-              cache_key.sub(type_key_prefix, "").match(short_name(opts[:filename]))
-            end || {}
-          else
-            files_set(local_type)
-          end
-          # below uses batches with a sleep in between to avoid overloading redis
-          files_set.each_slice(page_size).flat_map do |key_batch|
-            sleep sleep_time
-            @redis.pipelined do |pipeline|
-              key_batch.each do |key|
-                pipeline.hgetall(key)
-              end
+        files_set = if opts[:page]
+          raise "call coverage_for_types with paging"
+        elsif opts[:filename]
+          type_key_prefix = key_prefix(local_type)
+          # NOTE: a better way to extract filename from key would be better
+          files_set(local_type).select do |cache_key|
+            cache_key.sub(type_key_prefix, "").match(short_name(opts[:filename]))
+          end || {}
+        else
+          files_set(local_type)
+        end
+
+        # below uses batches with a sleep in between to avoid overloading redis
+        files_set = files_set.each_slice(page_size).flat_map do |key_batch|
+          sleep(0.01 * rand(1..10))
+          @redis.pipelined do |pipeline|
+            key_batch.each do |key|
+              pipeline.hgetall(key)
             end
           end
         end
 
-        cached_results.each_with_object({}) do |data_from_redis, hash|
+        files_set.each_with_object({}) do |data_from_redis, hash|
           add_coverage_for_file(data_from_redis, hash)
         end
       end

data/lib/coverband/adapters/memcached_store.rb

@@ -36,10 +36,6 @@ module Coverband
         @memcached.read(base_key) ? @memcached.read(base_key).bytesize : "N/A"
       end
 
-      def migrate!
-        raise NotImplementedError, "MemcachedStore doesn't support migrations"
-      end
-
       def type=(type)
         super
         reset_base_key

data/lib/coverband/adapters/null_store.rb

@@ -22,10 +22,6 @@ module Coverband
         0
       end
 
-      def migrate!
-        raise NotImplementedError, "NullStore doesn't support migrations"
-      end
-
       def coverage(_local_type = nil, opts = {})
         {}
       end

data/lib/coverband/adapters/redis_store.rb

@@ -45,28 +45,6 @@ module Coverband
         @redis.get(base_key) ? @redis.get(base_key).bytesize : "N/A"
       end
 
-      ###
-      # Current implementation moves from coverband3_1 to coverband_3_2
-      # In the future this can be made more general and support a more specific
-      # version format.
-      ###
-      def migrate!
-        reset_base_key
-        @format_version = "coverband3_1"
-        previous_data = coverage
-        if previous_data.empty?
-          puts "no previous data to migrate found"
-          exit 0
-        end
-        relative_path_report = previous_data.each_with_object({}) { |(key, vals), fixed_report|
-          fixed_report[Utils::RelativeFileConverter.convert(key)] = vals
-        }
-        clear!
-        reset_base_key
-        @format_version = REDIS_STORAGE_FORMAT_VERSION
-        save_coverage(merge_reports(coverage, relative_path_report, skip_expansion: true))
-      end
-
       def type=(type)
         super
         reset_base_key
@@ -94,6 +72,15 @@ module Coverband
         @redis
       end
 
+      def file_count
+        data = redis.get type_base_key(Coverband::RUNTIME_TYPE)
+        JSON.parse(data).keys.length
+      end
+
+      def cached_file_count
+        @cached_file_count ||= file_count
+      end
+
       private
 
       attr_reader :redis

data/lib/coverband/adapters/stdout_store.rb

@@ -21,10 +21,6 @@ module Coverband
         0
       end
 
-      def migrate!
-        raise NotImplementedError, "StdoutStore doesn't support migrations"
-      end
-
       def coverage(_local_type = nil, opts = {})
         {}
       end

data/lib/coverband/collectors/abstract_tracker.rb

@@ -127,7 +127,8 @@ module Coverband
       end
 
       def track_key?(key, options = {})
-        @ignore_patterns.none? { |pattern| key.to_s.include?(pattern) }
+        key = key.to_s
+        @ignore_patterns.none? { |pattern| key.match?(pattern) }
       end
 
       private
@@ -157,7 +158,11 @@ module Coverband
       end
 
       def class_key
-        @class_key ||= self.class.name.split("::").last
+        @class_key ||= if Coverband.configuration.redis_namespace
+          "#{Coverband.configuration.redis_namespace}_#{self.class.name.split("::").last}"
+        else
+          self.class.name.split("::").last
+        end
       end
     end
   end

data/lib/coverband/collectors/coverage.rb

@@ -14,10 +14,6 @@ module Coverband
     class Coverage
       include Singleton
 
-      def self.ruby_version_greater_than_or_equal_to?(version)
-        Gem::Version.new(RUBY_VERSION) >= Gem::Version.new(version)
-      end
-
       def reset_instance
         @project_directory = File.expand_path(Coverband.configuration.root)
         @ignore_patterns = Coverband.configuration.ignore
@@ -87,7 +83,6 @@ module Coverband
 
       def initialize
         @semaphore = Mutex.new
-        raise NotImplementedError, "Coverage needs Ruby > 2.3.0" if Gem::Version.new(RUBY_VERSION) < Gem::Version.new("2.3.0")
 
         require "coverage"
         if RUBY_PLATFORM == "java"
@@ -98,20 +93,14 @@ module Coverband
         if defined?(SimpleCov) && defined?(Rails) && defined?(Rails.env) && Rails.env.test?
           puts "Coverband: detected SimpleCov in test Env, allowing it to start Coverage"
           puts "Coverband: to ensure no error logs or missing Coverage call `SimpleCov.start` prior to requiring Coverband"
-        else
-          if ::Coverage.respond_to?(:state)
-            if ::Coverage.state == :idle
-              ::Coverage.start(oneshot_lines: Coverband.configuration.use_oneshot_lines_coverage)
-            elsif ::Coverage.state == :suspended
-              ::Coverage.resume
-            end
-          elsif Coverage.ruby_version_greater_than_or_equal_to?("2.6.0")
-            ::Coverage.start(oneshot_lines: Coverband.configuration.use_oneshot_lines_coverage) unless ::Coverage.running?
-          elsif Coverage.ruby_version_greater_than_or_equal_to?("2.5.0")
-            ::Coverage.start unless ::Coverage.running?
-          else
-            ::Coverage.start
+        elsif ::Coverage.respond_to?(:state)
+          if ::Coverage.state == :idle
+            ::Coverage.start(oneshot_lines: Coverband.configuration.use_oneshot_lines_coverage)
+          elsif ::Coverage.state == :suspended
+            ::Coverage.resume
           end
+        else
+          ::Coverage.start(oneshot_lines: Coverband.configuration.use_oneshot_lines_coverage) unless ::Coverage.running?
         end
         reset_instance
       end

data/lib/coverband/collectors/delta.rb

@@ -32,7 +32,7 @@ module Coverband
           transform_oneshot_lines_results(current_coverage)
         else
           new_results = generate
-          @@previous_coverage = current_coverage unless Coverband.configuration.simulate_oneshot_lines_coverage
+          @@previous_coverage = current_coverage
           new_results
         end
       end

data/lib/coverband/collectors/view_tracker.rb

@@ -17,6 +17,7 @@ module Coverband
 
       REPORT_ROUTE = "views_tracker"
       TITLE = "Views"
+      VIEWS_PATTERNS = %w[.erb$ .haml$ .slim$]
 
       def initialize(options = {})
         @project_directory = File.expand_path(Coverband.configuration.root)
@@ -24,6 +25,8 @@ module Coverband
         @roots = @roots.split(",") if @roots.is_a?(String)
 
         super
+
+        @ignore_patterns -= VIEWS_PATTERNS.map { |ignore_str| Regexp.new(ignore_str) }
       end
 
       def railtie!
@@ -84,7 +87,8 @@ module Coverband
         recently_used_views = used_keys.keys
         unused_views = all_keys - 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) } }
+        unused_views = unused_views.reject { |view| view.include?("/layouts/") && recently_used_views.any? { |used_view| view.include?(used_view) } }
+        unused_views.reject { |view| @ignore_patterns.any? { |pattern| view.match?(pattern) } }
       end
 
       def clear_key!(filename)
@@ -99,7 +103,7 @@ module Coverband
 
       def track_file?(file, options = {})
         (file.start_with?(@project_directory) || options[:layout]) &&
-          @ignore_patterns.none? { |pattern| file.include?(pattern) }
+          @ignore_patterns.none? { |pattern| file.match?(pattern) }
       end
 
       def concrete_target

data/lib/coverband/configuration.rb

@@ -10,7 +10,7 @@ module Coverband
       :reporter, :redis_namespace, :redis_ttl,
       :background_reporting_enabled,
       :test_env, :web_enable_clear, :gem_details, :web_debug, :report_on_exit,
-      :simulate_oneshot_lines_coverage,
+      :use_oneshot_lines_coverage, :simulate_oneshot_lines_coverage,
       :view_tracker, :defer_eager_loading_data,
       :track_routes, :track_redirect_routes, :route_tracker,
       :track_translations, :translations_tracker,
@@ -22,7 +22,7 @@ module Coverband
       :background_reporting_sleep_seconds, :reporting_wiggle,
       :send_deferred_eager_loading_data, :paged_reporting
 
-    attr_reader :track_gems, :ignore, :use_oneshot_lines_coverage
+    attr_reader :track_gems, :ignore
 
     #####
     # TODO: This is is brittle and not a great solution to avoid deploy time
@@ -33,7 +33,6 @@ module Coverband
     IGNORE_TASKS = ["coverband:clear",
                     "coverband:coverage",
                     "coverband:coverage_server",
-                    "coverband:migrate",
                     "assets:precompile",
                     "webpacker:compile",
                     "db:version",
@@ -49,7 +48,7 @@ module Coverband
     # Heroku when building assets runs code from a dynamic directory
     # /tmp was added to avoid coverage from /tmp/build directories during
     # heroku asset compilation
-    IGNORE_DEFAULTS = %w[vendor/ .erb$ .slim$ /tmp internal:prelude db/schema.rb]
+    IGNORE_DEFAULTS = %w[vendor/ /tmp internal:prelude db/schema.rb] + Collectors::ViewTracker::VIEWS_PATTERNS
 
     # Add in missing files which were never loaded
     # we need to know what all paths to check for unloaded files
@@ -62,7 +61,7 @@ module Coverband
     def reset
       @root = Dir.pwd
       @root_paths = []
-      @ignore = IGNORE_DEFAULTS.dup
+      @ignore = IGNORE_DEFAULTS.map { |ignore_str| Regexp.new(ignore_str) }
       @search_paths = TRACKED_DEFAULT_PATHS.dup
       @verbose = false
       @reporter = "scov"
@@ -84,7 +83,7 @@ module Coverband
       @web_debug = false
       @report_on_exit = true
       @use_oneshot_lines_coverage = ENV["ONESHOT"] || false
-      @simulate_oneshot_lines_coverage = ENV["SIMULATE_ONESHOT"] || false
+      @simulate_oneshot_lines_coverage = false # this is being deprecated
       @current_root = nil
       @all_root_paths = nil
       @all_root_patterns = nil
@@ -223,8 +222,8 @@ module Coverband
     # Don't allow the ignore to override things like gem tracking
     ###
     def ignore=(ignored_array)
-      ignored_array.map { |ignore_str| Regexp.new(ignore_str) }
-      @ignore = (@ignore + ignored_array).uniq
+      ignored_array = ignored_array.map { |ignore_str| Regexp.new(ignore_str) }
+      @ignore |= ignored_array
     rescue RegexpError
       logger.error "an invalid regular expression was passed in, ensure string are valid regex patterns #{ignored_array.join(",")}"
     end
@@ -253,19 +252,6 @@ module Coverband
         end
     end
 
-    def use_oneshot_lines_coverage=(value)
-      unless one_shot_coverage_implemented_in_ruby_version? || !value
-        raise(StandardError,
-          "One shot line coverage is only available in ruby >= 2.6")
-      end
-
-      @use_oneshot_lines_coverage = value
-    end
-
-    def one_shot_coverage_implemented_in_ruby_version?
-      Gem::Version.new(RUBY_VERSION) >= Gem::Version.new("2.6.0")
-    end
-
     def redis_url
       @redis_url ||= ENV["COVERBAND_REDIS_URL"] || ENV["REDIS_URL"]
     end

data/lib/coverband/integrations/background.rb

@@ -30,6 +30,8 @@ module Coverband
         logger.debug("Coverband: Starting background reporting") if Coverband.configuration.verbose
         sleep_seconds = Coverband.configuration.background_reporting_sleep_seconds.to_i
         @thread = Thread.new {
+          Thread.current.name = "Coverband Background Reporter"
+
           loop do
             if Coverband.configuration.reporting_wiggle
               sleep_seconds = Coverband.configuration.background_reporting_sleep_seconds.to_i + rand(Coverband.configuration.reporting_wiggle.to_i)

data/lib/coverband/integrations/sidekiq_swarm.rb

@@ -0,0 +1,8 @@
+# frozen_string_literal: true
+
+Sidekiq.configure_server do |config|
+  config.on(:fork) do
+    Coverband.start
+    Coverband.runtime_coverage!
+  end
+end