Skip to content

Commit df6a505

Browse files
author
Hwashiang Yu
committed
MC-4318: Slide Transparent Background Color Renders Incorrectly & Inconsistently On Stage & Storefront
- Moved expected assertion result to data entity - Added validation for after saving
1 parent 2505b40 commit df6a505

File tree

4 files changed

+26
-2
lines changed

4 files changed

+26
-2
lines changed

app/code/Magento/PageBuilder/Test/Mftf/Data/BannerData.xml

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -68,6 +68,13 @@
6868
<data key="cmsPageLinkName" unique="suffix">cms-page-link-</data>
6969
<data key="value">Good Night!</data>
7070
</entity>
71+
<!-- Background Color -->
72+
<entity name="PageBuilderBannerBackgroundColor_Empty" type="pagebuilder_banner_background_color_property">
73+
<data key="name">Background Color</data>
74+
<data key="section">background</data>
75+
<data key="fieldName">background_color</data>
76+
<data key="value">rgb(250, 250, 250)</data>
77+
</entity>
7178
<!-- Link -->
7279
<entity name="PageBuilderBannerLinkUrlProperty" type="pagebuilder_banner_linkUrl_property">
7380
<data key="name">Link</data>

app/code/Magento/PageBuilder/Test/Mftf/Data/SlideData.xml

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -40,6 +40,13 @@
4040
<data key="fieldName">min_height</data>
4141
<data key="value">300</data>
4242
</entity>
43+
<!-- Background Color -->
44+
<entity name="PageBuilderSlideItemBackgroundColor_Empty" type="pagebuilder_slide_item_background_color_property">
45+
<data key="name">Background Color</data>
46+
<data key="section">background</data>
47+
<data key="fieldName">background_color</data>
48+
<data key="value">rgb(250, 250, 250)</data>
49+
</entity>
4350
<!-- Link -->
4451
<entity name="PageBuilderSlideItemLinkURL_Empty" type="pagebuilder_slide_item_link_url_property">
4552
<data key="name">Link</data>

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

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -596,7 +596,7 @@
596596
</actionGroup>
597597
<executeJS function="return window.getComputedStyle(document.evaluate(&quot;{{BannerOnBackend.noBackgroundColor('1')}}&quot;, document.body).iterateNext()).backgroundColor" stepKey="grabDefaultBackgroundColor"/>
598598
<assertEquals stepKey="assertDefaultBackgroundColor">
599-
<expectedResult type="string">rgb(250, 250, 250)</expectedResult>
599+
<expectedResult type="string">{{PageBuilderBannerBackgroundColor_Empty.value}}</expectedResult>
600600
<actualResult type="variable">grabDefaultBackgroundColor</actualResult>
601601
</assertEquals>
602602
<actionGroup ref="saveAndContinueEditCmsPage" stepKey="saveAndContinueEditCmsPage"/>
@@ -606,6 +606,11 @@
606606
<actionGroup ref="validateBackgroundAttributesWithNoImageOrColor" stepKey="validateStage2">
607607
<argument name="section" value="BannerOnBackend"/>
608608
</actionGroup>
609+
<executeJS function="return window.getComputedStyle(document.evaluate(&quot;{{BannerOnBackend.noBackgroundColor('1')}}&quot;, document.body).iterateNext()).backgroundColor" stepKey="grabDefaultBackgroundColorAfterSave"/>
610+
<assertEquals stepKey="assertDefaultBackgroundColorAfterSave">
611+
<expectedResult type="string">{{PageBuilderBannerBackgroundColor_Empty.value}}</expectedResult>
612+
<actualResult type="variable">grabDefaultBackgroundColorAfterSave</actualResult>
613+
</assertEquals>
609614
<actionGroup ref="openPageBuilderEditPanel" stepKey="openEditMenuOnStage2">
610615
<argument name="contentType" value="PageBuilderBannerContentType"/>
611616
</actionGroup>

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

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -198,7 +198,7 @@
198198
<seeElement selector="{{SlideOnBackend.backgroundRepeat('1', PageBuilderBackgroundRepeat_Default.styleValue)}}" stepKey="seeBackgroundRepeatStage"/>
199199
<executeJS function="return window.getComputedStyle(document.evaluate(&quot;{{SlideOnBackend.noBackgroundColor('1')}}&quot;, document.body).iterateNext()).backgroundColor" stepKey="grabDefaultBackgroundColor"/>
200200
<assertEquals stepKey="assertDefaultBackgroundColor">
201-
<expectedResult type="string">rgb(250, 250, 250)</expectedResult>
201+
<expectedResult type="string">{{PageBuilderSlideItemBackgroundColor_Empty.value}}</expectedResult>
202202
<actualResult type="variable">grabDefaultBackgroundColor</actualResult>
203203
</assertEquals>
204204
<actionGroup ref="saveAndContinueEditCmsPage" stepKey="saveAndContinueEditCmsPage"/>
@@ -210,6 +210,11 @@
210210
<seeElement selector="{{SlideOnBackend.backgroundSize('1', PageBuilderBackgroundSize_Default.value)}}" stepKey="seeBackgroundSizeStage2"/>
211211
<seeElement selector="{{SlideOnBackend.backgroundPosition('1', PageBuilderBackgroundPosition_Default.styleValue)}}" stepKey="seeBackgroundPositionStage2"/>
212212
<seeElement selector="{{SlideOnBackend.backgroundRepeat('1', PageBuilderBackgroundRepeat_Default.styleValue)}}" stepKey="seeBackgroundRepeatStage2"/>
213+
<executeJS function="return window.getComputedStyle(document.evaluate(&quot;{{SlideOnBackend.noBackgroundColor('1')}}&quot;, document.body).iterateNext()).backgroundColor" stepKey="grabDefaultBackgroundColorAfterSave"/>
214+
<assertEquals stepKey="assertDefaultBackgroundColorAfterSave">
215+
<expectedResult type="string">{{PageBuilderSlideItemBackgroundColor_Empty.value}}</expectedResult>
216+
<actualResult type="variable">grabDefaultBackgroundColorAfterSave</actualResult>
217+
</assertEquals>
213218
<!-- Validate Edit Panel After Save -->
214219
<comment userInput="Validate Edit Panel After Save" stepKey="commentValidateEditPanel2"/>
215220
<actionGroup ref="openPageBuilderEditPanelSmallByIndex" stepKey="openEditPanel">

0 commit comments

Comments
 (0)