piggly-nsd 2.3.4 → 2.3.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 3aed72758cfe5ef63b8635586386845b2b3766a742cb398a9fc8121bd70e966f
-  data.tar.gz: 98e694a96ccff107b670de06d747b3a015a361a1e806c190e7bc6f9c36c0ec5f
+  metadata.gz: 736495bc6e9ad8785b755afb2b3a834cd0dff2e62983b0397e4d6fdf6f0fe3af
+  data.tar.gz: 958489e0a1a542230ad57dc8748ef4a231e9c03cea1452c34246887e9789746f
 SHA512:
-  metadata.gz: 889952ad1338856df2c539e199a18805236921b647add649937a9cfcec99578504915a000fd8174edeef13e485cab5cdda8a3ed03aa8e1a2273fe99bd5049235
-  data.tar.gz: 1de357aa82135b51d0c31b7515c102aa251bd489495f290aeaf71db302c2acc2bd19fee7dcc3410d7f1bf5e587b8852874994b02e61df6c03684114e5be53f9d
+  metadata.gz: ba81b385db041ea25c62a0777838d84ed89a165e489d6d4ef3f60ea908902acad3ba2a186e3eb7325a7520f407eead2ae3e6121fd8f4429664a3a9f00a848b01
+  data.tar.gz: e076e78ce56f6e8e7ed238f7fa354dfe9799b2707520394e5635162f8029359c581bf5d055b53efd0c0739b58410c59ee5cb9e94c08b7dd600ecf64f478fb7d4

data/README.md

