[dbsrgits/DBIx-Class.git] / t / 73oracle.t

{
  package    # hide from PAUSE
    DBICTest::Schema::ArtistFQN;

  use base 'DBIx::Class::Core';

  __PACKAGE__->table(
    defined $ENV{DBICTEST_ORA_USER}
      ? (uc $ENV{DBICTEST_ORA_USER}) . '.artist'
      : 'artist'
  );
  __PACKAGE__->add_columns(
    'artistid' => {
      data_type         => 'integer',
      is_auto_increment => 1,
    },
    'name' => {
      data_type   => 'varchar',
      size        => 100,
      is_nullable => 1,
    },
    'autoinc_col' => {
      data_type         => 'integer',
      is_auto_increment => 1,
    },
  );
  __PACKAGE__->set_primary_key(qw/ artistid autoinc_col /);

  1;
}

use strict;
use warnings;

use Test::Exception;
use Test::More;
use Sub::Name;

use lib qw(t/lib);
use DBICTest;
use DBIC::SqlMakerTest;

$ENV{NLS_SORT} = "BINARY";
$ENV{NLS_COMP} = "BINARY";
$ENV{NLS_LANG} = "AMERICAN";

my ($dsn,  $user,  $pass)  = @ENV{map { "DBICTEST_ORA_${_}" }  qw/DSN USER PASS/};

# optional:
my ($dsn2, $user2, $pass2) = @ENV{map { "DBICTEST_ORA_EXTRAUSER_${_}" } qw/DSN USER PASS/};

plan skip_all => 'Set $ENV{DBICTEST_ORA_DSN}, _USER and _PASS to run this test.'
  unless ($dsn && $user && $pass);

DBICTest::Schema->load_classes('ArtistFQN');

# This is in Core now, but it's here just to test that it doesn't break
DBICTest::Schema::Artist->load_components('PK::Auto');
# These are compat shims for PK::Auto...
DBICTest::Schema::CD->load_components('PK::Auto::Oracle');
DBICTest::Schema::Track->load_components('PK::Auto::Oracle');


##########
# recyclebin sometimes comes in the way
my $on_connect_sql = ["ALTER SESSION SET recyclebin = OFF"];

# iterate all tests on following options
my @tryopt = (
  { on_connect_do => $on_connect_sql },
  { quote_char => '"', on_connect_do => $on_connect_sql, },
);

# keep a database handle open for cleanup
my $dbh;

# test insert returning

# check if we indeed do support stuff
my $test_server_supports_insert_returning = do {
  my $v = DBICTest::Schema->connect($dsn, $user, $pass)
                   ->storage
                    ->_get_dbh
                     ->get_info(18);
  $v =~ /^(\d+)\.(\d+)/
    or die "Unparseable Oracle server version: $v\n";

# TODO find out which version supports the RETURNING syntax
# 8i has it and earlier docs are a 404 on oracle.com
  ( $1 > 8 || ($1 == 8 && $2 >= 1) ) ? 1 : 0;
};
is (
  DBICTest::Schema->connect($dsn, $user, $pass)->storage->_use_insert_returning,
  $test_server_supports_insert_returning,
  'insert returning capability guessed correctly'
);

my $schema;
for my $use_insert_returning ($test_server_supports_insert_returning
  ? (1,0)
  : (0)
) {

  no warnings qw/once/;
  local *DBICTest::Schema::connection = subname 'DBICTest::Schema::connection' => sub {
    my $s = shift->next::method (@_);
    $s->storage->_use_insert_returning ($use_insert_returning);
    $s;
  };

  for my $opt (@tryopt) {
    # clean all cached sequences from previous run
    for (map { values %{DBICTest::Schema->source($_)->columns_info} } (qw/Artist CD Track/) ) {
      delete $_->{sequence};
    }

    my $schema = DBICTest::Schema->connect($dsn, $user, $pass, $opt);

    $dbh = $schema->storage->dbh;
    my $q = $schema->storage->sql_maker->quote_char || '';

    do_creates($dbh, $q);

    _run_tests($schema, $opt);
  }
}

