coverband 4.2.0 → 4.2.4

data/test/coverband/reporters/base_test.rb

@@ -3,72 +3,8 @@
 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)
+  def setup
+    super
   end
 
   test '#merge_arrays basic merge preserves order and counts' do
@@ -96,14 +32,9 @@ class ReportsBaseTest < Minitest::Test
   end
 
   test "#get_current_scov_data_imp doesn't ignore folders with default ignore keys" do
-    @redis = Redis.new
-    store = Coverband::Adapters::RedisStore.new(@redis)
-    store.clear!
-
     Coverband.configure do |config|
       config.reporter          = 'std_out'
       config.root              = '/full/remote_app/path'
-      config.store             = store
     end
 
     key = '/a_path/that_has_erb_in/thepath.rb'
@@ -123,6 +54,7 @@ class ReportsBaseTest < Minitest::Test
   # rubocop:disable all
   ###
   test '#get_current_scov_data_imp merges multiples of file data' do
+    skip "@danmayer to take a closer look at this one"
     coverage = {'/base/66/app/controllers/dashboard_controller.rb' =>
        {"first_updated_at"=>1549610119,
         "last_updated_at"=>1549610200,
@@ -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)
-    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
@@ -165,4 +88,5 @@ class ReportsBaseTest < Minitest::Test
                 "data"=>[38, 38, 38, nil, 38, 38, nil, nil, 38, nil, 38, 80, 80, nil, 80, 80, 80, 80, 80, 80, 80, nil, nil, 38, nil, 38, 80, 55, 0, 0, 0, 0, nil, nil, nil, nil, 0, 0, nil, nil, 38, 80, 80, 80, nil, nil, nil, nil, nil, 38, 80, nil, nil, 38, 80, nil, nil]}
     assert_equal expected, Coverband::Reporters::Base.send(:get_current_scov_data_imp, store, roots)[:merged][key]
   end
+
 end

data/test/coverband/reporters/console_test.rb

@@ -7,23 +7,16 @@ class HTMLReportTest < Minitest::Test
 
   def setup
     super
-    @redis = Redis.new
-    @store = Coverband::Adapters::RedisStore.new(@redis)
-    @store.clear!
+    @store = Coverband.configuration.store
   end
 
   test 'report data' do
     Coverband.configure do |config|
-      config.reporter            = 'std_out'
-      config.store               = @store
-      config.reporting_frequency = 100.0
+      config.reporter = 'std_out'
     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]

data/test/coverband/reporters/html_test.rb

@@ -5,41 +5,91 @@ require File.expand_path('../../test_helper', File.dirname(__FILE__))
 class ReportHTMLTest < Minitest::Test
   def setup
     super
-    @redis = Redis.new
-    @store = Coverband::Adapters::RedisStore.new(@redis)
-    @store.clear!
-  end
-
-  test 'generate scov html report' do
+    @store = Coverband.configuration.store
     Coverband.configure do |config|
-      config.reporter          = 'scov'
       config.store             = @store
-      config.s3_bucket         = nil
-      config.ignore            = ['notsomething.rb']
+      config.root              = fixtures_root
+      config.ignore            = ['notsomething.rb', 'lib/*']
     end
     mock_file_hash
+  end
+
+  test 'generate dynamic content hosted html report' do
     @store.send(:save_report, basic_coverage)
 
     html = Coverband::Reporters::HTMLReport.new(@store,
-                                                   html: true,
-                                                   open_report: false).report
+                                                static: false,
+                                                open_report: false).report
     assert_match 'Generated by', html
   end
 
-  test 'generate scov report file' do
-    Coverband.configure do |config|
-      config.reporter          = 'scov'
-      config.store             = @store
-      config.s3_bucket         = nil
-      config.ignore            = ['notsomething.rb']
-    end
-    mock_file_hash
+  test 'files with no Coverage but in project are shown in reports' do
+    @store.send(:save_report, basic_source_fixture_coverage)
+
+    html = Coverband::Reporters::HTMLReport.new(@store,
+                                                static: false,
+                                                open_report: false).report
+    assert_match 'sample.rb', html
+    # in project, but not in coverage data
+    assert_match 'app/models/user.rb', html
+  end
+
+  test 'files with no Coverage but in project details page list warning' do
+    @store.send(:save_report, basic_coverage_full_path)
+
+    filename = basic_coverage_file_full_path
+    base_path = Dir.pwd
+    # in project, but not in coverage data
+    html = Coverband::Reporters::HTMLReport.new(Coverband.configuration.store,
+                                                filename: "#{Dir.pwd}/test/fixtures/app/models/user.rb",
+                                                base_path: base_path,
+                                                open_report: false).file_details
+    assert_match 'This file was never loaded', html
+  end
+
+  test 'generate static HTML report file' do
     @store.send(:save_report, basic_coverage)
 
     reporter = Coverband::Reporters::HTMLReport.new(@store,
-                                                    html: false,
+                                                    static: true,
                                                     open_report: false)
