coverband 4.2.1 → 4.2.2

data/test/benchmarks/init_rails.rake

@@ -0,0 +1,10 @@
+# frozen_string_literal: true
+
+require 'rails'
+require 'coverband'
+
+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/hash_redis_store_test.rb

@@ -0,0 +1,190 @@
+# frozen_string_literal: true
+
+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/', './')
+    end
+  end
+
+  def setup
+    super
+    @redis = Redis.new
+    # FIXME: remove dependency on configuration and instead pass this in as an argument
+    Coverband.configure do |config|
+      config.root_paths = ['app_path/']
+    end
+    @store = Coverband::Adapters::HashRedisStore.new(@redis, redis_namespace: 'coverband_test', relative_file_converter: MockRelativeFileConverter)
+    @store.clear!
+    Coverband.configuration.store = @store
+  end
+
+  def mock_time(time = Time.now)
+    Time.stubs(:now).returns(time)
+    time
+  end
+
+  def test_no_coverage
+    @store.save_report({})
+    assert_equal({}, @store.coverage)
+  end
+
+  class UnsupportedRedis
+    def info
+      { 'redis_version' => '2.5.0' }
+    end
+  end
+
+  def test_unsupported_redis
+    assert_raises RuntimeError do
+      Coverband::Adapters::HashRedisStore.new(UnsupportedRedis.new)
+    end
+  end
+
+  def test_coverage_for_file
+    yesterday = DateTime.now.prev_day.to_time
+    today = Time.now
+    mock_time(yesterday)
+    mock_file_hash
+    @store.save_report(
+      'app_path/dog.rb' => [0, 1, 2]
+    )
+    assert_equal(
+      {
+        'first_updated_at' => yesterday.to_i,
+        'last_updated_at' => yesterday.to_i,
+        'data' => [0, 1, 2]
+      },
+      @store.coverage['./dog.rb']
+    )
+    mock_time(today)
+    @store.save_report(
+      'app_path/dog.rb' => [1, 1, 0]
+    )
+    assert_equal(
+      {
+        'first_updated_at' => yesterday.to_i,
+        'last_updated_at' => today.to_i,
+        'data' => [1, 2, 2]
+      },
+      @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)
+    @store.save_report(
+      '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)
+    @store.save_report(
+      'app_path/dog.rb' => [0, 1, 2]
+    )
+    assert_equal(-1, @redis.ttl("#{REDIS_STORAGE_FORMAT_VERSION}.coverband_test.runtime../dog.rb.abcd"))
+  end
+
+  def test_coverage_for_multiple_files
+    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]
+    }
+    @store.save_report(data)
+    coverage = @store.coverage
+    assert_equal(
+      {
+        'first_updated_at' => current_time.to_i,
+        'last_updated_at' => current_time.to_i,
+        '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']
+  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']
+    @store.instance_eval { @file_hash_cache = {} }
+    mock_file_hash(hash: '123')
+    $debug = true
+    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]
+    }
+    @store.save_report(data)
+    assert_equal 1, @store.coverage.length
+    assert_equal [0, nil, 1, 2], @store.coverage['./dog.rb']['data']
+    @store.type = Coverband::RUNTIME_TYPE
+    data = {
+      'app_path/cat.rb' => [1, 2, 0, 1, 5]
+    }
+    @store.save_report(data)
+    assert_equal 1, @store.coverage.length
+    assert_equal [1, 2, 0, 1, 5], @store.coverage['./cat.rb']['data']
+  end
+
+  def test_coverage_type
+    mock_file_hash
+    @store.type = :eager_loading
+    data = {
+      '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']
+  end
+
+  def test_clear
+    mock_file_hash
+    @store.type = Coverband::EAGER_TYPE
+    data = {
+      '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]
+    }
+    @store.save_report(data)
+    assert_equal 1, @store.coverage.length
+    @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')
+  end
+
+  def test_clear_file
+    mock_file_hash
+    @store.type = :eager_loading
+    @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']
+  end
+end

