Attachment 102457 Details for Bug 24964 – Bug 24964: [19.11.x] Add tests

[patch] Bug 24964: [19.11.x] Add tests

Bug-24964-1911x-Add-tests.patch (text/plain), 4.50 KB, created by Jonathan Druart on 2020-04-06 14:56:06 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jonathan Druart

Created: 2020-04-06 14:56:06 UTC

Size: 4.50 KB

patch

obsolete

>From 2a5885bf7feeaed154b540b43322aa917a1b5477 Mon Sep 17 00:00:00 2001
>From: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
>Date: Wed, 25 Mar 2020 11:20:20 +0100
>Subject: [PATCH] Bug 24964: [19.11.x] Add tests
>
>Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
>
>Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
>---
> t/db_dependent/Utils/Datatables_Members.t | 68 +++++++++++++++++++++--
> 1 file changed, 62 insertions(+), 6 deletions(-)
>
>diff --git a/t/db_dependent/Utils/Datatables_Members.t b/t/db_dependent/Utils/Datatables_Members.t
>index d378f8fe97..7f1363946f 100644
>--- a/t/db_dependent/Utils/Datatables_Members.t
>+++ b/t/db_dependent/Utils/Datatables_Members.t
>@@ -17,7 +17,7 @@
> 
> use Modern::Perl;
> 
>-use Test::More tests => 50;
>+use Test::More tests => 51;
> 
> use C4::Context;
> use C4::Members;
>@@ -57,7 +57,8 @@ my $john_doe = $builder->build({
>             surname      => 'Doe',
>             branchcode   => $branchcode,
>             dateofbirth  => '1983-03-01',
>-            userid       => 'john.doe'
>+            userid       => 'john.doe',
>+            flags        => 0,
>         },
> });
> 
>@@ -69,7 +70,8 @@ my $john_smith = $builder->build({
>             surname      => 'Smith',
>             branchcode   => $branchcode,
>             dateofbirth  => '1982-02-01',
>-            userid       => 'john.smith'
>+            userid       => 'john.smith',
>+            flags        => 0,
>         },
> });
> 
>@@ -81,7 +83,8 @@ my $jane_doe = $builder->build({
>             surname      => 'Doe',
>             branchcode   => $branchcode,
>             dateofbirth  => '1983-03-01',
>-            userid       => 'jane.doe'
>+            userid       => 'jane.doe',
>+            flags        => 0,
>         },
> });
> my $jeanpaul_dupont = $builder->build({
>@@ -92,7 +95,8 @@ my $jeanpaul_dupont = $builder->build({
>             surname      => 'Dupont',
>             branchcode   => $branchcode,
>             dateofbirth  => '1982-02-01',
>-            userid       => 'jeanpaul.dupont'
>+            userid       => 'jeanpaul.dupont',
>+            flags        => 0,
>         },
> });
> my $dupont_brown = $builder->build({
>@@ -103,7 +107,8 @@ my $dupont_brown = $builder->build({
>             surname      => 'Brown',
>             branchcode   => $branchcode,
>             dateofbirth  => '1979-01-01',
>-            userid       => 'dupont.brown'
>+            userid       => 'dupont.brown',
>+            flags        => 0,
>         },
> });
> 
>@@ -461,5 +466,56 @@ subtest 'ExtendedPatronAttributes' => sub {
>         "'Dupont' is contained in 2 surnames and a patron attribute. Patron attribute one should not be displayed if searching in specific fields (Bug 18094)");
> };
> 
>+subtest 'Search with permissions' => sub {
>+    plan tests => 2;
>+
>+    my $superlibrarian = $builder->build_object(
>+        {
>+            class => 'Koha::Patrons',
>+            value => { branchcode => $branchcode, flags => 1 }
>+        }
>+    );
>+    my $librarian_with_full_permission = $builder->build_object(
>+        {
>+            class => 'Koha::Patrons',
>+            value => { branchcode => $branchcode, flags => 4100 }
>+        }
>+    );    # 4100 = 4096 (2^12 suggestions) + 4 (2^2 catalogue)
>+    my $librarian_with_subpermission = $builder->build_object(
>+        { class => 'Koha::Patrons', value => { branchcode => $branchcode } } );
>+    C4::Context->dbh->do(
>+        q|INSERT INTO user_permissions(borrowernumber, module_bit, code) VALUES(?,?,?)|,
>+        undef,
>+        $librarian_with_subpermission->borrowernumber,
>+        12,
>+        'suggestions_manage'
>+    );
>+
>+    my $search_results = C4::Utils::DataTables::Members::search(
>+        {
>+            searchmember     => "",
>+            searchfieldstype => 'standard',
>+            searchtype       => 'contain',
>+            branchcode       => $branchcode,
>+            has_permission   => {
>+                permission    => 'suggestions',
>+                subpermission => 'suggestions_manage'
>+            },
>+            dt_params => { iDisplayLength => 3, iDisplayStart => 0 },
>+        }
>+    );
>+    is( $search_results->{iTotalDisplayRecords},
>+        3, "We find 3 patrons with suggestions_manage permission" );
>+    is_deeply(
>+        [ map { $_->{borrowernumber} } @{ $search_results->{patrons} } ],
>+        [
>+            $superlibrarian->borrowernumber,
>+            $librarian_with_full_permission->borrowernumber,
>+            $librarian_with_subpermission->borrowernumber
>+        ],
>+        'We got the 3 patrons we expected'
>+    );
>+};
>+
> # End
> $schema->storage->txn_rollback;
>-- 
>2.20.1

Actions: View | Diff | Splinter Review

Attachments on bug 24964: 101668 | 101669 | 101686 | 101702 | 101813 | 101814 | 101815 | 101820 | 101821 | 101822 | 101823 | 102434 | 102435 | 102457 | 102458 | 102459 | 102460 | 102461