https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23177
--- Comment #12 from Nick Clemens <[email protected]> --- Created attachment 92175 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=92175&action=edit Bug 23177: [18.11.x] (QA follow-up) Move three subs from the middle to the top in Circulation.t Removed trailing comma for last sub too. Signed-off-by: Marcel de Rooy <[email protected]> Signed-off-by: Martin Renvoize <[email protected]> Bug 23177: (QA follow-up) Move rollback to the end Signed-off-by: Marcel de Rooy <[email protected]> Signed-off-by: Martin Renvoize <[email protected]> Bug 23177: (QA follow-up) Remove subtest txn_begin and rollback This resolves: DBIx::Class::Storage::txn_rollback(): A txn_rollback in nested transaction is ineffective! (depth 1) Removing unneeded subtest txn's. We should probably add them everywhere or just at global level. Test plan: With all three patches applied, run Circulation.t a few times. Signed-off-by: Marcel de Rooy <[email protected]> Signed-off-by: Martin Renvoize <[email protected]> Bug 23177: (RM follow-up) Further test clarifications This patch makes a few minor improvements to Circulation.t 1) Adds a name to some of the scoped blocks by converting them to subtests. 2) Adds output messages to some tests where they were missing. Signed-off-by: Martin Renvoize <[email protected]> Bug 23177: Rebase fixes -- You are receiving this mail because: You are watching all bug changes. _______________________________________________ Koha-bugs mailing list [email protected] https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
