made search_rs smarter about when to preserve the cache to fix mm prefetch usage
[dbsrgits/DBIx-Class.git] / t / 76joins.t
CommitLineData
70350518 1use strict;
2use warnings;
3
4use Test::More;
5use lib qw(t/lib);
6use DBICTest;
cb3e35d2 7use Data::Dumper;
70350518 8
a47e1233 9my $schema = DBICTest->init_schema();
0567538f 10
4d91ad3f 11my $orig_debug = $schema->storage->debug;
12
0567538f 13use IO::File;
14
15BEGIN {
16 eval "use DBD::SQLite";
17 plan $@
18 ? ( skip_all => 'needs DBD::SQLite for testing' )
2bd9c7c0 19 : ( tests => 64 );
0567538f 20}
21
58ff4acf 22# figure out if we've got a version of sqlite that is older than 3.2.6, in
23# which case COUNT(DISTINCT()) doesn't work
24my $is_broken_sqlite = 0;
25my ($sqlite_major_ver,$sqlite_minor_ver,$sqlite_patch_ver) =
26 split /\./, $schema->storage->dbh->get_info(18);
27if( $schema->storage->dbh->get_info(17) eq 'SQLite' &&
28 ( ($sqlite_major_ver < 3) ||
29 ($sqlite_major_ver == 3 && $sqlite_minor_ver < 2) ||
30 ($sqlite_major_ver == 3 && $sqlite_minor_ver == 2 && $sqlite_patch_ver < 6) ) ) {
31 $is_broken_sqlite = 1;
32}
485f6e10 33
0567538f 34# test the abstract join => SQL generator
35my $sa = new DBIC::SQL::Abstract;
36
37my @j = (
38 { child => 'person' },
39 [ { father => 'person' }, { 'father.person_id' => 'child.father_id' }, ],
40 [ { mother => 'person' }, { 'mother.person_id' => 'child.mother_id' } ],
41);
42my $match = 'person child JOIN person father ON ( father.person_id = '
43 . 'child.father_id ) JOIN person mother ON ( mother.person_id '
44 . '= child.mother_id )'
45 ;
46is( $sa->_recurse_from(@j), $match, 'join 1 ok' );
47
48my @j2 = (
49 { mother => 'person' },
50 [ [ { child => 'person' },
51 [ { father => 'person' },
52 { 'father.person_id' => 'child.father_id' }
53 ]
54 ],
55 { 'mother.person_id' => 'child.mother_id' }
56 ],
57);
58$match = 'person mother JOIN (person child JOIN person father ON ('
59 . ' father.person_id = child.father_id )) ON ( mother.person_id = '
60 . 'child.mother_id )'
61 ;
62is( $sa->_recurse_from(@j2), $match, 'join 2 ok' );
63
64my @j3 = (
65 { child => 'person' },
66 [ { father => 'person', -join_type => 'inner' }, { 'father.person_id' => 'child.father_id' }, ],
67 [ { mother => 'person', -join_type => 'inner' }, { 'mother.person_id' => 'child.mother_id' } ],
68);
69$match = 'person child INNER JOIN person father ON ( father.person_id = '
70 . 'child.father_id ) INNER JOIN person mother ON ( mother.person_id '
71 . '= child.mother_id )'
72 ;
73
74is( $sa->_recurse_from(@j3), $match, 'join 3 (inner join) ok');
75
ca7b9fdf 76my @j4 = (
77 { mother => 'person' },
78 [ [ { child => 'person', -join_type => 'left' },
79 [ { father => 'person', -join_type => 'right' },
80 { 'father.person_id' => 'child.father_id' }
81 ]
82 ],
83 { 'mother.person_id' => 'child.mother_id' }
84 ],
85);
86$match = 'person mother LEFT JOIN (person child RIGHT JOIN person father ON ('
87 . ' father.person_id = child.father_id )) ON ( mother.person_id = '
88 . 'child.mother_id )'
89 ;
90is( $sa->_recurse_from(@j4), $match, 'join 4 (nested joins + join types) ok');
91
635b9634 92my @j5 = (
93 { child => 'person' },
94 [ { father => 'person' }, { 'father.person_id' => \'!= child.father_id' }, ],
95 [ { mother => 'person' }, { 'mother.person_id' => 'child.mother_id' } ],
96);
97$match = 'person child JOIN person father ON ( father.person_id != '
98 . 'child.father_id ) JOIN person mother ON ( mother.person_id '
99 . '= child.mother_id )'
100 ;
101is( $sa->_recurse_from(@j5), $match, 'join 5 (SCALAR reference for ON statement) ok' );
102
103my @j6 = (
104 { child => 'person' },
105 [ { father => 'person' }, { 'father.person_id' => { '!=', '42' } }, ],
106 [ { mother => 'person' }, { 'mother.person_id' => 'child.mother_id' } ],
107);
11aada86 108$match = qr/^HASH reference arguments are not supported in JOINS - try using "\.\.\." instead/;
635b9634 109eval { $sa->_recurse_from(@j6) };
110like( $@, $match, 'join 6 (HASH reference for ON statement dies) ok' );
111
f9db5527 112my $rs = $schema->resultset("CD")->search(
0567538f 113 { 'year' => 2001, 'artist.name' => 'Caterwauler McCrae' },
114 { from => [ { 'me' => 'cd' },
115 [
116 { artist => 'artist' },
117 { 'me.artist' => 'artist.artistid' }
118 ] ] }
119 );
120
ebaefbc2 121cmp_ok( $rs + 0, '==', 1, "Single record in resultset");
0567538f 122
123is($rs->first->title, 'Forkful of bees', 'Correct record returned');
124
f9db5527 125$rs = $schema->resultset("CD")->search(
0567538f 126 { 'year' => 2001, 'artist.name' => 'Caterwauler McCrae' },
127 { join => 'artist' });
128
ebaefbc2 129cmp_ok( $rs + 0, '==', 1, "Single record in resultset");
0567538f 130
131is($rs->first->title, 'Forkful of bees', 'Correct record returned');
132
f9db5527 133$rs = $schema->resultset("CD")->search(
0567538f 134 { 'artist.name' => 'We Are Goth',
135 'liner_notes.notes' => 'Kill Yourself!' },
136 { join => [ qw/artist liner_notes/ ] });
137
ebaefbc2 138cmp_ok( $rs + 0, '==', 1, "Single record in resultset");
0567538f 139
140is($rs->first->title, 'Come Be Depressed With Us', 'Correct record returned');
141
8fe164b9 142# when using join attribute, make sure slice()ing all objects has same count as all()
f9db5527 143$rs = $schema->resultset("CD")->search(
8fe164b9 144 { 'artist' => 1 },
145 { join => [qw/artist/], order_by => 'artist.name' }
146);
147cmp_ok( scalar $rs->all, '==', scalar $rs->slice(0, $rs->count - 1), 'slice() with join has same count as all()' );
148
2bd9c7c0 149ok(!$rs->slice($rs->count+1000, $rs->count+1002)->count,
150 'Slicing beyond end of rs returns a zero count');
151
f9db5527 152$rs = $schema->resultset("Artist")->search(
0567538f 153 { 'liner_notes.notes' => 'Kill Yourself!' },
154 { join => { 'cds' => 'liner_notes' } });
155
156cmp_ok( $rs->count, '==', 1, "Single record in resultset");
157
158is($rs->first->name, 'We Are Goth', 'Correct record returned');
159
cb3e35d2 160# bug in 0.07000 caused attr (join/prefetch) to be modifed by search
161# so we check the search & attr arrays are not modified
162my $search = { 'artist.name' => 'Caterwauler McCrae' };
163my $attr = { prefetch => [ qw/artist liner_notes/ ],
164 order_by => 'me.cdid' };
165my $search_str = Dumper($search);
166my $attr_str = Dumper($attr);
167
168$rs = $schema->resultset("CD")->search($search, $attr);
0567538f 169
cb3e35d2 170is(Dumper($search), $search_str, 'Search hash untouched after search()');
171is(Dumper($attr), $attr_str, 'Attribute hash untouched after search()');
ebaefbc2 172cmp_ok($rs + 0, '==', 3, 'Correct number of records returned');
0567538f 173
d52170d4 174my $queries = 0;
9809a6df 175$schema->storage->debugcb(sub { $queries++; });
d52170d4 176$schema->storage->debug(1);
0567538f 177
178my @cd = $rs->all;
179
180is($cd[0]->title, 'Spoonful of bees', 'First record returned ok');
181
7cd300ea 182ok(!defined $cd[0]->liner_notes, 'No prefetch for NULL LEFT join');
0567538f 183
184is($cd[1]->{_relationship_data}{liner_notes}->notes, 'Buy Whiskey!', 'Prefetch for present LEFT JOIN');
185
dd417d06 186is(ref $cd[1]->liner_notes, 'DBICTest::LinerNotes', 'Prefetch returns correct class');
187
0567538f 188is($cd[2]->{_inflated_column}{artist}->name, 'Caterwauler McCrae', 'Prefetch on parent object ok');
189
d52170d4 190is($queries, 1, 'prefetch ran only 1 select statement');
191
4d91ad3f 192$schema->storage->debug($orig_debug);
193$schema->storage->debugobj->callback(undef);
0567538f 194
9b465d00 195# test for partial prefetch via columns attr
d20bb28e 196my $cd = $schema->resultset('CD')->find(1,
197 {
9b465d00 198 columns => [qw/title artist.name/],
0e35aa05 199 join => { 'artist' => {} }
d20bb28e 200 }
201);
202ok(eval { $cd->artist->name eq 'Caterwauler McCrae' }, 'single related column prefetched');
203
b3e8ac9b 204# start test for nested prefetch SELECT count
d52170d4 205$queries = 0;
db0e65d2 206$schema->storage->debugcb(sub { $queries++ });
d52170d4 207$schema->storage->debug(1);
b3e8ac9b 208
209$rs = $schema->resultset('Tag')->search(
210 {},
211 {
212 prefetch => { cd => 'artist' }
213 }
214);
215
216my $tag = $rs->first;
217
218is( $tag->cd->title, 'Spoonful of bees', 'step 1 ok for nested prefetch' );
219
220is( $tag->cd->artist->name, 'Caterwauler McCrae', 'step 2 ok for nested prefetch');
221
222# count the SELECTs
d52170d4 223#$selects++ if /SELECT(?!.*WHERE 1=0.*)/;
224is($queries, 1, 'nested prefetch ran exactly 1 select statement (excluding column_info)');
b3e8ac9b 225
d52170d4 226$queries = 0;
c5b7d799 227
d54c409f 228is($tag->search_related('cd')->search_related('artist')->first->name,
229 'Caterwauler McCrae',
230 'chained belongs_to->belongs_to search_related ok');
231
9809a6df 232is($queries, 0, 'chained search_related after belontgs_to->belongs_to prefetch ran no queries');
d54c409f 233
234$queries = 0;
235
d20bb28e 236$cd = $schema->resultset('CD')->find(1, { prefetch => 'artist' });
c5b7d799 237
238is($cd->{_inflated_column}{artist}->name, 'Caterwauler McCrae', 'artist prefetched correctly on find');
239
d52170d4 240is($queries, 1, 'find with prefetch ran exactly 1 select statement (excluding column_info)');
241
d54c409f 242$queries = 0;
243
95303b0a 244$schema->storage->debugcb(sub { $queries++; });
9809a6df 245
d54c409f 246$cd = $schema->resultset('CD')->find(1, { prefetch => { cd_to_producer => 'producer' } });
247
248is($cd->producers->first->name, 'Matt S Trout', 'many_to_many accessor ok');
249
0ed8d3b6 250is($queries, 1, 'many_to_many accessor with nested prefetch ran exactly 1 query');
d54c409f 251
252$queries = 0;
253
254my $producers = $cd->search_related('cd_to_producer')->search_related('producer');
255
256is($producers->first->name, 'Matt S Trout', 'chained many_to_many search_related ok');
257
9809a6df 258is($queries, 0, 'chained search_related after many_to_many prefetch ran no queries');
d54c409f 259
4d91ad3f 260$schema->storage->debug($orig_debug);
261$schema->storage->debugobj->callback(undef);
c5b7d799 262
b3e8ac9b 263$rs = $schema->resultset('Tag')->search(
264 {},
265 {
266 join => { cd => 'artist' },
267 prefetch => { cd => 'artist' }
268 }
269);
270
271cmp_ok( $rs->count, '>=', 0, 'nested prefetch does not duplicate joins' );
272
f9db5527 273my ($artist) = $schema->resultset("Artist")->search({ 'cds.year' => 2001 },
0567538f 274 { order_by => 'artistid DESC', join => 'cds' });
275
276is($artist->name, 'Random Boy Band', "Join search by object ok");
277
f9db5527 278my @cds = $schema->resultset("CD")->search({ 'liner_notes.notes' => 'Buy Merch!' },
0567538f 279 { join => 'liner_notes' });
280
281cmp_ok(scalar @cds, '==', 1, "Single CD retrieved via might_have");
282
283is($cds[0]->title, "Generic Manufactured Singles", "Correct CD retrieved");
284
f9db5527 285my @artists = $schema->resultset("Artist")->search({ 'tags.tag' => 'Shiny' },
0567538f 286 { join => { 'cds' => 'tags' } });
287
288cmp_ok( @artists, '==', 2, "two-join search ok" );
289
15c382be 290$rs = $schema->resultset("CD")->search(
291 {},
292 { group_by => [qw/ title me.cdid /] }
293);
294
485f6e10 295SKIP: {
296 skip "SQLite < 3.2.6 doesn't understand COUNT(DISTINCT())", 1
297 if $is_broken_sqlite;
298 cmp_ok( $rs->count, '==', 5, "count() ok after group_by on main pk" );
299}
15c382be 300
301cmp_ok( scalar $rs->all, '==', 5, "all() returns same count as count() after group_by on main pk" );
302
303$rs = $schema->resultset("CD")->search(
304 {},
305 { join => [qw/ artist /], group_by => [qw/ artist.name /] }
306);
307
485f6e10 308SKIP: {
309 skip "SQLite < 3.2.6 doesn't understand COUNT(DISTINCT())", 1
310 if $is_broken_sqlite;
311 cmp_ok( $rs->count, '==', 3, "count() ok after group_by on related column" );
312}
15c382be 313
8839560b 314$rs = $schema->resultset("Artist")->search(
6991aba3 315 {},
01c73d7b 316 { join => [qw/ cds /], group_by => [qw/ me.name /], having =>{ 'MAX(cds.cdid)'=> \'< 5' } }
6991aba3 317);
318
8839560b 319cmp_ok( $rs->all, '==', 2, "results ok after group_by on related column with a having" );
6991aba3 320
01c73d7b 321$rs = $rs->search( undef, { having =>{ 'count(*)'=> \'> 2' }});
6991aba3 322
8839560b 323cmp_ok( $rs->all, '==', 1, "count() ok after group_by on related column with a having" );
489709af 324
325$rs = $schema->resultset("Artist")->search(
326 { 'cds.title' => 'Spoonful of bees',
327 'cds_2.title' => 'Forkful of bees' },
328 { join => [ 'cds', 'cds' ] });
329
485f6e10 330SKIP: {
331 skip "SQLite < 3.2.6 doesn't understand COUNT(DISTINCT())", 1
332 if $is_broken_sqlite;
333 cmp_ok($rs->count, '==', 1, "single artist returned from multi-join");
334}
335
489709af 336is($rs->next->name, 'Caterwauler McCrae', "Correct artist returned");
337
d96034a1 338$cd = $schema->resultset('Artist')->first->create_related('cds',
24010dd8 339 {
340 title => 'Unproduced Single',
341 year => 2007
342});
343
344my $left_join = $schema->resultset('CD')->search(
345 { 'me.cdid' => $cd->cdid },
346 { prefetch => { cd_to_producer => 'producer' } }
347);
348
349cmp_ok($left_join, '==', 1, 'prefetch with no join record present');
8206c013 350
d52170d4 351$queries = 0;
db0e65d2 352$schema->storage->debugcb(sub { $queries++ });
887ce227 353$schema->storage->debug(1);
354
355my $tree_like =
356 $schema->resultset('TreeLike')->find(4,
357 { join => { parent => { parent => 'parent' } },
358 prefetch => { parent => { parent => 'parent' } } });
359
360is($tree_like->name, 'quux', 'Bottom of tree ok');
361$tree_like = $tree_like->parent;
362is($tree_like->name, 'baz', 'First level up ok');
363$tree_like = $tree_like->parent;
364is($tree_like->name, 'bar', 'Second level up ok');
365$tree_like = $tree_like->parent;
366is($tree_like->name, 'foo', 'Third level up ok');
367
4d91ad3f 368$schema->storage->debug($orig_debug);
369$schema->storage->debugobj->callback(undef);
887ce227 370
371cmp_ok($queries, '==', 1, 'Only one query run');
372
10ed6c25 373$tree_like = $schema->resultset('TreeLike')->search({'me.id' => 1});
af4cd3e5 374$tree_like = $tree_like->search_related('children')->search_related('children')->search_related('children')->first;
7e5b5f6e 375is($tree_like->name, 'quux', 'Tree search_related ok');
376
48d2be64 377$tree_like = $schema->resultset('TreeLike')->search_related('children',
378 { 'children.id' => 2, 'children_2.id' => 3 },
379 { prefetch => { children => 'children' } }
380 )->first;
b377c7e1 381is(eval { $tree_like->children->first->children->first->name }, 'quux',
382 'Tree search_related with prefetch ok');
10ed6c25 383
b377c7e1 384$tree_like = eval { $schema->resultset('TreeLike')->search(
10ed6c25 385 { 'children.id' => 2, 'children_2.id' => 5 },
3583d755 386 { join => [qw/children children/] }
9755b3df 387 )->search_related('children', { 'children_4.id' => 6 }, { prefetch => 'children' }
b377c7e1 388 )->first->children->first; };
389is(eval { $tree_like->name }, 'fong', 'Tree with multiple has_many joins ok');
10ed6c25 390
7e5b5f6e 391# test that collapsed joins don't get a _2 appended to the alias
392
393my $sql = '';
394$schema->storage->debugcb(sub { $sql = $_[1] });
395$schema->storage->debug(1);
396
397eval {
398 my $row = $schema->resultset('Artist')->search_related('cds', undef, {
399 join => 'tracks',
400 prefetch => 'tracks',
401 })->search_related('tracks')->first;
402};
403
9755b3df 404like( $sql, qr/^SELECT tracks_2\.trackid/, "join not collapsed for search_related" );
7e5b5f6e 405
4d91ad3f 406$schema->storage->debug($orig_debug);
407$schema->storage->debugobj->callback(undef);
907d3b9b 408
a08c907f 409$rs = $schema->resultset('Artist');
410$rs->create({ artistid => 4, name => 'Unknown singer-songwriter' });
411$rs->create({ artistid => 5, name => 'Emo 4ever' });
412@artists = $rs->search(undef, { prefetch => 'cds', order_by => 'artistid' });
413is(scalar @artists, 5, 'has_many prefetch with adjacent empty rows ok');
414
907d3b9b 415# -------------
416#
417# Tests for multilevel has_many prefetch
418
419# artist resultsets - with and without prefetch
420my $art_rs = $schema->resultset('Artist');
421my $art_rs_pr = $art_rs->search(
422 {},
423 {
424 join => [ { cds => ['tracks'] } ],
9809a6df 425 prefetch => [ { cds => ['tracks'] } ],
426 cache => 1 # last test needs this
907d3b9b 427 }
428);
429
430# This test does the same operation twice - once on a
431# set of items fetched from the db with no prefetch of has_many rels
432# The second prefetches 2 levels of has_many
433# We check things are the same by comparing the name or title
434# we build everything into a hash structure and compare the one
435# from each rs to see what differs
436
437sub make_hash_struc {
438 my $rs = shift;
439
440 my $struc = {};
441 foreach my $art ( $rs->all ) {
442 foreach my $cd ( $art->cds ) {
443 foreach my $track ( $cd->tracks ) {
444 $struc->{ $art->name }{ $cd->title }{ $track->title }++;
445 }
446 }
447 }
448 return $struc;
449}
450
d54c409f 451$queries = 0;
452$schema->storage->debugcb(sub { $queries++ });
453$schema->storage->debug(1);
454
907d3b9b 455my $prefetch_result = make_hash_struc($art_rs_pr);
d54c409f 456
457is($queries, 1, 'nested prefetch across has_many->has_many ran exactly 1 query');
458
907d3b9b 459my $nonpre_result = make_hash_struc($art_rs);
460
461is_deeply( $prefetch_result, $nonpre_result,
462 'Compare 2 level prefetch result to non-prefetch result' );
d54c409f 463
464$queries = 0;
465
466is($art_rs_pr->search_related('cds')->search_related('tracks')->first->title,
467 'Fowlin',
468 'chained has_many->has_many search_related ok'
469 );
470
9809a6df 471is($queries, 0, 'chained search_related after has_many->has_many prefetch ran no queries');