coverband 4.2.5 → 5.0.0.rc.4

data/test/integration/full_stack_test.rb

@@ -1,57 +1,46 @@
 # 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 FullStackTest < Minitest::Test
   REDIS_STORAGE_FORMAT_VERSION = Coverband::Adapters::RedisStore::REDIS_STORAGE_FORMAT_VERSION
-  TEST_RACK_APP = '../fake_app/basic_rack.rb'
+  TEST_RACK_APP = "../fake_app/basic_rack.rb"
 
   def setup
     super
     Coverband::Collectors::Coverage.instance.reset_instance
     Coverband.configure do |config|
-      config.s3_bucket = nil
       config.background_reporting_enabled = false
       config.track_gems = true
     end
     Coverband.start
     Coverband::Collectors::Coverage.instance.eager_loading!
-    @rack_file = require_unique_file 'fake_app/basic_rack.rb'
+    @rack_file = require_unique_file "fake_app/basic_rack.rb"
     Coverband.report_coverage
     Coverband::Collectors::Coverage.instance.runtime!
   end
 
-  test 'call app' do
-    request = Rack::MockRequest.env_for('/anything.json')
+  test "call app" do
+    request = Rack::MockRequest.env_for("/anything.json")
     middleware = Coverband::BackgroundMiddleware.new(fake_app_with_lines)
     results = middleware.call(request)
-    assert_equal 'Hello Rack!', results.last
+    assert_equal "Hello Rack!", results.last
     Coverband.report_coverage
     expected = [nil, nil, 0, nil, 0, 0, 1, nil, nil]
-    assert_equal expected, Coverband.configuration.store.coverage[@rack_file]['data']
+    assert_equal expected, Coverband.configuration.store.coverage[@rack_file]["data"]
 
     # additional calls increase count by 1
     middleware.call(request)
     Coverband.report_coverage
     expected = [nil, nil, 0, nil, 0, 0, 2, nil, nil]
-    assert_equal expected, Coverband.configuration.store.coverage[@rack_file]['data']
+    assert_equal expected, Coverband.configuration.store.coverage[@rack_file]["data"]
 
     # class coverage
     Coverband.eager_loading_coverage!
-    Coverband.configuration.store.coverage[@rack_file]['data']
+    Coverband.configuration.store.coverage[@rack_file]["data"]
     expected = [nil, nil, 1, nil, 1, 1, 0, nil, nil]
-  end
-
-  test 'call app with gem tracking' do
-    require 'rainbow'
-    Rainbow('this text is red').red
-    request = Rack::MockRequest.env_for('/anything.json')
-    middleware = Coverband::BackgroundMiddleware.new(fake_app_with_lines)
-    results = middleware.call(request)
-    assert_equal 'Hello Rack!', results.last
-    Coverband.report_coverage
-    assert Coverband.configuration.store.coverage.keys.any? { |key| key.end_with?('rainbow/global.rb') }
+    assert_equal expected, Coverband.configuration.store.coverage[@rack_file]["data"]
   end
 
   private

data/test/jruby_check.rb

@@ -1,8 +1,8 @@
-require 'coverage'
+require "coverage"
 
 Coverage.start
 
-require './test/dog.rb'
+require "./test/dog.rb"
 
 puts Coverage.peek_result
 
@@ -11,4 +11,3 @@ puts Dog.new.bark
 puts Coverage.peek_result
 
 puts "done"
-

data/test/rails4_dummy/Rakefile

@@ -1,6 +1,6 @@
 # Add your own tasks in files placed in lib/tasks ending in .rake,
 # for example lib/tasks/capistrano.rake, and they will automatically be available to Rake.
 
-require_relative 'config/application'
+require_relative "config/application"
 
 Rails.application.load_tasks

data/test/rails4_dummy/config.ru

@@ -1,4 +1,4 @@
 # This file is used by Rack-based servers to start the application.
 
-require ::File.expand_path('../config/environment', __FILE__)
+require ::File.expand_path("../config/environment", __FILE__)
 run Rails.application

data/test/rails4_dummy/config/application.rb

@@ -1,10 +1,10 @@
 # frozen_string_literal: true
 
