This is a live mirror of the Perl 5 development currently hosted at https://github.com/perl/perl5
Fix a Storable test to work in 5.8.9-
[perl5.git] / dist / Storable / t / restrict.t
index be7f408..65dac6f 100644 (file)
@@ -8,6 +8,7 @@
 
 sub BEGIN {
     unshift @INC, 't';
+    unshift @INC, 't/compat' if $] < 5.006002;
     if ($ENV{PERL_CORE}){
         require Config;
         if ($Config::Config{'extensions'} !~ /\bStorable\b/) {
@@ -19,7 +20,7 @@ sub BEGIN {
            print "1..0 # Skip: No Hash::Util pre 5.005\n";
            exit 0;
            # And doing this seems on 5.004 seems to create bogus warnings about
-           # unitialized variables, or coredumps in Perl_pp_padsv
+           # uninitialized variables, or coredumps in Perl_pp_padsv
        } elsif (!eval "require Hash::Util") {
             if ($@ =~ /Can\'t locate Hash\/Util\.pm in \@INC/s) {
                 print "1..0 # Skip: No Hash::Util:\n";
@@ -30,14 +31,12 @@ sub BEGIN {
         }
        unshift @INC, 't';
     }
-    require 'st-dump.pl';
 }
 
 
 use Storable qw(dclone freeze thaw);
-use Hash::Util qw(lock_hash unlock_value);
-
-print "1..100\n";
+use Hash::Util qw(lock_hash unlock_value lock_keys);
+use Test::More tests => 104;
 
 my %hash = (question => '?', answer => 42, extra => 'junk', undef => undef);
 lock_hash %hash;
@@ -67,37 +66,27 @@ sub testit {
 
   my @in_keys = sort keys %$hash;
   my @out_keys = sort keys %$copy;
-  unless (ok ++$test, "@in_keys" eq "@out_keys") {
-    print "# Failed: keys mis-match after deep clone.\n";
-    print "# Original keys: @in_keys\n";
-    print "# Copy's keys: @out_keys\n";
-  }
+  is("@in_keys", "@out_keys", "keys match after deep clone");
 
   # $copy = $hash;     # used in initial debug of the tests
 
-  ok ++$test, Internals::SvREADONLY(%$copy), "cloned hash restricted?";
+  is(Internals::SvREADONLY(%$copy), 1, "cloned hash restricted?");
 
-  ok ++$test, Internals::SvREADONLY($copy->{question}),
-    "key 'question' not locked in copy?";
+  is(Internals::SvREADONLY($copy->{question}), 1,
+     "key 'question' not locked in copy?");
 
-  ok ++$test, !Internals::SvREADONLY($copy->{answer}),
-    "key 'answer' not locked in copy?";
+  is(Internals::SvREADONLY($copy->{answer}), '',
+     "key 'answer' not locked in copy?");
 
   eval { $copy->{extra} = 15 } ;
-  unless (ok ++$test, !$@, "Can assign to reserved key 'extra'?") {
-    my $diag = $@;
-    $diag =~ s/\n.*\z//s;
-    print "# \$\@: $diag\n";
-  }
+  is($@, '', "Can assign to reserved key 'extra'?");
 
   eval { $copy->{nono} = 7 } ;
-  ok ++$test, $@, "Can not assign to invalid key 'nono'?";
+  isnt($@, '', "Can not assign to invalid key 'nono'?");
 
-  ok ++$test, exists $copy->{undef},
-    "key 'undef' exists";
+  is(exists $copy->{undef}, 1, "key 'undef' exists");
 
-  ok ++$test, !defined $copy->{undef},
-    "value for key 'undef' is undefined";
+  is($copy->{undef}, undef, "value for key 'undef' is undefined");
 }
 
 for $Storable::canonical (0, 1) {
@@ -119,11 +108,13 @@ for $Storable::canonical (0, 1) {
     for (0..16) {
       my $k = "k$_";
       eval { $copy->{$k} = undef } ;
-      unless (ok ++$test, !$@, "Can assign to reserved key '$k'?") {
-       my $diag = $@;
-       $diag =~ s/\n.*\z//s;
-       print "# \$\@: $diag\n";
-      }
+      is($@, '', "Can assign to reserved key '$k'?");
     }
+
+    my %hv;
+    $hv{a} = __PACKAGE__;
+    lock_keys %hv;
+    my $hv2 = &$cloner(\%hv);
+    ok eval { $$hv2{a} = 70 }, 'COWs do not become read-only';
   }
 }