X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=t%2Fsqlmaker%2Fmysql.t;h=5b3f33039218c1c552afa603fc105828b8b452ff;hb=2cfc22ddff9cb35524031dfc9d429d294b5e3d6e;hp=b5ce8a59ab3a34dda4e378a8a85e03363825702c;hpb=680e2ac9adda36197b8b880c25fa344376f5914e;p=dbsrgits%2FDBIx-Class.git diff --git a/t/sqlmaker/mysql.t b/t/sqlmaker/mysql.t index b5ce8a5..5b3f330 100644 --- a/t/sqlmaker/mysql.t +++ b/t/sqlmaker/mysql.t @@ -7,7 +7,6 @@ use lib qw(t/lib); use DBICTest; use DBICTest::Schema; use DBIC::SqlMakerTest; -use DBIC::DebugObj; my $schema = DBICTest::Schema->connect (DBICTest->_database, { quote_char => '`' }); # cheat @@ -17,54 +16,39 @@ bless ( $schema->storage, 'DBIx::Class::Storage::DBI::mysql' ); # check that double-subqueries are properly wrapped { - my ($sql, @bind); - my $debugobj = DBIC::DebugObj->new (\$sql, \@bind); - my $orig_debugobj = $schema->storage->debugobj; - my $orig_debug = $schema->storage->debug; - - $schema->storage->debugobj ($debugobj); - $schema->storage->debug (1); - # the expected SQL may seem wastefully nonsensical - this is due to # CD's tablename being \'cd', which triggers the "this can be anything" # mode, and forces a subquery. This in turn forces *another* subquery # because mysql is being mysql # Also we know it will fail - never deployed. All we care about is the - # SQL to compare - eval { $schema->resultset ('CD')->update({ genreid => undef }) }; - is_same_sql_bind ( - $sql, - \@bind, + # SQL to compare, hence the eval + $schema->is_executed_sql_bind( sub { + eval { $schema->resultset ('CD')->update({ genreid => undef }) } + },[[ 'UPDATE cd SET `genreid` = ? WHERE `cdid` IN ( SELECT * FROM ( SELECT `me`.`cdid` FROM cd `me` ) `_forced_double_subquery` )', - [ 'NULL' ], - 'Correct update-SQL with double-wrapped subquery', - ); + [ { dbic_colname => "genreid", sqlt_datatype => "integer" } => undef ], + ]], 'Correct update-SQL with double-wrapped subquery' ); # same comment as above - eval { $schema->resultset ('CD')->delete }; - is_same_sql_bind ( - $sql, - \@bind, + $schema->is_executed_sql_bind( sub { + eval { $schema->resultset ('CD')->delete } + }, [[ 'DELETE FROM cd WHERE `cdid` IN ( SELECT * FROM ( SELECT `me`.`cdid` FROM cd `me` ) `_forced_double_subquery` )', - [], - 'Correct delete-SQL with double-wrapped subquery', - ); + ]], 'Correct delete-SQL with double-wrapped subquery' ); # and a couple of really contrived examples (we test them live in t/71mysql.t) my $rs = $schema->resultset('Artist')->search({ name => { -like => 'baby_%' } }); my ($count_sql, @count_bind) = @${$rs->count_rs->as_query}; - eval { - $schema->resultset('Artist')->search( - { artistid => { - -in => $rs->get_column('artistid') - ->as_query - } }, - )->update({ name => \[ "CONCAT( `name`, '_bell_out_of_', $count_sql )", @count_bind ] }); - }; - - is_same_sql_bind ( - $sql, - \@bind, + $schema->is_executed_sql_bind( sub { + eval { + $schema->resultset('Artist')->search( + { artistid => { + -in => $rs->get_column('artistid') + ->as_query + } }, + )->update({ name => \[ "CONCAT( `name`, '_bell_out_of_', $count_sql )", @count_bind ] }); + } + }, [[ q( UPDATE `artist` SET `name` = CONCAT(`name`, '_bell_out_of_', ( @@ -84,18 +68,18 @@ bless ( $schema->storage, 'DBIx::Class::Storage::DBI::mysql' ); WHERE `name` LIKE ? ) `_forced_double_subquery` ) ), - [ ("'baby_%'") x 2 ], - ); - - eval { - $schema->resultset('CD')->search_related('artist', - { 'artist.name' => { -like => 'baby_with_%' } } - )->delete - }; - - is_same_sql_bind ( - $sql, - \@bind, + ( [ { dbic_colname => "name", sqlt_datatype => "varchar", sqlt_size => 100 } + => 'baby_%' ] + ) x 2 + ]]); + + $schema->is_executed_sql_bind( sub { + eval { + $schema->resultset('CD')->search_related('artist', + { 'artist.name' => { -like => 'baby_with_%' } } + )->delete + } + }, [[ q( DELETE FROM `artist` WHERE `artistid` IN ( @@ -109,11 +93,9 @@ bless ( $schema->storage, 'DBIx::Class::Storage::DBI::mysql' ); ) `_forced_double_subquery` ) ), - [ "'baby_with_%'" ], - ); - - $schema->storage->debugobj ($orig_debugobj); - $schema->storage->debug ($orig_debug); + [ { dbic_colname => "artist.name", sqlt_datatype => "varchar", sqlt_size => 100 } + => 'baby_with_%' ], + ]] ); } # Test support for straight joins