Bug 10386: improvements to VirtualShelves.t
authorMarcel de Rooy <m.de.rooy@rijksmuseum.nl>
Mon, 3 Jun 2013 13:27:50 +0000 (15:27 +0200)
committerTomas Cohen Arazi <tomascohen@gmail.com>
Mon, 15 Jul 2013 17:41:03 +0000 (14:41 -0300)
commita97b62379f82a94157916959109c7c1a75582d8a
tree83a9596ccf9e12cfd3e34b53bc3d4be2e12f1d2e
parent509972e3858d3b3c9f915d09a1e10f77cbe105fd
Bug 10386: improvements to VirtualShelves.t

Most important: Does no longer delete all shelves!
Checks if there are ten borrowers for testing. But even works without them :)
When creating or modifying lists, takes name clashes into consideration.

Small change to _CheckShelfName in VirtualShelves module. Making it possible to
check a name for a list whose owner has been set to NULL. Note that a test
like field=? with undef for placeholder will not work in MySql.

Test plan:
How do you test a test? Well, you could run it on various databases..
But for real hacking, you could also add some debug lines.
I tested this by forcing 10 undefs in @borrowernumbers.
And by overwriting the return value of randomname with an existing name.

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
(cherry picked from commit 93e87ca0b61611fc35238bf13eb226f477cf9c4e)
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
C4/VirtualShelves.pm
t/db_dependent/VirtualShelves.t