-require File.expand_path('boot', __dir__)
+require File.expand_path("boot", __dir__)
 
-require 'rails'
-require 'action_controller/railtie'
-require 'coverband'
+require "rails"
+require "action_controller/railtie"
+require "coverband"
 Bundler.require(*Rails.groups)
 
 module Rails4Dummy

data/test/rails4_dummy/config/boot.rb

@@ -1,3 +1,3 @@
-ENV['BUNDLE_GEMFILE'] ||= File.expand_path('../../Gemfile', __FILE__)
+ENV["BUNDLE_GEMFILE"] ||= File.expand_path("../../Gemfile", __FILE__)
 
-require 'bundler/setup' # Set up gems listed in the Gemfile.
+require "bundler/setup" # Set up gems listed in the Gemfile.

data/test/rails4_dummy/config/coverband.rb

@@ -1,3 +1,3 @@
 # frozen_string_literal: true
 
-require_relative '../../rails5_dummy/config/coverband'
+require_relative "../../rails5_dummy/config/coverband"

data/test/rails4_dummy/config/coverband_missing_redis.rb

@@ -1,3 +1,3 @@
 # frozen_string_literal: true
 
-require_relative '../../rails5_dummy/config/coverband_missing_redis'
+require_relative "../../rails5_dummy/config/coverband_missing_redis"

data/test/rails4_dummy/config/environment.rb

@@ -1,5 +1,5 @@
 # Load the Rails application.
-require File.expand_path('../application', __FILE__)
+require File.expand_path("../application", __FILE__)
 
 # Initialize the Rails application.
 Rails.application.initialize!

data/test/rails4_dummy/config/routes.rb

@@ -1,4 +1,4 @@
 Rails.application.routes.draw do
-  get 'dummy/show'
-  mount Coverband::Reporters::Web.new, at: '/coverage'
+  get "dummy/show"
+  mount Coverband::Reporters::Web.new, at: "/coverage"
 end

data/test/rails5_dummy/Rakefile

@@ -1,6 +1,6 @@
 # Add your own tasks in files placed in lib/tasks ending in .rake,
 # for example lib/tasks/capistrano.rake, and they will automatically be available to Rake.
 
-require_relative 'config/application'
+require_relative "config/application"
 
 Rails.application.load_tasks

data/test/rails5_dummy/config.ru

@@ -1,3 +1,3 @@
-require_relative 'config/environment'
+require_relative "config/environment"
 
 run Rails.application

data/test/rails5_dummy/config/application.rb

@@ -1,8 +1,8 @@
 # frozen_string_literal: true
 
-require 'rails'
-require 'action_controller/railtie'
-require 'coverband'
+require "rails"
+require "action_controller/railtie"
+require "coverband"
 Bundler.require(*Rails.groups)
 
 module Rails5Dummy

data/test/rails5_dummy/config/coverband.rb

@@ -1,15 +1,15 @@
 # frozen_string_literal: true
 
 Coverband.configure do |config|
-  config.root              = Dir.pwd
-  config.store             = Coverband::Adapters::RedisStore.new(Redis.new(db: 2, url: ENV['REDIS_URL']), redis_namespace: 'coverband_test') if defined? Redis
-  config.ignore            = %w[.erb$ .slim$]
-  config.root_paths        = []
-  config.logger            = Rails.logger
-  config.verbose           = true
+  config.root = Dir.pwd
+  config.store = Coverband::Adapters::RedisStore.new(Redis.new(db: 2, url: ENV["REDIS_URL"]), redis_namespace: "coverband_test") if defined? Redis
+  config.ignore = %w[.erb$ .slim$]
+  config.root_paths = []
+  config.logger = Rails.logger
+  config.verbose = true
   config.background_reporting_enabled = true
   config.track_gems = true
   config.gem_details = true
-  config.use_oneshot_lines_coverage = true if ENV['ONESHOT']
-  config.simulate_oneshot_lines_coverage = true if ENV['SIMULATE_ONESHOT']
+  config.use_oneshot_lines_coverage = true if ENV["ONESHOT"]
+  config.simulate_oneshot_lines_coverage = true if ENV["SIMULATE_ONESHOT"]
 end

