Bug 20854: (QA follow-up) Improve readability
authorTomas Cohen Arazi <tomascohen@theke.io>
Tue, 16 Mar 2021 19:50:34 +0000 (16:50 -0300)
committerFridolin Somers <fridolin.somers@biblibre.com>
Thu, 29 Apr 2021 11:57:51 +0000 (13:57 +0200)
commit9b3538678ee758632a5d945e3c8d3e5870465a67
tree4333d05d05b61e2b34f001e5b06c2356a97c1bb3
parent225517a0f3dfed730c1afe38e8a262f8fe02d28e
Bug 20854: (QA follow-up) Improve readability

The logout_cas method does too little tasks. In this case, I consider we
should not split it into a separate method. As the CAS implementation
lacked tests, it made sense to do it like this, so it is testable. But
now we have higher-level tests for logout_cas, we can bake the behavior
inside of it, and it is testable.

Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
(cherry picked from commit 7c3c066714ce568e507348f1e3a9a6b7db0c4815)
Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
C4/Auth_with_cas.pm