coverband 6.1.1 → 6.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ed2b063179cbc773c9b3b8fab3403820f9855c32a9de007c347b58873eecf13a
-  data.tar.gz: 7968b26ac22e3209b0bf2a9baabdfe107ed9029a0d4557eea5b80c22439e3278
+  metadata.gz: f23312d78597d5a4ccd34a6bd5284f384430ec78a4030ea4cd14e91a54d0226c
+  data.tar.gz: d52ca89becf12d1f2078ce951feda318415494fc2f1fe18a67b3b2514929b91a
 SHA512:
-  metadata.gz: 3bccf2dbc8e92d5980365a99b5ae2852fd8e04d224202a58e3a8490991b88eb88a881fbf620e4ea60d66cd4714a70f11a8015ebd96aa772a65a35e1fb642ea24
-  data.tar.gz: 784f5e0905989f865199261270072c397e520b2269ed387f0ecd1276ef5a7395fc5d4de7ec9990aceeda1cc883778bcfcf4c7c2260c792e81557914298778d87
+  metadata.gz: 2cdc25f77daf7adaf94b8010626b4292492dc8660bdd7ab062237cc1fa2eda9f145baaa91e20f18c4e64255285ecc0c471cb6a9d02e073184bcb4857ed342ec2
+  data.tar.gz: 8f7ea9062243e7e7d4fc304ade85e7fe21f4dc93c55140a0640242970af2638374626121f78aa4b07ae0f7ce47af8659d936a0a76b7029d8685d35a5effff21d

data/changes.md

@@ -1,3 +1,7 @@
+### Coverband 6.1.2
+
+* Fix for paging that would pull empty pages after getting all the data.
+
 ### Coverband 6.1.1
 
 * Performance fix making paged report loading 10X faster

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.1.1"
+  VERSION = "6.1.2"
 end

data/public/application.js

@@ -1,4 +1,4 @@
-$(document).ready(function() {
+$(document).ready(function () {
   // remove the url params like notice=message so they don't stick around
   window.history.replaceState(
     "object or string",
@@ -9,7 +9,7 @@ $(document).ready(function() {
     .delay(3000)
     .fadeOut("slow");
 
-  $(".del").click(function() {
+  $(".del").click(function () {
     if (!confirm("Do you want to delete")) {
       return false;
     }
@@ -42,7 +42,7 @@ $(document).ready(function() {
     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...");
@@ -55,14 +55,14 @@ $(document).ready(function() {
         url: `${$(".file_list").data("coverageurl")}/report_json?page=${page}`,
         type: 'GET',
         dataType: 'json',
-        success: function(data) {
+        success: function (data) {
           total_rows = data["iTotalRecords"];
           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 < 50 && all_data.length < total_rows) {
+
+          // the page less than 50 is to stop infinite loop in case of folks never clearing out old coverage reports
+          if (page < 50 && all_data.length <= total_rows && data["aaData"].length > 0) {
             setTimeout(() => {
               get_page(page);
             }, 10);
@@ -81,28 +81,28 @@ $(document).ready(function() {
   }
 
   src_link_click = (trigger_element) => {
-      var loader_url = $(trigger_element).attr("data-loader-url");
-      auto_click_anchor = null;
-      $(trigger_element).colorbox(jQuery.extend(colorbox_options, { href: loader_url}));
-    };
+    var loader_url = $(trigger_element).attr("data-loader-url");
+    auto_click_anchor = null;
+    $(trigger_element).colorbox(jQuery.extend(colorbox_options, { href: loader_url }));
+  };
 
   var colorbox_options = {
     transition: "none",
     opacity: 1,
     width: "95%",
     height: "95%",
-    onLoad: function() {
+    onLoad: function () {
       // 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) {
+        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() {
+    onCleanup: function () {
       window.location.hash = $(".group_tabs a:first").attr("href");
     }
   }
@@ -112,7 +112,7 @@ $(document).ready(function() {
   $(".file_list_container").hide();
 
   // Add tabs based upon existing file_list_containers
-  $(".file_list_container h2").each(function() {
+  $(".file_list_container h2").each(function () {
     var container_id = $(this)
       .parent()
       .attr("id");
@@ -132,16 +132,16 @@ $(document).ready(function() {
 
     $(".group_tabs").append(
       '<li><a href="#' +
-        container_id +
-        '">' +
-        group_name +
-        " " +
-        covered_percent +
-        "</a></li>"
+      container_id +
+      '">' +
+      group_name +
+      " " +
+      covered_percent +
+      "</a></li>"
     );
   });
 
-  $(".group_tabs a").each(function() {
+  $(".group_tabs a").each(function () {
     $(this).addClass(
       $(this)
         .attr("href")
@@ -150,12 +150,12 @@ $(document).ready(function() {
   });
 
   // Make sure tabs don't get ugly focus borders when active
-  $(".group_tabs a").live("focus", function() {
+  $(".group_tabs a").live("focus", function () {
     $(this).blur();
   });
 
   var favicon_path = $('link[rel="shortcut icon"]').attr("href");
-  $(".group_tabs a").live("click", function() {
+  $(".group_tabs a").live("click", function () {
     if (
       !$(this)
         .parent()
@@ -169,7 +169,7 @@ $(document).ready(function() {
         .addClass("active");
     }
     $(".file_list_container").hide();
-    $(".file_list_container" + $(this).attr("href")).show(function() {
+    $(".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(() => {
@@ -186,14 +186,14 @@ $(document).ready(function() {
           .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 relative file path?
     $('link[rel="shortcut icon"]').remove();
     $("head").append(
       '<link rel="shortcut icon" type="image/png" href="' +
-        favicon_path +
-        '" />'
+      favicon_path +
+      '" />'
     );
     return false;
   });

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: coverband
 version: !ruby/object:Gem::Version
-  version: 6.1.1
+  version: 6.1.2
 platform: ruby
 authors:
 - Dan Mayer
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2024-04-23 00:00:00.000000000 Z
+date: 2024-04-28 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: benchmark-ips