pronto-rustcov 0.1.6 → 0.1.9

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: bb33cf0ba6f6437b808ddc35ee00823c3cfbf03d463dc943b4255597dca768b2
-  data.tar.gz: 50f0acb9d7d0c3c240ca13e5191d7f871512d4280ad3ba92d25a4f6fd94fec8e
+  metadata.gz: 3aad469c0aeb544cb4bd49065731799cc6303a462833d143d77e9a0c805d8ef5
+  data.tar.gz: df7c0823144b3708ef7f022d7748f5b8afd72532340679f73d92c75708e454ff
 SHA512:
-  metadata.gz: 9eae0d6aa9a54c4dd6986fe8423b2a6e805ddc32af24b2321cbb6f1434af8596544eda819248313a45904585cf6bc17759f6f2e81de0f281e457c9bd3eeca0bf
-  data.tar.gz: 4d9156c24fe8183937b769a36ed7782384d4e8875794b1014106cc32d6546feddd2de5d9787190d306428810159ebb0b4bc5eefda7d8fbb108f1d566c9de4d3d
+  metadata.gz: 68ead906857eb7354e322342b9d0da8396192918275e11fdcff7bbc1f8a2e11880d2a16cdea439638c6043a439cd96f061930462bb16d7d97840f348d7570779
+  data.tar.gz: 417627cb6fa45276eec6b0507b36b54ad7e30da897a367cd8f78d14ea541d0a917231f133c22d97b93f7ce832322d6b509423a9eabc9e822205c39c9396455a4

data/lib/pronto/rustcov.rb

@@ -3,7 +3,13 @@ require 'pronto'
 module Pronto
   class Rustcov < Runner
     def run
-      one_message_per_file('target/lcov.info')
+      one_message_per_file(lcov_path)
+    end
+
+    private
+
+    def lcov_path
+      ENV['PRONTO_RUSTCOV_LCOV_PATH'] || ENV['LCOV_PATH'] || 'target/lcov.info'
     end
 
     def pronto_files_limit
@@ -19,9 +25,15 @@ module Pronto
 
       lcov = parse_lcov(lcov_path)
 
+      grouped = group_patches(@patches, lcov)
+
+      build_messages(grouped)
+    end
+
+    def group_patches(patches, lcov)
       grouped = Hash.new { |h, k| h[k] = [] }
 
-      @patches.sort_by { |patch| -patch.added_lines.count }.take(pronto_files_limit).each do |patch|
+      patches.each do |patch|
         next unless patch.added_lines.any?
         file_path = patch.new_file_full_path.to_s
         uncovered = lcov[file_path]
@@ -34,41 +46,65 @@ module Pronto
         end
       end
 
-      grouped.map do |patch, lines|
+      grouped.sort_by { |_, lines| -lines.count }.take(pronto_files_limit)
+    end
+
+    def build_messages(grouped)
+      messages = []
+
+      grouped.each do |patch, lines|
         linenos = lines.map(&:new_lineno).sort
-        ranges = linenos.chunk_while { |i, j| j == i + 1 }
-                        .take(pronto_messages_per_file_limit)
-                        .map { |group| group.size > 1 ? "#{group.first}–#{group.last}" : group.first.to_s }
-
-        message_text = "⚠️ Test coverage is missing for lines: #{ranges.join(', ')}"
-
-        # Attach the message to the first uncovered line
-        Pronto::Message.new(
-          patch.new_file_path,
-          lines.first,
-          :warning,
-          message_text,
-          nil,
-          self.class
-        )
+        line_ranges = linenos.chunk_while { |i, j| j == i + 1 }.to_a
+
+        best_ranges = line_ranges.sort_by { |range| [-range.size, range.first] }.take(pronto_messages_per_file_limit)
+
+        # If we have a message per file limit of N, then create N individual messages
+        # We'll take each range and create a separate message for it, up to the limit
+        best_ranges.each do |range|
+          message_text = format_message_text(range)
+
+          # Find the first line in this range for the message
+          first_line_in_range = lines.find { |line| line.new_lineno == range.first }
+
+          messages << Pronto::Message.new(
+            patch.new_file_path,
+            first_line_in_range,
+            :warning,
+            message_text,
+            nil,
+            self.class
+          )
+        end
       end
+
+      messages
     end
 
-    private
+    def format_message_text(range)
+      # Format the range as "start–end" or just the number if it's a single line
+      formatted_range = range.size > 1 ? "#{range.first}–#{range.last}" : range.first.to_s
+
+      "⚠️ Test coverage is missing for lines: #{formatted_range}"
+    end
 
     def parse_lcov(path)
       uncovered = Hash.new { |h, k| h[k] = [] }
       file = nil
 
-      File.foreach(path) do |line|
-        case line
-        when /^SF:(.+)/
-          file = File.expand_path($1.strip)
-        when /^DA:(\d+),0$/
-          uncovered[file] << $1.to_i if file
-        when /^end_of_record/
-          file = nil
+      begin
+        File.foreach(path) do |line|
+          case line
+          when /^SF:(.+)/
+            file = File.expand_path($1.strip)
+          when /^DA:(\d+),0$/
+            uncovered[file] << $1.to_i if file
+          when /^end_of_record/
+            file = nil
+          end
         end
+      rescue Errno::ENOENT
+        # File not found, raise a more informative error
+        fail "LCOV file not found at #{path}. Make sure your Rust tests were run with coverage enabled."
       end
 
       uncovered

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: pronto-rustcov
 version: !ruby/object:Gem::Version
-  version: 0.1.6
+  version: 0.1.9
 platform: ruby
 authors:
 - Pavel Lazureykis
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2025-05-14 00:00:00.000000000 Z
+date: 2025-05-15 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: pronto
@@ -24,6 +24,48 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: 0.11.0
+- !ruby/object:Gem::Dependency
+  name: rspec
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '3.12'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '3.12'
+- !ruby/object:Gem::Dependency
+  name: rake
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '13.0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '13.0'
+- !ruby/object:Gem::Dependency
+  name: simplecov
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 0.22.0
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 0.22.0
 description: This gem integrates Rust test coverage with Pronto via LCOV.
 email:
 - pavel@lazureykis.dev
@@ -44,7 +86,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '2.7'
+      version: '3.2'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="