kivitendo/t/bank/bank_transactions.t @ fb5b97ff
691a1293 | Geoffrey Richardson | use Test::More;
|
||
use strict;
|
||||
use lib 't';
|
||||
use utf8;
|
||||
use Carp;
|
||||
use Support::TestSetup;
|
||||
use Test::Exception;
|
||||
use List::Util qw(sum);
|
||||
use SL::DB::Buchungsgruppe;
|
||||
use SL::DB::Currency;
|
||||
use SL::DB::Exchangerate;
|
||||
use SL::DB::Customer;
|
||||
use SL::DB::Vendor;
|
||||
use SL::DB::Employee;
|
||||
use SL::DB::Invoice;
|
||||
use SL::DB::Part;
|
||||
use SL::DB::Unit;
|
||||
use SL::DB::TaxZone;
|
||||
use SL::DB::BankAccount;
|
||||
use SL::DB::PaymentTerm;
|
||||
use SL::DB::PurchaseInvoice;
|
||||
use SL::DB::BankTransaction;
|
||||
use SL::Controller::BankTransaction;
|
||||
use Data::Dumper;
|
||||
my ($customer, $vendor, $currency_id, @parts, $unit, $employee, $tax, $tax7, $tax_9, $taxzone, $payment_terms, $bank_account);
|
||||
my ($transdate1, $transdate2, $currency);
|
||||
my ($ar_chart,$bank,$ar_amount_chart, $ap_chart, $ap_amount_chart);
|
||||
my ($ar_transaction, $ap_transaction);
|
||||
sub clear_up {
|
||||
a81f877a | Sven Schöling | |||
691a1293 | Geoffrey Richardson | SL::DB::Manager::BankTransaction->delete_all(all => 1);
|
||
SL::DB::Manager::InvoiceItem->delete_all(all => 1);
|
||||
SL::DB::Manager::InvoiceItem->delete_all(all => 1);
|
||||
SL::DB::Manager::Invoice->delete_all(all => 1);
|
||||
SL::DB::Manager::PurchaseInvoice->delete_all(all => 1);
|
||||
SL::DB::Manager::Part->delete_all(all => 1);
|
||||
SL::DB::Manager::Customer->delete_all(all => 1);
|
||||
SL::DB::Manager::Vendor->delete_all(all => 1);
|
||||
SL::DB::Manager::BankAccount->delete_all(all => 1);
|
||||
SL::DB::Manager::PaymentTerm->delete_all(all => 1);
|
||||
SL::DB::Manager::Currency->delete_all(where => [ name => 'CUR' ]);
|
||||
};
|
||||
# starting test:
|
||||
Support::TestSetup::login();
|
||||
reset_state(); # initialise customers/vendors/bank/currency/...
|
||||
test1();
|
||||
test_overpayment_with_partialpayment();
|
||||
test_overpayment();
|
||||
test_skonto_exact();
|
||||
test_two_invoices();
|
||||
test_partial_payment();
|
||||
# remove all created data at end of test
|
||||
66796c03 | Geoffrey Richardson | clear_up();
|
||
691a1293 | Geoffrey Richardson | |||
done_testing();
|
||||
###### functions for setting up data
|
||||
sub reset_state {
|
||||
my %params = @_;
|
||||
$params{$_} ||= {} for qw(unit customer part tax vendor);
|
||||
clear_up();
|
||||
$transdate1 = DateTime->today;
|
||||
$transdate2 = DateTime->today->add(days => 5);
|
||||
$employee = SL::DB::Manager::Employee->current || croak "No employee";
|
||||
$tax = SL::DB::Manager::Tax->find_by(taxkey => 3, rate => 0.19, %{ $params{tax} }) || croak "No tax";
|
||||
$tax7 = SL::DB::Manager::Tax->find_by(taxkey => 2, rate => 0.07) || croak "No tax for 7\%";
|
||||
$taxzone = SL::DB::Manager::TaxZone->find_by( description => 'Inland') || croak "No taxzone";
|
||||
$tax_9 = SL::DB::Manager::Tax->find_by(taxkey => 9, rate => 0.19, %{ $params{tax} }) || croak "No tax";
|
||||
$currency_id = $::instance_conf->get_currency_id;
|
||||
$bank_account = SL::DB::BankAccount->new(
|
||||
account_number => '123',
|
||||
bank_code => '123',
|
||||
iban => '123',
|
||||
bic => '123',
|
||||
bank => '123',
|
||||
chart_id => SL::DB::Manager::Chart->find_by(description => 'Bank')->id,
|
||||
name => SL::DB::Manager::Chart->find_by(description => 'Bank')->description,
|
||||
)->save;
|
||||
$customer = SL::DB::Customer->new(
|
||||
name => 'Test Customer',
|
||||
currency_id => $currency_id,
|
||||
taxzone_id => $taxzone->id,
|
||||
iban => 'DE12500105170648489890',
|
||||
bic => 'TESTBIC',
|
||||
account_number => '648489890',
|
||||
mandate_date_of_signature => $transdate1,
|
||||
mandator_id => 'foobar',
|
||||
bank => 'Geizkasse',
|
||||
depositor => 'Test Customer',
|
||||
%{ $params{customer} }
|
||||
)->save;
|
||||
$payment_terms = SL::DB::PaymentTerm->new(
|
||||
description => 'payment',
|
||||
description_long => 'payment',
|
||||
terms_netto => '30',
|
||||
terms_skonto => '5',
|
||||
percent_skonto => '0.05',
|
||||
auto_calculation => 1,
|
||||
)->save;
|
||||
$vendor = SL::DB::Vendor->new(
|
||||
name => 'Test Vendor',
|
||||
currency_id => $currency_id,
|
||||
taxzone_id => $taxzone->id,
|
||||
payment_id => $payment_terms->id,
|
||||
iban => 'DE12500105170648489890',
|
||||
bic => 'TESTBIC',
|
||||
account_number => '648489890',
|
||||
bank => 'Geizkasse',
|
||||
depositor => 'Test Vendor',
|
||||
%{ $params{vendor} }
|
||||
)->save;
|
||||
$ar_chart = SL::DB::Manager::Chart->find_by( accno => '1400' ); # Forderungen
|
||||
$ap_chart = SL::DB::Manager::Chart->find_by( accno => '1600' ); # Verbindlichkeiten
|
||||
$bank = SL::DB::Manager::Chart->find_by( accno => '1200' ); # Bank
|
||||
$ar_amount_chart = SL::DB::Manager::Chart->find_by( accno => '8400' ); # Erlöse
|
||||
$ap_amount_chart = SL::DB::Manager::Chart->find_by( accno => '3400' ); # Wareneingang 19%
|
||||
}
|
||||
sub test_ar_transaction {
|
||||
my (%params) = @_;
|
||||
my $netamount = 100;
|
||||
my $amount = $params{amount} || $::form->round_amount(100 * 1.19,2);
|
||||
my $invoice = SL::DB::Invoice->new(
|
||||
invoice => 0,
|
||||
invnumber => $params{invnumber} || undef, # let it use its own invnumber
|
||||
amount => $amount,
|
||||
netamount => $netamount,
|
||||
transdate => $transdate1,
|
||||
taxincluded => 0,
|
||||
customer_id => $customer->id,
|
||||
taxzone_id => $customer->taxzone_id,
|
||||
currency_id => $currency_id,
|
||||
transactions => [],
|
||||
payment_id => $params{payment_id} || undef,
|
||||
notes => 'test_ar_transaction',
|
||||
);
|
||||
$invoice->add_ar_amount_row(
|
||||
amount => $invoice->netamount,
|
||||
chart => $ar_amount_chart,
|
||||
tax_id => $tax->id,
|
||||
);
|
||||
$invoice->create_ar_row(chart => $ar_chart);
|
||||
$invoice->save;
|
||||
is($invoice->currency_id , $currency_id , 'currency_id has been saved');
|
||||
is($invoice->netamount , 100 , 'ar amount has been converted');
|
||||
is($invoice->amount , 119 , 'ar amount has been converted');
|
||||
is($invoice->taxincluded , 0 , 'ar transaction doesn\'t have taxincluded');
|
||||
is(SL::DB::Manager::AccTransaction->find_by(chart_id => $ar_amount_chart->id , trans_id => $invoice->id)->amount , '100.00000' , $ar_amount_chart->accno . ': has been converted for currency');
|
||||
is(SL::DB::Manager::AccTransaction->find_by(chart_id => $ar_chart->id , trans_id => $invoice->id)->amount , '-119.00000' , $ar_chart->accno . ': has been converted for currency');
|
||||
return $invoice;
|
||||
};
|
||||
sub test_ap_transaction {
|
||||
my (%params) = @_;
|
||||
my $netamount = 100;
|
||||
my $amount = $::form->round_amount($netamount * 1.19,2);
|
||||
my $invoice = SL::DB::PurchaseInvoice->new(
|
||||
invoice => 0,
|
||||
invnumber => $params{invnumber} || 'test_ap_transaction',
|
||||
amount => $amount,
|
||||
netamount => $netamount,
|
||||
transdate => $transdate1,
|
||||
taxincluded => 0,
|
||||
vendor_id => $vendor->id,
|
||||
taxzone_id => $vendor->taxzone_id,
|
||||
currency_id => $currency_id,
|
||||
transactions => [],
|
||||
notes => 'test_ap_transaction',
|
||||
);
|
||||
$invoice->add_ap_amount_row(
|
||||
amount => $invoice->netamount,
|
||||
chart => $ap_amount_chart,
|
||||
tax_id => $tax_9->id,
|
||||
);
|
||||
$invoice->create_ap_row(chart => $ap_chart);
|
||||
$invoice->save;
|
||||
is($invoice->currency_id , $currency_id , 'currency_id has been saved');
|
||||
is($invoice->netamount , 100 , 'ap amount has been converted');
|
||||
is($invoice->amount , 119 , 'ap amount has been converted');
|
||||
is($invoice->taxincluded , 0 , 'ap transaction doesn\'t have taxincluded');
|
||||
is(SL::DB::Manager::AccTransaction->find_by(chart_id => $ap_amount_chart->id , trans_id => $invoice->id)->amount , '-100.00000' , $ap_amount_chart->accno . ': has been converted for currency');
|
||||
is(SL::DB::Manager::AccTransaction->find_by(chart_id => $ap_chart->id , trans_id => $invoice->id)->amount , '119.00000' , $ap_chart->accno . ': has been converted for currency');
|
||||
return $invoice;
|
||||
};
|
||||
###### test cases
|
||||
sub test1 {
|
||||
my $testname = 'test1';
|
||||
$ar_transaction = test_ar_transaction(invnumber => 'salesinv1');
|
||||
my $bt = $ar_transaction->create_bank_transaction or die "Couldn't create bank_transaction";
|
||||
$::form->{invoice_ids} = {
|
||||
$bt->id => [ $ar_transaction->id ]
|
||||
};
|
||||
a81f877a | Sven Schöling | |||
691a1293 | Geoffrey Richardson | my $bt_controller = SL::Controller::BankTransaction->new;
|
||
$bt_controller->action_save_invoices;
|
||||
$ar_transaction->load;
|
||||
$bt->load;
|
||||
is($ar_transaction->paid , '119.00000' , "$testname: salesinv1 was paid");
|
||||
is($ar_transaction->closed , 1 , "$testname: salesinv1 is closed");
|
||||
is($bt->invoice_amount , '119.00000' , "$testname: bt invoice amount was assigned");
|
||||
};
|
||||
sub test_skonto_exact {
|
||||
my $testname = 'test_skonto_exact';
|
||||
$ar_transaction = test_ar_transaction(invnumber => 'salesinv skonto',
|
||||
payment_id => $payment_terms->id,
|
||||
);
|
||||
my $bt = $ar_transaction->create_bank_transaction(amount => $ar_transaction->amount_less_skonto) or die "Couldn't create bank_transaction";
|
||||
$::form->{invoice_ids} = {
|
||||
$bt->id => [ $ar_transaction->id ]
|
||||
};
|
||||
$::form->{invoice_skontos} = {
|
||||
$bt->id => [ 'with_skonto_pt' ]
|
||||
};
|
||||
a81f877a | Sven Schöling | |||
691a1293 | Geoffrey Richardson | my $bt_controller = SL::Controller::BankTransaction->new;
|
||
$bt_controller->action_save_invoices;
|
||||
$ar_transaction->load;
|
||||
$bt->load;
|
||||
is($ar_transaction->paid , '119.00000' , "$testname: salesinv skonto was paid");
|
||||
is($ar_transaction->closed , 1 , "$testname: salesinv skonto is closed");
|
||||
is($bt->invoice_amount , '113.05000' , "$testname: bt invoice amount was assigned");
|
||||
};
|
||||
sub test_two_invoices {
|
||||
my $testname = 'test_two_invoices';
|
||||
my $ar_transaction_1 = test_ar_transaction(invnumber => 'salesinv_1');
|
||||
my $ar_transaction_2 = test_ar_transaction(invnumber => 'salesinv_2');
|
||||
my $bt = $ar_transaction_1->create_bank_transaction(amount => ($ar_transaction_1->amount + $ar_transaction_2->amount),
|
||||
purpose => "Rechnungen " . $ar_transaction_1->invnumber . " und " . $ar_transaction_2->invnumber,
|
||||
) or die "Couldn't create bank_transaction";
|
||||
$::form->{invoice_ids} = {
|
||||
$bt->id => [ $ar_transaction_1->id, $ar_transaction_2->id ]
|
||||
};
|
||||
a81f877a | Sven Schöling | |||
691a1293 | Geoffrey Richardson | my $bt_controller = SL::Controller::BankTransaction->new;
|
||
$bt_controller->action_save_invoices;
|
||||
$ar_transaction_1->load;
|
||||
$ar_transaction_2->load;
|
||||
$bt->load;
|
||||
is($ar_transaction_1->paid , '119.00000' , "$testname: salesinv_1 was paid");
|
||||
is($ar_transaction_1->closed , 1 , "$testname: salesinv_1 is closed");
|
||||
is($ar_transaction_2->paid , '119.00000' , "$testname: salesinv_2 was paid");
|
||||
is($ar_transaction_2->closed , 1 , "$testname: salesinv_2 is closed");
|
||||
is($bt->invoice_amount , '238.00000' , "$testname: bt invoice amount was assigned");
|
||||
};
|
||||
sub test_overpayment {
|
||||
my $testname = 'test_overpayment';
|
||||
$ar_transaction = test_ar_transaction(invnumber => 'salesinv overpaid');
|
||||
# amount 135 > 119
|
||||
my $bt = $ar_transaction->create_bank_transaction(amount => 135) or die "Couldn't create bank_transaction";
|
||||
$::form->{invoice_ids} = {
|
||||
$bt->id => [ $ar_transaction->id ]
|
||||
};
|
||||
a81f877a | Sven Schöling | |||
691a1293 | Geoffrey Richardson | my $bt_controller = SL::Controller::BankTransaction->new;
|
||
$bt_controller->action_save_invoices;
|
||||
$ar_transaction->load;
|
||||
$bt->load;
|
||||
is($ar_transaction->paid , '135.00000' , "$testname: 'salesinv overpaid' was overpaid");
|
||||
is($bt->invoice_amount , '135.00000' , "$testname: bt invoice amount was assigned overpaid amount");
|
||||
a81f877a | Sven Schöling | { local $TODO = 'this currently fails because closed ignores over-payments, see commit d90966c7';
|
||
691a1293 | Geoffrey Richardson | is($ar_transaction->closed , 0 , "$testname: 'salesinv overpaid' is open (via 'closed' method')");
|
||
a81f877a | Sven Schöling | }
|
||
691a1293 | Geoffrey Richardson | is($ar_transaction->open_amount == 0 ? 1 : 0 , 0 , "$testname: 'salesinv overpaid is open (via amount-paid)");
|
||
};
|
||||
sub test_overpayment_with_partialpayment {
|
||||
# two payments on different days, 10 and 119. If there is only one invoice we want it be overpaid.
|
||||
my $testname = 'test_overpayment_with_partialpayment';
|
||||
$ar_transaction = test_ar_transaction(invnumber => 'salesinv overpaid partial');
|
||||
my $bt_1 = $ar_transaction->create_bank_transaction(amount => 10) or die "Couldn't create bank_transaction";
|
||||
my $bt_2 = $ar_transaction->create_bank_transaction(amount => 119,
|
||||
transdate => DateTime->today->add(days => 5),
|
||||
) or die "Couldn't create bank_transaction";
|
||||
$::form->{invoice_ids} = {
|
||||
$bt_1->id => [ $ar_transaction->id ]
|
||||
};
|
||||
my $bt_controller_1 = SL::Controller::BankTransaction->new;
|
||||
$bt_controller_1->action_save_invoices;
|
||||
$::form->{invoice_ids} = {
|
||||
$bt_2->id => [ $ar_transaction->id ]
|
||||
};
|
||||
my $bt_controller_2 = SL::Controller::BankTransaction->new;
|
||||
$bt_controller_2->action_save_invoices;
|
||||
$ar_transaction->load;
|
||||
$bt_1->load;
|
||||
$bt_2->load;
|
||||
is($ar_transaction->paid , '129.00000' , "$testname: 'salesinv overpaid partial' was overpaid");
|
||||
is($bt_1->invoice_amount , '10.00000' , "$testname: bt_1 invoice amount was assigned overpaid amount");
|
||||
is($bt_2->invoice_amount , '119.00000' , "$testname: bt_2 invoice amount was assigned overpaid amount");
|
||||
};
|
||||
sub test_partial_payment {
|
||||
my $testname = 'test_partial_payment';
|
||||
$ar_transaction = test_ar_transaction(invnumber => 'salesinv partial payment');
|
||||
# amount 100 < 119
|
||||
my $bt = $ar_transaction->create_bank_transaction(amount => 100) or die "Couldn't create bank_transaction";
|
||||
$::form->{invoice_ids} = {
|
||||
$bt->id => [ $ar_transaction->id ]
|
||||
};
|
||||
a81f877a | Sven Schöling | |||
691a1293 | Geoffrey Richardson | my $bt_controller = SL::Controller::BankTransaction->new;
|
||
$bt_controller->action_save_invoices;
|
||||
$ar_transaction->load;
|
||||
$bt->load;
|
||||
is($ar_transaction->paid , '100.00000' , "$testname: 'salesinv partial payment' was partially paid");
|
||||
is($bt->invoice_amount , '100.00000' , "$testname: bt invoice amount was assigned partially paid amount");
|
||||
};
|
||||
1;
|