Bug 22877: (QA follow-up) Clarify intent of tests and remove one.
authorMartin Renvoize <martin.renvoize@ptfs-europe.com>
Thu, 9 May 2019 15:34:09 +0000 (16:34 +0100)
committerFridolin Somers <fridolin.somers@biblibre.com>
Mon, 10 Jun 2019 09:32:01 +0000 (11:32 +0200)
commit76074180ead15621b63ac84cedc43f3df151ca1c
tree21cb8209ce0fdfec481b9d6b37dca39a488f860e
parentb551b502f768317ff01323ba24c4e9bee0a3b74f
Bug 22877: (QA follow-up) Clarify intent of tests and remove one.

This patch embelishes the existing comments slightly and removes a test
which had a FIXME, but was in fact just a duplicate of the test directly
above it.  I beleive it became a duplicate after the subroutine signature
changed at some point recently.  I also move the bug 22877 specific test
into a subtest.

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Liz Rea <wizzyrea@gmail.com>
Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
(cherry picked from commit fa411c2f80d3464c9205f49fc31de492bc265c57)
Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
t/db_dependent/Circulation.t