|
Lines 20-26
Link Here
|
| 20 |
use Modern::Perl; |
20 |
use Modern::Perl; |
| 21 |
|
21 |
|
| 22 |
use Test::NoWarnings; |
22 |
use Test::NoWarnings; |
| 23 |
use Test::More tests => 16; |
23 |
use Test::More tests => 17; |
| 24 |
|
24 |
|
| 25 |
use Test::Exception; |
25 |
use Test::Exception; |
| 26 |
use Test::MockModule; |
26 |
use Test::MockModule; |
|
Lines 1259-1261
subtest 'strings_map() tests' => sub {
Link Here
|
| 1259 |
|
1259 |
|
| 1260 |
$schema->txn_rollback; |
1260 |
$schema->txn_rollback; |
| 1261 |
}; |
1261 |
}; |
| 1262 |
- |
1262 |
|
|
|
1263 |
subtest 'move_hold() tests' => sub { |
| 1264 |
plan tests => 11; |
| 1265 |
$schema->storage->txn_begin; |
| 1266 |
|
| 1267 |
my $patron = Koha::Patron->new( |
| 1268 |
{ |
| 1269 |
surname => 'Luke', |
| 1270 |
categorycode => 'PT', |
| 1271 |
branchcode => 'CPL' |
| 1272 |
} |
| 1273 |
)->store; |
| 1274 |
|
| 1275 |
my $patron_2 = Koha::Patron->new( |
| 1276 |
{ |
| 1277 |
surname => 'Gass', |
| 1278 |
categorycode => 'PT', |
| 1279 |
branchcode => 'CPL' |
| 1280 |
} |
| 1281 |
)->store; |
| 1282 |
|
| 1283 |
my $patron_3 = Koha::Patron->new( |
| 1284 |
{ |
| 1285 |
surname => 'Test', |
| 1286 |
categorycode => 'PT', |
| 1287 |
branchcode => 'CPL' |
| 1288 |
} |
| 1289 |
)->store; |
| 1290 |
|
| 1291 |
my $patron_4 = Koha::Patron->new( |
| 1292 |
{ |
| 1293 |
surname => 'Guy', |
| 1294 |
categorycode => 'PT', |
| 1295 |
branchcode => 'CPL' |
| 1296 |
} |
| 1297 |
)->store; |
| 1298 |
|
| 1299 |
my $biblio1 = $builder->build_sample_biblio(); |
| 1300 |
my $item_1 = $builder->build_sample_item( { biblionumber => $biblio1->biblionumber } ); |
| 1301 |
|
| 1302 |
my $biblio2 = $builder->build_sample_biblio(); |
| 1303 |
my $item_2 = $builder->build_sample_item( { biblionumber => $biblio2->biblionumber } ); |
| 1304 |
|
| 1305 |
my $biblio3 = $builder->build_sample_biblio(); |
| 1306 |
my $item_3 = $builder->build_sample_item( { biblionumber => $biblio3->biblionumber } ); |
| 1307 |
|
| 1308 |
my $biblio4 = $builder->build_sample_biblio(); |
| 1309 |
my $item_4 = $builder->build_sample_item( { biblionumber => $biblio4->biblionumber } ); |
| 1310 |
|
| 1311 |
# Create an item level hold |
| 1312 |
my $hold = Koha::Hold->new( |
| 1313 |
{ |
| 1314 |
borrowernumber => $patron->borrowernumber, |
| 1315 |
biblionumber => $biblio1->biblionumber, |
| 1316 |
itemnumber => $item_1->itemnumber, |
| 1317 |
branchcode => 'CPL', |
| 1318 |
} |
| 1319 |
)->store; |
| 1320 |
|
| 1321 |
# Create a record level hold |
| 1322 |
my $hold_2 = Koha::Hold->new( |
| 1323 |
{ |
| 1324 |
borrowernumber => $patron->borrowernumber, |
| 1325 |
biblionumber => $biblio1->biblionumber, |
| 1326 |
itemnumber => undef, |
| 1327 |
branchcode => 'CPL', |
| 1328 |
} |
| 1329 |
)->store; |
| 1330 |
|
| 1331 |
# Move the hold from item 1 to item 2 |
| 1332 |
my $result = $hold->move_hold( { new_itemnumber => $item_2->itemnumber } ); |
| 1333 |
ok( $result->{success}, 'Hold successfully moved' ); |
| 1334 |
$hold->discard_changes(); |
| 1335 |
is( $hold->biblionumber, $biblio2->biblionumber, 'Biblionumber updated correctly' ); |
| 1336 |
is( $hold->itemnumber, $item_2->itemnumber, 'Itemnumber updated correctly' ); |
| 1337 |
is( $hold->priority, 1, "Hold priority is set to 1" ); |
| 1338 |
|
| 1339 |
my $logs = Koha::ActionLogs->search( |
| 1340 |
{ |
| 1341 |
action => 'MODIFY', |
| 1342 |
module => 'HOLDS', |
| 1343 |
object => $hold->id |
| 1344 |
} |
| 1345 |
); |
| 1346 |
|
| 1347 |
is( $logs->count, 0, 'HoldsLog disabled, no logs added' ); |
| 1348 |
|
| 1349 |
my $result_2 = $hold_2->move_hold( { new_biblionumber => $biblio3->biblionumber } ); |
| 1350 |
ok( $result_2->{success}, 'Hold successfully moved' ); |
| 1351 |
$hold_2->discard_changes(); |
| 1352 |
is( $hold_2->biblionumber, $biblio3->biblionumber, 'Biblionumber updated correctly' ); |
| 1353 |
is( $hold->priority, 1, "Hold priority is set to 1" ); |
| 1354 |
|
| 1355 |
# Create an item level hold |
| 1356 |
my $hold_3 = Koha::Hold->new( |
| 1357 |
{ |
| 1358 |
borrowernumber => $patron_3->borrowernumber, |
| 1359 |
biblionumber => $biblio4->biblionumber, |
| 1360 |
itemnumber => $item_4->itemnumber, |
| 1361 |
branchcode => 'CPL', |
| 1362 |
priority => 1, |
| 1363 |
} |
| 1364 |
)->store; |
| 1365 |
|
| 1366 |
# Create an item level hold |
| 1367 |
my $hold_4 = Koha::Hold->new( |
| 1368 |
{ |
| 1369 |
borrowernumber => $patron_2->borrowernumber, |
| 1370 |
biblionumber => $biblio4->biblionumber, |
| 1371 |
itemnumber => $item_4->itemnumber, |
| 1372 |
branchcode => 'CPL', |
| 1373 |
priority => 2, |
| 1374 |
} |
| 1375 |
)->store; |
| 1376 |
|
| 1377 |
# Create an item level hold |
| 1378 |
my $hold_5 = Koha::Hold->new( |
| 1379 |
{ |
| 1380 |
borrowernumber => $patron_4->borrowernumber, |
| 1381 |
biblionumber => $biblio4->biblionumber, |
| 1382 |
itemnumber => $item_4->itemnumber, |
| 1383 |
branchcode => 'CPL', |
| 1384 |
priority => 3, |
| 1385 |
} |
| 1386 |
)->store; |
| 1387 |
|
| 1388 |
#enable HoldsLog |
| 1389 |
t::lib::Mocks::mock_preference( 'HoldsLog', 1 ); |
| 1390 |
|
| 1391 |
my $result_3 = $hold_2->move_hold( { new_biblionumber => $biblio4->biblionumber } ); |
| 1392 |
ok( $result_3->{success}, 'Hold successfully moved' ); |
| 1393 |
$hold_2->discard_changes; |
| 1394 |
is( $hold_2->priority, 4, "Hold priority is set to 4" ); |
| 1395 |
|
| 1396 |
my $logs_2 = Koha::ActionLogs->search( |
| 1397 |
{ |
| 1398 |
action => 'MODIFY', |
| 1399 |
module => 'HOLDS', |
| 1400 |
object => $hold_2->id |
| 1401 |
} |
| 1402 |
); |
| 1403 |
|
| 1404 |
is( $logs_2->count, 1, 'Hold modification was logged' ); |
| 1405 |
|
| 1406 |
#disable HoldsLog |
| 1407 |
t::lib::Mocks::mock_preference( 'HoldsLog', 1 ); |
| 1408 |
|
| 1409 |
$schema->storage->txn_rollback; |
| 1410 |
}; |