Skip to content

Commit 2329a14

Browse files
committed
Merge branch 'ACP2E-3460' of https://github.com/adobe-commerce-tier-4/magento2ce into PR-12-05-2024
2 parents ae1b356 + f9c5f0e commit 2329a14

File tree

3 files changed

+31
-17
lines changed

3 files changed

+31
-17
lines changed

app/code/Magento/Sales/Model/Order/Creditmemo/Sender/EmailSender.php

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
<?php
22
/**
3-
* Copyright © Magento, Inc. All rights reserved.
4-
* See COPYING.txt for license details.
3+
* Copyright 2016 Adobe
4+
* All Rights Reserved.
55
*/
66
namespace Magento\Sales\Model\Order\Creditmemo\Sender;
77

@@ -104,7 +104,9 @@ public function send(
104104

105105
$transport = [
106106
'order' => $order,
107+
'order_id' => $order->getId(),
107108
'creditmemo' => $creditmemo,
109+
'creditmemo_id' => $creditmemo->getId(),
108110
'comment' => $comment ? $comment->getComment() : '',
109111
'billing' => $order->getBillingAddress(),
110112
'payment_html' => $this->getPaymentHtml($order),

app/code/Magento/Sales/Test/Unit/Model/Order/Creditmemo/Sender/EmailSenderTest.php

Lines changed: 13 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
<?php
22
/**
3-
* Copyright © Magento, Inc. All rights reserved.
4-
* See COPYING.txt for license details.
3+
* Copyright 2016 Adobe
4+
* All Rights Reserved.
55
*/
66
declare(strict_types=1);
77

@@ -120,6 +120,10 @@ class EmailSenderTest extends TestCase
120120
*/
121121
private $senderBuilderFactoryMock;
122122

123+
private const CREDITMEMO_ID = 1;
124+
125+
private const ORDER_ID = 1;
126+
123127
/**
124128
* @inheritDoc
125129
*
@@ -154,9 +158,11 @@ protected function setUp(): void
154158

155159
$this->creditmemoMock = $this->getMockBuilder(\Magento\Sales\Model\Order\Creditmemo::class)
156160
->disableOriginalConstructor()
157-
->onlyMethods(['setEmailSent'])
161+
->onlyMethods(['setEmailSent', 'getId'])
158162
->addMethods(['setSendEmail'])
159163
->getMock();
164+
$this->creditmemoMock->method('getId')
165+
->willReturn(self::CREDITMEMO_ID);
160166

161167
$this->commentMock = $this->getMockBuilder(CreditmemoCommentCreationInterface::class)
162168
->disableOriginalConstructor()
@@ -192,6 +198,8 @@ protected function setUp(): void
192198
$this->orderMock->expects($this->any())
193199
->method('getPayment')
194200
->willReturn($this->paymentInfoMock);
201+
$this->orderMock->method('getId')
202+
->willReturn(self::ORDER_ID);
195203

196204
$this->paymentHelperMock = $this->getMockBuilder(Data::class)
197205
->disableOriginalConstructor()
@@ -285,7 +293,9 @@ public function testSend(
285293
if (!$configValue || $forceSyncMode) {
286294
$transport = [
287295
'order' => $this->orderMock,
296+
'order_id' => self::ORDER_ID,
288297
'creditmemo' => $this->creditmemoMock,
298+
'creditmemo_id' => self::CREDITMEMO_ID,
289299
'comment' => $isComment ? 'Comment text' : '',
290300
'billing' => $this->addressMock,
291301
'payment_html' => 'Payment Info Block',
Lines changed: 14 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1,39 +1,40 @@
11
<?php
22
/**
3-
* Copyright © Magento, Inc. All rights reserved.
4-
* See COPYING.txt for license details.
3+
* Copyright 2015 Adobe
4+
* All Rights Reserved.
55
*/
66
namespace Magento\Sales\Service\V1;
77

8+
use Magento\Framework\Webapi\Rest\Request;
9+
use Magento\Sales\Model\ResourceModel\Order\Creditmemo\Collection;
10+
use Magento\TestFramework\Helper\Bootstrap;
811
use Magento\TestFramework\TestCase\WebapiAbstract;
912

1013
/**
11-
* Class CreditmemoEmailTest
14+
* Test API call /creditmemo/{id}/emails
1215
*/
1316
class CreditmemoEmailTest extends WebapiAbstract
1417
{
15-
const SERVICE_VERSION = 'V1';
18+
private const SERVICE_VERSION = 'V1';
1619

17-
const SERVICE_NAME = 'salesCreditmemoManagementV1';
18-
19-
const CREDITMEMO_INCREMENT_ID = '100000001';
20+
private const SERVICE_NAME = 'salesCreditmemoManagementV1';
2021

2122
/**
2223
* @magentoApiDataFixture Magento/Sales/_files/creditmemo_with_list.php
2324
*/
2425
public function testCreditmemoEmail()
2526
{
26-
$objectManager = \Magento\TestFramework\Helper\Bootstrap::getObjectManager();
27+
$objectManager = Bootstrap::getObjectManager();
2728

28-
/** @var \Magento\Sales\Model\ResourceModel\Order\Creditmemo\Collection $creditmemoCollection */
29+
/** @var Collection $creditmemoCollection */
2930
$creditmemoCollection = $objectManager->get(
30-
\Magento\Sales\Model\ResourceModel\Order\Creditmemo\Collection::class
31+
Collection::class
3132
);
3233
$creditmemo = $creditmemoCollection->getFirstItem();
3334
$serviceInfo = [
3435
'rest' => [
3536
'resourcePath' => '/V1/creditmemo/' . $creditmemo->getId() . '/emails',
36-
'httpMethod' => \Magento\Framework\Webapi\Rest\Request::HTTP_METHOD_POST,
37+
'httpMethod' => Request::HTTP_METHOD_POST,
3738
],
3839
'soap' => [
3940
'service' => self::SERVICE_NAME,
@@ -42,6 +43,7 @@ public function testCreditmemoEmail()
4243
],
4344
];
4445
$requestData = ['id' => $creditmemo->getId()];
45-
$this->_webApiCall($serviceInfo, $requestData);
46+
$result = $this->_webApiCall($serviceInfo, $requestData);
47+
$this->assertTrue($result);
4648
}
4749
}

0 commit comments

Comments
 (0)