Bug 29234: Further clean Z3950 Tests
[koha-ffzg.git] / t / db_dependent / api / v1 / illrequests.t
old mode 100644 (file)
new mode 100755 (executable)
index bc990ee..31c220c
@@ -2,45 +2,43 @@
 
 # This file is part of Koha.
 #
-# 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 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.
+# 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, write to the Free Software Foundation, Inc.,
-# 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
+# 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 => 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 {
 
-    plan tests => 20;
+    plan tests => 30;
 
     # Mock ILLBackend (as object)
     my $backend = Test::MockObject->new;
@@ -70,18 +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  = $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(
@@ -90,7 +95,8 @@ subtest 'list() tests' => sub {
             value => {
                 backend        => 'Mock',
                 branchcode     => $library->branchcode,
-                borrowernumber => $patron->borrowernumber
+                borrowernumber => $patron_1->borrowernumber,
+                status         => 'STATUS1',
             }
         }
     );
@@ -102,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->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(
@@ -128,7 +130,8 @@ subtest 'list() tests' => sub {
             value => {
                 backend        => 'Mock',
                 branchcode     => $library->branchcode,
-                borrowernumber => $patron->borrowernumber
+                borrowernumber => $patron_2->borrowernumber,
+                status         => 'STATUS2',
             }
         }
     );
@@ -140,20 +143,33 @@ 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
+    $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' );
+    $t->get_ok( "//$userid:$password@/api/v1/illrequests" )
+      ->status_is(200)
+      ->json_is( [ $req2_formatted ] );
+
+    t::lib::Mocks::mock_preference( 'ILLHiddenRequestStatuses', 'STATUS2' );
+    $t->get_ok( "//$userid:$password@/api/v1/illrequests" )
+      ->status_is(200)
+      ->json_is( [ $req_formatted ] );
+
     $schema->storage->txn_rollback;
 };
 
@@ -176,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;