coverband 6.1.0 → 6.1.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 21e62f4add1bcd44cacd630e37e291a38ecfb924d8fa48698ecbe4ba69a99471
-  data.tar.gz: 4cc4dfd1289bdf13f77c78a7deb399c18075ec53a835abb45fa11011d0815bd4
+  metadata.gz: ed2b063179cbc773c9b3b8fab3403820f9855c32a9de007c347b58873eecf13a
+  data.tar.gz: 7968b26ac22e3209b0bf2a9baabdfe107ed9029a0d4557eea5b80c22439e3278
 SHA512:
-  metadata.gz: 2bcf294b5d07fc508b1468b0e3a2d993d15b1dae067bd147ca1d55ebc444cf6f7157fbd6e5afee205a7fa9e3648bee9fdab66cec6573ff31605c417346c858a3
-  data.tar.gz: 282523aea85ea90943c6a8bbf595a3c135f7a95534ac2046ccfeb379cea252ad4889b3e31799de8917b3b989a9492f55fee032f302f8aa8a8c619e870941c79c
+  metadata.gz: 3bccf2dbc8e92d5980365a99b5ae2852fd8e04d224202a58e3a8490991b88eb88a881fbf620e4ea60d66cd4714a70f11a8015ebd96aa772a65a35e1fb642ea24
+  data.tar.gz: 784f5e0905989f865199261270072c397e520b2269ed387f0ecd1276ef5a7395fc5d4de7ec9990aceeda1cc883778bcfcf4c7c2260c792e81557914298778d87

data/changes.md

@@ -1,3 +1,7 @@
+### Coverband 6.1.1
+
+* Performance fix making paged report loading 10X faster
+
 ### 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.

data/coverband.gemspec

@@ -36,6 +36,7 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency "memory_profiler"
   # breaking change in minitest and mocha...
   # 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-fork_executor"

data/lib/coverband/adapters/hash_redis_store.rb

@@ -231,14 +231,14 @@ module Coverband
           end
         end
 
+        # NOTE: This is kind of hacky, we find all the matching eager loading data
+        # for current page of runtime data.
         eager_key_pre = key_prefix(Coverband::EAGER_TYPE)
         runtime_key_pre = key_prefix(Coverband::RUNTIME_TYPE)
-        matched_file_set = files_set(Coverband::EAGER_TYPE)
-          .select do |eager_key, _val|
-          runtime_file_set.any? do |runtime_key|
-            (eager_key.sub(eager_key_pre, "") == runtime_key.sub(runtime_key_pre, ""))
-          end
-        end || []
+        matched_file_set = runtime_file_set.map do |runtime_key|
+          runtime_key.sub(runtime_key_pre, eager_key_pre)
+        end
+
         hash_data[Coverband::EAGER_TYPE] = matched_file_set.each_slice(page_size).flat_map do |key_batch|
           @redis.pipelined do |pipeline|
             key_batch.each do |key|
@@ -246,6 +246,7 @@ module Coverband
             end
           end
         end
+
         hash_data[Coverband::RUNTIME_TYPE] = hash_data[Coverband::RUNTIME_TYPE].each_with_object({}) do |data_from_redis, hash|
           add_coverage_for_file(data_from_redis, hash)
         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.0"
+  VERSION = "6.1.1"
 end

data/test/coverband/adapters/hash_redis_store_test.rb

@@ -242,4 +242,52 @@ class HashRedisStoreTest < Minitest::Test
       @store.coverage["./dog.rb"]
     )
   end
+
+  def test_split_coverage
+    @store = Coverband::Adapters::HashRedisStore.new(
+      @redis,
+      redis_namespace: "coverband_test",
+      relative_file_converter: MockRelativeFileConverter
+    )
+
+    mock_file_hash
+    yesterday = DateTime.now.prev_day.to_time
+    mock_time(yesterday)
+
+    @store.type = :eager_loading
+    data = {
+      "app_path/dog.rb" => [0, nil, 1]
+    }
+    @store.save_report(data)
+
+    @store.type = :runtime
+    @store.save_report(
+      "app_path/dog.rb" => [0, 1, 2]
+    )
+    redis_pipelined = Spy.on(@redis, :pipelined).and_call_through
+    assert_equal(
+      {
+        runtime: {
+          "./dog.rb" => {
+            "first_updated_at" => yesterday.to_i,
+            "last_updated_at" => yesterday.to_i,
+            "file_hash" => "abcd",
+            "data" => [0, 1, 2],
+            "timedata" => [nil, Time.at(yesterday.to_i), Time.at(yesterday.to_i)]
+          }
+        },
+        eager_loading: {
+          "./dog.rb" => {
+            "first_updated_at" => yesterday.to_i,
+            "last_updated_at" => nil,
+            "file_hash" => "abcd",
+            "data" => [0, nil, 1],
+            "timedata" => [nil, nil, Time.at(yesterday.to_i)]
+          }
+        }
+      },
+      @store.split_coverage([Coverband::RUNTIME_TYPE, Coverband::EAGER_TYPE], {}, {page: 1})
+    )
+    assert_equal 2, redis_pipelined.calls.count
+  end
 end

data/test/test_helper.rb

@@ -35,6 +35,7 @@ require "coverband/utils/results"
 require "coverband/reporters/html_report"
 require "coverband/reporters/json_report"
 require "webmock/minitest"
+require "spy/integration"
 
 require_relative "unique_files"
 $VERBOSE = original_verbosity

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: coverband
 version: !ruby/object:Gem::Version
-  version: 6.1.0
+  version: 6.1.1
 platform: ruby
 authors:
 - Dan Mayer
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2024-04-21 00:00:00.000000000 Z
+date: 2024-04-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: benchmark-ips
@@ -67,6 +67,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: spy
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: minitest
   requirement: !ruby/object:Gem::Requirement