coverband 4.2.7 → 5.0.0.rc.7

data/test/benchmarks/dog.rb

@@ -2,6 +2,6 @@
 
 class Dog
   def bark
-    'bark'
+    "bark"
   end
 end

data/test/benchmarks/init_rails.rake

@@ -1,10 +1,10 @@
 # frozen_string_literal: true
 
-require 'rails'
-require 'coverband'
+require "rails"
+require "coverband"
 
-desc 'Initialize rails'
-task 'init_rails' do
+desc "Initialize rails"
+task "init_rails" do
   Coverband.configure("./test/rails#{Rails::VERSION::MAJOR}_dummy/config/coverband.rb")
   require "./test/rails#{Rails::VERSION::MAJOR}_dummy/config/environment"
 end

data/test/coverband/adapters/base_test.rb

@@ -1,11 +1,10 @@
 # frozen_string_literal: true
 
-require File.expand_path('../../test_helper', File.dirname(__FILE__))
+require File.expand_path("../../test_helper", File.dirname(__FILE__))
 
 class AdaptersBaseTest < Minitest::Test
-
   def test_abstract_methods
-    abstract_methods = %w(clear! clear_file! migrate! size save_coverage coverage)
+    abstract_methods = %w[clear! clear_file! migrate! size save_coverage coverage]
     abstract_methods.each do |method|
       assert_raises RuntimeError do
         Coverband::Adapters::Base.new.send(method.to_sym)
@@ -32,10 +31,10 @@ class AdaptersBaseTest < Minitest::Test
     assert_equal [1, 1, nil, nil], Coverband::Adapters::Base.new.send(:array_add, latest, original)
   end
 
-  describe 'Coverband::Adapters::Base using file' do
+  describe "Coverband::Adapters::Base using file" do
     def setup
       super
-      @test_file_path = '/tmp/coverband_filestore_test_path.json'
+      @test_file_path = "/tmp/coverband_filestore_test_path.json"
       @store = Coverband::Adapters::FileStore.new(@test_file_path)
       mock_file_hash
     end
@@ -44,37 +43,37 @@ class AdaptersBaseTest < Minitest::Test
       old_time = 1541958097
       current_time = Time.now.to_i
       old_data = {
-        'first_updated_at' => old_time,
-        'last_updated_at' => current_time,
-        'file_hash' => 'abcd',
-        'data' => [5, 7, nil]
+        "first_updated_at" => old_time,
+        "last_updated_at" => current_time,
+        "file_hash" => "abcd",
+        "data" => [5, 7, nil]
       }
-      old_report = { '/projects/coverband_demo/config/coverband.rb' => old_data,
-                     '/projects/coverband_demo/config/initializers/assets.rb' => old_data,
-                     '/projects/coverband_demo/config/initializers/cookies_serializer.rb' => old_data }
-      new_report = { '/projects/coverband_demo/config/coverband.rb' => [5, 7, nil],
-                     '/projects/coverband_demo/config/initializers/filter_logging.rb' => [5, 7, nil],
-                     '/projects/coverband_demo/config/initializers/wrap_parameters.rb' => [5, 7, nil],
-                     '/projects/coverband_demo/app/controllers/application_controller.rb' => [5, 7, nil] }
+      old_report = {"/projects/coverband_demo/config/coverband.rb" => old_data,
+                    "/projects/coverband_demo/config/initializers/assets.rb" => old_data,
+                    "/projects/coverband_demo/config/initializers/cookies_serializer.rb" => old_data}
+      new_report = {"/projects/coverband_demo/config/coverband.rb" => [5, 7, nil],
+                    "/projects/coverband_demo/config/initializers/filter_logging.rb" => [5, 7, nil],
+                    "/projects/coverband_demo/config/initializers/wrap_parameters.rb" => [5, 7, nil],
+                    "/projects/coverband_demo/app/controllers/application_controller.rb" => [5, 7, nil]}
       expected_merge = {
-        'first_updated_at' => old_time,
-        'last_updated_at' => current_time,
-        'file_hash' => 'abcd',
-        'data' => [10, 14, nil]
+        "first_updated_at" => old_time,
+        "last_updated_at" => current_time,
+        "file_hash" => "abcd",
+        "data" => [10, 14, nil]
       }
       new_data = {
-        'first_updated_at' => current_time,
-        'last_updated_at' => current_time,
-        'file_hash' => 'abcd',
-        'data' => [5, 7, nil]
+        "first_updated_at" => current_time,
+        "last_updated_at" => current_time,
+        "file_hash" => "abcd",
+        "data" => [5, 7, nil]
       }
       expected_result = {
-        '/projects/coverband_demo/app/controllers/application_controller.rb' => new_data,
-        '/projects/coverband_demo/config/coverband.rb' => expected_merge,
-        '/projects/coverband_demo/config/initializers/assets.rb' => old_data,
-        '/projects/coverband_demo/config/initializers/cookies_serializer.rb' => old_data,
-        '/projects/coverband_demo/config/initializers/filter_logging.rb' => new_data,
-        '/projects/coverband_demo/config/initializers/wrap_parameters.rb' => new_data
+        "/projects/coverband_demo/app/controllers/application_controller.rb" => new_data,
+        "/projects/coverband_demo/config/coverband.rb" => expected_merge,
+        "/projects/coverband_demo/config/initializers/assets.rb" => old_data,
+        "/projects/coverband_demo/config/initializers/cookies_serializer.rb" => old_data,
+        "/projects/coverband_demo/config/initializers/filter_logging.rb" => new_data,
+        "/projects/coverband_demo/config/initializers/wrap_parameters.rb" => new_data
       }
       assert_equal expected_result, @store.send(:merge_reports, new_report, old_report)
     end

