Bug 22132: Unit tests
authorTomas Cohen Arazi <tomascohen@theke.io>
Tue, 15 Jan 2019 16:23:09 +0000 (13:23 -0300)
committerMartin Renvoize <martin.renvoize@ptfs-europe.com>
Fri, 1 Feb 2019 15:42:56 +0000 (15:42 +0000)
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>

Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
(cherry picked from commit 103670e726b9938aa1484b7486cba94adf7b69a9)
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>

t/db_dependent/api/v1/auth_basic.t [new file with mode: 0644]

diff --git a/t/db_dependent/api/v1/auth_basic.t b/t/db_dependent/api/v1/auth_basic.t
new file mode 100644 (file)
index 0000000..d689678
--- /dev/null
@@ -0,0 +1,107 @@
+#!/usr/bin/env perl
+
+# 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 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.
+
+use Modern::Perl;
+
+use Test::More tests => 2;
+use Test::Mojo;
+
+use t::lib::TestBuilder;
+use t::lib::Mocks;
+
+use MIME::Base64;
+
+my $schema  = Koha::Database->new->schema;
+my $builder = t::lib::TestBuilder->new;
+
+my $t = Test::Mojo->new('Koha::REST::V1');
+my $tx;
+
+subtest 'success tests' => sub {
+
+    plan tests => 5;
+
+    $schema->storage->txn_begin;
+
+    t::lib::Mocks::mock_preference( 'RESTBasicAuth', 1 );
+
+    my $password = 'AbcdEFG123';
+
+    my $patron = $builder->build_object(
+        { class => 'Koha::Patrons', value => { userid => 'tomasito', flags => 2**4 } } );
+    $patron->set_password($password);
+
+    my $credentials = encode_base64( $patron->userid . ':' . $password );
+
+    $tx = $t->ua->build_tx( GET => "/api/v1/patrons" );
+    $tx->req->headers->authorization("Basic $credentials");
+    $t->request_ok($tx)->status_is( 200, 'Successful authentication and permissions check' );
+
+    $patron->flags(undef)->store;
+
+    $tx = $t->ua->build_tx( GET => "/api/v1/patrons" );
+    $tx->req->headers->authorization("Basic $credentials");
+    $t->request_ok($tx)->status_is( 403, 'Successful authentication and not enough permissions' )
+        ->json_is(
+        '/error' => 'Authorization failure. Missing required permission(s).',
+        'Error message returned'
+        );
+
+    $schema->storage->txn_rollback;
+};
+
+subtest 'failure tests' => sub {
+
+    plan tests => 8;
+
+    $schema->storage->txn_begin;
+
+    my $password     = 'AbcdEFG123';
+    my $bad_password = '123456789';
+
+    t::lib::Mocks::mock_preference( 'RESTBasicAuth', 1 );
+
+    my $patron = $builder->build_object(
+        { class => 'Koha::Patrons', value => { userid => 'tomasito', flags => 2**4 } } );
+    $patron->set_password($password);
+
+    $tx = $t->ua->build_tx( GET => "/api/v1/patrons" );
+    $tx->req->headers->authorization("Basic ");
+    $t->request_ok($tx)->status_is( 401, 'No credentials passed' );
+
+    $patron->flags(undef)->store;
+
+    my $credentials = encode_base64( $patron->userid . ':' . $bad_password );
+
+    $tx = $t->ua->build_tx( GET => "/api/v1/patrons" );
+    $tx->req->headers->authorization("Basic $credentials");
+    $t->request_ok($tx)->status_is( 403, 'Successful authentication and not enough permissions' )
+        ->json_is( '/error' => 'Invalid password', 'Error message returned' );
+
+
+    t::lib::Mocks::mock_preference( 'RESTBasicAuth', 0 );
+
+    $tx = $t->ua->build_tx( GET => "/api/v1/patrons" );
+    $tx->req->headers->authorization("Basic $credentials");
+    $t->request_ok($tx)
+      ->status_is( 401, 'Basic authentication is disabled' )
+      ->json_is( '/error' => 'Basic authentication disabled', 'Expected error message rendered' );
+
+    $schema->storage->txn_rollback;
+};
+
+1;