coverband 4.2.0 → 4.2.1

data/test/forked/rails_full_stack_test.rb

@@ -0,0 +1,106 @@
+# frozen_string_literal: true
+
+require File.expand_path('../rails_test_helper', File.dirname(__FILE__))
+
+class RailsFullStackTest < Minitest::Test
+  include Capybara::DSL
+  include Capybara::Minitest::Assertions
+
+  def setup
+    super
+    rails_setup
+    # preload first coverage hit
+    Coverband.report_coverage
+    require 'rainbow'
+    Rainbow('this text is red').red
+  end
+
+  def teardown
+    super
+    Capybara.reset_sessions!
+    Capybara.use_default_driver
+  end
+
+  # We have to combine everything in one test
+  # because we can only initialize rails once per test
+  # run. Possibly fork test runs to avoid this problem in future?
+  unless ENV['COVERBAND_MEMORY_TEST']
+    test 'this is how we do it' do
+      visit '/dummy/show'
+      Coverband.report_coverage
+      assert_content('I am no dummy')
+      visit '/coverage'
+      within page.find('a', text: /dummy_controller.rb/).find(:xpath, '../..') do
+        assert_selector('td', text: '100.0 %')
+      end
+
+      # Test gems are reporting coverage
+      assert_content('Gems')
+      assert page.html.match('rainbow/wrapper.rb')
+
+      # Test eager load data stored separately
+      dummy_controller = "./test/rails#{Rails::VERSION::MAJOR}_dummy/app/controllers/dummy_controller.rb"
+      store.type = :eager_loading
+      eager_expected = [1, 1, 0, nil, nil]
+      results = store.coverage[dummy_controller]['data']
+      assert_equal(eager_expected, results)
+
+      store.type = Coverband::RUNTIME_TYPE
+      runtime_expected = [0, 0, 1, nil, nil]
+      results = store.coverage[dummy_controller]['data']
+    end
+  end
+
+  ###
+  # as we run it in single test mode via the benchmarks.
+  # Add new tests below this test
+  ###
+  if ENV['COVERBAND_MEMORY_TEST']
+    test 'memory usage' do
+      return unless ENV['COVERBAND_MEMORY_TEST']
+
+      # we don't want this to run during our standard test suite
+      # as the below profiler changes the runtime
+      # and shold only be included for isolated processes
+      begin
+        require 'memory_profiler'
+
+        # warmup
+        3.times do
+          visit '/dummy/show'
+          assert_content('I am no dummy')
+          Coverband.report_coverage
+        end
+
+        previous_out = $stdout
+        capture = StringIO.new
+        $stdout = capture
+
+        MemoryProfiler.report do
+          15.times do
+            visit '/dummy/show'
+            assert_content('I am no dummy')
+            Coverband.report_coverage
+            ###
+            # Set to nil not {} as it is easier to verify that no memory is retained when nil gets released
+            # don't use Coverband::Collectors::Delta.reset which sets to {}
+            #
+            # we clear this as this one variable is expected to retain memory and is a false positive
+            ###
+            Coverband::Collectors::Delta.class_variable_set(:@@previous_coverage, nil)
+            # needed to test older versions to discover when we had the regression
+            # Coverband::Collectors::Coverage.instance.send(:add_previous_results, nil)
+          end
+        end.pretty_print
+        data = $stdout.string
+        $stdout = previous_out
+        if data.match(/retained objects by gem(.*)retained objects by file/m)[0]&.match(/coverband/)
+          puts data
+          raise 'leaking memory!!!'
+        end
+      ensure
+        $stdout = previous_out
+      end
+    end
+  end
+end

data/test/forked/rails_rake_full_stack_test.rb

