Make sure unaliased selectors and prefetch coexist peacefully
[dbsrgits/DBIx-Class.git] / t / prefetch / with_limit.t
CommitLineData
a04af85f 1# Test to ensure we get a consistent result set wether or not we use the
2# prefetch option in combination rows (LIMIT).
3use strict;
4use warnings;
5
6use Test::More;
61fd5dfc 7use Test::Exception;
a04af85f 8use lib qw(t/lib);
9use DBICTest;
53c29913 10use DBIC::SqlMakerTest;
a04af85f 11
a04af85f 12my $schema = DBICTest->init_schema();
51a296b4 13
14
a04af85f 15my $no_prefetch = $schema->resultset('Artist')->search(
51a296b4 16 [ # search deliberately contrived
17 { 'artwork.cd_id' => undef },
18 { 'tracks.title' => { '!=' => 'blah-blah-1234568' }}
19 ],
96faafb4 20 { rows => 3, join => { cds => [qw/artwork tracks/] },
21 }
22);
23
24my $use_prefetch = $no_prefetch->search(
25 {},
25cac750 26 {
c9d29bb2 27 select => ['me.artistid', 'me.name'],
28 as => ['artistid', 'name'],
25cac750 29 prefetch => 'cds',
51a296b4 30 order_by => { -desc => 'name' },
25cac750 31 }
a04af85f 32);
33
36fd7f07 34# add a floating +select to make sure it does nto throw things off
35# we also expect it to appear in both selectors, as we can not know
36# for sure which part of the query it applies to (may be order_by,
37# maybe something else)
38#
39# we use a reference to the same array in bind vals, because
40# is_deeply picks up this difference too (not sure if bug or
41# feature)
42my $bind_one = [ __add => 1 ];
43$use_prefetch = $use_prefetch->search({}, {
44 '+select' => \[ 'me.artistid + ?', $bind_one ],
45});
46
47is_same_sql_bind (
48 $use_prefetch->as_query,
49 '(
50 SELECT me.artistid, me.name,
51 cds.cdid, cds.artist, cds.title, cds.year, cds.genreid, cds.single_track,
52 me.artistid + ?
53 FROM (
54 SELECT me.artistid, me.name,
55 me.artistid + ?
56 FROM artist me
57 LEFT JOIN cd cds
58 ON cds.artist = me.artistid
59 LEFT JOIN cd_artwork artwork
60 ON artwork.cd_id = cds.cdid
61 LEFT JOIN track tracks
62 ON tracks.cd = cds.cdid
63 WHERE artwork.cd_id IS NULL
64 OR tracks.title != ?
65 GROUP BY me.artistid, me.name, me.artistid + ?
66 ORDER BY name DESC LIMIT 3
67 ) me
68 LEFT JOIN cd cds
69 ON cds.artist = me.artistid
70 LEFT JOIN cd_artwork artwork
71 ON artwork.cd_id = cds.cdid
72 LEFT JOIN track tracks
73 ON tracks.cd = cds.cdid
74 WHERE artwork.cd_id IS NULL
75 OR tracks.title != ?
76 GROUP BY me.artistid, me.name, cds.cdid, cds.artist, cds.title, cds.year, cds.genreid, cds.single_track, me.artistid + ?
77 ORDER BY name DESC, cds.artist, cds.year ASC
78 )',
79 [
80 $bind_one, # outer select
81 $bind_one, # inner select
82 [ 'tracks.title' => 'blah-blah-1234568' ], # inner where
83 $bind_one, # inner group_by
84 [ 'tracks.title' => 'blah-blah-1234568' ], # outer where
85 $bind_one, # outer group_by
86 ],
87 'Expected SQL on complex limited prefetch'
88);
89
a04af85f 90is($no_prefetch->count, $use_prefetch->count, '$no_prefetch->count == $use_prefetch->count');
5624ba1f 91is(
9117ccfb 92 scalar ($no_prefetch->all),
93 scalar ($use_prefetch->all),
94 "Amount of returned rows is right"
5624ba1f 95);
96
9117ccfb 97my $artist_many_cds = $schema->resultset('Artist')->search ( {}, {
98 join => 'cds',
99 group_by => 'me.artistid',
100 having => \ 'count(cds.cdid) > 1',
101})->first;
102
103
5624ba1f 104$no_prefetch = $schema->resultset('Artist')->search(
9117ccfb 105 { artistid => $artist_many_cds->id },
5624ba1f 106 { rows => 1 }
107);
108
61fd5dfc 109$use_prefetch = $no_prefetch->search ({}, { prefetch => 'cds' });
5624ba1f 110
61fd5dfc 111my $normal_artist = $no_prefetch->single;
112my $prefetch_artist = $use_prefetch->find({ name => $artist_many_cds->name });
113my $prefetch2_artist = $use_prefetch->first;
5624ba1f 114
115is(
116 $prefetch_artist->cds->count,
117 $normal_artist->cds->count,
61fd5dfc 118 "Count of child rel with prefetch + rows => 1 is right (find)"
119);
120is(
121 $prefetch2_artist->cds->count,
122 $normal_artist->cds->count,
123 "Count of child rel with prefetch + rows => 1 is right (first)"
5624ba1f 124);
61fd5dfc 125
9117ccfb 126is (
127 scalar ($prefetch_artist->cds->all),
128 scalar ($normal_artist->cds->all),
61fd5dfc 129 "Amount of child rel rows with prefetch + rows => 1 is right (find)"
130);
131is (
132 scalar ($prefetch2_artist->cds->all),
133 scalar ($normal_artist->cds->all),
134 "Amount of child rel rows with prefetch + rows => 1 is right (first)"
135);
136
137throws_ok (
138 sub { $use_prefetch->single },
139 qr/resultsets prefetching has_many/,
140 'single() with multiprefetch is illegal',
9117ccfb 141);
01c781fe 142
14e26c5f 143throws_ok (
144 sub {
145 $use_prefetch->search(
146 {'tracks.title' => { '!=' => 'foo' }},
147 { order_by => \ 'some oddball literal sql', join => { cds => 'tracks' } }
148 )->next
149 }, qr/A required group_by clause could not be constructed automatically/,
150);
151
09707a31 152my $artist = $use_prefetch->search({'cds.title' => $artist_many_cds->cds->first->title })->next;
01c781fe 153is($artist->cds->count, 1, "count on search limiting prefetched has_many");
154
a3683eae 155# try with double limit
156my $artist2 = $use_prefetch->search({'cds.title' => { '!=' => $artist_many_cds->cds->first->title } })->slice (0,0)->next;
157is($artist2->cds->count, 2, "count on search limiting prefetched has_many");
158
53c29913 159# make sure 1:1 joins do not force a subquery (no point to exercise the optimizer, if at all available)
160# get cd's that have any tracks and their artists
161my $single_prefetch_rs = $schema->resultset ('CD')->search (
162 { 'me.year' => 2010, 'artist.name' => 'foo' },
163 { prefetch => ['tracks', 'artist'], rows => 15 },
164);
165is_same_sql_bind (
166 $single_prefetch_rs->as_query,
167 '(
168 SELECT
169 me.cdid, me.artist, me.title, me.year, me.genreid, me.single_track,
3d98c75e 170 tracks.trackid, tracks.cd, tracks.position, tracks.title, tracks.last_updated_on, tracks.last_updated_at,
53c29913 171 artist.artistid, artist.name, artist.rank, artist.charfield
172 FROM (
173 SELECT
174 me.cdid, me.artist, me.title, me.year, me.genreid, me.single_track
175 FROM cd me
176 JOIN artist artist ON artist.artistid = me.artist
177 WHERE ( ( artist.name = ? AND me.year = ? ) )
178 LIMIT 15
179 ) me
180 LEFT JOIN track tracks
181 ON tracks.cd = me.cdid
182 JOIN artist artist
183 ON artist.artistid = me.artist
184 WHERE ( ( artist.name = ? AND me.year = ? ) )
185 ORDER BY tracks.cd
186 )',
187 [
188 [ 'artist.name' => 'foo' ],
189 [ 'me.year' => 2010 ],
190 [ 'artist.name' => 'foo' ],
191 [ 'me.year' => 2010 ],
192 ],
193 'No grouping of non-multiplying resultsets',
194);
195
c9d29bb2 196done_testing;