Neither of these can fail now that we dep on 5.8.1 (missed in 55087b99)
Peter Rabbitson [Sat, 24 Jan 2015 19:53:22 +0000 (20:53 +0100)]
lib/DBIx/Class/Optional/Dependencies.pm
lib/DBIx/Class/SQLMaker/Oracle.pm
t/inflate/serialize.t

index 9afc2be..37e18c0 100644 (file)
@@ -51,7 +51,6 @@ my $datetime_basic = {
 };
 
 my $id_shortener = {
-  'Digest::MD5'                   => '0',
   'Math::BigInt'                  => '1.80',
   'Math::Base36'                  => '0.07',
 };
index 3cad115..b4c1584 100644 (file)
@@ -9,6 +9,7 @@ BEGIN {
   if (my $missing = DBIx::Class::Optional::Dependencies->req_missing_for ('id_shortener') ) {
     die "The following extra modules are required for Oracle-based Storages: $missing\n";
   }
+  require Digest::MD5;
 }
 
 use base 'DBIx::Class::SQLMaker';
index 5b1a437..a9bf5d7 100644 (file)
@@ -28,8 +28,6 @@ foreach my $serializer (@serializers) {
     }
 }
 
-plan (skip_all => "No suitable serializer found") unless $selected;
-
 DBICTest::Schema::Serialized->inflate_column( 'serialized',
     { inflate => $selected->{inflater},
       deflate => $selected->{deflater},