Skip to content

Commit f811cbc

Browse files
author
Eric Bohanon
committed
MAGETWO-70779: Add URL Rewrite not retrieving categories
- Functional Test
1 parent 539f6cd commit f811cbc

File tree

4 files changed

+13
-22
lines changed

4 files changed

+13
-22
lines changed

dev/tests/functional/tests/app/Magento/Catalog/Test/Repository/Category.xml

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -45,6 +45,16 @@
4545
<field name="include_in_menu" xsi:type="string">Yes</field>
4646
</dataset>
4747

48+
<dataset name="default_subcategory_with_single_quote_in_name">
49+
<field name="name" xsi:type="string">DefaultSubcategory\'%isolation%</field>
50+
<field name="url_key" xsi:type="string">default-subcategory-%isolation%</field>
51+
<field name="parent_id" xsi:type="array">
52+
<item name="dataset" xsi:type="string">default_category</item>
53+
</field>
54+
<field name="is_active" xsi:type="string">Yes</field>
55+
<field name="include_in_menu" xsi:type="string">Yes</field>
56+
</dataset>
57+
4858
<dataset name="default_subcategory_without_url_key">
4959
<field name="name" xsi:type="string">DefaultSubcategory%isolation%</field>
5060
<field name="parent_id" xsi:type="array">
@@ -169,15 +179,5 @@
169179
<item name="dataset" xsi:type="string">catalogProductSimple::default</item>
170180
</field>
171181
</dataset>
172-
173-
<dataset name="default_with_single_quote_in_name">
174-
<field name="name" xsi:type="string">Category\'%isolation%</field>
175-
<field name="url_key" xsi:type="string">category%isolation%</field>
176-
<field name="is_active" xsi:type="string">Yes</field>
177-
<field name="include_in_menu" xsi:type="string">Yes</field>
178-
<field name="parent_id" xsi:type="array">
179-
<item name="dataset" xsi:type="string">default_category</item>
180-
</field>
181-
</dataset>
182182
</repository>
183183
</config>

dev/tests/functional/tests/app/Magento/UrlRewrite/Test/Block/Adminhtml/Catalog/Category/Tree.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ public function selectCategory($category)
3232
{
3333
if ($category != null && $category->hasData('name')) {
3434
$this->_rootElement->find(
35-
"//a[contains(text(),'{$category->getName()}')]",
35+
"//a[contains(text(),\"{$category->getName()}\")]",
3636
Locator::SELECTOR_XPATH
3737
)->click();
3838
} else {

dev/tests/functional/tests/app/Magento/UrlRewrite/Test/TestCase/CategoryUrlRewriteTest.xml

Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -17,15 +17,5 @@
1717
<data name="categoryUpdates/data/url_key" xsi:type="string">UrlKey%isolation%</data>
1818
<constraint name="Magento\UrlRewrite\Test\Constraint\AssertCategoryUrlWithCustomStoreView" />
1919
</variation>
20-
<variation name="CategoryUrlRewriteTestVariation2">
21-
<data name="storeView/dataset" xsi:type="string">custom</data>
22-
<data name="childCategory/dataset" xsi:type="string">default</data>
23-
<data name="childCategory/data/category_products/dataset" xsi:type="string">catalogProductSimple::default_with_single_quote_in_name</data>
24-
<data name="parentCategory/dataset" xsi:type="string">default</data>
25-
<data name="parentCategory/data/category_products/dataset" xsi:type="string">catalogProductSimple::default_with_single_quote_in_name</data>
26-
<data name="categoryUpdates/data/use_default_url_key" xsi:type="string">No</data>
27-
<data name="categoryUpdates/data/url_key" xsi:type="string">UrlKey%isolation%</data>
28-
<constraint name="Magento\UrlRewrite\Test\Constraint\AssertCategoryUrlWithCustomStoreView" />
29-
</variation>
3020
</testCase>
3121
</config>

dev/tests/functional/tests/app/Magento/UrlRewrite/Test/TestCase/CreateCategoryRewriteEntityTest.php

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -68,7 +68,7 @@ public function __inject(
6868
$this->urlRewriteIndex = $urlRewriteIndex;
6969
$category = $fixtureFactory->createByCode(
7070
'category',
71-
['dataset' => 'default_subcategory']
71+
['dataset' => 'default_subcategory_with_single_quote_in_name']
7272
);
7373
$category->persist();
7474
return ['category' => $category];
@@ -87,6 +87,7 @@ public function test(UrlRewrite $urlRewrite, Category $category)
8787
$this->urlRewriteIndex->open();
8888
$this->urlRewriteIndex->getPageActionsBlock()->addNew();
8989
$this->urlRewriteEdit->getFormBlock()->fill($urlRewrite);
90+
9091
$this->urlRewriteEdit->getTreeBlock()->selectCategory($category);
9192
$this->urlRewriteEdit->getFormBlock()->fill($urlRewrite);
9293
$this->urlRewriteEdit->getPageMainActions()->save();

0 commit comments

Comments
 (0)