data/test/coverband/adapters/redis_store_test.rb

@@ -2,108 +2,110 @@
 
 require File.expand_path('../../test_helper', File.dirname(__FILE__))
 
-class RedisTest < Minitest::Test
-  REDIS_STORAGE_FORMAT_VERSION = Coverband::Adapters::RedisStore::REDIS_STORAGE_FORMAT_VERSION
+unless ENV['COVERBAND_HASH_REDIS_STORE']
+  class RedisTest < Minitest::Test
+    REDIS_STORAGE_FORMAT_VERSION = Coverband::Adapters::RedisStore::REDIS_STORAGE_FORMAT_VERSION
 
-  def setup
-    super
-    Coverband.configuration.redis_namespace = 'coverband_test'
-    @redis = Redis.new
-    @store = Coverband::Adapters::RedisStore.new(@redis, redis_namespace: 'coverband_test')
-    Coverband.configuration.store = @store
-    @store.clear!
-  end
-
-  def test_coverage
-    mock_file_hash
-    expected = basic_coverage
-    @store.save_report(expected)
-    assert_equal expected.keys, @store.coverage.keys
-    @store.coverage.each_pair do |key, data|
-      assert_equal expected[key], data['data']
+    def setup
+      super
+      Coverband.configuration.redis_namespace = 'coverband_test'
+      @redis = Redis.new
+      @store = Coverband::Adapters::RedisStore.new(@redis, redis_namespace: 'coverband_test')
+      Coverband.configuration.store = @store
+      @store.clear!
     end
-  end
 
-  def test_coverage_increments
-    mock_file_hash
-    expected = basic_coverage.dup
-    @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']
+    def test_coverage
+      mock_file_hash
+      expected = basic_coverage
+      @store.save_report(expected)
+      assert_equal expected.keys, @store.coverage.keys
+      @store.coverage.each_pair do |key, data|
+        assert_equal expected[key], data['data']
+      end
     end
-    @store.save_report(basic_coverage.dup)
-    assert_equal [0, 2, 4], @store.coverage['app_path/dog.rb']['data']
-  end
 
-  def test_store_coverage_by_type
-    mock_file_hash
-    expected = basic_coverage
-    @store.type = :eager_loading
-    @store.save_report(expected)
-    assert_equal expected.keys, @store.coverage.keys
-    @store.coverage.each_pair do |key, data|
-      assert_equal expected[key], data['data']
+    def test_coverage_increments
+      mock_file_hash
+      expected = basic_coverage.dup
+      @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']
+      end
+      @store.save_report(basic_coverage.dup)
+      assert_equal [0, 2, 4], @store.coverage['app_path/dog.rb']['data']
     end
 
-    @store.type = Coverband::RUNTIME_TYPE
-    assert_equal [], @store.coverage.keys
-  end
+    def test_store_coverage_by_type
+      mock_file_hash
+      expected = basic_coverage
+      @store.type = :eager_loading
+      @store.save_report(expected)
+      assert_equal expected.keys, @store.coverage.keys
+      @store.coverage.each_pair do |key, data|
+        assert_equal expected[key], data['data']
+      end
 
-  def test_merged_coverage_with_types
-    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.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']
-    assert_equal Coverband::RUNTIME_TYPE, @store.type
-  end
+      @store.type = Coverband::RUNTIME_TYPE
+      assert_equal [], @store.coverage.keys
+    end
 
-  def test_coverage_for_file
-    mock_file_hash
-    expected = basic_coverage
-    @store.save_report(expected)
-    assert_equal example_line, @store.coverage['app_path/dog.rb']['data']
-  end
+    def test_merged_coverage_with_types
+      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.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']
+      assert_equal Coverband::RUNTIME_TYPE, @store.type
+    end
 
