coverband 6.0.3.rc.3 → 6.1.0

data/lib/coverband/reporters/web.rb

@@ -112,11 +112,16 @@ module Coverband
       def index
         notice = "<strong>Notice:</strong> #{Rack::Utils.escape_html(request.params["notice"])}<br/>"
         notice = request.params["notice"] ? notice : ""
-        Coverband::Reporters::HTMLReport.new(Coverband.configuration.store,
+        page = (request.params["page"] || 1).to_i
+        options = {
           static: false,
           base_path: base_path,
           notice: notice,
-          open_report: false).report
+          open_report: false
+        }
+        options[:page] = page if Coverband.configuration.paged_reporting
+        Coverband::Reporters::HTMLReport.new(Coverband.configuration.store,
+          options).report
       end
 
       def json
@@ -136,16 +141,19 @@ module Coverband
       end
 
       def settings
+        return "" if Coverband.configuration.hide_settings
         Coverband::Utils::HTMLFormatter.new(nil, base_path: base_path).format_settings!
       end
 
       def display_abstract_tracker(tracker)
         notice = "<strong>Notice:</strong> #{Rack::Utils.escape_html(request.params["notice"])}<br/>"
         notice = request.params["notice"] ? notice : ""
-        Coverband::Utils::HTMLFormatter.new(nil,
+        options = {
           tracker: tracker,
           notice: notice,
-          base_path: base_path).format_abstract_tracker!
+          base_path: base_path
+        }
+        Coverband::Utils::HTMLFormatter.new(nil, options).format_abstract_tracker!
       end
 
       def view_tracker_data

data/lib/coverband/utils/html_formatter.rb

@@ -117,13 +117,6 @@ module Coverband
         puts "Encoding error file:#{source_file.filename} Coverband/ERB error #{e.message}."
       end
 
-      # Returns the html to ajax load a given source_file
-      def formatted_source_file_loader(result, source_file)
-        template("source_file_loader").result(binding)
-      rescue Encoding::CompatibilityError => e
-        puts "Encoding error file:#{source_file.filename} Coverband/ERB error #{e.message}."
-      end
-
       # Returns a table containing the given source files
       def formatted_file_list(title, result, source_files, options = {})
         title_id = title.gsub(/^[^a-zA-Z]+/, "").gsub(/[^a-zA-Z0-9\-\_]/, "")
@@ -162,11 +155,11 @@ module Coverband
         Digest::SHA1.hexdigest(source_file.filename)
       end
 
-      def timeago(time)
-        if time
+      def timeago(time, err_msg = "Not Available")
+        if time.respond_to?(:iso8601)
           "<abbr class=\"timeago\" title=\"#{time.iso8601}\">#{time.iso8601}</abbr>"
         else
-          "Not Available"
+          err_msg
         end
       end
 
@@ -175,7 +168,8 @@ module Coverband
       end
 
       def link_to_source_file(source_file)
