Skip to content

Commit 5ff6676

Browse files
committed
MC-3354: All Attributes Banner
- Fixing Banner button style test - Updating empty banner selector & data
1 parent 5ecdea2 commit 5ff6676

7 files changed

+63
-75
lines changed

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

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -54,6 +54,10 @@
5454
</entity>
5555
<!-- Message Text -->
5656
<entity name="PageBuilderBannerMessageDefaultProperty" type="pagebuilder_banner_message_property">
57+
<data key="name">Message Text</data>
58+
<data key="section">contents</data>
59+
<data key="role">textarea</data>
60+
<data key="value"/>
5761
<data key="placeholder">Edit Banner Text</data>
5862
</entity>
5963
<entity name="PageBuilderBannerMessageProperty" type="pagebuilder_banner_message_property">

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

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -67,7 +67,6 @@
6767
<element name="mobileCollagePadding" type="text" selector="(//div[@data-role='banner'])[{{arg1}}]//div[@data-element='mobile_image'][contains(@style,'padding: {{arg2}};')]" parameterized="true"/>
6868

6969
<!--fix or replace these-->
70-
<element name="poster" type="button" selector="div[data-role='banner'] .pagebuilder-mobile-hidden"/>
7170
<element name="collageLeft" type="button" selector="div[data-role='banner'] .pagebuilder-mobile-hidden[style*='40px calc(50% + 40px) 40px 40px'][style*='magento']"/>
7271
<element name="collageCentered" type="button" selector="div[data-role='banner'] .pagebuilder-mobile-hidden[style*='40px calc(25% + 40px)'][style*='magento']"/>
7372
<element name="collageRight" type="button" selector="div[data-role='banner'] .pagebuilder-mobile-hidden[style*='40px 40px 40px calc(50% + 40px)'][style*='magento']"/>
@@ -79,7 +78,7 @@
7978
<section name="BannerOnBackend">
8079
<element name="base" type="button" selector="(//div[contains(@class,'pagebuilder-banner') and contains(@class,'pagebuilder-content-type')])[{{arg1}}]" parameterized="true"/>
8180
<element name="overlay" type="button" selector="(//div[contains(@class,'pagebuilder-content-type') and contains(@class,'pagebuilder-banner')])[{{arg1}}]//div[@data-element='overlay']" parameterized="true"/>
82-
<element name="emptyBanner" type="button" selector="(//div[contains(@class,'pagebuilder-banner') and contains(@class,'pagebuilder-content-type')])[{{arg1}}]//div[contains(@class,'pagebuilder-empty-banner') and .='{{arg2}}']" parameterized="true"/>
81+
<element name="emptyBanner" type="button" selector="(//div[contains(@class,'pagebuilder-banner') and contains(@class,'pagebuilder-content-type')])[{{arg1}}]//div[contains(@class,'pagebuilder-empty-banner') and .='Edit Banner Text']" parameterized="true"/>
8382
<element name="editableButtonOnStage" type="button" selector="(//div[contains(@class,'pagebuilder-banner') and contains(@class,'pagebuilder-content-type')])[{{arg1}}]//button//span[@contenteditable='true']" parameterized="true"/>
8483
<element name="wysiwygLinkUrl" type="button" selector="[href*='{{arg1}}']" parameterized="true"/>
8584
<!-- Appearance Configuration -->

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -687,7 +687,7 @@
687687
<!-- Validate Stage -->
688688
<comment userInput="Validate Stage" stepKey="commentValidateStage"/>
689689
<waitForElementVisible selector="{{BannerOnBackend.collagePadding('1', PageBuilderBannerCollageCenterPaddingPropertyDefault.desktopValue)}}" stepKey="waitForCollageCenter"/>
690-
<seeElement selector="{{BannerOnBackend.emptyBanner('1', PageBuilderBannerMessageDefaultProperty.placeholder)}}" stepKey="seeEmptyBannerContent"/>
690+
<seeElement selector="{{BannerOnBackend.emptyBanner('1')}}" stepKey="seeEmptyBannerContent"/>
691691
<actionGroup ref="navigateToStoreFront" stepKey="navigateToStoreFront">
692692
<argument name="contentType" value="PageBuilderBannerContentType"/>
693693
</actionGroup>

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -686,7 +686,7 @@
686686
<actionGroup ref="switchToPageBuilderStage" stepKey="switchToPageBuilderStage"/>
687687
<!-- Validate Stage -->
688688
<comment userInput="Validate Stage" stepKey="commentValidateStage"/>
689-
<waitForElementVisible selector="{{BannerOnBackend.emptyBanner('1', PageBuilderBannerMessageDefaultProperty.placeholder)}}" stepKey="seeEmptyBannerContent"/>
689+
<waitForElementVisible selector="{{BannerOnBackend.emptyBanner('1')}}" stepKey="seeEmptyBannerContent"/>
690690
<waitForElementVisible selector="{{BannerOnBackend.collagePadding('1', PageBuilderBannerCollageLeftPaddingPropertyDefault.desktopValue)}}" stepKey="waitForCollageLeft"/>
691691
<actionGroup ref="navigateToStoreFront" stepKey="navigateToStoreFront">
692692
<argument name="contentType" value="PageBuilderBannerContentType"/>

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -686,7 +686,7 @@
686686
<actionGroup ref="switchToPageBuilderStage" stepKey="switchToPageBuilderStage"/>
687687
<!-- Validate Stage -->
688688
<comment userInput="Validate Stage" stepKey="commentValidateStage"/>
689-
<waitForElementVisible selector="{{BannerOnBackend.emptyBanner('1', PageBuilderBannerMessageDefaultProperty.placeholder)}}" stepKey="seeEmptyBannerContent"/>
689+
<waitForElementVisible selector="{{BannerOnBackend.emptyBanner('1')}}" stepKey="seeEmptyBannerContent"/>
690690
<waitForElementVisible selector="{{BannerOnBackend.collagePadding('1', PageBuilderBannerCollageRightPaddingPropertyDefault.desktopValue)}}" stepKey="waitForCollageRight"/>
691691
<actionGroup ref="navigateToStoreFront" stepKey="navigateToStoreFront">
692692
<argument name="contentType" value="PageBuilderBannerContentType"/>

0 commit comments

Comments
 (0)