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)
committerKyle M Hall <kyle@bywatersolutions.com>
Mon, 5 Dec 2016 15:32:57 +0000 (15:32 +0000)
commit15ed13845ff1c54605434a8f3100a72534af2351
tree3ee02a75fc7bf3f97222679171e8ae30ff70ec99
parentc275cf490fae608d9eec17d5ed55e5cd3f73c350
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>
t/db_dependent/CourseReserves.t