data/test/coverband/adapters/file_store_test.rb

@@ -1,38 +1,36 @@
 # frozen_string_literal: true
 
-require File.expand_path('../../test_helper', File.dirname(__FILE__))
+require File.expand_path("../../test_helper", File.dirname(__FILE__))
 
 class AdaptersFileStoreTest < Minitest::Test
-
   def test_covered_lines_when_no_file
-    @store = Coverband::Adapters::FileStore.new('')
+    @store = Coverband::Adapters::FileStore.new("")
     expected = {}
     assert_equal expected, @store.coverage
   end
 
-  describe 'Coverband::Adapters::FileStore with file' do
+  describe "Coverband::Adapters::FileStore with file" do
     def setup
       super
-      @test_file_path = '/tmp/coverband_filestore_test_path.json'
-      File.open(@test_file_path, 'w') { |f| f.write(test_data.to_json) }
+      @test_file_path = "/tmp/coverband_filestore_test_path.json"
+      previous_file_path = "#{@test_file_path}.#{::Process.pid}"
+      `rm #{@test_file_path}` if File.exist?(@test_file_path)
+      `rm #{previous_file_path}` if File.exist?(previous_file_path)
+      File.open(previous_file_path, "w") { |f| f.write(test_data.to_json) }
       @store = Coverband::Adapters::FileStore.new(@test_file_path)
     end
 
-    def test_size
-      assert @store.size > 1
-    end
-
     def test_coverage
-      assert_equal @store.coverage['dog.rb']['data'][0],  1
-      assert_equal @store.coverage['dog.rb']['data'][1],  2
+      assert_equal @store.coverage["dog.rb"]["data"][0], 1
+      assert_equal @store.coverage["dog.rb"]["data"][1], 2
     end
 
     def test_covered_lines_when_null
-      assert_nil @store.coverage['none.rb']
+      assert_nil @store.coverage["none.rb"]
     end
 
     def test_covered_files
-      assert_equal @store.covered_files, ['dog.rb']
+      assert @store.covered_files.include?("dog.rb")
     end
 
     def test_clear
@@ -42,16 +40,17 @@ class AdaptersFileStoreTest < Minitest::Test
 
     def test_save_report
       mock_file_hash
-      @store.send(:save_report, 'cat.rb' => [0, 1])
-      assert_equal @store.coverage['cat.rb']['data'][1], 1
+      @store.send(:save_report, "cat.rb" => [0, 1])
+      assert_equal @store.coverage["cat.rb"]["data"][1], 1
+      assert @store.size > 1
     end
 
     def test_data
       {
-        'dog.rb' => { 'data' => [1, 2, nil],
-                      'file_hash' => 'abcd',
-                      'first_updated_at' => 1541968729,
-                      'last_updated_at' => 1541968729 }
+        "dog.rb" => {"data" => [1, 2, nil],
+                     "file_hash" => "abcd",
+                     "first_updated_at" => 1541968729,
+                     "last_updated_at" => 1541968729}
       }
     end
   end

data/test/coverband/adapters/hash_redis_store_test.rb

@@ -1,13 +1,13 @@
 # frozen_string_literal: true
 