@@ -1,4 +1,4 @@
-[![CI](https://github.com/sergeiboikov/piggly/actions/workflows/ci.yml/badge.svg)](https://github.com/sergeiboikov/piggly/actions/workflows/ci.yml)
+[![CI](https://github.com/NSDDeveloper/piggly/actions/workflows/ci.yml/badge.svg)](https://github.com/NSDDeveloper/piggly/actions/workflows/ci.yml)
 # Piggly-NSD
 
 Code coverage reports for PostgreSQL PL/pgSQL stored procedures
@@ -60,7 +60,7 @@ This fork continues the version numbering from the original piggly project:
 
 To install the latest from github:
 
-    $ git clone https://github.com/sergeiboikov/piggly.git
+    $ git clone https://github.com/NSDDeveloper/piggly.git
     $ cd piggly
     $ bundle install
     $ bundle exec rake spec
@@ -139,6 +139,20 @@ To generate a SonarQube-compatible coverage report, use the `--sonar-report-path
 
 This generates an XML file in SonarQube's [generic test coverage format](https://docs.sonarsource.com/sonarqube-server/latest/analyzing-source-code/test-coverage/generic-test-data/).
 
+### Schema CSV Report
+
+To generate a schema-level CSV report, use the `--schema-csv-path` option:
+
+    $ piggly report -f messages.txt --schema-csv-path piggly/reports/coverage_by_schema.csv
+
+The CSV report groups procedures by schema and contains the following columns:
+
+- `No`
+- `Schema Name`
+- `Objects Count`
+- `Covered Objects`
+- `Line Coverage Percent`
+
 ## Running the Examples
 
     $ cd piggly
@@ -179,4 +193,4 @@ This generates an XML file in SonarQube's [generic test coverage format](https:/
 
 ## Bugs & Issues
 
-Please report any issues or feature requests on the [github tracker](https://github.com/sergeiboikov/piggly/issues).
+Please report any issues or feature requests on the [github tracker](https://github.com/NSDDeveloper/piggly/issues).

data/lib/piggly/command/report.rb

@@ -13,7 +13,7 @@ module Piggly
     class << Report
       def main(argv)
         require "pp"
-        io, config, sonar_path, html_report_requested = configure(argv)
+        io, config, sonar_path, schema_csv_path, html_report_requested = configure(argv)
 
         profile = Profile.new
         index   = Dumper::Index.new(config)
@@ -46,9 +46,14 @@ module Piggly
         if sonar_path
           create_sonar_report(config, procedures, profile, sonar_path)
         end
-        
-        unless html_report_requested || sonar_path
-          puts "Warning: No report output specified. Use -o for HTML reports or -x for Sonar report."
+
+        # Generate schema-level CSV report if requested
+        if schema_csv_path
+          create_schema_csv_report(config, procedures, profile, schema_csv_path)
+        end
+
+        unless html_report_requested || sonar_path || schema_csv_path
+          puts "Warning: No report output specified. Use -o for HTML reports, -x for Sonar report, or --schema-csv-path for CSV report."
         end
       end
 
@@ -138,9 +143,19 @@ module Piggly
         puts "Sonar coverage report written to: #{path}"
       end
 
+      # Create schema-level CSV coverage report
+      #
+      def create_schema_csv_report(config, procedures, profile, output_path)
+        puts "creating schema CSV coverage report"
+        reporter = Reporter::SchemaCsv.new(config, profile, output_path)
+        path = reporter.report(procedures)
+        puts "Schema CSV coverage report written to: #{path}"
+      end
+
       def configure(argv, config = Config.new)
         io = $stdin
         sonar_path = nil
+        schema_csv_path = nil
         html_report_requested = false
 
         p  = OptionParser.new do |o|
@@ -155,6 +170,9 @@ module Piggly
           o.on("-x", "--sonar-report-path PATH",  "generate Sonar coverage XML report at PATH") do |path|
             sonar_path = path
           end
+          o.on("--schema-csv-path PATH", "generate schema-level CSV coverage report at PATH") do |path|
+            schema_csv_path = path
+          end
           o.on("-a", "--accumulate",        "accumulate data from the previous run", &o_accumulate(config))
           o.on("-V", "--version",           "show version", &o_version(config))
           o.on("-h", "--help",              "show this message") { abort o.to_s }
@@ -170,9 +188,9 @@ module Piggly
         begin
           p.parse! argv
 
-          unless html_report_requested || sonar_path
+          unless html_report_requested || sonar_path || schema_csv_path
             raise OptionParser::MissingArgument,
-              "at least one report type required: use -o for HTML reports or -x for Sonar report"
+              "at least one report type required: use -o for HTML reports, -x for Sonar report, or --schema-csv-path for CSV report"
           end
           
           if io.eql?($stdin) and $stdin.tty?
@@ -180,7 +198,7 @@ module Piggly
               "stdin must be a pipe, or use --input PATH"
           end
 
-          return io, config, sonar_path, html_report_requested
+          return io, config, sonar_path, schema_csv_path, html_report_requested
         rescue OptionParser::InvalidOption,
                OptionParser::InvalidArgument,
                OptionParser::MissingArgument

data/lib/piggly/compiler/coverage_report.rb

@@ -23,7 +23,7 @@ module Piggly
         data = trace.compile(procedure)
 
         return :html  => traverse(data[:tree], profile),
-               :lines => 1 .. procedure.source(@config).count("\n") + 1
+               :lines => 1 .. Util::LineNumbers.count(procedure.source(@config))
       end
 
     protected

data/lib/piggly/compiler/line_coverage.rb

@@ -92,10 +92,10 @@ module Piggly
       # @param line_number [Integer] 1-based line number
       # @return [Boolean] true if line should be excluded
       def excluded_line?(source, line_number)
-        lines = source.split("\n")
-        return true if line_number < 1 || line_number > lines.length
-        
-        line_content = lines[line_number - 1].strip.downcase
+        max_line = Util::LineNumbers.count(source)
+        return true if line_number < 1 || line_number > max_line
+
+        line_content = source.lines[line_number - 1].strip.downcase
         
         # Exclude lines containing only structural keywords or comments
         excluded_patterns = [
@@ -137,11 +137,9 @@ module Piggly
         start_pos = [start_pos, source_len - 1].min if source_len > 0
         end_pos = [end_pos, source_len - 1].min if source_len > 0
         
-        # Calculate line numbers by counting newlines
-        # Line numbers are 1-based
-        start_line = source[0...start_pos].count("\n") + 1
-        end_line = source[0..end_pos].count("\n") + 1
-        
+        start_line = Util::LineNumbers.at_offset(source, start_pos)
+        end_line = Util::LineNumbers.at_offset(source, end_pos + 1)
+
         [start_line, end_line]
       end