X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=t%2F64db.t;h=b207a2940296f95d8a225b806fe1f5b2603afe20;hb=b9df8e39d03e6da167b2598f6ee70fb16c069112;hp=56a94e325c16ae0963c54a58c7edf93b8246220c;hpb=40708af1d856abcb5044605dcea6664726a9751e;p=dbsrgits%2FDBIx-Class.git

diff --git a/t/64db.t b/t/64db.t
index 56a94e3..b207a29 100644
--- a/t/64db.t
+++ b/t/64db.t
@@ -1,5 +1,5 @@
 use strict;
-use warnings;  
+use warnings;
 
 use Test::More;
 use lib qw(t/lib);
@@ -35,25 +35,21 @@ $schema->storage->txn_rollback;
 is($artist, undef, "Rollback ok");
 
 is_deeply (
-  get_storage_column_info ($schema->storage, 'collection'),
+  get_storage_column_info ($schema->storage, 'collection', qw/size is_nullable/),
   {
     collectionid => {
       data_type => 'INTEGER',
-      is_nullable => 0,
     },
     name => {
       data_type => 'varchar',
-      is_nullable => 0,
     },
   },
-  'Correctly retrieve column info (all columns non-nullable)'
+  'Correctly retrieve column info (no size or is_nullable)'
 );
 
-TODO: {
-  local $TODO = 'All current versions of SQLite seem to mis-report is_nullable';
-
+{
   is_deeply (
-    get_storage_column_info ($schema->storage, 'artist'),
+    get_storage_column_info ($schema->storage, 'artist', qw/size/),
     {
       'artistid' => {
           'data_type' => 'INTEGER',
@@ -66,6 +62,11 @@ TODO: {
       'rank' => {
           'data_type' => 'integer',
           'is_nullable' => 0,
+          'default_value' => '13',
+      },
+      'charfield' => {
+          'data_type' => 'char',
+          'is_nullable' => 1,
       },
     },
     'Correctly retrieve column info (mixed null and non-null columns)'
@@ -73,18 +74,23 @@ TODO: {
 };
 
 
+# Depending on test we need to strip away certain column info.
+#  - SQLite is known to report the size differently from release to release
+#  - Current DBD::SQLite versions do not implement NULLABLE
+#  - Some SQLite releases report stuff that isn't there as undef
+
 sub get_storage_column_info {
-  my ($storage, $table) = @_;
+  my ($storage, $table, @ignore) = @_;
 
   my $type_info = $storage->columns_info_for($table);
 
-  # I know this is gross but SQLite reports the size differently from release
-  # to release. At least this way the test still passes.
-  # Also it seems that some SQLite releases report stuff that isn't there as
-  # undef. So strip them out.
   for my $col (keys %$type_info) {
     for my $type (keys %{$type_info->{$col}}) {
-      if ($type eq 'size' or not defined $type_info->{$col}{$type} ) {
+      if (
+        grep { $type eq $_ } (@ignore)
+          or
+        not defined $type_info->{$col}{$type}
+      ) {
         delete $type_info->{$col}{$type};
       }
     }