@@ -0,0 +1,40 @@
+require File.expand_path('../rails_test_helper', File.dirname(__FILE__))
+require 'rails'
+
+class RailsRakeFullStackTest < Minitest::Test
+
+  test 'rake tasks shows coverage properly within eager_loading' do
+    store.instance_variable_set(:@redis_namespace, 'coverband_test')
+    store.clear!
+    system("COVERBAND_CONFIG=./test/rails#{Rails::VERSION::MAJOR}_dummy/config/coverband.rb bundle exec rake -f test/rails#{Rails::VERSION::MAJOR}_dummy/Rakefile middleware")
+    store.instance_variable_set(:@redis_namespace, 'coverband_test')
+    store.type = :eager_loading
+    pundit_file = store.coverage.keys.grep(/pundit.rb/).first
+    refute_nil pundit_file
+    pundit_coverage = store.coverage[pundit_file]
+    refute_nil pundit_coverage
+    assert_includes pundit_coverage['data'], 1
+
+    store.type = Coverband::RUNTIME_TYPE
+    if ENV['SIMULATE_ONESHOT']
+      pundit_coverage = store.get_coverage_report[Coverband::RUNTIME_TYPE][pundit_file]
+      assert pundit_coverage['data'].compact.all? { |el| el == 0}
+    else
+      pundit_coverage = store.coverage[pundit_file]
+      assert_nil pundit_coverage
+    end
+  end
+
+  test "ignored rake tasks don't add coverage" do
+    store.clear!
+    store.instance_variable_set(:@redis_namespace, 'coverband_test')
+    store.send(:save_report, basic_coverage_full_path)
+    output = `COVERBAND_CONFIG=./test/rails#{Rails::VERSION::MAJOR}_dummy/config/coverband.rb bundle exec rake -f test/rails#{Rails::VERSION::MAJOR}_dummy/Rakefile coverband:clear`
+    assert_nil output.match(/Coverband: Reported coverage via thread/)
+    coverage_report = store.get_coverage_report
+    empty_hash = {}
+    assert_equal empty_hash, coverage_report[Coverband::RUNTIME_TYPE]
+    assert_equal empty_hash, coverage_report[:eager_loading]
+    assert_equal empty_hash, coverage_report[:merged]
+  end
+end

data/test/integration/full_stack_test.rb

@@ -6,52 +6,54 @@ 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'
-  RELATIVE_FILE = './fake_app/basic_rack.rb'
 
   def setup
     super
     Coverband::Collectors::Coverage.instance.reset_instance
     Coverband.configure do |config|
-      config.reporting_frequency = 100.0
-      config.store = Coverband::Adapters::RedisStore.new(Redis.new)
+      config.store = Coverband::Adapters::RedisStore.new(Redis.new(), redis_namespace: 'coverband_test')
       config.s3_bucket = nil
-      config.background_reporting_enabled = false
+      config.background_reporting_enabled = true
       config.root_paths = ["#{File.expand_path('../', File.dirname(__FILE__))}/"]
       config.track_gems = true
     end
     Coverband.configuration.store.clear!
     Coverband.start
+    Coverband::Collectors::Coverage.instance.eager_loading!
+    @rack_file = require_unique_file 'fake_app/basic_rack.rb'
+    Coverband.report_coverage
     Coverband::Collectors::Coverage.instance.runtime!
-    @rack_file = File.expand_path(TEST_RACK_APP, File.dirname(__FILE__))
-    require @rack_file
-    # preload first coverage hit
-    Coverband::Collectors::Coverage.instance.report_coverage(true)
   end
 
   test 'call app' do
     request = Rack::MockRequest.env_for('/anything.json')
-    middleware = Coverband::Middleware.new(fake_app_with_lines)
+    middleware = Coverband::BackgroundMiddleware.new(fake_app_with_lines)
     results = middleware.call(request)
     assert_equal 'Hello Rack!', results.last
-    sleep(0.2)
+    Coverband.report_coverage
     expected = [nil, nil, 0, nil, 0, 0, 1, nil, nil]
-    assert_equal expected, Coverband.configuration.store.coverage[RELATIVE_FILE]['data']
+    assert_equal expected, Coverband.configuration.store.coverage[@rack_file]['data']
 
     # additional calls increase count by 1
     middleware.call(request)
