-BEGIN { delete $ENV{PERL_STRICTURES_EXTRA} }
+BEGIN { $ENV{PERL_STRICTURES_EXTRA} = 0 }
-# -e is sufficient here.
--e 't/smells-of-vcs/.git'
- or mkdir('t/smells-of-vcs/.git')
- or die "Couldn't create fake .git: $!";
+sub _eval { eval $_[0] }
+use strict;
+use warnings;
use Test::More qw(no_plan);
-our (@us, @expect);
-
-sub capture_stuff { [ $^H, ${^WARNING_BITS} ] }
-
-sub capture_us { push @us, capture_stuff }
-sub capture_expect { push @expect, capture_stuff }
+sub capture_hints {
+ my $code = shift;
+ $code .= q{
+ ;
+ my @h;
+ BEGIN { @h = ( $^H, ${^WARNING_BITS} ) }
+ @h;
+ };
+ my ($hints, $warning_bits) = _eval $code or die $@;
+ # ignore lexicalized hints
+ $hints &= ~ 0x20000;
+ $warning_bits = unpack "H*", $warning_bits
+ if defined $warning_bits;
+ return ($hints, $warning_bits);
+}
-{
- BEGIN { $ENV{PERL_STRICTURES_EXTRA} = 0 }
- use strictures 1;
- BEGIN { capture_us }
- BEGIN { delete $ENV{PERL_STRICTURES_EXTRA} }
+sub compare_hints {
+ my ($code_want, $code_got, $name) = @_;
+ my ($want_hints, $want_warnings) = capture_hints $code_want;
+ my ($hints, $warnings) = capture_hints $code_got;
+ is($hints, $want_hints, "Hints correct for $name");
+ is($warnings, $want_warnings, "Warnings correct for $name");
}
-{
+compare_hints q{
use strict;
use warnings FATAL => 'all';
- BEGIN { capture_expect }
-}
-
-# I'm assuming here we'll have more cases later. maybe not. eh.
+},
+q{
+ use strictures 1;
+},
+ 'version 1';
-foreach my $idx (0 .. $#us) {
- is($us[$idx][0], $expect[$idx][0], 'Hints ok for case '.($idx+1));
- is($us[$idx][1], $expect[$idx][1], 'Warnings ok for case '.($idx+1));
-}
+compare_hints q{
+ use strict;
+ use warnings 'all';
+ use warnings FATAL => @strictures::WARNING_CATEGORIES;
+ no warnings FATAL => @strictures::V2_NONFATAL;
+ use warnings @strictures::V2_NONFATAL;
+ no warnings @strictures::V2_DISABLE;
+},
+q{
+ use strictures 2;
+},
+ 'version 2';
my $v;
eval { $v = strictures->VERSION; 1 } or diag $@;
is $v, $strictures::VERSION, '->VERSION returns version correctly';
-SKIP: {
- skip 'Extra tests disabled on perls <= 5.008003', 1
- if $] < 5.008004;
- skip 'Not got all the modules to do this', 1
- unless eval {
- require indirect;
- require multidimensional;
- require bareword::filehandles;
- 1;
- };
- sub Foo::new { 1 }
- chdir("t/smells-of-vcs");
- local $strictures::Smells_Like_VCS = 1;
- foreach my $file (qw(lib/one.pm t/one.faket)) {
- ok(!eval { require $file; 1 }, "Failed to load ${file}");
- like($@, qr{Indirect call of method}, "Failed due to indirect.pm, ok");
- }
- ok(eval { require "other/one.pl"; 1 }, "Loaded other/one.pl ok");
- chdir("../..");
-}
-
-ok(!eval q{use strictures 2; 1; }, "Can't use strictures 2 (this is version 1)");
+my $next = int $v + 1;
+eval qq{ use strictures $next; };
-SKIP: {
- skip 'Extra tests disabled on perls <= 5.008003', 1
- if $] < 5.008004;
- local $ENV{PERL_STRICTURES_EXTRA} = 1;
- local $strictures::extra_load_states = undef;
- local @INC = ("t/dep_constellations/broken", @INC);
- local %INC = %INC;
- delete $INC{$_}
- for qw( indirect.pm multidimensional.pm bareword/filehandles.pm );
-
- {
- open my $fh, '>', \my $str;
- local *STDERR = $fh;
- strictures->import;
- like(
- $str,
- qr/Missing were:\n\n indirect multidimensional bareword::filehandles/,
- "failure to load all three extra deps is reported"
- );
- }
-
- {
- open my $fh, '>', \my $str;
- local *STDERR = $fh;
- strictures->import;
- ok( !$str, "extra dep load failure is not reported a second time" );
- }
-}
+like $@, qr/strictures version $next required/,
+ "Can't use strictures $next (this is version $v)";