-require File.expand_path('../../test_helper', File.dirname(__FILE__))
+require File.expand_path("../../test_helper", File.dirname(__FILE__))
 
 class HashRedisStoreTest < Minitest::Test
   REDIS_STORAGE_FORMAT_VERSION = Coverband::Adapters::HashRedisStore::REDIS_STORAGE_FORMAT_VERSION
 
   class MockRelativeFileConverter
     def self.convert(file)
-      file.sub('app_path/', './')
+      file.sub("app_path/", "./")
     end
   end
 
@@ -16,9 +16,9 @@ class HashRedisStoreTest < Minitest::Test
     @redis = Coverband::Test.redis
     # FIXME: remove dependency on configuration and instead pass this in as an argument
     Coverband.configure do |config|
-      config.root_paths = ['app_path/']
+      config.root_paths = ["app_path/"]
     end
-    @store = Coverband::Adapters::HashRedisStore.new(@redis, redis_namespace: 'coverband_test', relative_file_converter: MockRelativeFileConverter)
+    @store = Coverband::Adapters::HashRedisStore.new(@redis, redis_namespace: "coverband_test", relative_file_converter: MockRelativeFileConverter)
     Coverband.configuration.store = @store
   end
 
@@ -34,7 +34,7 @@ class HashRedisStoreTest < Minitest::Test
 
   class UnsupportedRedis
     def info
-      { 'redis_version' => '2.5.0' }
+      {"redis_version" => "2.5.0"}
     end
   end
 
@@ -50,46 +50,46 @@ class HashRedisStoreTest < Minitest::Test
     mock_time(yesterday)
     mock_file_hash
     @store.save_report(
-      'app_path/dog.rb' => [0, 1, 2]
+      "app_path/dog.rb" => [0, 1, 2]
     )
     assert_equal(
       {
-        'first_updated_at' => yesterday.to_i,
-        'last_updated_at' => yesterday.to_i,
-        'file_hash' => 'abcd',
-        'data' => [0, 1, 2]
+        "first_updated_at" => yesterday.to_i,
+        "last_updated_at" => yesterday.to_i,
+        "file_hash" => "abcd",
+        "data" => [0, 1, 2]
       },
-      @store.coverage['./dog.rb']
+      @store.coverage["./dog.rb"]
     )
     mock_time(today)
     @store.save_report(
-      'app_path/dog.rb' => [1, 1, 0]
+      "app_path/dog.rb" => [1, 1, 0]
     )
     assert_equal(
       {
-        'first_updated_at' => yesterday.to_i,
-        'last_updated_at' => today.to_i,
-        'file_hash' => 'abcd',
-        'data' => [1, 2, 2]
+        "first_updated_at" => yesterday.to_i,
+        "last_updated_at" => today.to_i,
+        "file_hash" => "abcd",
+        "data" => [1, 2, 2]
       },
-      @store.coverage['./dog.rb']
+      @store.coverage["./dog.rb"]
     )
   end
 
   def test_ttl_set
-    mock_file_hash(hash: 'abcd')
-    @store = Coverband::Adapters::HashRedisStore.new(@redis, redis_namespace: 'coverband_test', ttl: 3600, relative_file_converter: MockRelativeFileConverter)
+    mock_file_hash(hash: "abcd")
+    @store = Coverband::Adapters::HashRedisStore.new(@redis, redis_namespace: "coverband_test", ttl: 3600, relative_file_converter: MockRelativeFileConverter)
     @store.save_report(
-      'app_path/dog.rb' => [0, 1, 2]
+      "app_path/dog.rb" => [0, 1, 2]
     )
     assert_operator(@redis.ttl("#{REDIS_STORAGE_FORMAT_VERSION}.coverband_test.runtime../dog.rb.abcd"), :>, 0)
   end
 
   def test_no_ttl_set
-    mock_file_hash(hash: 'abcd')
-    @store = Coverband::Adapters::HashRedisStore.new(@redis, redis_namespace: 'coverband_test', ttl: nil, relative_file_converter: MockRelativeFileConverter)
+    mock_file_hash(hash: "abcd")
+    @store = Coverband::Adapters::HashRedisStore.new(@redis, redis_namespace: "coverband_test", ttl: nil, relative_file_converter: MockRelativeFileConverter)
     @store.save_report(
-      'app_path/dog.rb' => [0, 1, 2]
+      "app_path/dog.rb" => [0, 1, 2]
     )
     assert_equal(-1, @redis.ttl("#{REDIS_STORAGE_FORMAT_VERSION}.coverband_test.runtime../dog.rb.abcd"))
   end
