From b79befe267c171e1ffc4c6303a697e187769d10a Mon Sep 17 00:00:00 2001 From: Arthur Suzuki Date: Fri, 13 Jan 2023 14:56:30 +0100 Subject: [PATCH] Revert "Bug 32208: Extend Auth.t" This reverts commit 5529e5a0ca43ea46e59a51bd77022b089242c795. Signed-off-by: Arthur Suzuki --- t/db_dependent/Auth.t | 29 +++-------------------------- 1 file changed, 3 insertions(+), 26 deletions(-) diff --git a/t/db_dependent/Auth.t b/t/db_dependent/Auth.t index f9df4679ac..9fe22cbf72 100755 --- a/t/db_dependent/Auth.t +++ b/t/db_dependent/Auth.t @@ -109,8 +109,7 @@ subtest 'checkauth() tests' => sub { }; subtest 'While still logged in, relogin with another user' => sub { - plan tests => 6; - + plan tests => 4; my $patron = $builder->build_object({ class => 'Koha::Patrons', value => {} }); my $patron2 = $builder->build_object({ class => 'Koha::Patrons', value => {} }); # Create 'former' session @@ -136,11 +135,12 @@ subtest 'checkauth() tests' => sub { $cgi->param( -name => 'userid', -value => $patron2->userid ); $cgi->param( -name => 'password', -value => $password ); $cgi->param( -name => 'koha_login_context', -value => 1 ); - my ( @return, $stdout ); + my @return; { local *STDOUT; local %ENV; $ENV{REMOTE_ADDR} = '1.2.3.4'; + my $stdout; open STDOUT, '>', \$stdout; @return = C4::Auth::checkauth( $cgi, 0, {} ); close STDOUT; @@ -149,29 +149,6 @@ subtest 'checkauth() tests' => sub { is( $return[0], $patron2->userid, 'Login of patron2 approved' ); isnt( $return[2], $sessionID, 'Did not return previous session ID' ); ok( $return[2], 'New session ID not empty' ); - - # Similar situation: Relogin with former session of $patron, new user $patron2 has no permissions - $patron2->flags(undef)->store; - $session->param( 'number', $patron->id ); - $session->param( 'id', $patron->userid ); - $session->param( 'interface', 'intranet' ); - $session->flush; - $sessionID = $session->id; - C4::Context->_new_userenv($sessionID); - $cgi->param( -name => 'userid', -value => $patron2->userid ); - $cgi->param( -name => 'password', -value => $password ); - $cgi->param( -name => 'koha_login_context', -value => 1 ); - { - local *STDOUT; - local %ENV; - $ENV{REMOTE_ADDR} = '1.2.3.4'; - $stdout = q{}; - open STDOUT, '>', \$stdout; - @return = C4::Auth::checkauth( $cgi, 0, { catalogue => 1 }, 'intranet' ); # patron2 has no catalogue perm - close STDOUT; - } - like( $stdout, qr/You do not have permission to access this page/, 'No permission response' ); - is( @return, 0, 'checkauth returned failure' ); }; C4::Context->_new_userenv; # For next tests -- 2.39.5