Bug 17716: (followup) Remove dep on existing data and tidy
authorTomas Cohen Arazi <tomascohen@theke.io>
Fri, 2 Dec 2016 18:32:13 +0000 (15:32 -0300)
committerKatrin Fischer <katrin.fischer.83@web.de>
Mon, 5 Dec 2016 20:25:07 +0000 (21:25 +0100)
commit4b2dfed916567b13ddc406695858963ebf2aae6b
tree28c9667d0d9289101d4fc315be2846685a80ae2d
parent1ee1b7bbc46b92e655a69d248176401580ccdacd
Bug 17716: (followup) Remove dep on existing data and tidy

This patch removes the requirement for this tests for the DB to include
at least 10 borrowers to pass. Borrowers are now created on each run
using t::lib::TestBuilder and a loop.

Bonus: some tiny changes to tidy the file.

To test:
- Run:
  $ prove t/db_dependent/CourseReserves.t
SUCCESS => Tests pass with and without the patch.
- Sign off :-D

Signed-off-by: Mark Tompsett <mtompset@hotmail.com>
Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
t/db_dependent/CourseReserves.t