Skip to content

Commit 4879d15

Browse files
committed
MC-15742: Hard Coded 35px Margin Bottom On Slider on Storefront
- fix existing test to adjust for new slider height - add test coverage for slider height no dots
1 parent 9a53c18 commit 4879d15

File tree

2 files changed

+63
-1
lines changed

2 files changed

+63
-1
lines changed

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -107,7 +107,7 @@
107107
</entity>
108108
<!-- Container Height -->
109109
<entity name="PageBuilderEmptySliderContainerHeight" type="pagebuilder_slider_height_property">
110-
<data key="value">368</data>
110+
<data key="value">360</data>
111111
</entity>
112112
<!-- CSS Classes -->
113113
<entity name="PageBuilderSliderAdvancedCssClasses_Default" type="pagebuilder_advanced_css_classes_property">

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

Lines changed: 62 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -616,6 +616,68 @@
616616
<argument name="padding" value="PageBuilderPaddingProperty10"/>
617617
</actionGroup>
618618
</test>
619+
<test name="SliderWithNoDotsEqualToMinHeight" extends="RowWithBordersAndPaddingEqualToMinHeight">
620+
<annotations>
621+
<features value="PageBuilder"/>
622+
<stories value="Slider"/>
623+
<title value="Slider with no dots is equal to min height"/>
624+
<description value="A user can see Slider height with no dots is equal to min height"/>
625+
<severity value="AVERAGE"/>
626+
<useCaseId value="MC-15742"/>
627+
<testCaseId value="MC-18088"/>
628+
<group value="pagebuilder"/>
629+
<group value="pagebuilder-slide"/>
630+
<group value="pagebuilder-sliderCommon"/>
631+
</annotations>
632+
<actionGroup ref="addPageBuilderPageTitle" stepKey="enterPageTitle">
633+
<argument name="contentType" value="PageBuilderSliderContentType"/>
634+
</actionGroup>
635+
<actionGroup ref="expandPageBuilderPanelMenuSection" after="enterPageTitle" stepKey="expandMenuSection">
636+
<argument name="contentType" value="PageBuilderSliderContentType"/>
637+
</actionGroup>
638+
<actionGroup ref="dragContentTypeToStage" after="commentConfigureContentType" stepKey="dragContentTypeToStage">
639+
<argument name="contentType" value="PageBuilderSliderContentType"/>
640+
</actionGroup>
641+
<click selector="{{PageBuilderPanel.searchPanel}}" after="dragContentTypeToStage" stepKey="unFocusSlide"/>
642+
<actionGroup ref="openPageBuilderEditPanel" stepKey="openEditPanel">
643+
<argument name="contentType" value="PageBuilderSliderContentType"/>
644+
</actionGroup>
645+
<actionGroup ref="fillSlideOutPanelFieldGeneral" stepKey="enterMinHeightProperty">
646+
<argument name="property" value="PageBuilderSliderMinHeight_Default"/>
647+
</actionGroup>
648+
<actionGroup ref="conditionalClickSlideOutPanelFieldGeneral" after="enterMinHeightProperty" stepKey="enterShowDotsDisabled">
649+
<argument name="property" value="PageBuilderSliderShowDots_Disabled"/>
650+
</actionGroup>
651+
<comment userInput="keyForRemoval" stepKey="enterBorder"/>
652+
<comment userInput="keyForRemoval" stepKey="enterBorderColor"/>
653+
<comment userInput="keyForRemoval" stepKey="enterBorderWidth"/>
654+
<comment userInput="keyForRemoval" stepKey="enterPadding"/>
655+
<actionGroup ref="openPageBuilderEditPanelSmallByIndex" after="saveEditPanelSettings" stepKey="openSlideItemEditPanel">
656+
<argument name="section" value="SlideOnBackend"/>
657+
</actionGroup>
658+
<actionGroup ref="fillSlideOutPanelFieldGeneral" after="openSlideItemEditPanel" stepKey="enterMinHeightPropertySlideItem">
659+
<argument name="property" value="PageBuilderSlideItemMinimumHeight_300"/>
660+
</actionGroup>
661+
<actionGroup ref="fillSlideOutPanelField" after="enterMinHeightPropertySlideItem" stepKey="enterBackgroundColor">
662+
<argument name="property" value="PageBuilderBackgroundColor_Hex_Grey"/>
663+
</actionGroup>
664+
<actionGroup ref="saveEditPanelSettings" after="enterBackgroundColor" stepKey="saveEditPanelSettings2"/>
665+
<actionGroup ref="validateContentTypeHeightEqualToMinHeight" stepKey="validateContentTypeHeightBeforeSave">
666+
<argument name="content" value="{{SliderOnStage.containerCSS}}"/>
667+
<argument name="height" value="PageBuilderSliderMinHeight_Default"/>
668+
</actionGroup>
669+
<actionGroup ref="validateContentTypeHeightEqualToMinHeight" stepKey="validateContentTypeHeightAfterSave">
670+
<argument name="content" value="{{SliderOnStage.containerCSS}}"/>
671+
<argument name="height" value="PageBuilderSliderMinHeight_Default"/>
672+
</actionGroup>
673+
<actionGroup ref="navigateToStoreFront" stepKey="navigateToStoreFront">
674+
<argument name="contentType" value="PageBuilderSliderContentType"/>
675+
</actionGroup>
676+
<actionGroup ref="validateContentTypeHeightEqualToMinHeight" stepKey="validateContentTypeHeightStorefront">
677+
<argument name="content" value="{{SliderOnFrontend.all}}"/>
678+
<argument name="height" value="PageBuilderSliderMinHeight_Default"/>
679+
</actionGroup>
680+
</test>
619681
<test name="SliderMoveSlider">
620682
<annotations>
621683
<features value="PageBuilder"/>

0 commit comments

Comments
 (0)