Merge pull request #540 from utPLSQL/feature/coverage_false_negatives

jgebal · web-flow · commit 9ff76fca38a0 · 2017-12-31T15:11:51.000Z
Fixed coverage reporting for labelled `end loop` block.
diff --git a/source/core/coverage/ut_coverage.pkb b/source/core/coverage/ut_coverage.pkb
@@ -67,7 +67,7 @@ create or replace package body ut_coverage is
                      ) and
                      regexp_like(
                         s.text,
-                        '^\s*(((not)?\s*(overriding|final|instantiable)\s*)*(static|constructor|member)?\s*(procedure|function)|package(\s+body)|begin|end(\s+\S+)?\s*;)', 'i'
+                        '^([\t ]*(((not)?\s*(overriding|final|instantiable)[\t ]*)*(static|constructor|member)?[\t ]*(procedure|function)|package([\t ]+body)|begin|end([\t ]+\S+)*[ \t]*;))', 'i'
                      )
                     then 'Y'
                  end as to_be_skipped

Original file line number	Diff line number	Diff line change
`@@ -67,7 +67,7 @@ create or replace package body ut_coverage is`
`67`	`67`	`) and`
`68`	`68`	`regexp_like(`
`69`	`69`	`s.text,`
`70`		`- '^\s(((not)?\s(overriding\|final\|instantiable)\s)(static\|constructor\|member)?\s(procedure\|function)\|package(\s+body)\|begin\|end(\s+\S+)?\s;)', 'i'`
	`70`	`+ '^([\t ](((not)?\s(overriding\|final\|instantiable)[\t ])(static\|constructor\|member)?[\t ](procedure\|function)\|package([\t ]+body)\|begin\|end([\t ]+\S+)[ \t]*;))', 'i'`
`71`	`71`	`)`
`72`	`72`	`then 'Y'`
`73`	`73`	`end as to_be_skipped`