coverband 2.0.0 → 2.0.3

data/test/benchmarks/benchmark.rake

@@ -1,6 +1,5 @@
 # frozen_string_literal: true
 namespace :benchmarks do
-
   # https://github.com/evanphx/benchmark-ips
   # Enable and start GC before each job run. Disable GC afterwards.
   #
@@ -48,7 +47,7 @@ namespace :benchmarks do
     require 'benchmark'
     require 'benchmark/ips'
 
-    # TODO ok this is interesting and weird
+    # TODO: ok this is interesting and weird
     # basically the earlier I require coverage and
     # then require files the larger perf impact
     # this is somewhat expected but can lead to significant perf diffs
@@ -67,6 +66,16 @@ namespace :benchmarks do
     require File.join(File.dirname(__FILE__), 'dog')
   end
 
+  def benchmark_redis_store
+    redis = if ENV['REDIS_TEST_URL']
+              Redis.new(url: ENV['REDIS_TEST_URL'])
+            else
+              Redis.new
+            end
+    Coverband::Adapters::RedisStore.new(redis,
+                                        redis_namespace: 'coverband_bench')
+  end
+
   desc 'set up coverband tracepoint Redis'
   task :setup_redis do
     Coverband.configure do |config|
@@ -76,7 +85,7 @@ namespace :benchmarks do
       config.logger             = $stdout
       config.collector          = 'trace'
       config.memory_caching     = ENV['MEMORY_CACHE'] ? true : false
-      config.store              = Coverband::Adapters::RedisStore.new(Redis.new)
+      config.store              = benchmark_redis_store
     end
   end
 
@@ -88,7 +97,8 @@ namespace :benchmarks do
       config.logger             = $stdout
       config.collector          = 'trace'
       config.memory_caching     = ENV['MEMORY_CACHE'] ? true : false
-      config.store              = Coverband::Adapters::FileStore.new('/tmp/benchmark_store.json')
+      file_path                 = '/tmp/benchmark_store.json'
+      config.store              = Coverband::Adapters::FileStore.new(file_path)
     end
   end
 
@@ -104,7 +114,7 @@ namespace :benchmarks do
       config.logger             = $stdout
       config.collector          = 'coverage'
       config.memory_caching     = ENV['MEMORY_CACHE'] ? true : false
-      config.store              = Coverband::Adapters::RedisStore.new(Redis.new)
+      config.store              = benchmark_redis_store
     end
   end
 
@@ -138,12 +148,12 @@ namespace :benchmarks do
     10_000.times { Dog.new.bark }
   end
 
+  # puts "benchmark for: #{Coverband.configuration.inspect}"
+  # puts "store: #{Coverband.configuration.store.inspect}"
   def run_work(hold_work = false)
     suite = GCSuite.new
-    #puts "benchmark for: #{Coverband.configuration.inspect}"
-    #puts "store: #{Coverband.configuration.store.inspect}"
     Benchmark.ips do |x|
-      x.config(:time => 12, :warmup => 5, :suite => suite)
+      x.config(time: 12, warmup: 5, suite: suite)
       x.report 'coverband' do
         Coverband::Collectors::Base.instance.sample do
           work
@@ -159,6 +169,73 @@ namespace :benchmarks do
     Coverband::Collectors::Base.instance.reset_instance
   end
 
