Skip to content

Commit cad2625

Browse files
committed
MAGETWO-92070: Automate MFTF
- Fixed review comments
1 parent c6b094b commit cad2625

12 files changed

+58
-58
lines changed

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -478,7 +478,7 @@
478478
<argument name="property" defaultValue=""/>
479479
</arguments>
480480
<waitForElement selector="{{PageBuilderActionsSection.panelFieldControl(property.section, property.fieldName)}}" stepKey="waitForElement" />
481-
<waitForElementVisible selector="{{PageBuilderActionsSection.editFormUploadedImageSource(property.section, property.fieldName, property.baseFileName, property.fileType)}}" stepKey="seeUploadedFile"/>
481+
<waitForElementVisible selector="{{PageBuilderActionsSection.editFormUploadedImageSource(property.section, property.fieldName, property.fileName, property.fileType)}}" stepKey="seeUploadedFile"/>
482482
</actionGroup>
483483
<actionGroup name="seeNoImageUploadedOnSlideOut">
484484
<arguments>

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

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@
99
xsi:noNamespaceSchemaLocation="../../../../../../../dev/tests/acceptance/vendor/magento/magento2-functional-testing-framework/src/Magento/FunctionalTestingFramework/Test/etc/actionGroupSchema.xsd">
1010
<actionGroup name="addImageToStage">
1111
<arguments>
12-
<argument name="property" defaultValue=""/>
12+
<argument name="property"/>
1313
</arguments>
1414
<comment userInput="Attach image A to PageBuilder stage" stepKey="commentAttach"/>
1515
<attachFile userInput="{{property.value}}" selector="{{ImageOnStage.attachImageBtn}}" stepKey="attachImageFileOnStage"/>
@@ -19,8 +19,8 @@
1919
</actionGroup>
2020
<actionGroup name="verifyUploadImageButtons">
2121
<arguments>
22-
<argument name="contentBlockType" defaultValue=""/>
23-
<argument name="section" defaultValue=""/>
22+
<argument name="contentBlockType"/>
23+
<argument name="section"/>
2424
</arguments>
2525
<comment userInput="Hover uploaded image to check Upload New Image button and Select From Gallery button" stepKey="commentHover"/>
2626
<moveMouseOver selector="{{PageBuilderActionsSection.contentBlockInStage(contentBlockType.role)}}" x="10" y="0" stepKey="onMouseOverBannerContentBlockStage"/>
@@ -29,7 +29,7 @@
2929
</actionGroup>
3030
<actionGroup name="verifyImageOnSlideout">
3131
<arguments>
32-
<argument name="property" defaultValue=""/>
32+
<argument name="property"/>
3333
</arguments>
3434
<comment userInput="Verify image A display on preview of slide-out" stepKey="commentVerifyImage"/>
3535
<waitForElementVisible selector="{{ImageOnConfigurationPanel.imageName(property.value)}}" stepKey="seeImageNameOnPreview"/>

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

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,7 @@
5757
</arguments>
5858
<waitForElementVisible selector="{{page.base(index)}}" stepKey="waitForRow"/>
5959
<waitForElementVisible selector="{{page.backgroundColor(index, backgroundColor.rgb)}}" stepKey="waitForBackgroundColor"/>
60-
<waitForElementVisible selector="{{page.backgroundImage(index, backgroundImage.baseFileName, backgroundImage.fileType)}}" stepKey="waitForBackgroundImage"/>
60+
<waitForElementVisible selector="{{page.backgroundImage(index, backgroundImage.fileName, backgroundImage.fileType)}}" stepKey="waitForBackgroundImage"/>
6161
<waitForElementVisible selector="{{page.backgroundPosition(index, backgroundPosition.styleValue)}}" stepKey="waitForBackgroundPosition"/>
6262
<waitForElementVisible selector="{{page.backgroundSize(index, backgroundSize.value)}}" stepKey="waitForBackgroundSize"/>
6363
<waitForElementVisible selector="{{page.backgroundRepeat(index, backgroundRepeat.styleValue)}}" stepKey="waitForBackgroundRepeat"/>
@@ -76,7 +76,7 @@
7676
</arguments>
7777
<waitForElementVisible selector="{{page.base(index)}}" stepKey="waitForRow"/>
7878
<waitForElementVisible selector="{{page.backgroundColor(index, backgroundColor.rgb)}}" stepKey="waitForBackgroundColor"/>
79-
<waitForElementVisible selector="{{page.parallaxBackgroundImage(index, backgroundImage.baseFileName, backgroundImage.fileType)}}" stepKey="waitForBackgroundImage"/>
79+
<waitForElementVisible selector="{{page.parallaxBackgroundImage(index, backgroundImage.fileName, backgroundImage.fileType)}}" stepKey="waitForBackgroundImage"/>
8080
<waitForElementVisible selector="{{page.backgroundPosition(index, backgroundPosition.styleValue)}}" stepKey="waitForBackgroundPosition"/>
8181
<waitForElementVisible selector="{{page.backgroundSize(index, backgroundSize.value)}}" stepKey="waitForBackgroundSize"/>
8282
<waitForElementVisible selector="{{page.backgroundRepeat(index, backgroundRepeat.styleValue)}}" stepKey="waitForBackgroundRepeat"/>

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -100,7 +100,7 @@
100100
<data key="section">background</data>
101101
<data key="fieldName">background_image</data>
102102
<data key="value">magento.jpg</data>
103-
<data key="baseFileName">magento</data>
103+
<data key="fileName">magento</data>
104104
<data key="fileType">jpg</data>
105105
<data key="path">wysiwyg/magento</data>
106106
<data key="url">/(\b(https?|ftp|file):\/\/)[-A-Za-z0-9+&amp;@#/%?=~_|!:,.;]+[-A-Za-z0-9+&amp;@#/%=~_|]/g</data>

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

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,16 @@
1414
<data key="fieldName">slide_name</data>
1515
<data key="value">Slide 1 Name</data>
1616
</entity>
17+
<entity name="PageBuilderSliderBackgroundImageProperty" type="pagebuilder_background_image_property">
18+
<data key="name">Background Image</data>
19+
<data key="section">background</data>
20+
<data key="fieldName">background_image</data>
21+
<data key="value">magento.jpg</data>
22+
<data key="fileName">magento</data>
23+
<data key="fileType">jpg</data>
24+
<data key="path">wysiwyg/magento</data>
25+
<data key="url">/(\b(https?|ftp|file):\/\/)[-A-Za-z0-9+&amp;@#/%?=~_|!:,.;]+[-A-Za-z0-9+&amp;@#/%=~_|]/g</data>
26+
</entity>
1727
<entity name="PageBuilderSlideMinHeight" type="pagebuilder_slide_property">
1828
<data key="name">Minimum Height</data>
1929
<data key="section">appearance_fieldset</data>

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

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -89,6 +89,9 @@
8989
<element name="editFormUploadedImage" type="button" selector="//div[@data-index='{{arg1}}']/descendant::*[@name='{{arg2}}']//..//..//img" parameterized="true"/>
9090
<element name="editFormNoUploadedImage" type="button" selector="//span[text()='{{arg1}}']//parent::label//following-sibling::div//p[text()='Browse to find or drag image here']" parameterized="true"/>
9191
<element name="editFormUploadedImageSource" type="button" selector="//div[@data-index='{{arg1}}']/descendant::*[@name='{{arg2}}']//..//..//img[contains(@src,'{{arg3}}') and contains(@src,'{{arg4}}')]" parameterized="true"/>
92+
<element name="uploadLocalImageBtn" type="button" selector="//div[contains(@class,'pagebuilder-image-uploader-container')]//input[@name='background_image']"/>
93+
<element name="uploadNewImageBtn" type="button" selector="//label[text()='Upload Image']" />
94+
<element name="selectFromGalleryImageBtn" type="button" selector="//ul[contains(@class,'pagebuilder-options-links')]//label[text()='Select from Gallery']"/>
9295
</section>
9396
<section name="AdvancedForm">
9497
<element name="sectionName" type="button" selector="//div[@data-index='advanced']"/>

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

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -102,9 +102,6 @@
102102
<element name="bannerBorderRadiusStyle" type="button" selector="div[style*='border-radius: {{arg1}}']" parameterized="true"/>
103103
<element name="editFormBackgroundImagePreview" type="button" selector="//input[@name='background_image']/ancestor::div[@class='admin__field-control']/descendant::img"/>
104104
<element name="editFormBackgroundMobileImagePreview" type="button" selector="//input[@name='mobile_image']/ancestor::div[@class='admin__field-control']/descendant::img"/>
105-
<element name="uploadLocalImageBtn" type="button" selector="//div[contains(@class,'pagebuilder-image-uploader-container')]//input[@name='background_image']"/>
106-
<element name="uploadNewImageBtn" type="button" selector="//label[text()='Upload Image']" />
107-
<element name="selectFromGalleryImageBtn" type="button" selector="//ul[contains(@class,'pagebuilder-options-links')]//label[text()='Select from Gallery']"/>
108105
</section>
109106
<section name="BannerOnConfigurationPanel">
110107
<element name="tinyMCEIFrame" type="text" selector="pagebuilder_banner_form_message_ifr"/>

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

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -186,10 +186,7 @@
186186
<element name="slideBorderRadiusStyle" type="button" selector="div[style*='border-radius: {{arg1}}']" parameterized="true"/>
187187
<element name="editFormBackgroundImagePreview" type="button" selector="//input[@name='background_image']/ancestor::div[@class='admin__field-control']/descendant::img"/>
188188
<element name="editFormBackgroundMobileImagePreview" type="button" selector="//input[@name='mobile_image']/ancestor::div[@class='admin__field-control']/descendant::img"/>
189-
<element name="uploadLocalImageBtn" type="button" selector="//div[contains(@class,'pagebuilder-image-uploader-container')]//input[@name='background_image']"/>
190189
<element name="collageOverlay" type="button" selector=".pagebuilder-collage-overlay"/>
191-
<element name="uploadNewImageBtn" type="button" selector="//label[text()='Upload Image']" />
192-
<element name="selectFromGalleryImageBtn" type="button" selector="//ul[contains(@class,'pagebuilder-options-links')]//label[text()='Select from Gallery']"/>
193190
</section>
194191
<section name="SlideOnConfigurationPanel">
195192
<element name="tinyMCEIFrame" type="text" selector="pagebuilder_slide_form_content_ifr"/>

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

Lines changed: 19 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -982,7 +982,7 @@
982982
</test>
983983
<test name="BannerCanAddImageToBanner">
984984
<annotations>
985-
<features value="Pagebuilder: Banner"/>
985+
<features value="Pagebuilder"/>
986986
<stories value="MAGETWO-86533: [UI component] MediaGalleryImage: inside form"/>
987987
<title value="Add an image content block into banner"/>
988988
<description value="A user should be able to add an image into a banner."/>
@@ -1068,7 +1068,7 @@
10681068
</test>
10691069
<test name="AddImageToBannerStage">
10701070
<annotations>
1071-
<features value="Pagebuilder: Banner"/>
1071+
<features value="Pagebuilder"/>
10721072
<stories value="MAGETWO-75104: Add image into Slide, Banner from the Stage"/>
10731073
<title value="Add image into Banner from the Stage"/>
10741074
<description value="A user should be able to add image to Banner stage."/>
@@ -1080,8 +1080,6 @@
10801080
</annotations>
10811081
<before>
10821082
<actionGroup ref="LoginAsAdmin" stepKey="loginAsAdmin"/>
1083-
<actionGroup ref="navigateToAPageWithPageBuilder" stepKey="navigateToAPageWithPageBuilder"/>
1084-
<actionGroup ref="switchToPageBuilderStage" stepKey="switchToPageBuilderStage"/>
10851083
<createData entity="_defaultCmsPage" stepKey="createPreReqCMSPage" />
10861084
</before>
10871085
<after>
@@ -1099,14 +1097,14 @@
10991097
</actionGroup>
11001098
<!--Verify there are 2 buttons when hovering-->
11011099
<comment userInput="Verify there are 2 buttons when hovering" stepKey="commentVerifyButtons" />
1102-
<moveMouseOver selector="{{PageBuilderActionsSection.contentBlockInStage(PageBuilderBannerContentBlock.role)}}" x="10" y="0" stepKey="onMouseOverBannerContentBlockStage"/>
1100+
<moveMouseOver selector="{{PageBuilderActionsSection.contentBlockInStage(PageBuilderBannerContentBlock.role)}}" stepKey="onMouseOverBannerContentBlockStage"/>
11031101
<actionGroup ref="verifyUploadImageButtons" stepKey="verifyUploadImageButtons" >
11041102
<argument name="contentBlockType" value="PageBuilderBannerContentBlock"/>
1105-
<argument name="section" value="BannerOnBackend"/>
1103+
<argument name="section" value="PageBuilderActionsSection"/>
11061104
</actionGroup>
11071105
<!--Upload image from local machine -->
11081106
<comment userInput="Upload image from local machine" stepKey="commentUploadImageFromLocal" />
1109-
<attachFile userInput="{{PageBuilderLocalImagePropertyJpg.value}}" selector="{{BannerOnBackend.uploadLocalImageBtn}}" stepKey="attachImageFromLocalToStage"/>
1107+
<attachFile userInput="{{PageBuilderLocalImagePropertyJpg.value}}" selector="{{PageBuilderActionsSection.uploadLocalImageBtn}}" stepKey="attachImageFromLocalToStage"/>
11101108
<seeElement selector="{{ImageOnConfigurationPanel.imageSource(PageBuilderImagePropertyJpg.fileName)}}" stepKey="seeMainImageSource1"/>
11111109
<!--Upload image from Media Gallery Storage -->
11121110
<comment userInput="Upload image from Media Gallery Storage" stepKey="commentUploadImageFromStorage"/>
@@ -1117,30 +1115,30 @@
11171115
<argument name="ImageFolder" value="ImageFolder"/>
11181116
</actionGroup>
11191117
<actionGroup ref="attachImage" stepKey="attachImageFile1">
1120-
<argument name="Image" value="PageBuilderImagePropertyJpg"/>
1118+
<argument name="Image" value="PageBuilderBannerBackgroundImageProperty"/>
11211119
</actionGroup>
11221120
<actionGroup ref="saveImage" stepKey="saveImage"/>
1123-
<seeElement selector="{{ImageOnConfigurationPanel.imageSource(PageBuilderImagePropertyJpg.fileName)}}" stepKey="seeMainImageSource2"/>
1121+
<seeElement selector="{{ImageOnConfigurationPanel.imageSource(PageBuilderBannerBackgroundImageProperty.fileName)}}" stepKey="seeMainImageSource2"/>
11241122
<comment userInput="Go to Banner slide-out to verify uploaded image" stepKey="commentGoToBannerSlideOut"/>
11251123
<!--Verify uploaded image is displayed on slide-out-->
11261124
<comment userInput="Verify uploaded image is displayed on slide-out" stepKey="commentVerifyImageOnSlideout"/>
11271125
<actionGroup ref="openPageBuilderEditPanel" stepKey="openEditMenuOnStage">
11281126
<argument name="contentBlockType" value="PageBuilderBannerContentBlock"/>
11291127
</actionGroup>
1130-
<seeElement selector="{{ImageOnConfigurationPanel.imageName(PageBuilderImagePropertyJpg.value)}}" stepKey="seeImageNameOnPreview2"/>
1131-
<seeElement selector="{{ImageOnConfigurationPanel.imageSource(PageBuilderImagePropertyJpg.fileName)}}" stepKey="seeMainImageSource3"/>
1128+
<actionGroup ref="seeImageUploadedOnSlideOut" stepKey="seeImageOnSlideOut1">
1129+
<argument name="property" value="PageBuilderBannerBackgroundImageProperty"/>
1130+
</actionGroup>
11321131
<actionGroup ref="saveEditPanelSettings" stepKey="saveEditPanelSettings"/>
11331132
<actionGroup ref="saveAndContinueEditCmsPage" stepKey="saveAndContinueEditCmsPage"/>
11341133
<!-- Validate Storefront -->
11351134
<comment userInput="Validate Storefront" stepKey="commentValidateStorefront"/>
1136-
<openNewTab stepKey="openNewTab"/>
11371135
<amOnPage url="$$createPreReqCMSPage.identifier$$" stepKey="amOnCmsStorefront" />
11381136
<waitForPageLoad stepKey="waitForPageLoad3" />
1139-
<seeElement selector="{{ImageOnConfigurationPanel.imageSource(PageBuilderImagePropertyJpg.fileName)}}" stepKey="seeMainImageSourceOnFrontend"/>
1137+
<seeElement selector="{{ImageOnConfigurationPanel.imageSource(PageBuilderBannerBackgroundImageProperty.fileName)}}" stepKey="seeMainImageSourceOnFrontend"/>
11401138
</test>
11411139
<test name="VerifyUploadImageOnBannerStageDependOnEachTypeOfAppearance">
11421140
<annotations>
1143-
<features value="Pagebuilder: Banner"/>
1141+
<features value="Pagebuilder:"/>
11441142
<stories value="MAGETWO-75104: Add image into Slide, Banner from the Stage"/>
11451143
<title value="Add image into Banner from the Stage"/>
11461144
<description value="A user should be able to add image to Banner stage when switching between appearance type"/>
@@ -1152,8 +1150,6 @@
11521150
</annotations>
11531151
<before>
11541152
<actionGroup ref="LoginAsAdmin" stepKey="loginAsAdmin"/>
1155-
<actionGroup ref="navigateToAPageWithPageBuilder" stepKey="navigateToAPageWithPageBuilder"/>
1156-
<actionGroup ref="switchToPageBuilderStage" stepKey="switchToPageBuilderStage"/>
11571153
<createData entity="_defaultCmsPage" stepKey="createPreReqCMSPage" />
11581154
</before>
11591155
<after>
@@ -1178,24 +1174,24 @@
11781174
</actionGroup>
11791175
<actionGroup ref="saveEditPanelSettings" stepKey="saveEditPanelSettings1"/>
11801176
<comment userInput="Verify there are 2 buttons when hovering" stepKey="commentVerifyButtons1" />
1181-
<moveMouseOver selector="{{PageBuilderActionsSection.contentBlockInStage(PageBuilderBannerContentBlock.role)}}" x="10" y="0" stepKey="onMouseOverBannerContentBlockStage1"/>
1177+
<moveMouseOver selector="{{PageBuilderActionsSection.contentBlockInStage(PageBuilderBannerContentBlock.role)}}" stepKey="onMouseOverBannerContentBlockStage1"/>
11821178
<actionGroup ref="verifyUploadImageButtons" stepKey="verifyUploadImageButtons1" >
11831179
<argument name="contentBlockType" value="PageBuilderBannerContentBlock"/>
1184-
<argument name="section" value="BannerOnBackend"/>
1180+
<argument name="section" value="PageBuilderActionsSection"/>
11851181
</actionGroup>
11861182
<!--Verify there are 2 buttons display when user select appearance Center-->
11871183
<actionGroup ref="openPageBuilderEditPanel" stepKey="openEditMenuOnStage2">
11881184
<argument name="contentBlockType" value="PageBuilderBannerContentBlock"/>
11891185
</actionGroup>
11901186
<actionGroup ref="chooseVisualSelectOption" stepKey="enterAppearanceProperty2">
1191-
<argument name="property" value="PageBuilderSlideCollageCenteredProperty"/>
1187+
<argument name="property" value="PageBuilderBannerCollageCenteredProperty"/>
11921188
</actionGroup>
11931189
<actionGroup ref="saveEditPanelSettings" stepKey="saveEditPanelSettings2"/>
11941190
<comment userInput="Verify there are 2 buttons when hovering" stepKey="commentVerifyButtons2" />
1195-
<moveMouseOver selector="{{PageBuilderActionsSection.contentBlockInStage(PageBuilderBannerContentBlock.role)}}" x="10" y="0" stepKey="onMouseOverBannerContentBlockStage2"/>
1191+
<moveMouseOver selector="{{PageBuilderActionsSection.contentBlockInStage(PageBuilderBannerContentBlock.role)}}" stepKey="onMouseOverBannerContentBlockStage2"/>
11961192
<actionGroup ref="verifyUploadImageButtons" stepKey="verifyUploadImageButtons2" >
11971193
<argument name="contentBlockType" value="PageBuilderBannerContentBlock"/>
1198-
<argument name="section" value="BannerOnBackend"/>
1194+
<argument name="section" value="PageBuilderActionsSection"/>
11991195
</actionGroup>
12001196
<!--Verify there are 2 buttons display when user select appearance Right-->
12011197
<actionGroup ref="openPageBuilderEditPanel" stepKey="openEditMenuOnStage3">
@@ -1206,10 +1202,10 @@
12061202
</actionGroup>
12071203
<actionGroup ref="saveEditPanelSettings" stepKey="saveEditPanelSettings3"/>
12081204
<comment userInput="Verify there are 2 buttons when hovering" stepKey="commentVerifyButtons3" />
1209-
<moveMouseOver selector="{{PageBuilderActionsSection.contentBlockInStage(PageBuilderBannerContentBlock.role)}}" x="10" y="0" stepKey="onMouseOverBannerContentBlockStage3"/>
1205+
<moveMouseOver selector="{{PageBuilderActionsSection.contentBlockInStage(PageBuilderBannerContentBlock.role)}}" stepKey="onMouseOverBannerContentBlockStage3"/>
12101206
<actionGroup ref="verifyUploadImageButtons" stepKey="verifyUploadImageButtons3" >
12111207
<argument name="contentBlockType" value="PageBuilderBannerContentBlock"/>
1212-
<argument name="section" value="BannerOnBackend"/>
1208+
<argument name="section" value="PageBuilderActionsSection"/>
12131209
</actionGroup>
12141210
</test>
12151211
</tests>

0 commit comments

Comments
 (0)