Bug 19437: Rearranging tests for CancelExpiredReserves
authorMarcel de Rooy <m.de.rooy@rijksmuseum.nl>
Fri, 6 Oct 2017 08:41:29 +0000 (10:41 +0200)
committerJonathan Druart <jonathan.druart@bugs.koha-community.org>
Mon, 9 Oct 2017 16:47:02 +0000 (13:47 -0300)
commit511b3253576262878c953e7a3fe4a2941686ab1e
tree4730d6b2a9e63ac965e0ad25b314eb5db088997a
parent56500e8e676782155c5717e8db9786af394252d8
Bug 19437: Rearranging tests for CancelExpiredReserves

This patch originates from a QA Follow-up on bug 19260.

The first 19260 patch adds CancelExpiredReserves tests to Reserves.t.
But note that we already have some tests in Holds/CancelReserves.t.

This patch does:
Renames Holds/CancelReserves.t to Reserves/CancelExpiredReserves.t.
Rearranges modules there.
Moves its existing tests into a first subtest.
Moves the new subtest from Reserves.t to CancelExpiredReserves.t.
Replaces $dbh->do('DELETE FROM reserves').
Adds some TestBuilder statements for missing data (by the move): adding
biblio, item, borrower (removing slow AddMember call).

Test plan:
Run Reserves.t and Reserves/CancelExpiredReserves.t.

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Josef Moravec <josef.moravec@gmail.com>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
t/db_dependent/Holds/CancelReserves.t [deleted file]
t/db_dependent/Reserves.t
t/db_dependent/Reserves/CancelExpiredReserves.t [new file with mode: 0644]