Tomas Cohen Arazi
aa1049fdd3
Working on bug 31791, I found myself wondering if our current recursive code in C4::Auth::haspermission() would allow checking AND on subpermissions. As it is not documented in the POD or tested, I decided to write some unit tests for it. It turned out it was well supported, so I decided to submit the tests, and a small tweak in the POD to reflect that. To test: 1. Apply this patch 2. Run: $ ktd --shell k$ prove t/db_dependent/Auth/haspermission.t => SUCCESS: Tests pass! The code supports AND on subpermissions! 3. Sign off :-D Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de>
283 lines
10 KiB
Perl
Executable file
283 lines
10 KiB
Perl
Executable file
#!/usr/bin/perl
|
|
|
|
# Tests for C4::Auth::haspermission
|
|
|
|
# This file is part of Koha.
|
|
#
|
|
# Copyright 2016 Rijksmuseum
|
|
#
|
|
# Koha is free software; you can redistribute it and/or modify it
|
|
# under the terms of the GNU General Public License as published by
|
|
# the Free Software Foundation; either version 3 of the License, or
|
|
# (at your option) any later version.
|
|
#
|
|
# Koha is distributed in the hope that it will be useful, but
|
|
# WITHOUT ANY WARRANTY; without even the implied warranty of
|
|
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
|
|
# GNU General Public License for more details.
|
|
#
|
|
# You should have received a copy of the GNU General Public License
|
|
# along with Koha; if not, see <http://www.gnu.org/licenses>.
|
|
|
|
use Modern::Perl;
|
|
use Test::More tests => 5;
|
|
use Test::Exception;
|
|
|
|
use Koha::Database;
|
|
use t::lib::TestBuilder;
|
|
use C4::Auth qw( haspermission );
|
|
|
|
my $schema = Koha::Database->new->schema;
|
|
$schema->storage->txn_begin;
|
|
|
|
# Adding two borrowers and granular permissions for the second borrower
|
|
my $builder = t::lib::TestBuilder->new();
|
|
my $borr1 = $builder->build(
|
|
{
|
|
source => 'Borrower',
|
|
value => {
|
|
surname => 'Superlib',
|
|
flags => 1,
|
|
},
|
|
}
|
|
);
|
|
my $borr2 = $builder->build(
|
|
{
|
|
source => 'Borrower',
|
|
value => {
|
|
surname => 'Bor2',
|
|
flags => 2 + 4 + 2**11, # circulate, catalogue, acquisition
|
|
},
|
|
}
|
|
);
|
|
my $borr3 = $builder->build(
|
|
{
|
|
source => 'Borrower',
|
|
value => {
|
|
surname => 'Bor2',
|
|
flags => 2**13, # top level tools
|
|
},
|
|
}
|
|
);
|
|
$builder->build(
|
|
{
|
|
source => 'UserPermission',
|
|
value => {
|
|
borrowernumber => $borr2->{borrowernumber},
|
|
module_bit => 13, # tools
|
|
code => 'upload_local_cover_images',
|
|
},
|
|
}
|
|
);
|
|
$builder->build(
|
|
{
|
|
source => 'UserPermission',
|
|
value => {
|
|
borrowernumber => $borr2->{borrowernumber},
|
|
module_bit => 13, # tools
|
|
code => 'batch_upload_patron_images',
|
|
},
|
|
}
|
|
);
|
|
|
|
subtest 'undef top level tests' => sub {
|
|
|
|
plan tests => 1;
|
|
|
|
my $pass = haspermission( $borr2->{userid} );
|
|
ok($pass, "let through undef privs");
|
|
|
|
#throws_ok { my $r = haspermission( $borr1->{userid} ); }
|
|
#'Koha::Exceptions::WrongParameter',
|
|
# 'Exception thrown when missing $requiredflags';
|
|
#throws_ok { my $r = haspermission( $borr1->{userid}, undef ); }
|
|
#'Koha::Exceptions::WrongParameter', 'Exception thrown when explicit undef';
|
|
};
|
|
|
|
subtest 'scalar top level tests' => sub {
|
|
|
|
plan tests => 3;
|
|
|
|
# Check top level permission for superlibrarian
|
|
my $r = haspermission( $borr1->{userid}, 'circulate' );
|
|
is( ref($r), 'HASH', 'Superlibrarian/circulate' );
|
|
|
|
# Check specific top level permission(s) for borr2
|
|
$r = haspermission( $borr2->{userid}, 'circulate' );
|
|
is( ref($r), 'HASH', 'Borrower2/circulate' );
|
|
$r = haspermission( $borr2->{userid}, 'updatecharges' );
|
|
is( $r, 0, 'Borrower2/updatecharges should fail' );
|
|
};
|
|
|
|
subtest 'hashref top level AND tests' => sub {
|
|
|
|
plan tests => 16;
|
|
|
|
# Check top level permission for superlibrarian
|
|
my $r =
|
|
haspermission( $borr1->{userid}, { circulate => 1 } );
|
|
is( ref($r), 'HASH', 'Superlibrarian/circulate' );
|
|
|
|
# Check specific top level permission(s) for borr2
|
|
$r = haspermission( $borr2->{userid}, { circulate => 1, catalogue => 1 } );
|
|
is( ref($r), 'HASH', 'Borrower2/circulate' );
|
|
$r = haspermission( $borr2->{userid}, { updatecharges => 1 } );
|
|
is( $r, 0, 'Borrower2/updatecharges should fail' );
|
|
|
|
# Check granular permission with 1: means all subpermissions
|
|
$r = haspermission( $borr1->{userid}, { tools => 1 } );
|
|
is( ref($r), 'HASH', 'Superlibrarian/tools granular all' );
|
|
$r = haspermission( $borr2->{userid}, { tools => 1 } );
|
|
is( $r, 0, 'Borrower2/tools granular all should fail' );
|
|
|
|
# Check granular permission with *: means at least one subpermission
|
|
$r = haspermission( $borr1->{userid}, { tools => '*' } );
|
|
is( ref($r), 'HASH', 'Superlibrarian/tools granular *' );
|
|
$r = haspermission( $borr2->{userid}, { acquisition => '*' } );
|
|
is( ref($r), 'HASH', 'Borrower2/acq granular *' );
|
|
$r = haspermission( $borr2->{userid}, { tools => '*' } );
|
|
is( ref($r), 'HASH', 'Borrower2/tools granular *' );
|
|
$r = haspermission( $borr2->{userid}, { serials => '*' } );
|
|
is( $r, 0, 'Borrower2/serials granular * should fail' );
|
|
|
|
# Check granular permission with one or more specific subperms
|
|
$r = haspermission( $borr1->{userid}, { tools => 'edit_additional_contents' } );
|
|
is( ref($r), 'HASH', 'Superlibrarian/tools edit_additional_contents' );
|
|
$r = haspermission( $borr2->{userid}, { acquisition => 'budget_manage' } );
|
|
is( ref($r), 'HASH', 'Borrower2/acq budget_manage' );
|
|
$r = haspermission( $borr2->{userid},
|
|
{ acquisition => 'budget_manage', tools => 'edit_additional_contents' } );
|
|
is( $r, 0, 'Borrower2 (/acquisition|budget_manage AND /tools|edit_additional_contents) should fail' );
|
|
$r = haspermission(
|
|
$borr2->{userid},
|
|
{
|
|
tools => {
|
|
'upload_local_cover_images' => 1,
|
|
'batch_upload_patron_images' => 1
|
|
},
|
|
}
|
|
);
|
|
is( ref($r), 'HASH', 'Borrower2 (/tools|upload_local_cover_image AND /tools|batch_upload_patron_images) granular' );
|
|
$r = haspermission(
|
|
$borr3->{userid},
|
|
{
|
|
tools => {
|
|
'upload_local_cover_images' => 1,
|
|
'batch_upload_patron_images' => 1
|
|
},
|
|
}
|
|
);
|
|
is( ref($r), 'HASH', 'Borrower3 (/tools|upload_local_cover_image AND /tools|batch_upload_patron_images) granular' );
|
|
$r = haspermission(
|
|
$borr2->{userid},
|
|
{
|
|
tools => {
|
|
'upload_local_cover_images' => 1,
|
|
'edit_additional_contents' => 1
|
|
},
|
|
}
|
|
);
|
|
is( $r, 0, 'Borrower2 (/tools|upload_local_cover_image AND /tools|edit_additional_contents) granular' );
|
|
$r = haspermission(
|
|
$borr2->{userid},
|
|
{
|
|
tools => [ 'upload_local_cover_images', 'edit_additional_contents'],
|
|
}
|
|
);
|
|
is( ref($r), 'HASH', 'Borrower2 (/tools|upload_local_cover_image OR /tools|edit_additional_contents) granular' );
|
|
};
|
|
|
|
subtest 'arrayref top level OR tests' => sub {
|
|
|
|
plan tests => 13;
|
|
|
|
# Check top level permission for superlibrarian
|
|
my $r =
|
|
haspermission( $borr1->{userid}, [ 'circulate', 'editcatalogue' ] );
|
|
is( ref($r), 'HASH', 'Superlibrarian/circulate' );
|
|
|
|
# Check specific top level permission(s) for borr2
|
|
$r = haspermission( $borr2->{userid}, [ 'circulate', 'updatecharges' ] );
|
|
is( ref($r), 'HASH', 'Borrower2/circulate OR Borrower2/updatecharges' );
|
|
$r = haspermission( $borr2->{userid}, ['updatecharges', 'serials' ] );
|
|
is( $r, 0, 'Borrower2/updatecharges OR Borrower2/serials should fail' );
|
|
|
|
# Check granular permission with 1: means all subpermissions
|
|
$r = haspermission( $borr1->{userid}, [ 'tools' ] );
|
|
is( ref($r), 'HASH', 'Superlibrarian/tools granular all' );
|
|
$r = haspermission( $borr2->{userid}, [ 'tools' ] );
|
|
is( $r, 0, 'Borrower2/tools granular all should fail' );
|
|
|
|
# Check granular permission with *: means at least one subpermission
|
|
$r = haspermission( $borr1->{userid}, [ { tools => '*' } ] );
|
|
is( ref($r), 'HASH', 'Superlibrarian/tools granular *' );
|
|
$r = haspermission( $borr2->{userid}, [ { acquisition => '*' } ] );
|
|
is( ref($r), 'HASH', 'Borrower2/acq granular *' );
|
|
$r = haspermission( $borr2->{userid}, [ { tools => '*' } ] );
|
|
is( ref($r), 'HASH', 'Borrower2/tools granular *' );
|
|
$r = haspermission( $borr2->{userid}, [ { serials => '*' } ] );
|
|
is( $r, 0, 'Borrower2/serials granular * should fail' );
|
|
|
|
# Check granular permission with one or more specific subperms
|
|
$r = haspermission( $borr1->{userid}, [ { tools => 'edit_additional_contents' } ] );
|
|
is( ref($r), 'HASH', 'Superlibrarian/tools edit_additional_contents' );
|
|
$r =
|
|
haspermission( $borr2->{userid}, [ { acquisition => 'budget_manage' } ] );
|
|
is( ref($r), 'HASH', 'Borrower2/acq budget_manage' );
|
|
$r = haspermission( $borr2->{userid},
|
|
[ { acquisition => 'budget_manage'}, { tools => 'edit_additional_contents' } ] );
|
|
is( ref($r), 'HASH', 'Borrower2/two granular OR should pass' );
|
|
$r = haspermission(
|
|
$borr2->{userid},
|
|
[
|
|
{ tools => ['upload_local_cover_images'] },
|
|
{ tools => ['edit_additional_contents'] }
|
|
]
|
|
);
|
|
is( ref($r), 'HASH', 'Borrower2/tools granular OR subperms' );
|
|
};
|
|
|
|
$schema->storage->txn_rollback;
|
|
|
|
subtest 'AND on subpermissions' => sub {
|
|
|
|
plan tests => 2;
|
|
|
|
$schema->storage->txn_begin;
|
|
|
|
my $patron = $builder->build_object( { class => 'Koha::Patrons', value => { flags => 0 } } );
|
|
|
|
$builder->build(
|
|
{
|
|
source => 'UserPermission',
|
|
value => {
|
|
borrowernumber => $patron->id,
|
|
module_bit => 9, # editcatalogue
|
|
code => 'edit_catalogue',
|
|
},
|
|
}
|
|
);
|
|
|
|
my $r = haspermission( $patron->userid, { editcatalogue => { edit_catalogue => 1, advanced_editor => 1 } } );
|
|
is( $r, 0, "The user only has 'edit_catalogue' permissions, 0 returned" );
|
|
|
|
$builder->build(
|
|
{
|
|
source => 'UserPermission',
|
|
value => {
|
|
borrowernumber => $patron->id,
|
|
module_bit => 9, # editcatalogue
|
|
code => 'advanced_editor',
|
|
},
|
|
}
|
|
);
|
|
|
|
$r = haspermission( $patron->userid, { editcatalogue => { edit_catalogue => 1, advanced_editor => 1 } } );
|
|
|
|
ok(
|
|
$r->{editcatalogue}->{edit_catalogue} && $r->{editcatalogue}->{advanced_editor},
|
|
"The patron has 'edit_catalogue' and 'advanced_editor', both returned"
|
|
);
|
|
|
|
$schema->storage->txn_rollback;
|
|
};
|