===================================================================
RCS file: /cvs/palm/Palm-Keyring/t/keyring.t,v
retrieving revision 1.15
retrieving revision 1.20
diff -u -r1.15 -r1.20
--- palm/Palm-Keyring/t/keyring.t	2007/09/12 03:44:36	1.15
+++ palm/Palm-Keyring/t/keyring.t	2008/09/18 07:06:11	1.20
@@ -1,174 +1,332 @@
 #!/usr/bin/perl -T
-# $RedRiver: keyring.t,v 1.14 2007/08/10 04:13:31 andrew Exp $
+# $RedRiver: keyring.t,v 1.19 2008/09/18 02:02:50 andrew Exp $
 use strict;
 use warnings;
 
-use Test::More tests => 52;
+use Test::More tests => 198;
 use Data::Dumper;
 
-BEGIN { 
-    use_ok( 'Palm::PDB' ); 
-    use_ok( 'Palm::Keyring' ); 
+BEGIN {
+    use_ok('Palm::PDB');
+    use_ok('Palm::Keyring');
 }
 
-my $file = 'Keys-test.pdb';
-my $password = '12345';
+my $file         = 'Keys-test.pdb';
+my $password     = '12345';
 my $new_password = '54321';
 
 my @o = (
-    {
-        version  => 4,
+    {   version  => 4,
         password => $password,
     },
-    {
-        version      => 5,
-        password     => $password,
-        cipher       => 1,
+    {   version  => 5,
+        password => $password,
+        cipher   => 1,
     },
 );
 
+my $acct = {
+    0 => {
+        label    => 'name',
+        label_id => 0,
+        data     => 'test3',
+        font     => 0,
+    },
+    1 => {
+        label    => 'account',
+        label_id => 1,
+        data     => 'atestaccount',
+        font     => 0,
+    },
+    2 => {
+        label    => 'password',
+        label_id => 2,
+        data     => $password,
+        font     => 0,
+    },
+    3 => {
+        label    => 'lastchange',
+        label_id => 3,
+        data     => {
+            day   => 2,
+            month => 2,
+            year  => 99,
+        },
+        font => 0,
+    },
+    255 => {
+        label    => 'notes',
+        label_id => 255,
+        data     => 'now that really roxorZ!',
+        font     => 0,
+    },
+};
+
+my $bad_cipher      = 999;
+my %crypt_1_details = (
+    'default_iter'   => 1000,
+    'keylen'         => 24,
+    'blocksize'      => 8,
+    'name'           => 'DES_EDE3',
+    'alias'          => 'DES-EDE3',
+    'DES_odd_parity' => 1
+);
+
+my $bad_label       = 999;
+my $bad_label_name  = 'not_a_label_name';
+my %label_1_details = (
+    id   => 1,
+    name => 'account',
+);
+my %label_not_found_details = (
+    id   => $bad_label,
+    name => undef,
+);
+
+# Crypts
+is_deeply( Palm::Keyring::crypts(1), \%crypt_1_details, 'Got crypt 1' );
+is_deeply( Palm::Keyring::crypts('DES-EDE3'),
+    \%crypt_1_details, 'Got crypt DES-EDE3' );
+is( Palm::Keyring::crypts(), undef, "Didn't get crypt empty cipher" );
+is( Palm::Keyring::crypts($bad_cipher),
+    undef, "Didn't get crypt $bad_cipher" );
+
+# Bad Cipher
+eval { Palm::Keyring->new( { version => 5, cipher => $bad_cipher } ) };
+like(
+    $@,
+    qr/^Unknown \s cipher \s $bad_cipher/xms,
+    "Failed to create keyring with cipher $bad_cipher"
+);
+
+# Labels
+is_deeply( Palm::Keyring::labels(1), \%label_1_details, 'Got label 1' );
+is_deeply( Palm::Keyring::labels('account'),
+    \%label_1_details, 'Got label account' );
+is( Palm::Keyring::labels(), undef, "Didn't get label empty label" );
+is_deeply( Palm::Keyring::labels($bad_label),
+    \%label_not_found_details, "Got default label for $bad_label" );
+is( Palm::Keyring::labels($bad_label_name), undef, "Didn't get label for
+    $bad_label_name"
+);
+
+my $pdb;
+eval { $pdb = new Palm::Keyring( -file => 't/Keys-invalid_version.pdb' ) };
+like(
+    $@,
+    qr/^Unsupported \s Version \s 999/xms,
+    'Couldn\'t load pdb with invalid version'
+);
+
+eval { $pdb = new Palm::Keyring( -file => 't/Keys-invalid_cipher.pdb' ) };
+like(
+    $@,
+    qr/^Unknown \s cipher \s 999/xms,
+    'Couldn\'t load pdb with Unknown Cipher'
+);
+
+ok( $pdb = new Palm::Keyring( -file => 't/Keys-no_data.pdb' ),
+    'Loaded Palm::Keyring file with no data' );
+
+my $record;
+ok( $record = $pdb->append_Record(), 'Append Record' );
+ok( $pdb->Encrypt( $record, $password, $acct ),
+    'Encrypt account into record' );
+my $record2;
+ok( $record2 = $pdb->append_Record(), 'Append Record' );
+
+ok( $pdb->PackRecord($record), 'Pack Proper Record');
+eval{ $pdb->PackRecord($record2) };
+like( 
+    $@,
+    qr/^No \s encrypted \s content \s to \s pack/xms,
+    'Fail to pack record without encrypted content'
+);
+
+$record2->{encrypted} = '';
+eval{ $pdb->PackRecord($record2) };
+like( 
+    $@,
+    qr/^No \s ivec/xms,
+    'Fail to pack record without ivec'
+);
+
+$record2->{ivec} = 1;
+ok( $pdb->PackRecord($record2), 'Pack Proper Record');
+
+ok( $record = $pdb->ParseRecord(%{ $record }), 'Parse Proper Packed');
+
+$pdb->{version} = 999;
+eval{ $pdb->PackRecord($record) };
+like( $@, 
+    qr/^Unsupported \s Version \s 999/xms,
+    'Couldn\'t PackRecord with Invalid Version'
+);
+
+eval{ $pdb->ParseRecord(%{ $record2 }) };
+like( $@, 
+    qr/^Unsupported \s Version \s 999/xms,
+    'Couldn\'t ParseRecord with Invalid Version'
+);
+
+$pdb = undef;
+
+unlink $file;
+
 foreach my $options (@o) {
-    my $pdb;
-    my $record;
-    my $decrypted;
+    foreach my $config_type ( 'hashref', 'cgi-style', 'list' ) {
 
-    my $acct = {
-        0 => {
-            label => 'name',
-            label_id => 0,
-            data  => 'test3',
-            font  => 0,
-        },
-        1 => {
-            label => 'account',
-            label_id => 1,
-            data  => 'atestaccount',
-            font  => 0,
-        },
-        2 => {
-            label    => 'password',
-            label_id => 2,
-            data     => $password,
-            font  => 0,
-        },
-        3 => {
-            label => 'lastchange',
-            label_id => 3,
-            data => {
-                day   =>  2,
-                month =>  2,
-                year  => 99,
-            },
-            font  => 0,
-        },
-        255 => {
-            label => 'notes',
-            label_id => 255,
-            data  => 'now that really roxorZ!',
-            font  => 0,
-        },
-    };
+        my $pdb;
+        my $record;
+        my $decrypted;
 
+        my $Num_Tests_Left = 25;
     SKIP: {
-        if (defined $options->{cipher} && $options->{cipher} > 0) {
-            my $crypt = Palm::Keyring::crypts($options->{cipher});
-            skip 'Crypt::CBC not installed', 21 unless 
-                eval "require Crypt::CBC";
-            skip 'Crypt::' . $crypt->{name} . ' not installed', 21 unless 
-                eval "require Crypt::$crypt->{name}";
-        }
+            if ( defined $options->{cipher} && $options->{cipher} > 0 ) {
+                my $crypt = Palm::Keyring::crypts( $options->{cipher} );
+                skip 'Crypt::CBC not installed', $Num_Tests_Left
+                    unless eval "require Crypt::CBC";
+                if ($crypt) {
+                    skip 'Crypt::' . $crypt->{name} . ' not installed',
+                        $Num_Tests_Left
+                        unless eval "require Crypt::$crypt->{name}";
+                }
+                else {
+                    skip 'Unknown Crypt: ' . $options->{cipher},
+                        $Num_Tests_Left;
+                }
+            }
 
-        if ($options->{version} == 4) {
-            skip 'Crypt::DES not installed', 21 unless 
-                eval " require Crypt::DES ";
-            skip 'Digest::MD5 not installed', 21 unless 
-                eval " require Digest::MD5 ";
-        } elsif ($options->{version} == 5) {
-            skip 'Digest::HMAC_SHA1 not installed', 21 unless 
-                eval " require Digest::HMAC_SHA1 ";
-        }
+            if ( $options->{version} == 4 ) {
+                skip 'Crypt::DES not installed', $Num_Tests_Left
+                    unless eval "require Crypt::DES ";
+                skip 'Digest::MD5 not installed', $Num_Tests_Left
+                    unless eval "require Digest::MD5 ";
+            }
+            elsif ( $options->{version} == 5 ) {
+                skip 'Digest::HMAC_SHA1 not installed', $Num_Tests_Left
+                    unless eval "require Digest::HMAC_SHA1 ";
+            }
 
-        ok( $pdb = new Palm::Keyring($options), 
-            'new Palm::Keyring v' . $options->{version});
+            my @options = ($options);
+            if ( $config_type eq 'cgi-style' ) {
+                @options = (
+                    '-version'  => $options->{version},
+                    '-password' => $options->{password},
+                );
+                if ( $options->{cipher} ) {
+                    push @options, '-cipher', $options->{cipher};
+                }
+            }
+            elsif ( $config_type eq 'list' ) {
+                @options = ( $options->{password}, $options->{version} );
+                if ( $options->{cipher} ) {
+                    push @options, $options->{cipher};
+                }
+            }
 
-        ok( $record = $pdb->append_Record(), 'Append Record' );
+            ok( $pdb = new Palm::Keyring(@options),
+                'new Palm::Keyring v' . $options->{version}
+            );
 
-        ok( $pdb->Encrypt($record, $password, $acct), 
-            'Encrypt account into record' );
+            ok( $pdb->Write($file), 'Write "empty" file' );
 
-        ok( $pdb->Write($file), 'Write file' );
+            ok( $record = $pdb->append_Record(), 'Append Record' );
 
-        $pdb = undef;
+            ok( $pdb->Encrypt( $record, $password, $acct ),
+                'Encrypt account into record' );
 
-        ok( $pdb = new Palm::PDB(), 'new Palm::Keyring' );
+            ok( $pdb->Write($file), 'Write file' );
 
-        ok( $pdb->Load($file), 'Load File' );
+            $pdb = undef;
 
-        ok( $pdb->Password($password), 'Verify Password' );
+            ok( $pdb = new Palm::Keyring(), 'new Palm::Keyring' );
 
-        my $rec_num = 0;
-        ok( $decrypted = $pdb->Decrypt($pdb->{records}->[$rec_num]), 
-            'Decrypt record' );
+            ok( $pdb->Load($file), 'Load File' );
 
-        is( $decrypted->{2}->{data}, $password, 'Got password' );
+            ok( $pdb->Password($password), 'Verify Password' );
 
-        is_deeply( $decrypted, $acct, 'Account Matches' );
+            my $rec_num = 0;
+            ok( $decrypted = $pdb->Decrypt( $pdb->{records}->[$rec_num] ),
+                'Decrypt record' );
 
-        my $old_date = $decrypted->{3}->{data};
+            is( $decrypted->{2}->{data}, $password, 'Got password' );
 
-        ok( $pdb->Password($password, $new_password), 'Change PDB Password' );
+            is_deeply( $decrypted, $acct, 'Account Matches' );
 
-        ok( $decrypted = $pdb->Decrypt($pdb->{'records'}->[$rec_num]), 
-            'Decrypt with new password' );
+            my $old_date = $decrypted->{3}->{data};
 
-        my $new_date = $decrypted->{3}->{data};
+            ok( $pdb->Password( $password, $new_password ),
+                'Change PDB Password' );
 
-        is_deeply( $old_date, $new_date, 'Date didn\'t change' );
+            ok( $decrypted = $pdb->Decrypt( $pdb->{'records'}->[$rec_num] ),
+                'Decrypt with new password' );
 
-        $acct->{2}->{data} = $new_password;
+            my $new_date = $decrypted->{3}->{data};
 
-        $pdb->{records}->[$rec_num]->{plaintext} = $acct;
+            is_deeply( $old_date, $new_date, 'Date didn\'t change' );
 
-        ok(  $pdb->Encrypt($pdb->{'records'}->[$rec_num]), 'Change record' );
+            $decrypted->{2}->{data} = $new_password;
 
-        ok( $decrypted = $pdb->Decrypt($pdb->{'records'}->[$rec_num]), 
-            'Decrypt new record' );
+            $pdb->{records}->[$rec_num]->{plaintext} = $decrypted;
 
-        $new_date = $decrypted->{3}->{data};
+            ok( $pdb->Encrypt( $pdb->{'records'}->[$rec_num] ),
+                'Change record' );
 
-        my $od = join '/', map { $old_date->{$_} } sort keys %{ $old_date };
-        my $nd = join '/', map { $new_date->{$_} } sort keys %{ $new_date };
+            ok( $decrypted = $pdb->Decrypt( $pdb->{'records'}->[$rec_num] ),
+                'Decrypt new record' );
 
-        isnt( $od, $nd, 'Date changed');
+            $new_date = $decrypted->{3}->{data};
 
-        is( $decrypted->{2}->{data}, $new_password, 'Got new password' ); 
+            my $od = join '/', map { $old_date->{$_} } sort keys %{$old_date};
+            my $nd = join '/', map { $new_date->{$_} } sort keys %{$new_date};
 
-        my $last_decrypted = $decrypted;
+            isnt( $od, $nd, 'Date changed' );
 
-        $decrypted = {};
-        ok( $pdb->Password(), 'Forget password' );
+            is( $decrypted->{2}->{data}, $new_password, 'Got new password' );
 
-        eval{ $decrypted = $pdb->Decrypt($pdb->{'records'}->[$rec_num]) };
-        ok( $@, 'Don\'t decrypt' );
+            my $last_decrypted = $decrypted;
 
-        isnt( $decrypted->{password}, $new_password, 'Didn\'t get new password' );
+            $decrypted = {};
+            ok( $pdb->Password(), 'Forget password' );
 
-        ok( $pdb->Unlock($new_password), 'Unlock' );
+            eval {
+                $decrypted = $pdb->Decrypt( $pdb->{'records'}->[$rec_num] );
+            };
+            ok( $@, 'Don\'t decrypt' );
 
-        my @plaintext = map { $_->{plaintext} } @{ $pdb->{records} };
+            isnt( $decrypted->{password},
+                $new_password, 'Didn\'t get new password' );
 
-        is_deeply( $plaintext[0], $last_decrypted, 'Account Matches' );
+            ok( $pdb->Unlock($new_password), 'Unlock' );
 
-        ok( $pdb->Lock(), 'Lock' );
+            my @plaintext = map { $_->{plaintext} } @{ $pdb->{records} };
 
-        my $cleared_decrypted = {};
-        $cleared_decrypted->{0}= $last_decrypted->{0};
-        @plaintext = map { $_->{plaintext} } @{ $pdb->{records} };
+            is_deeply( $plaintext[0], $last_decrypted, 'Account Matches' );
 
-        is_deeply( $plaintext[0], $cleared_decrypted, 'Cleared records' );
+            ok( $pdb->Lock(), 'Lock' );
 
-        ok( unlink($file), 'Remove test pdb v' . $options->{version} );
+            my $cleared_decrypted = {};
+            $cleared_decrypted->{0} = $last_decrypted->{0};
+            @plaintext = map { $_->{plaintext} } @{ $pdb->{records} };
 
+            is_deeply( $plaintext[0], $cleared_decrypted, 'Cleared records' );
+
+            $pdb->{records}->[0]->{data} = undef;
+            ok( $pdb->Write($file), 'Write file' );
+            ok( $pdb->Load($file),  'Load File' );
+
+            $pdb->{version} = 999;
+            eval { $pdb->Write($file) };
+            like(
+                $@,
+                qr/^Unsupported \s Version \s 999/xms,
+                'Couldn\'t Write file with unsupported version'
+            );
+
+            ok( unlink($file), 'Remove test pdb v' . $options->{version} );
+
+        }
     }
 }