# New Ticket Created by  James Keenan 
# Please include the string:  [perl #45919]
# in the subject line of all future correspondence about this issue. 
# <URL: http://rt.perl.org/rt3/Ticket/Display.html?id=45919 >


Until Sept 24, lib/Parrot/BuildUtil.pm was one of those Perl 5  
components of Parrot which had 100% test coverage for statements,  
branches, conditions and subroutines.  But then one of our brethren  
added a new subroutine, generated_file_header(), to that package --  
without adding a corresponding test file!

[parrot] 509 $ !507
fns t/configure/ | xargs grep -n generated_file_header
[parrot] 510 $

Adding new code to thoroughly tested files without accompanying them  
with unit tests inevitably leads to lower test coverage (see http:// 
thenceforward.net/parrot/coverage/configure-build/lib-Parrot- 
BuildUtil-pm.html).  And since I'm the person who wrote the tests  
that got that package to 100% coverage, I take a personal interest in  
seeing that its coverage remains high.

I'd like to ask that a file be added to test this new functionality.   
You could probably start by copying t/configure/039-slurp_file.t into  
04-generated_file_header.t and working from there.

Thank you very much.
kid51

Reply via email to