Skip to content

Commit 26c3ff8

Browse files
committed
MC-4280: Row Vertical Alignment Doesn't Work Inside Block & Dynamic Block
- concat selectors for asserting block styles - stabilize test
1 parent 959467a commit 26c3ff8

File tree

3 files changed

+14
-17
lines changed

3 files changed

+14
-17
lines changed

app/code/Magento/PageBuilder/Test/Mftf/Section/PageBuilderBlockSection.xml

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -31,8 +31,6 @@
3131
<element name="margins" type="text" selector="(//div[contains(@class,'pagebuilder-block')])[{{arg1}}]//div[contains(@style,'margin: {{arg2}}px;')]" parameterized="true"/>
3232
<element name="padding" type="text" selector="(//div[contains(@class,'pagebuilder-block')])[{{arg1}}]//div[contains(@style,'padding: {{arg2}}px;')]" parameterized="true"/>
3333
<element name="overlay" type="text" selector="(//div[contains(@class,'pagebuilder-block') and contains(@class,'pagebuilder-block-overlay')])[{{arg1}}]" parameterized="true"/>
34-
<!-- Row -->
35-
<element name="rowInnerStyle" type="text" selector="(//div[contains(@class,'pagebuilder-block')])[{{arg1}}]//div[@data-role='row']//div[@data-element='inner'][contains(@style, '{{arg2}}')]" parameterized="true"/>
3634
</section>
3735
<section name="BlockOnGrid">
3836
<element name="searchBox" type="text" selector="//input[contains(@class,'data-grid-search-control')]"/>
@@ -45,6 +43,7 @@
4543
<element name="selectBlockBtn" type="button" selector="(//div[contains(@class,'pagebuilder-block')]//button/span[text()='Select Block'])"/>
4644
</section>
4745
<section name="BlockOnStorefront">
46+
<element name="ancestor" type="text" selector="[ancestor::div[@data-role='block']]"/>
4847
<element name="html" type="text" selector="(//div[@data-role='html'])[{{arg1}}]//a[contains(@class,'pagebuilder-button-primary')]" parameterized="true"/>
4948
<element name="base" type="text" selector="(//div[@data-role='block'])[{{arg1}}]" parameterized="true"/>
5049
<element name="hidden" type="text" selector="(//div[@data-role='block'])[{{arg1}}][contains(@style, 'display: none')]" parameterized="true"/>
@@ -63,7 +62,5 @@
6362
<element name="noCssClasses" type="text" selector="(//div[@data-role='row'])[{{arg1}}]/div[not(@class)]" parameterized="true"/>
6463
<element name="margins" type="text" selector="(//div[@data-role='block'])[{{arg1}}][contains(@style,'margin: {{arg2}}px;')]" parameterized="true"/>
6564
<element name="padding" type="text" selector="(//div[@data-role='block'])[{{arg1}}][contains(@style,'padding: {{arg2}}px;')]" parameterized="true"/>
66-
<!-- Row -->
67-
<element name="rowInnerStyle" type="text" selector="(//div[@data-role='block'])[{{arg1}}]//div[@data-role='row']//div[@data-element='inner'][contains(@style, '{{arg2}}')]" parameterized="true"/>
6865
</section>
6966
</sections>

