Do not attempt to load DBD::Oracle without envvars
Peter Rabbitson [Thu, 12 Mar 2015 14:10:15 +0000 (15:10 +0100)]
This is morally equivalent to 250d9e55, except without any of the new OptDep
shinies

t/inflate/datetime_oracle.t

index 9182f23..25a9366 100644 (file)
@@ -7,9 +7,6 @@ use DBIx::Class::Optional::Dependencies ();
 use lib qw(t/lib);
 use DBICTest;
 
-plan skip_all => 'Test needs ' . DBIx::Class::Optional::Dependencies->req_missing_for ('test_rdbms_oracle')
-  unless DBIx::Class::Optional::Dependencies->req_ok_for ('test_rdbms_oracle');
-
 my ($dsn, $user, $pass) = @ENV{map { "DBICTEST_ORA_${_}" } qw/DSN USER PASS/};
 
 if (not ($dsn && $user && $pass)) {
@@ -17,6 +14,10 @@ if (not ($dsn && $user && $pass)) {
          'Warning: This test drops and creates a table called \'event\'';
 }
 
+plan skip_all => 'Test needs ' . DBIx::Class::Optional::Dependencies->req_missing_for ('test_rdbms_oracle')
+  unless DBIx::Class::Optional::Dependencies->req_ok_for ('test_rdbms_oracle');
+
+
 # DateTime::Format::Oracle needs this set
 $ENV{NLS_DATE_FORMAT} = 'DD-MON-YY';
 $ENV{NLS_TIMESTAMP_FORMAT} = 'YYYY-MM-DD HH24:MI:SSXFF';