Bugzilla – Attachment 116508 Details for
Bug 27587
Use Basic auth on API tests
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 27587: Use Basic auth on API tests - illrequests.t
Bug-27587-Use-Basic-auth-on-API-tests---illrequest.patch (text/plain), 8.15 KB, created by
Jonathan Druart
on 2021-02-08 14:30:33 UTC
(
hide
)
Description:
Bug 27587: Use Basic auth on API tests - illrequests.t
Filename:
MIME Type:
Creator:
Jonathan Druart
Created:
2021-02-08 14:30:33 UTC
Size:
8.15 KB
patch
obsolete
>From de5dd7ea82f66558be11024ec7c72838e2cb56f7 Mon Sep 17 00:00:00 2001 >From: Tomas Cohen Arazi <tomascohen@theke.io> >Date: Tue, 2 Feb 2021 10:09:47 -0300 >Subject: [PATCH] Bug 27587: Use Basic auth on API tests - illrequests.t > >Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> >Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> >--- > t/db_dependent/api/v1/illrequests.t | 122 ++++++++++------------------ > 1 file changed, 42 insertions(+), 80 deletions(-) > >diff --git a/t/db_dependent/api/v1/illrequests.t b/t/db_dependent/api/v1/illrequests.t >index 269cb4d7e1..31c220c724 100755 >--- a/t/db_dependent/api/v1/illrequests.t >+++ b/t/db_dependent/api/v1/illrequests.t >@@ -18,25 +18,23 @@ > use Modern::Perl; > > use Test::More tests => 1; >+ > use Test::MockModule; > use Test::MockObject; > use Test::Mojo; >-use Test::Warn; > > use t::lib::TestBuilder; > use t::lib::Mocks; > >-use C4::Auth; > use Koha::Illrequests; > use Koha::DateUtils qw( format_sqldatetime ); > > my $schema = Koha::Database->new->schema; > my $builder = t::lib::TestBuilder->new; > >-t::lib::Mocks::mock_preference( 'SessionStorage', 'tmp' ); >+t::lib::Mocks::mock_preference( 'RESTBasicAuth', 1 ); > >-my $remote_address = '127.0.0.1'; >-my $t = Test::Mojo->new('Koha::REST::V1'); >+my $t = Test::Mojo->new('Koha::REST::V1'); > > subtest 'list() tests' => sub { > >@@ -70,19 +68,25 @@ subtest 'list() tests' => sub { > $schema->storage->txn_begin; > > Koha::Illrequests->search->delete; >- # ill => 22 (userflags.sql) >- my ( $borrowernumber, $session_id ) = create_user_and_session({ authorized => 22 }); >+ >+ # create an authorized user >+ my $patron = $builder->build_object({ >+ class => 'Koha::Patrons', >+ value => { flags => 2 ** 22 } # 22 => ill >+ }); >+ my $password = 'thePassword123'; >+ $patron->set_password({ password => $password, skip_validation => 1 }); >+ my $userid = $patron->userid; > > ## Authorized user tests > # No requests, so empty array should be returned >- my $tx = $t->ua->build_tx( GET => '/api/v1/illrequests' ); >- $tx->req->cookies( { name => 'CGISESSID', value => $session_id } ); >- $tx->req->env( { REMOTE_ADDR => $remote_address } ); >- $t->request_ok($tx)->status_is(200)->json_is( [] ); >+ $t->get_ok( "//$userid:$password@/api/v1/illrequests" ) >+ ->status_is(200) >+ ->json_is( [] ); > >- my $library = $builder->build_object( { class => 'Koha::Libraries' } ); >- my $patron_1 = $builder->build_object( { class => 'Koha::Patrons' } ); >- my $patron_2 = $builder->build_object( { class => 'Koha::Patrons' } ); >+ my $library = $builder->build_object( { class => 'Koha::Libraries' } ); >+ my $patron_1 = $builder->build_object( { class => 'Koha::Patrons' } ); >+ my $patron_2 = $builder->build_object( { class => 'Koha::Patrons' } ); > > # Create an ILL request > my $illrequest = $builder->build_object( >@@ -104,24 +108,20 @@ subtest 'list() tests' => sub { > my $req_formatted = add_formatted($response); > > # One illrequest created, should get returned >- $tx = $t->ua->build_tx( GET => '/api/v1/illrequests' ); >- $tx->req->cookies( { name => 'CGISESSID', value => $session_id } ); >- $tx->req->env( { REMOTE_ADDR => $remote_address } ); >- $t->request_ok($tx)->status_is(200)->json_is( [ $req_formatted ] ); >+ $t->get_ok( "//$userid:$password@/api/v1/illrequests" ) >+ ->status_is(200) >+ ->json_is( [ $req_formatted ] ); > > # One illrequest created, returned with augmented data >- $tx = $t->ua->build_tx( GET => >- '/api/v1/illrequests?embed=patron,library,capabilities,metadata,requested_partners' ); >- $tx->req->cookies( { name => 'CGISESSID', value => $session_id } ); >- $tx->req->env( { REMOTE_ADDR => $remote_address } ); >- $t->request_ok($tx)->status_is(200) >- ->json_has( '/0/patron', 'patron embedded' ) >- ->json_is( '/0/patron/patron_id', $patron_1->borrowernumber, 'The right patron is embeded') >- ->json_has( '/0/requested_partners', 'requested_partners embedded' ) >- ->json_has( '/0/capabilities', 'capabilities embedded' ) >- ->json_has( '/0/library', 'library embedded' ) >- ->json_has( '/0/metadata', 'metadata embedded' ) >- ->json_hasnt( '/1', 'Only one request was created' ); >+ $t->get_ok( "//$userid:$password@/api/v1/illrequests?embed=patron,library,capabilities,metadata,requested_partners" ) >+ ->status_is(200) >+ ->json_has( '/0/patron', 'patron embedded' ) >+ ->json_is( '/0/patron/patron_id', $patron_1->borrowernumber, 'The right patron is embeded') >+ ->json_has( '/0/requested_partners', 'requested_partners embedded' ) >+ ->json_has( '/0/capabilities', 'capabilities embedded' ) >+ ->json_has( '/0/library', 'library embedded' ) >+ ->json_has( '/0/metadata', 'metadata embedded' ) >+ ->json_hasnt( '/1', 'Only one request was created' ); > > # Create another ILL request > my $illrequest2 = $builder->build_object( >@@ -143,40 +143,31 @@ subtest 'list() tests' => sub { > my $req2_formatted = add_formatted($response2); > > # Two illrequest created, should get returned >- $tx = $t->ua->build_tx( GET => '/api/v1/illrequests' ); >- $tx->req->cookies( { name => 'CGISESSID', value => $session_id } ); >- $tx->req->env( { REMOTE_ADDR => $remote_address } ); >- $t->request_ok($tx)->status_is(200) >+ $t->get_ok( "//$userid:$password@/api/v1/illrequests" ) >+ ->status_is(200) > ->json_is( [ $req_formatted, $req2_formatted ] ); > > # Warn on unsupported query parameter >- $tx = $t->ua->build_tx( GET => '/api/v1/illrequests?request_blah=blah' ); >- $tx->req->cookies( { name => 'CGISESSID', value => $session_id } ); >- $tx->req->env( { REMOTE_ADDR => $remote_address } ); >- $t->request_ok($tx)->status_is(400)->json_is( >+ $t->get_ok( "//$userid:$password@/api/v1/illrequests?request_blah=blah" ) >+ ->status_is(400) >+ ->json_is( > [{ path => '/query/request_blah', message => 'Malformed query string'}] > ); > > # Test the borrowernumber parameter >- $tx = $t->ua->build_tx( GET => '/api/v1/illrequests?borrowernumber=' . >- $patron_2->borrowernumber ); >- $tx->req->cookies( { name => 'CGISESSID', value => $session_id } ); >- $tx->req->env( { REMOTE_ADDR => $remote_address } ); >- $t->request_ok($tx)->status_is(200)->json_is( [ $response2 ] ); >+ $t->get_ok( "//$userid:$password@/api/v1/illrequests?borrowernumber=" . $patron_2->borrowernumber ) >+ ->status_is(200) >+ ->json_is( [ $response2 ] ); > > # Test the ILLHiddenRequestStatuses syspref > t::lib::Mocks::mock_preference( 'ILLHiddenRequestStatuses', 'STATUS1' ); >- $tx = $t->ua->build_tx( GET => '/api/v1/illrequests' ); >- $tx->req->cookies( { name => 'CGISESSID', value => $session_id } ); >- $tx->req->env( { REMOTE_ADDR => $remote_address } ); >- $t->request_ok($tx)->status_is(200) >+ $t->get_ok( "//$userid:$password@/api/v1/illrequests" ) >+ ->status_is(200) > ->json_is( [ $req2_formatted ] ); > > t::lib::Mocks::mock_preference( 'ILLHiddenRequestStatuses', 'STATUS2' ); >- $tx = $t->ua->build_tx( GET => '/api/v1/illrequests' ); >- $tx->req->cookies( { name => 'CGISESSID', value => $session_id } ); >- $tx->req->env( { REMOTE_ADDR => $remote_address } ); >- $t->request_ok($tx)->status_is(200) >+ $t->get_ok( "//$userid:$password@/api/v1/illrequests" ) >+ ->status_is(200) > ->json_is( [ $req_formatted ] ); > > $schema->storage->txn_rollback; >@@ -201,32 +192,3 @@ sub add_formatted { > } > return $req; > } >- >-sub create_user_and_session { >- >- my $args = shift; >- my $dbh = C4::Context->dbh; >- >- my $flags = ( $args->{authorized} ) ? 2**$args->{authorized} : 0; >- >- my $user = $builder->build( >- { >- source => 'Borrower', >- value => { >- flags => $flags >- } >- } >- ); >- >- # Create a session for the authorized user >- my $session = C4::Auth::get_session(''); >- $session->param( 'number', $user->{borrowernumber} ); >- $session->param( 'id', $user->{userid} ); >- $session->param( 'ip', '127.0.0.1' ); >- $session->param( 'lasttime', time() ); >- $session->flush; >- >- return ( $user->{borrowernumber}, $session->id ); >-} >- >-1; >-- >2.20.1
<b>You cannot view the attachment while viewing its details because your browser does not support IFRAMEs. <a href="attachment.cgi?id=116508">View the attachment on a separate page</a>.</b>
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 27587
:
116211
|
116212
|
116213
|
116214
|
116215
|
116319
|
116320
|
116321
|
116322
|
116323
|
116505
|
116506
|
116507
| 116508 |
116509
|
118506