Skip to content

Commit 0eb4bcc

Browse files
committed
unit test case issue fix for checkout agreements AC-12479
1 parent 45c8940 commit 0eb4bcc

File tree

2 files changed

+2
-7
lines changed

2 files changed

+2
-7
lines changed

app/code/Magento/CheckoutAgreements/Model/AgreementsConfigProvider.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -103,7 +103,7 @@ protected function getAgreementsConfig()
103103
: nl2br($this->escaper->escapeHtml($agreement->getContent())),
104104
'checkboxText' => $isAgreementHtmlType
105105
? $agreement->getCheckboxText()
106-
: $this->escaper->escapeHtml($agreement->getCheckboxText()),
106+
: nl2br($this->escaper->escapeHtml($agreement->getCheckboxText())),
107107
'mode' => $agreement->getMode(),
108108
'agreementId' => $agreement->getAgreementId(),
109109
'contentHeight' => $agreement->getContentHeight()

app/code/Magento/CheckoutAgreements/Test/Unit/Model/AgreementsConfigProviderTest.php

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -98,7 +98,7 @@ public function testGetConfigIfContentIsHtml(): void
9898
'agreements' => [
9999
[
100100
'content' => $content,
101-
'checkboxText' => $escapedCheckboxText,
101+
'checkboxText' => $checkboxText,
102102
'mode' => $mode,
103103
'agreementId' => $agreementId,
104104
'contentHeight' => $contentHeight
@@ -122,11 +122,6 @@ public function testGetConfigIfContentIsHtml(): void
122122
->with($searchCriteriaMock)
123123
->willReturn([$agreement]);
124124

125-
$this->escaperMock->expects($this->once())
126-
->method('escapeHtml')
127-
->with($checkboxText)
128-
->willReturn($escapedCheckboxText);
129-
130125
$agreement->expects($this->once())->method('getIsHtml')->willReturn(true);
131126
$agreement->expects($this->once())->method('getContent')->willReturn($content);
132127
$agreement->expects($this->once())->method('getCheckboxText')->willReturn($checkboxText);

0 commit comments

Comments
 (0)