coverband 4.2.7 → 5.0.0.rc.3

data/test/coverband/configuration_test.rb

@@ -1,18 +1,17 @@
 # frozen_string_literal: true
 
-require File.expand_path('../test_helper', File.dirname(__FILE__))
+require File.expand_path("../test_helper", File.dirname(__FILE__))
 
 class BaseTest < Minitest::Test
   def setup
     super
     Coverband.configuration.reset
     Coverband.configure do |config|
-      config.root                = Dir.pwd
-      config.s3_bucket           = nil
-      config.root_paths          = ['/app_path/']
-      config.ignore              = ['config/envionments']
-      config.reporter            = 'std_out'
-      config.store               = Coverband::Adapters::RedisStore.new(Coverband::Test.redis, redis_namespace: 'coverband_test')
+      config.root = Dir.pwd
+      config.root_paths = ["/app_path/"]
+      config.ignore = ["config/envionments"]
+      config.reporter = "std_out"
+      config.store = Coverband::Adapters::RedisStore.new(Coverband::Test.redis, redis_namespace: "coverband_test")
     end
   end
 
@@ -22,9 +21,9 @@ class BaseTest < Minitest::Test
     assert_equal expected, Coverband.configuration.ignore
   end
 
-  test 'ignore works with plus equal' do
+  test "ignore works with plus equal" do
     Coverband.configure do |config|