app/code/Magento/PageBuilder/Test/Mftf/Section/PageBuilderRowSection.xml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -77,7 +77,7 @@
7777
<element name="hidden" type="text" selector="(//div[@data-role='row'])[{{arg1}}][contains(@style, 'display: none')]" parameterized="true"/>
7878
<element name="notHidden" type="text" selector="(//div[@data-role='row'])[{{arg1}}][not(contains(@style, 'display: none'))]" parameterized="true"/>
7979
<element name="computedWidth" type="text" selector="parseInt(window.getComputedStyle(document.querySelector('[data-role=row]')).width)"/>
80-
<element name="rowStyle" type="button" selector="(//div[@data-role='row'])[{{arg1}}]/div[contains(@style, '{{arg2}}')]" parameterized="true"/>
80+
<element name="rowStyle" type="button" selector="(//div[@data-role='row'])[{{arg1}}]//div[@data-element='inner'][contains(@style, '{{arg2}}')]" parameterized="true"/>
8181
<element name="rowWidthJS" type="text" selector="document.querySelectorAll('[data-appearance=\'{{arg1}}\']')[{{arg2}}].clientWidth" parameterized="true"/>
8282
<element name="rowH2ChildWidthJS" type="text" selector="document.querySelectorAll('div[data-appearance~=\'{{arg1}}\'] h2')[{{arg2}}].clientWidth" parameterized="true"/>
8383
<element name="appearance" type="text" selector="(//div[@data-role='row'])[{{arg1}}][@data-appearance='{{arg2}}']" parameterized="true"/>

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

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1114,26 +1114,25 @@
11141114
<title value="Row Content Can Be Vertically Aligned In Block"/>
11151115
<description value="A user can vertically align row contents in a Block"/>
11161116
<severity value="CRITICAL"/>
1117-
<useCaseId value="MC-914"/>
1117+
<useCaseId value="MC-4280"/>
11181118
<testCaseId value="MC-5061"/>
11191119
<group value="pagebuilder"/>
11201120
<group value="pagebuilder-block"/>
1121+
<group value="pagebuilder-row"/>
11211122
</annotations>
11221123
<before>
11231124
<actionGroup ref="LoginAsAdmin" stepKey="loginAsAdmin" />
1124-
<actionGroup ref="enablePageBuilderSetting" stepKey="enablePageBuilderSetting" />
11251125
<createData entity="_defaultBlock" stepKey="createPreReqBlock" />
1126+
<actionGroup ref="navigateToCreatedCMSBlockPage" stepKey="navigateToCreatedCMSBlockPage">
1127+
<argument name="CMSBlockPage" value="$$createPreReqBlock$$" />
1128+
</actionGroup>
11261129
</before>
11271130
<after>
11281131
<deleteData createDataKey="createPreReqBlock" stepKey="deletePreReqBlock" />
1129-
<actionGroup ref="DisabledWYSIWYG" stepKey="disableWYSIWYG"/>
11301132
<actionGroup ref="logout" stepKey="logout"/>
11311133
</after>
11321134
<!-- Edit Block and configure Row - Vertical Align Bottom -->
11331135
<comment userInput="Edit Block and configure row vertical align bottom" stepKey="commentConfigureRowVerticalAlignBottom"/>
1134-
<actionGroup ref="navigateToCreatedCMSBlockPage" stepKey="navigateToCreatedCMSBlockPage">
1135-
<argument name="CMSBlockPage" value="$$createPreReqBlock$$" />
1136-
</actionGroup>
11371136
<actionGroup ref="openPageBuilderEditPanel" stepKey="openEditMenuOnStage">
11381137
<argument name="contentType" value="PageBuilderRowContentType"/>
11391138
</actionGroup>
@@ -1145,6 +1144,7 @@
11451144
</actionGroup>
11461145
<actionGroup ref="saveEditPanelSettings" stepKey="saveEditPanelSettingsInBlock"/>
11471146
<click selector="{{CmsNewBlockBlockActionsSection.savePage}}" stepKey="saveBlock"/>
1147+
<waitForPageLoad time="30" stepKey="waitForStageToLoad"/>
11481148
<!-- Go to CMS Page and choose Block -->
11491149
<comment userInput="Go to CMS Page and choose Block" stepKey="commentGoToCMSPage"/>
11501150
<actionGroup ref="navigateToAPageWithPageBuilder" stepKey="navigateToAPageWithPageBuilder"/>
@@ -1164,20 +1164,20 @@
11641164
<actionGroup ref="saveEditPanelSettings" stepKey="saveEditPanelSettings"/>
11651165
<!-- Verify Vertical Alignment On Stage - Before Save -->
11661166
<comment userInput="Verify min height and vertical alignment on stage before save" stepKey="commentVerifyMinHeightAndVerticalAlignmentOnStageBeforeSave"/>
1167-
<seeElement selector="{{BlockOnStage.rowInnerStyle('1', PageBuilderMinHeightProperty_500.value)}}" stepKey="seeRowMinHeightBeforeSave"/>
1168-
<seeElement selector="{{BlockOnStage.rowInnerStyle('1', PageBuilderVerticalAlignmentPropertyBottom.style)}}" stepKey="seeRowVerticalAlignmentBottomBeforeSave"/>
1167+
<seeElement selector="({{RowOnStorefront.rowStyle('1', PageBuilderMinHeightProperty_500.style)}}{{BlockOnStorefront.ancestor}})[1]" stepKey="seeRowMinHeightBeforeSave"/>
1168+
<seeElement selector="({{RowOnStorefront.rowStyle('1', PageBuilderVerticalAlignmentPropertyBottom.style)}}{{BlockOnStorefront.ancestor}})[1]" stepKey="seeRowVerticalAlignmentBottomBeforeSave"/>
11691169
<actionGroup ref="saveAndContinueEditCmsPage" stepKey="saveAndContinueEditCmsPage"/>
11701170
<actionGroup ref="switchToPageBuilderStage" stepKey="switchToPageBuilderStageAfterSave"/>
11711171
<!-- Verify Vertical Alignment On Stage - After Save -->
11721172
<comment userInput="Verify min height and vertical alignment on stage after save" stepKey="commentVerifyMinHeightAndVerticalAlignmentOnStageAfterSave"/>
1173-
<seeElement selector="{{BlockOnStage.rowInnerStyle('1', PageBuilderMinHeightProperty_500.value)}}" stepKey="seeRowMinHeightAfterSave"/>
1174-
<seeElement selector="{{BlockOnStage.rowInnerStyle('1', PageBuilderVerticalAlignmentPropertyBottom.style)}}" stepKey="seeRowVerticalAlignmentBottomAfterSave"/>
1173+
<seeElement selector="({{RowOnStorefront.rowStyle('1', PageBuilderMinHeightProperty_500.style)}}{{BlockOnStorefront.ancestor}})[1]" stepKey="seeRowMinHeightAfterSave"/>
1174+
<seeElement selector="({{RowOnStorefront.rowStyle('1', PageBuilderVerticalAlignmentPropertyBottom.style)}}{{BlockOnStorefront.ancestor}})[1]" stepKey="seeRowVerticalAlignmentBottomAfterSave"/>
11751175
<!-- Verify Min Height and Vertical Alignment In Storefront -->
11761176
<comment userInput="Verify min height and vertical alignment in storefront" stepKey="commentVerifyMinHeightAndVerticalAlignmentInStorefront"/>
11771177
<actionGroup ref="navigateToStoreFront" stepKey="navigateToStoreFront">
11781178
<argument name="contentType" value="PageBuilderBlockContentType"/>
11791179
</actionGroup>
1180-
<seeElement selector="{{BlockOnStorefront.rowInnerStyle('1', PageBuilderMinHeightProperty_500.value)}}" stepKey="seeRowMinHeightStorefront"/>
1181-
<seeElement selector="{{BlockOnStorefront.rowInnerStyle('1', PageBuilderVerticalAlignmentPropertyBottom.style)}}" stepKey="seeRowVerticalAlignmentBottomStorefront"/>
1180+
<seeElement selector="({{RowOnStorefront.rowStyle('1', PageBuilderMinHeightProperty_500.style)}}{{BlockOnStorefront.ancestor}})[1]" stepKey="seeRowMinHeightStorefront"/>
1181+
<seeElement selector="({{RowOnStorefront.rowStyle('1', PageBuilderVerticalAlignmentPropertyBottom.style)}}{{BlockOnStorefront.ancestor}})[1]" stepKey="seeRowVerticalAlignmentBottomStorefront"/>
11821182
</test>
11831183
</tests>

0 commit comments

Comments
 (0)