LP#1755502 Fix hold user test user/gmcharlt/lp1755502_signoff
authorDan Wells <dbw2@calvin.edu>
Thu, 22 Mar 2018 14:31:58 +0000 (10:31 -0400)
committerGalen Charlton <gmc@equinoxinitiative.org>
Thu, 22 Mar 2018 17:52:56 +0000 (13:52 -0400)
commit02002bc3f23044e580b84b13ccdeaec1edb6e23c
treec67cdf35ea2705677d2f1ff1aecbc19e63a7d6ea
parent667bc1a5a3433b4cd936cd9d362668d23e6ad3b3
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