X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=t%2F76joins.t;h=101d71f694e8511ef7d4b93e81ca9d649afcd687;hb=71ea65e702defffaad1a30e8fb6b698c980e90c2;hp=1033b53d5cd5d4fb6f0795cec18af413348af3d8;hpb=cb3e35d2740817d0c5b8b52b77753f03c850708d;p=dbsrgits%2FDBIx-Class.git diff --git a/t/76joins.t b/t/76joins.t index 1033b53..101d71f 100644 --- a/t/76joins.t +++ b/t/76joins.t @@ -16,7 +16,7 @@ BEGIN { eval "use DBD::SQLite"; plan $@ ? ( skip_all => 'needs DBD::SQLite for testing' ) - : ( tests => 49 ); + : ( tests => 64 ); } # figure out if we've got a version of sqlite that is older than 3.2.6, in @@ -89,6 +89,26 @@ $match = 'person mother LEFT JOIN (person child RIGHT JOIN person father ON (' ; is( $sa->_recurse_from(@j4), $match, 'join 4 (nested joins + join types) ok'); +my @j5 = ( + { child => 'person' }, + [ { father => 'person' }, { 'father.person_id' => \'!= child.father_id' }, ], + [ { mother => 'person' }, { 'mother.person_id' => 'child.mother_id' } ], +); +$match = 'person child JOIN person father ON ( father.person_id != ' + . 'child.father_id ) JOIN person mother ON ( mother.person_id ' + . '= child.mother_id )' + ; +is( $sa->_recurse_from(@j5), $match, 'join 5 (SCALAR reference for ON statement) ok' ); + +my @j6 = ( + { child => 'person' }, + [ { father => 'person' }, { 'father.person_id' => { '!=', '42' } }, ], + [ { mother => 'person' }, { 'mother.person_id' => 'child.mother_id' } ], +); +$match = qr/^HASH reference arguments are not supported in JOINS - try using "\.\.\." instead/; +eval { $sa->_recurse_from(@j6) }; +like( $@, $match, 'join 6 (HASH reference for ON statement dies) ok' ); + my $rs = $schema->resultset("CD")->search( { 'year' => 2001, 'artist.name' => 'Caterwauler McCrae' }, { from => [ { 'me' => 'cd' }, @@ -126,6 +146,9 @@ $rs = $schema->resultset("CD")->search( ); cmp_ok( scalar $rs->all, '==', scalar $rs->slice(0, $rs->count - 1), 'slice() with join has same count as all()' ); +ok(!$rs->slice($rs->count+1000, $rs->count+1002)->count, + 'Slicing beyond end of rs returns a zero count'); + $rs = $schema->resultset("Artist")->search( { 'liner_notes.notes' => 'Kill Yourself!' }, { join => { 'cds' => 'liner_notes' } }); @@ -149,7 +172,7 @@ is(Dumper($attr), $attr_str, 'Attribute hash untouched after search()'); cmp_ok($rs + 0, '==', 3, 'Correct number of records returned'); my $queries = 0; -$schema->storage->debugcb(sub { $queries++ }); +$schema->storage->debugcb(sub { $queries++; }); $schema->storage->debug(1); my @cd = $rs->all; @@ -202,12 +225,38 @@ is($queries, 1, 'nested prefetch ran exactly 1 select statement (excluding colum $queries = 0; +is($tag->search_related('cd')->search_related('artist')->first->name, + 'Caterwauler McCrae', + 'chained belongs_to->belongs_to search_related ok'); + +is($queries, 0, 'chained search_related after belontgs_to->belongs_to prefetch ran no queries'); + +$queries = 0; + $cd = $schema->resultset('CD')->find(1, { prefetch => 'artist' }); is($cd->{_inflated_column}{artist}->name, 'Caterwauler McCrae', 'artist prefetched correctly on find'); is($queries, 1, 'find with prefetch ran exactly 1 select statement (excluding column_info)'); +$queries = 0; + +$schema->storage->debugcb(sub { $queries++; }); + +$cd = $schema->resultset('CD')->find(1, { prefetch => { cd_to_producer => 'producer' } }); + +is($cd->producers->first->name, 'Matt S Trout', 'many_to_many accessor ok'); + +is($queries, 1, 'many_to_many accessor with nested prefetch ran exactly 1 query'); + +$queries = 0; + +my $producers = $cd->search_related('cd_to_producer')->search_related('producer'); + +is($producers->first->name, 'Matt S Trout', 'chained many_to_many search_related ok'); + +is($queries, 0, 'chained search_related after many_to_many prefetch ran no queries'); + $schema->storage->debug($orig_debug); $schema->storage->debugobj->callback(undef); @@ -286,6 +335,19 @@ SKIP: { is($rs->next->name, 'Caterwauler McCrae', "Correct artist returned"); +$cd = $schema->resultset('Artist')->first->create_related('cds', + { + title => 'Unproduced Single', + year => 2007 +}); + +my $left_join = $schema->resultset('CD')->search( + { 'me.cdid' => $cd->cdid }, + { prefetch => { cd_to_producer => 'producer' } } +); + +cmp_ok($left_join, '==', 1, 'prefetch with no join record present'); + $queries = 0; $schema->storage->debugcb(sub { $queries++ }); $schema->storage->debug(1); @@ -343,3 +405,67 @@ like( $sql, qr/^SELECT tracks_2\.trackid/, "join not collapsed for search_relate $schema->storage->debug($orig_debug); $schema->storage->debugobj->callback(undef); + +$rs = $schema->resultset('Artist'); +$rs->create({ artistid => 4, name => 'Unknown singer-songwriter' }); +$rs->create({ artistid => 5, name => 'Emo 4ever' }); +@artists = $rs->search(undef, { prefetch => 'cds', order_by => 'artistid' }); +is(scalar @artists, 5, 'has_many prefetch with adjacent empty rows ok'); + +# ------------- +# +# Tests for multilevel has_many prefetch + +# artist resultsets - with and without prefetch +my $art_rs = $schema->resultset('Artist'); +my $art_rs_pr = $art_rs->search( + {}, + { + join => [ { cds => ['tracks'] } ], + prefetch => [ { cds => ['tracks'] } ], + cache => 1 # last test needs this + } +); + +# This test does the same operation twice - once on a +# set of items fetched from the db with no prefetch of has_many rels +# The second prefetches 2 levels of has_many +# We check things are the same by comparing the name or title +# we build everything into a hash structure and compare the one +# from each rs to see what differs + +sub make_hash_struc { + my $rs = shift; + + my $struc = {}; + foreach my $art ( $rs->all ) { + foreach my $cd ( $art->cds ) { + foreach my $track ( $cd->tracks ) { + $struc->{ $art->name }{ $cd->title }{ $track->title }++; + } + } + } + return $struc; +} + +$queries = 0; +$schema->storage->debugcb(sub { $queries++ }); +$schema->storage->debug(1); + +my $prefetch_result = make_hash_struc($art_rs_pr); + +is($queries, 1, 'nested prefetch across has_many->has_many ran exactly 1 query'); + +my $nonpre_result = make_hash_struc($art_rs); + +is_deeply( $prefetch_result, $nonpre_result, + 'Compare 2 level prefetch result to non-prefetch result' ); + +$queries = 0; + +is($art_rs_pr->search_related('cds')->search_related('tracks')->first->title, + 'Fowlin', + 'chained has_many->has_many search_related ok' + ); + +is($queries, 0, 'chained search_related after has_many->has_many prefetch ran no queries');