Ensure the custom rel cond resolver does not trigger forgotten compat shim
[dbsrgits/DBIx-Class.git] / t / relationship / custom.t
1 use strict;
2 use warnings;
3
4 use Test::More;
5 use Test::Exception;
6 use Test::Warn;
7 use lib qw(t/lib);
8 use DBICTest ':DiffSQL';
9
10 my $schema = DBICTest->init_schema();
11
12 $schema->resultset('Artist')->delete;
13 $schema->resultset('CD')->delete;
14
15 my $artist  = $schema->resultset("Artist")->create({ artistid => 21, name => 'Michael Jackson', rank => 20 });
16 my $artist2 = $schema->resultset("Artist")->create({ artistid => 22, name => 'Chico Buarque', rank => 1 }) ;
17 my $artist3 = $schema->resultset("Artist")->create({ artistid => 23, name => 'Ziraldo', rank => 1 });
18 my $artist4 = $schema->resultset("Artist")->create({ artistid => 24, name => 'Paulo Caruso', rank => 20 });
19
20 my @artworks;
21
22 foreach my $year (1975..1985) {
23   my $cd = $artist->create_related('cds', { year => $year, title => 'Compilation from ' . $year });
24   push @artworks, $cd->create_related('artwork', {});
25 }
26
27 foreach my $year (1975..1995) {
28   my $cd = $artist2->create_related('cds', { year => $year, title => 'Compilation from ' . $year });
29   push @artworks, $cd->create_related('artwork', {});
30 }
31
32 foreach my $artwork (@artworks) {
33   $artwork->create_related('artwork_to_artist', { artist => $_ }) for ($artist3, $artist4);
34 }
35
36
37 my $cds_80s_rs = $artist->cds_80s;
38 is_same_sql_bind(
39   $cds_80s_rs->as_query,
40   '(
41     SELECT me.cdid, me.artist, me.title, me.year, me.genreid, me.single_track
42       FROM cd me
43     WHERE ( ( me.artist = ? AND ( me.year < ? AND me.year > ? ) ) )
44   )',
45   [
46     [
47       {}
48         => 21
49     ],
50     [
51       { sqlt_datatype => 'varchar', sqlt_size => 100, dbic_colname => 'me.year' }
52         => 1990
53     ],
54     [
55       { sqlt_datatype => 'varchar', sqlt_size => 100, dbic_colname => 'me.year' }
56         => 1979
57     ],
58   ],
59 );
60 my @cds_80s = $cds_80s_rs->all;
61 is(@cds_80s, 6, '6 80s cds found (1980 - 1985)');
62 map { ok($_->year < 1990 && $_->year > 1979) } @cds_80s;
63
64
65 my $cds_90s_rs = $artist2->cds_90s;
66 is_same_sql_bind(
67   $cds_90s_rs->as_query,
68   '(
69     SELECT me.cdid, me.artist, me.title, me.year, me.genreid, me.single_track
70       FROM artist artist__row
71       JOIN cd me
72         ON ( me.artist = artist__row.artistid AND ( me.year < ? AND me.year > ? ) )
73       WHERE ( artist__row.artistid = ? )
74   )',
75   [
76     [
77       { sqlt_datatype => 'varchar', sqlt_size => 100, dbic_colname => 'me.year' }
78         => 2000
79     ],
80     [
81       { sqlt_datatype => 'varchar', sqlt_size => 100, dbic_colname => 'me.year' }
82         => 1989
83     ],
84     [ { sqlt_datatype => 'integer', dbic_colname => 'artist__row.artistid' }
85         => 22
86     ],
87   ]
88 );
89
90 # re-test with ::-containing moniker name
91 # (we don't have any currently, so fudge it with lots of local() )
92 {
93   local $schema->source('Artist')->{source_name} = 'Ar::Tist';
94   local $artist2->{related_resultsets};
95
96   is_same_sql_bind(
97     $artist2->cds_90s->as_query,
98     '(
99       SELECT me.cdid, me.artist, me.title, me.year, me.genreid, me.single_track
100         FROM artist ar_tist__row
101         JOIN cd me
102           ON ( me.artist = ar_tist__row.artistid AND ( me.year < ? AND me.year > ? ) )
103         WHERE ( ar_tist__row.artistid = ? )
104     )',
105     [
106       [
107         { sqlt_datatype => 'varchar', sqlt_size => 100, dbic_colname => 'me.year' }
108           => 2000
109       ],
110       [
111       { sqlt_datatype => 'varchar', sqlt_size => 100, dbic_colname => 'me.year' }
112           => 1989
113       ],
114       [ { sqlt_datatype => 'integer', dbic_colname => 'ar_tist__row.artistid' }
115           => 22
116       ],
117     ]
118   );
119 }
120
121
122 my @cds_90s = $cds_90s_rs->all;
123 is(@cds_90s, 6, '6 90s cds found (1990 - 1995) even with non-optimized search');
124 map { ok($_->year < 2000 && $_->year > 1989) } @cds_90s;
125
126 lives_ok {
127   my @cds_90s_95 = $artist2->cds_90s->search({ 'me.year' => 1995 });
128   is(@cds_90s_95, 1, '1 90s (95) cds found even with non-optimized search');
129   map { ok($_->year == 1995) } @cds_90s_95;
130 } 'should preserve chain-head "me" alias (API-consistency)';
131
132 # search for all artists prefetching published cds in the 80s...
133 my @all_artists_with_80_cds = $schema->resultset("Artist")->search
134   ({ 'cds_80s.cdid' => { '!=' => undef } }, { join => 'cds_80s', distinct => 1 });
135
136 is_deeply(
137   [ sort ( map { $_->year } map { $_->cds_80s->all } @all_artists_with_80_cds ) ],
138   [ sort (1980..1989, 1980..1985) ],
139   '16 correct cds found'
140 );
141
142 lives_ok {
143
144 my @all_artists_with_80_cds_pref = $schema->resultset("Artist")->search
145   ({ 'cds_80s.cdid' => { '!=' => undef } }, { prefetch => 'cds_80s' });
146
147 is_deeply(
148   [ sort ( map { $_->year } map { $_->cds_80s->all } @all_artists_with_80_cds_pref ) ],
149   [ sort (1980..1989, 1980..1985) ],
150   '16 correct cds found'
151 );
152
153 } 'prefetchy-fetchy-fetch';
154
155 # create_related a plain cd via the equoivalent coderef cond, with no extra conditions
156 lives_ok {
157   $artist->create_related('cds_cref_cond', { title => 'related creation via coderef cond', year => '2010' } );
158 } 'created_related with simple condition works';
159
160 # try to create_related a 80s cd
161 throws_ok {
162   $artist->create_related('cds_80s', { title => 'related creation 1' });
163 } qr/\QUnable to complete value inferrence - custom relationship 'cds_80s' on source 'Artist' returns conditions instead of values for column(s): 'year'/,
164 'Create failed - complex cond';
165
166 # now supply an explicit arg overwriting the ambiguous cond
167 my $cd_2020 = $artist->create_related('cds_80s', { title => 'related creation 2', year => '2020' });
168 my $id_2020 = $cd_2020->id;
169 is(
170   $schema->resultset('CD')->find($id_2020)->title,
171   'related creation 2',
172   '2020 CD created correctly'
173 );
174
175 # try a default year from a specific rel
176 my $id_1984 = $artist->create_related('cds_84', { title => 'related creation 3' })->id;
177 is(
178   $schema->resultset('CD')->find($id_1984)->title,
179   'related creation 3',
180   '1984 CD created correctly'
181 );
182
183 # try a specific everything via a non-simplified rel
184 throws_ok {
185   $artist->create_related('cds_90s', { title => 'related_creation 4', year => '2038' });
186 } qr/\QRelationship 'cds_90s' on source 'Artist' does not resolve to a join-free condition fragment/,
187 'Create failed - non-simplified rel';
188
189 # Do a self-join last-entry search
190 my @last_tracks;
191 for my $cd ($schema->resultset('CD')->search ({}, { order_by => 'cdid'})->all) {
192   push @last_tracks, $cd->tracks
193                          ->search ({}, { order_by => { -desc => 'position'} })
194                           ->next || ();
195 }
196
197 my $last_tracks_rs = $schema->resultset('Track')->search (
198   {'next_tracks.trackid' => undef},
199   { join => 'next_tracks', order_by => 'me.cd' },
200 );
201
202 is_deeply (
203   [$last_tracks_rs->get_column ('trackid')->all],
204   [ map { $_->trackid } @last_tracks ],
205   'last group-entry via self-join works',
206 );
207
208 is_deeply (
209   [map { $_->last_track->id } grep { $_->last_track } $schema->resultset('CD')->search ({}, { order_by => 'cdid', prefetch => 'last_track'})->all],
210   [ map { $_->trackid } @last_tracks ],
211   'last_track via insane subquery condition works',
212 );
213
214 is_deeply (
215   [map { $_->last_track->id } grep { $_->last_track } $schema->resultset('CD')->search ({}, { order_by => 'cdid'})->all],
216   [ map { $_->trackid } @last_tracks ],
217   'last_track via insane subquery condition works, even without prefetch',
218 );
219
220 my $artwork = $schema->resultset('Artwork')->search({},{ order_by => 'cd_id' })->first;
221 my @artists = $artwork->artists->all;
222 is(scalar @artists, 2, 'the two artists are associated');
223
224 my @artwork_artists = $artwork->artwork_to_artist->all;
225 foreach (@artwork_artists) {
226   lives_ok {
227     my $artista = $_->artist;
228     my $artistb = $_->artist_test_m2m;
229     ok($artista->rank < 10 ? $artistb : 1, 'belongs_to with custom rel works.');
230     my $artistc = $_->artist_test_m2m_noopt;
231     ok($artista->rank < 10 ? $artistc : 1, 'belongs_to with custom rel works even in non-simplified.');
232   } 'belongs_to works with custom rels';
233 }
234
235 @artists = ();
236 lives_ok {
237   @artists = $artwork->artists_test_m2m2->all;
238 } 'manytomany with extended rels in the has many works';
239 is(scalar @artists, 2, 'two artists');
240
241 @artists = ();
242 lives_ok {
243   @artists = $artwork->artists_test_m2m->all;
244 } 'can fetch many to many with optimized version';
245 is(scalar @artists, 1, 'only one artist is associated');
246
247 @artists = ();
248 lives_ok {
249   @artists = $artwork->artists_test_m2m_noopt->all;
250 } 'can fetch many to many with non-optimized version';
251 is(scalar @artists, 1, 'only one artist is associated');
252
253
254 # Make a single for each last_track
255 my @singles = map {
256   $_->create_related('cd_single', {
257     title => $_->title . ' (the single)',
258     artist => $artist,
259     year => 1999,
260   }) } @last_tracks
261 ;
262
263 # See if chaining works
264 is_deeply (
265   [ map { $_->title } $last_tracks_rs->search_related('cd_single')->all ],
266   [ map { $_->title } @singles ],
267   'Retrieved singles in proper order'
268 );
269
270 # See if prefetch works
271 is_deeply (
272   [ map { $_->cd_single->title } $last_tracks_rs->search({}, { prefetch => 'cd_single' })->all ],
273   [ map { $_->title } @singles ],
274   'Prefetched singles in proper order'
275 );
276
277 # test set_from_related/find_related with a belongs_to custom condition
278 my $preexisting_cd = $schema->resultset('CD')->find(1);
279
280 my $cd_single_track = $schema->resultset('CD')->create({
281   artist => $artist,
282   title => 'one one one',
283   year => 2001,
284   tracks => [{ title => 'uno uno uno' }]
285 });
286
287 my $single_track = $cd_single_track->tracks->next;
288
289 is(
290   $single_track->cd_cref_cond->title,
291   $cd_single_track->title,
292   'Got back the expected single-track cd title',
293 );
294
295 is_deeply
296   { $schema->resultset('Track')->find({ cd_cref_cond => { cdid => $cd_single_track->id } })->get_columns },
297   { $single_track->get_columns },
298   'Proper find with related via coderef cond',
299 ;
300
301 warnings_exist {
302   is_same_sql_bind(
303     $single_track->deliberately_broken_all_cd_tracks->as_query,
304     '(
305       SELECT me.trackid, me.cd, me.position, me.title, me.last_updated_on, me.last_updated_at
306         FROM track track__row
307         JOIN track me
308           ON me.cd = ?
309       WHERE track__row.trackid = ?
310     )',
311     [
312       [{ dbic_colname => "me.cd", sqlt_datatype => "integer" }
313         => "track__row.cd" ],
314       [{ dbic_colname => "track__row.trackid", sqlt_datatype => "integer" }
315         => 19 ],
316     ],
317     'Expected nonsensical JOIN cond',
318   ),
319 } qr/\Qrelationship 'deliberately_broken_all_cd_tracks' on source 'Track' specifies equality of column 'cd' and the *VALUE* 'cd' (you did not use the { -ident => ... } operator)/,
320   'Warning on 99.9999% malformed custom cond'
321 ;
322
323 $single_track->set_from_related( cd_cref_cond => undef );
324 ok $single_track->is_column_changed('cd');
325 is $single_track->get_column('cd'), undef, 'UNset from related via coderef cond';
326 is $single_track->cd, undef, 'UNset related object via coderef cond';
327
328 $single_track->discard_changes;
329
330 $single_track->set_from_related( cd_cref_cond => $preexisting_cd );
331 ok $single_track->is_column_changed('cd');
332 is $single_track->get_column('cd'), 1, 'set from related via coderef cond';
333 is_deeply
334   { $single_track->cd->get_columns },
335   { $preexisting_cd->get_columns },
336   'set from related via coderef cond inflates properly',
337 ;
338
339 throws_ok {
340   local $schema->source('Track')->relationship_info('cd_cref_cond')->{cond} = sub { 1,2,3 };
341   $schema->resultset('Track')->find({ cd_cref_cond => {} });
342 } qr/\QA custom condition coderef can return at most 2 conditions, but relationship 'cd_cref_cond' on source 'Track' returned extra values: 3/;
343
344 done_testing;