Skip to content

Commit b0d460a

Browse files
author
Hwashiang Yu
committed
MC-5810: Improve naming of the critical variables/parameters in the code and configuration
- Resolved pagebuilder product rule tree failure - Updated google maps warning message test
1 parent faf69fb commit b0d460a

File tree

3 files changed

+8
-6
lines changed

3 files changed

+8
-6
lines changed

app/code/Magento/PageBuilder/Test/Mftf/ActionGroup/ContentTypeMapActionGroup.xml

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -409,6 +409,11 @@
409409
<seeInCurrentUrl url="{{AdminContentManagementPage.url}}#cms_pagebuilder" stepKey="validateURL"/>
410410
<seeElement selector="{{ContentManagementSection.GoogleMapsAPIKeyInputField}}" stepKey="seeGoogleMapsAPIKeyInputField"/>
411411
</actionGroup>
412+
<actionGroup name="validateWarningMessageLinkNewTabTarget" extends="validateWarningMessageLink">
413+
<switchToNextTab stepKey="switchToNewConfigurationTab" after="clickLink"/>
414+
<closeTab stepKey="closeTab" after="seeGoogleMapsAPIKeyInputField"/>
415+
<switchToPreviousTab stepKey="switchToPreviousTab" after="closeTab"/>
416+
</actionGroup>
412417
<actionGroup name="validateAdvancedStyleWithAllUpdatedMapWithErrorOverlay" extends="validateAdvancedStyleWithAllUpdatedNotVisible">
413418
<waitForElementVisible selector="{{page.warningMessageLink(index)}}" stepKey="dontSeeElement"/>
414419
</actionGroup>

app/code/Magento/PageBuilder/Test/Mftf/Test/AdminPageBuilderMapGoogleMapsSettingsTests.xml

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -129,12 +129,9 @@
129129
<actionGroup ref="switchToPageBuilderStage" stepKey="switchToPageBuilderStage1"/>
130130
<!-- Stage: Validate Warning Message Link -->
131131
<comment userInput="Stage: Validate Warning Message Link" stepKey="commentValidateWarningMessageLinkStage"/>
132-
<actionGroup ref="validateWarningMessageLink" stepKey="validateWarningMessageLink1">
132+
<actionGroup ref="validateWarningMessageLinkNewTabTarget" stepKey="validateWarningMessageLink1">
133133
<argument name="section" value="MapOnStage"/>
134134
</actionGroup>
135-
<moveBack stepKey="moveBack1"/>
136-
<waitForPageLoad stepKey="waitForPageLoad1"/>
137-
<actionGroup ref="switchToPageBuilderStage" stepKey="switchToPageBuilderStage2"/>
138135
<!-- Edit Form: Validate Warning Message Link -->
139136
<comment userInput="Edit Form: Validate Warning Message Link" stepKey="commentValidateWarningMessageLinkEditForm"/>
140137
<actionGroup ref="openPageBuilderEditPanel" stepKey="openEditPanel1">
@@ -152,7 +149,7 @@
152149
<argument name="contentType" value="PageBuilderMapContentType"/>
153150
</actionGroup>
154151
<actionGroup ref="clickAddLocationButton" stepKey="clickAddLocationButton"/>
155-
<actionGroup ref="validateWarningMessageLink" stepKey="validateWarningMessageLink3">
152+
<actionGroup ref="validateWarningMessageLinkNewTabTarget" stepKey="validateWarningMessageLink3">
156153
<argument name="section" value="AddLocationForm"/>
157154
</actionGroup>
158155
</test>

app/code/Magento/PageBuilder/view/adminhtml/templates/form/element/conditions.phtml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ $role = 'pagebuilder-conditions-form-placeholder-' . $attribute;
1717
<div class="rule-tree">
1818
<fieldset id="<?= $block->escapeHtmlAttr($id) ?>" class="fieldset">
1919
<div class="rule-tree-wrapper">
20-
<div data-content-type="<?= $block->escapeHtmlAttr($role) ?>"></div>
20+
<div data-role="<?= $block->escapeHtmlAttr($role) ?>"></div>
2121
</div>
2222
</fieldset>
2323
</div>

0 commit comments

Comments
 (0)