Commit | Line | Data |
c0329273 |
1 | BEGIN { do "./t/lib/ANFANG.pm" or die ( $@ || $! ) } |
2 | |
70350518 |
3 | use strict; |
be83e9ec |
4 | use warnings; |
70350518 |
5 | |
6 | use Test::More; |
d6df786a |
7 | use Test::Exception; |
00f3b1c7 |
8 | use Test::Warn; |
c0329273 |
9 | |
a5a7bb73 |
10 | use DBICTest ':DiffSQL'; |
70350518 |
11 | |
ae515736 |
12 | my $schema = DBICTest->init_schema(); |
0567538f |
13 | |
f9db5527 |
14 | my @art = $schema->resultset("Artist")->search({ }, { order_by => 'name DESC'}); |
0567538f |
15 | |
d2f21b37 |
16 | is(@art, 3, "Three artists returned"); |
0567538f |
17 | |
18 | my $art = $art[0]; |
19 | |
20 | is($art->name, 'We Are Goth', "Correct order too"); |
21 | |
22 | $art->name('We Are In Rehab'); |
23 | |
24 | is($art->name, 'We Are In Rehab', "Accessor update ok"); |
25 | |
6dbea98e |
26 | my %dirty = $art->get_dirty_columns(); |
d2f21b37 |
27 | is(scalar(keys(%dirty)), 1, '1 dirty column'); |
6dbea98e |
28 | ok(grep($_ eq 'name', keys(%dirty)), 'name is dirty'); |
29 | |
0567538f |
30 | is($art->get_column("name"), 'We Are In Rehab', 'And via get_column'); |
31 | |
32 | ok($art->update, 'Update run'); |
33 | |
6dbea98e |
34 | my %not_dirty = $art->get_dirty_columns(); |
d2f21b37 |
35 | is(scalar(keys(%not_dirty)), 0, 'Nothing is dirty'); |
6dbea98e |
36 | |
00f3b1c7 |
37 | throws_ok ( sub { |
6dbea98e |
38 | my $ret = $art->make_column_dirty('name2'); |
00f3b1c7 |
39 | }, qr/No such column 'name2'/, 'Failed to make non-existent column dirty'); |
40 | |
6dbea98e |
41 | $art->make_column_dirty('name'); |
42 | my %fake_dirty = $art->get_dirty_columns(); |
d2f21b37 |
43 | is(scalar(keys(%fake_dirty)), 1, '1 fake dirty column'); |
6dbea98e |
44 | ok(grep($_ eq 'name', keys(%fake_dirty)), 'name is fake dirty'); |
45 | |
de5ce481 |
46 | ok($art->update, 'Update run'); |
47 | |
ae515736 |
48 | my $record_jp = $schema->resultset("Artist")->search(undef, { join => 'cds' })->search(undef, { prefetch => 'cds' })->next; |
49 | |
50 | ok($record_jp, "prefetch on same rel okay"); |
51 | |
52 | my $record_fn = $schema->resultset("Artist")->search(undef, { join => 'cds' })->search({'cds.cdid' => '1'}, {join => 'artist_undirected_maps'})->next; |
53 | |
54 | ok($record_fn, "funny join is okay"); |
55 | |
f9db5527 |
56 | @art = $schema->resultset("Artist")->search({ name => 'We Are In Rehab' }); |
0567538f |
57 | |
d2f21b37 |
58 | is(@art, 1, "Changed artist returned by search"); |
0567538f |
59 | |
d2f21b37 |
60 | is($art[0]->artistid, 3,'Correct artist too'); |
0567538f |
61 | |
d6df786a |
62 | lives_ok (sub { $art->delete }, 'Cascading delete on Ordered has_many works' ); # real test in ordered.t |
0567538f |
63 | |
f9db5527 |
64 | @art = $schema->resultset("Artist")->search({ }); |
0567538f |
65 | |
d2f21b37 |
66 | is(@art, 2, 'And then there were two'); |
0567538f |
67 | |
63bb9738 |
68 | is($art->in_storage, 0, "It knows it's dead"); |
0567538f |
69 | |
de5ce481 |
70 | lives_ok { $art->update } 'No changes so update should be OK'; |
71 | |
d6df786a |
72 | dies_ok ( sub { $art->delete }, "Can't delete twice"); |
0567538f |
73 | |
74 | is($art->name, 'We Are In Rehab', 'But the object is still live'); |
75 | |
76 | $art->insert; |
77 | |
78 | ok($art->in_storage, "Re-created"); |
79 | |
f9db5527 |
80 | @art = $schema->resultset("Artist")->search({ }); |
0567538f |
81 | |
d2f21b37 |
82 | is(@art, 3, 'And now there are three again'); |
0567538f |
83 | |
f9db5527 |
84 | my $new = $schema->resultset("Artist")->create({ artistid => 4 }); |
0567538f |
85 | |
d2f21b37 |
86 | is($new->artistid, 4, 'Create produced record ok'); |
0567538f |
87 | |
f9db5527 |
88 | @art = $schema->resultset("Artist")->search({ }); |
0567538f |
89 | |
d2f21b37 |
90 | is(@art, 4, "Oh my god! There's four of them!"); |
0567538f |
91 | |
92 | $new->set_column('name' => 'Man With A Fork'); |
93 | |
94 | is($new->name, 'Man With A Fork', 'set_column ok'); |
95 | |
96 | $new->discard_changes; |
97 | |
98 | ok(!defined $new->name, 'Discard ok'); |
99 | |
100 | $new->name('Man With A Spoon'); |
101 | |
102 | $new->update; |
103 | |
70350518 |
104 | my $new_again = $schema->resultset("Artist")->find(4); |
0567538f |
105 | |
106 | is($new_again->name, 'Man With A Spoon', 'Retrieved correctly'); |
107 | |
9bbd8963 |
108 | is($new_again->ID, 'DBICTest::Artist|artist|artistid=4', 'unique object id generated correctly'); |
1f6715ab |
109 | |
8273e845 |
110 | # test that store_column is called once for create() for non sequence columns |
52c53388 |
111 | { |
112 | ok(my $artist = $schema->resultset('Artist')->create({name => 'store_column test'})); |
113 | is($artist->name, 'X store_column test'); # used to be 'X X store...' |
b236052f |
114 | |
a22688ab |
115 | # call store_column even though the column doesn't seem to be dirty |
b236052f |
116 | $artist->name($artist->name); |
a22688ab |
117 | is($artist->name, 'X X store_column test'); |
b236052f |
118 | ok($artist->is_column_changed('name'), 'changed column marked as dirty'); |
119 | |
52c53388 |
120 | $artist->delete; |
121 | } |
122 | |
450e6dbf |
123 | # deprecation of rolled-out search |
124 | warnings_exist { |
125 | $schema->resultset('Artist')->search_rs(id => 4) |
126 | } qr/\Qsearch( %condition ) is deprecated/, 'Deprecation warning on ->search( %condition )'; |
127 | |
f9db5527 |
128 | is($schema->resultset("Artist")->count, 4, 'count ok'); |
0567538f |
129 | |
76cc4546 |
130 | |
b3e1f1f5 |
131 | # test find_or_new |
132 | { |
133 | my $existing_obj = $schema->resultset('Artist')->find_or_new({ |
134 | artistid => 4, |
135 | }); |
136 | |
137 | is($existing_obj->name, 'Man With A Spoon', 'find_or_new: found existing artist'); |
138 | ok($existing_obj->in_storage, 'existing artist is in storage'); |
139 | |
140 | my $new_obj = $schema->resultset('Artist')->find_or_new({ |
141 | artistid => 5, |
142 | name => 'find_or_new', |
143 | }); |
144 | |
145 | is($new_obj->name, 'find_or_new', 'find_or_new: instantiated a new artist'); |
63bb9738 |
146 | is($new_obj->in_storage, 0, 'new artist is not in storage'); |
b3e1f1f5 |
147 | } |
148 | |
f9db5527 |
149 | my $cd = $schema->resultset("CD")->find(1); |
076a6864 |
150 | my %cols = $cd->get_columns; |
151 | |
d2f21b37 |
152 | is(keys %cols, 6, 'get_columns number of columns ok'); |
076a6864 |
153 | |
154 | is($cols{title}, 'Spoonful of bees', 'get_columns values ok'); |
155 | |
156 | %cols = ( title => 'Forkful of bees', year => 2005); |
157 | $cd->set_columns(\%cols); |
158 | |
159 | is($cd->title, 'Forkful of bees', 'set_columns ok'); |
160 | |
161 | is($cd->year, 2005, 'set_columns ok'); |
162 | |
163 | $cd->discard_changes; |
164 | |
20518cb4 |
165 | # check whether ResultSource->columns returns columns in order originally supplied |
166 | my @cd = $schema->source("CD")->columns; |
571dced3 |
167 | |
a1cb5921 |
168 | is_deeply( \@cd, [qw/cdid artist title year genreid single_track/], 'column order'); |
571dced3 |
169 | |
82a96700 |
170 | $cd = $schema->resultset("CD")->search({ title => 'Spoonful of bees' }, { columns => ['title'] })->next; |
90f3f5ff |
171 | is($cd->title, 'Spoonful of bees', 'subset of columns returned correctly'); |
172 | |
02ddfe6e |
173 | $cd = $schema->resultset("CD")->search(undef, { '+columns' => [ { name => 'artist.name' } ], join => [ 'artist' ] })->find(1); |
5ac6a044 |
174 | |
175 | is($cd->title, 'Spoonful of bees', 'Correct CD returned with include'); |
176 | is($cd->get_column('name'), 'Caterwauler McCrae', 'Additional column returned'); |
177 | |
67ba6646 |
178 | # check if new syntax +columns also works for this |
53998003 |
179 | $cd = $schema->resultset("CD")->search(undef, { '+columns' => [ { name => 'artist.name' } ], join => [ 'artist' ] })->find(1); |
67ba6646 |
180 | |
181 | is($cd->title, 'Spoonful of bees', 'Correct CD returned with include'); |
182 | is($cd->get_column('name'), 'Caterwauler McCrae', 'Additional column returned'); |
183 | |
184 | # check if new syntax for +columns select specifiers works for this |
185 | $cd = $schema->resultset("CD")->search(undef, { '+columns' => [ {artist_name => 'artist.name'} ], join => [ 'artist' ] })->find(1); |
186 | |
187 | is($cd->title, 'Spoonful of bees', 'Correct CD returned with include'); |
188 | is($cd->get_column('artist_name'), 'Caterwauler McCrae', 'Additional column returned'); |
5ac6a044 |
189 | |
82a96700 |
190 | # update_or_insert |
f9db5527 |
191 | $new = $schema->resultset("Track")->new( { |
0567538f |
192 | trackid => 100, |
193 | cd => 1, |
0567538f |
194 | title => 'Insert or Update', |
43556c5d |
195 | last_updated_on => '1973-07-19 12:01:02' |
0567538f |
196 | } ); |
82a96700 |
197 | $new->update_or_insert; |
198 | ok($new->in_storage, 'update_or_insert insert ok'); |
0567538f |
199 | |
00f3b1c7 |
200 | throws_ok (sub { |
201 | $schema->class("Track")->load_components('DoesNotExist'); |
202 | }, qr!Can't locate DBIx/Class/DoesNotExist.pm!, 'exception on nonexisting component'); |
0567538f |
203 | |
1edaf6fe |
204 | is($schema->class("Artist")->field_name_for->{name}, 'artist name', 'mk_classdata usage ok'); |
90e6de6c |
205 | |
54540863 |
206 | my $search = [ { 'tags.tag' => 'Cheesy' }, { 'tags.tag' => 'Blue' } ]; |
207 | |
fb88ca2c |
208 | my $or_rs = $schema->resultset("CD")->search_rs($search, { join => 'tags', |
6aeb9185 |
209 | order_by => 'cdid' }); |
a258ee5d |
210 | is($or_rs->all, 5, 'Joined search with OR returned correct number of rows'); |
211 | is($or_rs->count, 5, 'Search count with OR ok'); |
54540863 |
212 | |
a258ee5d |
213 | my $collapsed_or_rs = $or_rs->search ({}, { distinct => 1 }); # induce collapse |
214 | is ($collapsed_or_rs->all, 4, 'Collapsed joined search with OR returned correct number of rows'); |
215 | is ($collapsed_or_rs->count, 4, 'Collapsed search count with OR ok'); |
6aeb9185 |
216 | |
00f3b1c7 |
217 | # make sure sure distinct on a grouped rs is warned about |
a2f22854 |
218 | { |
219 | my $cd_rs = $schema->resultset ('CD') |
220 | ->search ({}, { distinct => 1, group_by => 'title' }); |
221 | warnings_exist (sub { |
222 | $cd_rs->next; |
223 | }, qr/Useless use of distinct/, 'UUoD warning'); |
224 | } |
00f3b1c7 |
225 | |
1cc3ce1e |
226 | { |
d2f21b37 |
227 | my $tcount = $schema->resultset('Track')->search( |
286f32b3 |
228 | {}, |
d2f21b37 |
229 | { |
11d68671 |
230 | select => [ qw/position title/ ], |
231 | distinct => 1, |
286f32b3 |
232 | } |
233 | ); |
d2f21b37 |
234 | is($tcount->count, 13, 'multiple column COUNT DISTINCT ok'); |
235 | |
11d68671 |
236 | $tcount = $schema->resultset('Track')->search( |
237 | {}, |
238 | { |
239 | columns => [ qw/position title/ ], |
240 | distinct => 1, |
241 | } |
242 | ); |
243 | is($tcount->count, 13, 'multiple column COUNT DISTINCT ok'); |
244 | |
245 | $tcount = $schema->resultset('Track')->search( |
246 | {}, |
247 | { |
248 | group_by => [ qw/position title/ ] |
249 | } |
250 | ); |
8273e845 |
251 | is($tcount->count, 13, 'multiple column COUNT DISTINCT using column syntax ok'); |
f2de4889 |
252 | } |
584e74ed |
253 | |
f9db5527 |
254 | my $tag_rs = $schema->resultset('Tag')->search( |
6aeb9185 |
255 | [ { 'me.tag' => 'Cheesy' }, { 'me.tag' => 'Blue' } ]); |
256 | |
fb88ca2c |
257 | my $rel_rs = $tag_rs->search_related('cd', {}, { order_by => 'cd.cdid'} ); |
6aeb9185 |
258 | |
a258ee5d |
259 | is($rel_rs->count, 5, 'Related search ok'); |
6aeb9185 |
260 | |
d2f21b37 |
261 | is($or_rs->next->cdid, $rel_rs->next->cdid, 'Related object ok'); |
a4731ae0 |
262 | $or_rs->reset; |
263 | $rel_rs->reset; |
a953d8d9 |
264 | |
a2f22854 |
265 | # at this point there should be no active statements |
266 | # (finish() was called everywhere, either explicitly via |
267 | # reset() or on DESTROY) |
268 | for (keys %{$schema->storage->dbh->{CachedKids}}) { |
269 | fail("Unreachable cached statement still active: $_") |
270 | if $schema->storage->dbh->{CachedKids}{$_}->FETCH('Active'); |
271 | } |
272 | |
4c4ccf29 |
273 | my $tag = $schema->resultset('Tag')->search( |
02ddfe6e |
274 | [ { 'me.tag' => 'Blue' } ], |
275 | { columns => 'tagid' } |
276 | )->next; |
4c4ccf29 |
277 | |
d2f21b37 |
278 | ok($tag->has_column_loaded('tagid'), 'Has tagid loaded'); |
279 | ok(!$tag->has_column_loaded('tag'), 'Has not tag loaded'); |
4c4ccf29 |
280 | |
a953d8d9 |
281 | ok($schema->storage(), 'Storage available'); |
282 | |
16b4fd26 |
283 | { |
284 | my $rs = $schema->resultset("Artist")->search({ |
285 | -and => [ |
286 | artistid => { '>=', 1 }, |
287 | artistid => { '<', 3 } |
288 | ] |
289 | }); |
290 | |
84f7e8a1 |
291 | $rs->update({ rank => 6134 }); |
16b4fd26 |
292 | |
293 | my $art; |
294 | |
295 | $art = $schema->resultset("Artist")->find(1); |
84f7e8a1 |
296 | is($art->rank, 6134, 'updated first artist rank'); |
16b4fd26 |
297 | |
298 | $art = $schema->resultset("Artist")->find(2); |
84f7e8a1 |
299 | is($art->rank, 6134, 'updated second artist rank'); |
16b4fd26 |
300 | } |
301 | |
825135d8 |
302 | # test source_name |
303 | { |
304 | # source_name should be set for normal modules |
305 | is($schema->source('CD')->source_name, 'CD', 'source_name is set to moniker'); |
a4731ae0 |
306 | |
825135d8 |
307 | # test the result source that sets source_name explictly |
308 | ok($schema->source('SourceNameArtists'), 'SourceNameArtists result source exists'); |
0009fa49 |
309 | |
825135d8 |
310 | my @artsn = $schema->resultset('SourceNameArtists')->search({}, { order_by => 'name DESC' }); |
d2f21b37 |
311 | is(@artsn, 4, "Four artists returned"); |
8273e845 |
312 | |
b1fb2c94 |
313 | # make sure subclasses that don't set source_name are ok |
93405cf0 |
314 | ok($schema->source('ArtistSubclass'), 'ArtistSubclass exists'); |
825135d8 |
315 | } |
bab77431 |
316 | |
9c2c91ea |
317 | my $newbook = $schema->resultset( 'Bookmark' )->find(1); |
318 | |
d6df786a |
319 | lives_ok (sub { my $newlink = $newbook->link}, "stringify to false value doesn't cause error"); |
9c2c91ea |
320 | |
825135d8 |
321 | # test cascade_delete through many_to_many relations |
322 | { |
323 | my $art_del = $schema->resultset("Artist")->find({ artistid => 1 }); |
d6df786a |
324 | lives_ok (sub { $art_del->delete }, 'Cascading delete on Ordered has_many works' ); # real test in ordered.t |
d2f21b37 |
325 | is( $schema->resultset("CD")->search({artist => 1}), 0, 'Cascading through has_many top level.'); |
326 | is( $schema->resultset("CD_to_Producer")->search({cd => 1}), 0, 'Cascading through has_many children.'); |
825135d8 |
327 | } |
bab77431 |
328 | |
825135d8 |
329 | # test column_info |
330 | { |
331 | $schema->source("Artist")->{_columns}{'artistid'} = {}; |
d9916234 |
332 | $schema->source("Artist")->column_info_from_storage(1); |
bab77431 |
333 | |
825135d8 |
334 | my $typeinfo = $schema->source("Artist")->column_info('artistid'); |
335 | is($typeinfo->{data_type}, 'INTEGER', 'column_info ok'); |
336 | $schema->source("Artist")->column_info('artistid'); |
52416317 |
337 | ok($schema->source("Artist")->{_columns_info_loaded} == 1, 'Columns info loaded flag set'); |
338 | } |
339 | |
340 | # test columns_info |
341 | { |
342 | $schema->source("Artist")->{_columns}{'artistid'} = {}; |
343 | $schema->source("Artist")->column_info_from_storage(1); |
344 | $schema->source("Artist")->{_columns_info_loaded} = 0; |
345 | |
f45dc928 |
346 | my @undef_default = DBIx::Class::_ENV_::STRESSTEST_COLUMN_INFO_UNAWARE_STORAGE |
347 | ? () |
348 | : ( default_value => undef ) |
349 | ; |
350 | |
52416317 |
351 | is_deeply ( |
352 | $schema->source('Artist')->columns_info, |
353 | { |
354 | artistid => { |
355 | data_type => "INTEGER", |
f45dc928 |
356 | @undef_default, |
52416317 |
357 | is_nullable => 0, |
358 | size => undef |
359 | }, |
360 | charfield => { |
361 | data_type => "char", |
f45dc928 |
362 | @undef_default, |
52416317 |
363 | is_nullable => 1, |
364 | size => 10 |
365 | }, |
366 | name => { |
367 | data_type => "varchar", |
f45dc928 |
368 | @undef_default, |
52416317 |
369 | is_nullable => 1, |
370 | is_numeric => 0, |
371 | size => 100 |
372 | }, |
373 | rank => { |
374 | data_type => "integer", |
375 | default_value => 13, |
376 | is_nullable => 0, |
377 | size => undef |
378 | }, |
379 | }, |
380 | 'columns_info works', |
381 | ); |
382 | |
383 | ok($schema->source("Artist")->{_columns_info_loaded} == 1, 'Columns info loaded flag set'); |
384 | |
385 | is_deeply ( |
386 | $schema->source('Artist')->columns_info([qw/artistid rank/]), |
387 | { |
388 | artistid => { |
389 | data_type => "INTEGER", |
f45dc928 |
390 | @undef_default, |
52416317 |
391 | is_nullable => 0, |
392 | size => undef |
393 | }, |
394 | rank => { |
395 | data_type => "integer", |
396 | default_value => 13, |
397 | is_nullable => 0, |
398 | size => undef |
399 | }, |
400 | }, |
401 | 'limited columns_info works', |
402 | ); |
825135d8 |
403 | } |
bab77431 |
404 | |
a48e92d7 |
405 | # test source_info |
406 | { |
407 | my $expected = { |
408 | "source_info_key_A" => "source_info_value_A", |
409 | "source_info_key_B" => "source_info_value_B", |
410 | "source_info_key_C" => "source_info_value_C", |
411 | }; |
412 | |
413 | my $sinfo = $schema->source("Artist")->source_info; |
414 | |
415 | is_deeply($sinfo, $expected, 'source_info data works'); |
416 | } |
417 | |
825135d8 |
418 | # test remove_columns |
419 | { |
4738027b |
420 | is_deeply( |
421 | [$schema->source('CD')->columns], |
422 | [qw/cdid artist title year genreid single_track/], |
423 | 'initial columns', |
424 | ); |
425 | |
426 | $schema->source('CD')->remove_columns('coolyear'); #should not delete year |
427 | is_deeply( |
428 | [$schema->source('CD')->columns], |
429 | [qw/cdid artist title year genreid single_track/], |
430 | 'nothing removed when removing a non-existent column', |
431 | ); |
432 | |
433 | $schema->source('CD')->remove_columns('genreid', 'year'); |
434 | is_deeply( |
435 | [$schema->source('CD')->columns], |
436 | [qw/cdid artist title single_track/], |
437 | 'removed two columns', |
438 | ); |
439 | |
440 | my $priv_columns = $schema->source('CD')->_columns; |
441 | ok(! exists $priv_columns->{'year'}, 'year purged from _columns'); |
442 | ok(! exists $priv_columns->{'genreid'}, 'genreid purged from _columns'); |
825135d8 |
443 | } |
bab77431 |
444 | |
ade8df5b |
445 | # test resultsource->table return value when setting |
446 | { |
447 | my $class = $schema->class('Event'); |
ade8df5b |
448 | my $table = $class->table($class->table); |
449 | is($table, $class->table, '->table($table) returns $table'); |
450 | } |
0e80c4ca |
451 | |
452 | #make sure insert doesn't use set_column |
453 | { |
454 | my $en_row = $schema->resultset('Encoded')->new_result({encoded => 'wilma'}); |
455 | is($en_row->encoded, 'amliw', 'new encodes'); |
456 | $en_row->insert; |
457 | is($en_row->encoded, 'amliw', 'insert does not encode again'); |
458 | } |
3bb4eb8f |
459 | |
68888c09 |
460 | #make sure multicreate encoding still works |
461 | { |
462 | my $empl_rs = $schema->resultset('Employee'); |
463 | |
464 | my $empl = $empl_rs->create ({ |
465 | name => 'Secret holder', |
466 | secretkey => { |
467 | encoded => 'CAN HAZ', |
468 | }, |
469 | }); |
470 | is($empl->secretkey->encoded, 'ZAH NAC', 'correctly encoding on multicreate'); |
471 | |
472 | my $empl2 = $empl_rs->create ({ |
473 | name => 'Same secret holder', |
474 | secretkey => { |
475 | encoded => 'CAN HAZ', |
476 | }, |
477 | }); |
478 | is($empl2->secretkey->encoded, 'ZAH NAC', 'correctly encoding on preexisting multicreate'); |
479 | |
480 | $empl_rs->create ({ |
481 | name => 'cat1', |
482 | secretkey => { |
483 | encoded => 'CHEEZBURGER', |
484 | keyholders => [ |
485 | { |
486 | name => 'cat2', |
487 | }, |
488 | { |
489 | name => 'cat3', |
490 | }, |
491 | ], |
492 | }, |
493 | }); |
494 | |
495 | is($empl_rs->find({name => 'cat1'})->secretkey->encoded, 'REGRUBZEEHC', 'correct secret in database for empl1'); |
496 | is($empl_rs->find({name => 'cat2'})->secretkey->encoded, 'REGRUBZEEHC', 'correct secret in database for empl2'); |
497 | is($empl_rs->find({name => 'cat3'})->secretkey->encoded, 'REGRUBZEEHC', 'correct secret in database for empl3'); |
498 | |
499 | } |
500 | |
4376a157 |
501 | # make sure that obsolete handle-based source tracking continues to work for the time being |
502 | { |
503 | my $handle = $schema->source('Artist')->handle; |
504 | |
51c9ead2 |
505 | my $rowdata = { $schema->resultset('Artist')->next->get_columns }; |
4376a157 |
506 | |
507 | my $rs = DBIx::Class::ResultSet->new($handle); |
508 | my $rs_result = $rs->next; |
509 | isa_ok( $rs_result, 'DBICTest::Artist' ); |
510 | is_deeply ( |
511 | { $rs_result->get_columns }, |
512 | $rowdata, |
513 | 'Correct columns retrieved (rset/source link healthy)' |
514 | ); |
515 | |
516 | my $row = DBICTest::Artist->new({ -source_handle => $handle }); |
517 | is_deeply( |
518 | { $row->get_columns }, |
519 | {}, |
520 | 'No columns yet' |
521 | ); |
522 | |
523 | # store_column to fool the _orig_ident tracker |
524 | $row->store_column('artistid', $rowdata->{artistid}); |
525 | $row->in_storage(1); |
526 | |
527 | $row->discard_changes; |
528 | is_deeply( |
529 | { $row->get_columns }, |
530 | $rowdata, |
531 | 'Storage refetch successful' |
532 | ); |
533 | } |
534 | |
3f1d61d0 |
535 | # test to make sure that calling ->new() on a resultset object gives |
536 | # us a row object |
537 | { |
538 | my $new_artist = $schema->resultset('Artist')->new({}); |
539 | isa_ok( $new_artist, 'DBIx::Class::Row', '$rs->new gives a row object' ); |
540 | } |
541 | |
542 | |
3bb4eb8f |
543 | # make sure we got rid of the compat shims |
544 | SKIP: { |
7f08eb01 |
545 | my $remove_version = 0.083; |
546 | skip "Remove in $remove_version", 3 if $DBIx::Class::VERSION < $remove_version; |
3bb4eb8f |
547 | |
1225a9e0 |
548 | for (qw/compare_relationship_keys pk_depends_on resolve_condition/) { |
7f08eb01 |
549 | ok (! DBIx::Class::ResultSource->can ($_), "$_ no longer provided by DBIx::Class::ResultSource, removed before $remove_version"); |
3bb4eb8f |
550 | } |
551 | } |
42a87bbb |
552 | |
553 | #------------------------------ |
554 | # READ THIS BEFORE "FIXING" |
555 | #------------------------------ |
556 | # |
557 | # make sure we got rid of discard_changes mess - this is a mess and a source |
558 | # of great confusion. Here I simply die if the methods are available, which |
559 | # is wrong on its own (we *have* to provide some sort of back-compat, even |
560 | # if with warnings). Here is how I envision things should actually be. Also |
561 | # note that a lot of the deprecation can be started today (i.e. the switch |
562 | # from get_from_storage to copy_from_storage). So: |
563 | # |
564 | # $row->discard_changes => |
565 | # warning, and delegation to reload_from_storage |
566 | # |
567 | # $row->reload_from_storage => |
568 | # does what discard changes did in 0.08 - issues a query to the db |
569 | # and repopulates all column slots, regardless of dirty states etc. |
570 | # |
571 | # $row->revert_changes => |
572 | # does what discard_changes should have done initially (before it became |
573 | # a dual-purpose call). In order to make this work we will have to |
574 | # augment $row to carry its own initial-state, much like svn has a |
575 | # copy of the current checkout in contrast to cvs. |
576 | # |
577 | # my $db_row = $row->get_from_storage => |
578 | # warns and delegates to an improved name copy_from_storage, with the |
579 | # same semantics |
580 | # |
581 | # my $db_row = $row->copy_from_storage => |
582 | # a much better/descriptive name than get_from_storage |
583 | # |
584 | #------------------------------ |
585 | # READ THIS BEFORE "FIXING" |
586 | #------------------------------ |
587 | # |
588 | SKIP: { |
589 | skip "Something needs to be done before 0.09", 2 if $DBIx::Class::VERSION < 0.09; |
590 | |
591 | my $row = $schema->resultset ('Artist')->next; |
592 | |
593 | for (qw/discard_changes get_from_storage/) { |
594 | ok (! $row->can ($_), "$_ needs *some* sort of facelift before 0.09 ships - current state of affairs is unacceptable"); |
595 | } |
596 | } |
597 | |
73d47f9f |
598 | throws_ok { $schema->resultset} qr/resultset\(\) expects a source name/, 'resultset with no argument throws exception'; |
599 | |
4f52479b |
600 | throws_ok { $schema->source('Artist')->result_class->new( 'bugger' ) } qr/must be a hashref/; |
601 | |
42a87bbb |
602 | done_testing; |