Lines 19-25
Link Here
|
19 |
|
19 |
|
20 |
use Modern::Perl; |
20 |
use Modern::Perl; |
21 |
|
21 |
|
22 |
use Test::More tests => 14; |
22 |
use Test::More tests => 15; |
23 |
use Test::Exception; |
23 |
use Test::Exception; |
24 |
|
24 |
|
25 |
use C4::Circulation qw/AddIssue AddReturn/; |
25 |
use C4::Circulation qw/AddIssue AddReturn/; |
Lines 1029-1032
subtest "payout() tests" => sub {
Link Here
|
1029 |
$schema->storage->txn_rollback; |
1029 |
$schema->storage->txn_rollback; |
1030 |
}; |
1030 |
}; |
1031 |
|
1031 |
|
|
|
1032 |
subtest "reduce() tests" => sub { |
1033 |
|
1034 |
plan tests => 25; |
1035 |
|
1036 |
$schema->storage->txn_begin; |
1037 |
|
1038 |
# Create a borrower |
1039 |
my $categorycode = |
1040 |
$builder->build( { source => 'Category' } )->{categorycode}; |
1041 |
my $branchcode = $builder->build( { source => 'Branch' } )->{branchcode}; |
1042 |
|
1043 |
my $borrower = Koha::Patron->new( |
1044 |
{ |
1045 |
cardnumber => 'dariahall', |
1046 |
surname => 'Hall', |
1047 |
firstname => 'Daria', |
1048 |
} |
1049 |
); |
1050 |
$borrower->categorycode($categorycode); |
1051 |
$borrower->branchcode($branchcode); |
1052 |
$borrower->store; |
1053 |
|
1054 |
my $staff = Koha::Patron->new( |
1055 |
{ |
1056 |
cardnumber => 'bobby', |
1057 |
surname => 'Bloggs', |
1058 |
firstname => 'Bobby', |
1059 |
} |
1060 |
); |
1061 |
$staff->categorycode($categorycode); |
1062 |
$staff->branchcode($branchcode); |
1063 |
$staff->store; |
1064 |
|
1065 |
my $account = Koha::Account->new( { patron_id => $borrower->id } ); |
1066 |
|
1067 |
my $debit1 = Koha::Account::Line->new( |
1068 |
{ |
1069 |
borrowernumber => $borrower->borrowernumber, |
1070 |
amount => 20, |
1071 |
amountoutstanding => 20, |
1072 |
interface => 'commandline', |
1073 |
debit_type_code => 'LOST' |
1074 |
} |
1075 |
)->store(); |
1076 |
my $credit1 = Koha::Account::Line->new( |
1077 |
{ |
1078 |
borrowernumber => $borrower->borrowernumber, |
1079 |
amount => -20, |
1080 |
amountoutstanding => -20, |
1081 |
interface => 'commandline', |
1082 |
credit_type_code => 'CREDIT' |
1083 |
} |
1084 |
)->store(); |
1085 |
|
1086 |
is( $account->balance(), 0, "Account balance is 0" ); |
1087 |
is( $debit1->amountoutstanding, |
1088 |
20, 'Overdue fee has an amount outstanding of 20' ); |
1089 |
is( $credit1->amountoutstanding, |
1090 |
-20, 'Credit has an amount outstanding of -20' ); |
1091 |
|
1092 |
my $reduce_params = { |
1093 |
interface => 'commandline', |
1094 |
reduction_type => 'REFUND', |
1095 |
amount => 5, |
1096 |
staff_id => $staff->borrowernumber, |
1097 |
branch => $branchcode |
1098 |
}; |
1099 |
|
1100 |
throws_ok { $credit1->reduce($reduce_params); } |
1101 |
'Koha::Exceptions::Account::IsNotDebit', |
1102 |
'->reduce() can only be used with debits'; |
1103 |
|
1104 |
my @required = ( 'interface', 'reduction_type', 'amount' ); |
1105 |
for my $required (@required) { |
1106 |
my $params = {%$reduce_params}; |
1107 |
delete( $params->{$required} ); |
1108 |
throws_ok { |
1109 |
$debit1->reduce($params); |
1110 |
} |
1111 |
'Koha::Exceptions::MissingParameter', |
1112 |
"->reduce() requires the `$required` parameter is passed"; |
1113 |
} |
1114 |
|
1115 |
$reduce_params->{interface} = 'intranet'; |
1116 |
my @dependant_required = ( 'staff_id', 'branch' ); |
1117 |
for my $d (@dependant_required) { |
1118 |
my $params = {%$reduce_params}; |
1119 |
delete( $params->{$d} ); |
1120 |
throws_ok { |
1121 |
$debit1->reduce($params); |
1122 |
} |
1123 |
'Koha::Exceptions::MissingParameter', |
1124 |
"->reduce() requires the `$d` parameter is passed when interface is intranet"; |
1125 |
} |
1126 |
|
1127 |
throws_ok { |
1128 |
$debit1->reduce( |
1129 |
{ |
1130 |
interface => 'intranet', |
1131 |
staff_id => $staff->borrowernumber, |
1132 |
branch => $branchcode, |
1133 |
reduction_type => 'REFUND', |
1134 |
amount => 25 |
1135 |
} |
1136 |
); |
1137 |
} |
1138 |
'Koha::Exceptions::ParameterTooHigh', |
1139 |
'->reduce() cannot reduce more than original amount'; |
1140 |
|
1141 |
# Partial Reduction |
1142 |
# (Refund 5 on debt of 20) |
1143 |
my $reduction = $debit1->reduce($reduce_params); |
1144 |
|
1145 |
is( $reduction->amount() * 1, -5, "Reduce amount is -5" ); |
1146 |
is( $reduction->amountoutstanding() * 1, |
1147 |
0, "Reduce amountoutstanding is 0" ); |
1148 |
is( $debit1->amountoutstanding() * 1, |
1149 |
15, "Debit amountoutstanding reduced by 5 to 15" ); |
1150 |
is( $account->balance() * 1, -5, "Account balance is -5" ); |
1151 |
is( $reduction->status(), 'APPLIED', "Reduction status is 'APPLIED'" ); |
1152 |
|
1153 |
my $offsets = Koha::Account::Offsets->search( |
1154 |
{ credit_id => $reduction->id, debit_id => $debit1->id } ); |
1155 |
is( $offsets->count, 1, 'Only one offset is generated' ); |
1156 |
my $THE_offset = $offsets->next; |
1157 |
is( $THE_offset->amount * 1, |
1158 |
-5, 'Correct amount was applied against debit' ); |
1159 |
is( $THE_offset->type, 'REFUND', "Offset type set to 'REFUND'" ); |
1160 |
|
1161 |
# Zero offset created when zero outstanding |
1162 |
# (Refund another 5 on paid debt of 20) |
1163 |
$credit1->apply( { debits => [ $debit1 ] } ); |
1164 |
is($debit1->amountoutstanding + 0, 0, 'Debit1 amountoutstanding reduced to 0'); |
1165 |
$reduction = $debit1->reduce($reduce_params); |
1166 |
is( $reduction->amount() * 1, -5, "Reduce amount is -5" ); |
1167 |
is( $reduction->amountoutstanding() * 1, |
1168 |
-5, "Reduce amountoutstanding is -5" ); |
1169 |
|
1170 |
$offsets = Koha::Account::Offsets->search( |
1171 |
{ credit_id => $reduction->id, debit_id => $debit1->id } ); |
1172 |
is( $offsets->count, 1, 'Only one new offset is generated' ); |
1173 |
$THE_offset = $offsets->next; |
1174 |
is( $THE_offset->amount * 1, |
1175 |
0, 'Zero offset created for already paid off debit' ); |
1176 |
is( $THE_offset->type, 'REFUND', "Offset type set to 'REFUND'" ); |
1177 |
|
1178 |
# Compound reduction should not allow more than original amount |
1179 |
# (Reduction of 5 + 5 + 20 > 20) |
1180 |
$reduce_params->{amount} = 20; |
1181 |
throws_ok { |
1182 |
$debit1->reduce($reduce_params); |
1183 |
} |
1184 |
'Koha::Exceptions::ParameterTooHigh', |
1185 |
'->reduce cannot reduce mor than the original amount (combined reductions test)'; |
1186 |
|
1187 |
$schema->storage->txn_rollback; |
1188 |
}; |
1189 |
|
1032 |
1; |
1190 |
1; |
1033 |
- |
|
|