-  def test_coverage_with_simulate_oneshot_lines_coverage
-    Coverband.configuration.stubs(:simulate_oneshot_lines_coverage).returns(true)
-    mock_file_hash
-    expected = basic_coverage
-    @store.save_report(expected)
-    assert_equal example_line, @store.get_coverage_report[:runtime]['app_path/dog.rb']['data']
-  end
+    def test_coverage_for_file
+      mock_file_hash
+      expected = basic_coverage
+      @store.save_report(expected)
+      assert_equal example_line, @store.coverage['app_path/dog.rb']['data']
+    end
 
-  def test_coverage_when_null
-    assert_nil @store.coverage['app_path/dog.rb']
-  end
+    def test_coverage_with_simulate_oneshot_lines_coverage
+      Coverband.configuration.stubs(:simulate_oneshot_lines_coverage).returns(true)
+      mock_file_hash
+      expected = basic_coverage
+      @store.save_report(expected)
+      assert_equal example_line, @store.get_coverage_report[:runtime]['app_path/dog.rb']['data']
+    end
 
-  def test_clear
-    @redis.expects(:del).times(3)
-    @store.clear!
-  end
+    def test_coverage_when_null
+      assert_nil @store.coverage['app_path/dog.rb']
+    end
 
-  def test_clear_file
-    mock_file_hash
-    @store.type = :eager_loading
-    @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']
-  end
+    def test_clear
+      @redis.expects(:del).times(3)
+      @store.clear!
+    end
 
-  def test_size
-    mock_file_hash
-    @store.type = :eager_loading
-    @store.save_report('app_path/dog.rb' => [0, 1, 1])
-    assert @store.size > 1
-  end
+    def test_clear_file
+      mock_file_hash
+      @store.type = :eager_loading
+      @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']
+    end
 
-  def test_base_key
-    assert @store.send(:base_key).end_with?(Coverband::RUNTIME_TYPE.to_s)
+    def test_size
+      mock_file_hash
+      @store.type = :eager_loading
+      @store.save_report('app_path/dog.rb' => [0, 1, 1])
+      assert @store.size > 1
+    end
+
+    def test_base_key
+      assert @store.send(:base_key).end_with?(Coverband::RUNTIME_TYPE.to_s)
+    end
   end
 end

data/test/coverband/collectors/coverage_test.rb

@@ -73,7 +73,7 @@ class CollectorsCoverageTest < Minitest::Test
     Coverband.configuration.logger = logger
     @coverband.reset_instance
     Coverband::Adapters::RedisStore.any_instance.stubs(:save_report).raises('Oh no')
-    logger.expects(:error).times(3)
+    logger.expects(:error).at_least(3)
     error = assert_raises RuntimeError do
       @coverband.report_coverage
     end
@@ -87,6 +87,7 @@ class CollectorsCoverageTest < Minitest::Test
 
   test 'one shot line coverage disabled for ruby >= 2.6' do
     return unless Gem::Version.new(RUBY_VERSION) >= Gem::Version.new('2.5.0')
+
     Coverband::Collectors::Coverage.expects(:ruby_version_greater_than_or_equal_to?).with('2.6.0').returns(true)
     ::Coverage.expects(:running?).returns(false)
     ::Coverage.expects(:start).with(oneshot_lines: false)
@@ -95,6 +96,7 @@ class CollectorsCoverageTest < Minitest::Test
 
   test 'one shot line coverage enabled for ruby >= 2.6' do
     return unless Gem::Version.new(RUBY_VERSION) >= Gem::Version.new('2.5.0')
+
     Coverband.configuration.expects(:use_oneshot_lines_coverage).returns(true)
     Coverband::Collectors::Coverage.expects(:ruby_version_greater_than_or_equal_to?).with('2.6.0').returns(true)
     ::Coverage.expects(:running?).returns(false)
