piggly-nsd 2.3.3 → 2.3.5

data/spec/examples/util/line_numbers_spec.rb

@@ -0,0 +1,59 @@
+require 'spec_helper'
+
+module Piggly::Util
+
+  describe LineNumbers do
+    describe ".count" do
+      it "returns 0 for empty source" do
+        expect(LineNumbers.count("")).to eq(0)
+        expect(LineNumbers.count(nil)).to eq(0)
+      end
+
+      it "counts lines without a trailing newline" do
+        expect(LineNumbers.count("a\nb\nc")).to eq(3)
+      end
+
+      it "does not over-count when source ends with a newline" do
+        expect(LineNumbers.count("a\nb\nc\n")).to eq(3)
+      end
+
+      it "counts CRLF sources consistently" do
+        expect(LineNumbers.count("a\r\nb\r\nc\r\n")).to eq(3)
+      end
+
+      it "matches String#lines for a large trailing-newline body" do
+        source = (1..366).map { |i| "line#{i}" }.join("\n") + "\n"
+        expect(LineNumbers.count(source)).to eq(366)
+        expect(LineNumbers.count(source)).to eq(source.lines.count)
+      end
+    end
+
+    describe ".at_offset" do
+      let(:source) { "a\nb\nc\n" }
+
+      it "returns 1 for empty source" do
+        expect(LineNumbers.at_offset("", 0)).to eq(1)
+      end
+
+      it "maps byte offsets to 1-based line numbers" do
+        expect(LineNumbers.at_offset(source, 0)).to eq(1)
+        expect(LineNumbers.at_offset(source, 2)).to eq(2)
+        expect(LineNumbers.at_offset(source, 4)).to eq(3)
+      end
+
+      it "clamps offsets past EOF to the last line" do
+        expect(LineNumbers.at_offset(source, source.length)).to eq(3)
+        expect(LineNumbers.at_offset(source, source.length + 10)).to eq(3)
+      end
+
+      it "does not report a line beyond count when offset is on trailing newline" do
+        source = (1..366).map { |i| "line#{i}" }.join("\n") + "\n"
+        max_line = LineNumbers.count(source)
+
+        expect(LineNumbers.at_offset(source, source.length - 1)).to eq(max_line)
+        expect(LineNumbers.at_offset(source, source.length)).to eq(max_line)
+      end
+    end
+  end
+
+end

data/spec/issues/037_spec.rb

@@ -0,0 +1,30 @@
+require "spec_helper"
+
+module Piggly
+  describe "GET CURRENT DIAGNOSTICS" do
+    include GrammarHelper
+
+    it "can parse a GET CURRENT DIAGNOSTICS statement" do
+      body = "GET CURRENT DIAGNOSTICS l_wdc_inserted := ROW_COUNT;"
+
+      node = parse(:statement, body)
+      node.should be_statement
+    end
+
+    it "can parse a procedure with GET CURRENT DIAGNOSTICS" do
+      body = <<-SQL
+      DECLARE
+        l_wdc_inserted bigint;
+      BEGIN
+        INSERT INTO foo DEFAULT VALUES;
+        GET CURRENT DIAGNOSTICS l_wdc_inserted := ROW_COUNT;
+        RETURN l_wdc_inserted;
+      END;
+      SQL
+
+      node = parse(:start, body.strip.downcase)
+      node.count { |e| e.assignment? }.should == 0
+      node.count { |e| Parser::Nodes::Return === e }.should == 1
+    end
+  end
+end

data/spec/issues/case_in_condition_spec.rb

@@ -0,0 +1,50 @@
+require "spec_helper"
+
+module Piggly
+  describe "SQL CASE in PL/pgSQL conditions" do
+    include GrammarHelper
+
+    it "can parse IF with CASE in a function argument" do
+      body = <<-SQL
+      BEGIN
+        IF 0 = public.f_test(
+              p_doc_type_mnemo => line_rec.doc_type_mnemo,
+              p_person_id => CASE
+                               WHEN l_f_sub_acc = 1 THEN p_client_id
+                               WHEN l_f_sub_acc = 0 THEN l_person_id
+                             END,
+              p_repres_id => NULL
+            ) THEN
+          NULL;
+        END IF;
+      END;
+      SQL
+
+      node = parse(:start, body.strip.downcase)
+      node.count { |e| e.if? }.should == 1
+    end
+
+    it "can parse ELSIF with CASE in a function argument" do
+      body = <<-SQL
+      BEGIN
+        IF false THEN
+          NULL;
+        ELSIF 0 = public.f_test(
+                p_person_id => CASE WHEN l_flag = 1 THEN p_a ELSE p_b END
+              ) THEN
+          NULL;
+        END IF;
+      END;
+      SQL
+
+      node = parse(:start, body.strip.downcase)
+      node.count { |e| e.if? }.should == 2
+    end
+
+    it "does not break simple IF conditions" do
+      node = parse(:statement, "IF cond THEN a := 10; END IF;")
+      node.should be_statement
+      node.count { |e| e.if? }.should == 1
+    end
+  end
+end