sub _run_tests {
  my ($schema, $opt) = @_;

  my $q = $schema->storage->sql_maker->quote_char || '';

# test primary key handling with multiple triggers
  my ($new, $seq);

  my $new_artist = $schema->resultset('Artist')->create({ name => 'foo' });
  my $new_cd     = $schema->resultset('CD')->create({ artist => 1, title => 'EP C', year => '2003' });

  SKIP: {
    skip 'not detecting sequences when using INSERT ... RETURNING', 4
      if $schema->storage->_use_insert_returning;

    is($new_artist->artistid, 1, "Oracle Auto-PK worked for standard sqlt-like trigger");
    $seq = $new_artist->result_source->column_info('artistid')->{sequence};
    $seq = $$seq if ref $seq;
    like ($seq, qr/\.${q}artist_pk_seq${q}$/, 'Correct PK sequence selected for sqlt-like trigger');

    is($new_cd->cdid, 1, 'Oracle Auto-PK worked - using scalar ref as table name/custom weird trigger');
    $seq = $new_cd->result_source->column_info('cdid')->{sequence};
    $seq = $$seq if ref $seq;
    like ($seq, qr/\.${q}cd_seq${q}$/, 'Correct PK sequence selected for custom trigger');
  }

# test PKs again with fully-qualified table name
  my $artistfqn_rs = $schema->resultset('ArtistFQN');
  my $artist_rsrc = $artistfqn_rs->result_source;

  delete $artist_rsrc->column_info('artistid')->{sequence};
  $new = $artistfqn_rs->create( { name => 'bar' } );

  is_deeply( {map { $_ => $new->$_ } $artist_rsrc->primary_columns},
    { artistid => 2, autoinc_col => 2},
    "Oracle Multi-Auto-PK worked with fully-qualified tablename" );


  delete $artist_rsrc->column_info('artistid')->{sequence};
  $new = $artistfqn_rs->create( { name => 'bar', autoinc_col => 1000 } );

  is( $new->artistid, 3, "Oracle Auto-PK worked with fully-qualified tablename" );
  is( $new->autoinc_col, 1000, "Oracle Auto-Inc overruled with fully-qualified tablename");

  SKIP: {
    skip 'not detecting sequences when using INSERT ... RETURNING', 1
      if $schema->storage->_use_insert_returning;

    $seq = $new->result_source->column_info('artistid')->{sequence};
    $seq = $$seq if ref $seq;
    like ($seq, qr/\.${q}artist_pk_seq${q}$/, 'Correct PK sequence selected for sqlt-like trigger');
  }


# test LIMIT support
  for (1..6) {
    $schema->resultset('Artist')->create({ name => 'Artist ' . $_ });
  }
  my $it = $schema->resultset('Artist')->search( { name => { -like => 'Artist %' } }, {
    rows => 3,
    offset => 4,
    order_by => 'artistid'
  });

  is( $it->count, 2, "LIMIT count past end of RS ok" );
  is( $it->next->name, "Artist 5", "iterator->next ok" );
  is( $it->next->name, "Artist 6", "iterator->next ok" );
  is( $it->next, undef, "next past end of resultset ok" );


# test identifiers over the 30 char limit
  lives_ok {
    my @results = $schema->resultset('CD')->search(undef, {
      prefetch => 'very_long_artist_relationship',
      rows => 3,
      offset => 0,
    })->all;
    ok( scalar @results > 0, 'limit with long identifiers returned something');
  } 'limit with long identifiers executed successfully';


# test rel names over the 30 char limit
  my $query = $schema->resultset('Artist')->search({
    artistid => 1
  }, {
    prefetch => 'cds_very_very_very_long_relationship_name'
  });

  lives_and {
    is $query->first->cds_very_very_very_long_relationship_name->first->cdid, 1
  } 'query with rel name over 30 chars survived and worked';

  # rel name over 30 char limit with user condition
  # This requires walking the SQLA data structure.
  {
    local $TODO = 'user condition on rel longer than 30 chars';

    $query = $schema->resultset('Artist')->search({
      'cds_very_very_very_long_relationship_name.title' => 'EP C'
    }, {
      prefetch => 'cds_very_very_very_long_relationship_name'
    });

    lives_and {
      is $query->first->cds_very_very_very_long_relationship_name->first->cdid, 1
    } 'query with rel name over 30 chars and user condition survived and worked';
  }


# test join with row count ambiguity
  my $cd = $schema->resultset('CD')->next;
  my $track = $cd->create_related('tracks', { position => 1, title => 'Track1'} );
  my $tjoin = $schema->resultset('Track')->search({ 'me.title' => 'Track1'}, {
    join => 'cd', rows => 2
  });

  ok(my $row = $tjoin->next);

  is($row->title, 'Track1', "ambiguous column ok");


# check count distinct with multiple columns
  my $other_track = $schema->resultset('Track')->create({ cd => $cd->cdid, position => 1, title => 'Track2' });

  my $tcount = $schema->resultset('Track')->search(
    {},
    {
      select => [ qw/position title/ ],
      distinct => 1,
    }
  );
  is($tcount->count, 2, 'multiple column COUNT DISTINCT ok');

  $tcount = $schema->resultset('Track')->search(
    {},
    {
      columns => [ qw/position title/ ],
      distinct => 1,
    }
  );
  is($tcount->count, 2, 'multiple column COUNT DISTINCT ok');

  $tcount = $schema->resultset('Track')->search(
    {},
    {
      group_by => [ qw/position title/ ]
    }
  );
  is($tcount->count, 2, 'multiple column COUNT DISTINCT using column syntax ok');


# check group_by
  my $g_rs = $schema->resultset('Track')->search( undef, { columns=>[qw/trackid position/], group_by=> [ qw/trackid position/ ] , rows => 2, offset => 1 });
  is( scalar $g_rs->all, 1, "Group by with limit OK" );


# test with_deferred_fk_checks
  lives_ok {
    $schema->storage->with_deferred_fk_checks(sub {
      $schema->resultset('Track')->create({
        trackid => 999, cd => 999, position => 1, title => 'deferred FK track'
      });
      $schema->resultset('CD')->create({
        artist => 1, cdid => 999, year => '2003', title => 'deferred FK cd'
      });
    });
  } 'with_deferred_fk_checks code survived';

  is eval { $schema->resultset('Track')->find(999)->title }, 'deferred FK track',
    'code in with_deferred_fk_checks worked'; 

  throws_ok {
    $schema->resultset('Track')->create({
      trackid => 1, cd => 9999, position => 1, title => 'Track1'
    });
  } qr/constraint/i, 'with_deferred_fk_checks is off';


# test auto increment using sequences WITHOUT triggers
  for (1..5) {
    my $st = $schema->resultset('SequenceTest')->create({ name => 'foo' });
    is($st->pkid1, $_, "Oracle Auto-PK without trigger: First primary key");
    is($st->pkid2, $_ + 9, "Oracle Auto-PK without trigger: Second primary key");
    is($st->nonpkid, $_ + 19, "Oracle Auto-PK without trigger: Non-primary key");
  }
  my $st = $schema->resultset('SequenceTest')->create({ name => 'foo', pkid1 => 55 });
  is($st->pkid1, 55, "Oracle Auto-PK without trigger: First primary key set manually");


# test BLOBs
  SKIP: {
  TODO: {
    my %binstr = ( 'small' => join('', map { chr($_) } ( 1 .. 127 )) );
    $binstr{'large'} = $binstr{'small'} x 1024;

    my $maxloblen = length $binstr{'large'};
    note "Localizing LongReadLen to $maxloblen to avoid truncation of test data";
    local $dbh->{'LongReadLen'} = $maxloblen;

    my $rs = $schema->resultset('BindType');
    my $id = 0;

    if ($DBD::Oracle::VERSION eq '1.23') {
      throws_ok { $rs->create({ id => 1, blob => $binstr{large} }) }
        qr/broken/,
        'throws on blob insert with DBD::Oracle == 1.23';

      skip 'buggy BLOB support in DBD::Oracle 1.23', 7;
    }

    # disable BLOB mega-output
    my $orig_debug = $schema->storage->debug;
    $schema->storage->debug (0);

    local $TODO = 'Something is confusing column bindtype assignment when quotes are active'
      if $q;

    foreach my $type (qw( blob clob )) {
      foreach my $size (qw( small large )) {
        $id++;

        lives_ok { $rs->create( { 'id' => $id, $type => $binstr{$size} } ) }
        "inserted $size $type without dying";
        ok($rs->find($id)->$type eq $binstr{$size}, "verified inserted $size $type" );
      }
    }

    $schema->storage->debug ($orig_debug);
  }}


# test sequence detection from a different schema
  SKIP: {
  TODO: {
    skip ((join '',
      'Set DBICTEST_ORA_EXTRAUSER_DSN, _USER and _PASS to a *DIFFERENT* Oracle user',
      ' to run the cross-schema sequence detection test.'),
    1) unless $dsn2 && $user2 && $user2 ne $user;

    skip 'not detecting cross-schema sequence name when using INSERT ... RETURNING', 1
      if $schema->storage->_use_insert_returning;

    # Oracle8i Reference Release 2 (8.1.6) 
    #   http://download.oracle.com/docs/cd/A87860_01/doc/server.817/a76961/ch294.htm#993
    # Oracle Database Reference 10g Release 2 (10.2)
    #   http://download.oracle.com/docs/cd/B19306_01/server.102/b14237/statviews_2107.htm#sthref1297
    local $TODO = "On Oracle8i all_triggers view is empty, i don't yet know why..."
      if $schema->storage->_server_info->{normalized_dbms_version} < 9;

    my $schema2 = $schema->connect($dsn2, $user2, $pass2, $opt);

    my $schema1_dbh  = $schema->storage->dbh;
    $schema1_dbh->do("GRANT INSERT ON ${q}artist${q} TO " . uc $user2);
    $schema1_dbh->do("GRANT SELECT ON ${q}artist_pk_seq${q} TO " . uc $user2);
    $schema1_dbh->do("GRANT SELECT ON ${q}artist_autoinc_seq${q} TO " . uc $user2);


    my $rs = $schema2->resultset('ArtistFQN');
    delete $rs->result_source->column_info('artistid')->{sequence};

    lives_and {
      my $row = $rs->create({ name => 'From Different Schema' });
      ok $row->artistid;
    } 'used autoinc sequence across schemas';

    # now quote the sequence name (do_creates always uses an lc name)
    my $q_seq = $q
      ? '"artist_pk_seq"'
      : '"ARTIST_PK_SEQ"'
    ;
    delete $rs->result_source->column_info('artistid')->{sequence};
    $schema1_dbh->do(qq{
      CREATE OR REPLACE TRIGGER ${q}artist_insert_trg_pk${q}
      BEFORE INSERT ON ${q}artist${q}
      FOR EACH ROW
      BEGIN
        IF :new.${q}artistid${q} IS NULL THEN
          SELECT $q_seq.nextval
          INTO :new.${q}artistid${q}
          FROM DUAL;
        END IF;
      END;
    });


    lives_and {
      my $row = $rs->create({ name => 'From Different Schema With Quoted Sequence' });
      ok $row->artistid;
    } 'used quoted autoinc sequence across schemas';

    my $schema_name = uc $user;

    is_deeply $rs->result_source->column_info('artistid')->{sequence},
      \qq|${schema_name}.$q_seq|,
      'quoted sequence name correctly extracted';
  }}

  do_clean ($dbh);
}

