Bug 11338: add unit tests for DelItemCheck
authorGalen Charlton <gmc@esilibrary.com>
Wed, 25 Dec 2013 17:07:53 +0000 (17:07 +0000)
committerFridolin SOMERS <fridolin.somers@biblibre.com>
Fri, 3 Jan 2014 09:08:08 +0000 (10:08 +0100)
commitc65a5d73a7678cb4757566181d566e903a264aee
treef28a49cc577df32d1c843b570fb178c0af06ad2b
parent71b9096034cbc68e28d2db7c389ae341bd96e323
Bug 11338: add unit tests for DelItemCheck

This patch adds unit tests for two parts of DelItemCheck: checking
if the item is on loan, and checking if it is waiting on the hold
shelf.

To test:

[1] Verify that prove -v t/db_dependent/Circulation/IsItemIssued.t
    is successful.
[2] Verify that prove -v t/db_dependent/Reserves.t is *not*
    successful -- as it turns out, there was a latent bug where items
    waiting on the hold shelf or in transit to fill a hold could still
    be deleted without any warning.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
(cherry picked from commit 18888359b6744a1e65e34649c927e7c146f628f8)
Signed-off-by: Fridolin SOMERS <fridolin.somers@biblibre.com>
t/db_dependent/Circulation/IsItemIssued.t
t/db_dependent/Reserves.t