Skip to content

Commit 1a64984

Browse files
committed
MAGETWO-80353: PHP 7.1 compatibility: Jenkins Failures related to PHP 7.1 upgrade
- fix UpdateCustomVariableEntityTest test
1 parent c25732a commit 1a64984

File tree

2 files changed

+31
-6
lines changed

2 files changed

+31
-6
lines changed

dev/tests/functional/tests/app/Magento/Variable/Test/TestCase/UpdateCustomVariableEntityTest.php

Lines changed: 28 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@
1212
use Magento\Store\Test\Fixture\Store;
1313
use Magento\Mtf\Fixture\FixtureFactory;
1414
use Magento\Mtf\TestCase\Injectable;
15+
use Magento\Mtf\TestStep\TestStepFactory;
1516

1617
/**
1718
* Preconditions:
@@ -59,19 +60,36 @@ class UpdateCustomVariableEntityTest extends Injectable
5960
*/
6061
protected $store = null;
6162

63+
/**
64+
* Configuration setting.
65+
*
66+
* @var string
67+
*/
68+
private $configData;
69+
70+
/**
71+
* Factory for Test Steps.
72+
*
73+
* @var TestStepFactory
74+
*/
75+
private $testStepFactory;
76+
6277
/**
6378
* Injection data.
6479
*
6580
* @param SystemVariableIndex $systemVariableIndex
6681
* @param SystemVariableNew $systemVariableNew
82+
* @param TestStepFactory $testStepFactory
6783
* @return void
6884
*/
6985
public function __inject(
7086
SystemVariableIndex $systemVariableIndex,
71-
SystemVariableNew $systemVariableNew
87+
SystemVariableNew $systemVariableNew,
88+
TestStepFactory $testStepFactory
7289
) {
7390
$this->systemVariableIndexPage = $systemVariableIndex;
7491
$this->systemVariableNewPage = $systemVariableNew;
92+
$this->testStepFactory = $testStepFactory;
7593
}
7694

7795
/**
@@ -81,14 +99,22 @@ public function __inject(
8199
* @param SystemVariable $customVariable
82100
* @param SystemVariable $customVariableOrigin
83101
* @param string $saveAction
102+
* @param string|null $configData [optional]
84103
* @return array
85104
*/
86105
public function test(
87106
FixtureFactory $fixtureFactory,
88107
SystemVariable $customVariable,
89108
SystemVariable $customVariableOrigin,
90-
$saveAction
109+
$saveAction,
110+
$configData = null
91111
) {
112+
// Preconditions
113+
$this->configData = $configData;
114+
$this->testStepFactory->create(
115+
\Magento\Config\Test\TestStep\SetupConfigurationStep::class,
116+
['configData' => $this->configData]
117+
)->run();
92118
$this->store = $fixtureFactory->createByCode('store', ['dataset' => 'custom']);
93119
$this->store->persist();
94120
$customVariableOrigin->persist();

dev/tests/functional/tests/app/Magento/Variable/Test/TestCase/UpdateCustomVariableEntityTest.xml

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -19,8 +19,7 @@
1919
<constraint name="Magento\Variable\Test\Constraint\AssertCustomVariableInPage" />
2020
</variation>
2121
<variation name="UpdateCustomVariableEntityTestVariation2">
22-
<data name="tag" xsi:type="string">stable:no</data>
23-
<data name="issue" xsi:type="string">MAGETWO-80353: PHP 7.1 compatibility: Jenkins Failures related to PHP 7.1 upgrade</data>
22+
<data name="configData" xsi:type="string">add_store_code_to_urls</data>
2423
<data name="customVariable/data/code" xsi:type="string">variableCode%isolation%</data>
2524
<data name="customVariable/data/name" xsi:type="string">variableName%isolation%</data>
2625
<data name="customVariable/data/use_default_value" xsi:type="string">No</data>
@@ -32,6 +31,7 @@
3231
<constraint name="Magento\Variable\Test\Constraint\AssertCustomVariableInPage" />
3332
</variation>
3433
<variation name="UpdateCustomVariableEntityTestVariation3">
34+
<data name="configData" xsi:type="string">add_store_code_to_urls</data>
3535
<data name="customVariable/data/code" xsi:type="string">variableCode%isolation%</data>
3636
<data name="customVariable/data/name" xsi:type="string">variableName%isolation%</data>
3737
<data name="customVariable/data/use_default_value" xsi:type="string">No</data>
@@ -44,8 +44,7 @@
4444
<constraint name="Magento\Variable\Test\Constraint\AssertCustomVariableInPage" />
4545
</variation>
4646
<variation name="UpdateCustomVariableEntityTestVariation4">
47-
<data name="tag" xsi:type="string">stable:no</data>
48-
<data name="issue" xsi:type="string">MAGETWO-80353: PHP 7.1 compatibility: Jenkins Failures related to PHP 7.1 upgrade</data>
47+
<data name="configData" xsi:type="string">add_store_code_to_urls</data>
4948
<data name="customVariable/data/code" xsi:type="string">variableCode%isolation%</data>
5049
<data name="customVariable/data/name" xsi:type="string">variableName%isolation%</data>
5150
<data name="customVariable/data/use_default_value" xsi:type="string">No</data>

0 commit comments

Comments
 (0)