done_testing;

sub do_creates {
  my ($dbh, $q) = @_;

  do_clean($dbh);

  $dbh->do("CREATE SEQUENCE ${q}artist_autoinc_seq${q} START WITH 1 MAXVALUE 999999 MINVALUE 0");
  $dbh->do("CREATE SEQUENCE ${q}artist_pk_seq${q} START WITH 1 MAXVALUE 999999 MINVALUE 0");
  $dbh->do("CREATE SEQUENCE ${q}cd_seq${q} START WITH 1 MAXVALUE 999999 MINVALUE 0");
  $dbh->do("CREATE SEQUENCE ${q}track_seq${q} START WITH 1 MAXVALUE 999999 MINVALUE 0");

  $dbh->do("CREATE SEQUENCE ${q}nonpkid_seq${q} START WITH 20 MAXVALUE 999999 MINVALUE 0");
  # this one is always quoted as per manually specified sequence =>
  $dbh->do('CREATE SEQUENCE "pkid1_seq" START WITH 1 MAXVALUE 999999 MINVALUE 0');
  # this one is always unquoted as per manually specified sequence =>
  $dbh->do("CREATE SEQUENCE pkid2_seq START WITH 10 MAXVALUE 999999 MINVALUE 0");

  $dbh->do("CREATE TABLE ${q}artist${q} (${q}artistid${q} NUMBER(12), ${q}name${q} VARCHAR(255), ${q}autoinc_col${q} NUMBER(12), ${q}rank${q} NUMBER(38), ${q}charfield${q} VARCHAR2(10))");
  $dbh->do("ALTER TABLE ${q}artist${q} ADD (CONSTRAINT ${q}artist_pk${q} PRIMARY KEY (${q}artistid${q}))");

  $dbh->do("CREATE TABLE ${q}sequence_test${q} (${q}pkid1${q} NUMBER(12), ${q}pkid2${q} NUMBER(12), ${q}nonpkid${q} NUMBER(12), ${q}name${q} VARCHAR(255))");
  $dbh->do("ALTER TABLE ${q}sequence_test${q} ADD (CONSTRAINT ${q}sequence_test_constraint${q} PRIMARY KEY (${q}pkid1${q}, ${q}pkid2${q}))");

  # table cd will be unquoted => Oracle will see it as uppercase
  $dbh->do("CREATE TABLE cd (${q}cdid${q} NUMBER(12), ${q}artist${q} NUMBER(12), ${q}title${q} VARCHAR(255), ${q}year${q} VARCHAR(4), ${q}genreid${q} NUMBER(12), ${q}single_track${q} NUMBER(12))");
  $dbh->do("ALTER TABLE cd ADD (CONSTRAINT ${q}cd_pk${q} PRIMARY KEY (${q}cdid${q}))");

  $dbh->do("CREATE TABLE ${q}track${q} (${q}trackid${q} NUMBER(12), ${q}cd${q} NUMBER(12) REFERENCES CD(${q}cdid${q}) DEFERRABLE, ${q}position${q} NUMBER(12), ${q}title${q} VARCHAR(255), ${q}last_updated_on${q} DATE, ${q}last_updated_at${q} DATE, ${q}small_dt${q} DATE)");
  $dbh->do("ALTER TABLE ${q}track${q} ADD (CONSTRAINT ${q}track_pk${q} PRIMARY KEY (${q}trackid${q}))");

  $dbh->do("CREATE TABLE ${q}bindtype_test${q} (${q}id${q} integer NOT NULL PRIMARY KEY, ${q}bytea${q} integer NULL, ${q}blob${q} blob NULL, ${q}clob${q} clob NULL)");

  $dbh->do(qq{
    CREATE OR REPLACE TRIGGER ${q}artist_insert_trg_auto${q}
    BEFORE INSERT ON ${q}artist${q}
    FOR EACH ROW
    BEGIN
      IF :new.${q}autoinc_col${q} IS NULL THEN
        SELECT ${q}artist_autoinc_seq${q}.nextval
        INTO :new.${q}autoinc_col${q}
        FROM DUAL;
      END IF;
    END;
  });

  $dbh->do(qq{
    CREATE OR REPLACE TRIGGER ${q}artist_insert_trg_pk${q}
    BEFORE INSERT ON ${q}artist${q}
    FOR EACH ROW
    BEGIN
      IF :new.${q}artistid${q} IS NULL THEN
        SELECT ${q}artist_pk_seq${q}.nextval
        INTO :new.${q}artistid${q}
        FROM DUAL;
      END IF;
    END;
  });

  $dbh->do(qq{
    CREATE OR REPLACE TRIGGER ${q}cd_insert_trg${q}
    BEFORE INSERT OR UPDATE ON cd
    FOR EACH ROW

    DECLARE
    tmpVar NUMBER;

    BEGIN
      tmpVar := 0;

      IF :new.${q}cdid${q} IS NULL THEN
        SELECT ${q}cd_seq${q}.nextval
        INTO tmpVar
        FROM dual;

        :new.${q}cdid${q} := tmpVar;
      END IF;
    END;
  });

  $dbh->do(qq{
    CREATE OR REPLACE TRIGGER ${q}track_insert_trg${q}
    BEFORE INSERT ON ${q}track${q}
    FOR EACH ROW
    BEGIN
      IF :new.${q}trackid${q} IS NULL THEN
        SELECT ${q}track_seq${q}.nextval
        INTO :new.${q}trackid${q}
        FROM DUAL;
      END IF;
    END;
  });
}

