LP#1755502 Fix hold user test
authorDan Wells <dbw2@calvin.edu>
Thu, 22 Mar 2018 14:31:58 +0000 (10:31 -0400)
committerDan Wells <dbw2@calvin.edu>
Thu, 22 Mar 2018 18:14:18 +0000 (14:14 -0400)
commit80d9f5faa04e87f5b5e90b85c0f8c3ab0d3ed30d
tree0622a4786bdacd7cd155510c5d2b54ee4b13f2aa
parent2ffbacfc992eb0d49f5845e3c24ce0c185e93cb3
LP#1755502 Fix hold user test

Now that we are fleshing this field, we need to dig a little deeper
for the 'id' to test if we are the same user.

To test
-------
[1] Set up a patron with a hold request and with a item
    on the hold shelf that fills the hold.
[2] Check out the item to patron. Note that you
    will get a ITEM_ON_HOLDS_SHELF alert, even though the
    item is on hold for that patron. Cancel the alert.
[3] Apply the patch and repeat step 2. This time, the checkout
    will proceed.
[4] Set up another patron with a hold request and an item
    on the hold shelf, then checkout it at to a _different_ patron.
    Verify that the ITEM_ON_HOLDS_SHELF alert appears.

This patch also fixes test failures caught by the
19-lp1306666-abort-transit-copy-status.t live test.

Signed-off-by: Dan Wells <dbw2@calvin.edu>
Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>
Open-ILS/src/perlmods/lib/OpenILS/Application/Circ/Circulate.pm