--- trunk/t/1-validate.t	2006/09/11 11:57:30	665
+++ trunk/t/1-validate.t	2006/09/11 14:29:08	670
@@ -1,7 +1,7 @@
 #!/usr/bin/perl -w
 
 use strict;
-use Test::More tests => 51;
+use Test::More tests => 47;
 use Test::Exception;
 use blib;
 
@@ -21,6 +21,7 @@
 
 ok(my $v = new WebPAC::Validate(
 	path => "$abs_path/data/validate_test",
+	debug => $debug,
 ), "new");
 
 ok($v->{rules}, "rules exist");
@@ -54,7 +55,7 @@
 		my $tmp = $e;
 		while (@_) {
 			my $k = shift @_;
-			ok($tmp = $tmp->{$k}, "found $k");
+			ok($tmp = $tmp->{$k}, "found $k") if (defined($k));
 		}
 		diag "tmp: ",dump($tmp) if ($debug);
 		if ($tmp) {
@@ -73,7 +74,7 @@
 
 test_v({
 	'900' => 'foo'
-}, qw/field 900 not_repeatable/);
+}, qw/900 not_repeatable/);
 
 test_v({
 	'900' => [ qw/foo bar baz/ ]
@@ -81,7 +82,7 @@
 
 test_v({
 	'901' => [ qw/foo bar baz/ ]
-}, qw/field 901 missing_subfield/);
+}, qw/901 missing_subfield/);
 
 test_v({
 	'901' => [ { 'a' => 42 } ]
@@ -89,7 +90,7 @@
 
 test_v({
 	'901' => [ { 'b' => 42 } ]
-}, qw/field 901 subfield extra b/);
+}, qw/901 subfield extra b/);
 
 test_v({
 	'902' => [ { 'b' => 1 }, { 'c' => 2 } ]
@@ -97,7 +98,7 @@
 
 test_v({
 	'902' => [ { 'a' => 0 }, { 'b' => 1 }, { 'c' => 2 } ]
-}, qw/field 902 subfield extra a/);
+}, qw/902 subfield extra a/);
 
 test_v({
 	'903' => [ { 'a' => 0 }, { 'b' => 1 }, { 'c' => 2 } ]
@@ -105,13 +106,12 @@
 
 test_v({
 	'903' => [ { 'a' => 0 }, { 'b' => 1 }, { 'c' => 2 }, { 'd' => 3 } ]
-}, qw/field 903 subfield extra d/);
+}, qw/903 subfield extra d/);
 
 is_deeply(
-
-test_v({
-	'903' => [ { 'a' => 0 }, { 'b' => 1 }, { 'c' => 2 }, { 'd' => 3 }, { 'e' => 4 } ]
-}, qw/field 903 subfield extra/),
+	test_v({
+		'903' => [ { 'a' => 0 }, { 'b' => 1 }, { 'c' => 2 }, { 'd' => 3 }, { 'e' => 4 } ]
+	}, qw/903 subfield extra/),
 { 'd' => 1, 'e' => 1 }, 'additional fields d, e');
 
 test_v({
@@ -120,11 +120,11 @@
 
 test_v({
 	'904' => [ { 'b' => 1 } ]
-}, qw/field 904 subfield extra b/);
+}, qw/904 subfield extra b/);
 
 test_v({
 	'904' => [ { 'a' => [ 1,2 ] } ]
-}, qw/field 904 subfield extra_repeatable a/);
+}, qw/904 subfield extra_repeatable a/);
 
 test_v({
 	'905' => [ { 'a' => [ 1,2 ] } ]
@@ -134,15 +134,46 @@
 	'905' => [ ]
 });
 
-test_v({
-	'900' => 'foo',
-	'901' => [ qw/foo bar baz/ ],
-	'902' => [ { 'a' => 1, 'b' => [ 1,2 ] } ],
-	'903' => [ { 'a' => [ 1, 2 ], 'c' => 1, } ],
-	'904' => [ { 'b' => 1 } ],
-	'905' => [ { 'a' => 1 } ],
-});
+my $expected_error = {
+   900 => { not_repeatable => "probably bug in parsing input data" },
+   901 => { missing_subfield => "a required" },
+   902 => {
+            "dump"   => "^a1^b1^b2",
+            subfield => { extra => { a => 1 }, extra_repeatable => { b => 1 } },
+          },
+   903 => {
+            "dump"   => "^a1^a2^c1",
+            subfield => { extra_repeatable => { a => 1 } },
+          },
+   904 => { subfield => { extra => { b => 1 }, missing => { a => 1 } } },
+};
+
+
+is_deeply(
+	test_v({
+		'900' => 'foo',
+		'901' => [ qw/foo bar baz/ ],
+		'902' => [ { 'a' => 1, 'b' => [ 1,2 ] } ],
+		'903' => [ { 'a' => [ 1, 2 ], 'c' => 1, } ],
+		'904' => [ { 'b' => 1 } ],
+		'905' => [ { 'a' => 1 } ],
+	}, undef),
+$expected_error, 'validate without subfields');
+
+ok(my $r1 = $v->report, 'report');
+
+is_deeply(
+	test_v({
+		'900' => 'foo',
+		'901' => [ qw/foo bar baz/ ],
+		'902' => [ { 'a' => 1, 'b' => [ 1,2 ], subfields => [ qw/a 0 b 0 b 1/ ] } ],
+		'903' => [ { 'a' => [ 1, 2 ], 'c' => 1, subfields => [ qw/a 0 a 1 c 0/ ] } ],
+		'904' => [ { 'b' => 1, subfields => [ qw/b 0/ ] } ],
+		'905' => [ { 'a' => 1, subfields => [ qw/a 0/ ] } ],
+	}, undef),
+$expected_error, 'validate with subfields');
+
 
-diag "errors: ",dump( $v->all_errors );
+ok(my $r2 = $v->report, 'report');
 
-print $v->report;
+cmp_ok($r1, 'eq', $r2, 'subfields same as non-subfields');