LEFT join problem fixed
[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' )
8206c013 19 : ( tests => 63 );
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
f9db5527 149$rs = $schema->resultset("Artist")->search(
0567538f 150 { 'liner_notes.notes' => 'Kill Yourself!' },
151 { join => { 'cds' => 'liner_notes' } });
152
153cmp_ok( $rs->count, '==', 1, "Single record in resultset");
154
155is($rs->first->name, 'We Are Goth', 'Correct record returned');
156
cb3e35d2 157# bug in 0.07000 caused attr (join/prefetch) to be modifed by search
158# so we check the search & attr arrays are not modified
159my $search = { 'artist.name' => 'Caterwauler McCrae' };
160my $attr = { prefetch => [ qw/artist liner_notes/ ],
161 order_by => 'me.cdid' };
162my $search_str = Dumper($search);
163my $attr_str = Dumper($attr);
164
165$rs = $schema->resultset("CD")->search($search, $attr);
0567538f 166
cb3e35d2 167is(Dumper($search), $search_str, 'Search hash untouched after search()');
168is(Dumper($attr), $attr_str, 'Attribute hash untouched after search()');
ebaefbc2 169cmp_ok($rs + 0, '==', 3, 'Correct number of records returned');
0567538f 170
d52170d4 171my $queries = 0;
9809a6df 172$schema->storage->debugcb(sub { $queries++; });
d52170d4 173$schema->storage->debug(1);
0567538f 174
175my @cd = $rs->all;
176
177is($cd[0]->title, 'Spoonful of bees', 'First record returned ok');
178
7cd300ea 179ok(!defined $cd[0]->liner_notes, 'No prefetch for NULL LEFT join');
0567538f 180
181is($cd[1]->{_relationship_data}{liner_notes}->notes, 'Buy Whiskey!', 'Prefetch for present LEFT JOIN');
182
dd417d06 183is(ref $cd[1]->liner_notes, 'DBICTest::LinerNotes', 'Prefetch returns correct class');
184
0567538f 185is($cd[2]->{_inflated_column}{artist}->name, 'Caterwauler McCrae', 'Prefetch on parent object ok');
186
d52170d4 187is($queries, 1, 'prefetch ran only 1 select statement');
188
4d91ad3f 189$schema->storage->debug($orig_debug);
190$schema->storage->debugobj->callback(undef);
0567538f 191
9b465d00 192# test for partial prefetch via columns attr
d20bb28e 193my $cd = $schema->resultset('CD')->find(1,
194 {
9b465d00 195 columns => [qw/title artist.name/],
0e35aa05 196 join => { 'artist' => {} }
d20bb28e 197 }
198);
199ok(eval { $cd->artist->name eq 'Caterwauler McCrae' }, 'single related column prefetched');
200
b3e8ac9b 201# start test for nested prefetch SELECT count
d52170d4 202$queries = 0;
db0e65d2 203$schema->storage->debugcb(sub { $queries++ });
d52170d4 204$schema->storage->debug(1);
b3e8ac9b 205
206$rs = $schema->resultset('Tag')->search(
207 {},
208 {
209 prefetch => { cd => 'artist' }
210 }
211);
212
213my $tag = $rs->first;
214
215is( $tag->cd->title, 'Spoonful of bees', 'step 1 ok for nested prefetch' );
216
217is( $tag->cd->artist->name, 'Caterwauler McCrae', 'step 2 ok for nested prefetch');
218
219# count the SELECTs
d52170d4 220#$selects++ if /SELECT(?!.*WHERE 1=0.*)/;
221is($queries, 1, 'nested prefetch ran exactly 1 select statement (excluding column_info)');
b3e8ac9b 222
d52170d4 223$queries = 0;
c5b7d799 224
d54c409f 225is($tag->search_related('cd')->search_related('artist')->first->name,
226 'Caterwauler McCrae',
227 'chained belongs_to->belongs_to search_related ok');
228
9809a6df 229is($queries, 0, 'chained search_related after belontgs_to->belongs_to prefetch ran no queries');
d54c409f 230
231$queries = 0;
232
d20bb28e 233$cd = $schema->resultset('CD')->find(1, { prefetch => 'artist' });
c5b7d799 234
235is($cd->{_inflated_column}{artist}->name, 'Caterwauler McCrae', 'artist prefetched correctly on find');
236
d52170d4 237is($queries, 1, 'find with prefetch ran exactly 1 select statement (excluding column_info)');
238
d54c409f 239$queries = 0;
240
95303b0a 241$schema->storage->debugcb(sub { $queries++; });
9809a6df 242
d54c409f 243$cd = $schema->resultset('CD')->find(1, { prefetch => { cd_to_producer => 'producer' } });
244
245is($cd->producers->first->name, 'Matt S Trout', 'many_to_many accessor ok');
246
247TODO: {
248 local $TODO = 'use prefetched values for many_to_many accessor';
249
250 is($queries, 1, 'many_to_many accessor with nested prefetch ran exactly 1 query');
251}
252
253$queries = 0;
254
255my $producers = $cd->search_related('cd_to_producer')->search_related('producer');
256
257is($producers->first->name, 'Matt S Trout', 'chained many_to_many search_related ok');
258
9809a6df 259is($queries, 0, 'chained search_related after many_to_many prefetch ran no queries');
d54c409f 260
4d91ad3f 261$schema->storage->debug($orig_debug);
262$schema->storage->debugobj->callback(undef);
c5b7d799 263
b3e8ac9b 264$rs = $schema->resultset('Tag')->search(
265 {},
266 {
267 join => { cd => 'artist' },
268 prefetch => { cd => 'artist' }
269 }
270);
271
272cmp_ok( $rs->count, '>=', 0, 'nested prefetch does not duplicate joins' );
273
f9db5527 274my ($artist) = $schema->resultset("Artist")->search({ 'cds.year' => 2001 },
0567538f 275 { order_by => 'artistid DESC', join => 'cds' });
276
277is($artist->name, 'Random Boy Band', "Join search by object ok");
278
f9db5527 279my @cds = $schema->resultset("CD")->search({ 'liner_notes.notes' => 'Buy Merch!' },
0567538f 280 { join => 'liner_notes' });
281
282cmp_ok(scalar @cds, '==', 1, "Single CD retrieved via might_have");
283
284is($cds[0]->title, "Generic Manufactured Singles", "Correct CD retrieved");
285
f9db5527 286my @artists = $schema->resultset("Artist")->search({ 'tags.tag' => 'Shiny' },
0567538f 287 { join => { 'cds' => 'tags' } });
288
289cmp_ok( @artists, '==', 2, "two-join search ok" );
290
15c382be 291$rs = $schema->resultset("CD")->search(
292 {},
293 { group_by => [qw/ title me.cdid /] }
294);
295
485f6e10 296SKIP: {
297 skip "SQLite < 3.2.6 doesn't understand COUNT(DISTINCT())", 1
298 if $is_broken_sqlite;
299 cmp_ok( $rs->count, '==', 5, "count() ok after group_by on main pk" );
300}
15c382be 301
302cmp_ok( scalar $rs->all, '==', 5, "all() returns same count as count() after group_by on main pk" );
303
304$rs = $schema->resultset("CD")->search(
305 {},
306 { join => [qw/ artist /], group_by => [qw/ artist.name /] }
307);
308
485f6e10 309SKIP: {
310 skip "SQLite < 3.2.6 doesn't understand COUNT(DISTINCT())", 1
311 if $is_broken_sqlite;
312 cmp_ok( $rs->count, '==', 3, "count() ok after group_by on related column" );
313}
15c382be 314
8839560b 315$rs = $schema->resultset("Artist")->search(
6991aba3 316 {},
01c73d7b 317 { join => [qw/ cds /], group_by => [qw/ me.name /], having =>{ 'MAX(cds.cdid)'=> \'< 5' } }
6991aba3 318);
319
8839560b 320cmp_ok( $rs->all, '==', 2, "results ok after group_by on related column with a having" );
6991aba3 321
01c73d7b 322$rs = $rs->search( undef, { having =>{ 'count(*)'=> \'> 2' }});
6991aba3 323
8839560b 324cmp_ok( $rs->all, '==', 1, "count() ok after group_by on related column with a having" );
489709af 325
326$rs = $schema->resultset("Artist")->search(
327 { 'cds.title' => 'Spoonful of bees',
328 'cds_2.title' => 'Forkful of bees' },
329 { join => [ 'cds', 'cds' ] });
330
485f6e10 331SKIP: {
332 skip "SQLite < 3.2.6 doesn't understand COUNT(DISTINCT())", 1
333 if $is_broken_sqlite;
334 cmp_ok($rs->count, '==', 1, "single artist returned from multi-join");
335}
336
489709af 337is($rs->next->name, 'Caterwauler McCrae', "Correct artist returned");
338
24010dd8 339my $cd = $schema->resultset('Artist')->first->create_related('cds',
340 {
341 title => 'Unproduced Single',
342 year => 2007
343});
344
345my $left_join = $schema->resultset('CD')->search(
346 { 'me.cdid' => $cd->cdid },
347 { prefetch => { cd_to_producer => 'producer' } }
348);
349
350cmp_ok($left_join, '==', 1, 'prefetch with no join record present');
8206c013 351
d52170d4 352$queries = 0;
db0e65d2 353$schema->storage->debugcb(sub { $queries++ });
887ce227 354$schema->storage->debug(1);
355
356my $tree_like =
357 $schema->resultset('TreeLike')->find(4,
358 { join => { parent => { parent => 'parent' } },
359 prefetch => { parent => { parent => 'parent' } } });
360
361is($tree_like->name, 'quux', 'Bottom of tree ok');
362$tree_like = $tree_like->parent;
363is($tree_like->name, 'baz', 'First level up ok');
364$tree_like = $tree_like->parent;
365is($tree_like->name, 'bar', 'Second level up ok');
366$tree_like = $tree_like->parent;
367is($tree_like->name, 'foo', 'Third level up ok');
368
4d91ad3f 369$schema->storage->debug($orig_debug);
370$schema->storage->debugobj->callback(undef);
887ce227 371
372cmp_ok($queries, '==', 1, 'Only one query run');
373
10ed6c25 374$tree_like = $schema->resultset('TreeLike')->search({'me.id' => 1});
af4cd3e5 375$tree_like = $tree_like->search_related('children')->search_related('children')->search_related('children')->first;
7e5b5f6e 376is($tree_like->name, 'quux', 'Tree search_related ok');
377
48d2be64 378$tree_like = $schema->resultset('TreeLike')->search_related('children',
379 { 'children.id' => 2, 'children_2.id' => 3 },
380 { prefetch => { children => 'children' } }
381 )->first;
b377c7e1 382is(eval { $tree_like->children->first->children->first->name }, 'quux',
383 'Tree search_related with prefetch ok');
10ed6c25 384
b377c7e1 385$tree_like = eval { $schema->resultset('TreeLike')->search(
10ed6c25 386 { 'children.id' => 2, 'children_2.id' => 5 },
3583d755 387 { join => [qw/children children/] }
9755b3df 388 )->search_related('children', { 'children_4.id' => 6 }, { prefetch => 'children' }
b377c7e1 389 )->first->children->first; };
390is(eval { $tree_like->name }, 'fong', 'Tree with multiple has_many joins ok');
10ed6c25 391
7e5b5f6e 392# test that collapsed joins don't get a _2 appended to the alias
393
394my $sql = '';
395$schema->storage->debugcb(sub { $sql = $_[1] });
396$schema->storage->debug(1);
397
398eval {
399 my $row = $schema->resultset('Artist')->search_related('cds', undef, {
400 join => 'tracks',
401 prefetch => 'tracks',
402 })->search_related('tracks')->first;
403};
404
9755b3df 405like( $sql, qr/^SELECT tracks_2\.trackid/, "join not collapsed for search_related" );
7e5b5f6e 406
4d91ad3f 407$schema->storage->debug($orig_debug);
408$schema->storage->debugobj->callback(undef);
907d3b9b 409
a08c907f 410$rs = $schema->resultset('Artist');
411$rs->create({ artistid => 4, name => 'Unknown singer-songwriter' });
412$rs->create({ artistid => 5, name => 'Emo 4ever' });
413@artists = $rs->search(undef, { prefetch => 'cds', order_by => 'artistid' });
414is(scalar @artists, 5, 'has_many prefetch with adjacent empty rows ok');
415
907d3b9b 416# -------------
417#
418# Tests for multilevel has_many prefetch
419
420# artist resultsets - with and without prefetch
421my $art_rs = $schema->resultset('Artist');
422my $art_rs_pr = $art_rs->search(
423 {},
424 {
425 join => [ { cds => ['tracks'] } ],
9809a6df 426 prefetch => [ { cds => ['tracks'] } ],
427 cache => 1 # last test needs this
907d3b9b 428 }
429);
430
431# This test does the same operation twice - once on a
432# set of items fetched from the db with no prefetch of has_many rels
433# The second prefetches 2 levels of has_many
434# We check things are the same by comparing the name or title
435# we build everything into a hash structure and compare the one
436# from each rs to see what differs
437
438sub make_hash_struc {
439 my $rs = shift;
440
441 my $struc = {};
442 foreach my $art ( $rs->all ) {
443 foreach my $cd ( $art->cds ) {
444 foreach my $track ( $cd->tracks ) {
445 $struc->{ $art->name }{ $cd->title }{ $track->title }++;
446 }
447 }
448 }
449 return $struc;
450}
451
d54c409f 452$queries = 0;
453$schema->storage->debugcb(sub { $queries++ });
454$schema->storage->debug(1);
455
907d3b9b 456my $prefetch_result = make_hash_struc($art_rs_pr);
d54c409f 457
458is($queries, 1, 'nested prefetch across has_many->has_many ran exactly 1 query');
459
907d3b9b 460my $nonpre_result = make_hash_struc($art_rs);
461
462is_deeply( $prefetch_result, $nonpre_result,
463 'Compare 2 level prefetch result to non-prefetch result' );
d54c409f 464
465$queries = 0;
466
467is($art_rs_pr->search_related('cds')->search_related('tracks')->first->title,
468 'Fowlin',
469 'chained has_many->has_many search_related ok'
470 );
471
9809a6df 472is($queries, 0, 'chained search_related after has_many->has_many prefetch ran no queries');