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