Skip to content

Commit 8111a0f

Browse files
MTO-127: [Test] Import Advanced Pricing
- Stabilization
1 parent 56b19c5 commit 8111a0f

File tree

7 files changed

+18
-21
lines changed

7 files changed

+18
-21
lines changed

dev/tests/functional/tests/app/Magento/AdvancedPricingImportExport/Test/Constraint/AssertImportAdvancedPricing.php

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -86,9 +86,6 @@ private function getPreparePrices()
8686
foreach ($tierPrices as $tierPrice) {
8787
$resultProductArray[$productSku][] = $tierPrice;
8888
}
89-
if (isset($resultProductArray[$productSku])) {
90-
$resultProductArray[$productSku] = array_reverse($resultProductArray[$productSku]);
91-
}
9289
}
9390

9491
// Prepare tier prices data from csv file.

dev/tests/functional/tests/app/Magento/AdvancedPricingImportExport/Test/TestCase/ImportDataTest.xml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@
2626
</item>
2727
</item>
2828
</data>
29-
<constraint name="Magento\AdvancedPricingImportExport\Test\Constraint\AssertImportSuccessMessage" />
29+
<constraint name="Magento\ImportExport\Test\Constraint\AssertImportSuccessMessage" />
3030
<constraint name="Magento\AdvancedPricingImportExport\Test\Constraint\AssertImportAdvancedPricing" />
3131
</variation>
3232
<variation name="ImportProductsVariation2" ticketId="MAGETWO-46156" summary="Replace product tier prices">
@@ -48,7 +48,7 @@
4848
</item>
4949
</item>
5050
</data>
51-
<constraint name="Magento\AdvancedPricingImportExport\Test\Constraint\AssertImportSuccessMessage" />
51+
<constraint name="Magento\ImportExport\Test\Constraint\AssertImportSuccessMessage" />
5252
<constraint name="Magento\AdvancedPricingImportExport\Test\Constraint\AssertImportAdvancedPricing" />
5353
</variation>
5454
<variation name="ImportProductsVariation3" ticketId="MAGETWO-46157" summary="Delete product tier prices">
@@ -70,7 +70,7 @@
7070
</item>
7171
</item>
7272
</data>
73-
<constraint name="Magento\AdvancedPricingImportExport\Test\Constraint\AssertImportSuccessMessage" />
73+
<constraint name="Magento\ImportExport\Test\Constraint\AssertImportSuccessMessage" />
7474
<constraint name="Magento\AdvancedPricingImportExport\Test\Constraint\AssertImportAdvancedPricing" />
7575
</variation>
7676
<variation name="ImportProductsVariation4" ticketId="MAGETWO-46159" summary="Replace product tier prices in two websites">

dev/tests/functional/tests/app/Magento/AdvancedPricingImportExport/Test/_files/template/pricing/advanced_price_variation_1.php

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -9,17 +9,17 @@
99
'data_0' => [
1010
'sku' => '%sku%',
1111
'tier_price_website' => "All Websites [USD]",
12-
'tier_price_customer_group' => 'General',
12+
'tier_price_customer_group' => "NOT LOGGED IN",
1313
'tier_price_qty' => '10',
14-
'tier_price' => '8.00',
14+
'tier_price' => '9.00',
1515
'tier_price_value_type' => 'Fixed',
1616
],
1717
'data_1' => [
1818
'sku' => '%sku%',
1919
'tier_price_website' => "All Websites [USD]",
20-
'tier_price_customer_group' => "NOT LOGGED IN",
20+
'tier_price_customer_group' => 'General',
2121
'tier_price_qty' => '10',
22-
'tier_price' => '9.00',
22+
'tier_price' => '8.00',
2323
'tier_price_value_type' => 'Fixed',
2424
]
2525
],
Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
* Copyright © 2013-2017 Magento, Inc. All rights reserved.
44
* See COPYING.txt for license details.
55
*/
6-
namespace Magento\AdvancedPricingImportExport\Test\Constraint;
6+
namespace Magento\ImportExport\Test\Constraint;
77

