3 # Copyright 2015 Koha Development team
5 # This file is part of Koha
7 # Koha is free software; you can redistribute it and/or modify it
8 # under the terms of the GNU General Public License as published by
9 # the Free Software Foundation; either version 3 of the License, or
10 # (at your option) any later version.
12 # Koha is distributed in the hope that it will be useful, but
13 # WITHOUT ANY WARRANTY; without even the implied warranty of
14 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
15 # GNU General Public License for more details.
17 # You should have received a copy of the GNU General Public License
18 # along with Koha; if not, see <http://www.gnu.org/licenses>.
22 use Test::More tests => 18;
26 use Koha::Authority::Types;
28 use Koha::IssuingRules;
29 use Koha::Patron::Category;
30 use Koha::Patron::Categories;
34 use t::lib::TestBuilder;
38 my $schema = Koha::Database->new->schema;
39 $schema->storage->txn_begin;
40 my $builder = t::lib::TestBuilder->new;
42 is( ref(Koha::Authority::Types->find('')), 'Koha::Authority::Type', 'Koha::Objects->find should work if the primary key is an empty string' );
44 my @columns = Koha::Patrons->columns;
45 my $borrowernumber_exists = grep { /^borrowernumber$/ } @columns;
46 is( $borrowernumber_exists, 1, 'Koha::Objects->columns should return the table columns' );
48 subtest 'find' => sub {
50 my $patron = $builder->build({source => 'Borrower'});
51 my $patron_object = Koha::Patrons->find( $patron->{borrowernumber} );
52 is( $patron_object->borrowernumber, $patron->{borrowernumber}, '->find should return the correct object' );
54 eval { my @patrons = Koha::Patrons->find( $patron->{borrowernumber} ); };
55 like( $@, qr|^Cannot use "->find" in list context|, "->find should not be called in list context to avoid side-effects" );
57 # Test sending undef to find; should not generate a warning
58 warning_is { $patron = Koha::Patrons->find( undef ); }
59 "", "Sending undef does not trigger a DBIx warning";
60 warning_is { $patron = Koha::Patrons->find( undef, undef ); }
61 "", "Sending two undefs does not trigger a DBIx warning too";
64 subtest 'update' => sub {
67 $builder->build( { source => 'City', value => { city_country => 'UK' } } );
68 $builder->build( { source => 'City', value => { city_country => 'UK' } } );
69 $builder->build( { source => 'City', value => { city_country => 'UK' } } );
70 $builder->build( { source => 'City', value => { city_country => 'France' } } );
71 $builder->build( { source => 'City', value => { city_country => 'France' } } );
72 $builder->build( { source => 'City', value => { city_country => 'Germany' } } );
73 Koha::Cities->search( { city_country => 'UK' } )->update( { city_country => 'EU' } );
74 is( Koha::Cities->search( { city_country => 'EU' } )->count, 3, 'Koha::Objects->update should have updated the 3 rows' );
75 is( Koha::Cities->search( { city_country => 'UK' } )->count, 0, 'Koha::Objects->update should have updated the 3 rows' );
78 subtest 'reset' => sub {
81 my $patrons = Koha::Patrons->search;
82 my $first_borrowernumber = $patrons->next->borrowernumber;
83 my $second_borrowernumber = $patrons->next->borrowernumber;
84 is( ref( $patrons->reset ), 'Koha::Patrons', 'Koha::Objects->reset should allow chaining' );
85 is( ref( $patrons->reset->next ), 'Koha::Patron', 'Koha::Objects->reset should allow chaining' );
86 is( $patrons->reset->next->borrowernumber, $first_borrowernumber, 'Koha::Objects->reset should work as expected');
89 subtest 'delete' => sub {
92 my $patron_1 = $builder->build({source => 'Borrower'});
93 my $patron_2 = $builder->build({source => 'Borrower'});
94 is( Koha::Patrons->search({ -or => { borrowernumber => [ $patron_1->{borrowernumber}, $patron_2->{borrowernumber}]}})->delete, 2, '');
95 is( Koha::Patrons->search({ -or => { borrowernumber => [ $patron_1->{borrowernumber}, $patron_2->{borrowernumber}]}})->count, 0, '');
98 subtest 'new' => sub {
100 my $a_cat_code = 'A_CAT_CODE';
101 my $patron_category = Koha::Patron::Category->new( { categorycode => $a_cat_code } )->store;
102 is( Koha::Patron::Categories->find($a_cat_code)->category_type, 'A', 'Koha::Object->new should set the default value' );
103 Koha::Patron::Categories->find($a_cat_code)->delete;
104 $patron_category = Koha::Patron::Category->new( { categorycode => $a_cat_code, category_type => undef } )->store;
105 is( Koha::Patron::Categories->find($a_cat_code)->category_type, 'A', 'Koha::Object->new should set the default value even if the argument exists but is not defined' );
106 Koha::Patron::Categories->find($a_cat_code)->delete;
109 subtest 'find' => sub {
112 # check find on a single PK
113 my $patron = $builder->build({ source => 'Borrower' });
114 is( Koha::Patrons->find($patron->{borrowernumber})->surname,
115 $patron->{surname}, "Checking an arbitrary patron column after find"
117 # check find with unique column
118 my $obj = Koha::Patrons->find($patron->{cardnumber}, { key => 'cardnumber' });
119 is( $obj->borrowernumber, $patron->{borrowernumber},
120 'Find with unique column and key specified' );
121 # check find with an additional where clause in the attrs hash
122 # we do not expect to find something now
123 is( Koha::Patrons->find(
124 $patron->{borrowernumber},
125 { where => { surname => { '!=', $patron->{surname} }}},
126 ), undef, 'Additional where clause in find call' );
128 # check find with a composite FK
129 my $rule = $builder->build({ source => 'Issuingrule' });
130 my @pk = ( $rule->{branchcode}, $rule->{categorycode}, $rule->{itemtype} );
131 is( ref(Koha::IssuingRules->find(@pk)), "Koha::IssuingRule",
132 'Find returned a Koha object for composite primary key' );
134 is( Koha::Patrons->find(), undef, 'Find returns undef if no params passed' );
137 subtest 'search_related' => sub {
139 my $builder = t::lib::TestBuilder->new;
140 my $patron_1 = $builder->build( { source => 'Borrower' } );
141 my $patron_2 = $builder->build( { source => 'Borrower' } );
142 my $libraries = Koha::Patrons->search( { -or => { borrowernumber => [ $patron_1->{borrowernumber}, $patron_2->{borrowernumber} ] } } )->search_related('branchcode');
143 is( ref( $libraries ), 'Koha::Libraries', 'Koha::Objects->search_related should return an instanciated Koha::Objects-based object' );
144 is( $libraries->count, 2, 'Koha::Objects->search_related should work as expected' );
145 is( $libraries->next->branchcode, $patron_1->{branchcode}, 'Koha::Objects->search_related should work as expected' );
146 is( $libraries->next->branchcode, $patron_2->{branchcode}, 'Koha::Objects->search_related should work as expected' );
148 my @libraries = Koha::Patrons->search( { -or => { borrowernumber => [ $patron_1->{borrowernumber}, $patron_2->{borrowernumber} ] } } )->search_related('branchcode');
149 is( ref( $libraries[0] ), 'Koha::Library', 'Koha::Objects->search_related should return a list of Koha::Object-based objects' );
150 is( scalar(@libraries), 2, 'Koha::Objects->search_related should work as expected' );
151 is( $libraries[0]->branchcode, $patron_1->{branchcode}, 'Koha::Objects->search_related should work as expected' );
152 is( $libraries[1]->branchcode, $patron_2->{branchcode}, 'Koha::Objects->search_related should work as expected' );
155 subtest 'single' => sub {
157 my $builder = t::lib::TestBuilder->new;
158 my $patron_1 = $builder->build( { source => 'Borrower' } );
159 my $patron_2 = $builder->build( { source => 'Borrower' } );
160 my $patron = Koha::Patrons->search({}, { rows => 1 })->single;
161 is(ref($patron), 'Koha::Patron', 'Koha::Objects->single returns a single Koha::Patron object.');
162 warning_like { Koha::Patrons->search->single } qr/SQL that returns multiple rows/,
163 "Warning is presented if single is used for a result with multiple rows.";
166 subtest 'last' => sub {
168 my $builder = t::lib::TestBuilder->new;
169 my $patron_1 = $builder->build( { source => 'Borrower' } );
170 my $patron_2 = $builder->build( { source => 'Borrower' } );
171 my $last_patron = Koha::Patrons->search->last;
172 is( $last_patron->borrowernumber, $patron_2->{borrowernumber}, '->last should return the last inserted patron' );
173 $last_patron = Koha::Patrons->search({ borrowernumber => $patron_1->{borrowernumber} })->last;
174 is( $last_patron->borrowernumber, $patron_1->{borrowernumber}, '->last should work even if there is only 1 result' );
175 $last_patron = Koha::Patrons->search({ surname => 'should_not_exist' })->last;
176 is( $last_patron, undef, '->last should return undef if search does not return any results' );
179 subtest 'get_column' => sub {
181 my @cities = Koha::Cities->search;
182 my @city_names = map { $_->city_name } @cities;
183 is_deeply( [ Koha::Cities->search->get_column('city_name') ], \@city_names, 'Koha::Objects->get_column should be allowed' );
186 subtest 'Exceptions' => sub {
189 my $patron_borrowernumber = $builder->build({ source => 'Borrower' })->{ borrowernumber };
190 my $patron = Koha::Patrons->find( $patron_borrowernumber );
194 $patron->blah('blah');
196 ok( $_->isa('Koha::Exceptions::Object::MethodNotCoveredByTests'),
197 'Calling a non-covered method should raise a Koha::Exceptions::Object::MethodNotCoveredByTests exception' );
198 is( $_->message, 'The method Koha::Patron->blah is not covered by tests!', 'The message raised should contain the package and the method' );
202 $patron->set({ blah => 'blah' });
204 ok( $_->isa('Koha::Exceptions::Object::PropertyNotFound'),
205 'Setting a non-existent property should raise a Koha::Exceptions::Object::PropertyNotFound exception' );
210 Koha::Patrons->search->not_covered_yet;
212 ok( $_->isa('Koha::Exceptions::Object::MethodNotCoveredByTests'),
213 'Calling a non-covered method should raise a Koha::Exceptions::Object::MethodNotCoveredByTests exception' );
214 is( $_->message, 'The method Koha::Patrons->not_covered_yet is not covered by tests!', 'The message raised should contain the package and the method' );
218 Koha::Patrons->not_covered_yet;
220 ok( $_->isa('Koha::Exceptions::Object::MethodNotCoveredByTests'),
221 'Calling a non-covered method should raise a Koha::Exceptions::Object::MethodNotCoveredByTests exception' );
222 is( $_->message, 'The method Koha::Patrons->not_covered_yet is not covered by tests!', 'The message raised should contain the package and the method' );
226 $schema->storage->txn_rollback;
228 subtest '->is_paged and ->pager tests' => sub {
232 $schema->storage->txn_begin;
234 # Delete existing patrons
235 Koha::Checkouts->delete;
236 Koha::Patrons->delete;
239 $builder->build_object({ class => 'Koha::Patrons' });
242 # Non-paginated search
243 my $patrons = Koha::Patrons->search();
244 is( $patrons->count, 10, 'Search returns all patrons' );
245 ok( !$patrons->is_paged, 'Search is not paged' );
248 $patrons = Koha::Patrons->search( undef, { 'page' => 1, 'rows' => 3 } );
249 is( $patrons->count, 3, 'Search returns only one page, 3 patrons' );
250 ok( $patrons->is_paged, 'Search is paged' );
251 my $pager = $patrons->pager;
252 is( ref($patrons->pager), 'DBIx::Class::ResultSet::Pager',
253 'Koha::Objects->pager returns a valid DBIx::Class object' );
255 $schema->storage->txn_rollback;
258 subtest '->search() tests' => sub {
262 $schema->storage->txn_begin;
264 my $count = Koha::Patrons->search->count;
268 $builder->build_object({ class => 'Koha::Patrons' });
271 my $patrons = Koha::Patrons->search();
272 is( ref($patrons), 'Koha::Patrons', 'search in scalar context returns the Koha::Object-based type' );
273 my @patrons = Koha::Patrons->search();
274 is( scalar @patrons, $count + 10, 'search in list context returns a list of objects' );
277 is( ref($patrons[$i]), 'Koha::Patron', 'Objects in the list have the singular type' );
281 $schema->storage->txn_rollback;
284 subtest "to_api() tests" => sub {
288 $schema->storage->txn_begin;
290 my $city_1 = $builder->build_object( { class => 'Koha::Cities' } );
291 my $city_2 = $builder->build_object( { class => 'Koha::Cities' } );
293 my $cities = Koha::Cities->search(
295 cityid => [ $city_1->cityid, $city_2->cityid ]
297 { -orderby => { -desc => 'cityid' } }
300 is( $cities->count, 2, 'Count is correct' );
301 my $cities_api = $cities->to_api;
302 is( ref( $cities_api ), 'ARRAY', 'to_api returns an array' );
303 is_deeply( $cities_api->[0], $city_1->to_api, 'to_api returns the individual objects with ->to_api' );
304 is_deeply( $cities_api->[1], $city_2->to_api, 'to_api returns the individual objects with ->to_api' );
306 $schema->storage->txn_rollback;
309 subtest "TO_JSON() tests" => sub {
313 $schema->storage->txn_begin;
315 my $city_1 = $builder->build_object( { class => 'Koha::Cities' } );
316 my $city_2 = $builder->build_object( { class => 'Koha::Cities' } );
318 my $cities = Koha::Cities->search(
320 cityid => [ $city_1->cityid, $city_2->cityid ]
322 { -orderby => { -desc => 'cityid' } }
325 is( $cities->count, 2, 'Count is correct' );
326 my $cities_json = $cities->TO_JSON;
327 is( ref($cities_json), 'ARRAY', 'to_api returns an array' );
328 is_deeply( $cities_json->[0], $city_1->TO_JSON, 'TO_JSON returns the individual objects with ->TO_JSON' );
329 is_deeply( $cities_json->[1], $city_2->TO_JSON,'TO_JSON returns the individual objects with ->TO_JSON' );
331 $schema->storage->txn_rollback;
334 # Koha::Object[s] must behave the same as DBIx::Class
335 subtest 'Return same values as DBIx::Class' => sub {
338 subtest 'Delete' => sub {
341 $schema->storage->txn_begin;
343 subtest 'Simple Koha::Objects - Koha::Cities' => sub {
346 subtest 'Koha::Object->delete' => sub {
350 my ( $r_us, $e_us, $r_them, $e_them );
352 # CASE 1 - Delete an existing object
353 my $c = Koha::City->new( { city_name => 'city4test' } )->store;
354 try { $r_us = $c->delete; } catch { $e_us = $_ };
355 $c = $schema->resultset('City')->new( { city_name => 'city4test_2' } )->update_or_insert;
356 try { $r_them = $c->delete; } catch { $e_them = $_ };
357 ok( ref($r_us) && ref($r_them),
358 'Successful delete should return the object ' );
359 ok( !defined $e_us && !defined $e_them,
360 'Successful delete should not raise an exception' );
361 is( ref($r_us), 'Koha::City', 'Successful delete should return our Koha::Obect based object' );
363 # CASE 2 - Delete an object that is not in storage
364 try { $r_us = $r_us->delete; } catch { $e_us = $_ };
365 try { $r_them = $r_them->delete; } catch { $e_them = $_ };
367 defined $e_us && defined $e_them,
368 'Delete an object that is not in storage should raise an exception'
370 is( ref($e_us), 'DBIx::Class::Exception' )
371 ; # FIXME This needs adjustement, we want to throw a Koha::Exception
375 subtest 'Koha::Objects->delete' => sub {
379 my ( $r_us, $e_us, $r_them, $e_them );
381 # CASE 1 - Delete existing objects
382 my $city_1 = $builder->build_object({ class => 'Koha::Cities' });
383 my $city_2 = $builder->build_object({ class => 'Koha::Cities' });
384 my $city_3 = $builder->build_object({ class => 'Koha::Cities' });
385 my $cities = Koha::Cities->search(
397 try { $r_us = $cities->delete; } catch { $e_us = $_ };
399 $city_1 = $builder->build_object({ class => 'Koha::Cities' });
400 $city_2 = $builder->build_object({ class => 'Koha::Cities' });
401 $city_3 = $builder->build_object({ class => 'Koha::Cities' });
402 $cities = $schema->resultset('City')->search(
414 try { $r_them = $cities->delete; } catch { $e_them = $_ };
416 ok( $r_us == 3 && $r_them == 3 );
417 ok (!defined($e_us) && !defined($e_them));
419 # CASE 2 - One of the object is not in storage
420 $city_1 = $builder->build_object({ class => 'Koha::Cities' });
421 $city_2 = $builder->build_object({ class => 'Koha::Cities' });
422 $city_3 = $builder->build_object({ class => 'Koha::Cities' });
423 $cities = Koha::Cities->search(
435 $city_2->delete; # We delete one of the object
436 try { $r_us = $cities->delete; } catch { $e_us = $_ };
438 $city_1 = $builder->build_object({ class => 'Koha::Cities' });
439 $city_2 = $builder->build_object({ class => 'Koha::Cities' });
440 $city_3 = $builder->build_object({ class => 'Koha::Cities' });
441 $cities = $schema->resultset('City')->search(
453 $city_2->delete; # We delete one of the object
454 try { $r_them = $cities->delete; } catch { $e_them = $_ };
456 ok( $r_us == 2 && $r_them == 2 );
457 ok (!defined($e_us) && !defined($e_them));
461 subtest 'Overwritten Koha::Objects->delete - Koha::Patrons' => sub {
465 subtest 'Koha::Object->delete' => sub {
469 my ( $r_us, $e_us, $r_them, $e_them );
471 # CASE 1 - Delete an existing patron
472 my $patron = $builder->build_object({ class => 'Koha::Patrons' });
473 my $patron_data = $patron->unblessed;
476 $patron = Koha::Patron->new( $patron_data )->store;
477 try {$r_us = $patron->delete;} catch { $e_us = $_ };
478 $patron = $schema->resultset('Borrower')->new( $patron_data )->update_or_insert;
479 try {$r_them = $patron->delete;} catch { $e_them = $_ };
480 ok( ref($r_us) && ref($r_them),
481 'Successful delete should return the patron object' );
482 ok( !defined $e_us && !defined $e_them,
483 'Successful delete should not raise an exception' );
484 is( ref($r_us), 'Koha::Patron',
485 'Successful delete should return our Koha::Obect based object' );
487 # CASE 2 - Delete a patron that is not in storage
488 try { $r_us = $r_us->delete; } catch { $e_us = $_ };
489 try { $r_them = $r_them->delete; } catch { $e_them = $_ };
491 defined $e_us && defined $e_them,
492 'Delete a patron that is not in storage should raise an exception'
494 is( ref($e_us), 'DBIx::Class::Exception' )
495 ; # FIXME This needs adjustement, we want to throw a Koha::Exception
497 # CASE 3 - Delete a patron that cannot be deleted (as a checkout)
498 $patron = Koha::Patron->new($patron_data)->store;
499 $builder->build_object(
501 class => 'Koha::Checkouts',
502 value => { borrowernumber => $patron->borrowernumber }
505 try { $r_us = $r_us->delete; } catch { $e_us = $_ };
506 $patron = $schema->resultset('Borrower')->find( $patron->borrowernumber );
507 try { $r_them = $r_them->delete; } catch { $e_them = $_ };
509 defined $e_us && defined $e_them,
510 'Delete a patron that cannot be deleted should raise an exception'
512 is( ref($e_us), 'DBIx::Class::Exception' )
513 ; # FIXME This needs adjustement, we want to throw a Koha::Exception
516 subtest 'Koha::Objects->delete' => sub {
520 my ( $r_us, $e_us, $r_them, $e_them );
522 # CASE 1 - Delete existing objects
523 my $patron_1 = $builder->build_object({ class => 'Koha::Patrons' });
524 my $patron_2 = $builder->build_object({ class => 'Koha::Patrons' });
525 my $patron_3 = $builder->build_object({ class => 'Koha::Patrons' });
526 my $patrons = Koha::Patrons->search(
530 $patron_1->borrowernumber,
531 $patron_2->borrowernumber,
532 $patron_3->borrowernumber
538 try { $r_us = $patrons->delete; } catch { $e_us = $_ };
540 $patron_1 = $builder->build_object({ class => 'Koha::Patrons' });
541 $patron_2 = $builder->build_object({ class => 'Koha::Patrons' });
542 $patron_3 = $builder->build_object({ class => 'Koha::Patrons' });
543 $patrons = $schema->resultset('Borrower')->search(
547 $patron_1->borrowernumber,
548 $patron_2->borrowernumber,
549 $patron_3->borrowernumber
555 try { $r_them = $patrons->delete; } catch { $e_them = $_ };
557 ok( $r_us == 3 && $r_them == 3, '->delete should return the number of deleted patrons' );
558 ok (!defined($e_us) && !defined($e_them), '->delete should not raise exception if everything went well');
560 # CASE 2 - One of the patrons is not in storage
561 undef $_ for $r_us, $e_us, $r_them, $e_them;
562 $patron_1 = $builder->build_object({ class => 'Koha::Patrons' });
563 $patron_2 = $builder->build_object({ class => 'Koha::Patrons' });
564 $patron_3 = $builder->build_object({ class => 'Koha::Patrons' });
565 $patrons = Koha::Patrons->search(
569 $patron_1->borrowernumber,
570 $patron_2->borrowernumber,
571 $patron_3->borrowernumber
577 $patron_2->delete; # We delete one of the patron
578 try { $r_us = $patrons->delete; } catch { $e_us = $_ };
580 $patron_1 = $builder->build_object({ class => 'Koha::Patrons' });
581 $patron_2 = $builder->build_object({ class => 'Koha::Patrons' });
582 $patron_3 = $builder->build_object({ class => 'Koha::Patrons' });
583 $patrons = $schema->resultset('Borrower')->search(
587 $patron_1->borrowernumber,
588 $patron_2->borrowernumber,
589 $patron_3->borrowernumber
595 $patron_2->delete; # We delete one of the patron
596 try { $r_them = $patrons->delete; } catch { $e_them = $_ };
598 ok( $r_us == 2 && $r_them == 2, 'Delete patrons with one that was not in storage should delete the patrons' );
599 ok (!defined($e_us) && !defined($e_them), 'no exception should be raised if at least one patron was not in storage');
601 # CASE 3 - Delete a set of patrons with one that that cannot be deleted (as a checkout)
602 undef $_ for $r_us, $e_us, $r_them, $e_them;
603 $patron_1 = $builder->build_object({ class => 'Koha::Patrons' });
604 $patron_2 = $builder->build_object({ class => 'Koha::Patrons' });
605 $patron_3 = $builder->build_object({ class => 'Koha::Patrons' });
606 $patrons = Koha::Patrons->search(
610 $patron_1->borrowernumber,
611 $patron_2->borrowernumber,
612 $patron_3->borrowernumber
618 # Adding a checkout to patron_2
619 $builder->build_object(
621 class => 'Koha::Checkouts',
622 value => { borrowernumber => $patron_2->borrowernumber }
627 try { $r_us = $patrons->delete; } catch { $e_us = $_ };
629 qr{DBD::mysql::st execute failed: Cannot delete or update a parent row: a foreign key constraint fails},
630 "Foreign key constraint DBI error should be logged";
631 my $not_deleted_us = $patron_1->in_storage + $patron_2->in_storage + $patron_3->in_storage;
633 $patron_1 = $builder->build_object({ class => 'Koha::Patrons' });
634 $patron_2 = $builder->build_object({ class => 'Koha::Patrons' });
635 $patron_3 = $builder->build_object({ class => 'Koha::Patrons' });
636 $patrons = $schema->resultset('Borrower')->search(
640 $patron_1->borrowernumber,
641 $patron_2->borrowernumber,
642 $patron_3->borrowernumber
648 # Adding a checkout to patron_2
649 $builder->build_object(
651 class => 'Koha::Checkouts',
652 value => { borrowernumber => $patron_2->borrowernumber }
657 try { $r_them = $patrons->delete; } catch { $e_them = $_ };
659 qr{DBD::mysql::st execute failed: Cannot delete or update a parent row: a foreign key constraint fails},
660 "Foreign key constraint DBI error should be logged";
662 my $not_deleted_them = $patron_1->in_storage + $patron_2->in_storage + $patron_3->in_storage;
664 defined $e_us && defined $e_them,
665 'Delete patrons with one that cannot be deleted should raise an exception'
667 is( ref($e_us), 'DBIx::Class::Exception' )
668 ; # FIXME This needs adjustement, we want to throw a Koha::Exception
670 ok($not_deleted_us == 3 && $not_deleted_them == 3, 'If one patron cannot be deleted, none should have been deleted');
674 $schema->storage->txn_rollback;
679 subtest "attributes_from_api() tests" => sub {
683 $schema->storage->txn_begin;
685 my $cities_rs = Koha::Cities->new;
686 my $city = Koha::City->new;
688 my $api_attributes = {
694 $cities_rs->attributes_from_api($api_attributes),
695 $city->attributes_from_api($api_attributes)
698 $schema->storage->txn_rollback;