+  def fake_line_numbers
+    24.times.each_with_object({}) do |line, line_hash|
+      line_hash[(line + 1).to_s] = rand(5)
+    end
+  end
+
+  def fake_report
+    2934.times.each_with_object({}) do |file_number, hash|
+      hash["file#{file_number + 1}.rb"] = fake_line_numbers
+    end
+  end
+
+  def adjust_report(report)
+    report.keys.each do |file|
+      next unless rand < 0.15
+      report[file] = fake_line_numbers
+    end
+    report
+  end
+
+  def reporting_speed
+    report = fake_report
+    store = benchmark_redis_store
+
+    5.times { store.save_report(report) }
+    Benchmark.ips do |x|
+      x.config(time: 15, warmup: 5)
+      x.report('store_reports') { store.save_report(report) }
+    end
+  end
+
+  def reporting_memorycache_speed
+    report = fake_report
+    redis_store = benchmark_redis_store
+    cache_store = Coverband::Adapters::MemoryCacheStore.new(redis_store)
+
+    5.times do
+      redis_store.save_report(report)
+      cache_store.save_report(report)
+      adjust_report(report)
+    end
+    Benchmark.ips do |x|
+      x.config(time: 15, warmup: 5)
+      x.report('store_redis_reports') do
+        redis_store.save_report(report)
+        adjust_report(report)
+      end
+      x.report('store_cache_reports') do
+        cache_store.save_report(report)
+        adjust_report(report)
+      end
+      x.compare!
+    end
+  end
+
+  desc 'runs benchmarks on reporting large sets of files to redis'
+  task redis_reporting: [:setup] do
+    puts 'runs benchmarks on reporting large sets of files to redis'
+    reporting_speed
+  end
+
+  desc 'benchmarks: reporting large sets of files to redis using memory cache'
+  task cache_reporting: [:setup] do
+    puts 'benchmarks: reporting large sets of files to redis using memory cache'
+    reporting_memorycache_speed
+  end
+
   desc 'runs benchmarks on default redis setup'
   task run_redis: [:setup, :setup_redis] do
     puts 'Coverband tracepoint configured with default Redis store'
@@ -179,11 +256,13 @@ namespace :benchmarks do
 
   desc 'compare Coverband Ruby Coverage with normal Ruby'
   task :compare_coverage do
-    puts 'comparing with Coverage loaded and not, this takes some time for output...'
+    puts 'comparing Coverage loaded/not, this takes some time for output...'
     puts `COVERAGE=true rake benchmarks:run_coverage`
     puts `rake benchmarks:run_coverage`
   end
 end
 
 desc 'runs benchmarks'
-task benchmarks: ['benchmarks:run_file', 'benchmarks:run_redis', 'benchmarks:compare_coverage']
+task benchmarks: ['benchmarks:run_file',
+                  'benchmarks:run_redis',
+                  'benchmarks:compare_coverage']

data/test/test_helper.rb

@@ -59,11 +59,6 @@ end
 
 require 'coverband'
 
-unless File.exist?('./tmp/coverband_baseline.json')
-  `mkdir -p ./tmp`
-  `touch ./tmp/coverband_baseline.json`
-end
-
 Coverband.configure do |config|
   config.root              = Dir.pwd
   config.root_paths        = ['/app/']

data/test/unit/adapters_memory_cache_store_test.rb

@@ -5,52 +5,62 @@ require File.expand_path('../test_helper', File.dirname(__FILE__))
 module Coverband
   class MemoryCacheStoreTest < Test::Unit::TestCase
     def setup
-      Adapters::MemoryCacheStore.reset!
-      @store = mock('store')
+      Adapters::MemoryCacheStore.clear!
+      @redis = Redis.new
+      @store = Coverband::Adapters::RedisStore.new(@redis)
+      @store.clear!
       @memory_store = Adapters::MemoryCacheStore.new(@store)
     end
 
-    def data
-      {
-        'file1' => { 3 => 1, 5 => 2 },
-        'file2' => { 1 => 1, 2 => 1 }
-      }
-    end
-
     test 'it passes data into store' do
+      data = {
+        'file1' => { 1 => 0, 2 => 1, 3 => 0 },
+        'file2' => { 1 => 5, 2 => 2 }
+      }
       @store.expects(:save_report).with data
-      @store.expects(:covered_lines_for_file).with('file1').returns []
-      @store.expects(:covered_lines_for_file).with('file2').returns []
       @memory_store.save_report data
     end
 
-    test 'it passes data into store only once' do
+
+    test 'it filters coverage with same exact data' do
+      data = {
+        'file1' => { 1 => 0, 2 => 1, 3 => 0 },
+        'file2' => { 1 => 5, 2 => 2 }
+      }
       @store.expects(:save_report).once.with data
-      @store.expects(:covered_lines_for_file).with('file1').returns []
-      @store.expects(:covered_lines_for_file).with('file2').returns []
       2.times { @memory_store.save_report data }
     end
+    
+    test 'it filters coverage for files with same exact data' do
+ 
+      report_first_request = {
+        'file1' => { 1 => 0, 2 => 1, 3 => 0 },
+        'file2' => { 1 => 5, 2 => 2 }
+      }
 
