Skip to content

Commit 736ce0d

Browse files
committed
MC-20639: MyAccount :: Order Details :: Refund (creditMemo) Details by Order Number
- cleanup test
1 parent 45a955b commit 736ce0d

File tree

1 file changed

+18
-22
lines changed

1 file changed

+18
-22
lines changed

dev/tests/api-functional/testsuite/Magento/GraphQl/Sales/CreditmemoTest.php

Lines changed: 18 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,6 @@
1818
use Magento\Sales\Model\Order;
1919
use Magento\Sales\Model\Order\CreditmemoFactory;
2020
use Magento\Sales\Model\ResourceModel\Order\Collection as OrderCollection;
21-
use Magento\Sales\Model\ResourceModel\Order\Creditmemo\Collection as CreditMemoCollection;
2221
use Magento\Sales\Model\Service\CreditmemoService;
2322
use Magento\TestFramework\Helper\Bootstrap;
2423
use Magento\TestFramework\TestCase\GraphQlAbstract;
@@ -51,9 +50,6 @@ class CreditmemoTest extends GraphQlAbstract
5150
/** @var OrderRepositoryInterface */
5251
private $orderRepository;
5352

54-
/** @var CreditMemoCollection */
55-
private $creditMemoCollection;
56-
5753
/** @var SearchCriteriaBuilder */
5854
private $searchCriteriaBuilder;
5955

@@ -74,7 +70,6 @@ protected function setUp(): void
7470
$this->orderCollection = $objectManager->get(OrderCollection::class);
7571
$this->orderRepository = $objectManager->get(OrderRepositoryInterface::class);
7672
$this->creditMemoService = $objectManager->get(CreditmemoService::class);
77-
$this->creditMemoCollection = $objectManager->get(CreditMemoCollection::class);
7873
}
7974

8075
/**
@@ -172,14 +167,14 @@ public function testCreditMemoForBundledProductsWithPartialRefund()
172167
$this->setPaymentMethod($cartId, $paymentMethod);
173168
$orderNumber = $this->placeOrder($cartId);
174169
$this->prepareInvoice($orderNumber, 2);
175-
// Create a credit memo
170+
176171
$order = $this->order->loadByIncrementId($orderNumber);
177172
/** @var Order\Item $orderItem */
178173
$orderItem = current($order->getAllItems());
179174
$orderItem->setQtyRefunded(1);
180175
$order->addItem($orderItem);
181176
$order->save();
182-
177+
// Create a credit memo
183178
$creditMemo = $this->creditMemoFactory->createByOrder($order, $order->getData());
184179
$creditMemo->setOrder($order);
185180
$creditMemo->setState(1);
@@ -252,13 +247,7 @@ public function testCreditMemoForBundledProductsWithPartialRefund()
252247
$creditMemos = $firstOrderItem['credit_memos'] ?? [];
253248
$this->assertResponseFields($creditMemos, $expectedCreditMemoData);
254249
$this->deleteOrder();
255-
$searchCriteria = $this->searchCriteriaBuilder->addFilter('increment_id', $orderNumber)
256-
->create();
257-
$creditmemoRepository = Bootstrap::getObjectManager()->get(CreditmemoRepositoryInterface::class);
258-
$creditmemos = $creditmemoRepository->getList($searchCriteria)->getItems();
259-
foreach ($creditmemos as $creditmemo) {
260-
$creditmemoRepository->delete($creditmemo);
261-
}
250+
$this->cleanUpCreditMemos($orderNumber);
262251
}
263252

264253
/**
@@ -307,7 +296,6 @@ public function testCreditMemoForBundleProductWithTaxesAndDiscounts()
307296
$creditMemo->save();
308297

309298
$this->creditMemoService->refund($creditMemo, true);
310-
//$this->prepareCreditmemoAndRefund($orderNumber);
311299
$response = $this->getCustomerOrderWithCreditMemoQuery();
312300
$expectedCreditMemoData = [
313301
[
@@ -377,13 +365,7 @@ public function testCreditMemoForBundleProductWithTaxesAndDiscounts()
377365
$creditMemos = $firstOrderItem['credit_memos'] ?? [];
378366
$this->assertResponseFields($creditMemos, $expectedCreditMemoData);
379367
$this->deleteOrder();
380-
$searchCriteria = $this->searchCriteriaBuilder->addFilter('increment_id', $orderNumber)
381-
->create();
382-
$creditmemoRepository = Bootstrap::getObjectManager()->get(CreditmemoRepositoryInterface::class);
383-
$creditmemos = $creditmemoRepository->getList($searchCriteria)->getItems();
384-
foreach ($creditmemos as $creditmemo) {
385-
$creditmemoRepository->delete($creditmemo);
386-
}
368+
$this->cleanUpCreditMemos($orderNumber);
387369
}
388370

389371
/**
@@ -729,6 +711,20 @@ private function deleteOrder(): void
729711
$registry->register('isSecureArea', false);
730712
}
731713

714+
/**
715+
* @param $orderNumber
716+
*/
717+
private function cleanUpCreditMemos($orderNumber): void
718+
{
719+
$creditmemoRepository = Bootstrap::getObjectManager()->get(CreditmemoRepositoryInterface::class);
720+
$searchCriteria = $this->searchCriteriaBuilder->addFilter('increment_id', $orderNumber)
721+
->create();
722+
$creditmemos = $creditmemoRepository->getList($searchCriteria)->getItems();
723+
foreach ($creditmemos as $creditmemo) {
724+
$creditmemoRepository->delete($creditmemo);
725+
}
726+
}
727+
732728
/**
733729
* Get CustomerOrder with credit memo details
734730
*

0 commit comments

Comments
 (0)