Fixes according to review

Author: VladimirZaets

app/code/Magento/Catalog/Helper/Output.php

@@ -219,7 +219,7 @@ public function categoryAttribute($category, $attributeHtml, $attributeName)
         if ($attributeHtml !== null
             && $attribute->getIsHtmlAllowedOnFront()
             && $attribute->getIsWysiwygEnabled()
-            && $this->isDirectivesExists($attributeHtml)
+            && $this->isDirectivesExists((string)$attributeHtml)
 
         ) {
             $attributeHtml = $this->_getTemplateProcessor()->filter($attributeHtml);
@@ -238,7 +238,7 @@ public function categoryAttribute($category, $attributeHtml, $attributeName)
      * @param string $attributeHtml
      * @return bool
      */
-    public function isDirectivesExists($attributeHtml)
+    public function isDirectivesExists(string $attributeHtml): bool
     {
         $matches = false;
         foreach ($this->directivePatterns as $pattern) {

app/code/Magento/Customer/Controller/Account/LoginPost.php

@@ -204,11 +204,11 @@ public function execute()
                         return $resultRedirect;
                     }
                 } catch (EmailNotConfirmedException $e) {
-                    $value = $this->customerUrl->getEmailConfirmationUrl($login['username']);
-                    $message = __(
-                        'This account is not confirmed. <a href="%1">Click here</a> to resend confirmation email.',
-                        $value
+                    $this->messageManager->addComplexErrorMessage(
+                        'confirmAccountErrorMessage',
+                        ['url' => $this->customerUrl->getEmailConfirmationUrl($login['username'])]
                     );
+                    $this->session->setUsername($login['username']);
                 } catch (AuthenticationException $e) {
                     $message = __(
                         'The account sign-in was incorrect or your account is disabled temporarily. '
@@ -223,7 +223,7 @@ public function execute()
                     );
                 } finally {
                     if (isset($message)) {
-                        $this->messageManager->addError($message);
+                        $this->messageManager->addErrorMessage($message);
                         $this->session->setUsername($login['username']);
                     }
                 }

app/code/Magento/Customer/Test/Unit/Controller/Account/LoginPostTest.php

@@ -551,14 +551,12 @@ protected function mockExceptions($exception, $username)
                     ->with($username)
                     ->willReturn($url);
 
-                $message = __(
-                    'This account is not confirmed.' .
-                    ' <a href="%1">Click here</a> to resend confirmation email.',
-                    $url
-                );
                 $this->messageManager->expects($this->once())
-                    ->method('addError')
-                    ->with($message)
+                    ->method('addComplexErrorMessage')
+                    ->with(
+                        'confirmAccountErrorMessage',
+                        ['url' => $url]
+                    )
                     ->willReturnSelf();
 
                 $this->session->expects($this->once())
@@ -569,7 +567,7 @@ protected function mockExceptions($exception, $username)
 
             case \Magento\Framework\Exception\AuthenticationException::class:
                 $this->messageManager->expects($this->once())
-                    ->method('addError')
+                    ->method('addErrorMessage')
                     ->with(
                         __(
                             'The account sign-in was incorrect or your account is disabled temporarily. '
@@ -597,7 +595,7 @@ protected function mockExceptions($exception, $username)
                     . 'Please wait and try again later.'
                 );
                 $this->messageManager->expects($this->once())
-                    ->method('addError')
+                    ->method('addErrorMessage')
                     ->with($message)
                     ->willReturnSelf();
                 $this->session->expects($this->once())

app/code/Magento/Customer/etc/frontend/di.xml

@@ -92,6 +92,12 @@
                         <item name="template" xsi:type="string">Magento_Customer::messages/confirmAccountSuccessMessage.phtml</item>
                     </item>
                 </item>
+                <item name="confirmAccountErrorMessage" xsi:type="array">
+                    <item name="renderer" xsi:type="const">\Magento\Framework\View\Element\Message\Renderer\BlockRenderer::CODE</item>
+                    <item name="data" xsi:type="array">
+                        <item name="template" xsi:type="string">Magento_Customer::messages/confirmAccountErrorMessage.phtml</item>
+                    </item>
+                </item>
                 <item name="customerVatShippingAddressSuccessMessage" xsi:type="array">
                     <item name="renderer" xsi:type="const">\Magento\Framework\View\Element\Message\Renderer\BlockRenderer::CODE</item>
                     <item name="data" xsi:type="array">

app/code/Magento/Customer/view/frontend/templates/messages/confirmAccountErrorMessage.phtml

@@ -0,0 +1,11 @@
+<?php
+/**
+ * Copyright © Magento, Inc. All rights reserved.
+ * See COPYING.txt for license details.
+ */
+
+/** @var \Magento\Framework\View\Element\Template $block */
+/** @var \Magento\Framework\Escaper $escaper */
+
+?>
+<?= $escaper->escapeHtml(__('This account is not confirmed. <a href="%1">Click here</a> to resend confirmation email.', $block->getData('url')), ['a']);

dev/tests/integration/testsuite/Magento/Catalog/Helper/OutputTest.php

@@ -122,7 +122,7 @@ protected function _testAttribute($method, $entityCode, $expectedResult)
         try {
             $this->assertEquals(
                 $expectedResult,
-                $this->_helper->{$method}(uniqid(), "<p>line1</p>\nline2", $attributeName)
+                $this->_helper->{$method}(uniqid(), __("<p>line1</p>\nline2"), $attributeName)
             );
 
             $attribute->setIsHtmlAllowedOnFront($isHtml)->setIsWysiwygEnabled($isWysiwyg);

dev/tests/integration/testsuite/Magento/Customer/Controller/Account/LoginPostTest.php

@@ -28,6 +28,11 @@ class LoginPostTest extends AbstractController
     /** @var EncoderInterface */
     private $urlEncoder;
 
+    /**
+     * @var Url
+     */
+    private $customerUrl;
+
     /**
      * @inheritdoc
      */
@@ -37,6 +42,7 @@ protected function setUp()
 
         $this->session = $this->_objectManager->get(Session::class);
         $this->urlEncoder = $this->_objectManager->get(EncoderInterface::class);
+        $this->customerUrl = $this->_objectManager->get(Url::class);
     }
 
     /**
@@ -107,13 +113,16 @@ public function missingParametersDataProvider(): array
      */
     public function testLoginWithUnconfirmedPassword(): void
     {
-        $this->markTestSkipped('Blocked by MC-31370.');
         $email = 'unconfirmedcustomer@example.com';
         $this->prepareRequest($email, 'Qwert12345');
         $this->dispatch('customer/account/loginPost');
         $this->assertEquals($email, $this->session->getUsername());
+        $message = __(
+            'This account is not confirmed. <a href="%1">Click here</a> to resend confirmation email.',
+            $this->customerUrl->getEmailConfirmationUrl($this->session->getUsername())
+        );
         $this->assertSessionMessages(
-            $this->equalTo([(string)__('This account is not confirmed. Click here to resend confirmation email.')]),
+            $this->equalTo([(string)$message]),
             MessageInterface::TYPE_ERROR
         );
     }