-    Coverband::Utils::HTMLFormatter.any_instance.expects(:format!).once
+    Coverband::Utils::HTMLFormatter.any_instance.expects(:format_static_html!).once
     reporter.report
   end
+
+  test 'generate dynamic content detailed file report' do
+    @store.send(:save_report, basic_coverage_full_path)
+
+    filename = basic_coverage_file_full_path
+    base_path = '/coverage'
+    html = Coverband::Reporters::HTMLReport.new(Coverband.configuration.store,
+                                                filename: filename,
+                                                base_path: base_path,
+                                                open_report: false).file_details
+    assert_match 'Coverage first seen', html
+  end
+
+  test 'generate dynamic content detailed file report handles missing file' do
+    @store.send(:save_report, basic_coverage_full_path)
+
+    filename = 'missing_path'
+    base_path = '/coverage'
+    html = Coverband::Reporters::HTMLReport.new(Coverband.configuration.store,
+                                                filename: filename,
+                                                base_path: base_path,
+                                                open_report: false).file_details
+    assert_match 'File No Longer Available', html
+  end
+
+  test 'generate dynamic content detailed file report does not allow loading real non project files' do
+    @store.send(:save_report, basic_coverage_full_path)
+
+    filename = "#{test_root}/test_helper.rb"
+    base_path = '/coverage'
+    html = Coverband::Reporters::HTMLReport.new(Coverband.configuration.store,
+                                                filename: filename,
+                                                base_path: base_path,
+                                                open_report: false).file_details
+    assert_match 'File No Longer Available', html
+  end
 end

data/test/coverband/reporters/web_test.rb

@@ -37,16 +37,6 @@ if Gem::Version.new(RUBY_VERSION) >= Gem::Version.new('2.2.0')
         post '/clear'
         assert_equal 301, last_response.status
       end
-
-      test 'collect_coverage' do
-        post '/collect_coverage'
-        assert_equal 301, last_response.status
-      end
-
-      test 'reload_files' do
-        post '/reload_files'
-        assert_equal 301, last_response.status
-      end
     end
   end
 end

data/test/coverband/utils/absolute_file_converter_test.rb