# clean up our mess
sub do_clean {

  my $dbh = shift || return;

  for my $q ('', '"') {
    my @clean = (
      "DROP TRIGGER ${q}track_insert_trg${q}",
      "DROP TRIGGER ${q}cd_insert_trg${q}",
      "DROP TRIGGER ${q}artist_insert_trg_auto${q}",
      "DROP TRIGGER ${q}artist_insert_trg_pk${q}",
      "DROP SEQUENCE ${q}nonpkid_seq${q}",
      "DROP SEQUENCE ${q}pkid2_seq${q}",
      "DROP SEQUENCE ${q}pkid1_seq${q}",
      "DROP SEQUENCE ${q}track_seq${q}",
      "DROP SEQUENCE ${q}cd_seq${q}",
      "DROP SEQUENCE ${q}artist_autoinc_seq${q}",
      "DROP SEQUENCE ${q}artist_pk_seq${q}",
      "DROP TABLE ${q}bindtype_test${q}",
      "DROP TABLE ${q}sequence_test${q}",
      "DROP TABLE ${q}track${q}",
      "DROP TABLE ${q}cd${q}",
      "DROP TABLE ${q}artist${q}",
    );
    eval { $dbh -> do ($_) } for @clean;
  }
}

END {
  if ($dbh) {
    local $SIG{__WARN__} = sub {};
    do_clean($dbh);
    $dbh->disconnect;
  }
}
Commit	Line	Data
cb464582	1	{
	2	package # hide from PAUSE
	3	DBICTest::Schema::ArtistFQN;
	4
	5	use base 'DBIx::Class::Core';
	6
	7	__PACKAGE__->table(
07cda1c5	8	defined $ENV{DBICTEST_ORA_USER}
07cda1c5	9	? (uc $ENV{DBICTEST_ORA_USER}) . '.artist'
cb464582	10	: 'artist'
	11	);
	12	__PACKAGE__->add_columns(
07cda1c5	13	'artistid' => {
	14	data_type => 'integer',
	15	is_auto_increment => 1,
	16	},
	17	'name' => {
	18	data_type => 'varchar',
	19	size => 100,
	20	is_nullable => 1,
	21	},
	22	'autoinc_col' => {
	23	data_type => 'integer',
	24	is_auto_increment => 1,
	25	},
cb464582	26	);
bf51641f	27	__PACKAGE__->set_primary_key(qw/ artistid autoinc_col /);
cb464582	28
	29	1;
	30	}
	31
70350518	32	use strict;
e6dd7b42	33	use warnings;
70350518	34
5db2758d	35	use Test::Exception;
70350518	36	use Test::More;
bf51641f	37	use Sub::Name;
ab6e0924	38
70350518	39	use lib qw(t/lib);
70350518	40	use DBICTest;
8ce8340f	41	use DBIC::SqlMakerTest;
0567538f	42
994dc91b	43	$ENV{NLS_SORT} = "BINARY";
	44	$ENV{NLS_COMP} = "BINARY";
	45	$ENV{NLS_LANG} = "AMERICAN";
	46
df6e3f5c	47	my ($dsn, $user, $pass) = @ENV{map { "DBICTEST_ORA_${_}" } qw/DSN USER PASS/};
	48
	49	# optional:
9d7d2f00	50	my ($dsn2, $user2, $pass2) = @ENV{map { "DBICTEST_ORA_EXTRAUSER_${_}" } qw/DSN USER PASS/};
0567538f	51
12e05c15	52	plan skip_all => 'Set $ENV{DBICTEST_ORA_DSN}, _USER and _PASS to run this test.'
0567538f	53	unless ($dsn && $user && $pass);
0567538f	54
cb464582	55	DBICTest::Schema->load_classes('ArtistFQN');
0567538f	56
07cda1c5	57	# This is in Core now, but it's here just to test that it doesn't break
	58	DBICTest::Schema::Artist->load_components('PK::Auto');
	59	# These are compat shims for PK::Auto...
	60	DBICTest::Schema::CD->load_components('PK::Auto::Oracle');
	61	DBICTest::Schema::Track->load_components('PK::Auto::Oracle');
ba12b23f	62
0567538f	63
07cda1c5	64	##########
	65	# recyclebin sometimes comes in the way
	66	my $on_connect_sql = ["ALTER SESSION SET recyclebin = OFF"];
0567538f	67
07cda1c5	68	# iterate all tests on following options
	69	my @tryopt = (
	70	{ on_connect_do => $on_connect_sql },
	71	{ quote_char => '"', on_connect_do => $on_connect_sql, },
	72	);
	73
	74	# keep a database handle open for cleanup
	75	my $dbh;
