pronto-rustcov 0.1.5 → 0.1.7

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 4b90415468e5df17f944a01aaf70d5cfcedf235d6219cb6decfd7444fd5316aa
-  data.tar.gz: 7fa74a181f664b1f2f6790a46a96f5c75ccc23439f23348bcdfb3a5f00d385dc
+  metadata.gz: 7df000545899a6dcf0010858dce8b6b70fbd490b3267ce907167289be36c746f
+  data.tar.gz: 1e7e6c583a9be88702d442f9da91921e0bb1debed9563af582f1e06d356cc575
 SHA512:
-  metadata.gz: b0723c5c9ced93c32082537a66721a769dba75bb9f54a1113213c6ab7a921872f44b219d6cbb7d6856996ea7d6f881d1381f087f8214099aeb6fbe45d7fa6473
-  data.tar.gz: edc3c3699d2c03e1421708159891f8d4011c99e2a0b31951901f02e51bb95905b7f36feb16b03ebfbd5829fdb7f9cb298b835865cfd376b1354bd5323145184f
+  metadata.gz: 9d3cc9a905b9ed10d597c8e2a414a1fe4d368c486cb80d0adb9751df1c594f745597b5edbfc0087780154df510691fb6b5de3147fdc5f06ef4eac5c1312e7478
+  data.tar.gz: a04b4aa9f19c3dd7b6bf2dd555f35129c5a89b1432997f6e1e9b0c5ada57d3a558e0a8eb570d36fc81f89fea27cb00f0c8d3aa06568228d3a170d848d3f7d4d0

data/lib/pronto/rustcov.rb

@@ -3,11 +3,21 @@ require 'pronto'
 module Pronto
   class Rustcov < Runner
     def run
-      one_message_per_file('target/lcov.info')
+      one_message_per_file(lcov_path)
     end
 
-    def messages_limit
-      ENV['PRONTO_RUSTCOV_MESSAGES_LIMIT']&.to_i || 5
+    private
+
+    def lcov_path
+      ENV['PRONTO_RUSTCOV_LCOV_PATH'] || ENV['LCOV_PATH'] || 'target/lcov.info'
+    end
+
+    def pronto_files_limit
+      ENV['PRONTO_RUSTCOV_FILES_LIMIT']&.to_i || 5
+    end
+
+    def pronto_messages_per_file_limit
+      ENV['PRONTO_RUSTCOV_MESSAGES_PER_FILE_LIMIT']&.to_i || 5
     end
 
     def one_message_per_file(lcov_path)
@@ -15,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(messages_limit).each do |patch|
+      patches.sort_by { |patch| -patch.added_lines.count }.take(pronto_files_limit).each do |patch|
         next unless patch.added_lines.any?
         file_path = patch.new_file_full_path.to_s
         uncovered = lcov[file_path]
@@ -30,40 +46,66 @@ module Pronto
         end
       end
 
-      grouped.map do |patch, lines|
+      grouped
+    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 }
-                        .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
+
+        # 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
+        line_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
+          )
+
+          # Stop if we've reached the limit of messages per file
+          break if messages.count { |m| m.path == patch.new_file_path } >= pronto_messages_per_file_limit
+        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.5
+  version: 0.1.7
 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:
   - - ">="