Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the Perl-Critic CPAN distribution.

Report information
The Basics
Id: 21718
Status: resolved
Priority: 0/
Queue: Perl-Critic

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

Bug Information
Severity: Unimportant
Broken in: (no value)
Fixed in: (no value)



Subject: No skip for File::Slurp in includes.t
FYI, trunk barks because no File::Slurp was installed AUTHOR_TEST check. t/94_includes.........................Can't locate File/Slurp.pm in @INC (@INC contains: C:\Perl-Critic\blib\lib C:\Perl-Critic\blib\arch C: /Development/Perl/584/lib C:/Development/Perl/584/site/lib . C:/Development/Perl/584/lib C:/Development/Perl/584/site/lib .) at t/94_include s.t line 5.
Subject: 94_includes.t.patch
Index: t/94_includes.t =================================================================== --- t/94_includes.t (revision 668) +++ t/94_includes.t (working copy) @@ -2,11 +2,17 @@ use warnings; use strict; -use File::Slurp; use File::Find; use PPI::Document; use Test::More; +BEGIN { + eval 'use File::Slurp'; + if($@) { + plan skip_all => 'File::Slurp not installed'; + } +} + if (!$ENV{TEST_AUTHOR}) { plan skip_all => 'Author test'; }
Heh, I committed an easier patch! Thanks. -- Chris --- t/94_includes.t (revision 668) +++ t/94_includes.t (working copy) @@ -2,7 +2,6 @@ use warnings; use strict; -use File::Slurp; use File::Find; use PPI::Document; use Test::More;