X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=t%2F30sqlt-new-diff-mysql.t;h=9ed3108ee0b9214a99f73dd4d0c73d823bde9ce5;hb=d8cf2279bd5e25db979d6251c5648d69fc5a298d;hp=4f3ad16e2993fe68adbc6d5d246ccdb0d83f5b81;hpb=74ca32cee1a50c5c166a0e3f8cc1ecdcfe76ad23;p=dbsrgits%2FSQL-Translator.git diff --git a/t/30sqlt-new-diff-mysql.t b/t/30sqlt-new-diff-mysql.t index 4f3ad16..9ed3108 100644 --- a/t/30sqlt-new-diff-mysql.t +++ b/t/30sqlt-new-diff-mysql.t @@ -13,11 +13,11 @@ use Test::SQL::Translator qw(maybe_plan); use SQL::Translator::Schema::Constants; use Storable 'dclone'; -plan tests => 7; +plan tests => 9; use_ok('SQL::Translator::Diff') or die "Cannot continue\n"; -my $tr = SQL::Translator->new; +my $tr = SQL::Translator->new; my ( $source_schema, $target_schema, $parsed_sql_schema ) = map { my $t = SQL::Translator->new; @@ -34,84 +34,117 @@ my ( $source_schema, $target_schema, $parsed_sql_schema ) = map { } (qw( create1.yml create2.yml )); # Test for differences -my $out = SQL::Translator::Diff::schema_diff( $source_schema, 'MySQL', $target_schema, 'MySQL', { no_batch_alters => 1} ); +my @out = SQL::Translator::Diff::schema_diff( + $source_schema, 'MySQL', + $target_schema, 'MySQL', + { + no_batch_alters => 1, + producer_args => { quote_table_names => 0 } + } +); + +ok( @out, 'Got a list' ); + +my $out = join('', @out); + eq_or_diff($out, <<'## END OF DIFF', "Diff as expected"); --- Convert schema 'create1.yml' to 'create2.yml': +-- Convert schema 'create1.yml' to 'create2.yml':; BEGIN; SET foreign_key_checks=0; - CREATE TABLE added ( id integer(11) ); - SET foreign_key_checks=1; - ALTER TABLE old_name RENAME TO new_name; + ALTER TABLE employee DROP FOREIGN KEY FK5302D47D93FE702E; + ALTER TABLE person DROP INDEX UC_age_name; + ALTER TABLE person DROP INDEX u_name; + ALTER TABLE employee DROP COLUMN job_title; + ALTER TABLE new_name ADD COLUMN new_field integer; -ALTER TABLE person ADD COLUMN is_rock_star tinyint(4) DEFAULT '1'; + +ALTER TABLE person ADD COLUMN is_rock_star tinyint(4) DEFAULT 1; + ALTER TABLE person CHANGE COLUMN person_id person_id integer(11) NOT NULL auto_increment; + ALTER TABLE person CHANGE COLUMN name name varchar(20) NOT NULL; -ALTER TABLE person CHANGE COLUMN age age integer(11) DEFAULT '18'; -ALTER TABLE person CHANGE COLUMN iq iq integer(11) DEFAULT '0'; + +ALTER TABLE person CHANGE COLUMN age age integer(11) DEFAULT 18; + +ALTER TABLE person CHANGE COLUMN iq iq integer(11) DEFAULT 0; + ALTER TABLE person CHANGE COLUMN description physical_description text; + ALTER TABLE person ADD UNIQUE INDEX unique_name (name); + ALTER TABLE employee ADD CONSTRAINT FK5302D47D93FE702E_diff FOREIGN KEY (employee_id) REFERENCES person (person_id); + ALTER TABLE person ADD UNIQUE UC_person_id (person_id); + ALTER TABLE person ADD UNIQUE UC_age_name (age, name); + ALTER TABLE person ENGINE=InnoDB; + ALTER TABLE deleted DROP FOREIGN KEY fk_fake; + DROP TABLE deleted; + COMMIT; + ## END OF DIFF $out = SQL::Translator::Diff::schema_diff($source_schema, 'MySQL', $target_schema, 'MySQL', { ignore_index_names => 1, - ignore_constraint_names => 1 + ignore_constraint_names => 1, + producer_args => { quote_table_names => 0 }, }); eq_or_diff($out, <<'## END OF DIFF', "Diff as expected"); --- Convert schema 'create1.yml' to 'create2.yml': +-- Convert schema 'create1.yml' to 'create2.yml':; BEGIN; SET foreign_key_checks=0; - CREATE TABLE added ( id integer(11) ); - SET foreign_key_checks=1; - ALTER TABLE employee DROP COLUMN job_title; + ALTER TABLE old_name RENAME TO new_name, ADD COLUMN new_field integer; + ALTER TABLE person DROP INDEX UC_age_name, - ADD COLUMN is_rock_star tinyint(4) DEFAULT '1', + ADD COLUMN is_rock_star tinyint(4) DEFAULT 1, CHANGE COLUMN person_id person_id integer(11) NOT NULL auto_increment, CHANGE COLUMN name name varchar(20) NOT NULL, - CHANGE COLUMN age age integer(11) DEFAULT '18', - CHANGE COLUMN iq iq integer(11) DEFAULT '0', + CHANGE COLUMN age age integer(11) DEFAULT 18, + CHANGE COLUMN iq iq integer(11) DEFAULT 0, CHANGE COLUMN description physical_description text, ADD UNIQUE UC_person_id (person_id), ADD UNIQUE UC_age_name (age, name), ENGINE=InnoDB; + ALTER TABLE deleted DROP FOREIGN KEY fk_fake; + DROP TABLE deleted; + COMMIT; + ## END OF DIFF @@ -119,9 +152,9 @@ COMMIT; $out = SQL::Translator::Diff::schema_diff($source_schema, 'MySQL', $source_schema, 'MySQL' ); eq_or_diff($out, <<'## END OF DIFF', "No differences found"); --- Convert schema 'create1.yml' to 'create1.yml': +-- Convert schema 'create1.yml' to 'create1.yml':; --- No differences found +-- No differences found; ## END OF DIFF @@ -145,41 +178,42 @@ eq_or_diff($out, <<'## END OF DIFF', "No differences found"); my $field = $target_schema->get_table('employee')->get_field('employee_id'); $field->data_type('integer'); $field->size(0); - $out = SQL::Translator::Diff::schema_diff($schema, 'MySQL', $target_schema, 'MySQL' ); + $out = SQL::Translator::Diff::schema_diff($schema, 'MySQL', $target_schema, 'MySQL', { producer_args => { quote_table_names => 0 } } ); eq_or_diff($out, <<'## END OF DIFF', "No differences found"); --- Convert schema 'create.sql' to 'create2.yml': +-- Convert schema 'create.sql' to 'create2.yml':; BEGIN; SET foreign_key_checks=0; - CREATE TABLE added ( id integer(11) ); - SET foreign_key_checks=1; - ALTER TABLE employee DROP FOREIGN KEY FK5302D47D93FE702E, DROP COLUMN job_title, ADD CONSTRAINT FK5302D47D93FE702E_diff FOREIGN KEY (employee_id) REFERENCES person (person_id); + ALTER TABLE person DROP INDEX UC_age_name, DROP INDEX u_name, - ADD COLUMN is_rock_star tinyint(4) DEFAULT '1', + ADD COLUMN is_rock_star tinyint(4) DEFAULT 1, CHANGE COLUMN person_id person_id integer(11) NOT NULL auto_increment, CHANGE COLUMN name name varchar(20) NOT NULL, - CHANGE COLUMN age age integer(11) DEFAULT '18', - CHANGE COLUMN iq iq integer(11) DEFAULT '0', + CHANGE COLUMN age age integer(11) DEFAULT 18, + CHANGE COLUMN iq iq integer(11) DEFAULT 0, CHANGE COLUMN description physical_description text, ADD UNIQUE INDEX unique_name (name), ADD UNIQUE UC_person_id (person_id), ADD UNIQUE UC_age_name (age, name), ENGINE=InnoDB; + DROP TABLE deleted; + COMMIT; + ## END OF DIFF } @@ -217,16 +251,19 @@ COMMIT; my $out = SQL::Translator::Diff::schema_diff($s1, 'MySQL', $s2, 'MySQL' ); eq_or_diff($out, <<'## END OF DIFF', "Batch alter of constraints work for InnoDB"); --- Convert schema 'Schema 1' to 'Schema 2': +-- Convert schema 'Schema 1' to 'Schema 2':; BEGIN; ALTER TABLE employee DROP FOREIGN KEY FK5302D47D93FE702E_diff; + ALTER TABLE employee ADD COLUMN new integer, ADD CONSTRAINT FK5302D47D93FE702E_diff FOREIGN KEY (employee_id) REFERENCES person (person_id) ON DELETE CASCADE, ADD CONSTRAINT new_constraint FOREIGN KEY (employee_id) REFERENCES patty (fake); + COMMIT; + ## END OF DIFF } @@ -264,7 +301,7 @@ COMMIT; my $out = SQL::Translator::Diff::schema_diff($s1, 'MySQL', $s2, 'MySQL' ); eq_or_diff($out, <<'## END OF DIFF', "Alter/drop constraints works with rename table"); --- Convert schema 'Schema 3' to 'Schema 4': +-- Convert schema 'Schema 3' to 'Schema 4':; BEGIN; @@ -272,6 +309,26 @@ ALTER TABLE employee RENAME TO fnord, DROP FOREIGN KEY bar_fk, ADD CONSTRAINT foo_fk FOREIGN KEY (employee_id) REFERENCES foo (id); + +COMMIT; + +## END OF DIFF + + # Test quoting works too. + $out = SQL::Translator::Diff::schema_diff($s1, 'MySQL', $s2, 'MySQL', + { producer_args => { quote_table_names => '`' } } + ); + eq_or_diff($out, <<'## END OF DIFF', "Quoting can be turned on"); +-- Convert schema 'Schema 3' to 'Schema 4':; + +BEGIN; + +ALTER TABLE `employee` RENAME TO `fnord`, + DROP FOREIGN KEY bar_fk, + ADD CONSTRAINT foo_fk FOREIGN KEY (employee_id) REFERENCES `foo` (id); + + COMMIT; + ## END OF DIFF }