coverband 6.1.4 → 6.1.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a3ffdf0c9a4cefbcbc728967f36ed20a90755074864a2fe858859b739e6912bb
-  data.tar.gz: e853e687820d45bbf414f51efd3a51f996b05f5eff0dcecac95351b076b3300d
+  metadata.gz: 032cf0d003aefa2d222472e12412e352f5059cfffc93645810692e60b3dc6602
+  data.tar.gz: b094364d6cc2dbfbc2527209f4bf03025c06199443a0b9fa0380851afe7a00ae
 SHA512:
-  metadata.gz: 3f2f69c9de574808771f002af9fc4903fed8ec9af0d52288457759ff21b1c8a3c0e42111a6ea59c8830893fc57acf05e300f8520903dda920a894419710ad386
-  data.tar.gz: 20073041f076062cefeeeb8bcec74d703086d70c86c6f7b0b2511474f5c0cd2b9cb61510245cca49104dff53a8979249a2b4dae2bfad85b09682c9dd56ce5628
+  metadata.gz: 14ce46ede0f55f2162878529c8226a022518df0a0a22c83ad323d0856fad43b99a9f694b3b629171b86872bfd16626da9772388cae5a665a5e00e1493ef81b8b
+  data.tar.gz: c394482fdfddfa2d70dab2453517f2c490e66e1b9ef463a5a097413deb41540158a13a0626a2f3c4e62fc263de3052df6d99c7c7765ddb9ae867074b5b943e94

data/.github/workflows/main.yml

@@ -19,10 +19,14 @@ jobs:
         # truffleruby,
         # truffleruby-head,
         # removing jruby again to flaky
-        gemfile: [ Gemfile.rails6.0, Gemfile.rails6.1, Gemfile.rails7.0, Gemfile.rails7.1 ]
+        gemfile: [ Gemfile.rails7.0, Gemfile.rails7.1, Gemfile.rails7.2, Gemfile.rails8.0 ]
         # need to add support for multiple gemfiles
-        ruby: ["2.7", "3.0", "3.1", "3.2", "3.3"]
+        ruby: ["3.1", "3.2", "3.3", "3.4", "ruby-head"]
         redis-version: [4, 5, 6, 7]
+        exclude:
+          # Rails 8 requires ruby 3.2+.
+          - gemfile: 'rails_8.0'
+            ruby: '3.1'
     runs-on: ${{ matrix.os }}-latest
     steps:
       - uses: actions/checkout@v4
@@ -33,6 +37,14 @@ jobs:
         with:
           ruby-version: ${{ matrix.ruby }}
           bundler-cache: true # runs 'bundle install' and caches installed gems automatically
-      - run: bundle exec standardrb
-      - run: bundle exec rake
+      - run: bundle exec rake test:all
       - run: "RUBYOPT='--enable=frozen-string-literal --debug=frozen-string-literal' bundle exec rake"
+  starndardrb:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+      - uses: ruby/setup-ruby@v1
+        with:
+          ruby-version: 3.1
+          bundler-cache: true # runs 'bundle install' and caches installed gems automatically
+      - run: bundle exec standardrb --format github

data/.gitignore

@@ -24,4 +24,6 @@ temp_results
 .env
 log/
 test/unique_files
-test/rails5_dummy/tmp
+test/rails7_dummy/tmp
+CLAUDE.md
+.claude/

data/.rubocop.yml

@@ -2,4 +2,4 @@ require: standard
 
 inherit_gem:
   standard:
-    - config/ruby-2.7.yml
+    - config/ruby-3.1.yml

data/.standard.yml

@@ -1,4 +1,4 @@
-ruby_version: 2.7
+ruby_version: 3.1
 fix: false # default: false
 parallel: true # default: false
 format: progress # default: Standard::Formatter

data/Gemfile

@@ -4,7 +4,7 @@ source "https://rubygems.org"
 
 # Specify your gem's dependencies in coverband.gemspec
 gemspec
-gem "rails", "~>7"
+gem "rails" # latest
 gem "haml"
 gem "slim"
 gem "webrick"

data/{Gemfile.rails7 → Gemfile.rails7.2}

