Bug 21134: Correctly throw the exception in Koha::Patron::Modification->approve
authorJonathan Druart <jonathan.druart@bugs.koha-community.org>
Tue, 31 Jul 2018 15:43:54 +0000 (12:43 -0300)
committerFridolin Somers <fridolin.somers@biblibre.com>
Fri, 21 Sep 2018 09:46:26 +0000 (11:46 +0200)
commitc110912af3f4ff5f41eea4668d38863a951d82c0
tree111cf6acbd11b57b6305131832f323ba0caedb1c
parentd802bf6722f9ac79b9460edc307c656cd3358763
Bug 21134: Correctly throw the exception in Koha::Patron::Modification->approve

koha_1       |     # Looks like you planned 20 tests but ran 9.
koha_1       |
koha_1       | #   Failed test 'approve tests'
koha_1       | #   at t/db_dependent/Koha/Patron/Modifications.t line
259.
koha_1       | [Something went wrong]# Looks like your test exited with
255 just after 5.
koha_1       | [21:13:47] t/db_dependent/Koha/Patron/Modifications.t
.....................

"Something went wrong" should never been thrown,
Koha::Patron::Modification->approve raises $@, but it's not defined, it
should raise $_ instead

Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
(cherry picked from commit 6e99cfbca7e904b7664163838a334c487c94e462)
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
(cherry picked from commit c6133753ed661902a9ec3cce14190c7d3d683ef0)
Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
Koha/Patron/Modification.pm