--- trunk/t/3-normalize.t	2006/07/02 20:14:21	564
+++ trunk/t/3-normalize.t	2006/07/03 21:08:07	574
@@ -2,7 +2,7 @@
 
 use strict;
 
-use Test::More tests => 103;
+use Test::More tests => 140;
 use Test::Exception;
 use Cwd qw/abs_path/;
 use blib;
@@ -150,9 +150,10 @@
 	$eval_t = substr($eval_t,0,$max_eval_output) . '...' if (length($eval_t) > $max_eval_output);
 	$eval_t =~ s/\\/\\\\/gs;
 
-	my $v = eval "$t";
-	ok(! $@, $@ ? dump_error($@, $t) : "eval: $eval_t");
-	$v;
+	my @__ret;
+	eval "\@__ret = $t";
+	ok(! $@, $@ ? dump_error($@, $t) : "eval: $eval_t = " . dump(@__ret));
+	return \@__ret;
 }
 
 {
@@ -359,6 +360,7 @@
 	#
 	# MARC
 	#
+	_debug( 4 );
 
 	test_s(qq{ marc_indicators('900',1,2) });
 	test_s(qq{ marc('900','a', rec('200') ) });
@@ -496,13 +498,25 @@
 		'split_rec_on',
 		{ '200' => [{ a => 'foo ; bar', b => 42, c => 'baz' }] },
 		qq{ split_rec_on('200','a', qr/\\s*;\\s*/, 1) },
-		'foo',
+		[ 'foo' ],
 	);
 	test_rule(
 		'split_rec_on',
 		{ '200' => [{ a => 'foo ; bar', b => 42, c => 'baz' }] },
 		qq{ split_rec_on('200','a', qr/\\s*;\\s*/, 2) },
-		'bar',
+		[ 'bar' ],
+	);
+	test_rule(
+		'split_rec_on no part',
+		{ '200' => [{ a => 'foo ; bar', b => 42, c => 'baz' }] },
+		qq{ split_rec_on('200','a', qr/\\s*;\\s*/) },
+		[ 'foo', 'bar' ],
+	);
+	test_rule(
+		'split_rec_on no record',
+		{},
+		qq{ split_rec_on('200','a', qr/\\s*;\\s*/) },
+		[ '' ],
 	);
 
 	test_rec_rules(
@@ -531,5 +545,74 @@
 	ok(marc_leader(), 'marc_leader get');
 	diag "leader: ", dump(marc_leader()) if ($debug);
 	is_deeply(marc_leader(), { '06' => 42, 11 => 5 }, "marc_leader full");
+
+	test_rule(
+		'rec1(000)',
+		{ '000' => [ 42 ]},
+		qq{ rec('000') },
+		[ 42 ],
+	);
+
+	test_rec_rules(
+		'marc(001,rec(000))',
+		{ '000' => [ 42 ]},
+		qq{
+			marc('001', rec('000') );
+		},
+		[
+			[ '001', 42, ]
+		],
+	);
+
+	test_rec_rules(
+		'marc_remove',
+		{ '200' => [{ a => 42, b => 'bar', c => 'baz' }] },
+		qq{
+			marc('900', 'a', rec('200','a') );
+			marc('900', 'b', rec('200','b') );
+			marc_remove('900','b');
+			marc('900', 'b', rec('200','c') );
+			marc_remove('900','a');
+		},
+		[
+			[ '900', ' ', ' ', 'b', 'baz' ],
+		],
+	);
+
+	test_rec_rules(
+		'marc_duplicate',
+		{ '200' => [{ a => 42, b => 'bar', c => 'baz', d => 'bing', e => 'bong' }] },
+		qq{
+			marc('900', 'a', rec('200','a') );
+			marc('900', 'b', rec('200','b') );
+			marc_duplicate;
+			marc_remove('900','b');
+			marc('900', 'b', rec('200','c') );
+			marc_duplicate;
+			marc_remove('900','b');
+			marc('900', 'b', rec('200','d') );
+			marc_duplicate;
+			marc_remove('900','b');
+			marc('900', 'b', rec('200','e') );
+		},
+		[
+			# this will return FIRST record
+			[ '900', ' ', ' ', 'a', 42, 'b', 'bar' ],
+		],
+	);
+
+	my $i = 0;
+	foreach my $v ( qw/bar baz bing bong/ ) {
+
+		ok(@marc = WebPAC::Normalize::_get_marc_fields( offset => $i ),
+			"_get_marc_fields( offset => $i )"
+		);
+		diag "marc $i = ", dump( @marc ) if ($debug);
+		is_deeply( \@marc,
+			[ [ '900', ' ', ' ', 'a', 42, 'b', $v ] ],
+			"MARC copy $i has $v",
+		);
+		$i++;
+	}
 }