-    test 'it only passes files and lines we have not hit yet' do
-      second_data = {
-        'file1' => { 3 => 1, 5 => 1, 10 => 1 },
-        'file2' => { 1 => 1, 2 => 1 }
+      report_second_request = {
+        'file1' => { 1 => 0, 2 => 1, 3 => 0 },
+        'file2' => { 1 => 5, 2 => 3 }
       }
-      @store.expects(:covered_lines_for_file).with('file1').returns []
-      @store.expects(:covered_lines_for_file).with('file2').returns []
-      @store.expects(:save_report).once.with data
-      @store.expects(:save_report).once.with(
-        'file1' => { 10 => 1 }
-      )
-      @memory_store.save_report data
-      @memory_store.save_report second_data
+      @store.expects(:save_report).with({
+        'file1' => { 1 => 0, 2 => 1, 3 => 0 },
+        'file2' => { 1 => 5, 2 => 2 }
+      })
+      @store.expects(:save_report).with({
+        'file2' => { 1 => 5, 2 => 3 }
+      })
+      @memory_store.save_report(report_first_request)
+      @memory_store.save_report(report_second_request)
     end
 
     test 'it initializes cache with what is in store' do
-      @store.expects(:covered_lines_for_file).with('file1').returns [3, 5]
-      @store.expects(:covered_lines_for_file).with('file2').returns [2]
-      @store.expects(:save_report).with('file2' => { 1 => 1 })
-      @memory_store.save_report data
+      data = {
+        'file1' => { 1 => 0, 2 => 1, 3 => 0 },
+        'file2' => { 1 => 5, 2 => 2 }
+      }
+      Coverband::Adapters::RedisStore.new(@redis).save_report(data)
+      @store.expects(:save_report).never
+      @memory_store.save_report(data)
     end
   end
 end

data/test/unit/adapters_redis_store_test.rb

@@ -17,13 +17,13 @@ class RedisTest < Test::Unit::TestCase
 
   def test_coverage
     @redis.sadd(BASE_KEY, 'dog.rb')
-    @store.send(:store_map, "#{BASE_KEY}.dog.rb", example_hash)
-    expected = { 'dog.rb' => example_hash }
+    @store.send(:pipelined_save, combined_report)
+    expected = {'dog.rb' => example_hash}
     assert_equal expected, @store.coverage
   end
 
   def test_covered_lines_for_file
-    @store.send(:store_map, "#{BASE_KEY}.dog.rb", example_hash)
+    @store.send(:pipelined_save, combined_report)
     assert_equal [["1", "1"], ["2", "2"]], @store.covered_lines_for_file('dog.rb').sort
   end
 
@@ -47,6 +47,15 @@ class RedisTest < Test::Unit::TestCase
 
   private
 
+  def combined_report
+    {
+      "#{BASE_KEY}.dog.rb" => {
+        new: example_hash,
+        existing: {}
+      }
+    }
+  end
+
   def test_data
     {
       '/Users/danmayer/projects/cover_band_server/app.rb' => { 54 => 1, 55 => 2 },

data/test/unit/collectors_base_test.rb

@@ -49,6 +49,7 @@ class CollectorsBaseTest < Test::Unit::TestCase
     coverband.instance_variable_set('@store', nil)
     assert_equal false, coverband.instance_variable_get('@enabled')
     logger.expects(:info).at_least_once
+    logger.stubs('debug')
     coverband.sample { 1 + 1 }
     assert_equal true, coverband.instance_variable_get('@enabled')
   end
@@ -62,8 +63,8 @@ class CollectorsBaseTest < Test::Unit::TestCase
     coverband.instance_variable_set('@store', nil)
     assert_equal false, coverband.instance_variable_get('@enabled')
     logger.expects(:info).at_least_once
+    logger.stubs('debug')
     coverband.start
-    logger.info(1 + 1)
     coverband.stop
     coverband.save
   end
@@ -74,6 +75,7 @@ class CollectorsBaseTest < Test::Unit::TestCase
     coverband.instance_variable_set('@sample_percentage', 100.0)
     coverband.instance_variable_set('@verbose', true)
     Coverband.configuration.logger.stubs('info')
+    Coverband.configuration.logger.stubs('debug')
     store = Coverband::Adapters::RedisStore.new(Redis.new)
     coverband.instance_variable_set('@store', store)
     store.expects(:save_report).once.with(has_entries(dog_file => {5 => 5}))

data/test/unit/collectors_coverage_test.rb

@@ -61,7 +61,8 @@ if Gem::Version.new(RUBY_VERSION) >= Gem::Version.new('2.3.0')
       coverband.instance_variable_set('@logger', logger)
       coverband.instance_variable_set('@store', nil)
       assert_equal false, coverband.instance_variable_get('@enabled')
-      logger.expects(:info).at_least_once
+      logger.stubs('info')
+      logger.expects(:debug).at_least_once
       coverband.sample { 1 + 1 }
       assert_equal true, coverband.instance_variable_get('@enabled')
     end
@@ -74,6 +75,7 @@ if Gem::Version.new(RUBY_VERSION) >= Gem::Version.new('2.3.0')
       coverband.instance_variable_set('@store', nil)
       assert_equal false, coverband.instance_variable_get('@enabled')
       logger.expects(:info).at_least_once
+      logger.stubs('debug')
       coverband.start
       logger.info(1 + 1)
       coverband.stop
@@ -83,8 +85,6 @@ if Gem::Version.new(RUBY_VERSION) >= Gem::Version.new('2.3.0')
     test 'allow reporting to redis start stop save' do
       dog_file = File.expand_path('./dog.rb', File.dirname(__FILE__))
       coverband.instance_variable_set('@sample_percentage', 100.0)
-      coverband.instance_variable_set('@verbose', true)
-      Coverband.configuration.logger.stubs('info')
       store = Coverband::Adapters::RedisStore.new(Redis.new)
       coverband.instance_variable_set('@store', store)
 

data/test/unit/collectors_trace_test.rb

@@ -50,6 +50,7 @@ class CollectorsTraceTest < Test::Unit::TestCase
     coverband.instance_variable_set('@store', nil)
     assert_equal false, coverband.instance_variable_get('@enabled')
     logger.expects(:info).at_least_once
+    logger.stubs('debug')
     coverband.sample { 1 + 1 }
     assert_equal true, coverband.instance_variable_get('@enabled')
   end
@@ -63,8 +64,8 @@ class CollectorsTraceTest < Test::Unit::TestCase
     coverband.instance_variable_set('@store', nil)
     assert_equal false, coverband.instance_variable_get('@enabled')
     logger.expects(:info).at_least_once
+    logger.stubs('debug')
     coverband.start
-    logger.info(1 + 1)
     coverband.stop
     coverband.save
   end
@@ -75,6 +76,7 @@ class CollectorsTraceTest < Test::Unit::TestCase
     coverband.instance_variable_set('@sample_percentage', 100.0)
     coverband.instance_variable_set('@verbose', true)
     Coverband.configuration.logger.stubs('info')
+    Coverband.configuration.logger.stubs('debug')
     store = Coverband::Adapters::RedisStore.new(Redis.new)
     coverband.instance_variable_set('@store', store)
     store.expects(:save_report).once.with(has_entries(dog_file => { 5 => 5 }))

data/test/unit/reports_console_test.rb

@@ -15,6 +15,15 @@ class SimpleCovReportTest < Test::Unit::TestCase
     {'1' => '1', '2' => '2'}
   end
 
+  def combined_report
+    {
+      "#{BASE_KEY}.test/unit/dog.rb" => {
+        new: example_hash,
+        existing: {}
+      }
+    }
+  end
+
   test 'report data' do
     Coverband.configure do |config|
       config.redis             = @redis
@@ -25,7 +34,7 @@ class SimpleCovReportTest < Test::Unit::TestCase
     Coverband::Reporters::ConsoleReport.expects(:current_root).returns('./test/unit')
 
     @redis.sadd(BASE_KEY, 'test/unit/dog.rb')
-    @store.send(:store_map, "#{BASE_KEY}.test/unit/dog.rb", example_hash)
+    @store.send(:pipelined_save, combined_report)
 
     report = Coverband::Reporters::ConsoleReport.report(@store)
     expected = {"test/unit/dog.rb"=>[1, 2, nil, nil, nil, nil, nil]}

data/test/unit/reports_simple_cov_test.rb

@@ -15,6 +15,15 @@ class ReportsSimpleCovTest < Test::Unit::TestCase
     {'1' => '1', '2' => '2'}
   end
 
+  def combined_report
+    {
+      "#{BASE_KEY}.test/unit/dog.rb" => {
+        new: example_hash,
+        existing: {}
+      }
+    }
+  end
+
   test 'generate scov report' do
     Coverband.configure do |config|
       config.redis             = @redis
@@ -26,7 +35,7 @@ class ReportsSimpleCovTest < Test::Unit::TestCase
     Coverband.configuration.logger.stubs('info')
 
     @redis.sadd(BASE_KEY, 'test/unit/dog.rb')
-    @store.send(:store_map, "#{BASE_KEY}.test/unit/dog.rb", example_hash)
+    @store.send(:pipelined_save, combined_report)
 
     SimpleCov.expects(:track_files)
     SimpleCov.expects(:add_not_loaded_files).returns({})
@@ -48,7 +57,7 @@ class ReportsSimpleCovTest < Test::Unit::TestCase
     Coverband::Reporters::SimpleCovReport.expects(:current_root).at_least_once.returns('/tmp/root_dir')
 
     @redis.sadd(BASE_KEY, 'test/unit/dog.rb')
-    @store.send(:store_map, "#{BASE_KEY}.test/unit/dog.rb", example_hash)
+    @store.send(:pipelined_save, combined_report)
     SimpleCov.expects(:track_files)
     SimpleCov.expects(:add_not_loaded_files).returns('fake_file.rb' => [1])
     SimpleCov::Result.any_instance.expects(:format!)

data/test/unit/reports_web_test.rb

@@ -0,0 +1,37 @@
+# frozen_string_literal: true
+
+require File.expand_path('../test_helper', File.dirname(__FILE__))
+require 'aws-sdk'
+require File.expand_path('../../lib/coverband/reporters/web', File.dirname(__FILE__))
+require 'rack/test'
+
+ENV['RACK_ENV'] = 'test'
+
+if Gem::Version.new(RUBY_VERSION) >= Gem::Version.new('2.2.0')
+  module Coverband
+    class S3WebTest < Test::Unit::TestCase
+      include Rack::Test::Methods
+
+      def app
+        Coverband::Reporters::Web.new
+      end
+
+      # TODO add tests for all endpoints
+      test 'renders index content' do
+        get '/'
+        assert last_response.ok?
+        assert_match 'Coverband Web Admin Index', last_response.body
+      end
+
+      test 'renders show content' do
+        Coverband.configuration.s3_bucket = 'coverage-bucket'
+        s3 = mock('s3')
+        Aws::S3::Client.expects(:new).returns(s3)
+        s3.expects(:get_object).with(bucket: 'coverage-bucket', key: 'coverband/index.html').returns mock('response', body: mock('body', read: 'content'))
+        get '/show'
+        assert last_response.ok?
+        assert_equal 'content', last_response.body
+      end
+    end
+  end
+end

data/test/unit/s3_report_writer_test.rb

@@ -18,7 +18,13 @@ module Coverband
       File.expects(:read).at_least(0).returns("content ./assets/#{html_version}/")
       object.expects(:put).with(body: 'content ')
       Aws::S3::Resource.expects(:new).returns(s3)
-      S3ReportWriter.new('coverage-bucket').persist!
+
+      s3_writer_options = {
+        region: 'us-west-1',
+        access_key_id: '',
+        secret_access_key: ''
+      }
+      S3ReportWriter.new('coverage-bucket', s3_writer_options).persist!
     end
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: coverband
 version: !ruby/object:Gem::Version
-  version: 2.0.0
+  version: 2.0.3
 platform: ruby
 authors:
 - Dan Mayer
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-07-23 00:00:00.000000000 Z
+date: 2018-10-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: aws-sdk
@@ -108,20 +108,6 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-- !ruby/object:Gem::Dependency
-  name: sinatra
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
 - !ruby/object:Gem::Dependency
   name: test-unit
   requirement: !ruby/object:Gem::Requirement
@@ -207,16 +193,13 @@ files:
 - README.md
 - Rakefile
 - changes.md
+- config.ru
 - coverband.gemspec
-- docs/coverband_details.png
-- docs/coverband_index.png
-- docs/coverband_install.gif
 - lib/coverband.rb
 - lib/coverband/adapters/base.rb
 - lib/coverband/adapters/file_store.rb
 - lib/coverband/adapters/memory_cache_store.rb
 - lib/coverband/adapters/redis_store.rb
-- lib/coverband/baseline.rb
 - lib/coverband/collectors/base.rb
 - lib/coverband/collectors/coverage.rb
 - lib/coverband/collectors/trace.rb
@@ -225,8 +208,8 @@ files:
 - lib/coverband/reporters/base.rb
 - lib/coverband/reporters/console_report.rb
 - lib/coverband/reporters/simple_cov_report.rb
+- lib/coverband/reporters/web.rb
 - lib/coverband/s3_report_writer.rb
-- lib/coverband/s3_web.rb
 - lib/coverband/tasks.rb
 - lib/coverband/version.rb
 - test/benchmarks/.gitignore
@@ -237,7 +220,6 @@ files:
 - test/unit/adapters_file_store_test.rb
 - test/unit/adapters_memory_cache_store_test.rb
 - test/unit/adapters_redis_store_test.rb
-- test/unit/baseline_test.rb
 - test/unit/collectors_base_test.rb
 - test/unit/collectors_coverage_test.rb
 - test/unit/collectors_trace_test.rb
@@ -247,8 +229,8 @@ files:
 - test/unit/reports_base_test.rb
 - test/unit/reports_console_test.rb
 - test/unit/reports_simple_cov_test.rb
+- test/unit/reports_web_test.rb
 - test/unit/s3_report_writer_test.rb
-- test/unit/s3_web_test.rb
 homepage: https://github.com/danmayer/coverband
 licenses:
 - MIT
@@ -269,7 +251,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.7.3
+rubygems_version: 2.5.1
 signing_key: 
 specification_version: 4
 summary: Rack middleware to help measure production code usage (LOC runtime usage)
@@ -282,7 +264,6 @@ test_files:
 - test/unit/adapters_file_store_test.rb
 - test/unit/adapters_memory_cache_store_test.rb
 - test/unit/adapters_redis_store_test.rb
-- test/unit/baseline_test.rb
 - test/unit/collectors_base_test.rb
 - test/unit/collectors_coverage_test.rb
 - test/unit/collectors_trace_test.rb
@@ -292,5 +273,5 @@ test_files:
 - test/unit/reports_base_test.rb
 - test/unit/reports_console_test.rb
 - test/unit/reports_simple_cov_test.rb
+- test/unit/reports_web_test.rb
 - test/unit/s3_report_writer_test.rb
-- test/unit/s3_web_test.rb

data/lib/coverband/baseline.rb

@@ -1,44 +0,0 @@
-# frozen_string_literal: true
-
-module Coverband
-  class Baseline
-    def self.record
-      require 'coverage'
-      Coverage.start
-      yield
-
-      project_directory = File.expand_path(Coverband.configuration.root)
-      results = Coverage.result
-      results = results.reject { |key, _val| !key.match(project_directory) || Coverband.configuration.ignore.any? { |pattern| key.match(/#{pattern}/) } }
-
-      Coverband.configuration.store.save_report(convert_coverage_format(results))
-    end
-
-    def self.parse_baseline(_back_compat = nil)
-      Coverband.configuration.store.coverage
-    end
-
-    def self.exclude_files(files)
-      Coverband.configuration.ignore.each do |ignore|
-        path = Coverband.configuration.root + "/#{ignore}"
-        excludes = File.directory?(path) ? Dir.glob("#{path}/**/*") : [path]
-        files -= excludes
-      end
-      files
-    end
-
-    private
-
-    def self.convert_coverage_format(results)
-      file_map = {}
-      results.each_pair do |file, data|
-        lines_map = {}
-        data.each_with_index do |hits, index|
-          lines_map[(index + 1)] = hits unless hits.nil?
-        end
-        file_map[file] = lines_map
-      end
-      file_map
-    end
-  end
-end

data/lib/coverband/s3_web.rb

@@ -1,25 +0,0 @@
-# frozen_string_literal: true
-
-require 'sinatra/base'
-
-module Coverband
-  class S3Web < Sinatra::Base
-    set :public_folder, proc(){ File.expand_path('public', Gem::Specification.find_by_name('simplecov-html').full_gem_path) }
-
-    get '/' do
-      s3.get_object(bucket: Coverband.configuration.s3_bucket, key: 'coverband/index.html').body.read
-    end
-
-    private
-
-    def s3
-      begin
-        require 'aws-sdk'
-      rescue StandardError
-        Coverband.configuration.logger.error "coverband requires 'aws-sdk' in order use S3ReportWriter."
-        return
-      end
-      @s3 ||= Aws::S3::Client.new
-    end
-  end
-end