single_cov 1.9.1 → 1.10.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 033c6118fe2eb88114dab333f21c65b789ae0841b52ecfc5d7e96ce086f1c36d
-  data.tar.gz: f6dbb0e79ccd6764efa21f943d1e10e31b84ba7a1bff80f4af7d92dcf3b7ea42
+  metadata.gz: 8d35277b97cbcabd001c11de169153d1eda320dfcfbd7ad516406ee811e623eb
+  data.tar.gz: c5d9fb7c34b7cb1a67a772a8ce870dbfc40cc6b1ae9ee859a951bbc3cbf93120
 SHA512:
-  metadata.gz: 939772cc7264d4ab62255b608b9ef6d493c75c6ed0e96f7e1c1d5f97e967a79bc93624c6449a5cffb5973e527a5ce9dae204e01b336e8ecb22dbea9779fd19d1
-  data.tar.gz: 8d101d81129a40851a00b9f7157d1e1424c0845f3f0185c64fe890b08c3f453dc11dbcd756275768d286a6817e1483205cabf18683428ecbdb884cd4877118d7
+  metadata.gz: 64b8de808afd738fae4817c0701c12905882a4122ac50b53cc3e13694f78a831bcbaa433a7e45115682ef3ea2beefd7b59d2442872ad2ce11a82999007e3c6c7
+  data.tar.gz: 7637465f16cadc6a8beec088abd9fff899a3d9052617dd9c89d786ee24e565cb6fa8a8508a69407d63c013088aa11a3b765ac152198fc395460a490536525295

data/lib/single_cov/version.rb

@@ -1,4 +1,4 @@
 # frozen_string_literal: true
 module SingleCov
-  VERSION = "1.9.1"
+  VERSION = "1.10.0"
 end

data/lib/single_cov.rb

@@ -40,7 +40,7 @@ module SingleCov
         # ignore lines that are marked as uncovered via comments
         # TODO: warn when using uncovered but the section is indeed covered
         content = File.readlines("#{root}/#{file}")
-        uncovered.reject! do |line_start, _, _, _|
+        uncovered.reject! do |line_start, _, _, _, _|
           content[line_start - 1].match?(UNCOVERED_COMMENT_MARKER)
         end
         next if uncovered.size == expected_uncovered
@@ -146,18 +146,13 @@ module SingleCov
     def uncovered(coverage)
       return coverage unless coverage.is_a?(Hash) # just lines
 
-      # [nil, 1, 0, 1, 0] -> [3, 5]
-      uncovered_lines = coverage.fetch(:lines)
-        .each_with_index
-        .select { |c, _| c == 0 }
-        .map { |_, i| i + 1 }
-        .compact
-
+      uncovered_lines = indexes(coverage.fetch(:lines), 0).map! { |i| i + 1 }
       uncovered_branches = uncovered_branches(coverage[:branches] || {})
-      uncovered_branches.reject! { |k| uncovered_lines.include?(k[0]) } # remove duplicates
+      uncovered_branches.reject! { |br| uncovered_lines.include?(br[0]) } # ignore branch when whole line is uncovered
 
+      # combine lines and branches while keeping them sorted
       all = uncovered_lines.concat uncovered_branches
-      all.sort_by! { |line_start, char_start, _, _| [line_start, char_start || 0] } # branches are unsorted
+      all.sort_by! { |line_start, char_start, _, _, _| [line_start, char_start || 0] } # branches are unsorted
       all
     end
 
@@ -174,19 +169,21 @@ module SingleCov
       (!defined?(@main_process_pid) || @main_process_pid == Process.pid)
     end
 
+    # {[branch_id] => {[branch_part] => coverage}} --> uncovered location
     def uncovered_branches(coverage)
-      # {[branch_id] => {[branch_part] => coverage}} --> {branch_part -> sum-of-coverage}
-      sum = Hash.new(0)
+      sum = {}
       coverage.each_value do |branch|
-        branch.each do |k, v|
-          sum[k.slice(2, 4)] += v
+        branch.filter_map do |part, c|
+          location = [part[2], part[3] + 1, part[4], part[5] + 1] # locations can be duplicated
+          type = part[0]
+          info = (sum[location] ||= [0, nil])
+          info[0] += c
+          info[1] = type if type == :else # only else is important to track since it often is not in the code
         end
       end
 
-      sum.select! { |_, v| v == 0 } # keep missing coverage
-      found = sum.map { |k, _| [k[0], k[1] + 1, k[2], k[3] + 1] }
-      found.uniq!
-      found
+      # keep location and type of missing coverage
+      sum.filter_map { |k, v| k + [v[1]] if v[0] == 0 }
     end
 
     def default_tests
@@ -197,6 +194,10 @@ module SingleCov
       Dir["#{root}/#{pattern}"].map! { |f| f.sub("#{root}/", '') }
     end
 
+    def indexes(list, find)
+      list.each_with_index.filter_map { |v, i| i if v == find }
+    end
+
     # do not ask for coverage when SimpleCov already does or it conflicts
     def coverage_results
       if defined?(SimpleCov) && (result = SimpleCov.instance_variable_get(:@result))
@@ -327,13 +328,13 @@ module SingleCov
         [
           "#{file} new uncovered lines introduced #{details}",
           red("Lines missing coverage:"),
-          *uncovered.map do |line_start, char_start, line_end, char_end|
+          *uncovered.map do |line_start, char_start, line_end, char_end, type|
             if char_start # branch coverage
               if line_start == line_end
                 "#{file}:#{line_start}:#{char_start}-#{char_end}"
               else # possibly unreachable since branches always seem to be on the same line
                 "#{file}:#{line_start}:#{char_start}-#{line_end}:#{char_end}"
-              end
+              end + (type ? " # #{type}" : "")
             else
               "#{file}:#{line_start}"
             end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: single_cov
 version: !ruby/object:Gem::Version
-  version: 1.9.1
+  version: 1.10.0
 platform: ruby
 authors:
 - Michael Grosser
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-03-15 00:00:00.000000000 Z
+date: 2023-05-20 00:00:00.000000000 Z
 dependencies: []
 description:
 email: michael@grosser.it