Remove the $ENV{PERL_CORE} boilerplate from Storable's tests.
Nicholas Clark [Sun, 30 Aug 2009 10:43:13 +0000 (11:43 +0100)]
32 files changed:
ext/Storable/t/attach_errors.t
ext/Storable/t/attach_singleton.t
ext/Storable/t/blessed.t
ext/Storable/t/canonical.t
ext/Storable/t/circular_hook.t
ext/Storable/t/code.t
ext/Storable/t/compat01.t
ext/Storable/t/compat06.t
ext/Storable/t/croak.t
ext/Storable/t/dclone.t
ext/Storable/t/downgrade.t
ext/Storable/t/forgive.t
ext/Storable/t/freeze.t
ext/Storable/t/integer.t
ext/Storable/t/interwork56.t
ext/Storable/t/just_plain_nasty.t
ext/Storable/t/lock.t
ext/Storable/t/malice.t
ext/Storable/t/overload.t
ext/Storable/t/recurse.t
ext/Storable/t/restrict.t
ext/Storable/t/retrieve.t
ext/Storable/t/sig_die.t
ext/Storable/t/store.t
ext/Storable/t/threads.t
ext/Storable/t/tied.t
ext/Storable/t/tied_hook.t
ext/Storable/t/tied_items.t
ext/Storable/t/utf8.t
ext/Storable/t/utf8hash.t
ext/Storable/t/weak.t
t/TEST

