Bug 13321: Rename variables

[koha.git] / t / db_dependent / Acquisition / TransferOrder.t

diff --git a/t/db_dependent/Acquisition/TransferOrder.t b/t/db_dependent/Acquisition/TransferOrder.t
index fd294e2..efb8e00 100644 (file)
--- a/t/db_dependent/Acquisition/TransferOrder.t
+++ b/t/db_dependent/Acquisition/TransferOrder.t
@@ -2,14 +2,24 @@
 
 use Modern::Perl;
 
-use Test::More;
+use Test::More tests => 13;
+use C4::Context;
 use C4::Acquisition;
 use C4::Biblio;
 use C4::Items;
 use C4::Bookseller;
 use C4::Budgets;
+use Koha::Database;
+use Koha::DateUtils;
+use Koha::Acquisition::Order;
 use MARC::Record;
 
+my $schema = Koha::Database->new()->schema();
+$schema->storage->txn_begin();
+
+my $dbh = C4::Context->dbh;
+$dbh->{RaiseError} = 1;
+
 my $booksellerid1 = C4::Bookseller::AddBookseller(
     {
         name => "my vendor 1",
@@ -48,39 +58,53 @@ my $budget = C4::Budgets::GetBudget( $budgetid );
 my ($biblionumber, $biblioitemnumber) = AddBiblio(MARC::Record->new, '');
 my $itemnumber = AddItem({}, $biblionumber);
 
-my ( undef, $ordernumber ) = C4::Acquisition::NewOrder(
+my $order = Koha::Acquisition::Order->new(
     {
         basketno => $basketno1,
-        quantity => 24,
+        quantity => 2,
         biblionumber => $biblionumber,
         budget_id => $budget->{budget_id},
     }
-);
-NewOrderItem($itemnumber, $ordernumber);
+)->insert;
+my $ordernumber = $order->{ordernumber};
+$order->add_item( $itemnumber );
 
 # Begin tests
-my $order;
 is(scalar GetOrders($basketno1), 1, "1 order in basket1");
 ($order) = GetOrders($basketno1);
 is(scalar GetItemnumbersFromOrder($order->{ordernumber}), 1, "1 item in basket1's order");
 is(scalar GetOrders($basketno2), 0, "0 order in basket2");
 
-diag("Transfering order to basket2");
+# Transfering order to basket2
 my $newordernumber = TransferOrder($ordernumber, $basketno2);
 is(scalar GetOrders($basketno1), 0, "0 order in basket1");
 is(scalar GetOrders($basketno2), 1, "1 order in basket2");
+
+# Determine if the transfer marked things cancelled properly.
+is($order->{orderstatus},'new','Before the transfer, the order status should be new');
+($order) = GetOrders($basketno1, { 'cancelled' => 1 });
+is($order->{orderstatus},'cancelled','After the transfer, the order status should be set to cancelled');
+
 ($order) = GetOrders($basketno2);
 is(scalar GetItemnumbersFromOrder($order->{ordernumber}), 1, "1 item in basket2's order");
 
-END {
-    C4::Acquisition::DelOrder( $biblionumber, $ordernumber );
-    C4::Acquisition::DelOrder( $biblionumber, $newordernumber );
-    C4::Budgets::DelBudget( $budgetid );
-    C4::Acquisition::DelBasket( $basketno1 );
-    C4::Bookseller::DelBookseller( $booksellerid1 );
-    C4::Acquisition::DelBasket( $basketno2 );
-    C4::Bookseller::DelBookseller( $booksellerid2 );
-    C4::Biblio::DelBiblio($biblionumber);
-};
-
-done_testing;
+# Bug 11552
+my $orders = SearchOrders({ ordernumber => $newordernumber });
+is ( scalar( @$orders ), 1, 'SearchOrders returns 1 order with newordernumber' );
+$orders = SearchOrders({ ordernumber => $ordernumber });
+is ( scalar( @$orders ), 1, 'SearchOrders returns 1 order with [old]ordernumber' );
+is ( $orders->[0]->{ordernumber}, $newordernumber, 'SearchOrders returns newordernumber if [old]ordernumber is given' );
+
+my $neworder = GetOrder( $newordernumber );
+
+ModReceiveOrder({
+    biblionumber => $biblionumber,
+    order       => $neworder,
+    quantityreceived => 2, 
+});
+CancelReceipt( $newordernumber );
+$order = GetOrder( $newordernumber );
+is ( $order->{ordernumber}, $newordernumber, 'Regression test Bug 11549: After a transfer, receive and cancel the receive should be possible.' );
+is ( $order->{basketno}, $basketno2, 'Regression test Bug 11549: The order still exist in the basket where the transfer has been done.');
+
+$schema->storage->txn_rollback();