From: Peter Rabbitson Date: Tue, 19 May 2009 17:26:21 +0000 (+0000) Subject: Make sure pg test can restart after half-way failures X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=commitdiff_plain;h=7603d2a5b93541932b94593c0f12e12386c83ef0;p=dbsrgits%2FDBIx-Class-Historic.git Make sure pg test can restart after half-way failures --- diff --git a/t/72pg.t b/t/72pg.t index 2178ce0..05470a6 100644 --- a/t/72pg.t +++ b/t/72pg.t @@ -72,6 +72,8 @@ $schema->source("Artist")->name("testschema.artist"); $schema->source("SequenceTest")->name("testschema.sequence_test"); { local $SIG{__WARN__} = sub {}; + _cleanup ($dbh); + $dbh->do("CREATE SCHEMA testschema;"); $dbh->do("CREATE TABLE testschema.artist (artistid serial PRIMARY KEY, name VARCHAR(100), rank INTEGER NOT NULL DEFAULT '13', charfield CHAR(10), arrayfield INTEGER[]);"); $dbh->do("CREATE TABLE testschema.sequence_test (pkid1 integer, pkid2 integer, nonpkid integer, name VARCHAR(100), CONSTRAINT pk PRIMARY KEY(pkid1, pkid2));"); @@ -264,16 +266,21 @@ for (1..5) { my $st = $schema->resultset('SequenceTest')->create({ name => 'foo', pkid1 => 55 }); is($st->pkid1, 55, "Oracle Auto-PK without trigger: First primary key set manually"); - -END { - if($dbh) { - $dbh->do("DROP TABLE testschema.artist;"); - $dbh->do("DROP TABLE testschema.casecheck;"); - $dbh->do("DROP TABLE testschema.sequence_test;"); - $dbh->do("DROP TABLE testschema.array_test;"); - $dbh->do("DROP SEQUENCE pkid1_seq"); - $dbh->do("DROP SEQUENCE pkid2_seq"); - $dbh->do("DROP SEQUENCE nonpkid_seq"); - $dbh->do("DROP SCHEMA testschema;"); - } +sub _cleanup { + my $dbh = shift or return; + + for my $stat ( + 'DROP TABLE testschema.artist', + 'DROP TABLE testschema.casecheck', + 'DROP TABLE testschema.sequence_test', + 'DROP TABLE testschema.array_test', + 'DROP SEQUENCE pkid1_seq', + 'DROP SEQUENCE pkid2_seq', + 'DROP SEQUENCE nonpkid_seq', + 'DROP SCHEMA testschema', + ) { + eval { $dbh->do ($stat) }; + } } + +END { _cleanup($dbh) }