From 00cf699c82aeea46ef5a72bf344fd306430c5aba Mon Sep 17 00:00:00 2001
From: Nick Clemens <nick@bywatersolutions.com>
Date: Tue, 17 Apr 2018 14:37:11 +0000
Subject: [PATCH] Bug 18736: (follow-up) Add tests and FIXME for
 GetbudgetsPlanCell

prove t/db_dependent/Budgets.t

Signed-off-by: Julian Maurice <julian.maurice@biblibre.com>

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>

Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
---
 C4/Budgets.pm            |  4 +++-
 t/db_dependent/Budgets.t | 35 +++++++++++++++++++++++++++++++++--
 2 files changed, 36 insertions(+), 3 deletions(-)

diff --git a/C4/Budgets.pm b/C4/Budgets.pm
index 39bec9ebe9..d689ad80be 100644
--- a/C4/Budgets.pm
+++ b/C4/Budgets.pm
@@ -208,12 +208,13 @@ sub SetOwnerToFundHierarchy {
 
 # -------------------------------------------------------------------
 sub GetBudgetsPlanCell {
-    my ( $cell, $period, $budget ) = @_;
+    my ( $cell, $period, $budget ) = @_; #FIXME we don't use $period or $budget
     my ($actual, $sth);
     my $dbh = C4::Context->dbh;
     my $roundsql = _get_rounding_sql(qq|ecost_tax_included|);
     if ( $cell->{'authcat'} eq 'MONTHS' ) {
         # get the actual amount
+        # FIXME we should consider quantity
         $sth = $dbh->prepare( qq|
 
             SELECT SUM(| .  $roundsql . qq|) AS actual FROM aqorders
@@ -223,6 +224,7 @@ sub GetBudgetsPlanCell {
         $sth->execute( $cell->{'budget_id'} );
     } elsif ( $cell->{'authcat'} eq 'BRANCHES' ) {
         # get the actual amount
+        # FIXME we should consider quantity
         $sth = $dbh->prepare( qq|
 
             SELECT SUM(| . $roundsql . qq|) FROM aqorders
diff --git a/t/db_dependent/Budgets.t b/t/db_dependent/Budgets.t
index 816d97a5b1..f127a77271 100755
--- a/t/db_dependent/Budgets.t
+++ b/t/db_dependent/Budgets.t
@@ -14,6 +14,7 @@ use Koha::Acquisition::Orders;
 use Koha::Acquisition::Funds;
 use Koha::Patrons;
 use Koha::Number::Price;
+use Koha::Items;
 
 use t::lib::TestBuilder;
 use t::lib::Mocks;
@@ -966,11 +967,12 @@ subtest 'GetBudgetSpent and GetBudgetOrdered' => sub {
     is( @$hierarchy[0]->{total_ordered},6,"After adding invoice adjustment on child budget, budget hierarchy shows 6 ordered");
 };
 
-subtest 'OrderPriceRounding GetBudgetSpent GetBudgetOrdered tests' => sub {
+subtest 'GetBudgetSpent GetBudgetOrdered GetBudgetsPlanCell tests' => sub {
 
-    plan tests => 8;
+    plan tests => 12;
 
 #Let's build an order, we need a couple things though
+    t::lib::Mocks::mock_preference('OrderPriceRounding','nearest_cent');
 
     my $spent_biblio = $builder->build({ source => 'Biblio' });
     my $spent_basket = $builder->build({ source => 'Aqbasket', value => { is_standing => 0 } });
@@ -1017,6 +1019,35 @@ subtest 'OrderPriceRounding GetBudgetSpent GetBudgetOrdered tests' => sub {
     $spent_ordered = GetBudgetOrdered( $spent_order->{budget_id} );
     is($spent_ordered,'78.8',"We expect the ordered amount to be equal to the estimated price rounded times quantity");
 
+#Let's test some budget planning
+#Regression tests for bug 18736
+    #We need an item to test by BRANCHES
+    my $item_1 = $builder->build({ source => 'Item' });
+    my $order_item_1 = $builder->build({ source => 'AqordersItem', value => { ordernumber => $spent_order->{ordernumber}, itemnumber => $item_1->{itemnumber}  } });
+    my $spent_fund = Koha::Acquisition::Funds->find( $spent_order->{budget_id} );
+    my $cell = {
+        authcat => 'MONTHS',
+        cell_authvalue => $spent_order->{entrydate}, #normally this is just the year/month but full won't hurt us here
+        budget_id => $spent_order->{budget_id},
+        budget_period_id => $spent_fund->budget_period_id,
+    };
+    t::lib::Mocks::mock_preference('OrderPriceRounding','');
+    my ( $actual ) = GetBudgetsPlanCell( $cell, undef, undef); #we are only testing the actual for now
+    is ( $actual, '9.854200', "We expect this to be an exact order cost"); #really we should expect cost*quantity but we don't
+    t::lib::Mocks::mock_preference('OrderPriceRounding','nearest_cent');
+    ( $actual ) = GetBudgetsPlanCell( $cell, undef, undef); #we are only testing the actual for now
+    is ( $actual, '9.8500', "We expect this to be a rounded order cost"); #really we should expect cost*quantity but we don't
+    $cell->{authcat} = 'BRANCHES';
+    $cell->{authvalue} = $item_1->{homebranch};
+    ( $actual ) = GetBudgetsPlanCell( $cell, undef, undef); #we are only testing the actual for now
+    t::lib::Mocks::mock_preference('OrderPriceRounding','');
+    ( $actual ) = GetBudgetsPlanCell( $cell, undef, undef); #we are only testing the actual for now
+    is ( $actual, '9.854200', "We expect this to be full cost for items from the branch"); #here we rely on items having been created for each order, again quantity should be considered
+    t::lib::Mocks::mock_preference('OrderPriceRounding','nearest_cent');
+    ( $actual ) = GetBudgetsPlanCell( $cell, undef, undef); #we are only testing the actual for now
+    is ( $actual, '9.8500', "We expect this to be rounded cost for items from the branch"); #here we rely on items having been created for each order, again quantity should be considered
+
+
 #Okay, now we can receive the order, giving the price as the user would
 
     $spent_orderinfo->{unitprice} = 9.85; #we are paying what we expected
-- 
2.39.5