coverband 4.2.0.beta → 4.2.0.rc1

data/lib/coverband/reporters/html_report.rb

@@ -11,7 +11,7 @@ module Coverband
         base_path = options.fetch(:base_path) { nil }
 
         # list all files, even if not tracked by Coverband (0% coverage)
-        tracked_glob = "#{current_root}/{app,lib,config}/**/*.{rb}"
+        tracked_glob = "#{Coverband.configuration.current_root}/{app,lib,config}/**/*.{rb}"
         report_files = Coverband::Utils::Result.add_not_loaded_files(scov_style_report, tracked_glob)
         # apply coverband filters
         filtered_report_files = {}

data/lib/coverband/reporters/web.rb

@@ -36,7 +36,7 @@ module Coverband
           when /.*\.(css|js|gif|png)/
             @static.call(env)
           when %r{\/settings}
-            [200, { 'Content-Type' => 'text/json' }, [settings]]
+            [200, { 'Content-Type' => 'text/html' }, [settings]]
           when %r{\/debug_data}
             [200, { 'Content-Type' => 'text/json' }, [debug_data]]
           when %r{\/$}
@@ -58,7 +58,7 @@ module Coverband
       end
 
       def settings
-        Coverband.configuration.to_h.to_json
+        Coverband::Utils::HTMLFormatter.new(nil, base_path: base_path).format_settings!
       end
 
       def debug_data
@@ -66,7 +66,7 @@ module Coverband
       end
 
       def collect_coverage
-        Coverband::Collectors::Coverage.instance.report_coverage(true)
+        Coverband.report_coverage(true)
         notice = 'coverband coverage collected'
         [301, { 'Location' => "#{base_path}?notice=#{notice}" }, []]
       end

data/lib/coverband/utils/file_path_helper.rb