@@ -98,98 +98,98 @@ class HashRedisStoreTest < Minitest::Test
     current_time = mock_time
     mock_file_hash
     data = {
-      'app_path/dog.rb' => [0, nil, 1, 2],
-      'app_path/cat.rb' => [1, 2, 0, 1, 5],
-      'app_path/ferrit.rb' => [1, 5, nil, 2, nil]
+      "app_path/dog.rb" => [0, nil, 1, 2],
+      "app_path/cat.rb" => [1, 2, 0, 1, 5],
+      "app_path/ferrit.rb" => [1, 5, nil, 2, nil]
     }
     @store.save_report(data)
-    coverage = @store.coverage
+    @store.coverage
     assert_equal(
       {
-        'first_updated_at' => current_time.to_i,
-        'last_updated_at' => current_time.to_i,
-        'file_hash' => 'abcd',
-        'data' => [0, nil, 1, 2]
-      }, @store.coverage['./dog.rb']
+        "first_updated_at" => current_time.to_i,
+        "last_updated_at" => current_time.to_i,
+        "file_hash" => "abcd",
+        "data" => [0, nil, 1, 2]
+      }, @store.coverage["./dog.rb"]
     )
-    assert_equal [1, 2, 0, 1, 5], @store.coverage['./cat.rb']['data']
-    assert_equal [1, 5, nil, 2, nil], @store.coverage['./ferrit.rb']['data']
+    assert_equal [1, 2, 0, 1, 5], @store.coverage["./cat.rb"]["data"]
+    assert_equal [1, 5, nil, 2, nil], @store.coverage["./ferrit.rb"]["data"]
   end
 
   def test_file_hash_change
-    mock_file_hash(hash: 'abc')
-    @store.save_report('app_path/dog.rb' => [0, nil, 1, 2])
-    coverage = @store.coverage
-    assert_equal [0, nil, 1, 2], @store.coverage['./dog.rb']['data']
+    mock_file_hash(hash: "abc")
+    @store.save_report("app_path/dog.rb" => [0, nil, 1, 2])
+    @store.coverage
+    assert_equal [0, nil, 1, 2], @store.coverage["./dog.rb"]["data"]
     @store.instance_eval { @file_hash_cache = {} }
-    mock_file_hash(hash: '123')
-    assert_nil @store.coverage['./dog.rb']
+    mock_file_hash(hash: "123")
+    assert_nil @store.coverage["./dog.rb"]
   end
 
   def test_type
     mock_file_hash
     @store.type = :eager_loading
     data = {
-      'app_path/dog.rb' => [0, nil, 1, 2]
+      "app_path/dog.rb" => [0, nil, 1, 2]
     }
     @store.save_report(data)
     assert_equal 1, @store.coverage.length
-    assert_equal [0, nil, 1, 2], @store.coverage['./dog.rb']['data']
+    assert_equal [0, nil, 1, 2], @store.coverage["./dog.rb"]["data"]
     # eager_loading doesn't set last_updated_at
-    assert_nil @store.coverage['./dog.rb']['last_updated_at']
+    assert_nil @store.coverage["./dog.rb"]["last_updated_at"]
     @store.type = Coverband::RUNTIME_TYPE
     data = {
-      'app_path/cat.rb' => [1, 2, 0, 1, 5]
+      "app_path/cat.rb" => [1, 2, 0, 1, 5]
     }
     current_time = Time.now.to_i
     @store.save_report(data)
     assert_equal 1, @store.coverage.length
-    assert_equal [1, 2, 0, 1, 5], @store.coverage['./cat.rb']['data']
-    assert current_time <= @store.coverage['./cat.rb']['last_updated_at']
+    assert_equal [1, 2, 0, 1, 5], @store.coverage["./cat.rb"]["data"]
+    assert current_time <= @store.coverage["./cat.rb"]["last_updated_at"]
   end
 
   def test_coverage_type
     mock_file_hash
     @store.type = :eager_loading
     data = {
-      'app_path/dog.rb' => [0, nil, 1, 2]
+      "app_path/dog.rb" => [0, nil, 1, 2]
     }
     @store.save_report(data)
     @store.type = Coverband::RUNTIME_TYPE
