Convert ext/SDBM_File/t/sdbm.t to Test::More.

1 files changed, 93 insertions, 100 deletions

diff --git a/ext/SDBM_File/t/sdbm.t b/ext/SDBM_File/t/sdbm.t
index f7b0ae64f2..3af6a583ce 100644
--- a/ext/SDBM_File/t/sdbm.t
+++ b/ext/SDBM_File/t/sdbm.t
@@ -13,44 +13,34 @@ BEGIN {
 use strict;
 use warnings;
 
-sub ok
-{
-    my $no = shift ;
-    my $result = shift ;
-
-    print "not " unless $result ;
-    print "ok $no\n" ;
-}
+use Test::More tests => 81;
 
 require SDBM_File;
 #If Fcntl is not available, try 0x202 or 0x102 for O_RDWR|O_CREAT
 use Fcntl;
 
-print "1..80\n";
-
 unlink <Op_dbmx.*>;
 
 umask(0);
 my %h ;
-ok(1, tie %h,'SDBM_File','Op_dbmx', O_RDWR|O_CREAT, 0640);
+isa_ok(tie(%h, 'SDBM_File', 'Op_dbmx', O_RDWR|O_CREAT, 0640), 'SDBM_File');
 
 my $Dfile = "Op_dbmx.pag";
 if (! -e $Dfile) {
 	($Dfile) = <Op_dbmx.*>;
 }
-if ($^O eq 'amigaos' || $^O eq 'os2' || $^O eq 'MSWin32' || $^O eq 'NetWare' || $^O eq 'dos' || $^O eq 'cygwin') {
-    print "ok 2 # Skipped: different file permission semantics\n";
-}
-else {
+SKIP: {
+    skip "different file permission semantics on $^O", 1
+	if $^O eq 'amigaos' || $^O eq 'os2' || $^O eq 'MSWin32' || $^O eq 'NetWare' || $^O eq 'dos' || $^O eq 'cygwin';
     my ($dev,$ino,$mode,$nlink,$uid,$gid,$rdev,$size,$atime,$mtime,$ctime,
      $blksize,$blocks) = stat($Dfile);
-    print (($mode & 0777) == ($^O eq 'vos' ? 0750 : 0640) ? "ok 2\n" : "not ok 2\n");
+    is($mode & 0777, 0640);
 }
 my $i = 0;
 while (my ($key,$value) = each(%h)) {
     $i++;
 }
-print (!$i ? "ok 3\n" : "not ok 3\n");
+is($i, 0);
 
 $h{'goner1'} = 'snork';
 
@@ -72,7 +62,7 @@ $h{'goner2'} = 'snork';
 delete $h{'goner2'};
 
 untie(%h);
-print (tie(%h,'SDBM_File','Op_dbmx', O_RDWR, 0640) ? "ok 4\n" : "not ok 4\n");
+isa_ok(tie(%h, 'SDBM_File', 'Op_dbmx', O_RDWR, 0640), 'SDBM_File');
 
 $h{'j'} = 'J';
 $h{'k'} = 'K';
@@ -100,7 +90,8 @@ delete $h{'goner3'};
 my @keys = keys(%h);
 my @values = values(%h);
 
-if ($#keys == 29 && $#values == 29) {print "ok 5\n";} else {print "not ok 5\n";}
+is($#keys, 29);
+is($#values, 29);
 
 while (my ($key,$value) = each(%h)) {
     if ($key eq $keys[$i] && $value eq $values[$i] && $key eq lc($value)) {
@@ -109,30 +100,29 @@ while (my ($key,$value) = each(%h)) {
     }
 }
 
-if ($i == 30) {print "ok 6\n";} else {print "not ok 6\n";}
+is($i, 30);
 
 @keys = ('blurfl', keys(%h), 'dyick');
-if ($#keys == 31) {print "ok 7\n";} else {print "not ok 7\n";}
+is($#keys, 31);
 
 $h{'foo'} = '';
 $h{''} = 'bar';
 
-# check cache overflow and numeric keys and contents
 my $ok = 1;
 for ($i = 1; $i < 200; $i++) { $h{$i + 0} = $i + 0; }
 for ($i = 1; $i < 200; $i++) { $ok = 0 unless $h{$i} == $i; }
-print ($ok ? "ok 8\n" : "not ok 8\n");
+is($ok, 1, 'check cache overflow and numeric keys and contents');
 
 my ($dev,$ino,$mode,$nlink,$uid,$gid,$rdev,$size,$atime,$mtime,$ctime,
    $blksize,$blocks) = stat($Dfile);
-print ($size > 0 ? "ok 9\n" : "not ok 9\n");
+cmp_ok($size, '>', 0);
 
 @h{0..200} = 200..400;
 my @foo = @h{0..200};
-print join(':',200..400) eq join(':',@foo) ? "ok 10\n" : "not ok 10\n";
+is(join(':',200..400), join(':',@foo));
 
-print ($h{'foo'} eq '' ? "ok 11\n" : "not ok 11\n");
-print ($h{''} eq 'bar' ? "ok 12\n" : "not ok 12\n");
+is($h{'foo'}, '');
+is($h{''}, 'bar');
 
 
 {
@@ -183,22 +173,22 @@ EOM
     BEGIN { push @INC, '.'; }
 
     eval 'use SubDB ; use Fcntl ;';
-    main::ok(13, $@ eq "") ;
+    main::is($@, "");
     my %h ;
     my $X ;
     eval '
 	$X = tie(%h, "SubDB","dbhash_tmp", O_RDWR|O_CREAT, 0640 );
 	' ;
 
-    main::ok(14, $@ eq "") ;
+    main::is($@, "");
 
     my $ret = eval '$h{"fred"} = 3 ; return $h{"fred"} ' ;
-    main::ok(15, $@ eq "") ;
-    main::ok(16, $ret == 5) ;
+    main::is($@, "");
+    main::is($ret, 5);
 
     $ret = eval '$X->A_new_method("fred") ' ;
-    main::ok(17, $@ eq "") ;
-    main::ok(18, $ret eq "[[5]]") ;
+    main::is($@, "");
+    main::is($ret, "[[5]]");
 
     undef $X;
     untie(%h);
@@ -206,8 +196,8 @@ EOM
 
 }
 
-ok(19, !exists $h{'goner1'});
-ok(20, exists $h{'foo'});
+is(exists $h{goner1}, '');
+is(exists $h{foo}, 1);
 
 untie %h;
 unlink <Op_dbmx*>, $Dfile;
@@ -227,7 +217,8 @@ unlink <Op_dbmx*>, $Dfile;
    }
    
    unlink <Op_dbmx*>;
-   ok(21, $db = tie(%h, 'SDBM_File','Op_dbmx', O_RDWR|O_CREAT, 0640)) ;
+   $db = tie %h, 'SDBM_File', 'Op_dbmx', O_RDWR|O_CREAT, 0640;
+   isa_ok($db, 'SDBM_File');
 
    $db->filter_fetch_key   (sub { $fetch_key = $_ }) ;
    $db->filter_store_key   (sub { $store_key = $_ }) ;
@@ -238,17 +229,17 @@ unlink <Op_dbmx*>, $Dfile;
 
    $h{"fred"} = "joe" ;
    #                   fk   sk     fv   sv
-   ok(22, checkOutput( "", "fred", "", "joe")) ;
+   ok(checkOutput("", "fred", "", "joe"));
 
    ($fetch_key, $store_key, $fetch_value, $store_value) = ("") x 4 ;
-   ok(23, $h{"fred"} eq "joe");
+   is($h{"fred"}, "joe");
    #                   fk    sk     fv    sv
-   ok(24, checkOutput( "", "fred", "joe", "")) ;
+   ok(checkOutput("", "fred", "joe", ""));
 
    ($fetch_key, $store_key, $fetch_value, $store_value) = ("") x 4 ;
-   ok(25, $db->FIRSTKEY() eq "fred") ;
+   is($db->FIRSTKEY(), "fred");
    #                    fk     sk  fv  sv
-   ok(26, checkOutput( "fred", "", "", "")) ;
+   ok(checkOutput("fred", "", "", ""));
 
    # replace the filters, but remember the previous set
    my ($old_fk) = $db->filter_fetch_key   
@@ -263,17 +254,17 @@ unlink <Op_dbmx*>, $Dfile;
    ($fetch_key, $store_key, $fetch_value, $store_value) = ("") x 4 ;
    $h{"Fred"} = "Joe" ;
    #                   fk   sk     fv    sv
-   ok(27, checkOutput( "", "fred", "", "Jxe")) ;
+   ok(checkOutput("", "fred", "", "Jxe"));
 
    ($fetch_key, $store_key, $fetch_value, $store_value) = ("") x 4 ;
-   ok(28, $h{"Fred"} eq "[Jxe]");
+   is($h{"Fred"}, "[Jxe]");
    #                   fk   sk     fv    sv
-   ok(29, checkOutput( "", "fred", "[Jxe]", "")) ;
+   ok(checkOutput("", "fred", "[Jxe]", ""));
 
    ($fetch_key, $store_key, $fetch_value, $store_value) = ("") x 4 ;
-   ok(30, $db->FIRSTKEY() eq "FRED") ;
+   is($db->FIRSTKEY(), "FRED");
    #                   fk   sk     fv    sv
-   ok(31, checkOutput( "FRED", "", "", "")) ;
+   ok(checkOutput("FRED", "", "", ""));
 
    # put the original filters back
    $db->filter_fetch_key   ($old_fk);
@@ -283,15 +274,15 @@ unlink <Op_dbmx*>, $Dfile;
 
    ($fetch_key, $store_key, $fetch_value, $store_value) = ("") x 4 ;
    $h{"fred"} = "joe" ;
-   ok(32, checkOutput( "", "fred", "", "joe")) ;
+   ok(checkOutput("", "fred", "", "joe"));
 
    ($fetch_key, $store_key, $fetch_value, $store_value) = ("") x 4 ;
-   ok(33, $h{"fred"} eq "joe");
-   ok(34, checkOutput( "", "fred", "joe", "")) ;
+   is($h{"fred"}, "joe");
+   ok(checkOutput("", "fred", "joe", ""));
 
    ($fetch_key, $store_key, $fetch_value, $store_value) = ("") x 4 ;
-   ok(35, $db->FIRSTKEY() eq "fred") ;
-   ok(36, checkOutput( "fred", "", "", "")) ;
+   is($db->FIRSTKEY(), "fred");
+   ok(checkOutput("fred", "", "", ""));
 
    # delete the filters
    $db->filter_fetch_key   (undef);
@@ -301,15 +292,15 @@ unlink <Op_dbmx*>, $Dfile;
 
    ($fetch_key, $store_key, $fetch_value, $store_value) = ("") x 4 ;
    $h{"fred"} = "joe" ;
-   ok(37, checkOutput( "", "", "", "")) ;
+   ok(checkOutput("", "", "", ""));
 
    ($fetch_key, $store_key, $fetch_value, $store_value) = ("") x 4 ;
-   ok(38, $h{"fred"} eq "joe");
-   ok(39, checkOutput( "", "", "", "")) ;
+   is($h{"fred"}, "joe");
+   ok(checkOutput("", "", "", ""));
 
    ($fetch_key, $store_key, $fetch_value, $store_value) = ("") x 4 ;
-   ok(40, $db->FIRSTKEY() eq "fred") ;
-   ok(41, checkOutput( "", "", "", "")) ;
+   is($db->FIRSTKEY(), "fred");
+   ok(checkOutput("", "", "", ""));
 
    undef $db ;
    untie %h;
@@ -322,7 +313,8 @@ unlink <Op_dbmx*>, $Dfile;
     my (%h, $db) ;
 
     unlink <Op_dbmx*>;
-    ok(42, $db = tie(%h, 'SDBM_File','Op_dbmx', O_RDWR|O_CREAT, 0640)) ;
+    $db = tie %h, 'SDBM_File', 'Op_dbmx', O_RDWR|O_CREAT, 0640;
+    isa_ok($db, 'SDBM_File');
 
     my %result = () ;
 
@@ -346,32 +338,32 @@ unlink <Op_dbmx*>, $Dfile;
     $_ = "original" ;
 
     $h{"fred"} = "joe" ;
-    ok(43, $result{"store key"} eq "store key - 1: [fred]");
-    ok(44, $result{"store value"} eq "store value - 1: [joe]");
-    ok(45, !defined $result{"fetch key"} );
-    ok(46, !defined $result{"fetch value"} );
-    ok(47, $_ eq "original") ;
-
-    ok(48, $db->FIRSTKEY() eq "fred") ;
-    ok(49, $result{"store key"} eq "store key - 1: [fred]");
-    ok(50, $result{"store value"} eq "store value - 1: [joe]");
-    ok(51, $result{"fetch key"} eq "fetch key - 1: [fred]");
-    ok(52, ! defined $result{"fetch value"} );
-    ok(53, $_ eq "original") ;
+    is($result{"store key"}, "store key - 1: [fred]");
+    is($result{"store value"}, "store value - 1: [joe]");
+    is($result{"fetch key"}, undef);
+    is($result{"fetch value"}, undef);
+    is($_, "original");
+
+    is($db->FIRSTKEY(), "fred");
+    is($result{"store key"}, "store key - 1: [fred]");
+    is($result{"store value"}, "store value - 1: [joe]");
+    is($result{"fetch key"}, "fetch key - 1: [fred]");
+    is($result{"fetch value"}, undef);
+    is($_, "original");
 
     $h{"jim"}  = "john" ;
-    ok(54, $result{"store key"} eq "store key - 2: [fred jim]");
-    ok(55, $result{"store value"} eq "store value - 2: [joe john]");
-    ok(56, $result{"fetch key"} eq "fetch key - 1: [fred]");
-    ok(57, ! defined $result{"fetch value"} );
-    ok(58, $_ eq "original") ;
-
-    ok(59, $h{"fred"} eq "joe");
-    ok(60, $result{"store key"} eq "store key - 3: [fred jim fred]");
-    ok(61, $result{"store value"} eq "store value - 2: [joe john]");
-    ok(62, $result{"fetch key"} eq "fetch key - 1: [fred]");
-    ok(63, $result{"fetch value"} eq "fetch value - 1: [joe]");
-    ok(64, $_ eq "original") ;
+    is($result{"store key"}, "store key - 2: [fred jim]");
+    is($result{"store value"}, "store value - 2: [joe john]");
+    is($result{"fetch key"}, "fetch key - 1: [fred]");
+    is($result{"fetch value"}, undef);
+    is($_, "original");
+
+    is($h{"fred"}, "joe");
+    is($result{"store key"}, "store key - 3: [fred jim fred]");
+    is($result{"store value"}, "store value - 2: [joe john]");
+    is($result{"fetch key"}, "fetch key - 1: [fred]");
+    is($result{"fetch value"}, "fetch value - 1: [joe]");
+    is($_, "original");
 
     undef $db ;
     untie %h;
@@ -383,12 +375,13 @@ unlink <Op_dbmx*>, $Dfile;
    my (%h, $db) ;
    unlink <Op_dbmx*>;
 
-   ok(65, $db = tie(%h, 'SDBM_File','Op_dbmx', O_RDWR|O_CREAT, 0640)) ;
+   $db = tie %h, 'SDBM_File', 'Op_dbmx', O_RDWR|O_CREAT, 0640;
+   isa_ok($db, 'SDBM_File');
 
    $db->filter_store_key (sub { $_ = $h{$_} }) ;
 
    eval '$h{1} = 1234' ;
-   ok(66, $@ =~ /^recursion detected in filter_store_key at/ );
+   like($@, qr/^recursion detected in filter_store_key at/);
    
    undef $db ;
    untie %h;
@@ -406,10 +399,9 @@ unlink <Op_dbmx*>, $Dfile;
     my $a = "";
     local $SIG{__WARN__} = sub {$a = $_[0]} ;
     
-    ok(67, tie(%h, 'SDBM_File','Op_dbmx', O_RDWR|O_CREAT, 0640)) ;
+    isa_ok(tie(%h, 'SDBM_File', 'Op_dbmx', O_RDWR|O_CREAT, 0640), 'SDBM_File');
     $h{ABC} = undef;
-    ok(68, $a eq "") ;
-
+    is($a, "");
     untie %h;
     unlink <Op_dbmx*>;
 }
@@ -424,27 +416,28 @@ unlink <Op_dbmx*>, $Dfile;
     unlink <Op_dbmx*>;
     my $bad_key = 0 ;
     my %h = () ;
-    ok(69, my $db = tie(%h, 'SDBM_File','Op_dbmx', O_RDWR|O_CREAT, 0640)) ;
+    my $db = tie %h, 'SDBM_File','Op_dbmx', O_RDWR|O_CREAT, 0640;
+    isa_ok($db, 'SDBM_File');
     $db->filter_fetch_key (sub { $_ =~ s/^Beta_/Alpha_/ if defined $_}) ;
     $db->filter_store_key (sub { $bad_key = 1 if /^Beta_/ ; $_ =~ s/^Alpha_/Beta_/}) ;
 
     $h{'Alpha_ABC'} = 2 ;
     $h{'Alpha_DEF'} = 5 ;
 
-    ok(70, $h{'Alpha_ABC'} == 2);
-    ok(71, $h{'Alpha_DEF'} == 5);
+    is($h{'Alpha_ABC'}, 2);
+    is($h{'Alpha_DEF'}, 5);
 
     my ($k, $v) = ("","");
     while (($k, $v) = each %h) {}
-    ok(72, $bad_key == 0);
+    is($bad_key, 0);
 
     $bad_key = 0 ;
     foreach $k (keys %h) {}
-    ok(73, $bad_key == 0);
+    is($bad_key, 0);
 
     $bad_key = 0 ;
     foreach $v (values %h) {}
-    ok(74, $bad_key == 0);
+    is($bad_key, 0);
 
     undef $db ;
     untie %h ;
@@ -458,7 +451,8 @@ unlink <Op_dbmx*>, $Dfile;
    my %h ;
    unlink <Op1_dbmx*>;
 
-   ok(75, my $db = tie(%h, 'SDBM_File','Op1_dbmx', O_RDWR|O_CREAT, 0640)) ;
+   my $db = tie %h, 'SDBM_File', 'Op1_dbmx', O_RDWR|O_CREAT, 0640;
+   isa_ok($db, 'SDBM_File');
 
    $db->filter_fetch_key   (sub { }) ;
    $db->filter_store_key   (sub { }) ;
@@ -468,10 +462,10 @@ unlink <Op_dbmx*>, $Dfile;
    $_ = "original" ;
 
    $h{"fred"} = "joe" ;
-   ok(76, $h{"fred"} eq "joe");
+   is($h{"fred"}, "joe");
 
-   eval { map { $h{$_} } (1, 2, 3) };
-   ok (77, ! $@);
+   eval { grep { $h{$_} } (1, 2, 3) };
+   is($@, '');
 
 
    # delete the filters
@@ -482,15 +476,14 @@ unlink <Op_dbmx*>, $Dfile;
 
    $h{"fred"} = "joe" ;
 
-   ok(78, $h{"fred"} eq "joe");
+   is($h{"fred"}, "joe");
 
-   ok(79, $db->FIRSTKEY() eq "fred") ;
+   is($db->FIRSTKEY(), "fred");
    
    eval { map { $h{$_} } (1, 2, 3) };
-   ok (80, ! $@);
+   is($@, '');
 
    undef $db ;
    untie %h;
    unlink <Op1_dbmx*>;
 }
-exit ;