data/test/rails5_dummy/config/coverband_missing_redis.rb

@@ -1,15 +1,15 @@
 # frozen_string_literal: true
 
 Coverband.configure do |config|
-  config.root              = Dir.pwd
-  config.store             = Coverband::Adapters::RedisStore.new(Redis.new(db: 2, url: 'redis://127.0.0.1:123'), redis_namespace: 'coverband_test') if defined? Redis
-  config.ignore            = %w[vendor .erb$ .slim$]
-  config.root_paths        = []
-  config.logger            = Rails.logger
-  config.verbose           = true
+  config.root = Dir.pwd
+  config.store = Coverband::Adapters::RedisStore.new(Redis.new(db: 2, url: "redis://127.0.0.1:123"), redis_namespace: "coverband_test") if defined? Redis
+  config.ignore = %w[vendor .erb$ .slim$]
+  config.root_paths = []
+  config.logger = Rails.logger
+  config.verbose = true
   config.background_reporting_enabled = true
   config.track_gems = true
   config.gem_details = true
-  config.use_oneshot_lines_coverage = true if ENV['ONESHOT']
-  config.simulate_oneshot_lines_coverage = true if ENV['SIMULATE_ONESHOT']
+  config.use_oneshot_lines_coverage = true if ENV["ONESHOT"]
+  config.simulate_oneshot_lines_coverage = true if ENV["SIMULATE_ONESHOT"]
 end

data/test/rails5_dummy/config/environment.rb

@@ -1,2 +1,2 @@
-require_relative 'application'
+require_relative "application"
 Rails.application.initialize!

data/test/rails5_dummy/config/routes.rb

@@ -1,4 +1,4 @@
 Rails.application.routes.draw do
-  get 'dummy/show'
-  mount Coverband::Reporters::Web.new, at: '/coverage'
+  get "dummy/show"
+  mount Coverband::Reporters::Web.new, at: "/coverage"
 end

data/test/rails6_dummy/Rakefile

@@ -1,6 +1,6 @@
 # Add your own tasks in files placed in lib/tasks ending in .rake,
 # for example lib/tasks/capistrano.rake, and they will automatically be available to Rake.
 
-require_relative 'config/application'
+require_relative "config/application"
 
 Rails.application.load_tasks

data/test/rails6_dummy/config.ru

@@ -1,4 +1,4 @@
 # This file is used by Rack-based servers to start the application.
 
-require ::File.expand_path('../config/environment', __FILE__)
+require ::File.expand_path("../config/environment", __FILE__)
 run Rails.application

data/test/rails6_dummy/config/application.rb

@@ -1,10 +1,10 @@
 # frozen_string_literal: true
 
-require File.expand_path('boot', __dir__)
+require File.expand_path("boot", __dir__)
 
-require 'rails'
-require 'action_controller/railtie'
-require 'coverband'
+require "rails"
+require "action_controller/railtie"
+require "coverband"
 Bundler.require(*Rails.groups)
 
 module Rails6Dummy

data/test/rails6_dummy/config/boot.rb

@@ -1,3 +1,3 @@
-ENV['BUNDLE_GEMFILE'] ||= File.expand_path('../../Gemfile', __FILE__)
+ENV["BUNDLE_GEMFILE"] ||= File.expand_path("../../Gemfile", __FILE__)
 
-require 'bundler/setup' # Set up gems listed in the Gemfile.
+require "bundler/setup" # Set up gems listed in the Gemfile.

data/test/rails6_dummy/config/coverband.rb

@@ -1,3 +1,3 @@
 # frozen_string_literal: true
 
-require_relative '../../rails5_dummy/config/coverband'
+require_relative "../../rails5_dummy/config/coverband"

data/test/rails6_dummy/config/coverband_missing_redis.rb

@@ -1,3 +1,3 @@
 # frozen_string_literal: true
 
-require_relative '../../rails5_dummy/config/coverband_missing_redis'
+require_relative "../../rails5_dummy/config/coverband_missing_redis"

data/test/rails6_dummy/config/environment.rb

@@ -1,5 +1,5 @@
 # Load the Rails application.