-    assert_equal [0, nil, 1, 2], @store.coverage(:eager_loading)['./dog.rb']['data']
+    assert_equal [0, nil, 1, 2], @store.coverage(:eager_loading)["./dog.rb"]["data"]
   end
 
   def test_clear
     mock_file_hash
     @store.type = Coverband::EAGER_TYPE
     data = {
-      'app_path/dog.rb' => [0, nil, 1, 2]
+      "app_path/dog.rb" => [0, nil, 1, 2]
     }
     @store.save_report(data)
     assert_equal 1, @store.coverage.length
     @store.type = Coverband::RUNTIME_TYPE
     data = {
-      'app_path/cat.rb' => [1, 2, 0, 1, 5]
+      "app_path/cat.rb" => [1, 2, 0, 1, 5]
     }
     @store.save_report(data)
     assert_equal 1, @store.coverage.length
-    @redis.set('random', 'data')
+    @redis.set("random", "data")
     @store.clear!
     @store.type = Coverband::RUNTIME_TYPE
     assert @store.coverage.empty?
     @store.type = :eager_loading
     assert @store.coverage.empty?
-    assert_equal 'data', @redis.get('random')
+    assert_equal "data", @redis.get("random")
   end
 
   def test_clear_file
     mock_file_hash
     @store.type = :eager_loading
-    @store.save_report('app_path/dog.rb' => [0, 1, 1])
+    @store.save_report("app_path/dog.rb" => [0, 1, 1])
     @store.type = Coverband::RUNTIME_TYPE
-    @store.save_report('app_path/dog.rb' => [1, 0, 1])
-    assert_equal [1, 1, 2], @store.get_coverage_report[:merged]['./dog.rb']['data']
-    @store.clear_file!('app_path/dog.rb')
-    assert_nil @store.get_coverage_report[:merged]['./dog.rb']
+    @store.save_report("app_path/dog.rb" => [1, 0, 1])
+    assert_equal [1, 1, 2], @store.get_coverage_report[:merged]["./dog.rb"]["data"]
+    @store.clear_file!("app_path/dog.rb")
+    assert_nil @store.get_coverage_report[:merged]["./dog.rb"]
   end
 end

data/test/coverband/adapters/redis_store_test.rb

@@ -1,8 +1,8 @@
 # frozen_string_literal: true
 
-require File.expand_path('../../test_helper', File.dirname(__FILE__))
+require File.expand_path("../../test_helper", File.dirname(__FILE__))
 
-unless ENV['COVERBAND_HASH_REDIS_STORE']
+unless ENV["COVERBAND_HASH_REDIS_STORE"]
   class RedisTest < Minitest::Test
     REDIS_STORAGE_FORMAT_VERSION = Coverband::Adapters::RedisStore::REDIS_STORAGE_FORMAT_VERSION
 
@@ -18,7 +18,7 @@ unless ENV['COVERBAND_HASH_REDIS_STORE']
       @store.save_report(expected)
       assert_equal expected.keys, @store.coverage.keys
       @store.coverage.each_pair do |key, data|
-        assert_equal expected[key], data['data']
+        assert_equal expected[key], data["data"]
       end
     end
 
@@ -28,20 +28,20 @@ unless ENV['COVERBAND_HASH_REDIS_STORE']
       @store.save_report(basic_coverage.dup)
       assert_equal expected.keys, @store.coverage.keys
       @store.coverage.each_pair do |key, data|
-        assert_equal expected[key], data['data']
+        assert_equal expected[key], data["data"]
       end
       current_time = Time.now.to_i
       @store.save_report(basic_coverage.dup)
-      assert_equal [0, 2, 4], @store.coverage['app_path/dog.rb']['data']
-      assert current_time <= @store.coverage['app_path/dog.rb']['last_updated_at']
+      assert_equal [0, 2, 4], @store.coverage["app_path/dog.rb"]["data"]
+      assert current_time <= @store.coverage["app_path/dog.rb"]["last_updated_at"]
     end
 
     def test_file_hash_change
-      mock_file_hash(hash: 'abc')
-      @store.save_report('app_path/dog.rb' => [0, nil, 1, 2])
-      assert_equal [0, nil, 1, 2], @store.coverage['app_path/dog.rb']['data']
-      mock_file_hash(hash: '123')
-      assert_nil @store.coverage['app_path/dog.rb']
+      mock_file_hash(hash: "abc")
+      @store.save_report("app_path/dog.rb" => [0, nil, 1, 2])
+      assert_equal [0, nil, 1, 2], @store.coverage["app_path/dog.rb"]["data"]
+      mock_file_hash(hash: "123")
+      assert_nil @store.coverage["app_path/dog.rb"]
     end
 
     def test_store_coverage_by_type
