Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the Test-Strict CPAN distribution.

Report information
The Basics
Id: 60953
Status: new
Priority: 0/
Queue: Test-Strict

People
Owner: Nobody in particular
Requestors: hma [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: Important
Broken in: 0.14
Fixed in: (no value)



Subject: all_cover_ok() doesn't respect taint switches
I think you just forgot to include the taint switches in the all_cover_ok() test. I hope this can easily resolved by using the already existing _taint_switch() function. The applied small patch works for me.
Subject: Strict.patch
*** Strict.pm.orig 2010-02-14 01:49:40.000000000 +0100 --- Strict.pm 2010-09-01 23:20:29.000000000 +0200 *************** *** 358,364 **** } foreach my $file ( @all_files ) { $file = _untaint($file); ! `$perl_bin -MDevel::Cover=$DEVEL_COVER_OPTIONS $file`; $Test->ok(! $?, "Coverage captured from $file" ); } $Test->ok(my $cover = `$cover_bin 2>&1`, "Got cover"); --- 358,369 ---- } foreach my $file ( @all_files ) { $file = _untaint($file); ! ! # Add the -t -T switches if they are set in the #! line ! my $switch = _taint_switch($file) || ''; ! $switch = "-$switch " if $switch; ! ! `$perl_bin $switch-MDevel::Cover=$DEVEL_COVER_OPTIONS $file`; $Test->ok(! $?, "Coverage captured from $file" ); } $Test->ok(my $cover = `$cover_bin 2>&1`, "Got cover");