-require File.expand_path('../application', __FILE__)
+require File.expand_path("../application", __FILE__)
 
 # Initialize the Rails application.
 Rails.application.initialize!

data/test/rails6_dummy/config/routes.rb

@@ -1,4 +1,4 @@
 Rails.application.routes.draw do
-  get 'dummy/show'
-  mount Coverband::Reporters::Web.new, at: '/coverage'
+  get "dummy/show"
+  mount Coverband::Reporters::Web.new, at: "/coverage"
 end

data/test/rails_test_helper.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
-require 'minitest'
-require 'minitest/fork_executor'
+require "minitest"
+require "minitest/fork_executor"
 
 # Forked executor includes autorun which does not work with qrush/m
 # https://github.com/qrush/m/issues/26
@@ -9,25 +9,25 @@ require 'minitest/fork_executor'
 if defined?(M)
   Minitest.class_eval do
     def self.autorun
-      puts 'No autorunning'
+      puts "No autorunning"
     end
   end
 end
 
 Minitest.parallel_executor = Minitest::ForkExecutor.new
-require File.expand_path('./test_helper', File.dirname(__FILE__))
-require 'capybara'
-require 'capybara/minitest'
+require File.expand_path("./test_helper", File.dirname(__FILE__))
+require "capybara"
+require "capybara/minitest"
 def rails_setup
-  ENV['RAILS_ENV'] = 'test'
-  require 'rails'
+  ENV["RAILS_ENV"] = "test"
+  require "rails"
   # coverband must be required after rails
   Coverband.configure("./test/rails#{Rails::VERSION::MAJOR}_dummy/config/coverband.rb")
-  load 'coverband/utils/railtie.rb'
+  load "coverband/utils/railtie.rb"
 
   require_relative "../test/rails#{Rails::VERSION::MAJOR}_dummy/config/environment"
-  require 'capybara/rails'
+  require "capybara/rails"
   # Our coverage report is wrapped in display:none as of now
   Capybara.ignore_hidden_elements = false
-  require 'mocha/minitest'
+  require "mocha/minitest"
 end

data/test/test_helper.rb

@@ -1,28 +1,36 @@
 # frozen_string_literal: true
 
-# This lets us ignore warnings from our test dependencies when loaded
-# We do this because of aws-sdk-s3
-# aws-sdk-s3 is written in such a way to cause dozens of warnings
 original_verbosity = $VERBOSE
 $VERBOSE = nil
-require 'rubygems'
-require 'aws-sdk-s3'
-require 'simplecov'
-require 'coveralls'
-require 'minitest/autorun'
-require 'mocha/minitest'
-require 'ostruct'
-require 'json'
-require 'redis'
-require 'resque'
+require "rubygems"
+require "simplecov"
+require "coveralls"
+require "minitest/autorun"
+require "mocha/minitest"
+require "ostruct"
+require "json"
+require "redis"
+require "resque"
+
+require "coverband"
+require "coverband/reporters/web"
+require "coverband/utils/html_formatter"
+require "coverband/utils/result"
+require "coverband/utils/file_list"
+require "coverband/utils/source_file"
+require "coverband/utils/lines_classifier"
+require "coverband/utils/results"
+require "coverband/reporters/html_report"
+require "webmock/minitest"
+
 # require 'pry-byebug' unless ENV['CI'] # Ruby 2.3 on CI crashes on pry & JRuby doesn't support it
-require_relative 'unique_files'
+require_relative "unique_files"
 $VERBOSE = original_verbosity
 
-unless ENV['ONESHOT'] || ENV['SIMULATE_ONESHOT']
+unless ENV["ONESHOT"] || ENV["SIMULATE_ONESHOT"]
   SimpleCov.formatter = Coveralls::SimpleCov::Formatter
   SimpleCov.start do
-    add_filter 'test/forked'
+    add_filter "test/forked"
   end
 
   Coveralls.wear!
@@ -37,14 +45,15 @@ module Coverband
     end
 
     def self.reset
