Bug 17941 add tests to cover the optimization of the nested loop
authorFrancesco Rivetti <oha@oha.it>
Mon, 20 Feb 2017 10:02:40 +0000 (11:02 +0100)
committerKyle M Hall <kyle@bywatersolutions.com>
Fri, 3 Mar 2017 16:50:27 +0000 (16:50 +0000)
commitb1249dcc34d22722c23fb3a231b61bcd42f82550
treeb52bddde3cfc26b96002b1f6c16376b0c3e8b4e2
parent08b1ebec75623c84967fd09611cf1fef4442c200
Bug 17941 add tests to cover the optimization of the nested loop

the new code skip testing for more borrowers if the item can't be renewed
this require more tests for the case where 2+ items are reserved.

I tried to add 1 more reserve to the main test suit, but too many other tests
rely on specific holds, and I couldn't get around it.

Instead, I added a subtest that consider the specific simple case and leave the
other test cases exactly has they were designed.

Signed-off-by: Mark Tompsett <mtompset@hotmail.com>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Amended-patch: Just fix 3 indendations

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
t/db_dependent/Circulation.t