ACP2E-3045: Order closed without fully refunded

Author: aplapana

app/code/Magento/Sales/Model/ResourceModel/Order/Handler/State.php

@@ -28,7 +28,12 @@ public function check(Order $order)
                 ->setStatus($order->getConfig()->getStateDefaultStatus(Order::STATE_PROCESSING));
             $currentState = Order::STATE_PROCESSING;
         }
-        if ($order->isCanceled() || $order->canUnhold() || $order->canInvoice() || $this->orderHasOpenInvoices($order) || (int) $order->getTotalDue() > 0) {
+        if ($order->isCanceled() ||
+            $order->canUnhold() ||
+            $order->canInvoice() ||
+            $this->orderHasOpenInvoices($order) ||
+            (int) $order->getTotalDue() > 0
+        ) {
             return $this;
         }
 

app/code/Magento/Sales/Test/Unit/Model/ResourceModel/Order/Handler/StateTest.php

@@ -8,10 +8,8 @@
 namespace Magento\Sales\Test\Unit\Model\ResourceModel\Order\Handler;
 
 use Magento\Sales\Model\Order;
-use Magento\Sales\Model\Order\Address;
 use Magento\Sales\Model\Order\Item;
 use Magento\Sales\Model\Order\Invoice;
-use Magento\Sales\Model\ResourceModel\Order\Address\Collection;
 use Magento\Sales\Model\ResourceModel\Order\Handler\State;
 use Magento\Sales\Model\ResourceModel\Order\Invoice\Collection as InvoiceCollection;
 use PHPUnit\Framework\MockObject\MockObject;
@@ -50,7 +48,8 @@ protected function setUp(): void
                     'getStatus',
                     'getAllItems',
                     'getInvoiceCollection',
-                    'getTotalQtyOrdered'
+                    'getTotalQtyOrdered',
+                    'getTotalDue'
                 ]
             )
             ->disableOriginalConstructor()