0567538f	76
bf51641f	77	# test insert returning
	78
	79	# check if we indeed do support stuff
	80	my $test_server_supports_insert_returning = do {
	81	my $v = DBICTest::Schema->connect($dsn, $user, $pass)
	82	->storage
	83	->_get_dbh
	84	->get_info(18);
	85	$v =~ /^(\d+)\.(\d+)/
	86	or die "Unparseable Oracle server version: $v\n";
	87
	88	# TODO find out which version supports the RETURNING syntax
	89	# 8i has it and earlier docs are a 404 on oracle.com
	90	( $1 > 8 \|\| ($1 == 8 && $2 >= 1) ) ? 1 : 0;
	91	};
	92	is (
	93	DBICTest::Schema->connect($dsn, $user, $pass)->storage->_use_insert_returning,
	94	$test_server_supports_insert_returning,
	95	'insert returning capability guessed correctly'
	96	);
	97
	98	my $schema;
	99	for my $use_insert_returning ($test_server_supports_insert_returning
	100	? (1,0)
	101	: (0)
	102	) {
	103
	104	no warnings qw/once/;
	105	local *DBICTest::Schema::connection = subname 'DBICTest::Schema::connection' => sub {
	106	my $s = shift->next::method (@_);
	107	$s->storage->_use_insert_returning ($use_insert_returning);
	108	$s;
	109	};
	110
	111	for my $opt (@tryopt) {
	112	# clean all cached sequences from previous run
	113	for (map { values %{DBICTest::Schema->source($_)->columns_info} } (qw/Artist CD Track/) ) {
	114	delete $_->{sequence};
	115	}
	116
	117	my $schema = DBICTest::Schema->connect($dsn, $user, $pass, $opt);
07cda1c5	118
bf51641f	119	$dbh = $schema->storage->dbh;
	120	my $q = $schema->storage->sql_maker->quote_char \|\| '';
	121
	122	do_creates($dbh, $q);
	123
	124	_run_tests($schema, $opt);
	125	}
	126	}