-      Coverband.configuration.redis_namespace = 'coverband_test'
-      Coverband.configuration.store.instance_variable_set(:@redis_namespace, 'coverband_test')
+      Coverband.configuration.redis_namespace = "coverband_test"
+      Coverband.configuration.store.instance_variable_set(:@redis_namespace, "coverband_test")
       Coverband.configuration.store.class.class_variable_set(:@@path_cache, {})
       Coverband.configuration.reset
       Coverband::Collectors::Coverage.instance.reset_instance
       Coverband::Utils::RelativeFileConverter.reset
       Coverband::Utils::AbsoluteFileConverter.reset
-      Coverband.configuration.redis_namespace = 'coverband_test'
+      Coverband.configuration.reporting_wiggle = 0
+      Coverband.configuration.redis_namespace = "coverband_test"
       Coverband::Background.stop
       Coverband.configuration.store.instance_variable_set(:@redis, redis)
       redis.flushdb
@@ -61,19 +70,19 @@ Minitest::Test.class_eval do
   prepend Coverband::Test
 end
 
-TEST_COVERAGE_FILE = '/tmp/fake_file.json'
+TEST_COVERAGE_FILE = "/tmp/fake_file.json"
 
-$LOAD_PATH.unshift(File.join(File.dirname(__FILE__), '..', 'lib'))
+$LOAD_PATH.unshift(File.join(File.dirname(__FILE__), "..", "lib"))
 $LOAD_PATH.unshift(File.dirname(__FILE__))
 
 Mocha::Configuration.prevent(:stubbing_method_unnecessarily)
 Mocha::Configuration.prevent(:stubbing_non_existent_method)
 
 def test(name, &block)
-  test_name = "test_#{name.gsub(/\s+/, '_')}".to_sym
+  test_name = "test_#{name.gsub(/\s+/, "_")}".to_sym
   defined = begin
               instance_method(test_name)
-            rescue StandardError
+            rescue
               false
             end
   raise "#{test_name} is already defined in #{self}" if defined
@@ -87,7 +96,7 @@ def test(name, &block)
   end
 end
 
-def mock_file_hash(hash: 'abcd')
+def mock_file_hash(hash: "abcd")
   Coverband::Utils::FileHasher.expects(:hash).at_least_once.returns(hash)
 end
 
@@ -96,15 +105,15 @@ def example_line
 end
 
 def basic_coverage
-  { 'app_path/dog.rb' => example_line }
+  {"app_path/dog.rb" => example_line}
 end
 
 def basic_coverage_full_path
-  { basic_coverage_file_full_path => example_line }
+  {basic_coverage_file_full_path => example_line}
 end
 
 def basic_source_fixture_coverage
-  { source_fixture('sample.rb') => example_line }
+  {source_fixture("sample.rb") => example_line}
 end
 
 def basic_coverage_file_full_path
@@ -112,11 +121,11 @@ def basic_coverage_file_full_path
 end
 
 def source_fixture(filename)
-  File.expand_path(File.join(File.dirname(__FILE__), 'fixtures', filename))
+  File.expand_path(File.join(File.dirname(__FILE__), "fixtures", filename))
 end
 
 def fixtures_root
-  File.expand_path(File.join(File.dirname(__FILE__), 'fixtures'))
+  File.expand_path(File.join(File.dirname(__FILE__), "fixtures"))
 end
 
 def test_root
@@ -127,13 +136,13 @@ end
 # This handles an issue where the store is setup in tests prior to being able to set the namespace
 ###
 def store
-  if Coverband.configuration.store.redis_namespace == 'coverband_test'
-    Coverband.configuration.store
+  if Coverband.configuration.store.redis_namespace == "coverband_test"
+    # noop
   else
-    Coverband.configuration.redis_namespace = 'coverband_test'
+    Coverband.configuration.redis_namespace = "coverband_test"
     Coverband.configuration.instance_variable_set(:@store, nil)
-    Coverband.configuration.store
   end
+  Coverband.configuration.store
 end
 
 # Taken from http://stackoverflow.com/questions/4459330/how-do-i-temporarily-redirect-stderr-in-ruby
@@ -150,7 +159,7 @@ ensure
   $stderr = previous_stderr
 end
 
-require 'coverband'
+require "coverband"
 
 Coverband::Configuration.class_eval do
   def test_env