Skip to content

Commit a756249

Browse files
author
Joan He
committed
Merge remote-tracking branch 'origin/MAGETWO-90719-reorder-exception' into BugFixPR
2 parents a7893f9 + 6a86074 commit a756249

File tree

4 files changed

+14
-13
lines changed

4 files changed

+14
-13
lines changed

app/code/Magento/Catalog/Test/Mftf/Test/StorefrontPurchaseProductWithCustomOptions.xml

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -16,9 +16,7 @@
1616
<description value="Admin should be able to sell products with different variants of their own"/>
1717
<severity value="CRITICAL"/>
1818
<testCaseId value="MAGETWO-61717"/>
19-
<skip>
20-
<issueId value="MAGETWO-90719"/>
21-
</skip>
19+
<group value="Catalog"/>
2220
</annotations>
2321
<before>
2422
<createData entity="Simple_US_Customer" stepKey="createCustomer"/>

composer.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@
3838
"elasticsearch/elasticsearch": "~2.0|~5.1",
3939
"magento/composer": "~1.4.0",
4040
"magento/magento-composer-installer": ">=0.1.11",
41-
"magento/zendframework1": "~1.14.0",
41+
"magento/zendframework1": "~1.14.1",
4242
"monolog/monolog": "^1.17",
4343
"oyejorge/less.php": "~1.7.0",
4444
"pelago/emogrifier": "^2.0.0",

composer.lock

Lines changed: 6 additions & 6 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

dev/tests/integration/testsuite/Magento/Catalog/Model/Product/Option/Type/File/ValidatorInfoTest.php

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -30,6 +30,9 @@ class ValidatorInfoTest extends \PHPUnit\Framework\TestCase
3030
*/
3131
protected $validateFactoryMock;
3232

33+
/**
34+
* {@inheritdoc}
35+
*/
3336
protected function setUp()
3437
{
3538
$this->objectManager = \Magento\TestFramework\Helper\Bootstrap::getObjectManager();
@@ -111,11 +114,11 @@ public function testExceptionWithoutErrors()
111114
*/
112115
public function testValidate()
113116
{
114-
$validateMock = $this->createPartialMock(\Zend_Validate::class, ['isValid']);
115-
$validateMock->expects($this->once())->method('isValid')->will($this->returnValue(true));
117+
//use actual zend class to test changed functionality
118+
$validate = $this->objectManager->create(\Zend_Validate::class);
116119
$this->validateFactoryMock->expects($this->once())
117120
->method('create')
118-
->will($this->returnValue($validateMock));
121+
->will($this->returnValue($validate));
119122
$this->assertTrue(
120123
$this->model->validate(
121124
$this->getOptionValue(),

0 commit comments

Comments
 (0)