t/db_dependent/Koha/Account.t

   1 #!/usr/bin/perl
   2
   3 # Copyright 2018 Koha Development team
   4 #
   5 # This file is part of Koha
   6 #
   7 # Koha is free software; you can redistribute it and/or modify it
   8 # under the terms of the GNU General Public License as published by
   9 # the Free Software Foundation; either version 3 of the License, or
  10 # (at your option) any later version.
  11 #
  12 # Koha is distributed in the hope that it will be useful, but
  13 # WITHOUT ANY WARRANTY; without even the implied warranty of
  14 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
  15 # GNU General Public License for more details.
  16 #
  17 # You should have received a copy of the GNU General Public License
  18 # along with Koha; if not, see <http://www.gnu.org/licenses>
  19
  20 use Modern::Perl;
  21
  22 use Test::More tests => 8;
  23 use Test::MockModule;
  24 use Test::Exception;
  25
  26 use Koha::Account;
  27 use Koha::Account::Lines;
  28 use Koha::Account::Offsets;
  29
  30
  31 use t::lib::Mocks;
  32 use t::lib::TestBuilder;
  33
  34 my $schema  = Koha::Database->new->schema;
  35 my $builder = t::lib::TestBuilder->new;
  36
  37 subtest 'new' => sub {
  38
  39     plan tests => 2;
  40
  41     $schema->storage->txn_begin;
  42
  43     throws_ok { Koha::Account->new(); } qr/No patron id passed in!/, 'Croaked on bad call to new';
  44
  45     my $patron  = $builder->build_object({ class => 'Koha::Patrons' });
  46     my $account = Koha::Account->new( { patron_id => $patron->borrowernumber } );
  47     is( defined $account, 1, "Account is defined" );
  48
  49     $schema->storage->txn_rollback;
  50 };
  51
  52 subtest 'outstanding_debits() tests' => sub {
  53
  54     plan tests => 22;
  55
  56     $schema->storage->txn_begin;
  57
  58     my $patron  = $builder->build_object({ class => 'Koha::Patrons' });
  59     my $account = $patron->account;
  60
  61     my @generated_lines;
  62     push @generated_lines, Koha::Account::Line->new({ borrowernumber => $patron->id, amount => 1, amountoutstanding => 1 })->store;
  63     push @generated_lines, Koha::Account::Line->new({ borrowernumber => $patron->id, amount => 2, amountoutstanding => 2 })->store;
  64     push @generated_lines, Koha::Account::Line->new({ borrowernumber => $patron->id, amount => 3, amountoutstanding => 3 })->store;
  65     push @generated_lines, Koha::Account::Line->new({ borrowernumber => $patron->id, amount => 4, amountoutstanding => 4 })->store;
  66
  67     my $lines     = $account->outstanding_debits();
  68     my @lines_arr = $account->outstanding_debits();
  69
  70     is( ref($lines), 'Koha::Account::Lines', 'Called in scalar context, outstanding_debits returns a Koha::Account::Lines object' );
  71     is( $lines->total_outstanding, 10, 'Outstandig debits total is correctly calculated' );
  72
  73     my $i = 0;
  74     foreach my $line ( @{ $lines->as_list } ) {
  75         my $fetched_line = Koha::Account::Lines->find( $generated_lines[$i]->id );
  76         is_deeply( $line->unblessed, $fetched_line->unblessed, "Fetched line matches the generated one ($i)" );
  77         is_deeply( $lines_arr[$i]->unblessed, $fetched_line->unblessed, "Fetched line matches the generated one ($i)" );
  78         is( ref($lines_arr[$i]), 'Koha::Account::Line', 'outstanding_debits returns a list of Koha::Account::Line objects in list context' );
  79         $i++;
  80     }
  81     my $patron_2 = $builder->build_object({ class => 'Koha::Patrons' });
  82     Koha::Account::Line->new({ borrowernumber => $patron_2->id, amountoutstanding => -2 })->store;
  83     my $just_one = Koha::Account::Line->new({ borrowernumber => $patron_2->id, amount => 3, amountoutstanding =>  3 })->store;
  84     Koha::Account::Line->new({ borrowernumber => $patron_2->id, amount => -6, amountoutstanding => -6 })->store;
  85     $lines = $patron_2->account->outstanding_debits();
  86     is( $lines->total_outstanding, 3, "Total if some outstanding debits and some credits is only debits" );
  87     is( $lines->count, 1, "With 1 outstanding debits, we get back a Lines object with 1 lines" );
  88     my $the_line = Koha::Account::Lines->find( $just_one->id );
  89     is_deeply( $the_line->unblessed, $lines->next->unblessed, "We get back the one correct line");
  90
  91     my $patron_3 = $builder->build_object({ class => 'Koha::Patrons' });
  92     Koha::Account::Line->new({ borrowernumber => $patron_2->id, amount => -2,   amountoutstanding => -2 })->store;
  93     Koha::Account::Line->new({ borrowernumber => $patron_2->id, amount => -20,  amountoutstanding => -20 })->store;
  94     Koha::Account::Line->new({ borrowernumber => $patron_2->id, amount => -200, amountoutstanding => -200 })->store;
  95     $lines = $patron_3->account->outstanding_debits();
  96     is( $lines->total_outstanding, 0, "Total if no outstanding debits total is 0" );
  97     is( $lines->count, 0, "With 0 outstanding debits, we get back a Lines object with 0 lines" );
  98
  99     my $patron_4  = $builder->build_object({ class => 'Koha::Patrons' });
 100     my $account_4 = $patron_4->account;
 101     $lines = $account_4->outstanding_debits();
 102     is( $lines->total_outstanding, 0, "Total if no outstanding debits is 0" );
 103     is( $lines->count, 0, "With no outstanding debits, we get back a Lines object with 0 lines" );
 104
 105     # create a pathological credit with amountoutstanding > 0 (BZ 14591)
 106     Koha::Account::Line->new({ borrowernumber => $patron_4->id, amount => -3, amountoutstanding => 3 })->store();
 107     $lines = $account_4->outstanding_debits();
 108     is( $lines->count, 0, 'No credits are confused with debits because of the amountoutstanding value' );
 109
 110     $schema->storage->txn_rollback;
 111 };
 112
 113 subtest 'outstanding_credits() tests' => sub {
 114
 115     plan tests => 17;
 116
 117     $schema->storage->txn_begin;
 118
 119     my $patron  = $builder->build_object({ class => 'Koha::Patrons' });
 120     my $account = $patron->account;
 121
 122     my @generated_lines;
 123     push @generated_lines, $account->add_credit({ amount => 1 });
 124     push @generated_lines, $account->add_credit({ amount => 2 });
 125     push @generated_lines, $account->add_credit({ amount => 3 });
 126     push @generated_lines, $account->add_credit({ amount => 4 });
 127
 128     my $lines     = $account->outstanding_credits();
 129     my @lines_arr = $account->outstanding_credits();
 130
 131     is( ref($lines), 'Koha::Account::Lines', 'Called in scalar context, outstanding_credits returns a Koha::Account::Lines object' );
 132     is( $lines->total_outstanding, -10, 'Outstandig credits total is correctly calculated' );
 133
 134     my $i = 0;
 135     foreach my $line ( @{ $lines->as_list } ) {
 136         my $fetched_line = Koha::Account::Lines->find( $generated_lines[$i]->id );
 137         is_deeply( $line->unblessed, $fetched_line->unblessed, "Fetched line matches the generated one ($i)" );
 138         is_deeply( $lines_arr[$i]->unblessed, $fetched_line->unblessed, "Fetched line matches the generated one ($i)" );
 139         is( ref($lines_arr[$i]), 'Koha::Account::Line', 'outstanding_debits returns a list of Koha::Account::Line objects in list context' );
 140         $i++;
 141     }
 142
 143     my $patron_2 = $builder->build_object({ class => 'Koha::Patrons' });
 144     $account  = $patron_2->account;
 145     $lines       = $account->outstanding_credits();
 146     is( $lines->total_outstanding, 0, "Total if no outstanding credits is 0" );
 147     is( $lines->count, 0, "With no outstanding credits, we get back a Lines object with 0 lines" );
 148
 149     # create a pathological debit with amountoutstanding < 0 (BZ 14591)
 150     Koha::Account::Line->new({ borrowernumber => $patron_2->id, amount => 2, amountoutstanding => -3 })->store();
 151     $lines = $account->outstanding_credits();
 152     is( $lines->count, 0, 'No debits are confused with credits because of the amountoutstanding value' );
 153
 154     $schema->storage->txn_rollback;
 155 };
 156
 157 subtest 'add_credit() tests' => sub {
 158
 159     plan tests => 15;
 160
 161     $schema->storage->txn_begin;
 162
 163     # delete logs and statistics
 164     my $action_logs = $schema->resultset('ActionLog')->search()->count;
 165     my $statistics = $schema->resultset('Statistic')->search()->count;
 166
 167     my $patron  = $builder->build_object( { class => 'Koha::Patrons' } );
 168     my $account = Koha::Account->new( { patron_id => $patron->borrowernumber } );
 169
 170     is( $account->balance, 0, 'Patron has no balance' );
 171
 172     # Disable logs
 173     t::lib::Mocks::mock_preference( 'FinesLog', 0 );
 174
 175     my $line_1 = $account->add_credit(
 176         {   amount      => 25,
 177             description => 'Payment of 25',
 178             library_id  => $patron->branchcode,
 179             note        => 'not really important',
 180             type        => 'payment',
 181             user_id     => $patron->id
 182         }
 183     );
 184
 185     is( $account->balance, -25, 'Patron has a balance of -25' );
 186     is( $schema->resultset('ActionLog')->count(), $action_logs + 0, 'No log was added' );
 187     is( $schema->resultset('Statistic')->count(), $statistics + 1, 'Action added to statistics' );
 188     is( $line_1->accounttype, $Koha::Account::account_type_credit->{'payment'}, 'Account type is correctly set' );
 189
 190     # Enable logs
 191     t::lib::Mocks::mock_preference( 'FinesLog', 1 );
 192
 193     my $sip_code = "1";
 194     my $line_2 = $account->add_credit(
 195         {   amount      => 37,
 196             description => 'Payment of 37',
 197             library_id  => $patron->branchcode,
 198             note        => 'not really important',
 199             user_id     => $patron->id,
 200             sip         => $sip_code
 201         }
 202     );
 203
 204     is( $account->balance, -62, 'Patron has a balance of -25' );
 205     is( $schema->resultset('ActionLog')->count(), $action_logs + 1, 'Log was added' );
 206     is( $schema->resultset('Statistic')->count(), $statistics + 2, 'Action added to statistics' );
 207     is( $line_2->accounttype, $Koha::Account::account_type_credit->{'payment'} . $sip_code, 'Account type is correctly set' );
 208
 209     # offsets have the credit_id set to accountlines_id, and debit_id is undef
 210     my $offset_1 = Koha::Account::Offsets->search({ credit_id => $line_1->id })->next;
 211     my $offset_2 = Koha::Account::Offsets->search({ credit_id => $line_2->id })->next;
 212
 213     is( $offset_1->credit_id, $line_1->id, 'No debit_id is set for credits' );
 214     is( $offset_1->debit_id, undef, 'No debit_id is set for credits' );
 215     is( $offset_2->credit_id, $line_2->id, 'No debit_id is set for credits' );
 216     is( $offset_2->debit_id, undef, 'No debit_id is set for credits' );
 217
 218     my $line_3 = $account->add_credit(
 219         {   amount      => 20,
 220             description => 'Manual credit applied',
 221             library_id  => $patron->branchcode,
 222             user_id     => $patron->id,
 223             type        => 'forgiven'
 224         }
 225     );
 226
 227     is( $schema->resultset('ActionLog')->count(), $action_logs + 2, 'Log was added' );
 228     is( $schema->resultset('Statistic')->count(), $statistics + 2, 'No action added to statistics, because of credit type' );
 229
 230     $schema->storage->txn_rollback;
 231 };
 232
 233 subtest 'add_debit() tests' => sub {
 234
 235     plan tests => 13;
 236
 237     $schema->storage->txn_begin;
 238
 239     # delete logs and statistics
 240     my $action_logs = $schema->resultset('ActionLog')->search()->count;
 241     my $statistics  = $schema->resultset('Statistic')->search()->count;
 242
 243     my $patron = $builder->build_object( { class => 'Koha::Patrons' } );
 244     my $account =
 245       Koha::Account->new( { patron_id => $patron->borrowernumber } );
 246
 247     is( $account->balance, 0, 'Patron has no balance' );
 248
 249     throws_ok {
 250     $account->add_debit(
 251         {
 252             amount      => -5,
 253             description => 'amount validation failure',
 254             library_id  => $patron->branchcode,
 255             note        => 'this should fail anyway',
 256             type        => 'rent',
 257             user_id     => $patron->id
 258         }
 259     ); } 'Koha::Exceptions::Account::AmountNotPositive', 'Expected validation exception thrown (amount)';
 260
 261     throws_ok {
 262     $account->add_debit(
 263         {
 264             amount      => 5,
 265             description => 'type validation failure',
 266             library_id  => $patron->branchcode,
 267             note        => 'this should fail anyway',
 268             type        => 'failure',
 269             user_id     => $patron->id
 270         }
 271     ); } 'Koha::Exceptions::Account::UnrecognisedType', 'Expected validation exception thrown (type)';
 272
 273     # Disable logs
 274     t::lib::Mocks::mock_preference( 'FinesLog', 0 );
 275
 276     my $line_1 = $account->add_debit(
 277         {
 278             amount      => 25,
 279             description => 'Rental charge of 25',
 280             library_id  => $patron->branchcode,
 281             note        => 'not really important',
 282             type        => 'rent',
 283             user_id     => $patron->id
 284         }
 285     );
 286
 287     is( $account->balance, 25, 'Patron has a balance of 25' );
 288     is(
 289         $schema->resultset('ActionLog')->count(),
 290         $action_logs + 0,
 291         'No log was added'
 292     );
 293     is(
 294         $line_1->accounttype,
 295         $Koha::Account::account_type_debit->{'rent'},
 296         'Account type is correctly set'
 297     );
 298
 299     # Enable logs
 300     t::lib::Mocks::mock_preference( 'FinesLog', 1 );
 301
 302     my $sip_code = "1";
 303     my $line_2   = $account->add_debit(
 304         {
 305             amount      => 37,
 306             description => 'Rental charge of 37',
 307             library_id  => $patron->branchcode,
 308             note        => 'not really important',
 309             type        => 'rent',
 310             user_id     => $patron->id,
 311         }
 312     );
 313
 314     is( $account->balance, 62, 'Patron has a balance of 62' );
 315     is(
 316         $schema->resultset('ActionLog')->count(),
 317         $action_logs + 1,
 318         'Log was added'
 319     );
 320     is(
 321         $line_2->accounttype,
 322         $Koha::Account::account_type_debit->{'rent'},
 323         'Account type is correctly set'
 324     );
 325
 326     # offsets have the debit_id set to accountlines_id, and credit_id is undef
 327     my $offset_1 =
 328       Koha::Account::Offsets->search( { debit_id => $line_1->id } )->next;
 329     my $offset_2 =
 330       Koha::Account::Offsets->search( { debit_id => $line_2->id } )->next;
 331
 332     is( $offset_1->debit_id,  $line_1->id, 'debit_id is set for debit 1' );
 333     is( $offset_1->credit_id, undef,       'credit_id is not set for debit 1' );
 334     is( $offset_2->debit_id,  $line_2->id, 'debit_id is set for debit 2' );
 335     is( $offset_2->credit_id, undef,       'credit_id is not set for debit 2' );
 336
 337     $schema->storage->txn_rollback;
 338 };
 339
 340 subtest 'lines() tests' => sub {
 341
 342     plan tests => 1;
 343
 344     $schema->storage->txn_begin;
 345
 346     my $patron  = $builder->build_object({ class => 'Koha::Patrons' });
 347     my $account = $patron->account;
 348
 349     my @generated_lines;
 350
 351     # Add Credits
 352     push @generated_lines, $account->add_credit({ amount => 1 });
 353     push @generated_lines, $account->add_credit({ amount => 2 });
 354     push @generated_lines, $account->add_credit({ amount => 3 });
 355     push @generated_lines, $account->add_credit({ amount => 4 });
 356
 357     # Add Debits
 358     push @generated_lines, Koha::Account::Line->new({ borrowernumber => $patron->id, amountoutstanding => 1 })->store;
 359     push @generated_lines, Koha::Account::Line->new({ borrowernumber => $patron->id, amountoutstanding => 2 })->store;
 360     push @generated_lines, Koha::Account::Line->new({ borrowernumber => $patron->id, amountoutstanding => 3 })->store;
 361     push @generated_lines, Koha::Account::Line->new({ borrowernumber => $patron->id, amountoutstanding => 4 })->store;
 362
 363     # Paid Off
 364     push @generated_lines, Koha::Account::Line->new({ borrowernumber => $patron->id, amountoutstanding => 0 })->store;
 365     push @generated_lines, Koha::Account::Line->new({ borrowernumber => $patron->id, amountoutstanding => 0 })->store;
 366
 367     my $lines = $account->lines;
 368     is( $lines->_resultset->count, 10, "All accountlines (debits, credits and paid off) were fetched");
 369
 370     $schema->storage->txn_rollback;
 371 };
 372
 373 subtest 'reconcile_balance' => sub {
 374
 375     plan tests => 4;
 376
 377     subtest 'more credit than debit' => sub {
 378
 379         plan tests => 6;
 380
 381         $schema->storage->txn_begin;
 382
 383         my $patron  = $builder->build_object({ class => 'Koha::Patrons' });
 384         my $account = $patron->account;
 385
 386         # Add Credits
 387         $account->add_credit({ amount => 1 });
 388         $account->add_credit({ amount => 2 });
 389         $account->add_credit({ amount => 3 });
 390         $account->add_credit({ amount => 4 });
 391         $account->add_credit({ amount => 5 });
 392
 393         # Add Debits TODO: replace for calls to add_debit when time comes
 394         Koha::Account::Line->new({ borrowernumber => $patron->id, amount => 1, amountoutstanding => 1 })->store;
 395         Koha::Account::Line->new({ borrowernumber => $patron->id, amount => 2, amountoutstanding => 2 })->store;
 396         Koha::Account::Line->new({ borrowernumber => $patron->id, amount => 3, amountoutstanding => 3 })->store;
 397         Koha::Account::Line->new({ borrowernumber => $patron->id, amount => 4, amountoutstanding => 4 })->store;
 398
 399         # Paid Off
 400         Koha::Account::Line->new({ borrowernumber => $patron->id, amount => 1, amountoutstanding => 0 })->store;
 401         Koha::Account::Line->new({ borrowernumber => $patron->id, amount => 1, amountoutstanding => 0 })->store;
 402
 403         is( $account->balance(), -5, "Account balance is -5" );
 404         is( $account->outstanding_debits->total_outstanding, 10, 'Outstanding debits sum 10' );
 405         is( $account->outstanding_credits->total_outstanding, -15, 'Outstanding credits sum -15' );
 406
 407         $account->reconcile_balance();
 408
 409         is( $account->balance(), -5, "Account balance is -5" );
 410         is( $account->outstanding_debits->total_outstanding, 0, 'No outstanding debits' );
 411         is( $account->outstanding_credits->total_outstanding, -5, 'Outstanding credits sum -5' );
 412
 413         $schema->storage->txn_rollback;
 414     };
 415
 416     subtest 'same debit as credit' => sub {
 417
 418         plan tests => 6;
 419
 420         $schema->storage->txn_begin;
 421
 422         my $patron  = $builder->build_object({ class => 'Koha::Patrons' });
 423         my $account = $patron->account;
 424
 425         # Add Credits
 426         $account->add_credit({ amount => 1 });
 427         $account->add_credit({ amount => 2 });
 428         $account->add_credit({ amount => 3 });
 429         $account->add_credit({ amount => 4 });
 430
 431         # Add Debits TODO: replace for calls to add_debit when time comes
 432         Koha::Account::Line->new({ borrowernumber => $patron->id, amount => 1, amountoutstanding => 1 })->store;
 433         Koha::Account::Line->new({ borrowernumber => $patron->id, amount => 2, amountoutstanding => 2 })->store;
 434         Koha::Account::Line->new({ borrowernumber => $patron->id, amount => 3, amountoutstanding => 3 })->store;
 435         Koha::Account::Line->new({ borrowernumber => $patron->id, amount => 4, amountoutstanding => 4 })->store;
 436
 437         # Paid Off
 438         Koha::Account::Line->new({ borrowernumber => $patron->id, amount => 1, amountoutstanding => 0 })->store;
 439         Koha::Account::Line->new({ borrowernumber => $patron->id, amount => 1, amountoutstanding => 0 })->store;
 440
 441         is( $account->balance(), 0, "Account balance is 0" );
 442         is( $account->outstanding_debits->total_outstanding, 10, 'Outstanding debits sum 10' );
 443         is( $account->outstanding_credits->total_outstanding, -10, 'Outstanding credits sum -10' );
 444
 445         $account->reconcile_balance();
 446
 447         is( $account->balance(), 0, "Account balance is 0" );
 448         is( $account->outstanding_debits->total_outstanding, 0, 'No outstanding debits' );
 449         is( $account->outstanding_credits->total_outstanding, 0, 'Outstanding credits sum 0' );
 450
 451         $schema->storage->txn_rollback;
 452     };
 453
 454     subtest 'more debit than credit' => sub {
 455
 456         plan tests => 6;
 457
 458         $schema->storage->txn_begin;
 459
 460         my $patron  = $builder->build_object({ class => 'Koha::Patrons' });
 461         my $account = $patron->account;
 462
 463         # Add Credits
 464         $account->add_credit({ amount => 1 });
 465         $account->add_credit({ amount => 2 });
 466         $account->add_credit({ amount => 3 });
 467         $account->add_credit({ amount => 4 });
 468
 469         # Add Debits TODO: replace for calls to add_debit when time comes
 470         Koha::Account::Line->new({ borrowernumber => $patron->id, amount => 1, amountoutstanding => 1 })->store;
 471         Koha::Account::Line->new({ borrowernumber => $patron->id, amount => 2, amountoutstanding => 2 })->store;
 472         Koha::Account::Line->new({ borrowernumber => $patron->id, amount => 3, amountoutstanding => 3 })->store;
 473         Koha::Account::Line->new({ borrowernumber => $patron->id, amount => 4, amountoutstanding => 4 })->store;
 474         Koha::Account::Line->new({ borrowernumber => $patron->id, amount => 5, amountoutstanding => 5 })->store;
 475
 476         # Paid Off
 477         Koha::Account::Line->new({ borrowernumber => $patron->id, amount => 1, amountoutstanding => 0 })->store;
 478         Koha::Account::Line->new({ borrowernumber => $patron->id, amount => 1, amountoutstanding => 0 })->store;
 479
 480         is( $account->balance(), 5, "Account balance is 5" );
 481         is( $account->outstanding_debits->total_outstanding, 15, 'Outstanding debits sum 15' );
 482         is( $account->outstanding_credits->total_outstanding, -10, 'Outstanding credits sum -10' );
 483
 484         $account->reconcile_balance();
 485
 486         is( $account->balance(), 5, "Account balance is 5" );
 487         is( $account->outstanding_debits->total_outstanding, 5, 'Outstanding debits sum 5' );
 488         is( $account->outstanding_credits->total_outstanding, 0, 'Outstanding credits sum 0' );
 489
 490         $schema->storage->txn_rollback;
 491     };
 492
 493     subtest 'credits are applied to older debits first' => sub {
 494
 495         plan tests => 9;
 496
 497         $schema->storage->txn_begin;
 498
 499         my $patron  = $builder->build_object({ class => 'Koha::Patrons' });
 500         my $account = $patron->account;
 501
 502         # Add Credits
 503         $account->add_credit({ amount => 1 });
 504         $account->add_credit({ amount => 3 });
 505
 506         # Add Debits TODO: replace for calls to add_debit when time comes
 507         my $debit_1 = Koha::Account::Line->new({ borrowernumber => $patron->id, amount => 1, amountoutstanding => 1 })->store;
 508         my $debit_2 = Koha::Account::Line->new({ borrowernumber => $patron->id, amount => 2, amountoutstanding => 2 })->store;
 509         my $debit_3 = Koha::Account::Line->new({ borrowernumber => $patron->id, amount => 3, amountoutstanding => 3 })->store;
 510
 511         is( $account->balance(), 2, "Account balance is 2" );
 512         is( $account->outstanding_debits->total_outstanding, 6, 'Outstanding debits sum 6' );
 513         is( $account->outstanding_credits->total_outstanding, -4, 'Outstanding credits sum -4' );
 514
 515         $account->reconcile_balance();
 516
 517         is( $account->balance(), 2, "Account balance is 2" );
 518         is( $account->outstanding_debits->total_outstanding, 2, 'Outstanding debits sum 2' );
 519         is( $account->outstanding_credits->total_outstanding, 0, 'Outstanding credits sum 0' );
 520
 521         $debit_1->discard_changes;
 522         is( $debit_1->amountoutstanding + 0, 0, 'Old debit payed' );
 523         $debit_2->discard_changes;
 524         is( $debit_2->amountoutstanding + 0, 0, 'Old debit payed' );
 525         $debit_3->discard_changes;
 526         is( $debit_3->amountoutstanding + 0, 2, 'Newest debit only partially payed' );
 527
 528         $schema->storage->txn_rollback;
 529     };
 530 };
 531
 532 subtest 'pay() tests' => sub {
 533
 534     plan tests => 2;
 535
 536     $schema->storage->txn_begin;
 537
 538     my $patron  = $builder->build_object({ class => 'Koha::Patrons' });
 539     my $library = $builder->build_object({ class => 'Koha::Libraries' });
 540     my $account = $patron->account;
 541
 542     my $context = Test::MockModule->new('C4::Context');
 543     $context->mock( 'userenv', { branch => $library->id } );
 544
 545     my $credit_1_id = $account->pay({ amount => 200 });
 546     my $credit_1    = Koha::Account::Lines->find( $credit_1_id );
 547
 548     is( $credit_1->branchcode, undef, 'No branchcode is set if library_id was not passed' );
 549
 550     my $credit_2_id = $account->pay({ amount => 150, library_id => $library->id });
 551     my $credit_2    = Koha::Account::Lines->find( $credit_2_id );
 552
 553     is( $credit_2->branchcode, $library->id, 'branchcode set because library_id was passed' );
 554
 555     $schema->storage->txn_rollback;
 556 };