Added guard to make sure values that cannot be read correctly aren't stored, plus...

[dbsrgits/DBM-Deep.git] / t / 26_scalar_ref.t

diff --git a/t/26_scalar_ref.t b/t/26_scalar_ref.t
index 405c6fe..7a77c39 100644 (file)
--- a/t/26_scalar_ref.t
+++ b/t/26_scalar_ref.t
@@ -1,6 +1,7 @@
 use strict;
 
-use Test::More tests => 7;
+use Test::More tests => 10;
+use Test::Exception;
 use File::Temp qw( tempfile tempdir );
 
 use_ok( 'DBM::Deep' );
@@ -8,32 +9,48 @@ use_ok( 'DBM::Deep' );
 my $dir = tempdir( CLEANUP => 1 );
 my ($fh, $filename) = tempfile( 'tmpXXXX', UNLINK => 1, DIR => $dir );
 
+my $x = 25;
 {
     my $db = DBM::Deep->new( $filename );
 
-    my $x = 25;
-    my $y = 30;
-    $db->{scalar} = $x;
-    $db->{scalarref} = \$y;
-    $db->{selfref} = \$x;
+    throws_ok {
+        $db->{scalarref} = \$x;
+    } qr/Storage of variables of type 'SCALAR' is not supported/,
+    'Storage of scalar refs not supported';
 
-    is( $db->{scalar}, $x, "Scalar retrieved ok" );
+    throws_ok {
+        $db->{scalarref} = \\$x;
+    } qr/Storage of variables of type 'REF' is not supported/,
+    'Storage of ref refs not supported';
+
+    throws_ok {
+        $db->{scalarref} = sub { 1 };
+    } qr/Storage of variables of type 'CODE' is not supported/,
+    'Storage of code refs not supported';
+
+    throws_ok {
+        $db->{scalarref} = $db->_get_self->_fh;
+    } qr/Storage of variables of type 'GLOB' is not supported/,
+    'Storage of glob refs not supported';
+
+    $db->{scalar} = $x;
     TODO: {
-        todo_skip "Scalar refs aren't implemented yet", 2;
-        is( ${$db->{scalarref}}, 30, "Scalarref retrieved ok" );
-        is( ${$db->{selfref}}, 25, "Scalarref to stored scalar retrieved ok" );
+        todo_skip "Refs to DBM::Deep objects aren't implemented yet", 2;
+        lives_ok {
+            $db->{selfref} = \$db->{scalar};
+        } "Refs to DBM::Deep objects are ok";
+
+        is( ${$db->{selfref}}, $x, "A ref to a DBM::Deep object is ok" );
     }
 }
 
 {
     my $db = DBM::Deep->new( $filename );
 
-    my $x = 25;
-    my $y = 30;
     is( $db->{scalar}, $x, "Scalar retrieved ok" );
     TODO: {
-        todo_skip "Scalar refs aren't implemented yet", 2;
+        todo_skip "Refs to DBM::Deep objects aren't implemented yet", 2;
         is( ${$db->{scalarref}}, 30, "Scalarref retrieved ok" );
-        is( ${$db->{selfref}}, 25, "Scalarref to stored scalar retrieved ok" );
+        is( ${$db->{selfref}}, 26, "Scalarref to stored scalar retrieved ok" );
     }
 }