Bug 20942: Unit tests for /patrons/{patron_id}/account
authorTomas Cohen Arazi <tomascohen@theke.io>
Wed, 13 Jun 2018 17:02:30 +0000 (14:02 -0300)
committerNick Clemens <nick@bywatersolutions.com>
Wed, 18 Jul 2018 16:49:26 +0000 (16:49 +0000)
This patch adds tests for the /patrons/{patron_id}/account endpoint.
To test:

- Run:
  $ kshell
 k$ prove t/db_dependent/api/v1/patrons_accounts.t
=> FAIL: Tests should fail because the endpoint is not implemented.

Signed-off-by: Josef Moravec <josef.moravec@gmail.com>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
t/db_dependent/api/v1/patrons_accounts.t [new file with mode: 0644]

diff --git a/t/db_dependent/api/v1/patrons_accounts.t b/t/db_dependent/api/v1/patrons_accounts.t
new file mode 100644 (file)
index 0000000..dc770f4
--- /dev/null
@@ -0,0 +1,143 @@
+#!/usr/bin/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 => 1;
+
+use Test::Mojo;
+use Test::Warn;
+
+use t::lib::TestBuilder;
+use t::lib::Mocks;
+
+use C4::Accounts qw(manualinvoice);
+use C4::Auth;
+use Koha::Account::Line;
+
+my $schema  = Koha::Database->new->schema;
+my $builder = t::lib::TestBuilder->new;
+
+# FIXME: sessionStorage defaults to mysql, but it seems to break transaction handling
+# this affects the other REST api tests
+t::lib::Mocks::mock_preference( 'SessionStorage', 'tmp' );
+
+my $remote_address = '127.0.0.1';
+my $t              = Test::Mojo->new('Koha::REST::V1');
+
+subtest 'get_balance() tests' => sub {
+
+    plan tests => 9;
+
+    $schema->storage->txn_begin;
+
+    my ( $patron_id, $session_id ) = create_user_and_session({ authorized => 0 });
+    my $patron = Koha::Patrons->find($patron_id);
+
+    my $tx = $t->ua->build_tx(GET => "/api/v1/patrons/$patron_id/account");
+    $tx->req->cookies({ name => 'CGISESSID', value => $session_id });
+    $tx->req->env({ REMOTE_ADDR => '127.0.0.1' });
+    $t->request_ok($tx)
+      ->status_is(200)
+      ->json_is( { balance => 0.00 } );
+
+    my $account_line_1 = Koha::Account::Line->new(
+        {
+            borrowernumber    => $patron->borrowernumber,
+            date              => \'NOW()',
+            amount            => 50,
+            description       => "A description",
+            accounttype       => "N", # New card
+            amountoutstanding => 50,
+            manager_id        => $patron->borrowernumber,
+        }
+    )->store();
+    $account_line_1->discard_changes;
+
+    my $account_line_2 = Koha::Account::Line->new(
+        {
+            borrowernumber    => $patron->borrowernumber,
+            date              => \'NOW()',
+            amount            => 50.01,
+            description       => "A description",
+            accounttype       => "N", # New card
+            amountoutstanding => 50.01,
+            manager_id        => $patron->borrowernumber,
+        }
+    )->store();
+    $account_line_2->discard_changes;
+
+    $tx = $t->ua->build_tx( GET => "/api/v1/patrons/$patron_id/account" );
+    $tx->req->cookies( { name => 'CGISESSID', value => $session_id } );
+    $tx->req->env( { REMOTE_ADDR => '127.0.0.1' } );
+    $t->request_ok($tx)->status_is(200)->json_is(
+        {   balance           => 100.01,
+            outstanding_lines => [
+                Koha::REST::V1::Patrons::Account::_to_api( $account_line_1->TO_JSON ),
+                Koha::REST::V1::Patrons::Account::_to_api( $account_line_2->TO_JSON )
+
+            ]
+        }
+    );
+
+    Koha::Account->new({ patron_id => $patron_id })->pay(
+        {   amount       => 100.01,
+            note         => 'He paid!',
+            description  => 'Finally!',
+            library_id   => $patron->branchcode,
+            account_type => 'Pay',
+            offset_type  => 'Payment'
+        }
+    );
+
+    $tx = $t->ua->build_tx( GET => "/api/v1/patrons/$patron_id/account" );
+    $tx->req->cookies( { name => 'CGISESSID', value => $session_id } );
+    $tx->req->env( { REMOTE_ADDR => '127.0.0.1' } );
+    $t->request_ok($tx)->status_is(200)->json_is( { balance => 0 } );
+
+    $schema->storage->txn_rollback;
+};
+
+sub create_user_and_session {
+
+    my $args  = shift;
+    my $flags = ( $args->{authorized} ) ? 16 : 0;
+
+    my $user = $builder->build(
+        {
+            source => 'Borrower',
+            value  => {
+                flags => $flags,
+                gonenoaddress => 0,
+                lost => 0,
+                email => 'nobody@example.com',
+                emailpro => 'nobody@example.com',
+                B_email => 'nobody@example.com'
+            }
+        }
+    );
+
+    # 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 );
+}