much more sane implementation of to_hash which now include

[MARC-Fast] / t / 001_marc.t

diff --git a/t/001_marc.t b/t/001_marc.t
index 990d078..c05fcd6 100755 (executable)
--- a/t/001_marc.t
+++ b/t/001_marc.t
@@ -3,7 +3,7 @@
 use strict;
 use blib;
 
-use Test::More tests => 53;
+use Test::More tests => 63;
 use Test::Exception;
 
 BEGIN {
@@ -11,6 +11,9 @@ BEGIN {
 }
 
 my $debug = shift @ARGV;
+
+my $marc_file = 't/camel.usmarc';
+
 if ( $debug ) {
        eval { require Data::Dump; };
        $debug = 0 if ($@);
@@ -25,11 +28,13 @@ $param{marcdb} = '/foo/bar/file';
 
 throws_ok { $marc = MARC::Fast->new(%param); } qr/foo.bar/, "marcdb exist";
 
-$param{marcdb} = 'data/unimarc.iso';
+$param{marcdb} = $marc_file if -e $marc_file;
 
 SKIP: {
        skip "no $param{marcdb} test file ", 37 unless (-e $param{marcdb});
 
+       diag "marc file: $marc_file";
+
        ok($marc = MARC::Fast->new(%param), "new");
 
        isa_ok ($marc, 'MARC::Fast');
@@ -45,16 +50,20 @@ SKIP: {
        ok(! $marc->fetch($marc->count + 1), "fetch max+1:".($marc->count+1));
 
        foreach (1 .. 10) {
-               ok($marc->fetch($_), "fetch $_");
+               ok($marc->fetch($_), "fetch($_)");
 
                ok($marc->last_leader, "last_leader $_");
 
-               ok(my $hash = $marc->to_hash($_), "to_hash $_");
+               ok(my $hash = $marc->to_hash($_), "to_hash($_)");
                diag "to_hash($_) = ",Data::Dump::dump($hash) if ($debug);
-               ok(my $ascii = $marc->to_ascii($_), "to_ascii $_");
+
+               ok(my $hash_sf = $marc->to_hash($_, include_subfields => 1), "to_hash($_,include_subfields)");
+               diag "to_hash($_, include_subfields => 1) = ",Data::Dump::dump($hash_sf) if ($debug);
+
+               ok(my $ascii = $marc->to_ascii($_), "to_ascii($_)");
                diag "to_ascii($_) ::\n$ascii" if ($debug);
        }
 
-       ok(! $marc->fetch(0), "fetch 0 again");
+       ok(! $marc->fetch(0), "fetch(0) again");
        ok(! $marc->last_leader, "no last_leader");
 }