# DBM::Deep Test
##
use strict;
-use Test::More tests => 2;
-use File::Temp qw( tempfile tempdir );
-use Fcntl qw( :flock );
+use Test::More tests => 17;
+use Test::Deep;
+use Test::Exception;
+use t::common qw( new_fh );
use_ok( 'DBM::Deep' );
-my $dir = tempdir( CLEANUP => 1 );
-my ($fh, $filename) = tempfile( 'tmpXXXX', UNLINK => 1, DIR => $dir );
-flock $fh, LOCK_UN;
-my $db = DBM::Deep->new( $filename );
+# Failure cases to make sure that things are caught right.
+foreach my $type ( DBM::Deep->TYPE_HASH, DBM::Deep->TYPE_ARRAY ) {
+ my ($fh, $filename) = new_fh();
+ my $db = DBM::Deep->new({
+ file => $filename,
+ fh => $fh,
+ type => $type,
+ });
+
+ # Load a scalar
+ throws_ok {
+ $db->import( 'foo' );
+ } qr/Cannot import a scalar/, "Importing a scalar to type '$type' fails";
+
+ # Load a ref of the wrong type
+ # Load something with bad stuff in it
+ my $x = 3;
+ if ( $type eq 'A' ) {
+ throws_ok {
+ $db->import( { foo => 'bar' } );
+ } qr/Cannot import a hash into an array/, "Wrong type fails";
+
+ throws_ok {
+ $db->import( [ \$x ] );
+ } qr/Storage of references of type 'SCALAR' is not supported/, "Bad stuff fails";
+ }
+ else {
+ throws_ok {
+ $db->import( [ 1 .. 3 ] );
+ } qr/Cannot import an array into a hash/, "Wrong type fails";
+
+ throws_ok {
+ $db->import( { foo => \$x } );
+ } qr/Storage of references of type 'SCALAR' is not supported/, "Bad stuff fails";
+ }
+}
+
+{
+ my ($fh, $filename) = new_fh();
+ my $db = DBM::Deep->new({
+ file => $filename,
+ fh => $fh,
+ autobless => 1,
+ });
##
# Create structure in memory
##
-my $struct = {
- key1 => "value1",
- key2 => "value2",
- array1 => [ "elem0", "elem1", "elem2" ],
- hash1 => {
- subkey1 => "subvalue1",
- subkey2 => "subvalue2"
- }
-};
+ my $struct = {
+ key1 => "value1",
+ key2 => "value2",
+ array1 => [ "elem0", "elem1", "elem2" ],
+ hash1 => {
+ subkey1 => "subvalue1",
+ subkey2 => "subvalue2",
+ subkey3 => bless( { a => 'b' }, 'Foo' ),
+ }
+ };
-##
-# Import entire thing
-##
-$db->import( $struct );
-undef $struct;
+ $db->import( $struct );
-##
-# Make sure everything is there
-##
-ok(
- ($db->{key1} eq "value1") &&
- ($db->{key2} eq "value2") &&
- ($db->{array1} &&
- ($db->{array1}->[0] eq "elem0") &&
- ($db->{array1}->[1] eq "elem1") &&
- ($db->{array1}->[2] eq "elem2")
- ) &&
- ($db->{hash1} &&
- ($db->{hash1}->{subkey1} eq "subvalue1") &&
- ($db->{hash1}->{subkey2} eq "subvalue2")
- )
-);
+ cmp_deeply(
+ $db,
+ noclass({
+ key1 => 'value1',
+ key2 => 'value2',
+ array1 => [ 'elem0', 'elem1', 'elem2', ],
+ hash1 => {
+ subkey1 => "subvalue1",
+ subkey2 => "subvalue2",
+ subkey3 => useclass( bless { a => 'b' }, 'Foo' ),
+ },
+ }),
+ "Everything matches",
+ );
+
+ $struct->{foo} = 'bar';
+ is( $struct->{foo}, 'bar', "\$struct has foo and it's 'bar'" );
+ ok( !exists $db->{foo}, "\$db doesn't have the 'foo' key, so \$struct is not tied" );
+
+ $struct->{hash1}->{foo} = 'bar';
+ is( $struct->{hash1}->{foo}, 'bar', "\$struct->{hash1} has foo and it's 'bar'" );
+ ok( !exists $db->{hash1}->{foo}, "\$db->{hash1} doesn't have the 'foo' key, so \$struct->{hash1} is not tied" );
+}
+
+{
+ my ($fh, $filename) = new_fh();
+ my $db = DBM::Deep->new({
+ file => $filename,
+ fh => $fh,
+ type => DBM::Deep->TYPE_ARRAY,
+ });
+
+ my $struct = [
+ 1 .. 3,
+ [ 2, 4, 6 ],
+ bless( [], 'Bar' ),
+ { foo => [ 2 .. 4 ] },
+ ];
+
+ $db->import( $struct );
+
+ cmp_deeply(
+ $db,
+ noclass([
+ 1 .. 3,
+ [ 2, 4, 6 ],
+ useclass( bless( [], 'Bar' ) ),
+ { foo => [ 2 .. 4 ] },
+ ]),
+ "Everything matches",
+ );
+
+ push @$struct, 'bar';
+ is( $struct->[-1], 'bar', "\$struct has 'bar' at the end" );
+ ok( $db->[-1], "\$db doesn't have the 'bar' value at the end, so \$struct is not tied" );
+}
+
+# Failure case to verify that rollback occurs
+{
+ my ($fh, $filename) = new_fh();
+ my $db = DBM::Deep->new({
+ file => $filename,
+ fh => $fh,
+ autobless => 1,
+ });
+
+ $db->{foo} = 'bar';
+
+ my $x;
+ my $struct = {
+ key1 => [
+ 2, \$x, 3,
+ ],
+ };
+
+ eval {
+ $db->import( $struct );
+ };
+ like( $@, qr/Storage of references of type 'SCALAR' is not supported/, 'Error message correct' );
+
+ TODO: {
+ local $TODO = "Importing cannot occur within a transaction yet.";
+ cmp_deeply(
+ $db,
+ noclass({
+ foo => 'bar',
+ }),
+ "Everything matches",
+ );
+ }
+}
+
+__END__
+
+Need to add tests for:
+ - Failure case (have something tied or a glob or something like that)
+ - Where we already have $db->{hash1} to make sure that it's not overwritten