-        %(<a href="##{id source_file}" class="src_link" title="#{shortened_filename source_file}">#{shortened_filename source_file}</a>)
+        data_loader_url = "#{base_path}load_file_details?filename=#{source_file.filename}"
+        %(<a href="##{id source_file}" class="src_link" title="#{shortened_filename source_file}" data-loader-url="#{data_loader_url}" onclick="src_link_click(this)">#{shortened_filename source_file}</a>)
       end
     end
   end

data/lib/coverband/utils/results.rb

@@ -44,6 +44,8 @@ module Coverband
         eager_file = get_eager_file(source_file)
         runtime_file = get_runtime_file(source_file)
 
+        return 0 unless runtime_file
+
         return runtime_file.covered_lines_count unless eager_file
 
         eager_file.relevant_lines - eager_file.covered_lines_count

data/lib/coverband/utils/source_file.rb

@@ -25,13 +25,15 @@ module Coverband
         attr_reader :coverage
         # Whether this line was skipped
         attr_reader :skipped
+        # The coverage data posted time for this line: either nil (never), nil (missed) or Time instance (last posted)
+        attr_reader :coverage_posted
 
         # Lets grab some fancy aliases, shall we?
         alias source src
         alias line line_number
         alias number line_number
 
-        def initialize(src, line_number, coverage)
+        def initialize(src, line_number, coverage, coverage_posted = nil)
           raise ArgumentError, "Only String accepted for source" unless src.is_a?(String)
           raise ArgumentError, "Only Integer accepted for line_number" unless line_number.is_a?(Integer)
           raise ArgumentError, "Only Integer and nil accepted for coverage" unless coverage.is_a?(Integer) || coverage.nil?
@@ -40,6 +42,7 @@ module Coverband
           @line_number = line_number
           @coverage = coverage
           @skipped = false
+          @coverage_posted = coverage_posted
         end
 
         # Returns true if this is a line that should have been covered, but was not
@@ -82,6 +85,8 @@ module Coverband
       attr_reader :filename
       # The array of coverage data received from the Coverage.result
       attr_reader :coverage
+      # The array of coverage timedata received from the Coverage.result
+      attr_reader :coverage_posted
 
       # the date this version of the file first started to record coverage
       attr_reader :first_updated_at
@@ -96,6 +101,7 @@ module Coverband
         @runtime_relavant_lines = nil
         if file_data.is_a?(Hash)
           @coverage = file_data["data"]
+          @coverage_posted = file_data["timedata"] || [] # NOTE: only implement timedata for HashRedisStore
           @first_updated_at = @last_updated_at = NOT_AVAILABLE
           @first_updated_at = Time.at(file_data["first_updated_at"]) if file_data["first_updated_at"]
           @last_updated_at = Time.at(file_data["last_updated_at"]) if file_data["last_updated_at"]
@@ -139,7 +145,12 @@ module Coverband
         coverage_exceeding_source_warn if coverage.size > src.size
 
         lines = src.map.with_index(1) { |src, i|
-          Coverband::Utils::SourceFile::Line.new(src, i, never_loaded ? 0 : coverage[i - 1])
+          Coverband::Utils::SourceFile::Line.new(
+            src,
+            i,
+            never_loaded ? 0 : coverage[i - 1],
+            (never_loaded || !coverage_posted.is_a?(Array)) ? nil : coverage_posted[i - 1]
+          )
         }
 
         process_skipped_lines(lines)
@@ -200,6 +211,10 @@ module Coverband
         lines[index]&.coverage
       end
 
+      def line_coverage_posted(index)
+        lines[index]&.coverage_posted
+      end
+
       # Returns all lines that should have been, but were not covered
       # as instances of SimpleCov::SourceFile::Line
       def missed_lines

data/lib/coverband/version.rb

@@ -5,5 +5,5 @@
 # use format "4.2.1.rc.1" ~> 4.2.1.rc to prerelease versions like v4.2.1.rc.2 and v4.2.1.rc.3
 ###
 module Coverband
-  VERSION = "6.0.3.rc.3"
+  VERSION = "6.1.0"
 end

data/lib/coverband.rb

@@ -91,8 +91,8 @@ module Coverband
     coverage_instance.eager_loading!
   end
 
-  def self.eager_loading_coverage(&block)
-    coverage_instance.eager_loading(&block)
+  def self.eager_loading_coverage(...)
+    coverage_instance.eager_loading(...)
   end
 
   def self.runtime_coverage!
@@ -130,7 +130,6 @@ module Coverband
       ###
       def initialize
         require "coverband/reporters/web"
-        require "coverband/reporters/web_pager"
         require "coverband/utils/html_formatter"
         require "coverband/utils/result"
         require "coverband/utils/file_list"
@@ -148,19 +147,4 @@ module Coverband
       end
     end
   end
-
-  module Reporters
-    class WebPager < Web
-      def initialize
-        require "coverband/reporters/web"
-        require "coverband/reporters/web_pager"
-        super
-      end
-
-      def self.call(env)
-        @app ||= new
-        @app.call(env)
-      end
-    end
-  end
 end

data/lua/lib/persist-coverage.lua

@@ -21,6 +21,9 @@ for _, file_data in ipairs(files_data) do
   redis.call('HSETNX', hash_key, 'first_updated_at', first_updated_at)
   for line, coverage in pairs(file_data.coverage) do
     redis.call("HINCRBY", hash_key, line, coverage)
+    if coverage > 0 then
+      redis.call("HSET", hash_key, line .. "_last_posted", ARGV[1])
+    end
   end
   if ttl and ttl ~= cjson.null then
     redis.call("EXPIRE", hash_key, ttl)

data/public/application.css

@@ -687,9 +687,17 @@ a.src_link {
   background: url("./magnify.png") no-repeat left 50%;
   padding-left: 18px; }
 
+a.src_link.strong {
+  font-weight: bold;
+}
+
 .red a.src_link {
     color: #990000;
-   }
+}
+
+a.src_link.red {
+  color: #990000;
+}
 
 tr, td {
   margin: 0;
@@ -817,6 +825,21 @@ td {
 .red {
   color: #990000; }
 
+span.red {
+    color: #990000; }
+
+td:has(> span.red) { 
+  color: #990000;
+}
+
+td:has(> span.yellow) { 
+  color: #ddaa00;
+}
+
+td:has(> span.green) { 
+  color: #009900;
+}
+
 .yellow {
   color: #ddaa00; }
 

data/public/application.js

@@ -16,7 +16,7 @@ $(document).ready(function() {
   });
 
   // Configuration for fancy sortable tables for source file groups
-  $(".file_list").dataTable({
+  var tableOptions = {
     aaSorting: [[1, "asc"]],
     bPaginate: false,
     bJQueryUI: true,
@@ -30,19 +30,26 @@ $(document).ready(function() {
       null,
       null,
       null
-    ]
-  });
+    ],
+  }
+
+  $(".file_list").dataTable(tableOptions);
 
-  // TODO: add support for searching...
-  // hmm should I just use manual paging? or load more...
+  // TODO: add support for searching on server side
   // best docs on our version of datatables 1.7 https://datatables.net/beta/1.7/examples/server_side/server_side.html
-  // TODO: fix bug where we hardcoded /coverage we need to pull it from the path it is mounted on
   if ($(".file_list.unsorted").length == 1) {
-    var current_rows = 0;
+    $(".dataTables_empty").html("loading...");
     var total_rows = 0;
     var page = 1;
+    var all_data = [];
+    
+    // load and render page content before we start the loop
+    // perhaps move this into a datatable ready event
+    $(".dataTables_empty").html("loading...");
+    setTimeout(() => {
+      get_page(page);
+    }, 1200);
 
-    // write a function to get a page of data and add it to the table
     function get_page(page) {
       $.ajax({
         url: `${$(".file_list").data("coverageurl")}/report_json?page=${page}`,
@@ -50,96 +57,56 @@ $(document).ready(function() {
         dataType: 'json',
         success: function(data) {
           total_rows = data["iTotalRecords"];
-          // NOTE: we request 250 at a time, but we seem to have some files that we have as a list but 0 coverage,
-          // so we don't get back 250 per page... to ensure we we need to account for filtered out and empty files
-          // this 250 at the moment is synced to the 250 in the hash redis store
-          current_rows += 250; //data["aaData"].length;
-          console.log(current_rows);
-          console.log(total_rows);
-          $(".file_list.unsorted").dataTable().fnAddData(data["aaData"]);
+          all_data = all_data.concat(data["aaData"]);
+          $(".dataTables_empty").html("loading... on " + all_data.length + " of " + total_rows + " files");
           page += 1;
+;
           // the page less than 100 is to stop infinite loop in case of folks never clearing out old coverage reports
-          if (page < 100 && current_rows < total_rows) {
-            get_page(page);
+          if (page < 50 && all_data.length < total_rows) {
+            setTimeout(() => {
+              get_page(page);
+            }, 10);
+          } else {
+            $(".file_list.unsorted").dataTable().fnAddData(all_data);
+            // allow rendering to complete before we click the anchor
+            setTimeout(() => {
+              if (window.auto_click_anchor && $(window.auto_click_anchor).length > 0) {
+                $(window.auto_click_anchor).click();
+              }
+            }, 50)
           }
         }
       });
     }
-    get_page(page);
   }
 
-
-  // Syntax highlight all files up front - deactivated
-  // $('.source_table pre code').each(function(i, e) {hljs.highlightBlock(e, '  ')});
   src_link_click = (trigger_element) => {
-      // Get the source file element that corresponds to the clicked element
-      var source_table = $(".shared_source_table");
       var loader_url = $(trigger_element).attr("data-loader-url");
+      auto_click_anchor = null;
       $(trigger_element).colorbox(jQuery.extend(colorbox_options, { href: loader_url}));
-  
-      // If not highlighted yet, do it!
-      if (!source_table.hasClass("highlighted")) {
-        source_table.find("pre code").each(function(i, e) {
-          hljs.highlightBlock(e, "  ");
-        });
-        source_table.addClass("highlighted");
-      }
     };
-  window.src_link_click = src_link_click;
 
-  // Syntax highlight source files on first toggle of the file view popup
-  $("a.src_link").click(src_link_click(this));
-
-  var prev_anchor;
-  var curr_anchor;
   var colorbox_options = {
-    open: true,
     transition: "none",
-    // inline: true,
     opacity: 1,
     width: "95%",
     height: "95%",
     onLoad: function() {
-      // TODO: move source highlighting here
-      prev_anchor = curr_anchor ? curr_anchor : jQuery.url.attr("anchor");
-      curr_anchor = this.href.split("#")[1];
-      window.location.hash = curr_anchor;
+      // If not highlighted yet, do it!
+      var source_table = $(".shared_source_table");
+      if (!source_table.hasClass("highlighted")) {
+        source_table.find("pre code").each(function(i, e) {
+          hljs.highlightBlock(e, "  ");
+        });
+        source_table.addClass("highlighted");
+      }
+      window.location.hash = this.href.split("#")[1];
     },
     onCleanup: function() {
-      if (prev_anchor && prev_anchor != curr_anchor) {
-        $('a[href="#' + prev_anchor + '"]').click();
-        curr_anchor = prev_anchor;
-      } else {
-        $(".group_tabs a:first").click();
-        prev_anchor = curr_anchor;
-        curr_anchor = $(".group_tabs a:first").attr("href");
-      }
-      window.location.hash = curr_anchor;
+      window.location.hash = $(".group_tabs a:first").attr("href");
     }
   }
 
-  src_link_colorbox = (trigger_element) => {
-    $(trigger_element).colorbox(colorbox_options);
-  };
-  window.src_link_colorbox = src_link_colorbox;
-
-  // Set-up of popup for source file views
-  // TODO: drop the static source view even for not paged coverband, then delete all this
-  $("a.src_link").colorbox(colorbox_options);
-
-  window.onpopstate = function(event) {
-    if (location.hash.substring(0, 2) == "#_") {
-      $.colorbox.close();
-      curr_anchor = jQuery.url.attr("anchor");
-    } else {
-      if ($("#colorbox").is(":hidden")) {
-        console.log("pop");
-        // $('a.src_link[href="' + location.hash + '"]').colorbox({ open: true });
-        $('.shared_source_table').colorbox({ open: true });
-      }
-    }
-  };
-
   // Hide src files and file list container after load
   $(".source_files").hide();
   $(".file_list_container").hide();
@@ -157,15 +124,20 @@ $(document).ready(function() {
       .find(".covered_percent")
       .first()
       .html();
+    if (covered_percent) {
+      covered_percent = "(" + covered_percent + ")";
+    } else {
+      covered_percent = "";
+    }
 
     $(".group_tabs").append(
       '<li><a href="#' +
         container_id +
         '">' +
         group_name +
-        " (" +
+        " " +
         covered_percent +
-        ")</a></li>"
+        "</a></li>"
     );
   });
 
@@ -197,15 +169,26 @@ $(document).ready(function() {
         .addClass("active");
     }
     $(".file_list_container").hide();
-    $(".file_list_container" + $(this).attr("href")).show();
-    window.location.href =
-      window.location.href.split("#")[0] +
-      $(this)
-        .attr("href")
-        .replace("#", "#_");
-
+    $(".file_list_container" + $(this).attr("href")).show(function() {
+      // If we have an anchor to click, click it
+      // allow rendering to complete before we click the anchor
+      setTimeout(() => {
+        if (window.auto_click_anchor && $(window.auto_click_anchor).length > 0) {
+          $(window.auto_click_anchor).click();
+        }
+      }, 30);
+    });
+    // Below the #_ is a hack to show we have processed the hash change
+    if (!window.auto_click_anchor) {
+      window.location.href =
+        window.location.href.split("#")[0] +
+        $(this)
+          .attr("href")
+          .replace("#", "#_");
+    }
+  
     // Force favicon reload - otherwise the location change containing anchor would drop the favicon...
-    // Works only on firefox, but still... - Anyone know a better solution to force favicon on local file?
+    // Works only on firefox, but still... - Anyone know a better solution to force favicon on local relative file path?
     $('link[rel="shortcut icon"]').remove();
     $("head").append(
       '<link rel="shortcut icon" type="image/png" href="' +
@@ -215,18 +198,21 @@ $(document).ready(function() {
     return false;
   });
 
+  // The below function handles turning initial anchors in links to navigate to correct tab
   if (jQuery.url.attr("anchor")) {
     var anchor = jQuery.url.attr("anchor");
-    // source file hash
     if (anchor.length == 40) {
-      console.log("I need to fix deep links to source, the click call wont work anymore");
-      // $("a.src_link[href=#" + anchor + "]").click();
+      // handles deep links to source files
+      window.auto_click_anchor = "a.src_link[href=#" + anchor + "]";
+      $(".group_tabs a:first").click();
     } else {
+      // handles a # anchor that needs to be processed into a #_ completed action
       if ($(".group_tabs a." + anchor.replace("_", "")).length > 0) {
         $(".group_tabs a." + anchor.replace("_", "")).click();
       }
     }
   } else {
+    // No anchor, so click the first navigation tab
     $(".group_tabs a:first").click();
   }
 

data/public/dependencies.js

@@ -88,7 +88,7 @@ var hljs=new function(){function l(o){return o.replace(/&/gm,"&amp;").replace(/<
             xhrError: "This content failed to load.",
             imgError: "This image failed to load.",
 
-            // accessbility
+            // accessibility
             returnFocus: true,
             trapFocus: true,
 
@@ -103,7 +103,7 @@ var hljs=new function(){function l(o){return o.replace(/&/gm,"&amp;").replace(/<
                 return this.rel;
             },
             href: function() {
-                // using this.href would give the absolute url, when the href may have been intedned as a selector (e.g. '#container')
+                // using this.href would give the absolute url, when the href may have been intended as a selector (e.g. '#container')
                 return $(this).attr('href');
             },
             title: function() {
@@ -145,7 +145,7 @@ var hljs=new function(){function l(o){return o.replace(/&/gm,"&amp;").replace(/<
         $prev,
         $close,
         $groupControls,
-        $events = $('<a/>'), // $({}) would be prefered, but there is an issue with jQuery 1.4.2
+        $events = $('<a/>'), // $({}) would be preferred, but there is an issue with jQuery 1.4.2
 
     // Variables for cached values or use across multiple functions
         settings,

data/test/coverband/adapters/hash_redis_store_test.rb

@@ -57,7 +57,8 @@ class HashRedisStoreTest < Minitest::Test
         "first_updated_at" => yesterday.to_i,
         "last_updated_at" => yesterday.to_i,
         "file_hash" => "abcd",
-        "data" => [0, 1, 2]
+        "data" => [0, 1, 2],
+        "timedata" => [nil, Time.at(yesterday.to_i), Time.at(yesterday.to_i)]
       },
       @store.coverage["./dog.rb"]
     )
@@ -65,15 +66,12 @@ class HashRedisStoreTest < Minitest::Test
     @store.save_report(
       "app_path/dog.rb" => [1, 1, 0]
     )
-    assert_equal(
-      {
-        "first_updated_at" => yesterday.to_i,
-        "last_updated_at" => today.to_i,
-        "file_hash" => "abcd",
-        "data" => [1, 2, 2]
-      },
-      @store.coverage["./dog.rb"]
-    )
+
+    assert_equal("abcd", @store.coverage["./dog.rb"]["file_hash"])
+    assert_equal(today.to_i, @store.coverage["./dog.rb"]["last_updated_at"])
+    assert_equal(yesterday.to_i, @store.coverage["./dog.rb"]["first_updated_at"])
+    assert_equal([1, 2, 2], @store.coverage["./dog.rb"]["data"])
+    assert_equal([Time.at(today.to_i), Time.at(today.to_i), Time.at(yesterday.to_i)], @store.coverage["./dog.rb"]["timedata"])
   end
 
   def test_ttl_set
@@ -109,7 +107,8 @@ class HashRedisStoreTest < Minitest::Test
         "first_updated_at" => current_time.to_i,
         "last_updated_at" => current_time.to_i,
         "file_hash" => "abcd",
-        "data" => [0, nil, 1, 2]
+        "data" => [0, nil, 1, 2],
+        "timedata" => [nil, nil, Time.at(current_time.to_i), Time.at(current_time.to_i)]
       }, @store.coverage["./dog.rb"]
     )
     assert_equal [1, 2, 0, 1, 5], @store.coverage["./cat.rb"]["data"]
@@ -213,7 +212,8 @@ class HashRedisStoreTest < Minitest::Test
         "first_updated_at" => yesterday.to_i,
         "last_updated_at" => yesterday.to_i,
         "file_hash" => "abcd",
-        "data" => [0, 1, 2]
+        "data" => [0, 1, 2],
+        "timedata" => [nil, Time.at(yesterday.to_i), Time.at(yesterday.to_i)]
       },
       @store.coverage["./dog.rb"]
     )
@@ -225,7 +225,8 @@ class HashRedisStoreTest < Minitest::Test
         "first_updated_at" => yesterday.to_i,
         "last_updated_at" => yesterday.to_i,
         "file_hash" => "abcd",
-        "data" => [0, 1, 2]
+        "data" => [0, 1, 2],
+        "timedata" => [nil, Time.at(yesterday.to_i), Time.at(yesterday.to_i)]
       },
       @store.coverage["./dog.rb"]
     )
@@ -235,7 +236,8 @@ class HashRedisStoreTest < Minitest::Test
         "first_updated_at" => yesterday.to_i,
         "last_updated_at" => yesterday.to_i,
         "file_hash" => "abcd",
-        "data" => [0, 2, 4]
+        "data" => [0, 2, 4],
+        "timedata" => [nil, Time.at(yesterday.to_i), Time.at(yesterday.to_i)]
       },
       @store.coverage["./dog.rb"]
     )

data/test/coverband/collectors/route_tracker_test.rb

@@ -33,14 +33,32 @@ class RouterTrackerTest < Minitest::Test
     tracker = Coverband::Collectors::RouteTracker.new(store: store, roots: "dir")
 
     payload = {
-      request: Payload.new("path", "GET")
+      request: Payload.new("path", "GET"),
+      status: 302,
+      location: "https://coverband.dev/"
     }
     tracker.track_key(payload)
     tracker.save_report
     assert_equal [route_hash], tracker.logged_keys
   end
 
-  test "track controller routes" do
+  test "track redirect routes when track_redirect_routes is false" do
+    Coverband.configuration.track_redirect_routes = false
+
+    store = fake_store
+    tracker = Coverband::Collectors::RouteTracker.new(store: store, roots: "dir")
+
+    payload = {
+      request: Payload.new("path", "GET"),
+      status: 302,
+      location: "https://coverband.dev/"
+    }
+    tracker.track_key(payload)
+    tracker.save_report
+    assert_equal [], tracker.logged_keys
+  end
+
+  test "track controller routes in Rails < 6.1" do
     store = fake_store
     route_hash = {controller: "some/controller", action: "index", url_path: nil, verb: "GET"}
     store.raw_store.expects(:hset).with(tracker_key, route_hash.to_s, anything)
@@ -57,6 +75,27 @@ class RouterTrackerTest < Minitest::Test
     assert_equal [route_hash], tracker.logged_keys
   end
 
+  test "track controller routes in Rails >= 6.1" do
+    store = fake_store
+    route_hash = {controller: "some/controller", action: "index", url_path: nil, verb: "GET"}
+    store.raw_store.expects(:hset).with(tracker_key, route_hash.to_s, anything)
+    tracker = Coverband::Collectors::RouteTracker.new(store: store, roots: "dir")
+    payload = {
+      params: {
+        "controller" => "some/controller",
+        "action" => "index"
+      },
+      controller: "SomeController",
+      action: "index",
+      path: "path",
+      method: "GET",
+      request: Payload.new("path", "GET")
+    }
+    tracker.track_key(payload)
+    tracker.save_report
+    assert_equal [route_hash], tracker.logged_keys
+  end
+
   test "report used routes" do
     store = fake_store
     route_hash = {controller: "some/controller", action: "index", url_path: nil, verb: "GET"}

data/test/coverband/reporters/json_test.rb

@@ -38,7 +38,7 @@ class ReportJSONTest < Minitest::Test
     json = Coverband::Reporters::JSONReport.new(@store).report
     parsed = JSON.parse(json)
 
-    expected_keys = ["never_loaded", "runtime_percentage", "lines_of_code", "lines_covered", "lines_runtime", "lines_missed", "covered_percent", "covered_strength"]
+    expected_keys = ["filename", "hash", "never_loaded", "runtime_percentage", "lines_of_code", "lines_covered", "lines_runtime", "lines_missed", "covered_percent", "covered_strength"]
 
     assert_equal parsed["files"].length, 2
     parsed["files"].keys.each do |file|