data/spec/issues/commit_spec.rb

@@ -0,0 +1,34 @@
+require "spec_helper"
+
+module Piggly
+  describe "standalone transaction statements" do
+    include GrammarHelper
+
+    it "can parse commit without trailing expression" do
+      node = parse(:statement, "commit;")
+      node.should be_statement
+      node.count { |e| e.sql? }.should == 1
+      node.find { |e| e.sql? }.source_text.should == "commit;"
+    end
+
+    it "can parse rollback without trailing expression" do
+      node = parse(:statement, "rollback;")
+      node.should be_statement
+      node.count { |e| e.sql? }.should == 1
+    end
+
+    it "can parse a procedure with multiple commit statements" do
+      body = <<-SQL
+      BEGIN
+        call i_schema.create_choice1(p_session_id => l_session_id);
+        commit;
+        call i_schema.create_choice2(l_session_id);
+        commit;
+      END;
+      SQL
+
+      node = parse(:start, body.strip.downcase)
+      node.count { |e| e.sql? }.should == 4
+    end
+  end
+end

data/spec/spec_helper.rb

@@ -26,7 +26,7 @@ module Piggly
 
     COMMENTS = ["abc defghi", "abc -- abc", "quote's", "a 'str'"]
 
-    SQLWORDS = %w[select insert update delete drop alter commit set start]
+    SQLWORDS = %w[select insert update delete merge drop alter commit set start]
 
     KEYWORDS = %w[as := = alias begin by constant continue
                   cursor debug declare diagnostics else elsif elseif

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: piggly-nsd
 version: !ruby/object:Gem::Version
-  version: 2.3.3
+  version: 2.3.5
 platform: ruby
 authors:
 - Kvle Putnam
@@ -37,6 +37,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: 0.18.4
+- !ruby/object:Gem::Dependency
+  name: csv
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '3.3'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '3.3'
 description: PostgreSQL PL/pgSQL stored procedure code coverage (NSD fork)
 email: putnam.kvle@gmail.com
 executables:
@@ -56,6 +70,7 @@ files:
 - lib/piggly/compiler.rb
 - lib/piggly/compiler/cache_dir.rb
 - lib/piggly/compiler/coverage_report.rb
+- lib/piggly/compiler/line_coverage.rb
 - lib/piggly/compiler/trace_compiler.rb
 - lib/piggly/config.rb
 - lib/piggly/dumper.rb
@@ -79,6 +94,8 @@ files:
 - lib/piggly/reporter/resources/highlight.js
 - lib/piggly/reporter/resources/piggly.css
 - lib/piggly/reporter/resources/sortable.js
+- lib/piggly/reporter/schema_csv.rb
+- lib/piggly/reporter/sonar.rb
 - lib/piggly/tags.rb
 - lib/piggly/task.rb
 - lib/piggly/util.rb
@@ -86,6 +103,7 @@ files:
 - lib/piggly/util/cacheable.rb
 - lib/piggly/util/enumerable.rb
 - lib/piggly/util/file.rb
+- lib/piggly/util/line_numbers.rb
 - lib/piggly/util/process_queue.rb
 - lib/piggly/util/thunk.rb
 - lib/piggly/version.rb
@@ -121,12 +139,14 @@ files:
 - spec/examples/reporter/html/dsl_spec.rb
 - spec/examples/reporter/html/index_spec.rb
 - spec/examples/reporter/html_spec.rb
+- spec/examples/reporter/schema_csv_spec.rb
 - spec/examples/reporter_spec.rb
 - spec/examples/tags_spec.rb
 - spec/examples/task_spec.rb
 - spec/examples/util/cacheable_spec.rb
 - spec/examples/util/enumerable_spec.rb
 - spec/examples/util/file_spec.rb
+- spec/examples/util/line_numbers_spec.rb
 - spec/examples/util/process_queue_spec.rb
 - spec/examples/util/thunk_spec.rb
 - spec/examples/version_spec.rb
@@ -136,9 +156,12 @@ files:
 - spec/issues/028_spec.rb
 - spec/issues/032_spec.rb
 - spec/issues/036_spec.rb
+- spec/issues/037_spec.rb
+- spec/issues/case_in_condition_spec.rb
+- spec/issues/commit_spec.rb
 - spec/spec_helper.rb
 - spec/spec_suite.rb
-homepage: https://github.com/sergeiboikov/piggly
+homepage: https://github.com/NSDDeveloper/piggly
 licenses:
 - BSD-2-Clause
 metadata: {}