Converge ext/[GNOS]DBM_File/t/[gnos]dbm.t by using the same filename.

Choose the 1 dot form used by sdbm.t, to keep VMS happy. Also, propagate into
ndbm.t the part of the test for 20001013.009 that cbc5248d01a71061 missed.
Move the exist tests from f4b9d8806d76b352 earlier in sdbm.t, to increase
consistency - the alternative attempts to have 2 DBM files open simultaneously,
which ODBM_File doesn't support. (Implied TODO: add an explicit test for this
to the other 3.)

1 files changed, 35 insertions, 27 deletions

diff --git a/ext/NDBM_File/t/ndbm.t b/ext/NDBM_File/t/ndbm.t
index 8bbd2933b8..7a2ae7069c 100644
--- a/ext/NDBM_File/t/ndbm.t
+++ b/ext/NDBM_File/t/ndbm.t
@@ -13,21 +13,21 @@ BEGIN {
 use strict;
 use warnings;
 
-use Test::More tests => 78;
+use Test::More tests => 79;
 
 require NDBM_File;
 #If Fcntl is not available, try 0x202 or 0x102 for O_RDWR|O_CREAT
 use Fcntl;
 
-unlink <Op.dbmx*>;
+unlink <Op_dbmx.*>;
 
 umask(0);
 my %h;
-isa_ok(tie(%h,'NDBM_File','Op.dbmx', O_RDWR|O_CREAT, 0640), 'NDBM_File');
+isa_ok(tie(%h,'NDBM_File','Op_dbmx', O_RDWR|O_CREAT, 0640), 'NDBM_File');
 
-my $Dfile = "Op.dbmx.pag";
+my $Dfile = "Op_dbmx.pag";
 if (! -e $Dfile) {
-	($Dfile) = <Op.dbmx*>;
+	($Dfile) = <Op_dbmx*>;
 }
 SKIP: {
     skip "different file permission semantics on $^O", 1
@@ -62,7 +62,7 @@ $h{'goner2'} = 'snork';
 delete $h{'goner2'};
 
 untie(%h);
-isa_ok(tie(%h,'NDBM_File','Op.dbmx', O_RDWR, 0640), 'NDBM_File');
+isa_ok(tie(%h,'NDBM_File','Op_dbmx', O_RDWR, 0640), 'NDBM_File');
 
 $h{'j'} = 'J';
 $h{'k'} = 'K';
@@ -125,7 +125,7 @@ is($h{'foo'}, '');
 is($h{''}, 'bar');
 
 untie %h;
-unlink <Op.dbmx*>, $Dfile;
+unlink <Op_dbmx*>, $Dfile;
 
 {
    # sub-class test
@@ -173,13 +173,14 @@ EOM
     close FILE ;
 
     BEGIN { push @INC, '.'; }
+    unlink <dbhash_tmp*> ;
 
     eval 'use SubDB ; use Fcntl ; ';
     main::is($@, "");
     my %h ;
     my $X ;
     eval '
-	$X = tie(%h, "SubDB","dbhash.tmp", O_RDWR|O_CREAT, 0640 );
+	$X = tie(%h, "SubDB","dbhash_tmp", O_RDWR|O_CREAT, 0640 );
 	' ;
 
     main::is($@, "");
@@ -194,10 +195,13 @@ EOM
 
     undef $X;
     untie(%h);
-    unlink "SubDB.pm", <dbhash.tmp*> ;
+    unlink "SubDB.pm", <dbhash_tmp.*> ;
 
 }
 
+untie %h;
+unlink <Op_dbmx*>, $Dfile;
+
 {
    # DBM Filter tests
    my (%h, $db) ;
@@ -212,8 +216,8 @@ EOM
 	   $_ eq 'original' ;
    }
    
-   unlink <Op.dbmx*>;
-   $db = tie %h, 'NDBM_File', 'Op.dbmx', O_RDWR|O_CREAT, 0640;
+   unlink <Op_dbmx*>;
+   $db = tie %h, 'NDBM_File', 'Op_dbmx', O_RDWR|O_CREAT, 0640;
    isa_ok($db, 'NDBM_File');
 
    $db->filter_fetch_key   (sub { $fetch_key = $_ }) ;
@@ -300,7 +304,7 @@ EOM
 
    undef $db ;
    untie %h;
-   unlink <Op.dbmx*>;
+   unlink <Op_dbmx*>;
 }
 
 {    
@@ -308,8 +312,8 @@ EOM
 
     my (%h, $db) ;
 
-    unlink <Op.dbmx*>;
-    $db = tie %h, 'NDBM_File', 'Op.dbmx', O_RDWR|O_CREAT, 0640;
+    unlink <Op_dbmx*>;
+    $db = tie %h, 'NDBM_File', 'Op_dbmx', O_RDWR|O_CREAT, 0640;
     isa_ok($db, 'NDBM_File');
 
     my %result = () ;
@@ -363,15 +367,15 @@ EOM
 
     undef $db ;
     untie %h;
-    unlink <Op.dbmx*>;
+    unlink <Op_dbmx*>;
 }		
 
 {
    # DBM Filter recursion detection
    my (%h, $db) ;
-   unlink <Op.dbmx*>;
+   unlink <Op_dbmx*>;
 
-   $db = tie %h, 'NDBM_File', 'Op.dbmx', O_RDWR|O_CREAT, 0640;
+   $db = tie %h, 'NDBM_File', 'Op_dbmx', O_RDWR|O_CREAT, 0640;
    isa_ok($db, 'NDBM_File');
 
    $db->filter_store_key (sub { $_ = $h{$_} }) ;
@@ -381,7 +385,7 @@ EOM
    
    undef $db ;
    untie %h;
-   unlink <Op.dbmx*>;
+   unlink <Op_dbmx*>;
 }
 
 {
@@ -390,12 +394,16 @@ EOM
     # test that $hash{KEY} = undef doesn't produce the warning
     #     Use of uninitialized value in null operation 
 
-    unlink <Op.dbmx*>;
+    unlink <Op_dbmx*>;
     my %h ;
     my $a = "";
     local $SIG{__WARN__} = sub {$a = $_[0]} ;
-    
-    isa_ok(tie(%h, 'NDBM_File','Op.dbmx', O_RDWR|O_CREAT, 0640), 'NDBM_File');
+
+    isa_ok(tie(%h, 'NDBM_File','Op_dbmx', O_RDWR|O_CREAT, 0640), 'NDBM_File');
+    $h{ABC} = undef;
+    is($a, "");
+    untie %h;
+    unlink <Op_dbmx*>;
 }
 
 {
@@ -405,10 +413,10 @@ EOM
     # modified key doesn't get passed to NEXTKEY.
     # Also Test "keys" & "values" while we are at it.
 
-    unlink <Op.dbmx*>;
+    unlink <Op_dbmx*>;
     my $bad_key = 0 ;
     my %h = () ;
-    my $db = tie %h, 'NDBM_File','Op.dbmx', O_RDWR|O_CREAT, 0640;
+    my $db = tie %h, 'NDBM_File','Op_dbmx', O_RDWR|O_CREAT, 0640;
     isa_ok($db, 'NDBM_File');
     $db->filter_fetch_key (sub { $_ =~ s/^Beta_/Alpha_/ if defined $_}) ;
     $db->filter_store_key (sub { $bad_key = 1 if /^Beta_/ ; $_ =~ s/^Alpha_/Beta_/}) ;
@@ -433,7 +441,7 @@ EOM
 
     undef $db ;
     untie %h ;
-    unlink <Op.dbmx*>;
+    unlink <Op_dbmx*>;
 }
 
 
@@ -441,9 +449,9 @@ EOM
    # Check that DBM Filter can cope with read-only $_
 
    my %h ;
-   unlink <Op.dbmx*>;
+   unlink <Op1_dbmx*>;
 
-   my $db = tie %h, 'NDBM_File','Op.dbmx', O_RDWR|O_CREAT, 0640;
+   my $db = tie %h, 'NDBM_File','Op1_dbmx', O_RDWR|O_CREAT, 0640;
    isa_ok($db, 'NDBM_File');
 
    $db->filter_fetch_key   (sub { }) ;
@@ -477,5 +485,5 @@ EOM
 
    undef $db ;
    untie %h;
-   unlink <Op.dbmx*>;
+   unlink <Op1_dbmx*>;
 }