07cda1c5	127
bf51641f	128	sub _run_tests {
bf51641f	129	my ($schema, $opt) = @_;
07cda1c5	130
bf51641f	131	my $q = $schema->storage->sql_maker->quote_char \|\| '';
ab4f4e4c	132
ab4f4e4c	133	# test primary key handling with multiple triggers
07cda1c5	134	my ($new, $seq);
0567538f	135
bf51641f	136	my $new_artist = $schema->resultset('Artist')->create({ name => 'foo' });
bf51641f	137	my $new_cd = $schema->resultset('CD')->create({ artist => 1, title => 'EP C', year => '2003' });
6f5f880d	138
bf51641f	139	SKIP: {
	140	skip 'not detecting sequences when using INSERT ... RETURNING', 4
	141	if $schema->storage->_use_insert_returning;
	142
	143	is($new_artist->artistid, 1, "Oracle Auto-PK worked for standard sqlt-like trigger");
	144	$seq = $new_artist->result_source->column_info('artistid')->{sequence};
	145	$seq = $$seq if ref $seq;
	146	like ($seq, qr/\.${q}artist_pk_seq${q}$/, 'Correct PK sequence selected for sqlt-like trigger');
	147
	148	is($new_cd->cdid, 1, 'Oracle Auto-PK worked - using scalar ref as table name/custom weird trigger');
	149	$seq = $new_cd->result_source->column_info('cdid')->{sequence};
	150	$seq = $$seq if ref $seq;
	151	like ($seq, qr/\.${q}cd_seq${q}$/, 'Correct PK sequence selected for custom trigger');
	152	}
e6dd7b42	153
07cda1c5	154	# test PKs again with fully-qualified table name
	155	my $artistfqn_rs = $schema->resultset('ArtistFQN');
	156	my $artist_rsrc = $artistfqn_rs->result_source;
ab4f4e4c	157
07cda1c5	158	delete $artist_rsrc->column_info('artistid')->{sequence};
07cda1c5	159	$new = $artistfqn_rs->create( { name => 'bar' } );
ab4f4e4c	160
bf51641f	161	is_deeply( {map { $_ => $new->$_ } $artist_rsrc->primary_columns},
	162	{ artistid => 2, autoinc_col => 2},
	163	"Oracle Multi-Auto-PK worked with fully-qualified tablename" );
cb464582	164
ab4f4e4c	165
07cda1c5	166	delete $artist_rsrc->column_info('artistid')->{sequence};
	167	$new = $artistfqn_rs->create( { name => 'bar', autoinc_col => 1000 } );
	168
	169	is( $new->artistid, 3, "Oracle Auto-PK worked with fully-qualified tablename" );
	170	is( $new->autoinc_col, 1000, "Oracle Auto-Inc overruled with fully-qualified tablename");
bf51641f	171
	172	SKIP: {
	173	skip 'not detecting sequences when using INSERT ... RETURNING', 1
	174	if $schema->storage->_use_insert_returning;
	175
	176	$seq = $new->result_source->column_info('artistid')->{sequence};
	177	$seq = $$seq if ref $seq;
	178	like ($seq, qr/\.${q}artist_pk_seq${q}$/, 'Correct PK sequence selected for sqlt-like trigger');
	179	}
ab4f4e4c	180
ab4f4e4c	181
ab4f4e4c	182	# test LIMIT support
07cda1c5	183	for (1..6) {
ab4f4e4c	184	$schema->resultset('Artist')->create({ name => 'Artist ' . $_ });
07cda1c5	185	}
	186	my $it = $schema->resultset('Artist')->search( { name => { -like => 'Artist %' } }, {
	187	rows => 3,
	188	offset => 4,
	189	order_by => 'artistid'
	190	});
	191
	192	is( $it->count, 2, "LIMIT count past end of RS ok" );
	193	is( $it->next->name, "Artist 5", "iterator->next ok" );
	194	is( $it->next->name, "Artist 6", "iterator->next ok" );
	195	is( $it->next, undef, "next past end of resultset ok" );
	196
	197
	198	# test identifiers over the 30 char limit
	199	lives_ok {
	200	my @results = $schema->resultset('CD')->search(undef, {
	201	prefetch => 'very_long_artist_relationship',
	202	rows => 3,
	203	offset => 0,
	204	})->all;
	205	ok( scalar @results > 0, 'limit with long identifiers returned something');
	206	} 'limit with long identifiers executed successfully';
ab4f4e4c	207
ab4f4e4c	208
62d4dbae	209	# test rel names over the 30 char limit
ab4f4e4c	210	my $query = $schema->resultset('Artist')->search({
ab4f4e4c	211	artistid => 1
6c0230de	212	}, {
	213	prefetch => 'cds_very_very_very_long_relationship_name'
	214	});
	215
	216	lives_and {
07cda1c5	217	is $query->first->cds_very_very_very_long_relationship_name->first->cdid, 1
ab4f4e4c	218	} 'query with rel name over 30 chars survived and worked';
	219
	220	# rel name over 30 char limit with user condition
	221	# This requires walking the SQLA data structure.
	222	{
	223	local $TODO = 'user condition on rel longer than 30 chars';
	224
	225	$query = $schema->resultset('Artist')->search({
	226	'cds_very_very_very_long_relationship_name.title' => 'EP C'
	227	}, {
	228	prefetch => 'cds_very_very_very_long_relationship_name'
	229	});
	230
	231	lives_and {
	232	is $query->first->cds_very_very_very_long_relationship_name->first->cdid, 1
	233	} 'query with rel name over 30 chars and user condition survived and worked';
	234	}
07cda1c5	235
6c0230de	236
9900b569	237	# test join with row count ambiguity
07cda1c5	238	my $cd = $schema->resultset('CD')->next;
	239	my $track = $cd->create_related('tracks', { position => 1, title => 'Track1'} );
	240	my $tjoin = $schema->resultset('Track')->search({ 'me.title' => 'Track1'}, {
	241	join => 'cd', rows => 2
	242	});
d2a3958e	243
07cda1c5	244	ok(my $row = $tjoin->next);
	245
	246	is($row->title, 'Track1', "ambiguous column ok");
2660b14e	247
d2a3958e	248
2660b14e	249
286f32b3	250	# check count distinct with multiple columns
07cda1c5	251	my $other_track = $schema->resultset('Track')->create({ cd => $cd->cdid, position => 1, title => 'Track2' });
11d68671	252
07cda1c5	253	my $tcount = $schema->resultset('Track')->search(
	254	{},
	255	{
	256	select => [ qw/position title/ ],
	257	distinct => 1,
	258	}
	259	);
	260	is($tcount->count, 2, 'multiple column COUNT DISTINCT ok');
11d68671	261
07cda1c5	262	$tcount = $schema->resultset('Track')->search(
	263	{},
	264	{
	265	columns => [ qw/position title/ ],
	266	distinct => 1,
	267	}
	268	);
	269	is($tcount->count, 2, 'multiple column COUNT DISTINCT ok');
286f32b3	270
07cda1c5	271	$tcount = $schema->resultset('Track')->search(
	272	{},
	273	{
	274	group_by => [ qw/position title/ ]
	275	}
	276	);
	277	is($tcount->count, 2, 'multiple column COUNT DISTINCT using column syntax ok');
2660b14e	278
e8e971f2	279
07cda1c5	280	# check group_by
	281	my $g_rs = $schema->resultset('Track')->search( undef, { columns=>[qw/trackid position/], group_by=> [ qw/trackid position/ ] , rows => 2, offset => 1 });
	282	is( scalar $g_rs->all, 1, "Group by with limit OK" );
	283
bd691933	284
b7b18f32	285	# test with_deferred_fk_checks
07cda1c5	286	lives_ok {
	287	$schema->storage->with_deferred_fk_checks(sub {
	288	$schema->resultset('Track')->create({
	289	trackid => 999, cd => 999, position => 1, title => 'deferred FK track'
	290	});
	291	$schema->resultset('CD')->create({
	292	artist => 1, cdid => 999, year => '2003', title => 'deferred FK cd'
	293	});
b7b18f32	294	});
07cda1c5	295	} 'with_deferred_fk_checks code survived';
b7b18f32	296
07cda1c5	297	is eval { $schema->resultset('Track')->find(999)->title }, 'deferred FK track',
	298	'code in with_deferred_fk_checks worked';
	299
	300	throws_ok {
	301	$schema->resultset('Track')->create({
	302	trackid => 1, cd => 9999, position => 1, title => 'Track1'
	303	});
	304	} qr/constraint/i, 'with_deferred_fk_checks is off';
b7b18f32	305
b7b18f32	306
ccd6f984	307	# test auto increment using sequences WITHOUT triggers
07cda1c5	308	for (1..5) {
39b8d119	309	my $st = $schema->resultset('SequenceTest')->create({ name => 'foo' });
	310	is($st->pkid1, $_, "Oracle Auto-PK without trigger: First primary key");
	311	is($st->pkid2, $_ + 9, "Oracle Auto-PK without trigger: Second primary key");
	312	is($st->nonpkid, $_ + 19, "Oracle Auto-PK without trigger: Non-primary key");
07cda1c5	313	}
	314	my $st = $schema->resultset('SequenceTest')->create({ name => 'foo', pkid1 => 55 });
	315	is($st->pkid1, 55, "Oracle Auto-PK without trigger: First primary key set manually");
	316
ccd6f984	317
12e05c15	318	# test BLOBs
07cda1c5	319	SKIP: {
	320	TODO: {
	321	my %binstr = ( 'small' => join('', map { chr($_) } ( 1 .. 127 )) );
	322	$binstr{'large'} = $binstr{'small'} x 1024;
8068691e	323
07cda1c5	324	my $maxloblen = length $binstr{'large'};
	325	note "Localizing LongReadLen to $maxloblen to avoid truncation of test data";
	326	local $dbh->{'LongReadLen'} = $maxloblen;
5db2758d	327
07cda1c5	328	my $rs = $schema->resultset('BindType');
07cda1c5	329	my $id = 0;
5db2758d	330
07cda1c5	331	if ($DBD::Oracle::VERSION eq '1.23') {
	332	throws_ok { $rs->create({ id => 1, blob => $binstr{large} }) }
	333	qr/broken/,
	334	'throws on blob insert with DBD::Oracle == 1.23';
5db2758d	335
07cda1c5	336	skip 'buggy BLOB support in DBD::Oracle 1.23', 7;
07cda1c5	337	}
5db2758d	338
07cda1c5	339	# disable BLOB mega-output
	340	my $orig_debug = $schema->storage->debug;
	341	$schema->storage->debug (0);
f3f6c13a	342
07cda1c5	343	local $TODO = 'Something is confusing column bindtype assignment when quotes are active'
07cda1c5	344	if $q;
5db2758d	345
07cda1c5	346	foreach my $type (qw( blob clob )) {
	347	foreach my $size (qw( small large )) {
	348	$id++;
d7f20fdf	349
07cda1c5	350	lives_ok { $rs->create( { 'id' => $id, $type => $binstr{$size} } ) }
	351	"inserted $size $type without dying";
	352	ok($rs->find($id)->$type eq $binstr{$size}, "verified inserted $size $type" );
	353	}
d7f20fdf	354	}
f3f6c13a	355
07cda1c5	356	$schema->storage->debug ($orig_debug);
	357	}}
	358
5db2758d	359
df6e3f5c	360	# test sequence detection from a different schema
07cda1c5	361	SKIP: {
	362	TODO: {
	363	skip ((join '',
	364	'Set DBICTEST_ORA_EXTRAUSER_DSN, _USER and _PASS to a DIFFERENT Oracle user',
bf51641f	365	' to run the cross-schema sequence detection test.'),
07cda1c5	366	1) unless $dsn2 && $user2 && $user2 ne $user;
9d7d2f00	367
bf51641f	368	skip 'not detecting cross-schema sequence name when using INSERT ... RETURNING', 1
	369	if $schema->storage->_use_insert_returning;
	370
07cda1c5	371	# Oracle8i Reference Release 2 (8.1.6)
	372	# http://download.oracle.com/docs/cd/A87860_01/doc/server.817/a76961/ch294.htm#993
	373	# Oracle Database Reference 10g Release 2 (10.2)
	374	# http://download.oracle.com/docs/cd/B19306_01/server.102/b14237/statviews_2107.htm#sthref1297
	375	local $TODO = "On Oracle8i all_triggers view is empty, i don't yet know why..."
	376	if $schema->storage->_server_info->{normalized_dbms_version} < 9;
606b30c3	377
bf51641f	378	my $schema2 = $schema->connect($dsn2, $user2, $pass2, $opt);
df6e3f5c	379
07cda1c5	380	my $schema1_dbh = $schema->storage->dbh;
	381	$schema1_dbh->do("GRANT INSERT ON ${q}artist${q} TO " . uc $user2);
	382	$schema1_dbh->do("GRANT SELECT ON ${q}artist_pk_seq${q} TO " . uc $user2);
bf51641f	383	$schema1_dbh->do("GRANT SELECT ON ${q}artist_autoinc_seq${q} TO " . uc $user2);
df6e3f5c	384
df6e3f5c	385
07cda1c5	386	my $rs = $schema2->resultset('ArtistFQN');
07cda1c5	387	delete $rs->result_source->column_info('artistid')->{sequence};
df6e3f5c	388
07cda1c5	389	lives_and {
	390	my $row = $rs->create({ name => 'From Different Schema' });
	391	ok $row->artistid;
	392	} 'used autoinc sequence across schemas';
	393
	394	# now quote the sequence name (do_creates always uses an lc name)
	395	my $q_seq = $q
	396	? '"artist_pk_seq"'
	397	: '"ARTIST_PK_SEQ"'
	398	;
	399	delete $rs->result_source->column_info('artistid')->{sequence};
	400	$schema1_dbh->do(qq{
	401	CREATE OR REPLACE TRIGGER ${q}artist_insert_trg_pk${q}
	402	BEFORE INSERT ON ${q}artist${q}
	403	FOR EACH ROW
	404	BEGIN
	405	IF :new.${q}artistid${q} IS NULL THEN
	406	SELECT $q_seq.nextval
	407	INTO :new.${q}artistid${q}
	408	FROM DUAL;
	409	END IF;
	410	END;
	411	});
72044892	412
72044892	413
07cda1c5	414	lives_and {
	415	my $row = $rs->create({ name => 'From Different Schema With Quoted Sequence' });
	416	ok $row->artistid;
	417	} 'used quoted autoinc sequence across schemas';
72044892	418
07cda1c5	419	my $schema_name = uc $user;
72044892	420
07cda1c5	421	is_deeply $rs->result_source->column_info('artistid')->{sequence},
	422	\qq\|${schema_name}.$q_seq\|,
	423	'quoted sequence name correctly extracted';
	424	}}
	425
	426	do_clean ($dbh);
	427	}
