X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=t%2F94versioning.t;h=a21141a65fb93043ac472fe28f7cec03f2d2c6a0;hb=4549ba4966c026a49409c7dde70f32b2512250b0;hp=1c8026cd697fa436ab3448eb096b8241fcf58472;hpb=e2c0df8e0b707050eb005ac6f68548f857a36acf;p=dbsrgits%2FDBIx-Class.git diff --git a/t/94versioning.t b/t/94versioning.t index 1c8026c..a21141a 100644 --- a/t/94versioning.t +++ b/t/94versioning.t @@ -1,50 +1,274 @@ -#!/usr/bin/perl use strict; use warnings; + use Test::More; +use Test::Warn; +use Test::Exception; + +my ($dsn, $user, $pass); BEGIN { - eval "use DBD::SQLite; use SQL::Translator;"; - plan $@ - ? ( skip_all => 'needs DBD::SQLite and SQL::Translator for testing' ) - : ( tests => 6 ); + ($dsn, $user, $pass) = @ENV{map { "DBICTEST_MYSQL_${_}" } qw/DSN USER PASS/}; + + plan skip_all => 'Set $ENV{DBICTEST_MYSQL_DSN}, _USER and _PASS to run this test' + unless ($dsn); + + require DBIx::Class; + plan skip_all => + 'Test needs ' . DBIx::Class::Optional::Dependencies->req_missing_for ('deploy') + unless DBIx::Class::Optional::Dependencies->req_ok_for ('deploy') } +use Path::Class; +use File::Copy; +use Time::HiRes qw/time sleep/; + use lib qw(t/lib); +use DBICTest; # do not remove even though it is not used + +use_ok('DBICVersion_v1'); + +my $version_table_name = 'dbix_class_schema_versions'; +my $old_table_name = 'SchemaVersions'; + +my $ddl_dir = dir ('t', 'var'); +mkdir ($ddl_dir) unless -d $ddl_dir; + +my $fn = { + v1 => $ddl_dir->file ('DBICVersion-Schema-1.0-MySQL.sql'), + v2 => $ddl_dir->file ('DBICVersion-Schema-2.0-MySQL.sql'), + v3 => $ddl_dir->file ('DBICVersion-Schema-3.0-MySQL.sql'), + trans_v12 => $ddl_dir->file ('DBICVersion-Schema-1.0-2.0-MySQL.sql'), + trans_v23 => $ddl_dir->file ('DBICVersion-Schema-2.0-3.0-MySQL.sql'), +}; + +my $schema_v1 = DBICVersion::Schema->connect($dsn, $user, $pass, { ignore_version => 1 }); +eval { $schema_v1->storage->dbh->do('drop table ' . $version_table_name) }; +eval { $schema_v1->storage->dbh->do('drop table ' . $old_table_name) }; + +is($schema_v1->ddl_filename('MySQL', '1.0', $ddl_dir), $fn->{v1}, 'Filename creation working'); +unlink( $fn->{v1} ) if ( -e $fn->{v1} ); +$schema_v1->create_ddl_dir('MySQL', undef, $ddl_dir); + +ok(-f $fn->{v1}, 'Created DDL file'); +$schema_v1->deploy({ add_drop_table => 1 }); + +my $tvrs = $schema_v1->{vschema}->resultset('Table'); +is($schema_v1->_source_exists($tvrs), 1, 'Created schema from DDL file'); + +# loading a new module defining a new version of the same table +DBICVersion::Schema->_unregister_source ('Table'); +use_ok('DBICVersion_v2'); + +my $schema_v2 = DBICVersion::Schema->connect($dsn, $user, $pass, { ignore_version => 1 }); +{ + unlink($fn->{v2}); + unlink($fn->{trans_v12}); -use_ok('DBICVersionOrig'); + is($schema_v2->get_db_version(), '1.0', 'get_db_version ok'); + is($schema_v2->schema_version, '2.0', 'schema version ok'); + $schema_v2->create_ddl_dir('MySQL', '2.0', $ddl_dir, '1.0'); + ok(-f $fn->{trans_v12}, 'Created DDL file'); -my $db_file = "t/var/versioning.db"; -unlink($db_file) if -e $db_file; -unlink($db_file . "-journal") if -e $db_file . "-journal"; -mkdir("t/var") unless -d "t/var"; -unlink('t/var/DBICVersion-Schema-1.0-SQLite.sql'); + warnings_like ( + sub { $schema_v2->upgrade() }, + qr/DB version .+? is lower than the schema version/, + 'Warn before upgrade', + ); -my $schema_orig = DBICVersion::Schema->connect("dbi:SQLite:$db_file"); -# $schema->storage->ensure_connected(); + is($schema_v2->get_db_version(), '2.0', 'db version number upgraded'); -is($schema_orig->ddl_filename('SQLite', 't/var', '1.0'), 't/var/DBICVersion-Schema-1.0-SQLite.sql', 'Filename creation working'); -$schema_orig->create_ddl_dir('SQLite', undef, 't/var'); + lives_ok ( sub { + $schema_v2->storage->dbh->do('select NewVersionName from TestVersion'); + }, 'new column created' ); -ok(-f 't/var/DBICVersion-Schema-1.0-SQLite.sql', 'Created DDL file'); -## do this here or let Versioned.pm do it? -# $schema->deploy(); + warnings_exist ( + sub { $schema_v2->create_ddl_dir('MySQL', '2.0', $ddl_dir, '1.0') }, + [ + qr/Overwriting existing DDL file - $fn->{v2}/, + qr/Overwriting existing diff file - $fn->{trans_v12}/, + ], + 'An overwrite warning generated for both the DDL and the diff', + ); +} + +{ + my $schema_version = DBICVersion::Schema->connect($dsn, $user, $pass); + lives_ok (sub { + $schema_version->storage->dbh->do('select * from ' . $version_table_name); + }, 'version table exists'); + + lives_ok (sub { + $schema_version->storage->dbh->do("DROP TABLE IF EXISTS $old_table_name"); + $schema_version->storage->dbh->do("RENAME TABLE $version_table_name TO $old_table_name"); + }, 'versions table renamed to old style table'); + + $schema_version = DBICVersion::Schema->connect($dsn, $user, $pass); + is($schema_version->get_db_version, '2.0', 'transition from old table name to new okay'); + + dies_ok (sub { + $schema_version->storage->dbh->do('select * from ' . $old_table_name); + }, 'old version table gone'); + +} + +# repeat the v1->v2 process for v2->v3 before testing v1->v3 +DBICVersion::Schema->_unregister_source ('Table'); +use_ok('DBICVersion_v3'); -my $tvrs = $schema_orig->resultset('Table'); -is($schema_orig->exists($tvrs), 1, 'Created schema from DDL file'); +my $schema_v3 = DBICVersion::Schema->connect($dsn, $user, $pass, { ignore_version => 1 }); +{ + unlink($fn->{v3}); + unlink($fn->{trans_v23}); -eval "use DBICVersionNew"; -my $schema_new = DBICVersion::Schema->connect("dbi:SQLite:$db_file"); + is($schema_v3->get_db_version(), '2.0', 'get_db_version 2.0 ok'); + is($schema_v3->schema_version, '3.0', 'schema version 3.0 ok'); + $schema_v3->create_ddl_dir('MySQL', '3.0', $ddl_dir, '2.0'); + ok(-f $fn->{trans_v23}, 'Created DDL 2.0 -> 3.0 file'); -unlink('t/var/DBICVersion-Schema-2.0-SQLite.sql'); -unlink('t/var/DBICVersion-Schema-1.0-2.0-SQLite.sql'); -$schema_new->create_ddl_dir('SQLite', undef, 't/var', '1.0'); -ok(-f 't/var/DBICVersion-Schema-1.0-2.0-SQLite.sql', 'Created DDL upgrade file'); + warnings_exist ( + sub { $schema_v3->upgrade() }, + qr/DB version .+? is lower than the schema version/, + 'Warn before upgrade', + ); -## create new to pick up filedata for upgrade files we just made (on_connect) -my $schema_upgrade = DBICVersion::Schema->connect("dbi:SQLite:$db_file"); + is($schema_v3->get_db_version(), '3.0', 'db version number upgraded'); + + lives_ok ( sub { + $schema_v3->storage->dbh->do('select ExtraColumn from TestVersion'); + }, 'new column created'); +} + +# now put the v1 schema back again +{ + # drop all the tables... + eval { $schema_v1->storage->dbh->do('drop table ' . $version_table_name) }; + eval { $schema_v1->storage->dbh->do('drop table ' . $old_table_name) }; + eval { $schema_v1->storage->dbh->do('drop table TestVersion') }; + + { + local $DBICVersion::Schema::VERSION = '1.0'; + $schema_v1->deploy; + } + is($schema_v1->get_db_version(), '1.0', 'get_db_version 1.0 ok'); +} + +# attempt v1 -> v3 upgrade +{ + local $SIG{__WARN__} = sub { warn if $_[0] !~ /Attempting upgrade\.$/ }; + $schema_v3->upgrade(); + is($schema_v3->get_db_version(), '3.0', 'db version number upgraded'); +} + +# Now, try a v1 -> v3 upgrade with a file that has comments strategically placed in it. +# First put the v1 schema back again... +{ + # drop all the tables... + eval { $schema_v1->storage->dbh->do('drop table ' . $version_table_name) }; + eval { $schema_v1->storage->dbh->do('drop table ' . $old_table_name) }; + eval { $schema_v1->storage->dbh->do('drop table TestVersion') }; + + { + local $DBICVersion::Schema::VERSION = '1.0'; + $schema_v1->deploy; + } + is($schema_v1->get_db_version(), '1.0', 'get_db_version 1.0 ok'); +} + +# add a "harmless" comment before one of the statements. +system( qq($^X -pi -e "s/ALTER/-- this is a comment\nALTER/" $fn->{trans_v23};) ); + +# Then attempt v1 -> v3 upgrade +{ + local $SIG{__WARN__} = sub { warn if $_[0] !~ /Attempting upgrade\.$/ }; + $schema_v3->upgrade(); + is($schema_v3->get_db_version(), '3.0', 'db version number upgraded to 3.0'); + + # make sure that the column added after the comment is actually added. + lives_ok ( sub { + $schema_v3->storage->dbh->do('select ExtraColumn from TestVersion'); + }, 'new column created'); +} + + +# check behaviour of DBIC_NO_VERSION_CHECK env var and ignore_version connect attr +{ + my $schema_version = DBICVersion::Schema->connect($dsn, $user, $pass); + eval { + $schema_version->storage->dbh->do("DELETE from $version_table_name"); + }; + + + warnings_like ( sub { + $schema_version = DBICVersion::Schema->connect($dsn, $user, $pass); + }, qr/Your DB is currently unversioned/, 'warning detected without env var or attr' ); + + warnings_like ( sub { + $schema_version = DBICVersion::Schema->connect($dsn, $user, $pass, { ignore_version => 1 }); + }, [], 'warning not detected with attr set'); + + + local $ENV{DBIC_NO_VERSION_CHECK} = 1; + warnings_like ( sub { + $schema_version = DBICVersion::Schema->connect($dsn, $user, $pass); + }, [], 'warning not detected with env var set'); + + warnings_like ( sub { + $schema_version = DBICVersion::Schema->connect($dsn, $user, $pass, { ignore_version => 0 }); + }, qr/Your DB is currently unversioned/, 'warning detected without env var or attr'); +} + +# attempt a deploy/upgrade cycle within one second +{ + eval { $schema_v2->storage->dbh->do('drop table ' . $version_table_name) }; + eval { $schema_v2->storage->dbh->do('drop table ' . $old_table_name) }; + eval { $schema_v2->storage->dbh->do('drop table TestVersion') }; + + # this attempts to sleep until the turn of the second + my $t = time(); + sleep (int ($t) + 1 - $t); + note ('Fast deploy/upgrade start: ', time() ); + + { + local $DBICVersion::Schema::VERSION = '2.0'; + $schema_v2->deploy; + } + + local $SIG{__WARN__} = sub { warn if $_[0] !~ /Attempting upgrade\.$/ }; + $schema_v2->upgrade(); + + is($schema_v2->get_db_version(), '3.0', 'Fast deploy/upgrade'); +}; + +# Check that it Schema::Versioned deals with new/all forms of connect arguments. +{ + my $get_db_version_run = 0; + + no warnings qw/once redefine/; + local *DBIx::Class::Schema::Versioned::get_db_version = sub { + $get_db_version_run = 1; + return $_[0]->schema_version; + }; + + # Make sure the env var isn't whats triggering it + local $ENV{DBIC_NO_VERSION_CHECK} = 0; + + DBICVersion::Schema->connect({ + dsn => $dsn, + user => $user, + pass => $pass, + ignore_version => 1 + }); + + ok($get_db_version_run == 0, "attributes pulled from hashref connect_info"); + $get_db_version_run = 0; + + DBICVersion::Schema->connect( $dsn, $user, $pass, { ignore_version => 1 } ); + ok($get_db_version_run == 0, "attributes pulled from list connect_info"); +} + +unless ($ENV{DBICTEST_KEEP_VERSIONING_DDL}) { + unlink $_ for (values %$fn); +} -## do this here or let Versioned.pm do it? -$schema_upgrade->upgrade(); -$tvrs = $schema_upgrade->resultset('Table'); -is($schema_upgrade->exists($tvrs), 1, 'Upgraded schema from DDL file'); +done_testing;