-      config.ignore += ['config/initializers']
+      config.ignore += ["config/initializers"]
     end
     Coverband::Collectors::Coverage.instance.reset_instance
     expected = ["vendor/",
@@ -38,23 +37,12 @@ class BaseTest < Minitest::Test
     assert_equal expected, Coverband.configuration.ignore
   end
 
-  test 'ignore' do
+  test "ignore" do
     Coverband::Collectors::Coverage.instance.reset_instance
-    assert !Coverband.configuration.gem_paths.first.nil?
+    assert !Coverband.configuration.ignore.first.nil?
   end
 
-  test 'gem_paths' do
-    Coverband::Collectors::Coverage.instance.reset_instance
-    assert !Coverband.configuration.gem_paths.first.nil?
-  end
-
-  test 'groups' do
-    Coverband::Collectors::Coverage.instance.reset_instance
-    Coverband.configuration.track_gems = true
-    assert_equal %w[App Gems], Coverband.configuration.groups.keys
-  end
-
-  test 'all_root_paths' do
+  test "all_root_paths" do
     Coverband::Collectors::Coverage.instance.reset_instance
     current_paths = Coverband.configuration.root_paths.dup
     # verify previous bug fix
@@ -64,30 +52,16 @@ class BaseTest < Minitest::Test
     assert_equal current_paths, Coverband.configuration.root_paths
   end
 
-  test 's3 options' do
-    Coverband::Collectors::Coverage.instance.reset_instance
-    Coverband.configure do |config|
-      config.s3_bucket = 'bucket'
-      config.s3_region = 'region'
-      config.s3_access_key_id = 'key_id'
-      config.s3_secret_access_key = 'secret'
-    end
-    assert_equal 'bucket', Coverband.configuration.s3_bucket
-    assert_equal 'region', Coverband.configuration.s3_region
-    assert_equal 'key_id', Coverband.configuration.s3_access_key_id
-    assert_equal 'secret', Coverband.configuration.s3_secret_access_key
-  end
-
-  test 'store raises issues' do
+  test "store raises issues" do
     Coverband::Collectors::Coverage.instance.reset_instance
     assert_raises RuntimeError do
       Coverband.configure do |config|
-        config.store = 'fake'
+        config.store = "fake"
       end
     end
   end
 
-  test 'use_oneshot_lines_coverage' do
+  test "use_oneshot_lines_coverage" do
     refute Coverband.configuration.use_oneshot_lines_coverage
 
     Coverband.configuration.stubs(:one_shot_coverage_implemented_in_ruby_version?).returns(true)
@@ -101,7 +75,7 @@ class BaseTest < Minitest::Test
     exception = assert_raises Exception do
       Coverband.configuration.use_oneshot_lines_coverage = true
     end
-    assert_equal 'One shot line coverage is only available in ruby >= 2.6', exception.message
+    assert_equal "One shot line coverage is only available in ruby >= 2.6", exception.message
     refute Coverband.configuration.use_oneshot_lines_coverage
   end
 end

data/test/coverband/coverband_test.rb

@@ -1,42 +1,42 @@
 # frozen_string_literal: true
 
-require File.expand_path('../test_helper', File.dirname(__FILE__))
+require File.expand_path("../test_helper", File.dirname(__FILE__))
 
 class CoverbandTest < Minitest::Test
-  test 'Coverband#start kicks off background reporting if enabled and not in rack server' do
-    Coverband.configuration.stubs(:background_reporting_enabled).returns(:true)
+  test "Coverband#start kicks off background reporting if enabled and not in rack server" do
+    Coverband.configuration.stubs(:background_reporting_enabled).returns(true)
     Coverband::RackServerCheck.expects(:running?).returns(false)
     Coverband::Background.expects(:start)
     Coverband.start
   end
 
-  test 'Coverband#start delays background reporting if enabled and running in a rack server' do
+  test "Coverband#start delays background reporting if enabled and running in a rack server" do
     Coverband.configuration.stubs(:background_reporting_enabled).returns(true)
     Coverband::RackServerCheck.expects(:running?).returns(true)
     Coverband::Background.expects(:start).never
     Coverband.start
   end
 
-  test 'Coverband#start does not kick off background reporting if not enabled' do
+  test "Coverband#start does not kick off background reporting if not enabled" do
     Coverband.configuration.stubs(:background_reporting_enabled).returns(false)
     Coverband::Background.expects(:start).never
     ::Coverband.start
   end
 
-  test 'Coverband#configured? works' do
+  test "Coverband#configured? works" do
     Coverband.configure
     assert Coverband.configured?
   end
 
-  test 'Eager load coverage block' do
-    Coverband.eager_loading_coverage {
-      #some code
+  test "Eager load coverage block" do
+    Coverband.eager_loading_coverage do
+      # some code
       1 + 1
-    }
+    end
     assert_equal :runtime, Coverband.configuration.store.type
   end
 
-  test 'Eager load coverage' do
+  test "Eager load coverage" do
     Coverband.eager_loading_coverage!
     assert_equal :eager_loading, Coverband.configuration.store.type
     Coverband.runtime_coverage!

data/test/coverband/integrations/background_middleware_test.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
-require File.expand_path('../../test_helper', File.dirname(__FILE__))
-require 'rack'
+require File.expand_path("../../test_helper", File.dirname(__FILE__))
+require "rack"
 
 class BackgroundMiddlewareTest < Minitest::Test
   def setup
@@ -11,23 +11,23 @@ class BackgroundMiddlewareTest < Minitest::Test
     end
   end
 
-  test 'call app' do
-    request = Rack::MockRequest.env_for('/anything.json')
+  test "call app" do
+    request = Rack::MockRequest.env_for("/anything.json")
     Coverband::Collectors::Coverage.instance.reset_instance
     middleware = Coverband::BackgroundMiddleware.new(fake_app)
     results = middleware.call(request)
-    assert_equal '/anything.json', results.last
+    assert_equal "/anything.json", results.last
   end
 
-  test 'pass all rack lint checks' do
+  test "pass all rack lint checks" do
     Coverband::Collectors::Coverage.instance.reset_instance
     app = Rack::Lint.new(Coverband::BackgroundMiddleware.new(fake_app))
-    env = Rack::MockRequest.env_for('/hello')
+    env = Rack::MockRequest.env_for("/hello")
     app.call(env)
   end
 
-  test 'starts background reporter when configured' do
-    request = Rack::MockRequest.env_for('/anything.json')
+  test "starts background reporter when configured" do
+    request = Rack::MockRequest.env_for("/anything.json")
     Coverband.configuration.stubs(:background_reporting_enabled).returns(true)
     Coverband::Background.expects(:start)
     middleware = Coverband::BackgroundMiddleware.new(fake_app)
@@ -38,7 +38,7 @@ class BackgroundMiddlewareTest < Minitest::Test
 
   def fake_app
     @fake_app ||= lambda do |env|
-      [200, { 'Content-Type' => 'text/plain' }, env['PATH_INFO']]
+      [200, {"Content-Type" => "text/plain"}, env["PATH_INFO"]]
     end
   end
 end

data/test/coverband/integrations/background_test.rb

@@ -1,10 +1,11 @@
 # frozen_string_literal: true
 
-require File.expand_path('../../test_helper', File.dirname(__FILE__))
+require File.expand_path("../../test_helper", File.dirname(__FILE__))
 
 class BackgroundTest < Minitest::Test
   class ThreadDouble < Struct.new(:alive)
-    def exit; end
+    def exit
+    end
 
     def alive?
       alive

data/test/coverband/integrations/rack_server_check_test.rb

@@ -1,22 +1,22 @@
 # frozen_string_literal: true
 
-require File.expand_path('../../test_helper', File.dirname(__FILE__))
+require File.expand_path("../../test_helper", File.dirname(__FILE__))
 
 class RackServerCheckTest < Minitest::Test
-  test 'returns true when running in rack server' do
-    caller_locations = ['blah/lib/rack/server.rb'].map { |path| OpenStruct.new(path: path, label: 'foo') }
+  test "returns true when running in rack server" do
+    caller_locations = ["blah/lib/rack/server.rb"].map { |path| OpenStruct.new(path: path, label: "foo") }
     Kernel.expects(:caller_locations).returns(caller_locations)
     assert(Coverband::RackServerCheck.running?)
   end
 
-  test 'returns false when not running in rack server' do
-    caller_locations = ['blah/lib/sidekiq/worker.rb'].map { |path| OpenStruct.new(path: path, label: 'foo') }
+  test "returns false when not running in rack server" do
+    caller_locations = ["blah/lib/sidekiq/worker.rb"].map { |path| OpenStruct.new(path: path, label: "foo") }
     Kernel.expects(:caller_locations).returns(caller_locations)
     refute(Coverband::RackServerCheck.running?)
   end
 
-  test 'returns true if running within a rails server' do
-    caller_locations = [OpenStruct.new(path: '/lib/rails/commands/commands_tasks.rb', label: 'server')]
+  test "returns true if running within a rails server" do
+    caller_locations = [OpenStruct.new(path: "/lib/rails/commands/commands_tasks.rb", label: "server")]
     Kernel.expects(:caller_locations).returns(caller_locations)
     assert(Coverband::RackServerCheck.running?)
   end

data/test/coverband/integrations/report_middleware_test.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
-require File.expand_path('../../test_helper', File.dirname(__FILE__))
-require 'coverband/integrations/report_middleware'
+require File.expand_path("../../test_helper", File.dirname(__FILE__))
+require "coverband/integrations/report_middleware"
 
 class ReportMiddlewareTest < Minitest::Test
   def setup
@@ -11,21 +11,21 @@ class ReportMiddlewareTest < Minitest::Test
     end
   end
 
-  test 'reports coverage' do
-    request = Rack::MockRequest.env_for('/anything.json')
+  test "reports coverage" do
+    request = Rack::MockRequest.env_for("/anything.json")
     Coverband::Collectors::Coverage.instance.expects(:report_coverage)
     middleware = Coverband::ReportMiddleware.new(fake_app)
     middleware.call(request)
   end
 
-  test 'reports coverage when an error is raised' do
-    request = Rack::MockRequest.env_for('/anything.json')
+  test "reports coverage when an error is raised" do
+    request = Rack::MockRequest.env_for("/anything.json")
     Coverband::Collectors::Coverage.instance.reset_instance
     Coverband::Collectors::Coverage.instance.expects(:report_coverage).once
     middleware = Coverband::ReportMiddleware.new(fake_app_raise_error)
     begin
       middleware.call(request)
-    rescue StandardError
+    rescue
       nil
     end
   end
@@ -34,11 +34,11 @@ class ReportMiddlewareTest < Minitest::Test
 
   def fake_app
     @fake_app ||= lambda do |env|
-      [200, { 'Content-Type' => 'text/plain' }, env['PATH_INFO']]
+      [200, {"Content-Type" => "text/plain"}, env["PATH_INFO"]]
     end
   end
 
   def fake_app_raise_error
-    @fake_app_raise_error ||= -> { raise 'hell' }
+    @fake_app_raise_error ||= -> { raise "hell" }
   end
 end

data/test/coverband/integrations/resque_worker_test.rb

@@ -1,11 +1,11 @@
 # frozen_string_literal: true
 
-require File.expand_path('../../test_helper', File.dirname(__FILE__))
+require File.expand_path("../../test_helper", File.dirname(__FILE__))
 
 class ResqueWorkerTest < Minitest::Test
   def enqueue_and_run_job
     Resque.enqueue(TestResqueJob)
-    queue = ENV['QUEUE'] = 'resque_coverband'
+    ENV["QUEUE"] = "resque_coverband"
     worker = Resque::Worker.new
     worker.startup
     worker.work_one_job
@@ -21,9 +21,9 @@ class ResqueWorkerTest < Minitest::Test
     Resque.redis = redis
   end
 
-  test 'resque job coverage' do
-    relative_job_file = './test/coverband/integrations/test_resque_job.rb'
-    resque_job_file = File.expand_path('./test_resque_job.rb', File.dirname(__FILE__))
+  test "resque job coverage" do
+    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
 
     enqueue_and_run_job
@@ -35,13 +35,13 @@ class ResqueWorkerTest < Minitest::Test
     Coverband.runtime_coverage!
     report = Coverband.configuration.store.get_coverage_report
 
-    if RUBY_PLATFORM == 'java'
+    if RUBY_PLATFORM == "java"
       # NOTE: the todo test only issue seems to be slightly different in JRuby
       # were nothing is showing up as runtime Coverage... This appears to be a test only issue
-      assert_equal 1, report[Coverband::EAGER_TYPE][relative_job_file]['data'][6]
+      assert_equal 1, report[Coverband::EAGER_TYPE][relative_job_file]["data"][6]
     else
-      assert_equal 0, report[Coverband::EAGER_TYPE][relative_job_file]['data'][6]
-      assert_equal 1, report[Coverband::RUNTIME_TYPE][relative_job_file]['data'][6]
+      assert_equal 0, report[Coverband::EAGER_TYPE][relative_job_file]["data"][6]
+      assert_equal 1, report[Coverband::RUNTIME_TYPE][relative_job_file]["data"][6]
     end
   end
 end

data/test/coverband/integrations/test_resque_job.rb

@@ -4,6 +4,6 @@ class TestResqueJob
   @queue = :resque_coverband
 
   def self.perform
-    'resque job perform'
+    "resque job perform"
   end
 end

data/test/coverband/reporters/base_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 ReportsBaseTest < Minitest::Test
   def setup
     super
   end
 
-  test '#merge_arrays basic merge preserves order and counts' do
+  test "#merge_arrays basic merge preserves order and counts" do
     first = [0, 0, 1, 0, 1]
     second = [nil, 0, 1, 0, 0]
     expects = [0, 0, 2, 0, 1]
@@ -15,7 +15,7 @@ class ReportsBaseTest < Minitest::Test
     assert_equal expects, Coverband::Reporters::Base.send(:merge_arrays, first, second)
   end
 
-  test '#merge_arrays basic merge preserves order and counts different lengths' do
+  test "#merge_arrays basic merge preserves order and counts different lengths" do
     first = [0, 0, 1, 0, 1]
     second = [nil, 0, 1, 0, 0, 0, 0, 1]
     expects = [0, 0, 2, 0, 1, 0, 0, 1]
@@ -23,7 +23,7 @@ class ReportsBaseTest < Minitest::Test
     assert_equal expects, Coverband::Reporters::Base.send(:merge_arrays, first, second)
   end
 
-  test '#merge_arrays basic merge preserves nils' do
+  test "#merge_arrays basic merge preserves nils" do
     first = [0, 1, 2, nil, nil, nil]
     second = [0, 1, 2, nil, 0, 1, 2]
     expects = [0, 2, 4, nil, 0, 1, 2]
@@ -33,16 +33,16 @@ class ReportsBaseTest < Minitest::Test
 
   test "#get_current_scov_data_imp doesn't ignore folders with default ignore keys" do
     Coverband.configure do |config|
-      config.reporter          = 'std_out'
-      config.root              = '/full/remote_app/path'
+      config.reporter = "std_out"
+      config.root = "/full/remote_app/path"
     end
 
-    key = '/a_path/that_has_erb_in/thepath.rb'
-    roots = ['/app/', '/full/remote_app/path/']
+    key = "/a_path/that_has_erb_in/thepath.rb"
+    roots = ["/app/", "/full/remote_app/path/"]
 
     lines_hit = [1, 3, 6]
     store.stubs(:merged_coverage).returns(key => lines_hit)
-    expected = { key => [1, 3, 6] }
+    expected = {key => [1, 3, 6]}
 
     assert_equal expected, Coverband::Reporters::Base.send(:get_current_scov_data_imp, store, roots)[:merged]
   end

data/test/coverband/reporters/console_test.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require File.expand_path('../../test_helper', File.dirname(__FILE__))
+require File.expand_path("../../test_helper", File.dirname(__FILE__))
 
 class HTMLReportTest < Minitest::Test
   REDIS_STORAGE_FORMAT_VERSION = Coverband::Adapters::RedisStore::REDIS_STORAGE_FORMAT_VERSION
@@ -10,17 +10,17 @@ class HTMLReportTest < Minitest::Test
     @store = Coverband.configuration.store
   end
 
-  test 'report data' do
+  test "report data" do
     Coverband.configure do |config|
-      config.reporter = 'std_out'
+      config.reporter = "std_out"
     end
-    Coverband.configuration.logger.stubs('info')
+    Coverband.configuration.logger.stubs("info")
     mock_file_hash
-    Coverband::Utils::RelativeFileConverter.expects(:convert).with('app_path/dog.rb').returns('./dog.rb')
+    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]
-    expected = { './dog.rb' => [0, 1, 2] }
+    expected = {"./dog.rb" => [0, 1, 2]}
     assert_equal(expected.keys, report.keys)
   end
 end