@@ -0,0 +1,56 @@
+# frozen_string_literal: true
+
+require File.expand_path('../../test_helper', File.dirname(__FILE__))
+
+module Coverband
+  module Utils
+    class AbsoluteFileConverterTest < ::Minitest::Test
+      def test_convert
+        converter = AbsoluteFileConverter.new([FileUtils.pwd])
+        assert_equal("#{FileUtils.pwd}/lib/coverband.rb", converter.convert('./lib/coverband.rb'))
+      end
+
+      def test_convert_multiple_roots
+        converter = AbsoluteFileConverter.new(['/foo/bar', FileUtils.pwd])
+        assert_equal("#{FileUtils.pwd}/Rakefile", converter.convert('./Rakefile'))
+      end
+
+      test 'relative_path_to_full leave filename from a key with a local path' do
+        converter = AbsoluteFileConverter.new(['/app/', '/full/remote_app/path/'])
+        assert_equal '/full/remote_app/path/is/a/path.rb', converter.convert('/full/remote_app/path/is/a/path.rb')
+      end
+
+      test 'relative_path_to_full fix filename from a key with a swappable path' do
+        key = '/app/is/a/path.rb'
+        converter = AbsoluteFileConverter.new(['/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, converter.convert(key)
+      end
+
+      test 'relative_path_to_full fix filename a changing deploy path with quotes' do
+        converter = AbsoluteFileConverter.new(['/box/apps/app_name/releases/\\d+/', '/full/remote_app/path/'])
+        expected_path = '/full/remote_app/path/app/models/user.rb'
+        key = '/box/apps/app_name/releases/20140725203539/app/models/user.rb'
+        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, converter.convert(key)
+        assert_equal expected_path, converter.convert(key)
+      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'
+        converter = AbsoluteFileConverter.new(['/var/local/company/company.d/[0-9]*/', "#{current_app_root}/"])
+
+        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, converter.convert(key)
+        assert_equal expected_path, converter.convert(key)
+      end
+    end
+  end
+end

data/test/coverband/utils/file_groups_test.rb

@@ -3,16 +3,18 @@
 require File.expand_path('../../test_helper', File.dirname(__FILE__))
 
 describe Coverband::Utils::FileGroups do
-  FAKE_GEM_PATH = 'fake/gem/path'
+  FAKE_GEM_PATH = '/Users/danmayer/.rvm/gems/ruby-2.6.2/gems'
+  SECONDAY_GEM_PATH = '/Users/danmayer/.rvm/rubies/ruby-2.6.2/lib/ruby/gems/2.6.0/gems'
   subject do
     controller_lines = [nil, 2, 2, 0, nil, nil, 0, nil, nil, nil]
     files = [
       Coverband::Utils::SourceFile.new(source_fixture('sample.rb'), [nil, 1, 1, 1, nil, nil, 1, 1, nil, nil]),
       Coverband::Utils::SourceFile.new(source_fixture('app/models/user.rb'), [nil, 1, 1, 1, nil, nil, 1, 0, nil, nil]),
       Coverband::Utils::SourceFile.new(source_fixture('app/controllers/sample_controller.rb'), controller_lines),
-      Coverband::Utils::SourceFile.new("#{FAKE_GEM_PATH}/gem_name.rb", controller_lines)
+      Coverband::Utils::SourceFile.new("#{FAKE_GEM_PATH}/gem_name-1.0.0/gem_name.rb", controller_lines),
+      Coverband::Utils::SourceFile.new("#{SECONDAY_GEM_PATH}/gem_two_name-1.1.1/gem_two.rb", controller_lines)
     ]
-    Coverband.configuration.expects(:gem_paths).at_least_once.returns([FAKE_GEM_PATH])
+    Coverband.configuration.expects(:gem_paths).at_least_once.returns([FAKE_GEM_PATH, SECONDAY_GEM_PATH])
     Coverband.configuration.track_gems = true
     Coverband::Utils::FileGroups.new(files)
   end
@@ -22,7 +24,11 @@ describe Coverband::Utils::FileGroups do
   end
 
   it 'has gem files' do
-    assert_equal "#{FAKE_GEM_PATH}/gem_name.rb", subject.grouped_results['Gems'].first.first.short_name
+    assert_equal "gem_name-1.0.0", subject.grouped_results['Gems'].first.first.gem_name
+  end
+
+  it 'has gem files from secondary gem paths' do
+    assert_equal "gem_two_name-1.1.1", subject.grouped_results['Gems'][1].first.gem_name
   end
 end
 
@@ -50,6 +56,6 @@ describe Coverband::Utils::FileGroups, :vendored_gems do
   end
 
   it 'does has gem files' do
-    assert_equal 'gem_name.rb', subject.grouped_results['Gems'].first.first.short_name
+    assert_equal '.gem_name.rb', subject.grouped_results['Gems'].first.first.short_name
   end
 end

data/test/coverband/utils/file_hasher_test.rb

@@ -0,0 +1,29 @@
+# frozen_string_literal: true
+
+require File.expand_path('../../test_helper', File.dirname(__FILE__))
+
+module Coverband
+  module Utils
+    class FileHasherTest < Minitest::Test
+      def test_hash_same_file
+        refute_nil FileHasher.hash('./test/dog.rb')
+        assert_equal(FileHasher.hash('./test/dog.rb'), FileHasher.hash('./test/dog.rb'))
+        assert_equal(FileHasher.hash(File.expand_path('./test/dog.rb')), FileHasher.hash('./test/dog.rb'))
+      end
+
+      def test_hash_different_files
+        refute_equal(FileHasher.hash('./test/dog.rb'), FileHasher.hash('./lib/coverband.rb'))
+      end
+
+      def test_hash_file_not_exists
+        assert_nil(FileHasher.hash('./made_up_file.py'))
+      end
+
+      def test_hash_gem_file
+        spec = Gem::Specification.find_by_name('rainbow')
+        assert FileHasher.hash("./gems/rainbow-#{spec.version}/lib/rainbow.rb",
+                               path_converter: AbsoluteFileConverter.new(Coverband.configuration.gem_paths))
+      end
+    end
+  end
+end

data/test/coverband/utils/file_list_test.rb

@@ -9,7 +9,7 @@ require File.expand_path('../../test_helper', File.dirname(__FILE__))
 describe Coverband::Utils::FileList do
   subject do
     original_result = {
-      source_fixture('sample.rb') => [nil, 1, 1, 1, nil, nil, 1, 1, nil, nil],
+      source_fixture('sample.rb') => {'first_updated_at' => Time.at(0), 'data' => [nil, 1, 1, 1, nil, nil, 1, 1, nil, nil]},
       source_fixture('app/models/user.rb') => [nil, 1, 1, 1, nil, nil, 1, 0, nil, nil],
       source_fixture('app/controllers/sample_controller.rb') => [nil, 2, 2, 0, nil, nil, 0, nil, nil, nil]
     }
@@ -44,11 +44,11 @@ describe Coverband::Utils::FileList do
     assert_equal [50.0, 80.0, 100.0], subject.covered_percentages
   end
 
-  it 'has the correct least covered file' do
-    assert subject.least_covered_file.match(/sample_controller.rb/)
-  end
-
   it 'has the correct covered strength' do
     assert_equal 0.9285714285714286, subject.covered_strength
   end
+
+  it 'has correct first_seen_at' do
+    assert_equal Time.at(0), subject.first_seen_at
+  end
 end

data/test/coverband/utils/html_formatter_test.rb

@@ -0,0 +1,41 @@
+# frozen_string_literal: true
+
+require File.expand_path('../../test_helper', File.dirname(__FILE__))
+
+class HTMLFormatterTest < Minitest::Test
+  def setup
+    super
+    @store = Coverband::Adapters::RedisStore.new(Coverband::Test.redis, redis_namespace: 'coverband_test')
+  end
+
+  test 'generate dynamic content hosted html report' do
+    Coverband.configure do |config|
+      config.store             = @store
+      config.ignore            = ['notsomething.rb']
+    end
+    mock_file_hash
+    @store.send(:save_report, basic_coverage_full_path)
+
+    notice = nil
+    base_path = '/coverage'
+    filtered_report_files = Coverband::Reporters::Base.report(@store, {})
+    html = Coverband::Utils::HTMLFormatter.new(filtered_report_files,
+                                               base_path: base_path,
+                                               notice: notice).format_dynamic_html!
+    assert_match 'loading source data', html
+  end
+
+  test 'generate static HTML report file' do
+    Coverband.configure do |config|
+      config.store             = @store
+      config.ignore            = ['notsomething.rb']
+    end
+    mock_file_hash
+    @store.send(:save_report, basic_coverage_full_path)
+
+    filtered_report_files = Coverband::Reporters::Base.report(@store, {})
+    Coverband::Utils::HTMLFormatter.new(filtered_report_files).format_static_html!
+    html = File.read("#{Coverband.configuration.root}/coverage/index.html")
+    assert_match 'Coverage first seen', html
+  end
+end

data/test/coverband/utils/relative_file_converter_test.rb

@@ -0,0 +1,39 @@
+# frozen_string_literal: true
+
+require File.expand_path('../../test_helper', File.dirname(__FILE__))
+
+module Coverband
+  module Utils
+    class RelativeFileConverterTest < ::Minitest::Test
+      def test_convert
+        converter = RelativeFileConverter.new(['/bar/tmp/'])
+        assert_equal('./gracie.rb', converter.convert('/bar/tmp/gracie.rb'))
+      end
+
+      def test_convert_without_leading_forward_slash
+        converter = RelativeFileConverter.new(['/foo/bar'])
+        assert_equal('./file.rb', converter.convert('/foo/bar/file.rb'))
+      end
+
+      def test_multiple_roots
+        converter = RelativeFileConverter.new(['/bar/tmp/', '/foo/bar/'])
+        assert_equal('./josie.rb', converter.convert('/foo/bar/josie.rb'))
+      end
+
+      def test_no_match
+        converter = RelativeFileConverter.new(['/bar/tmp/', '/foo/bar/'])
+        assert_equal('/foo/josie.rb', converter.convert('/foo/josie.rb'))
+      end
+
+      def test_middle_path_match
+        converter = RelativeFileConverter.new(['/bar/tmp/', '/foo/bar/'])
+        assert_equal('/tmp/foo/bar/josie.rb', converter.convert('/tmp/foo/bar/josie.rb'))
+      end
+
+      def test_already_relative_file
+        converter = RelativeFileConverter.new(['/bar/tmp/', '/foo/bar/'])
+        assert_equal('./foo/bar/josie.rb', converter.convert('./foo/bar/josie.rb'))
+      end
+    end
+  end
+end

data/test/coverband/utils/result_test.rb

@@ -48,57 +48,16 @@ describe 'result' do
         assert_equal [80.0, 80.0, 100.0], subject.covered_percentages
       end
 
-      it 'has accurate least covered file' do
-        assert subject.least_covered_file.match(/sample_controller.rb/)
-      end
-
-      [:covered_percent,
-       :covered_percentages,
-       :least_covered_file,
-       :covered_strength,
-       :covered_lines,
-       :missed_lines,
-       :total_lines].each do |msg|
+      %i[covered_percent
+         covered_percentages
+         covered_strength
+         covered_lines
+         missed_lines
+         total_lines].each do |msg|
         it "responds to #{msg}" do
           assert(subject.respond_to?(msg))
         end
       end
-
-      describe 'dumped with to_hash' do
-        it 'is a hash' do
-          assert subject.to_hash.is_a?(Hash)
-        end
-
-        describe 'loaded back with from_hash' do
-          let(:dumped_result) do
-            Coverband::Utils::Result.from_hash(subject.to_hash)
-          end
-
-          it 'has 3 source files' do
-            assert_equal subject.source_files.count, dumped_result.source_files.count
-          end
-
-          it 'has the same covered_percent' do
-            assert_equal subject.covered_percent, dumped_result.covered_percent
-          end
-
-          it 'has the same covered_percentages' do
-            assert_equal subject.covered_percentages, dumped_result.covered_percentages
-          end
-
-          it 'has the same timestamp' do
-            assert_equal subject.created_at.to_i, dumped_result.created_at.to_i
-          end
-
-          it 'has the same command_name' do
-            assert_equal subject.command_name, dumped_result.command_name
-          end
-
-          it 'has the same original_result' do
-            assert_equal subject.original_result, dumped_result.original_result
-          end
-        end
-      end
     end
   end
 end

data/test/coverband/utils/results_test.rb

@@ -0,0 +1,54 @@
+# frozen_string_literal: true
+
+require File.expand_path('../../test_helper', File.dirname(__FILE__))
+
+describe 'results' do
+  describe 'with a (mocked) Coverage.result' do
+    let(:source_file) { Coverband::Utils::SourceFile.new(source_fixture('app/models/user.rb'), run_lines) }
+    let(:eager_lines) { [nil, 1, 1, 0, nil, nil, 1, 0, nil, nil] }
+    let(:run_lines) { [nil, nil, nil, 1, nil, nil, nil,nil, nil, nil] }
+    let(:original_result) do
+      orig = {
+        Coverband::MERGED_TYPE => {source_fixture('app/models/user.rb') => eager_lines},
+      }
+      orig.merge!({Coverband::EAGER_TYPE => {source_fixture('app/models/user.rb') => eager_lines}}) if eager_lines
+      orig.merge!({Coverband::RUNTIME_TYPE => {source_fixture('app/models/user.rb') => run_lines}}) if run_lines
+      orig
+    end
+    subject { Coverband::Utils::Results.new(original_result) }
+
+    describe 'runtime relevant lines is supported' do
+      it 'has correct runtime relevant coverage' do
+        assert_equal 50.0, subject.runtime_relevant_coverage(source_file)
+      end
+
+      it 'has correct runtime relevant lines' do
+        assert_equal 2, subject.runtime_relavent_lines(source_file)
+      end
+    end
+
+    describe 'runtime relevant lines when no runtime coverage exists' do
+      let(:run_lines) { nil }
+
+      it 'has correct runtime relevant lines' do
+        assert_equal 0.0, subject.runtime_relevant_coverage(source_file)
+      end
+
+      it 'has correct runtime relevant lines' do
+        assert_equal 2, subject.runtime_relavent_lines(source_file)
+      end
+    end
+
+    describe 'runtime relevant lines when no eager coverage exists' do
+      let(:eager_lines) { nil }
+
+      it 'has correct runtime relevant lines' do
+        assert_equal 100.0, subject.runtime_relevant_coverage(source_file)
+      end
+
+      it 'has correct runtime relevant lines' do
+        assert_equal 1, subject.runtime_relavent_lines(source_file)
+      end
+    end
+  end
+end

data/test/coverband/utils/s3_report_test.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require File.expand_path('../../test_helper', File.dirname(__FILE__))
 
 module Coverband