Tighten limits on code coverage context selection.

If the last } of a function was marked as uncovered then the context selection would overrun into the next function. Start checking context on the current line to prevent this. Make the same change for start context even though it doesn't seem to have an issue.
pgbackrest · Nov 13, 2018 · acb579c · acb579c
1 parent 086bc35
commit acb579c
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/test/lib/pgBackRestTest/Common/CoverageTest.pm b/test/lib/pgBackRestTest/Common/CoverageTest.pm
@@ -178,7 +178,7 @@ sub coverageGenerate
             foreach my $iLine (sort(keys(%{$rhCoverage->{$strFile}{line}})))
             {
                 # Run back to the beginning of the function comment
-                for (my $iLineIdx = $iLine - 1; $iLineIdx >= 0; $iLineIdx--)
+                for (my $iLineIdx = $iLine; $iLineIdx >= 0; $iLineIdx--)
                 {
                     if (!defined($rhCoverage->{$strFile}{line}{sprintf("%09d", $iLineIdx)}))
                     {
@@ -189,7 +189,7 @@ sub coverageGenerate
                 }
 
                 # Run forward to the end of the function
-                for (my $iLineIdx = $iLine + 1; $iLineIdx < @stryC; $iLineIdx++)
+                for (my $iLineIdx = $iLine; $iLineIdx < @stryC; $iLineIdx++)
                 {
                     if (!defined($rhCoverage->{$strFile}{line}{sprintf("%09d", $iLineIdx)}))
                     {