@@ -4,7 +4,7 @@ source 'https://rubygems.org'
 
 # Specify your gem's dependencies in coverband.gemspec
 gemspec
-gem 'rails', '~>7'
+gem 'rails', '~>7.2.0'
 gem "haml"
 gem "slim"
-gem "webrick"
+gem "webrick"

data/{Gemfile.rails6.0 → Gemfile.rails8.0}

@@ -1,10 +1,10 @@
 # frozen_string_literal: true
 
-source "https://rubygems.org"
+source 'https://rubygems.org'
 
 # Specify your gem's dependencies in coverband.gemspec
 gemspec
-gem "rails", "~>6.0.0"
+gem 'rails', '~>8.0.0'
 gem "haml"
 gem "slim"
 gem "webrick"

data/README.md

@@ -116,6 +116,39 @@ The columns in the web UI are as follows:
 When viewing an individual file, a line of code such as a class or method definition may appear green because it is eager loaded by the application, but still not be hit at all in runtime by actual users.  
 
 ![example of a file with lines not hit at runtime](https://user-images.githubusercontent.com/96786/63541229-aa98a580-c4eb-11e9-8eb8-c004fe1369db.png)
+#### Coverage first seen and Last activity recorded
+In the context of Coverband, a file is considered relevant for monitoring if it is located within certain specified project paths. Specifically, Coverband monitors all *.rb files located in the following paths:
+
+- app/
+
+- lib/
+
+- config/
+
+This applies as long as these files are not included in the list of exclusions defined in "config.ignore".
+
+#### Coverage first seen
+This attribute shows the exact date and time when Coverband first detected the file. This detection happens when:
+
+- The file is loaded by Ruby (e.g., during application startup in Rails via autoload or require).
+
+- The file becomes relevant for monitoring, even if no lines within it have been executed yet.
+
+- By default, Coverband resets a file’s coverage data and updates its “Coverage first seen” timestamp whenever the file changes, ensuring the coverage reflects the file’s new state.
+
+#### Last activity recorded
+This attribute shows the most recent time when any line in the file was executed at runtime. It specifically reflects runtime execution and does not include the file’s load time.
+
+- When any line in the file is executed at runtime (e.g., a method is called, a route is triggered, or a conditional is evaluated), Coverband updates the “Last activity recorded” timestamp.
+
+- This only occurs when the file’s lines are actually executed. If a file is loaded but none of its lines are executed, the “Last activity recorded” timestamp will not be updated.
+
+- It helps determine if a file is actively being used in the application. If a file has no recent activity, it may be a candidate for review or removal.
+
+#### Example
+- If a file has a “Coverage first seen” timestamp but no “Last activity recorded”, this indicates that the file is being loaded but no runtime activity is occurring (e.g., it contains unused methods or classes).
+
+- If both attributes are populated and the “Last activity recorded” timestamp is recent, the file is actively used in the application.
 
 ### Mounting as a Rack App
 
@@ -305,12 +338,10 @@ Now that Coverband uses MD5 hashes there should be no reason to manually clear c
 
 `rake coverband:clear`
 
-This can also be done through the web if `config.web_enable_clear` is enabled.
-
-**NOTE**: The previous task does not clear the trackers data (views, routes, translations, etc).
-To clear trackers data, run
+This will clear both the coverage and trackers data. It can also be done through the web if `config.web_enable_clear` is enabled.
 
-`rake coverband:clear_tracker`
+To clear only coverage data, run `rake coverband:clear_coverage`.
+To clear only trackers data, run `rake coverband:clear_tracker`.
 
 ### Adding Rake Tasks outside of Rails
 
@@ -399,14 +430,14 @@ Outputs:
 
 # Prerequisites
 
-- Coverband 3.0.X+ requires Ruby 2.3+
+- Ruby 3.1+
 - Coverband currently requires Redis for production usage
 
 ### Ruby and Rails Version Support
 
 We will match Heroku & Ruby's support lifetime, supporting the last 3 major Ruby releases. For details see [supported runtimes](https://devcenter.heroku.com/articles/ruby-support#supported-runtimes).
 
-For Rails, we will follow the policy of the [Rails team maintenance policy](https://guides.rubyonrails.org/maintenance_policy.html). We officially support the last two major release versions, while providing minimal support (major bugs / security fixes) for an additional version. This means at the moment we primarily target Rails 6.x, 5.x, and will try to keep current functionality working for Rails 4.x but may release new features that do not work on that target.
+For Rails, we will follow the policy of the [Rails team maintenance policy](https://guides.rubyonrails.org/maintenance_policy.html). We officially support the last two major release versions, while providing minimal support (major bugs / security fixes) for an additional version. At the moment we primarily target Rails 7.0+.
 
 ### JRuby Support
 

data/Rakefile

@@ -8,7 +8,7 @@ RuboCop::RakeTask.new
 
 task default: %i[test]
 
-task "test:all": %i[rubocop test forked_tests benchmarks:memory benchmarks]
+task "test:all": %i[test forked_tests benchmarks:memory benchmarks]
 
 task :test
 require "rake/testtask"

data/changes.md

@@ -1,3 +1,22 @@
+### Coverband 6.1.6
+
+* Check for Rails::Engine - Stephen Wetzel
+* Support views located in components directory, as well as sub-directories like packs/app/views - Derek Kniffin
+
+### Coverband 6.1.5
+
+* add testing for ruby-head
+* drop support for older Ruby and Rails version as matching our policy
+* remove ostruct as it is no longer in ruby-head
+* fix issue requiring base64 to be added to folks gem files
+* added support for track_key method allowing for code to specifically mark things tracked
+* improvements to CI thanks ydah
+* Update README to include functionality details for "Coverage first seen" and "Last activity recorded" thanks juanarevalo
+* fixes for Rack 3 - thanks Aaron Pfeifer
+* Do not mark single-line and empty methods as dead - fatkodima
+* fix on clear_trackers rake task
+* Consider eager loaded code too when detecting dead methods - fatkodima
+
 ### Coverband 6.1.4
 
 * fix on ignoring to many view files thx @fatkodima
@@ -17,7 +36,7 @@
 * improved documentation @jjb
 * add back static output summary @bessey
 
-other cleanup, small fixes, and updated mostly basic maintence, by me.
+other cleanup, small fixes, and updated mostly basic maintenance, by me.
 
 ### Coverband 6.1.2
 
@@ -29,7 +48,7 @@ other cleanup, small fixes, and updated mostly basic maintence, by me.
 
 ### Coverband 6.1.0
 
-This release has a number of smaller fixes and improvements. It includes a sizable refactoring around the UI which should simplify improvements going forward. This release is mostly targetting large projects with 6K+ ruby files, use the new `config.paged_reporting = true` option with the HashRedisStore to enable paged reporting for large projects. The HashRedisStore now also includes the last time a line in a file was executed.
+This release has a number of smaller fixes and improvements. It includes a sizable refactoring around the UI which should simplify improvements going forward. This release is mostly targeting large projects with 6K+ ruby files, use the new `config.paged_reporting = true` option with the HashRedisStore to enable paged reporting for large projects. The HashRedisStore now also includes the last time a line in a file was executed.
 
 * Thanks to @FeLvi-zzz for the last time accessed support for the Hash Redis Store
 * Thanks to @alpaca-tc for the improvements on the route tracker
@@ -42,7 +61,7 @@ This release has a number of smaller fixes and improvements. It includes a sizab
 * This release addresses large projects and adds in paged reporting
   * to ensure even on projects with 10K+ files it can load on heroku under the 30s timeout
   * only supports HashRedis store
-  * faster UI for web UI in general which should be noticable on non paged reports
+  * faster UI for web UI in general which should be noticeable on non paged reports
   * reduce redis calls
 
 ### Coverband 6.0.2

data/coverband.gemspec

@@ -20,7 +20,7 @@ Gem::Specification.new do |spec|
   spec.executables = spec.files.grep(%r{^bin/}) { |f| File.basename(f) }
   spec.require_paths = %w[lib]
 
-  spec.required_ruby_version = ">= 2.7"
+  spec.required_ruby_version = ">= 3.1"
 
   spec.metadata = {
     "homepage_uri" => "https://github.com/danmayer/coverband",
@@ -62,4 +62,5 @@ Gem::Specification.new do |spec|
   # quality adapter, I think this is more honest about requirements and reduces confusion
   # without this there was a race condition on calling coverband configure before redis was loaded
   spec.add_runtime_dependency "redis", ">= 3.0"
+  spec.add_runtime_dependency "base64"
 end

data/lib/coverband/adapters/hash_redis_store.rb

@@ -205,9 +205,7 @@ module Coverband
 
         data = coverage_data_from_redis(data_from_redis)
         timedata = coverage_time_data_from_redis(data_from_redis)
-        hash[file] = data_from_redis.select do |meta_data_key, _value|
-          META_DATA_KEYS.include?(meta_data_key)
-        end.merge!("data" => data, "timedata" => timedata)
+        hash[file] = data_from_redis.slice(*META_DATA_KEYS).merge!("data" => data, "timedata" => timedata)
         hash[file][LAST_UPDATED_KEY] =
           (hash[file][LAST_UPDATED_KEY].nil? || hash[file][LAST_UPDATED_KEY] == "") ? nil : hash[file][LAST_UPDATED_KEY].to_i
         hash[file].merge!(LAST_UPDATED_KEY => hash[file][LAST_UPDATED_KEY],

data/lib/coverband/collectors/abstract_tracker.rb

@@ -109,7 +109,7 @@ module Coverband
 
       # This is the basic rails version supported, if there is something more unique over ride in subclass
       def self.supported_version?
-        defined?(Rails) && defined?(Rails::VERSION) && Rails::VERSION::STRING.split(".").first.to_i >= 5
+        defined?(Rails::VERSION) && Rails::VERSION::STRING.split(".").first.to_i >= 7
       end
 
       def route

data/lib/coverband/collectors/route_tracker.rb

@@ -13,7 +13,7 @@ module Coverband
       TITLE = "Routes"
 
       def initialize(options = {})
-        if Rails&.respond_to?(:version) && Gem::Version.new(Rails.version) >= Gem::Version.new("6.0.0") && Gem::Version.new(Rails.version) < Gem::Version.new("7.1.0")
+        if Rails&.respond_to?(:version) && Gem::Version.new(Rails.version) < Gem::Version.new("7.1.0")
           require_relative "../utils/rails6_ext"
         end
 
@@ -52,7 +52,7 @@ module Coverband
       end
 
       def self.supported_version?
-        defined?(Rails) && defined?(Rails::VERSION) && Rails::VERSION::STRING.split(".").first.to_i >= 6
+        defined?(Rails) && defined?(Rails::VERSION) && Rails::VERSION::STRING.split(".").first.to_i >= 7
       end
 
       def unused_keys(used_keys = nil)

data/lib/coverband/collectors/view_tracker.rb

@@ -108,7 +108,7 @@ module Coverband
 
       def concrete_target
         if defined?(Rails.application)
-          Dir.glob("#{@project_directory}/app/views/**/*.html.{erb,haml,slim}")
+          Dir.glob("#{@project_directory}/**/app/{views,components}/**/*.html.{erb,haml,slim}")
         else
           []
         end

data/lib/coverband/collectors/view_tracker_service.rb

@@ -24,7 +24,7 @@ module Coverband
       end
 
       def self.supported_version?
-        defined?(Rails) && defined?(Rails::VERSION) && Rails::VERSION::STRING.split(".").first.to_i >= 4
+        defined?(Rails::VERSION) && Rails::VERSION::STRING.split(".").first.to_i >= 7
       end
 
       private

data/lib/coverband/configuration.rb

@@ -224,8 +224,6 @@ module Coverband
     def ignore=(ignored_array)
       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
 
     def current_root

data/lib/coverband/reporters/base.rb

@@ -85,7 +85,7 @@ module Coverband
         # why do we need to merge covered files data?
         # basically because paths on machines or deployed hosts could be different, so
         # two different keys could point to the same filename or `line_key`
-        # this happens when deployment has a dynmaic path or the path change during deployment (hot code reload)
+        # this happens when deployment has a dynamic path or the path change during deployment (hot code reload)
         # TODO: think we are filtering based on ignore while sending to the store
         # and as we also pull it out here
         ###

data/lib/coverband/utils/dead_methods.rb

@@ -37,7 +37,11 @@ module Coverband
       end
 
       def self.scan_all
-        coverage = Coverband.configuration.store.coverage
+        # If the file was loaded during eager loading and then its code is never executed
+        # during runtime, then it will not have any runtime coverage. When reporting
+        # dead methods, we need to look at all the files discovered during the eager loading
+        # and runtime phases.
+        coverage = Coverband.configuration.store.get_coverage_report[Coverband::MERGED_TYPE]
         coverage.flat_map do |file_path, coverage|
           scan(file_path: file_path, coverage: coverage["data"])
         end

data/lib/coverband/utils/method_definition_scanner.rb

@@ -32,11 +32,23 @@ if defined?(RubyVM::AbstractSyntaxTree)
           private
 
           def first_line_number
-            @method_definition.first_line_number + 1
+            if multiline?
+              @method_definition.first_line_number + 1
+            else
+              @method_definition.first_line_number
+            end
           end
 
           def last_line_number
-            @method_definition.last_line_number - 1
+            if multiline?
+              @method_definition.last_line_number - 1
+            else
+              @method_definition.last_line_number
+            end
+          end
+
+          def multiline?
+            @method_definition.last_line_number - @method_definition.first_line_number > 1
           end
         end
 

data/lib/coverband/utils/railtie.rb

@@ -7,7 +7,7 @@ module Coverband
       super
     end
   end
-  Rails::Engine.prepend(RailsEagerLoad)
+  Rails::Engine.prepend(RailsEagerLoad) if defined? ::Rails::Engine
 
   class Railtie < Rails::Railtie
     initializer "coverband.configure" do |app|

data/lib/coverband/utils/tasks.rb

@@ -12,7 +12,7 @@ namespace :coverband do
     FileUtils.cp(config_template, "./config/coverband.rb")
   end
 
-  desc "console formated report of Coverband code coverage"
+  desc "console formatted report of Coverband code coverage"
   task :coverage do
     require "coverband/utils/html_formatter"
     require "coverband/utils/result"
@@ -23,7 +23,7 @@ namespace :coverband do
     Coverband::Reporters::ConsoleReport.report(Coverband.configuration.store)
   end
 
-  desc "JSON formated report of Coverband code coverage"
+  desc "JSON formatted report of Coverband code coverage"
   task :coverage_json do
     require "coverband/utils/html_formatter"
     require "coverband/utils/result"
@@ -51,7 +51,7 @@ namespace :coverband do
   # gem "simplecov-html", require: false
   # the file is written to coverage/index.html
   ###
-  desc "static HTML formated report of Coverband code coverage"
+  desc "static HTML formatted report of Coverband code coverage"
   task :coverage_html do
     require "coverband/utils/html_formatter"
     require "coverband/utils/result"
@@ -113,7 +113,16 @@ namespace :coverband do
     if Coverband.configuration.store.is_a?(Coverband::Adapters::FileStore)
       Coverband.configuration.store.merge_mode = true
     end
-    Rack::Server.start app: Coverband::Reporters::Web.new,
+
+    begin
+      require "rackup/server"
+      server_class = Rackup::Server
+    rescue LoadError
+      require "rack/server"
+      server_class = Rack::Server
+    end
+
+    server_class.start app: Coverband::Reporters::Web.new,
       Port: ENV.fetch("COVERBAND_COVERAGE_PORT", 9022).to_i
   end
 
@@ -129,10 +138,16 @@ namespace :coverband do
   end
 
   ###
-  # clear data helpful for development or after configuration issues
+  # clear all coverband data
+  ###
+  desc "reset Coverband coverage and trackers data, helpful for development, debugging, etc"
+  task clear: [:clear_coverage, :clear_tracker]
+
+  ###
+  # clear coverband coverage data
   ###
   desc "reset Coverband coverage data, helpful for development, debugging, etc"
-  task :clear do
+  task :clear_coverage do
     Coverband.configuration.store.clear!
   end
 
@@ -141,6 +156,9 @@ namespace :coverband do
   ###
   desc "reset Coverband trackers data (view, routes, translations, etc), helpful for development, debugging, etc"
   task :clear_tracker do
+    # Load rails-related trackers, if the gem is used in a rails app.
+    Coverband.configuration.railtie! if defined?(Rails::Railtie)
+
     trackers = Coverband.configuration.trackers
     trackers.each(&:reset_recordings)
   end

data/lib/coverband/version.rb

@@ -5,5 +5,5 @@
 # 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 = "6.1.4"
+  VERSION = "6.1.6"
 end

data/lib/coverband.rb

@@ -100,6 +100,32 @@ module Coverband
     coverage_instance.runtime!
   end
 
+  # Track a key with the specified tracker.
+  # @param tracker_type [Symbol] The type of tracker to use (e.g., :view_tracker, :translations_tracker, :routes_tracker)
+  # @param key [String] The key to track
+  # @return [Boolean] True if tracking was successful, false otherwise
+  # @raise [ArgumentError] If the tracker_type is not supported
+  def self.track_key(tracker_type, key)
+    return false unless key
+
+    supported_trackers = [:view_tracker, :translations_tracker, :routes_tracker]
+
+    unless supported_trackers.include?(tracker_type)
+      raise ArgumentError, "Unsupported tracker type: #{tracker_type}. Must be one of: #{supported_trackers.join(", ")}"
+    end
+
+    begin
+      tracker = configuration.send(tracker_type)
+      return false unless tracker&.respond_to?(:track_key)
+
+      tracker.track_key(key)
+      true
+    rescue => e
+      configuration.logger.error "Coverband: Failed to track key '#{key}' with tracker '#{tracker_type}'. Error: #{e.message}"
+      false
+    end
+  end
+
   private_class_method def self.coverage_instance
     Coverband::Collectors::Coverage.instance
   end

data/test/benchmarks/benchmark.rake

@@ -98,11 +98,11 @@ namespace :benchmarks do
     Coverband::Collectors::Coverage.instance.reset_instance
   end
 
-  lines = 45
-  non_nil_lines = 18
   def fake_line_numbers
+    lines = 45
+    non_nil_lines = 18
     lines.times.map do |line|
-      coverage = (line < non_nil_lines) ? rand(5) : nil
+      (line < non_nil_lines) ? rand(5) : nil
     end
   end
 
@@ -146,7 +146,7 @@ namespace :benchmarks do
       x.report("store_reports_all") { store.save_report(report) }
     end
     keys_subset = report.keys.first(100)
-    report_subset = report.select { |key, _value| keys_subset.include?(key) }
+    report_subset = report.slice(*keys_subset)
     Benchmark.ips do |x|
       x.config(time: 20, warmup: 5)
       x.report("store_reports_subset") { store.save_report(report_subset) }
@@ -155,6 +155,7 @@ namespace :benchmarks do
 
   def measure_memory
     require "memory_profiler"
+    require "stringio"
     report = fake_report
     store = benchmark_redis_store
     store.clear!
@@ -182,7 +183,7 @@ namespace :benchmarks do
 
   def measure_memory_report_coverage
     require "memory_profiler"
-    report = fake_report
+    fake_report
     store = benchmark_redis_store
     store.clear!
     mock_files(store)
@@ -249,6 +250,7 @@ namespace :benchmarks do
 
   desc "checks memory of collector"
   task memory_check: [:setup] do
+    require "stringio"
     require "objspace"
     puts "memory load check"
     puts(ObjectSpace.memsize_of_all / 2**20)
@@ -256,37 +258,29 @@ namespace :benchmarks do
     # about 2mb
     puts(ObjectSpace.memsize_of(data) / 2**20)
 
-    json_data = JSON.parse(data)
+    JSON.parse(data)
     # this seems to just show the value of the pointer
     # puts(ObjectSpace.memsize_of(json_data) / 2**20)
     # implies json takes 10-12 mb
     puts(ObjectSpace.memsize_of_all / 2**20)
-
-    json_data = nil
     GC.start
-    json_data = JSON.parse(data)
+    JSON.parse(data)
     # this seems to just show the value of the pointer
     # puts(ObjectSpace.memsize_of(json_data) / 2**20)
     # implies json takes 10-12 mb
     puts(ObjectSpace.memsize_of_all / 2**20)
-
-    json_data = nil
     GC.start
-    json_data = JSON.parse(data)
+    JSON.parse(data)
     # this seems to just show the value of the pointer
     # puts(ObjectSpace.memsize_of(json_data) / 2**20)
     # implies json takes 10-12 mb
     puts(ObjectSpace.memsize_of_all / 2**20)
-
-    json_data = nil
     GC.start
-    json_data = JSON.parse(data)
+    JSON.parse(data)
     # this seems to just show the value of the pointer
     # puts(ObjectSpace.memsize_of(json_data) / 2**20)
     # implies json takes 10-12 mb
     puts(ObjectSpace.memsize_of_all / 2**20)
-
-    json_data = nil
     GC.start
     puts(ObjectSpace.memsize_of_all / 2**20)
 
@@ -316,7 +310,7 @@ namespace :benchmarks do
   desc "runs memory leak check via Rails tests"
   task memory_rails: [:setup] do
     puts "runs memory rails test to ensure we dont leak"
-    puts `COVERBAND_MEMORY_TEST=true bundle exec test/forked/rails_full_stack_test.rb`
+    puts `COVERBAND_MEMORY_TEST=true bundle exec ruby -I test test/forked/rails_full_stack_test.rb`
   end
 
   desc "runs memory leak checks"

data/test/coverband/collectors/coverage_test.rb

@@ -23,7 +23,7 @@ class CollectorsCoverageTest < Minitest::Test
     file = require_unique_file
     coverband.report_coverage
     coverage = Coverband.configuration.store.coverage
-    assert_equal(coverage[file]["data"], [nil, nil, 1, 1, 0, nil, nil])
+    assert_equal(coverage[file]["data"], [nil, nil, 1, 1, 0, nil, nil, 1, nil, 1, nil, nil])
   end
 
   test "Dog method and class coverage" do
@@ -31,7 +31,7 @@ class CollectorsCoverageTest < Minitest::Test
     Dog.new.bark
     coverband.report_coverage
     coverage = Coverband.configuration.store.coverage
-    assert_equal(coverage["./test/dog.rb"]["data"], [nil, nil, 1, 1, 1, nil, nil])
+    assert_equal(coverage["./test/dog.rb"]["data"], [nil, nil, 1, 1, 1, nil, nil, 1, nil, 1, nil, nil])
   end
 
   test "Dog eager load coverage" do
@@ -45,7 +45,7 @@ class CollectorsCoverageTest < Minitest::Test
     coverband.eager_loading!
     coverage = Coverband.configuration.store.coverage[file]
     refute_nil coverage, "Eager load coverage is present"
-    assert_equal(coverage["data"], [nil, nil, 1, 1, 0, nil, nil])
+    assert_equal(coverage["data"], [nil, nil, 1, 1, 0, nil, nil, 1, nil, 1, nil, nil])
   end
 
   test "gets coverage instance" do
@@ -78,16 +78,12 @@ class CollectorsCoverageTest < Minitest::Test
   end
 
   test "using coverage state idle with ruby >= 3.1.0" do
-    return unless Gem::Version.new(RUBY_VERSION) >= Gem::Version.new("3.1.0")
-
     ::Coverage.expects(:state).returns(:idle)
     ::Coverage.expects(:start).with(oneshot_lines: false)
     Coverband::Collectors::Coverage.send(:new)
   end
 
   test "using coverage state suspended with ruby >= 3.1.0" do
-    return unless Gem::Version.new(RUBY_VERSION) >= Gem::Version.new("3.1.0")
-
     ::Coverage.expects(:state).returns(:suspended).at_least_once
     ::Coverage.expects(:resume)
     Coverband::Collectors::Coverage.send(:new)

data/test/coverband/collectors/route_tracker_test.rb

@@ -1,7 +1,6 @@
 # frozen_string_literal: true
 
 require File.expand_path("../../test_helper", File.dirname(__FILE__))
-require "ostruct"
 
 class RouterTrackerTest < Minitest::Test
   # NOTE: using struct vs open struct as open struct has a special keyword method that overshadows the method value on Ruby 2.x