@@ -104,6 +106,7 @@ class CollectorsCoverageTest < Minitest::Test
 
   test 'one shot line coverage for ruby >= 2.6 when already running' do
     return unless Gem::Version.new(RUBY_VERSION) >= Gem::Version.new('2.5.0')
+
     Coverband::Collectors::Coverage.expects(:ruby_version_greater_than_or_equal_to?).with('2.6.0').returns(true)
     ::Coverage.expects(:running?).returns(true)
     ::Coverage.expects(:start).never
@@ -112,10 +115,11 @@ class CollectorsCoverageTest < Minitest::Test
 
   test 'no one shot line coverage for ruby < 2.6' do
     return unless Gem::Version.new(RUBY_VERSION) >= Gem::Version.new('2.5.0')
+
     Coverband::Collectors::Coverage.expects(:ruby_version_greater_than_or_equal_to?).with('2.6.0').returns(false)
     Coverband::Collectors::Coverage.expects(:ruby_version_greater_than_or_equal_to?).with('2.5.0').returns(true)
     ::Coverage.expects(:running?).returns(false)
-    ::Coverage.expects(:start).with()
+    ::Coverage.expects(:start).with
     Coverband::Collectors::Coverage.send(:new)
   end
 end

data/test/coverband/collectors/view_tracker_test.rb

@@ -0,0 +1,77 @@
+require File.expand_path('../../test_helper', File.dirname(__FILE__))
+
+class ReporterTest < Minitest::Test
+
+  def setup
+    super
+    fake_store.raw_store.del('render_tracker')
+  end
+
+  test "init correctly" do
+    Coverband::Collectors::ViewTracker.expects(:supported_version?).returns(true)
+    tracker = Coverband::Collectors::ViewTracker.new(:store => "store", :roots => 'dir')
+    assert_equal 'dir', tracker.roots.first
+    assert_equal 'store', tracker.store
+    assert_equal [], tracker.target
+    assert_equal [], tracker.logged_views
+  end
+
+  test "track partials" do
+    Coverband::Collectors::ViewTracker.expects(:supported_version?).returns(true)
+    store = fake_store
+    file_path = "#{File.expand_path(Coverband.configuration.root)}/file"
+    store.raw_store.expects(:sadd).with('render_tracker', file_path)
+    tracker = Coverband::Collectors::ViewTracker.new(store: store, roots: 'dir')
+    tracker.track_views('name', 'start', 'finish', 'id', {:identifier => file_path})
+    tracker.report_views_tracked
+    assert_equal [file_path], tracker.logged_views
+  end
+
+  test "track layouts" do
+    Coverband::Collectors::ViewTracker.expects(:supported_version?).returns(true)
+    store = fake_store
+    file_path = "#{File.expand_path(Coverband.configuration.root)}/layout"
+    store.raw_store.expects(:sadd).with('render_tracker', file_path)
+    tracker = Coverband::Collectors::ViewTracker.new(store: store, roots: 'dir')
+    tracker.track_views('name', 'start', 'finish', 'id', {:layout => file_path})
+    tracker.report_views_tracked
+    assert_equal [file_path], tracker.logged_views
+  end
+
+  test "report used partials" do
+    Coverband::Collectors::ViewTracker.expects(:supported_version?).returns(true)
+    store = fake_store
+    file_path = "#{File.expand_path(Coverband.configuration.root)}/file"
+    tracker = Coverband::Collectors::ViewTracker.new(store: store, roots: 'dir')
+    tracker.track_views('name', 'start', 'finish', 'id', {:identifier => file_path})
+    tracker.report_views_tracked
+    assert_equal [file_path], tracker.used_views
+  end
+
+  test "report unused partials" do
+    Coverband::Collectors::ViewTracker.expects(:supported_version?).returns(true)
+    store = fake_store
+    file_path = "#{File.expand_path(Coverband.configuration.root)}/file"
+    target = [file_path, 'not_used']
+    tracker = Coverband::Collectors::ViewTracker.new(store: store, roots: 'dir', target: target)
+    tracker.track_views('name', 'start', 'finish', 'id', {:identifier => file_path})
+    tracker.report_views_tracked
+    assert_equal ['not_used'], tracker.unused_views
+  end
+
+  test "reset store" do
+    Coverband::Collectors::ViewTracker.expects(:supported_version?).returns(true)
+    store = fake_store
+    store.raw_store.expects(:del).with('render_tracker')
+    tracker = Coverband::Collectors::ViewTracker.new(store: store, roots: 'dir')
+    tracker.track_views('name', 'start', 'finish', 'id', {:identifier => 'file'})
+    tracker.reset_recordings
+  end
+
+  protected
+
+  def fake_store
+    @fake_store ||= Coverband::Adapters::RedisStore.new(Redis.new, redis_namespace: 'coverband_test')
+  end
+
+end