df6e3f5c	428
86cc4156	429	done_testing;
86cc4156	430
df6e3f5c	431	sub do_creates {
07cda1c5	432	my ($dbh, $q) = @_;
	433
	434	do_clean($dbh);
	435
	436	$dbh->do("CREATE SEQUENCE ${q}artist_autoinc_seq${q} START WITH 1 MAXVALUE 999999 MINVALUE 0");
	437	$dbh->do("CREATE SEQUENCE ${q}artist_pk_seq${q} START WITH 1 MAXVALUE 999999 MINVALUE 0");
	438	$dbh->do("CREATE SEQUENCE ${q}cd_seq${q} START WITH 1 MAXVALUE 999999 MINVALUE 0");
	439	$dbh->do("CREATE SEQUENCE ${q}track_seq${q} START WITH 1 MAXVALUE 999999 MINVALUE 0");
	440
	441	$dbh->do("CREATE SEQUENCE ${q}nonpkid_seq${q} START WITH 20 MAXVALUE 999999 MINVALUE 0");
	442	# this one is always quoted as per manually specified sequence =>
	443	$dbh->do('CREATE SEQUENCE "pkid1_seq" START WITH 1 MAXVALUE 999999 MINVALUE 0');
	444	# this one is always unquoted as per manually specified sequence =>
df6e3f5c	445	$dbh->do("CREATE SEQUENCE pkid2_seq START WITH 10 MAXVALUE 999999 MINVALUE 0");
df6e3f5c	446
07cda1c5	447	$dbh->do("CREATE TABLE ${q}artist${q} (${q}artistid${q} NUMBER(12), ${q}name${q} VARCHAR(255), ${q}autoinc_col${q} NUMBER(12), ${q}rank${q} NUMBER(38), ${q}charfield${q} VARCHAR2(10))");
07cda1c5	448	$dbh->do("ALTER TABLE ${q}artist${q} ADD (CONSTRAINT ${q}artist_pk${q} PRIMARY KEY (${q}artistid${q}))");
df6e3f5c	449
07cda1c5	450	$dbh->do("CREATE TABLE ${q}sequence_test${q} (${q}pkid1${q} NUMBER(12), ${q}pkid2${q} NUMBER(12), ${q}nonpkid${q} NUMBER(12), ${q}name${q} VARCHAR(255))");
07cda1c5	451	$dbh->do("ALTER TABLE ${q}sequence_test${q} ADD (CONSTRAINT ${q}sequence_test_constraint${q} PRIMARY KEY (${q}pkid1${q}, ${q}pkid2${q}))");
df6e3f5c	452
07cda1c5	453	# table cd will be unquoted => Oracle will see it as uppercase
	454	$dbh->do("CREATE TABLE cd (${q}cdid${q} NUMBER(12), ${q}artist${q} NUMBER(12), ${q}title${q} VARCHAR(255), ${q}year${q} VARCHAR(4), ${q}genreid${q} NUMBER(12), ${q}single_track${q} NUMBER(12))");
	455	$dbh->do("ALTER TABLE cd ADD (CONSTRAINT ${q}cd_pk${q} PRIMARY KEY (${q}cdid${q}))");
df6e3f5c	456
07cda1c5	457	$dbh->do("CREATE TABLE ${q}track${q} (${q}trackid${q} NUMBER(12), ${q}cd${q} NUMBER(12) REFERENCES CD(${q}cdid${q}) DEFERRABLE, ${q}position${q} NUMBER(12), ${q}title${q} VARCHAR(255), ${q}last_updated_on${q} DATE, ${q}last_updated_at${q} DATE, ${q}small_dt${q} DATE)");
07cda1c5	458	$dbh->do("ALTER TABLE ${q}track${q} ADD (CONSTRAINT ${q}track_pk${q} PRIMARY KEY (${q}trackid${q}))");
df6e3f5c	459
07cda1c5	460	$dbh->do("CREATE TABLE ${q}bindtype_test${q} (${q}id${q} integer NOT NULL PRIMARY KEY, ${q}bytea${q} integer NULL, ${q}blob${q} blob NULL, ${q}clob${q} clob NULL)");
12e05c15	461
df6e3f5c	462	$dbh->do(qq{
07cda1c5	463	CREATE OR REPLACE TRIGGER ${q}artist_insert_trg_auto${q}
07cda1c5	464	BEFORE INSERT ON ${q}artist${q}
ab4f4e4c	465	FOR EACH ROW
ab4f4e4c	466	BEGIN
07cda1c5	467	IF :new.${q}autoinc_col${q} IS NULL THEN
	468	SELECT ${q}artist_autoinc_seq${q}.nextval
	469	INTO :new.${q}autoinc_col${q}
ab4f4e4c	470	FROM DUAL;
	471	END IF;
	472	END;
	473	});
07cda1c5	474
ab4f4e4c	475	$dbh->do(qq{
07cda1c5	476	CREATE OR REPLACE TRIGGER ${q}artist_insert_trg_pk${q}
07cda1c5	477	BEFORE INSERT ON ${q}artist${q}
df6e3f5c	478	FOR EACH ROW
df6e3f5c	479	BEGIN
07cda1c5	480	IF :new.${q}artistid${q} IS NULL THEN
	481	SELECT ${q}artist_pk_seq${q}.nextval
	482	INTO :new.${q}artistid${q}
df6e3f5c	483	FROM DUAL;
	484	END IF;
	485	END;
	486	});
07cda1c5	487
df6e3f5c	488	$dbh->do(qq{
07cda1c5	489	CREATE OR REPLACE TRIGGER ${q}cd_insert_trg${q}
df6e3f5c	490	BEFORE INSERT OR UPDATE ON cd
df6e3f5c	491	FOR EACH ROW
07cda1c5	492
6f5f880d	493	DECLARE
	494	tmpVar NUMBER;
	495
df6e3f5c	496	BEGIN
6f5f880d	497	tmpVar := 0;
6f5f880d	498
07cda1c5	499	IF :new.${q}cdid${q} IS NULL THEN
07cda1c5	500	SELECT ${q}cd_seq${q}.nextval
6f5f880d	501	INTO tmpVar
	502	FROM dual;
	503
07cda1c5	504	:new.${q}cdid${q} := tmpVar;
df6e3f5c	505	END IF;
	506	END;
	507	});
07cda1c5	508
df6e3f5c	509	$dbh->do(qq{
07cda1c5	510	CREATE OR REPLACE TRIGGER ${q}track_insert_trg${q}
07cda1c5	511	BEFORE INSERT ON ${q}track${q}
df6e3f5c	512	FOR EACH ROW
df6e3f5c	513	BEGIN
07cda1c5	514	IF :new.${q}trackid${q} IS NULL THEN
	515	SELECT ${q}track_seq${q}.nextval
	516	INTO :new.${q}trackid${q}
df6e3f5c	517	FROM DUAL;
	518	END IF;
	519	END;
	520	});
	521	}
	522