data/test/coverband/reporters/html_test.rb

@@ -1,95 +1,83 @@
 # frozen_string_literal: true
 
-require File.expand_path('../../test_helper', File.dirname(__FILE__))
+require File.expand_path("../../test_helper", File.dirname(__FILE__))
 
 class ReportHTMLTest < Minitest::Test
   def setup
     super
     @store = Coverband.configuration.store
     Coverband.configure do |config|
-      config.store             = @store
-      config.root              = fixtures_root
-      config.ignore            = ['notsomething.rb', 'lib/*']
+      config.store = @store
+      config.root = fixtures_root
+      config.ignore = ["notsomething.rb", "lib/*"]
     end
     mock_file_hash
   end
 
-  test 'generate dynamic content hosted html report' do
+  test "generate dynamic content hosted html report" do
     @store.send(:save_report, basic_coverage)
 
     html = Coverband::Reporters::HTMLReport.new(@store,
-                                                static: false,
-                                                open_report: false).report
-    assert_match 'Generated by', html
+      open_report: false).report
+    assert_match "Generated by", html
   end
 
-  test 'files with no Coverage but in project are shown in reports' do
+  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
+      open_report: false).report
+    assert_match "sample.rb", html
     # in project, but not in coverage data
-    assert_match 'app/models/user.rb', html
+    assert_match "app/models/user.rb", html
   end
 
-  test 'files with no Coverage but in project details page list warning' do
+  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
+    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,
-                                                    static: true,
-                                                    open_report: false)
-    Coverband::Utils::HTMLFormatter.any_instance.expects(:format_static_html!).once
-    reporter.report
+      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 dynamic content detailed file report' do
+  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'
+    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
+      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
+  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'
+    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
+      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
+  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'
+    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
+      filename: filename,
+      base_path: base_path,
+      open_report: false).file_details
+    assert_match "File No Longer Available", html
   end
 end