data/test/coverband/integrations/resque_worker_test.rb

@@ -15,16 +15,15 @@ class ResqueWorkerTest < Minitest::Test
     super
     Coverband.configure do |config|
       config.background_reporting_enabled = false
-      config.root_paths = ["#{File.expand_path('../', File.dirname(__FILE__))}/"]
     end
     Coverband.configuration.store.instance_variable_set(:@redis_namespace, 'coverband_test')
     Coverband.start
-    redis = Coverband.configuration.store.send(:redis)
+    redis = Coverband.configuration.store.instance_eval { @redis }
     Resque.redis = redis
   end
 
   test 'resque job coverage' do
-    relative_job_file = './integrations/test_resque_job.rb'
+    relative_job_file = './test/coverband/integrations/test_resque_job.rb'
     resque_job_file = File.expand_path('./test_resque_job.rb', File.dirname(__FILE__))
     require resque_job_file
 

data/test/coverband/reporters/base_test.rb

@@ -3,74 +3,6 @@
 require File.expand_path('../../test_helper', File.dirname(__FILE__))
 
 class ReportsBaseTest < Minitest::Test
-  test 'relative_path_to_full fix filename from a key with a swappable path' do
-    Coverband.configure do |config|
-      config.reporter          = 'std_out'
-      config.root              = '/full/remote_app/path'
-    end
-
-    key = '/app/is/a/path.rb'
-    # the code takes config.root expands and adds a '/' for the final path in roots
-    roots = ['/app/', '/full/remote_app/path/']
-
-    expected_path = '/full/remote_app/path/is/a/path.rb'
-    File.expects(:exist?).with(key).returns(false)
-    File.expects(:exist?).with(expected_path).returns(true)
-    assert_equal expected_path, Coverband::Reporters::Base.send(:relative_path_to_full, key, roots)
-  end
-
-  test 'relative_path_to_full fix filename a changing deploy path with quotes' do
-    Coverband.configure do |config|
-      config.reporter          = 'std_out'
-      config.root              = '/full/remote_app/path'
-    end
-
-    expected_path = '/full/remote_app/path/app/models/user.rb'
-    key = '/box/apps/app_name/releases/20140725203539/app/models/user.rb'
-    roots = ['/box/apps/app_name/releases/\\d+/', '/full/remote_app/path/']
-    File.expects(:exist?).with('/box/apps/app_name/releases/\\d+/app/models/user.rb').returns(false)
-    File.expects(:exist?).with(expected_path).returns(true)
-    assert_equal expected_path, Coverband::Reporters::Base.send(:relative_path_to_full, key, roots)
-    File.expects(:exist?).with('/box/apps/app_name/releases/\\d+/app/models/user.rb').returns(false)
-    File.expects(:exist?).with(expected_path).returns(true)
-    roots = ['/box/apps/app_name/releases/\d+/', '/full/remote_app/path/']
-    assert_equal expected_path, Coverband::Reporters::Base.send(:relative_path_to_full, key, roots)
-  end
-
-  test 'relative_path_to_full fix filename a changing deploy path real world examples' do
-    current_app_root = '/var/local/company/company.d/79'
-    Coverband.configure do |config|
-      config.reporter          = 'std_out'
-      config.root              = current_app_root
-    end
-
-    expected_path = '/var/local/company/company.d/79/app/controllers/dashboard_controller.rb'
-    key = '/var/local/company/company.d/78/app/controllers/dashboard_controller.rb'
-
-    File.expects(:exist?).with('/var/local/company/company.d/[0-9]*/app/controllers/dashboard_controller.rb').returns(false)
-    File.expects(:exist?).with(expected_path).returns(true)
-    roots = ['/var/local/company/company.d/[0-9]*/', "#{current_app_root}/"]
-    assert_equal expected_path, Coverband::Reporters::Base.send(:relative_path_to_full, key, roots)
-    File.expects(:exist?).with('/var/local/company/company.d/[0-9]*/app/controllers/dashboard_controller.rb').returns(false)
-    File.expects(:exist?).with(expected_path).returns(true)
-    roots = ['/var/local/company/company.d/[0-9]*/', "#{current_app_root}/"]
-    assert_equal expected_path, Coverband::Reporters::Base.send(:relative_path_to_full, key, roots)
-  end
-
-  test 'relative_path_to_full leave filename from a key with a local path' do
-    Coverband.configure do |config|
-      config.reporter          = 'std_out'
-      config.root              = '/full/remote_app/path'
-    end
-
-    key = '/full/remote_app/path/is/a/path.rb'
-    # the code takes config.root expands and adds a '/' for the final path in roots
-    roots = ['/app/', '/full/remote_app/path/']
-
-    expected_path = '/full/remote_app/path/is/a/path.rb'
-    assert_equal expected_path, Coverband::Reporters::Base.send(:relative_path_to_full, key, roots)
-  end
-
   test '#merge_arrays basic merge preserves order and counts' do
     first = [0, 0, 1, 0, 1]
     second = [nil, 0, 1, 0, 0]
