Merge the relationship resolution rework
[dbsrgits/DBIx-Class.git] / t / relationship / core.t
CommitLineData
c0329273 1BEGIN { do "./t/lib/ANFANG.pm" or die ( $@ || $! ) }
2
70350518 3use strict;
2f926160 4use warnings;
70350518 5
4b8dcc58 6use Test::More;
2f926160 7use Test::Exception;
8a67d9cf 8use Test::Warn;
c0329273 9
a5a7bb73 10use DBICTest ':DiffSQL';
4b8dcc58 11
a47e1233 12my $schema = DBICTest->init_schema();
5efe4c79 13
0567538f 14# has_a test
f9db5527 15my $cd = $schema->resultset("CD")->find(4);
d48427a7 16my ($artist) = $cd->search_related('artist');
0567538f 17is($artist->name, 'Random Boy Band', 'has_a search_related ok');
18
19# has_many test with an order_by clause defined
f9db5527 20$artist = $schema->resultset("Artist")->find(1);
d48427a7 21my @cds = $artist->search_related('cds');
07037f89 22is( $cds[1]->title, 'Spoonful of bees', 'has_many search_related with order_by ok' );
0567538f 23
24# search_related with additional abstract query
d48427a7 25@cds = $artist->search_related('cds', { title => { like => '%of%' } } );
0567538f 26is( $cds[1]->title, 'Forkful of bees', 'search_related with abstract query ok' );
27
28# creating a related object
d48427a7 29$artist->create_related( 'cds', {
30 title => 'Big Flop',
31 year => 2005,
32} );
07037f89 33
c89815db 34my $big_flop_cd = ($artist->search_related('cds'))[3];
35is( $big_flop_cd->title, 'Big Flop', 'create_related ok' );
36
49eeb48d 37# make sure we are not making pointless select queries when a FK IS NULL
38$schema->is_executed_querycount( sub {
c89815db 39 $big_flop_cd->genre; #should not trigger a select query
49eeb48d 40}, 0, 'No SELECT made for belongs_to if key IS NULL');
41
42$schema->is_executed_querycount( sub {
cef1bdda 43 $big_flop_cd->genre_inefficient; #should trigger a select query
49eeb48d 44}, 1, 'SELECT made for belongs_to if key IS NULL when undef_on_null_fk disabled');
0567538f 45
5b89a768 46my( $rs_from_list ) = $artist->search_related_rs('cds');
660cf1be 47isa_ok( $rs_from_list, 'DBIx::Class::ResultSet', 'search_related_rs in list context returns rs' );
5b89a768 48
49( $rs_from_list ) = $artist->cds_rs();
660cf1be 50isa_ok( $rs_from_list, 'DBIx::Class::ResultSet', 'relation_rs in list context returns rs' );
5b89a768 51
0567538f 52# count_related
53is( $artist->count_related('cds'), 4, 'count_related ok' );
54
55# set_from_related
f9db5527 56my $track = $schema->resultset("Track")->create( {
0567538f 57 trackid => 1,
58 cd => 3,
59 position => 98,
60 title => 'Hidden Track'
61} );
62$track->set_from_related( cd => $cd );
63
0818c9a7 64# has_relationship
65ok(! $track->has_relationship( 'foo' ), 'Track has no relationship "foo"');
66ok($track->has_relationship( 'disc' ), 'Track has relationship "disk"' );
67
70350518 68is($track->disc->cdid, 4, 'set_from_related ok, including alternative accessor' );
0567538f 69
2c037e6b 70$track->set_from_related( cd => undef );
71
72ok( !defined($track->cd), 'set_from_related with undef ok');
73
35f5c265 74$track = $schema->resultset("Track")->new( {} );
75$track->cd;
8273e845 76$track->set_from_related( cd => $cd );
35f5c265 77ok ($track->cd, 'set_from_related ok after using the accessor' );
2c037e6b 78
0567538f 79# update_from_related, the same as set_from_related, but it calls update afterwards
f9db5527 80$track = $schema->resultset("Track")->create( {
0567538f 81 trackid => 2,
82 cd => 3,
365d06b7 83 title => 'Hidden Track 2'
0567538f 84} );
85$track->update_from_related( cd => $cd );
86
450e6dbf 87my $t_cd = ($schema->resultset("Track")->search({ cd => 4, title => 'Hidden Track 2' }))[0]->cd;
0567538f 88
70350518 89is( $t_cd->cdid, 4, 'update_from_related ok' );
0567538f 90
91# find_or_create_related with an existing record
92$cd = $artist->find_or_create_related( 'cds', { title => 'Big Flop' } );
93is( $cd->year, 2005, 'find_or_create_related on existing record ok' );
94
95# find_or_create_related creating a new record
96$cd = $artist->find_or_create_related( 'cds', {
97 title => 'Greatest Hits',
98 year => 2006,
99} );
100is( $cd->title, 'Greatest Hits', 'find_or_create_related new record ok' );
e02b9964 101
0567538f 102@cds = $artist->search_related('cds');
103is( ($artist->search_related('cds'))[4]->title, 'Greatest Hits', 'find_or_create_related new record search ok' );
104
87772e46 105$artist->delete_related( cds => { title => 'Greatest Hits' });
450e6dbf 106cmp_ok( $schema->resultset("CD")->search({ title => 'Greatest Hits' }), '==', 0, 'delete_related ok' );
0567538f 107
b3e1f1f5 108# find_or_new_related with an existing record
109$cd = $artist->find_or_new_related( 'cds', { title => 'Big Flop' } );
110is( $cd->year, 2005, 'find_or_new_related on existing record ok' );
111ok( $cd->in_storage, 'find_or_new_related on existing record: is in_storage' );
112
113# find_or_new_related instantiating a new record
114$cd = $artist->find_or_new_related( 'cds', {
115 title => 'Greatest Hits 2: Louder Than Ever',
116 year => 2007,
117} );
118is( $cd->title, 'Greatest Hits 2: Louder Than Ever', 'find_or_new_related new record ok' );
63bb9738 119is( $cd->in_storage, 0, 'find_or_new_related on a new record: not in_storage' );
b3e1f1f5 120
e02b9964 121$cd->artist(undef);
122my $newartist = $cd->find_or_new_related( 'artist', {
123 name => 'Random Boy Band Two',
124 artistid => 200,
125} );
e02b9964 126is($newartist->name, 'Random Boy Band Two', 'find_or_new_related new artist record with id');
127is($newartist->id, 200, 'find_or_new_related new artist id set');
128
8273e845 129lives_ok(
130 sub {
f623c6ee 131 my $new_bookmark = $schema->resultset("Bookmark")->new_result( {} );
132 my $new_related_link = $new_bookmark->new_related( 'link', {} );
133 },
134 'No back rel'
135);
136
3f8affda 137throws_ok {
138 my $new_bookmark = $schema->resultset("Bookmark")->new_result( {} );
139 $new_bookmark->new_related( no_such_rel => {} );
140} qr/No such relationship 'no_such_rel'/, 'creating in uknown rel throws';
f623c6ee 141
b3f358b5 142# many_to_many helper tests
f9db5527 143$cd = $schema->resultset("CD")->find(1);
fb88ca2c 144my @producers = $cd->producers(undef, { order_by => 'producerid'} );
7411204b 145is( $producers[0]->name, 'Matt S Trout', 'many_to_many ok' );
b3f358b5 146is( $cd->producers_sorted->next->name, 'Bob The Builder',
147 'sorted many_to_many ok' );
450e6dbf 148is( $cd->producers_sorted({producerid => 3})->next->name, 'Fred The Phenotype',
b3f358b5 149 'sorted many_to_many with search condition ok' );
7411204b 150
3a868fb2 151$cd = $schema->resultset('CD')->find(2);
b3f358b5 152my $prod_rs = $cd->producers();
153my $prod_before_count = $schema->resultset('Producer')->count;
154is( $prod_rs->count, 0, "CD doesn't yet have any producers" );
3a868fb2 155my $prod = $schema->resultset('Producer')->find(1);
156$cd->add_to_producers($prod);
3a868fb2 157is( $prod_rs->count(), 1, 'many_to_many add_to_$rel($obj) count ok' );
b3f358b5 158is( $prod_rs->first->name, 'Matt S Trout',
159 'many_to_many add_to_$rel($obj) ok' );
3a868fb2 160$cd->remove_from_producers($prod);
ac36a402 161$cd->add_to_producers($prod, {attribute => 1});
162is( $prod_rs->count(), 1, 'many_to_many add_to_$rel($obj, $link_vals) count ok' );
163is( $cd->cd_to_producer->first->attribute, 1, 'many_to_many $link_vals ok');
164$cd->remove_from_producers($prod);
165$cd->set_producers([$prod], {attribute => 2});
166is( $prod_rs->count(), 1, 'many_to_many set_$rel($obj, $link_vals) count ok' );
167is( $cd->cd_to_producer->first->attribute, 2, 'many_to_many $link_vals ok');
168$cd->remove_from_producers($prod);
b3f358b5 169is( $schema->resultset('Producer')->find(1)->name, 'Matt S Trout',
170 "producer object exists after remove of link" );
171is( $prod_rs->count, 0, 'many_to_many remove_from_$rel($obj) ok' );
303cf522 172$cd->add_to_producers({ name => 'Testy McProducer' });
b3f358b5 173is( $schema->resultset('Producer')->count, $prod_before_count+1,
174 'add_to_$rel($hash) inserted a new producer' );
303cf522 175is( $prod_rs->count(), 1, 'many_to_many add_to_$rel($hash) count ok' );
b3f358b5 176is( $prod_rs->first->name, 'Testy McProducer',
177 'many_to_many add_to_$rel($hash) ok' );
178$cd->add_to_producers({ name => 'Jack Black' });
179is( $prod_rs->count(), 2, 'many_to_many add_to_$rel($hash) count ok' );
8a67d9cf 180
181warnings_like {
182 $cd->set_producers($schema->resultset('Producer')->all);
183 is( $cd->producers->count(), $prod_before_count+2,
184 'many_to_many set_$rel(@objs) count ok' );
185
186 $cd->set_producers($schema->resultset('Producer')->find(1));
187 is( $cd->producers->count(), 1, 'many_to_many set_$rel($obj) count ok' );
188} [
189 ( qr/\QCalling 'set_producers' with a list of items to link to is deprecated, use an arrayref instead/ ) x 2
190], 'Warnings on deprecated invocation of set_* found';
191
69831300 192warnings_like {
193 is( $cd->producers( producerid => '666' )->count, 0 );
194} [
195 qr/\Qsearch( %condition ) is deprecated/
196], 'Warning properly bubbled from search()';
197
2d9a96fd 198# the undef-attr-arg at the end is deliberate: this is what FormFu does
199$cd->set_producers([$schema->resultset('Producer')->all], undef);
8273e845 200is( $cd->producers->count(), $prod_before_count+2,
21c45f7b 201 'many_to_many set_$rel(\@objs) count ok' );
202$cd->set_producers([$schema->resultset('Producer')->find(1)]);
203is( $cd->producers->count(), 1, 'many_to_many set_$rel([$obj]) count ok' );
303cf522 204
4017fe64 205throws_ok {
206 $cd->remove_from_producers({ fake => 'hash' })
69831300 207} qr/expects an object/, 'remove_from_$rel($hash) dies correctly';
303cf522 208
4017fe64 209throws_ok {
210 $cd->add_to_producers()
69831300 211} qr/expects an object or hashref/, 'add_to_$rel(undef) dies correctly';
303cf522 212
3bd6e3e0 213# many_to_many stresstest
214my $twokey = $schema->resultset('TwoKeys')->find(1,1);
215my $fourkey = $schema->resultset('FourKeys')->find(1,2,3,4);
216
217is( $twokey->fourkeys->count, 0, 'twokey has no fourkeys' );
218$twokey->add_to_fourkeys($fourkey, { autopilot => 'engaged' });
219my $got_fourkey = $twokey->fourkeys({ sensors => 'online' })->first;
220is( $twokey->fourkeys->count, 1, 'twokey has one fourkey' );
221is( $got_fourkey->$_, $fourkey->$_,
222 'fourkeys row has the correct value for column '.$_ )
223 for (qw(foo bar hello goodbye sensors));
224$twokey->remove_from_fourkeys($fourkey);
225is( $twokey->fourkeys->count, 0, 'twokey has no fourkeys' );
b3f358b5 226is( $twokey->fourkeys_to_twokeys->count, 0,
227 'twokey has no links to fourkey' );
228
ac36a402 229
ac8a5ba4 230my $undef_artist_cd = $schema->resultset("CD")->new_result({ 'title' => 'badgers', 'year' => 2007 });
5ae153d7 231ok(! $undef_artist_cd->has_column_loaded('artist'), 'FK not loaded');
370f2ba2 232is($undef_artist_cd->search_related('artist')->count, 0, '0=1 search when FK does not exist and object not yet in db');
4017fe64 233lives_ok {
2c5c07ec 234 $undef_artist_cd->related_resultset('artist')->new({name => 'foo'});
4017fe64 235} 'Object created on a resultset related to not yet inserted object';
981299aa 236lives_ok{
237 $schema->resultset('Artwork')->new_result({})->cd;
238} 'undef_on_null_fk does not choke on empty conds';
239
ac8a5ba4 240my $def_artist_cd = $schema->resultset("CD")->new_result({ 'title' => 'badgers', 'year' => 2007, artist => undef });
241is($def_artist_cd->has_column_loaded('artist'), 1, 'FK loaded');
242is($def_artist_cd->search_related('artist')->count, 0, 'closed search on null FK');
3a868fb2 243
5efe4c79 244# test undirected many-to-many relationship (e.g. "related artists")
af439a0e 245my $undir_maps = $schema->resultset("Artist")
246 ->search ({artistid => 1})
247 ->search_related ('artist_undirected_maps');
5efe4c79 248is($undir_maps->count, 1, 'found 1 undirected map for artist 1');
af439a0e 249is_same_sql_bind (
250 $undir_maps->as_query,
251 '(
252 SELECT artist_undirected_maps.id1, artist_undirected_maps.id2
253 FROM artist me
95c73ab2 254 JOIN artist_undirected_map artist_undirected_maps
af439a0e 255 ON artist_undirected_maps.id1 = me.artistid OR artist_undirected_maps.id2 = me.artistid
256 WHERE ( artistid = ? )
257 )',
0e773352 258 [[ { sqlt_datatype => 'integer', dbic_colname => 'artistid' }
259 => 1 ]],
af439a0e 260 'expected join sql produced',
261);
5efe4c79 262
f9db5527 263$undir_maps = $schema->resultset("Artist")->find(2)->artist_undirected_maps;
5efe4c79 264is($undir_maps->count, 1, 'found 1 undirected map for artist 2');
265
de54e8bd 266{
267 my $artist_to_mangle = $schema->resultset('Artist')->find(2);
268
d8516e92 269 throws_ok {
270 $artist_to_mangle->set_from_related( artist_undirected_maps => { id1 => 42 } )
271 } qr/\QUnable to complete value inferrence - relationship 'artist_undirected_maps' on source 'Artist' results in expression(s) instead of definitive values: ( artistid = ? OR artistid IS NULL )/,
272 'Expected exception on unresovable set_from_related'
273 ;
de54e8bd 274
275 ok( ! $artist_to_mangle->is_changed, 'Unresolvable set_from_related did not alter object' );
276
277 $artist_to_mangle->set_from_related( artist_undirected_maps => {} );
278 ok( $artist_to_mangle->is_changed, 'Definitive set_from_related did alter object' );
279 is (
280 $artist_to_mangle->id,
281 undef,
282 'Correctly unset id on definitive outcome of OR condition',
283 );
284}
285
ad3d2d7c 286my $mapped_rs = $undir_maps->search_related('mapped_artists');
287
288my @art = $mapped_rs->all;
5efe4c79 289
290cmp_ok(@art, '==', 2, "Both artist returned from map");
291
ad3d2d7c 292my $searched = $mapped_rs->search({'mapped_artists.artistid' => {'!=', undef}});
293
294cmp_ok($searched->count, '==', 2, "Both artist returned from map after adding another condition");
295
597508e8 296# check join through cascaded has_many relationships (also empty has_many rels)
b8d4bd90 297$artist = $schema->resultset("Artist")->find(1);
298my $trackset = $artist->cds->search_related('tracks');
597508e8 299is($trackset->count, 10, "Correct number of tracks for artist");
300is($trackset->all, 10, "Correct number of track objects for artist");
ad3d2d7c 301
0f57d214 302# now see about updating eveything that belongs to artist 2 to artist 3
303$artist = $schema->resultset("Artist")->find(2);
304my $nartist = $schema->resultset("Artist")->find(3);
305cmp_ok($artist->cds->count, '==', 1, "Correct orig #cds for artist");
306cmp_ok($nartist->cds->count, '==', 1, "Correct orig #cds for artist");
307$artist->cds->update({artist => $nartist->id});
308cmp_ok($artist->cds->count, '==', 0, "Correct new #cds for artist");
309cmp_ok($nartist->cds->count, '==', 2, "Correct new #cds for artist");
310
6bf6ba2f 311# check if is_foreign_key_constraint attr is set
312my $rs_normal = $schema->source('Track');
313my $relinfo = $rs_normal->relationship_info ('cd');
314cmp_ok($relinfo->{attrs}{is_foreign_key_constraint}, '==', 1, "is_foreign_key_constraint defined for belongs_to relationships.");
315
316my $rs_overridden = $schema->source('ForceForeign');
317my $relinfo_with_attr = $rs_overridden->relationship_info ('cd_3');
318cmp_ok($relinfo_with_attr->{attrs}{is_foreign_key_constraint}, '==', 0, "is_foreign_key_constraint defined for belongs_to relationships with attr.");
caac1708 319
8273e845 320# check that relationships below left join relationships are forced to left joins
56b73f83 321# when traversing multiple belongs_to
322my $cds = $schema->resultset("CD")->search({ 'me.cdid' => 5 }, { join => { single_track => 'cd' } });
caac1708 323is($cds->count, 1, "subjoins under left joins force_left (string)");
324
56b73f83 325$cds = $schema->resultset("CD")->search({ 'me.cdid' => 5 }, { join => { single_track => ['cd'] } });
caac1708 326is($cds->count, 1, "subjoins under left joins force_left (arrayref)");
327
56b73f83 328$cds = $schema->resultset("CD")->search({ 'me.cdid' => 5 }, { join => { single_track => { cd => {} } } });
caac1708 329is($cds->count, 1, "subjoins under left joins force_left (hashref)");
af439a0e 330
331done_testing;