@@ -51,7 +51,7 @@ unless ENV['COVERBAND_HASH_REDIS_STORE']
       @store.save_report(expected)
       assert_equal expected.keys, @store.coverage.keys
       @store.coverage.each_pair do |key, data|
-        assert_equal expected[key], data['data']
+        assert_equal expected[key], data["data"]
       end
       @store.type = Coverband::RUNTIME_TYPE
       assert_equal [], @store.coverage.keys
@@ -61,14 +61,14 @@ unless ENV['COVERBAND_HASH_REDIS_STORE']
       mock_file_hash
       assert_equal Coverband::RUNTIME_TYPE, @store.type
       @store.type = :eager_loading
-      @store.save_report('app_path/dog.rb' => [0, 1, 1])
+      @store.save_report("app_path/dog.rb" => [0, 1, 1])
       # eager_loading doesn't set last_updated_at
-      assert_nil @store.coverage['app_path/dog.rb']['last_updated_at']
+      assert_nil @store.coverage["app_path/dog.rb"]["last_updated_at"]
       @store.type = Coverband::RUNTIME_TYPE
       current_time = Time.now.to_i
-      @store.save_report('app_path/dog.rb' => [1, 0, 1])
-      assert_equal [1, 1, 2], @store.get_coverage_report[:merged]['app_path/dog.rb']['data']
-      assert current_time <= @store.coverage['app_path/dog.rb']['last_updated_at']
+      @store.save_report("app_path/dog.rb" => [1, 0, 1])
+      assert_equal [1, 1, 2], @store.get_coverage_report[:merged]["app_path/dog.rb"]["data"]
+      assert current_time <= @store.coverage["app_path/dog.rb"]["last_updated_at"]
       assert_equal Coverband::RUNTIME_TYPE, @store.type
     end
 
@@ -76,7 +76,7 @@ unless ENV['COVERBAND_HASH_REDIS_STORE']
       mock_file_hash
       expected = basic_coverage
       @store.save_report(expected)
-      assert_equal example_line, @store.coverage['app_path/dog.rb']['data']
+      assert_equal example_line, @store.coverage["app_path/dog.rb"]["data"]
     end
 
     def test_coverage_with_simulate_oneshot_lines_coverage
@@ -84,11 +84,11 @@ unless ENV['COVERBAND_HASH_REDIS_STORE']
       mock_file_hash
       expected = basic_coverage
       @store.save_report(expected)
-      assert_equal example_line, @store.get_coverage_report[:runtime]['app_path/dog.rb']['data']
+      assert_equal example_line, @store.get_coverage_report[:runtime]["app_path/dog.rb"]["data"]
     end
 
     def test_coverage_when_null
-      assert_nil @store.coverage['app_path/dog.rb']
+      assert_nil @store.coverage["app_path/dog.rb"]
     end
 
     def test_clear
@@ -99,18 +99,18 @@ unless ENV['COVERBAND_HASH_REDIS_STORE']
     def test_clear_file
       mock_file_hash
       @store.type = :eager_loading
-      @store.save_report('app_path/dog.rb' => [0, 1, 1])
+      @store.save_report("app_path/dog.rb" => [0, 1, 1])
       @store.type = Coverband::RUNTIME_TYPE
-      @store.save_report('app_path/dog.rb' => [1, 0, 1])
-      assert_equal [1, 1, 2], @store.get_coverage_report[:merged]['app_path/dog.rb']['data']
-      @store.clear_file!('app_path/dog.rb')
-      assert_nil @store.get_coverage_report[:merged]['app_path/dog.rb']
+      @store.save_report("app_path/dog.rb" => [1, 0, 1])
+      assert_equal [1, 1, 2], @store.get_coverage_report[:merged]["app_path/dog.rb"]["data"]
+      @store.clear_file!("app_path/dog.rb")
+      assert_nil @store.get_coverage_report[:merged]["app_path/dog.rb"]
     end
 
     def test_size
       mock_file_hash
       @store.type = :eager_loading
-      @store.save_report('app_path/dog.rb' => [0, 1, 1])
+      @store.save_report("app_path/dog.rb" => [0, 1, 1])
       assert @store.size > 1
     end