Skip to content

Commit af2a832

Browse files
committed
MAGETWO-46014: delayed error messages
1 parent eec31f6 commit af2a832

File tree

2 files changed

+43
-5
lines changed

2 files changed

+43
-5
lines changed

dev/tests/integration/framework/Magento/TestFramework/TestCase/AbstractController.php

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -228,7 +228,8 @@ public function assertSessionMessages(
228228
* @param string|null $messageType
229229
* @return array
230230
*/
231-
private function getCookieMessages($messageType = null) {
231+
private function getCookieMessages($messageType = null)
232+
{
232233
/** @var $cookieManager CookieManagerInterface */
233234
$cookieManager = $this->_objectManager->get(CookieManagerInterface::class);
234235
try {

dev/tests/integration/framework/tests/unit/testsuite/Magento/Test/TestCase/ControllerAbstractTest.php

Lines changed: 41 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@
44
* See COPYING.txt for license details.
55
*/
66
namespace Magento\Test\TestCase;
7+
use Magento\Framework\Stdlib\CookieManagerInterface;
78

89
/**
910
* @SuppressWarnings(PHPMD.CouplingBetweenObjects)
@@ -15,11 +16,15 @@ class ControllerAbstractTest extends \Magento\TestFramework\TestCase\AbstractCon
1516
/** @var \PHPUnit_Framework_MockObject_MockObject | \Magento\Framework\Message\Manager */
1617
private $messageManager;
1718

19+
/** @var \PHPUnit_Framework_MockObject_MockObject | CookieManagerInterface */
20+
private $cookieManagerMock;
21+
1822
protected function setUp()
1923
{
2024
$testObjectManager = new \Magento\Framework\TestFramework\Unit\Helper\ObjectManager($this);
2125

2226
$this->messageManager = $this->getMock('\Magento\Framework\Message\Manager', [], [], '', false);
27+
$this->cookieManagerMock = $this->getMock(CookieManagerInterface::class, [], [], '', false);
2328
$request = $testObjectManager->getObject('Magento\TestFramework\Request');
2429
$response = $testObjectManager->getObject('Magento\TestFramework\Response');
2530
$this->_objectManager = $this->getMock(
@@ -37,6 +42,7 @@ protected function setUp()
3742
['Magento\Framework\App\RequestInterface', $request],
3843
['Magento\Framework\App\ResponseInterface', $response],
3944
['Magento\Framework\Message\Manager', $this->messageManager],
45+
[CookieManagerInterface::class, $this->cookieManagerMock],
4046
]
4147
)
4248
);
@@ -139,16 +145,20 @@ public function testAssertSessionMessagesSuccess(array $expectedMessages, $messa
139145
public function assertSessionMessagesDataProvider()
140146
{
141147
return [
142-
'message waning type filtering' => [
143-
['some_warning'],
148+
'message warning type filtering' => [
149+
['some_warning', 'warning_cookie'],
144150
\Magento\Framework\Message\MessageInterface::TYPE_WARNING,
145151
],
146152
'message error type filtering' => [
147-
['error_one', 'error_two'],
153+
['error_one', 'error_two', 'error_cookie'],
148154
\Magento\Framework\Message\MessageInterface::TYPE_ERROR,
149155
],
156+
'message notice type filtering' => [
157+
['some_notice', 'notice_cookie'],
158+
\Magento\Framework\Message\MessageInterface::TYPE_NOTICE,
159+
],
150160
'message success type filtering' => [
151-
['success!'],
161+
['success!', 'success_cookie'],
152162
\Magento\Framework\Message\MessageInterface::TYPE_SUCCESS,
153163
],
154164
];
@@ -166,6 +176,10 @@ public function testAssertSessionMessagesAll()
166176
'error_two',
167177
'some_notice',
168178
'success!',
179+
'warning_cookie',
180+
'notice_cookie',
181+
'success_cookie',
182+
'error_cookie',
169183
]
170184
)
171185
);
@@ -192,5 +206,28 @@ private function addSessionMessages()
192206
->addMessage(new \Magento\Framework\Message\Success('success!'));
193207
$this->messageManager->expects($this->any())->method('getMessages')
194208
->will($this->returnValue($messagesCollection));
209+
210+
$cookieMessages = [
211+
[
212+
'type' => 'warning',
213+
'text' => 'warning_cookie',
214+
],
215+
[
216+
'type' => 'notice',
217+
'text' => 'notice_cookie',
218+
],
219+
[
220+
'type' => 'success',
221+
'text' => 'success_cookie',
222+
],
223+
[
224+
'type' => 'error',
225+
'text' => 'error_cookie',
226+
],
227+
];
228+
229+
$this->cookieManagerMock->expects($this->any())
230+
->method('getCookie')
231+
->willReturn(\Zend_Json::encode($cookieMessages));
195232
}
196233
}

0 commit comments

Comments
 (0)