-    sleep(0.2)
+    Coverband.report_coverage
     expected = [nil, nil, 0, nil, 0, 0, 2, nil, nil]
-    assert_equal expected, Coverband.configuration.store.coverage[RELATIVE_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']
+    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::Middleware.new(fake_app_with_lines)
+    middleware = Coverband::BackgroundMiddleware.new(fake_app_with_lines)
     results = middleware.call(request)
     assert_equal 'Hello Rack!', results.last
-    sleep(0.1)
+    Coverband.report_coverage
     assert Coverband.configuration.store.coverage.keys.any? { |key| key.end_with?('rainbow/global.rb') }
   end
 

data/test/rails4_dummy/Rakefile

@@ -0,0 +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'
+
+Rails.application.load_tasks

data/test/rails4_dummy/config/application.rb

@@ -1,15 +1,14 @@
-require File.expand_path('../boot', __FILE__)
+# frozen_string_literal: true
+
+require File.expand_path('boot', __dir__)
+
+require 'rails'
+require 'action_controller/railtie'
+require 'coverband'
+Bundler.require(*Rails.groups)
 
-require "rails"
-require "action_controller/railtie"
 module Rails4Dummy
   class Application < Rails::Application
-    # Coverband needs to be setup before any of the initializers to capture usage of them
-    Coverband.configure(File.open("#{Rails.root}/config/coverband.rb"))
-    config.middleware.use Coverband::Middleware
-    config.before_initialize do
-      Coverband.start
-    end
     config.eager_load = true
   end
 end

data/test/rails4_dummy/config/coverband.rb

@@ -1,11 +1,13 @@
 Coverband.configure do |config|
   config.root              = Dir.pwd
-  config.store             = Coverband::Adapters::RedisStore.new(Redis.new(url: ENV['REDIS_URL'])) if defined? Redis
+  config.store             = Coverband::Adapters::RedisStore.new(Redis.new(url: ENV['REDIS_URL']), redis_namespace: 'coverband_test') if defined? Redis
   config.ignore            = %w[vendor .erb$ .slim$]
   config.root_paths        = []
-  config.logger              = Rails.logger
+  config.logger            = Rails.logger
   config.verbose           = true
-  config.background_reporting_enabled = false
+  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']
 end

data/test/rails5_dummy/Rakefile

@@ -0,0 +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'
+
+Rails.application.load_tasks

data/test/rails5_dummy/config/application.rb

@@ -1,16 +1,12 @@
-require "rails"
-require "action_controller/railtie"
+# frozen_string_literal: true
+
+require 'rails'
+require 'action_controller/railtie'
 require 'coverband'
+Bundler.require(*Rails.groups)
 
-module Dummy
+module Rails5Dummy
   class Application < Rails::Application
-    # Coverband needs to be setup before any of the initializers to capture usage of them
-    Coverband.configure(File.open("#{Rails.root}/config/coverband.rb"))
-    config.middleware.use Coverband::Middleware
-    config.before_initialize do
-      Coverband.start
-    end
     config.eager_load = true
   end
 end
-

data/test/rails5_dummy/config/coverband.rb

@@ -1,11 +1,13 @@
 Coverband.configure do |config|
   config.root              = Dir.pwd
-  config.store             = Coverband::Adapters::RedisStore.new(Redis.new(url: ENV['REDIS_URL'])) if defined? Redis
+  config.store             = Coverband::Adapters::RedisStore.new(Redis.new(url: ENV['REDIS_URL']), 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 = false
+  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']
 end

data/test/rails_test_helper.rb

@@ -1,16 +1,33 @@
+# frozen_string_literal: true
+
+require 'minitest'
+require 'minitest/fork_executor'
+
+# Forked executor includes autorun which does not work with qrush/m
+# https://github.com/qrush/m/issues/26
+# https://github.com/seattlerb/minitest/blob/master/lib/minitest/autorun.rb
+if defined?(M)
+  Minitest.class_eval do
+    def self.autorun
+      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'
 def rails_setup
-  ENV["RAILS_ENV"] = "test"
+  ENV['RAILS_ENV'] = 'test'
   require 'rails'
-  #coverband must be required after rails
-  load 'coverband/utils/railtie.rb'
+  # coverband must be required after rails
   Coverband.configure("./test/rails#{Rails::VERSION::MAJOR}_dummy/config/coverband.rb")
+  load 'coverband/utils/railtie.rb'
+
   require_relative "../test/rails#{Rails::VERSION::MAJOR}_dummy/config/environment"
   require 'capybara/rails'
-  #Our coverage report is wrapped in display:none as of now
+  # Our coverage report is wrapped in display:none as of now
   Capybara.ignore_hidden_elements = false
   require 'mocha/minitest'
 end
-

data/test/test_helper.rb

@@ -7,8 +7,8 @@ original_verbosity = $VERBOSE
 $VERBOSE = nil
 require 'rubygems'
 require 'aws-sdk-s3'
-require 'coveralls'
 require 'simplecov'
+require 'coveralls'
 require 'minitest/autorun'
 require 'mocha/minitest'
 require 'ostruct'
@@ -19,17 +19,29 @@ require 'pry-byebug'
 require_relative 'unique_files'
 $VERBOSE = original_verbosity
 
-Coveralls.wear!
+unless ENV['ONESHOT'] || ENV['SIMULATE_ONESHOT']
+  SimpleCov.formatter = Coveralls::SimpleCov::Formatter
+  SimpleCov.start do
+    add_filter 'test/forked'
+  end
+
+  Coveralls.wear!
+end
+
 
 module Coverband
   module Test
     def self.reset
-      [:eager_loading, nil].each do |type|
+      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, {})
+      [:eager_loading, :runtime].each do |type|
         Coverband.configuration.store.type = type
         Coverband.configuration.store.clear!
       end
       Coverband.configuration.reset
       Coverband::Collectors::Coverage.instance.reset_instance
+      Coverband.configuration.redis_namespace = 'coverband_test'
       Coverband::Background.stop
     end
 
@@ -60,6 +72,7 @@ def test(name, &block)
               false
             end
   raise "#{test_name} is already defined in #{self}" if defined
+
   if block_given?
     define_method(test_name, &block)
   else
@@ -83,16 +96,41 @@ def basic_coverage
   { 'app_path/dog.rb' => example_line }
 end
 
+def basic_coverage_full_path
+  { basic_coverage_file_full_path => example_line }
+end
+
+def basic_source_fixture_coverage
+  { source_fixture('sample.rb') => example_line }
+end
+
+def basic_coverage_file_full_path
+  "#{test_root}/dog.rb"
+end
+
 def source_fixture(filename)
   File.expand_path(File.join(File.dirname(__FILE__), 'fixtures', filename))
 end
 
+def fixtures_root
+  File.expand_path(File.join(File.dirname(__FILE__), 'fixtures'))
+end
+
 def test_root
   File.expand_path(File.join(File.dirname(__FILE__)))
 end
 
+###
+# This handles an issue where the store is setup in tests prior to being able to set the namespace
+###
 def store
-  Coverband.configuration.store
+  if Coverband.configuration.store.redis_namespace=='coverband_test'
+    Coverband.configuration.store
+  else
+    Coverband.configuration.redis_namespace = 'coverband_test'
+    Coverband.configuration.instance_variable_set(:@store, nil)
+    Coverband.configuration.store
+  end
 end
 
 # Taken from http://stackoverflow.com/questions/4459330/how-do-i-temporarily-redirect-stderr-in-ruby

data/test/unique_files.rb

@@ -2,22 +2,30 @@
 
 require 'securerandom'
 require 'fileutils'
+require 'erb'
+require 'ostruct'
 
-UNIQUE_FILES_DIR = "./test/unique_files"
+UNIQUE_FILES_DIR = './test/unique_files'
 
-def require_unique_file(file = 'dog.rb')
-  dir = "#{UNIQUE_FILES_DIR}/#{SecureRandom.uuid}"
-  FileUtils.mkdir_p(dir)
-  temp_file = "#{dir}/#{file}"
-  File.open(temp_file, 'w'){ |w| w.write(File.read("./test/#{file}")) }
+def require_unique_file(file = 'dog.rb', variables = {})
+  uuid = SecureRandom.uuid
+  dir = "#{UNIQUE_FILES_DIR}/#{uuid}"
+  file_name = file.sub('.erb', '')
+  temp_file = "#{dir}/#{file_name}"
+  FileUtils.mkdir_p(Pathname.new(temp_file).dirname.to_s)
+  file_contents = File.read("./test/#{file}")
+  file_contents = ERB.new(file_contents).result(OpenStruct.new(variables).instance_eval { binding }) if variables.any?
+  File.open(temp_file, 'w') { |w| w.write(file_contents) }
   require temp_file
-  temp_file
+  Coverband::Utils::FilePathHelper.full_path_to_relative(File.expand_path(temp_file))
 end
 
 def remove_unique_files
   FileUtils.rm_r(UNIQUE_FILES_DIR) if File.exist?(UNIQUE_FILES_DIR)
 end
 
-Minitest.after_run do
-  remove_unique_files
+if defined?(Minitest)
+  Minitest.after_run do
+    remove_unique_files
+  end
 end

data/views/file_list.erb

@@ -39,7 +39,7 @@
             <%= link_to_source_file(source_file) %>
           </td>
           <td class="<%= coverage_css_class(source_file.covered_percent) %> strong"><%= source_file.covered_percent.round(2).to_s %> %</td>
-          <% runtime_percentage = result.file_with_type(source_file, Coverband::RUNTIME_TYPE).try(:covered_percent).try(:round, 2) %>
+          <% runtime_percentage = result.runtime_relevant_coverage(source_file) %>
           <td class="<%= "#{coverage_css_class(runtime_percentage)}" %> strong">
             <%= "#{runtime_percentage || '0'} %" %>
           </td>
@@ -47,7 +47,7 @@
           <td><%= source_file.covered_lines.count + source_file.missed_lines.count %></td>
           <td><%= source_file.covered_lines.count %></td>
           <td>
-            <%= result.file_with_type(source_file, Coverband::RUNTIME_TYPE).try(:covered_lines).try(:count) || 0 %>
+            <%= result.file_with_type(source_file, Coverband::RUNTIME_TYPE)&.covered_lines_count || 0 %>
           </td>
           <td><%= source_file.missed_lines.count %></td>
           <td><%= source_file.covered_strength %></td>

data/views/gem_list.erb

@@ -20,8 +20,10 @@
       <tr>
         <th>Gem</th>
         <th>% covered</th>
+        <th>% runtime</th>
         <th>Relevant Lines</th>
         <th>Lines covered</th>
+        <th>Lines runtime</th>
         <th>Lines missed</th>
         <th>Avg. Hits / Line</th>
       </tr>
@@ -37,10 +39,17 @@
             <% end %>
           </td>
           <td class="<%= coverage_css_class(source_file.covered_percent) %> strong"><%= source_file.covered_percent.round(2).to_s %> %</td>
+          <% runtime_percentage = result.group_file_list_with_type(title, source_file, Coverband::RUNTIME_TYPE)&.formatted_covered_percent %>
+          <td class="<%= "#{coverage_css_class(runtime_percentage)}" %> strong">
+            <%= "#{runtime_percentage || '0'} %" %>
+          </td>
           <td><%= source_file.covered_lines + source_file.missed_lines %></td>
           <td><%= source_file.covered_lines %></td>
+          <td>
+            <%= result.group_file_list_with_type(title, source_file, Coverband::RUNTIME_TYPE)&.covered_lines || 0 %>
+          </td>
           <td><%= source_file.missed_lines %></td>
-          <td><%= source_file.covered_strength %></td>
+          <td><%= source_file.covered_strength.round(1) %></td>
         </tr>
       <% end %>
     </tbody>

data/views/layout.erb

@@ -18,8 +18,6 @@
       <div id="header">
         <a href='<%= base_path %>'>Coverband Admin</a> &nbsp;
         <a href='<%= base_path %>settings'>Info</a> &nbsp;
-        <%= button("#{base_path}collect_coverage", 'force coverage collection') %> &nbsp;
-        <%= button("#{base_path}reload_files", 'reload Coverband files') %> &nbsp;
         <% if Coverband.configuration.web_enable_clear %>
           <%= button("#{base_path}clear", 'clear coverage report', delete: true) %>
         <% end %>
@@ -31,6 +29,11 @@
       <div class="notice"><%= notice %></div>
       <% end %>
       <div class="timestamp">Generated <%= timeago(Time.now) %></div>
+      <br/>
+      <div class="timestamp">
+        Coverage Recording Started: <%= timeago(result.source_files.first_seen_at) %>
+      </div>
+
       <ul class="group_tabs"></ul>
 
       <div id="content">
@@ -48,7 +51,11 @@
       <div class="source_files">
       <% result.source_files.each do |source_file| %>
         <% if (!source_file.gem? || (view_gems? && gem_details? && source_file.gem? )) %>
-          <%= formatted_source_file_loader(result, source_file) %>
+          <% if static_html? %>
+            <%= formatted_source_file(result, source_file) %>
+          <% else %>
+            <%= formatted_source_file_loader(result, source_file) %>
+          <% end %>
         <% end %>
       <% end %>
       </div>

data/views/source_file.erb

@@ -2,14 +2,23 @@
   <div class="header">
     <h3><%= shortened_filename source_file %></h3>
     <h4>
-      <span class="<%= coverage_css_class(source_file.covered_percent) %>"><%= source_file.covered_percent.round(2).to_s %> %</span>
-       covered
+      <span class="<%= coverage_css_class(source_file.covered_percent) %>">
+        <%= source_file.covered_percent.round(2).to_s %> %
+      </span>
+      covered,
+
+      <span class="<%= coverage_css_class(source_file.covered_percent) %>">
+        <%= result.runtime_relevant_coverage(source_file) %> %
+      </span>
+      runtime covered
+
       <% if Coverband.configuration.web_enable_clear %>
         <%= button("#{base_path}clear_file?filename=#{source_file.relative_path}", 'clear file coverage') %> &nbsp;
       <% end %>
     </h4>
     <div>
       <b><%= source_file.lines_of_code %></b> relevant lines.
+      <b><%= result.runtime_relavent_lines(source_file) %></b> runtime relevant lines.
       <span class="green"><b><%= source_file.covered_lines.count %></b> lines covered</span> and
       <span class="red"><b><%= source_file.missed_lines.count %></b> lines missed.</span>
     </div>
@@ -25,9 +34,9 @@
         <li class="<%= line.status %>" data-hits="<%= line.coverage ? line.coverage : '' %>" data-linenumber="<%= line.number %>">
           <% if line.covered? %><span class="hits">
             load:
-            <%= result.file_with_type(source_file, Coverband::EAGER_TYPE).try(:lines).try(:[], index).try(:coverage) || 0 %>,
+            <%= result.file_with_type(source_file, Coverband::EAGER_TYPE)&.line_coverage(index) || 0 %>,
             runtime:
-            <%= result.file_with_type(source_file, Coverband::RUNTIME_TYPE).try(:lines).try(:[], index).try(:coverage) || 0 %>
+            <%= result.file_with_type(source_file, Coverband::RUNTIME_TYPE)&.line_coverage(index) || 0 %>
             all: <%= line.coverage %>
           </span><% end %>
           <% if line.skipped? %><span class="hits">skipped</span><% end %>

data/views/source_file_loader.erb

@@ -1,6 +1,6 @@
 <div class="source_table" id="<%= id source_file %>" data-loader-url="<%= base_path %>load_file_details?filename=<%= source_file.filename %>">
   <div class='loader'>
-    loading...
+    loading source data...
     <br/>
     <img src="<%= assets_path('loading.gif') %>" alt="loading"/>
   </div>