From d1f8d5bbd2d221457787262b6724049294d1ec56 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Tom=C3=A1s=20Cohen=20Arazi?= Date: Wed, 30 Sep 2026 11:41:21 -0300 Subject: [PATCH] Bug 42719: (QA follow-up) Adjust tests MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit This patch makes the tests reflect the design decision made on the bug report. The author for the original 'No CGISESSID then error' patch signed off on this so I'm confident this is the right thing to do. Signed-off-by: Tomás Cohen Arazi Signed-off-by: Pedro Amorim --- t/db_dependent/api/v1/idp.t | 10 +--------- 1 file changed, 1 insertion(+), 9 deletions(-) diff --git a/t/db_dependent/api/v1/idp.t b/t/db_dependent/api/v1/idp.t index 9fd115b8ca..58074ed201 100755 --- a/t/db_dependent/api/v1/idp.t +++ b/t/db_dependent/api/v1/idp.t @@ -286,7 +286,7 @@ subtest 'domain endpoint tests' => sub { }; subtest 'oauth login tests' => sub { - plan tests => 6; + plan tests => 4; $schema->storage->txn_begin; @@ -322,14 +322,6 @@ subtest 'oauth login tests' => sub { $t->get_ok("/api/v1/public/oauth/login/oauth_test/opac")->status_is(302); - $tx = $t->ua->build_tx( GET => "/api/v1/public/oauth/login/oauth_test/opac" ); - $tx->req->cookies( { name => 'NOTCGISESSID', value => $session_id } ); - $tx->req->env( { REMOTE_ADDR => $remote_address } ); - $t->request_ok($tx)->header_like( - Location => qr/\?auth_error=No%20user%20session%20found/, - "Redirect to error when no CGISESSID provided" - ); - $schema->storage->txn_rollback; };