88
use Magento\ImportExport\Test\Page\Adminhtml\AdminImportIndex;
99
use Magento\Mtf\Constraint\AbstractConstraint;

dev/tests/functional/tests/app/Magento/ImportExport/Test/Fixture/Import/File.php

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -206,15 +206,15 @@ private function preparePlaceHolders()
206206
$key = 0;
207207
foreach ($this->entities as $entity) {
208208
$entityData = $this->prepareEntityData($entity);
209-
foreach ($this->csvTemplate['entity_' . $key] as $tierKey => $tier) {
210-
$values = implode('', array_values($tier));
209+
foreach ($this->csvTemplate['entity_' . $key] as $entityKey => $importedEntityData) {
210+
$values = implode('', array_values($importedEntityData));
211211
preg_match_all('/\%(.*)\%/U', $values, $indexes);
212212
foreach ($indexes[1] as $index) {
213213
if (isset($entityData[$index])) {
214-
$placeholders['entity_' . $key][$tierKey]["%{$index}%"] = $entityData[$index];
214+
$placeholders['entity_' . $key][$entityKey]["%{$index}%"] = $entityData[$index];
215215
}
216216
if (isset($entityData['code'])) {
217-
$placeholders['entity_' . $key][$tierKey][$entityData['code']]
217+
$placeholders['entity_' . $key][$entityKey][$entityData['code']]
218218
= isset($entityData[$entityData['code']])
219219
? $entityData[$entityData['code']]
220220
: 'Main Website';

dev/tests/functional/tests/app/Magento/ImportExport/Test/Page/Adminhtml/AdminExportIndex.xml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,8 +7,8 @@
77
-->
88
<config xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:noNamespaceSchemaLocation="../../../../../../../vendor/magento/mtf/etc/pages.xsd">
99
<page name="AdminExportIndex" area="Adminhtml" mca="admin/export/index" module="Magento_ImportExport">
10-
<block name="filterExport" class="Magento\ImportExport\Test\Block\Adminhtml\Export\Filter" locator="#export_filter_container" strategy="css selector"/>
11-
<block name="exportForm" class="Magento\ImportExport\Test\Block\Adminhtml\Export\Edit\Form" locator="#container" strategy="css selector"/>
10+
<block name="filterExport" class="Magento\ImportExport\Test\Block\Adminhtml\Export\Filter" locator="#export_filter_container" strategy="css selector" />
11+
<block name="exportForm" class="Magento\ImportExport\Test\Block\Adminhtml\Export\Edit\Form" locator="#container" strategy="css selector" />
1212
<block name="messagesBlock" class="Magento\Backend\Test\Block\Messages" locator="#messages" strategy="css selector" />
1313
</page>
1414
</config>

dev/tests/functional/tests/app/Magento/ImportExport/Test/TestStep/ChangeCurrencyOnCustomWebsiteStep.php

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@
1010
use Magento\Mtf\Fixture\FixtureFactory;
1111

1212
/**
13-
* Create custom store step.
13+
* Change currency on custom website step.
1414
*/
1515
class ChangeCurrencyOnCustomWebsiteStep implements TestStepInterface
1616
{
@@ -38,12 +38,12 @@ class ChangeCurrencyOnCustomWebsiteStep implements TestStepInterface
3838
/**
3939
* @param FixtureFactory $fixtureFactory
4040
* @param ImportData $import
41-
* @param bool|null $changeCurrency
41+
* @param bool $changeCurrency
4242
*/
4343
public function __construct(
4444
FixtureFactory $fixtureFactory,
4545
ImportData $import,
46-
$changeCurrency
46+
$changeCurrency = false
4747
) {
4848
$this->fixtureFactory = $fixtureFactory;
4949
$this->import = $import;
@@ -53,7 +53,7 @@ public function __construct(
5353
/**
5454
* Fill import form.
5555
*
56-
* @return void
56+
* @return array
5757
*/
5858
public function run()
5959
{

0 commit comments

Comments
 (0)