Bug 25623: Make sure oauth.t rolls back the db
authorTomas Cohen Arazi <tomascohen@theke.io>
Fri, 29 May 2020 17:41:25 +0000 (14:41 -0300)
committerLucas Gass <lucas@bywatersolutions.com>
Wed, 17 Jun 2020 22:55:22 +0000 (22:55 +0000)
commit0734017ab62e05ee4b365ac4546dc68acefa86ca
tree5ae25a7a1e5e17d1a34a9450ea655116482ae102
parentc6519eb540d615358bb0d55730be2a744a4c751d
Bug 25623: Make sure oauth.t rolls back the db

This patch adds a missing transaction wrapping one of the subtests. This
is probably an oversight when refactoring this file.

To test:
1. Open the patron categories page
2. Run:
   $ kshell
  k$ prove t/db_dependent/api/v1/oauth.t
=> SUCCESS: Tests pass
3. Reload the patron categories page
=> FAIL: There's random data in there
4. Apply this patch and repeat 2
5. Reload the patron categories page
=> SUCCESS: The random data from the previous tests is still there, but
no new random data
=> SUCCESS: Tests still pass
6. Sign off :-D

Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Signed-off-by: Julian Maurice <julian.maurice@biblibre.com>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
(cherry picked from commit f898cef4f4b5a9f424f968a58b96898f26f80035)

Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
t/db_dependent/api/v1/oauth.t