gitea
/
Koha-clms
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Bug 20402: Fix oauth.t 

GET /patrons now requires { "borrowers": 1 } instead of
{ "borrowers": "edit_borrowers" }

Signed-off-by: Josef Moravec <josef.moravec@gmail.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
This commit is contained in:
Julian Maurice 2018-04-12 09:17:43 +02:00 committed by Jonathan Druart
parent 43a4b3c22c
commit ccc034195e
1 changed files with 5 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
t/db_dependent/api/v1/oauth.t
View File

@ -21,6 +21,7 @@ use Test::More tests => 1;
use Test::Mojo;
use Koha::Database;
use Koha::Patrons;
use t::lib::Mocks;
use t::lib::TestBuilder;
@ -34,13 +35,14 @@ subtest '/oauth/token tests' => sub {
$schema->storage->txn_begin;
my $patron = $builder->build({
my $borrower = $builder->build({
source => 'Borrower',
value => {
surname => 'Test OAuth',
flags => 0,
},
});
my $patron = Koha::Patrons->find($borrower->{borrowernumber});
# Missing parameter grant_type
$t->post_ok('/api/v1/oauth/token')
@ -60,7 +62,7 @@ subtest '/oauth/token tests' => sub {
t::lib::Mocks::mock_config('api_client', {
'client_id' => $client_id,
'client_secret' => $client_secret,
patron_id => $patron->{borrowernumber},
patron_id => $patron->borrowernumber,
});
my $formData = {
@ -85,14 +87,7 @@ subtest '/oauth/token tests' => sub {
$t->request_ok($tx)->status_is(403);
# With access token and permissions, it returns 200
$builder->build({
source => 'UserPermission',
value => {
borrowernumber => $patron->{borrowernumber},
module_bit => 4, # borrowers
code => 'edit_borrowers',
},
});
$patron->flags(2**4)->store;
$tx = $t->ua->build_tx(GET => '/api/v1/patrons');
$tx->req->headers->authorization("Bearer $access_token");
$t->request_ok($tx)->status_is(200);