0567538f	523	# clean up our mess
07cda1c5	524	sub do_clean {
	525
	526	my $dbh = shift \|\| return;
	527
	528	for my $q ('', '"') {
	529	my @clean = (
	530	"DROP TRIGGER ${q}track_insert_trg${q}",
	531	"DROP TRIGGER ${q}cd_insert_trg${q}",
	532	"DROP TRIGGER ${q}artist_insert_trg_auto${q}",
	533	"DROP TRIGGER ${q}artist_insert_trg_pk${q}",
	534	"DROP SEQUENCE ${q}nonpkid_seq${q}",
	535	"DROP SEQUENCE ${q}pkid2_seq${q}",
	536	"DROP SEQUENCE ${q}pkid1_seq${q}",
	537	"DROP SEQUENCE ${q}track_seq${q}",
	538	"DROP SEQUENCE ${q}cd_seq${q}",
	539	"DROP SEQUENCE ${q}artist_autoinc_seq${q}",
	540	"DROP SEQUENCE ${q}artist_pk_seq${q}",
	541	"DROP TABLE ${q}bindtype_test${q}",
	542	"DROP TABLE ${q}sequence_test${q}",
	543	"DROP TABLE ${q}track${q}",
	544	"DROP TABLE ${q}cd${q}",
	545	"DROP TABLE ${q}artist${q}",
	546	);
	547	eval { $dbh -> do ($_) } for @clean;
	548	}
	549	}
	550
3ff5b740	551	END {
07cda1c5	552	if ($dbh) {
	553	local $SIG{__WARN__} = sub {};
	554	do_clean($dbh);
	555	$dbh->disconnect;
df6e3f5c	556	}
3ff5b740	557	}