@@ -139,19 +71,10 @@ class ReportsBaseTest < Minitest::Test
         "file_hash"=>"14dc84e940e26cbfb9ac79b43862e762",
         "data"=>[16, 16, 16, nil, 16, 16, nil, nil, 16, nil, 16, 32, 32, nil, 32, 32, 32, 32, 32, 32, 32, nil, nil, 16, nil, 16, 32, 23, 0, 0, 0, 0, nil, nil, nil, nil, 0, 0, nil, nil, 16, 32, 32, 32, nil, nil, nil, nil, nil, 16, 32, nil, nil, 16, 32, nil, nil]}
       }
-    @redis = Redis.new
-    store = Coverband::Adapters::RedisStore.new(@redis, redis_namespace: 'coverband_test')
-    store.clear!
-
-    Coverband.configure do |config|
-      config.reporter          = 'std_out'
-      config.root              = '/base/78/app/'
-      config.store             = store
-    end
-
     key = '/base/78/app/app/controllers/dashboard_controller.rb'
     roots = ['/base/[0-9]*/', '/base/78/app/']
 
+    Coverband.configuration.stubs(:all_root_paths).returns(roots)
     lines_hit = [1, 3, 6]
     store.stubs(:merged_coverage).returns(coverage)
     File.expects(:exist?).at_least_once

data/test/coverband/reporters/console_test.rb

@@ -19,10 +19,7 @@ class HTMLReportTest < Minitest::Test
     end
     Coverband.configuration.logger.stubs('info')
     mock_file_hash
-    Coverband.configuration
-             .expects(:current_root)
-             .at_least_once
-             .returns('app_path')
+    Coverband::Utils::RelativeFileConverter.expects(:convert).with('app_path/dog.rb').returns('./dog.rb')
     @store.send(:save_report, basic_coverage)
 
     report = Coverband::Reporters::ConsoleReport.report(@store)[:merged]