index 85971db..ffa41f9 100644 (file)
 # which Storable should (correctly) throw errors.
 
 sub BEGIN {
-    if ($ENV{PERL_CORE}){
-       chdir('t') if -d 't';
-       @INC = ('.', '../lib');
-    } else {
-       unshift @INC, 't';
-    }
+    unshift @INC, 't';
     require Config; import Config;
     if ($ENV{PERL_CORE} and $Config{'extensions'} !~ /\bStorable\b/) {
         print "1..0 # Skip: Storable was not built\n";
index 475204f..e850a16 100644 (file)
 # which should see both structs pointing to the same object.
 
 sub BEGIN {
-    if ($ENV{PERL_CORE}){
-       chdir('t') if -d 't';
-       @INC = ('.', '../lib');
-    } else {
-       unshift @INC, 't';
-    }
+    unshift @INC, 't';
     require Config; import Config;
     if ($ENV{PERL_CORE} and $Config{'extensions'} !~ /\bStorable\b/) {
         print "1..0 # Skip: Storable was not built\n";
index 842674f..7c0494c 100644 (file)
@@ -7,12 +7,7 @@
 #
 
 sub BEGIN {
-    if ($ENV{PERL_CORE}){
-       chdir('t') if -d 't';
-       @INC = ('.', '../lib', '../ext/Storable/t');
-    } else {
-       unshift @INC, 't';
-    }
+    unshift @INC, 't';
     require Config; import Config;
     if ($ENV{PERL_CORE} and $Config{'extensions'} !~ /\bStorable\b/) {
         print "1..0 # Skip: Storable was not built\n";
index 7b7936c..204a235 100644 (file)
@@ -7,12 +7,7 @@
 #  
 
 sub BEGIN {
-    if ($ENV{PERL_CORE}){
-       chdir('t') if -d 't';
-       @INC = ('.', '../lib');
-    } else {
-       unshift @INC, 't';
-    }
+    unshift @INC, 't';
     require Config; import Config;
     if ($ENV{PERL_CORE} and $Config{'extensions'} !~ /\bStorable\b/) {
         print "1..0 # Skip: Storable was not built\n";
index 782b3d3..48f4be4 100644 (file)
 # which Storable should (correctly) throw errors.
 
 sub BEGIN {
-    if ($ENV{PERL_CORE}){
-       chdir('t') if -d 't';
-       @INC = ('.', '../lib');
-    } else {
-       unshift @INC, 't';
-    }
+    unshift @INC, 't';
     require Config; import Config;
     if ($ENV{PERL_CORE} and $Config{'extensions'} !~ /\bStorable\b/) {
         print "1..0 # Skip: Storable was not built\n";
index c3c988b..dd2a96e 100644 (file)
@@ -7,12 +7,7 @@
 #
 
 sub BEGIN {
-    if ($ENV{PERL_CORE}){
-       chdir('t') if -d 't';
-       @INC = ('.', '../lib');
-    } else {
-       unshift @INC, 't';
-    }
+    unshift @INC, 't';
     require Config; import Config;
     if ($ENV{PERL_CORE} and $Config{'extensions'} !~ /\bStorable\b/) {
         print "1..0 # Skip: Storable was not built\n";
index 536d85e..9b47212 100644 (file)
@@ -1,12 +1,7 @@
 #!perl -w
 
 BEGIN {
-    if ($ENV{PERL_CORE}){
-        chdir('t') if -d 't';
-        @INC = ('.', '../lib', '../ext/Storable/t');
-    } else {
-        unshift @INC, 't';
-    }
+    unshift @INC, 't';
     require Config; import Config;
     if ($ENV{PERL_CORE} and $Config{'extensions'} !~ /\bStorable\b/) {
         print "1..0 # Skip: Storable was not built\n";
index 7dbf1b8..6d8ade3 100644 (file)
@@ -7,12 +7,7 @@
 #
 
 BEGIN {
-    if ($ENV{PERL_CORE}){
-       chdir('t') if -d 't';
-       @INC = ('.', '../lib', '../ext/Storable/t');
-    } else {
-       unshift @INC, 't';
-    }
+    unshift @INC, 't';
     require Config; import Config;
     if ($ENV{PERL_CORE} and $Config{'extensions'} !~ /\bStorable\b/) {
         print "1..0 # Skip: Storable was not built\n";
index 96c7cb3..ecd2bf8 100644 (file)
@@ -6,10 +6,6 @@
 # In particular, don't use Test; as this covers up the problem.
 
 sub BEGIN {
-    if ($ENV{PERL_CORE}){
-       chdir('t') if -d 't';
-       @INC = ('.', '../lib');
-    }
     if ($ENV{PERL_CORE}) {
        require Config; import Config;
        %Config=%Config if 0; # cease -w
index c3aa180..078cd81 100644 (file)
@@ -7,12 +7,7 @@
 #
 
 sub BEGIN {
-    if ($ENV{PERL_CORE}){
-       chdir('t') if -d 't';
-       @INC = ('.', '../lib', '../ext/Storable/t');
-    } else {
-       unshift @INC, 't';
-    }
+    unshift @INC, 't';
     require Config; import Config;
     if ($ENV{PERL_CORE} and $Config{'extensions'} !~ /\bStorable\b/) {
         print "1..0 # Skip: Storable was not built\n";
index d977a00..76bd05a 100644 (file)
 # are encountered.
 
 sub BEGIN {
-    if ($ENV{PERL_CORE}){
-       chdir('t') if -d 't';
-       @INC = ('.', '../lib');
-    } else {
-       unshift @INC, 't';
-    }
+    unshift @INC, 't';
     require Config; import Config;
     if ($ENV{PERL_CORE} and $Config{'extensions'} !~ /\bStorable\b/) {
         print "1..0 # Skip: Storable was not built\n";
index 1c07a57..495edc3 100644 (file)
 #
 
 sub BEGIN {
-    if ($ENV{PERL_CORE}){
-       chdir('t') if -d 't';
-       @INC = ('.', '../lib');
-    } else {
-       unshift @INC, 't';
-    }
+    unshift @INC, 't';
     require Config; import Config;
     if ($ENV{PERL_CORE} and $Config{'extensions'} !~ /\bStorable\b/) {
         print "1..0 # Skip: Storable was not built\n";
index d69e3b4..e76b669 100644 (file)
@@ -7,12 +7,7 @@
 #
 
 sub BEGIN {
-    if ($ENV{PERL_CORE}){
-       chdir('t') if -d 't';
-       @INC = ('.', '../lib', '../ext/Storable/t');
-    } else {
-       unshift @INC, 't';
-    }
+    unshift @INC, 't';
     require Config; import Config;
     if ($ENV{PERL_CORE} and $Config{'extensions'} !~ /\bStorable\b/) {
         print "1..0 # Skip: Storable was not built\n";
index ec7961e..2c22235 100644 (file)
 # are encountered.
 
 sub BEGIN {
-    if ($ENV{PERL_CORE}){
-       chdir('t') if -d 't';
-       @INC = ('.', '../lib');
-    } else {
-       unshift @INC, 't';
-    }
+    unshift @INC, 't';
     require Config; import Config;
     if ($ENV{PERL_CORE} and $Config{'extensions'} !~ /\bStorable\b/) {
         print "1..0 # Skip: Storable was not built\n";
index 5339c0f..4e9b414 100644 (file)
 # on Unix systems with IV as long long works.
 
 sub BEGIN {
-    if ($ENV{PERL_CORE}){
-       chdir('t') if -d 't';
-       @INC = ('.', '../lib');
-    } else {
-       unshift @INC, 't';
-    }
+    unshift @INC, 't';
     require Config; import Config;
     if ($ENV{PERL_CORE} and $Config{'extensions'} !~ /\bStorable\b/) {
         print "1..0 # Skip: Storable was not built\n";
index 77271fb..85eaa90 100644 (file)
@@ -6,12 +6,7 @@
 #  Everyone's invited! :-D
 
 sub BEGIN {
-    if ($ENV{PERL_CORE}){
-        chdir('t') if -d 't';
-        @INC = ('.', '../lib');
-    } else {
-        unshift @INC, 't';
-    }
+    unshift @INC, 't';
     require Config; import Config;
     if ($ENV{PERL_CORE} and $Config{'extensions'} !~ /\bStorable\b/) {
         print "1..0 # Skip: Storable was not built\n";
index e93cef4..14b5f42 100644 (file)
@@ -7,12 +7,7 @@
 #
 
 sub BEGIN {
-    if ($ENV{PERL_CORE}){
-       chdir('t') if -d 't';
-       @INC = ('.', '../lib', '../ext/Storable/t');
-    } else {
-       unshift @INC, 't';
-    }
+    unshift @INC, 't';
     require Config; import Config;
     if ($ENV{PERL_CORE} and $Config{'extensions'} !~ /\bStorable\b/) {
         print "1..0 # Skip: Storable was not built\n";
index 648b15f..ddb65dd 100644 (file)
 # It also acts as a test for read_header
 
 sub BEGIN {
-    if ($ENV{PERL_CORE}){
-       chdir('t') if -d 't';
-       @INC = ('.', '../lib', '../ext/Storable/t');
-    } else {
-       # This lets us distribute Test::More in t/
-       unshift @INC, 't';
-    }
+    # This lets us distribute Test::More in t/
+    unshift @INC, 't';
     require Config; import Config;
     if ($ENV{PERL_CORE} and $Config{'extensions'} !~ /\bStorable\b/) {
         print "1..0 # Skip: Storable was not built\n";
index 93f0794..22fccfb 100644 (file)
@@ -7,12 +7,7 @@
 #  
 
 sub BEGIN {
-    if ($ENV{PERL_CORE}){
-       chdir('t') if -d 't';
-       @INC = ('.', '../lib', '../ext/Storable/t');
-    } else {
-       unshift @INC, 't';
-    }
+    unshift @INC, 't';
     require Config; import Config;
     if ($ENV{PERL_CORE} and $Config{'extensions'} !~ /\bStorable\b/) {
         print "1..0 # Skip: Storable was not built\n";
index 18b461b..d7dcb0e 100644 (file)
@@ -7,12 +7,7 @@
 #  
 
 sub BEGIN {
-    if ($ENV{PERL_CORE}){
-       chdir('t') if -d 't';
-       @INC = ('.', '../lib', '../ext/Storable/t');
-    } else {
-       unshift @INC, 't';
-    }
+    unshift @INC, 't';
     require Config; import Config;
     if ($ENV{PERL_CORE} and $Config{'extensions'} !~ /\bStorable\b/) {
         print "1..0 # Skip: Storable was not built\n";
index d5c4bd6..be7f408 100644 (file)
@@ -7,9 +7,8 @@
 #
 
 sub BEGIN {
-    chdir('t') if -d 't';
+    unshift @INC, 't';
     if ($ENV{PERL_CORE}){
-       @INC = ('.', '../lib', '../ext/Storable/t');
         require Config;
         if ($Config::Config{'extensions'} !~ /\bStorable\b/) {
             print "1..0 # Skip: Storable was not built\n";
index b022511..2e44d5d 100644 (file)
@@ -7,12 +7,7 @@
 #
 
 sub BEGIN {
-    if ($ENV{PERL_CORE}){
-       chdir('t') if -d 't';
-       @INC = ('.', '../lib', '../ext/Storable/t');
-    } else {
-       unshift @INC, 't';
-    }
+    unshift @INC, 't';
     require Config; import Config;
     if ($ENV{PERL_CORE} and $Config{'extensions'} !~ /\bStorable\b/) {
         print "1..0 # Skip: Storable was not built\n";
index 0350752..d2390a7 100644 (file)
@@ -7,12 +7,7 @@
 #
 
 sub BEGIN {
-    if ($ENV{PERL_CORE}){
-       chdir('t') if -d 't';
-       @INC = ('.', '../lib');
-    } else {
-       unshift @INC, 't';
-    }
+    unshift @INC, 't';
     require Config; import Config;
     if ($ENV{PERL_CORE} and $Config{'extensions'} !~ /\bStorable\b/) {
         print "1..0 # Skip: Storable was not built\n";
index fc2141c..61bcf7d 100644 (file)
@@ -7,12 +7,7 @@
 #
 
 sub BEGIN {
-    if ($ENV{PERL_CORE}){
-       chdir('t') if -d 't';
-       @INC = ('.', '../lib', '../ext/Storable/t');
-    } else {
-       unshift @INC, 't';
-    }
+    unshift @INC, 't';
     require Config; import Config;
     if ($ENV{PERL_CORE} and $Config{'extensions'} !~ /\bStorable\b/) {
         print "1..0 # Skip: Storable was not built\n";
index 664566e..72efdda 100644 (file)
 # thread when it starts
 
 sub BEGIN {
-    if ($ENV{PERL_CORE}){
-       chdir('t') if -d 't';
-       @INC = ('.', '../lib');
-    } else {
-       unshift @INC, 't';
-    }
+    unshift @INC, 't';
     require Config; import Config;
     if ($ENV{PERL_CORE} and $Config{'extensions'} !~ /\bStorable\b/) {
         print "1..0 # Skip: Storable was not built\n";
index 662d9ae..9a7f571 100644 (file)
@@ -7,12 +7,7 @@
 #
 
 sub BEGIN {
-    if ($ENV{PERL_CORE}){
-       chdir('t') if -d 't';
-       @INC = ('.', '../lib', '../ext/Storable/t');
-    } else {
-       unshift @INC, 't';
-    }
+    unshift @INC, 't';
     require Config; import Config;
     if ($ENV{PERL_CORE} and $Config{'extensions'} !~ /\bStorable\b/) {
         print "1..0 # Skip: Storable was not built\n";
index 28b5363..8f2846e 100644 (file)
@@ -7,12 +7,7 @@
 #
 
 sub BEGIN {
-    if ($ENV{PERL_CORE}){
-       chdir('t') if -d 't';
-       @INC = ('.', '../lib', '../ext/Storable/t');
-    } else {
-       unshift @INC, 't';
-    }
+    unshift @INC, 't';
     require Config; import Config;
     if ($ENV{PERL_CORE} and $Config{'extensions'} !~ /\bStorable\b/) {
         print "1..0 # Skip: Storable was not built\n";
index ba918bf..bd15e5c 100644 (file)
 #
 
 sub BEGIN {
-    if ($ENV{PERL_CORE}){
-       chdir('t') if -d 't';
-       @INC = ('.', '../lib', '../ext/Storable/t');
-    } else {
-       unshift @INC, 't';
-    }
+    unshift @INC, 't';
     require Config; import Config;
     if ($ENV{PERL_CORE} and $Config{'extensions'} !~ /\bStorable\b/) {
         print "1..0 # Skip: Storable was not built\n";
index ab0f62c..67b7917 100644 (file)
@@ -12,12 +12,7 @@ sub BEGIN {
        print "1..0 # Skip: no utf8 support\n";
        exit 0;
     }
-    if ($ENV{PERL_CORE}){
-       chdir('t') if -d 't';
-       @INC = ('.', '../lib', '../ext/Storable/t');
-    } else {
-       unshift @INC, 't';
-    }
+    unshift @INC, 't';
     require Config; import Config;
     if ($ENV{PERL_CORE} and $Config{'extensions'} !~ /\bStorable\b/) {
         print "1..0 # Skip: Storable was not built\n";
index a529ea5..7eac651 100644 (file)
@@ -5,17 +5,7 @@ sub BEGIN {
        print "1..0 # Skip: no utf8 hash key support\n";
        exit 0;
     }
-    if ($ENV{PERL_CORE}){
-       chdir('t') if -d 't';
-       @INC = ('.', '../lib');
-        if ($^O eq 'MacOS') {
-            # Look, I'm using this fully-qualified variable more than once!
-            my $arch = $MacPerl::Architecture;
-            push @INC, "::lib:${MacPerl::Architecture}:";
-        }
-    } else {
-       unshift @INC, 't';
-    }
+    unshift @INC, 't';
     require Config; import Config;
     if ($ENV{PERL_CORE}){
        if($Config{'extensions'} !~ /\bStorable\b/) {
index a8b708b..a2be7a1 100644 (file)
@@ -7,13 +7,8 @@
 #
 
 sub BEGIN {
-  if ($ENV{PERL_CORE}){
-    chdir('t') if -d 't';
-    @INC = ('.', '../lib', '../ext/Storable/t');
-  } else {
-    # This lets us distribute Test::More in t/
-    unshift @INC, 't';
-  }
+  # This lets us distribute Test::More in t/
+  unshift @INC, 't';
   require Config; import Config;
   if ($ENV{PERL_CORE} and $Config{'extensions'} !~ /\bStorable\b/) {
     print "1..0 # Skip: Storable was not built\n";
diff --git a/t/TEST b/t/TEST
index 0231332..470d6f1 100755 (executable)
--- a/t/TEST
+++ b/t/TEST
@@ -39,7 +39,6 @@ my %temp_no_core =
      '../ext/IPC-SysV' => 1,
      '../ext/MIME-Base64' => 1,
      '../ext/Safe' => 'fixme',
-     '../ext/Storable' => 'fixme',
      '../ext/Time-HiRes' => 1,
      '../ext/Unicode-Normalize' => 1,
     );