@@ -0,0 +1,57 @@
+# frozen_string_literal: true
+
+####
+# Helper functions for shared logic related to file path manipulation
+####
+module Coverband
+  module Utils
+    module FilePathHelper
+      ###
+      # Takes a full path and converts to a relative path
+      ###
+      def full_path_to_relative(full_path)
+        relative_filename = full_path
+        Coverband.configuration.all_root_paths.each do |root|
+          relative_filename = relative_filename.gsub(/^#{root}/, './')
+          # once we have a relative path break out of the loop
+          break if relative_filename.start_with? './'
+        end
+        relative_filename
+      end
+
+      ###
+      # relative_path_to_full code takes:
+      # relative_path: which is a full path the same as reported by Coverage
+      # roots: if a collection of all possible full app paths
+      #    EX: [Coverband.configuration.root_paths, "#{current_root}/"]
+      # The LAST item should be the current file system root
+      # it expands that expands and adds a '/' as that isn't there from Dir.pwd
+      #
+      # NOTEs on configuration.root_paths usage
+      # strings: matching is pretty simple for full string paths
+      # regex: to get regex to work for changing deploy directories
+      #        the regex must be double escaped in double quotes
+      #          (if using \d for example)
+      #        or use single qoutes
+      #        example: '/box/apps/app_name/releases/\d+/'
+      #        example: '/var/local/company/company.d/[0-9]*/'
+      ###
+      def relative_path_to_full(relative_path, roots)
+        relative_filename = relative_path
+        local_filename = relative_filename
+        roots.each do |root|
+          relative_filename = relative_filename.gsub(/^#{root}/, './')
+        end
+        # the filename for our reports is expected to be a full path.
+        # roots.last should be roots << current_root}/
+        # a fully expanded path of config.root
+        # filename = filename.gsub('./', roots.last)
+        # above only works for app files
+        # we need to rethink some of this logic
+        # gems aren't at project root and can have multiple locations
+        local_root = roots.find { |root| File.exist?(relative_filename.gsub('./', root)) }
+        local_root ? relative_filename.gsub('./', local_root) : local_filename
+      end
+    end
+  end
+end

data/lib/coverband/utils/html_formatter.rb

@@ -18,7 +18,7 @@ module Coverband
       def initialize(report, options = {})
         @notice = options.fetch(:notice) { nil }
         @base_path = options.fetch(:base_path) { nil }
-        @coverage_result = Coverband::Utils::Result.new(report)
+        @coverage_result = Coverband::Utils::Result.new(report) if report
       end
 
       def format!
@@ -29,8 +29,16 @@ module Coverband
         format_html(@coverage_result)
       end
 
+      def format_settings!
+        format_settings
+      end
+
       private
 
+      def format_settings
+        template('settings').result(binding)
+      end
+
       def format(result)
         Dir[File.join(File.dirname(__FILE__), '../../../public/*')].each do |path|
           FileUtils.cp_r(path, asset_output_path)

data/lib/coverband/utils/railtie.rb

@@ -7,7 +7,7 @@ module Coverband
     end
 
     config.after_initialize do
-      Coverband::Collectors::Coverage.instance.report_coverage(true)
+      Coverband.report_coverage(true)
     end
 
     rake_tasks do

data/lib/coverband/utils/tasks.rb

@@ -25,4 +25,13 @@ namespace :coverband do
     environment
     Coverband.configuration.store.clear!
   end
+
+  ###
+  # clear data helpful for development or after configuration issues
+  ###
+  desc 'upgrade previous Coverband datastore to latest format'
+  task :migrate do
+    environment
+    Coverband.configuration.store.migrate!
+  end
 end

data/lib/coverband/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Coverband
-  VERSION = '4.2.0.beta'
+  VERSION = '4.2.0.rc1'
 end

data/test/benchmarks/benchmark.rake

@@ -134,7 +134,7 @@ namespace :benchmarks do
       x.config(time: 12, warmup: 5, suite: suite)
       x.report 'coverband' do
         work
-        Coverband::Collectors::Coverage.instance.report_coverage
+        Coverband.report_coverage(true)
       end
       x.report 'no coverband' do
         work
@@ -176,6 +176,10 @@ namespace :benchmarks do
         @file_hash_cache[file] = Digest::MD5.file(__FILE__).hexdigest
       end
     end
+
+    def store.full_path_to_relative(file)
+      file
+    end
   end
 
   def reporting_speed
@@ -215,12 +219,43 @@ namespace :benchmarks do
     $stdout = previous_out
   end
 
+  def measure_configure_memory
+    require 'memory_profiler'
+    # warmup
+    3.times { Coverband.configure }
+
+    previous_out = $stdout
+    capture = StringIO.new
+    $stdout = capture
+
+    MemoryProfiler.report do
+      10.times do
+        Coverband.configure do |config|
+          redis_url = ENV['CACHE_REDIS_URL'] || ENV['REDIS_URL']
+          config.store = Coverband::Adapters::RedisStore.new(Redis.new(url: redis_url), redis_namespace: 'coverband_data')
+        end
+      end
+    end.pretty_print
+    data = $stdout.string
+    $stdout = previous_out
+    puts data
+    raise 'leaking memory!!!' unless data.match('Total retained:  0 bytes')
+  ensure
+    $stdout = previous_out
+  end
+
   desc 'runs memory reporting on Redis store'
   task memory_reporting: [:setup] do
     puts 'runs memory benchmarking to ensure we dont leak'
     measure_memory
   end
 
+  desc 'runs memory reporting on configure'
+  task memory_configure_reporting: [:setup] do
+    puts 'runs memory benchmarking on configure to ensure we dont leak'
+    measure_configure_memory
+  end
+
   desc 'runs memory leak check via Rails tests'
   task memory_rails: [:setup] do
     puts 'runs memory rails test to ensure we dont leak'

data/test/benchmarks/coverage_fork.sh

@@ -0,0 +1,37 @@
+#!/usr/bin/env ruby
+
+# This is a small script to illustrate how previous results get wiped out by forking
+# this has implications of forking processes like Resque...
+# this in the end would cause coverage.array_diff
+# with previous results to add NEGATIVE code hits to the stored Coverage
+# which in turn causes all sorts of crazy issues.
+#
+# ruby test/benchmarks/coverage_fork.rb
+# in parent before fork
+# {"/Users/danmayer/projects/coverband/test/dog.rb"=>[nil, nil, 1, 1, 2, nil, nil]}
+# in child after fork
+# {"/Users/danmayer/projects/coverband/test/dog.rb"=>[nil, nil, 0, 0, 0, nil, nil]}
+# now triggering hits
+# {"/Users/danmayer/projects/coverband/test/dog.rb"=>[nil, nil, 0, 0, 3, nil, nil]}
+#
+# I believe this might be related to CoW and GC... not sure
+# http://patshaughnessy.net/2012/3/23/why-you-should-be-excited-about-garbage-collection-in-ruby-2-0
+#
+# NOTE: That the child now has 0 hits where previously method definitions had 1
+# this causes all sorts of bad things to happen.
+require 'coverage'
+Coverage.start
+load './test/dog.rb'
+Dog.new.bark
+Dog.new.bark
+puts 'in parent before fork'
+puts Coverage.peek_result
+fork do
+  puts 'in child after fork'
+  puts Coverage.peek_result
+  puts 'now triggering hits'
+  Dog.new.bark
+  Dog.new.bark
+  Dog.new.bark
+  puts Coverage.peek_result
+end

data/test/{unit/adapters_base_test.rb → coverband/adapters/base_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 AdaptersBaseTest < Minitest::Test
   def setup

data/test/{unit/adapters_file_store_test.rb → coverband/adapters/file_store_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 AdaptersFileStoreTest < Minitest::Test
   def setup

data/test/{unit/adapters_redis_store_test.rb → coverband/adapters/redis_store_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 RedisTest < Minitest::Test
   REDIS_STORAGE_FORMAT_VERSION = Coverband::Adapters::RedisStore::REDIS_STORAGE_FORMAT_VERSION

data/test/{unit/collectors_coverage_test.rb → coverband/collectors/coverage_test.rb}

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
-require File.expand_path('../test_helper', File.dirname(__FILE__))
-require File.expand_path('./dog', File.dirname(__FILE__))
+require File.expand_path('../../test_helper', File.dirname(__FILE__))
+require File.expand_path('../../dog', File.dirname(__FILE__))
 
 if Gem::Version.new(RUBY_VERSION) >= Gem::Version.new('2.3.0')
   class CollectorsCoverageTest < Minitest::Test
@@ -44,5 +44,19 @@ if Gem::Version.new(RUBY_VERSION) >= Gem::Version.new('2.3.0')
       @coverband.reset_instance
       @coverband.report_coverage
     end
+
+    test 'default tmp ignores' do
+      heroku_build_file = '/tmp/build_81feca8c72366e4edf020dc6f1937485/config/initializers/assets.rb'
+      assert_equal false, @coverband.send(:track_file?, heroku_build_file)
+    end
+
+    test '#array_diff never returns negative hits' do
+      # this can occur if a process forks after initializing the previous results
+      # see test/benchmarks/coverage_fork.rb
+      latest = [0, nil]
+      original = [1, nil]
+      expected = [0, nil]
+      assert_equal expected, @coverband.send(:array_diff, latest, original)
+    end
   end
 end

data/test/{unit → coverband}/configuration_test.rb

@@ -16,22 +16,32 @@ class BaseTest < Minitest::Test
     end
   end
 
-  test 'defaults ' do
+  test 'defaults' do
     coverband = Coverband::Collectors::Coverage.instance.reset_instance
     assert_equal ['vendor', 'internal:prelude', 'schema.rb'], coverband.instance_variable_get('@ignore_patterns')
   end
 
-  test 'gem_paths ' do
+  test 'gem_paths' do
     Coverband::Collectors::Coverage.instance.reset_instance
     assert Coverband.configuration.gem_paths.first != nil
   end
 
-  test 'groups ' do
+  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
+    Coverband::Collectors::Coverage.instance.reset_instance
+    current_paths = Coverband.configuration.root_paths.dup
+    # verify previous bug fix
+    # it would extend the root_paths instance variable on each invokation
+    Coverband.configuration.all_root_paths
+    Coverband.configuration.all_root_paths
+    assert_equal current_paths, Coverband.configuration.root_paths
+  end
+
   test 's3 options' do
     Coverband::Collectors::Coverage.instance.reset_instance
     Coverband.configure do |config|

data/test/{unit → coverband/integrations}/background_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 BackgroundTest < Minitest::Test
   def setup

data/test/{unit → coverband/integrations}/middleware_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__))
 require 'rack'
 
 class MiddlewareTest < Minitest::Test

data/test/{unit/rack_server_checkout_test.rb → coverband/integrations/rack_server_check_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 RackServerCheckTest < Minitest::Test
   test 'returns true when running in rack server' do

data/test/{unit → coverband/integrations}/resque_worker_test.rb

@@ -1,18 +1,21 @@
 # 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'
-    Resque::Worker.new.work_one_job
+    worker = Resque::Worker.new
+    worker.startup
+    worker.work_one_job
   end
 
   def setup
     super
     Coverband.configure do |config|
       config.background_reporting_enabled = false
+      config.root_paths = ["#{File.expand_path('../', File.dirname(__FILE__))}/"]
     end
     Coverband.start
     redis = Coverband.configuration.store.send(:redis)
@@ -20,16 +23,14 @@ class ResqueWorkerTest < Minitest::Test
   end
 
   test 'resque job coverage' do
+    relative_job_file = './integrations/test_resque_job.rb'
     resque_job_file = File.expand_path('./test_resque_job.rb', File.dirname(__FILE__))
     require resque_job_file
 
-    #report after loading the file in parent process
-    Coverband::Collectors::Coverage.instance.report_coverage(true)
-
     enqueue_and_run_job
 
-    puts "assert_equal 1, Coverband.configuration.store.coverage['#{resque_job_file}']['data'][4]"
-    assert_equal 1, Coverband.configuration.store.coverage[resque_job_file]['data'][4]
+    assert !Coverband::Background.running?
+
+    assert_equal 1, Coverband.configuration.store.coverage[relative_job_file]['data'][4]
   end
 end
-

data/test/coverband/reporters/base_test.rb

@@ -0,0 +1,168 @@
+# frozen_string_literal: true
+
+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]
+    expects = [0, 0, 2, 0, 1]
+
+    assert_equal expects, Coverband::Reporters::Base.send(:merge_arrays, first, second)
+  end
+
+  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]
+
+    assert_equal expects, Coverband::Reporters::Base.send(:merge_arrays, first, second)
+  end
+
+  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]
+
+    assert_equal expects, Coverband::Reporters::Base.send(:merge_arrays, first, second)
+  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'
+    roots = ['/app/', '/full/remote_app/path/']
+
+    lines_hit = [1, 3, 6]
+    store.stubs(:coverage).returns(key => lines_hit)
+    expected = { key => [1, 3, 6] }
+
+    assert_equal expected, Coverband::Reporters::Base.send(:get_current_scov_data_imp, store, roots)
+  end
+
+  ###
+  # This test uses real world example data which helped uncover a bug
+  # The copied data doesn't format easily and isn't worth the effort to meet
+  # string style
+  # rubocop:disable all
+  ###
+  test '#get_current_scov_data_imp merges multiples of file data' do
+    coverage = {'/base/66/app/controllers/dashboard_controller.rb' =>
+       {"first_updated_at"=>1549610119,
+        "last_updated_at"=>1549610200,
+        "file_hash"=>"14dc84e940e26cbfb9ac79b43862e762",
+        "data"=>[1, 1, 1, nil, 1, 1, nil, nil, 1, nil, 1, 26, 26, nil, 26, 26, 26, 26, 26, 26, 26, nil, nil, 1, nil, 1, 26, 19, 0, 0, 0, 0, nil, nil, nil, nil, 0, 0, nil, nil, 1, 26, 26, 26, nil, nil, nil, nil, nil, 1, 26, nil, nil, 1, 26, nil, nil]},
+     '/base/78/app/controllers/dashboard_controller.rb' =>
+       {"first_updated_at"=>1549658574,
+        "last_updated_at"=>1549729830,
+        "file_hash"=>"14dc84e940e26cbfb9ac79b43862e762",
+        "data"=>[21, 21, 21, nil, 21, 21, nil, nil, 21, nil, 21, 22, 22, nil, 22, 22, 22, 22, 22, 22, 22, nil, nil, 21, nil, 21, 22, 13, 0, 0, 0, 0, nil, nil, nil, nil, 0, 0, nil, nil, 21, 22, 22, 22, nil, nil, nil, nil, nil, 21, 22, nil, nil, 21, 22, nil, nil]},
+    '/base/70/app/controllers/dashboard_controller.rb' =>
+       {"first_updated_at"=>1549617873,
+        "last_updated_at"=>1549618094,
+        "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/']
+
+    lines_hit = [1, 3, 6]
+    store.stubs(:coverage).returns(coverage)
+    File.expects(:exist?).at_least_once
+      .with('/base/[0-9]*/app/controllers/dashboard_controller.rb')
+      .returns(false)
+    File.expects(:exist?).at_least_once.with(key).returns(true)
+
+    expected = {"first_updated_at"=>1549617873,
+                "last_updated_at"=>1549618094,
+                "file_hash"=>"14dc84e940e26cbfb9ac79b43862e762",
+                "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)[key]
+  end
+end