View | Details | Raw Unified | Return to bug 28206
Collapse All | Expand All

(-)a/t/db_dependent/Holds.t (-4 / +20 lines)
Lines 326-332 ok( Link Here
326
}
326
}
327
327
328
subtest 'AllowHoldsOnDamagedItems' => sub {
328
subtest 'AllowHoldsOnDamagedItems' => sub {
329
    plan tests => 5;
329
    plan tests => 7;
330
330
331
    my $library    = $builder->build_object( { class => 'Koha::Libraries' } );
331
    my $library    = $builder->build_object( { class => 'Koha::Libraries' } );
332
    my $biblio     = $builder->build_sample_biblio;
332
    my $biblio     = $builder->build_sample_biblio;
Lines 340-352 subtest 'AllowHoldsOnDamagedItems' => sub { Link Here
340
        }
340
        }
341
    );
341
    );
342
342
343
    my $itemnumber = $builder->build_sample_item(
343
    my $item = $builder->build_sample_item(
344
        {
344
        {
345
            library      => $library->branchcode,
345
            library      => $library->branchcode,
346
            biblionumber => $biblio->biblionumber,
346
            biblionumber => $biblio->biblionumber,
347
            damaged      => 1
347
            damaged      => 1
348
        }
348
        }
349
    )->itemnumber;
349
    );
350
    my $itemnumber = $item->itemnumber;
350
    t::lib::Mocks::mock_preference( 'AllowHoldsOnDamagedItems', 1 );
351
    t::lib::Mocks::mock_preference( 'AllowHoldsOnDamagedItems', 1 );
351
    is(
352
    is(
352
        CanItemBeReserved( $patron->borrowernumber, $itemnumber )->{status},
353
        CanItemBeReserved( $patron->borrowernumber, $itemnumber )->{status},
Lines 381-386 subtest 'AllowHoldsOnDamagedItems' => sub { Link Here
381
        !defined( ( CheckReserves($itemnumber) )[1] ), "Hold cannot be trapped for damaged item with AllowHoldsOnDamagedItems disabled"
382
        !defined( ( CheckReserves($itemnumber) )[1] ), "Hold cannot be trapped for damaged item with AllowHoldsOnDamagedItems disabled"
382
    );
383
    );
383
384
385
386
387
    t::lib::Mocks::mock_preference( 'AllowHoldsOnDamagedItems', 0 );
388
    $hold = Koha::Hold->new(
389
        {
390
            borrowernumber => $patron->borrowernumber,
391
            itemnumber     => $itemnumber,
392
            biblionumber   => $biblio->biblionumber,
393
        }
394
    )->store();
395
396
    t::lib::Mocks::mock_userenv({ branchcode => $library->branchcode });
397
    my $checkout = AddIssue( $patron->unblessed, $item->barcode );
398
    ok( $checkout, 'AddIssue should have checkout the item' );
399
    is( Koha::Holds->find($hold->reserve_id), undef, 'AddIssue should have filled the hold' );
400
384
};
401
};
385
402
386
# Items that are not for loan, but holdable should not be trapped until they are available for loan
403
# Items that are not for loan, but holdable should not be trapped until they are available for loan
387
- 

Return to bug 28206