From 308836db3a9392736ead1f1a6229c1701b90a428 Mon Sep 17 00:00:00 2001 From: Marcel de Rooy Date: Fri, 9 Sep 2022 09:09:48 +0000 Subject: [PATCH] Bug 30588: (QA follow-up) Remove erroneous FIXME Content-Type: text/plain; charset=utf-8 Comment speaks for itself. Look at surrounding code. Signed-off-by: Marcel de Rooy --- t/db_dependent/api/v1/two_factor_auth.t | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/t/db_dependent/api/v1/two_factor_auth.t b/t/db_dependent/api/v1/two_factor_auth.t index 63075a6925..3a890c3d65 100755 --- a/t/db_dependent/api/v1/two_factor_auth.t +++ b/t/db_dependent/api/v1/two_factor_auth.t @@ -176,8 +176,7 @@ subtest 'send_otp_token' => sub { $tx->req->cookies( { name => 'CGISESSID', value => $session->id } ); $tx->req->env( { REMOTE_ADDR => $remote_address } ); - # FIXME This is not correct They are authenticated! - # Patron is not authenticated yet + # Session is still anonymous, no borrowernumber yet: Unauthorized $t->request_ok($tx)->status_is(401); # Patron is partially authenticated (credentials correct) -- 2.20.1