Skip to content

Commit 9546277

Browse files
author
Korshenko, Oleksii(okorshenko)
committed
Merge pull request #476 from magento-dragons/W3
[DRAGONS] BugFix(W3)
2 parents 1b5272c + dc2f6e6 commit 9546277

File tree

10 files changed

+26
-46
lines changed

10 files changed

+26
-46
lines changed

app/code/Magento/Catalog/view/adminhtml/web/js/options.js

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -41,9 +41,6 @@ define([
4141
data.intype = this.getOptionInputType();
4242
}
4343

44-
if (!this.totalItems) {
45-
data.checked = 'checked';
46-
}
4744
element = this.template({
4845
data: data
4946
});

app/code/Magento/ConfigurableProduct/view/adminhtml/templates/catalog/product/edit/attribute/steps/attributes_values.phtml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@
1111
<div data-bind="scope: '<?= /* @noEscape */ $block->getComponentName()?>'">
1212
<h2 class="steps-wizard-title"><?= /* @noEscape */ __('Step 2: Attribute Values') ?></h2>
1313
<div class="steps-wizard-info">
14-
<span><?= /* @noEscape */ __('Select values from each attribute to include in this product. Each unique combination of values creates a unigue product SKU.') ?></span>
14+
<span><?= $block->escapeHtml(__('Select from the following attribute values for this product. Each unique combination of values creates a unique product SKU.')) ?></span>
1515
</div>
1616
<div data-bind="foreach: attributes, sortableList: attributes">
1717

app/code/Magento/Swatches/view/adminhtml/templates/catalog/product/edit/attribute/steps/attributes_values.phtml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@
1111
<div data-bind="scope: '<?= /* @escapeNotVerified */ $block->getComponentName()?>'">
1212
<h2 class="steps-wizard-title"><?= /* @escapeNotVerified */ __('Step 2: Attribute Values') ?></h2>
1313
<div class="steps-wizard-info">
14-
<span><?= /* @escapeNotVerified */ __('Select values from each attribute to include in this product. Each unique combination of values creates a unigue product SKU.') ?></span>
14+
<span><?= $block->escapeHtml(__('Select from the following attribute values for this product. Each unique combination of values creates a unique product SKU.')) ?></span>
1515
</div>
1616
<div data-bind="foreach: attributes, sortableList: attributes">
1717

dev/tests/functional/tests/app/Magento/Catalog/Test/Block/Adminhtml/Category/Edit/CategoryForm.php

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -48,6 +48,7 @@ public function fill(FixtureInterface $fixture, SimpleElement $element = null)
4848
{
4949
if ($fixture->hasData('store_id')) {
5050
$store = $fixture->getStoreId();
51+
$this->browser->find($this->header)->hover();
5152
$storeSwitcherBlock = $this->browser->find($this->storeSwitcherBlock);
5253
$storeSwitcherBlock->find($this->dropdownBlock, Locator::SELECTOR_CSS, 'liselectstore')->setValue($store);
5354
$modalElement = $this->browser->find($this->confirmModal);

dev/tests/functional/tests/app/Magento/Catalog/Test/Block/Product/ProductList/TopToolbar.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ class TopToolbar extends Block
2828
*/
2929
public function getSelectSortType()
3030
{
31-
return $this->_rootElement->find($this->sorter)->getValue();
31+
return $this->_rootElement->find("#sorter")->getElements('option[selected]')[0]->getText();
3232
}
3333

3434
/**

dev/tests/functional/tests/app/Magento/Catalog/Test/Constraint/AssertCategoryPage.php

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -205,12 +205,12 @@ protected function verifyDisplaySettings(array $categoryData)
205205
//TODO: verify display_mode
206206

207207
if (isset($categoryData['default_sort_by'])) {
208-
$sortBy = strtolower($categoryData['default_sort_by']);
209-
$sortType = $this->categoryViewPage->getTopToolbar()->getSelectSortType();
210-
if ($sortBy != $sortType) {
208+
$expected = $categoryData['default_sort_by'];
209+
$actual = $this->categoryViewPage->getTopToolbar()->getSelectSortType();
210+
if ($expected != $actual) {
211211
$errorMessage[] = 'Wrong sorting type.'
212-
. "\nExpected: " . $sortBy
213-
. "\nActual: " . $sortType;
212+
. "\nExpected: " . $expected
213+
. "\nActual: " . $actual;
214214
}
215215
}
216216

@@ -257,7 +257,7 @@ protected function verifySearchEngineOptimization(array $categoryData)
257257
};
258258

259259
if (isset($categoryData['meta_title'])) {
260-
$actual = $this->categoryViewPage->getMetaInformation()->getTitle();
260+
$actual = $this->browser->getTitle();
261261
if ($categoryData['meta_title'] != $actual) {
262262
$errorMessage[] = 'Wrong page title.'
263263
. "\nExpected: " . $categoryData['meta_title']

dev/tests/functional/tests/app/Magento/Catalog/Test/Page/Category/CatalogCategoryView.xml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,6 @@
77
-->
88
<config xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:noNamespaceSchemaLocation="../../../../../../../vendor/magento/mtf/etc/pages.xsd">
99
<page name="CatalogCategoryView" area="Category" mca="catalog/category/view" module="Magento_Catalog">
10-
<block name="metaInformation" class="Magento\Cms\Test\Block\MetaInformation" locator="head" strategy="css selector" />
1110
<block name="topmenu" class="Magento\Theme\Test\Block\Html\Topmenu" locator=".navigation" strategy="css selector" />
1211
<block name="titleBlock" class="Magento\Theme\Test\Block\Html\Title" locator=".page-title-wrapper h1.page-title .base" strategy="css selector"/>
1312
<block name="breadcrumbs" class="Magento\Theme\Test\Block\Html\Breadcrumbs" locator=".breadcrumbs" strategy="css selector"/>

dev/tests/functional/tests/app/Magento/Catalog/Test/TestCase/Category/UpdateCategoryEntityTest.xml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -43,6 +43,7 @@
4343
<constraint name="Magento\Catalog\Test\Constraint\AssertCategoryIsNotActive" />
4444
</variation>
4545
<variation name="UpdateCategoryEntityTestVariation4_ChangeCategoryNameOnStoreView" summary="Update Category with custom Store View.">
46+
<data name="issue" xsi:type="string">Not implemented yet: MAGETWO-48516</data>
4647
<data name="category/data/store_id/dataset" xsi:type="string">custom</data>
4748
<data name="category/data/use_default_name" xsi:type="string">No</data>
4849
<data name="category/data/name" xsi:type="string">Category %isolation%</data>

dev/tests/functional/tests/app/Magento/Cms/Test/Block/MetaInformation.php

Lines changed: 0 additions & 33 deletions
This file was deleted.
Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,15 @@
1+
<?xml version="1.0"?>
2+
<!--
3+
/**
4+
* Copyright © 2016 Magento. All rights reserved.
5+
* See COPYING.txt for license details.
6+
*/
7+
-->
8+
<config xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
9+
xsi:noNamespaceSchemaLocation="../../../../../vendor/magento/mtf/Magento/Mtf/TestRunner/etc/testRunner.xsd">
10+
<rule scope="testsuite">
11+
<allow>
12+
<namespace value="Magento\Catalog\Test\TestCase\Category" />
13+
</allow>
14+
</rule>
15+
</config>

0 commit comments

Comments
 (0)