Skip to content

AdminAdobeStockInsertRenditionImageFromGalleryFileSizeTest #1835

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Sep 21, 2020

Conversation

engcom-Golf
Copy link
Contributor

@engcom-Golf engcom-Golf commented Sep 15, 2020

Description (*)

Related Pull Requests

https://github.com/magento/magento2/pull/30074/files

Cover issue by MFTF test by adding image from adobe stock inventory

Fixed Issues (if relevant)

#1806

Manual testing scenarios (*)

Preconditions:

Set rendition image size 50x50

Steps

  1. Open Category page
  2. Open Media Gallery
  3. Save image preview from Adobe Stock
  4. Add saved preview to category
  5. Assert correct file (1) size is shown

Copy link
Member

@sivaschenko sivaschenko left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the pull request @engcom-Golf ! Please see my comment

<createData entity="SimpleSubCategory" stepKey="category"/>
<actionGroup ref="AdminLoginActionGroup" stepKey="loginAsAdmin"/>
<!-- Prepare configuration -->
<magentoCLI command="config:set system/media_gallery_renditions/width 50" stepKey="prepareWidth"/>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please introduce an action group to set configuration values to MediaGalleryRenditions modules (please create a new pull request to magento2 for this purpose)

@sivaschenko sivaschenko added has dependency Priority: P3 May be fixed according to the position in the backlog Severity: S3 Affects non-critical data or functionality and does not force users to employ a workaround and removed has dependency labels Sep 17, 2020
@sivaschenko sivaschenko changed the title magento/adobe-stock-integration#1806 add mftf test AdminAdobeStockInsertRenditionImageFromGalleryFileSizeTest Sep 21, 2020
@sivaschenko
Copy link
Member

@magento run Functional Tests CE,Functional Tests EE,Functional Tests B2B

@ghost
Copy link

ghost commented Sep 21, 2020

Hi @engcom-Golf, thank you for your contribution!
Please, complete Contribution Survey, it will take less than a minute.
Your feedback will help us to improve contribution process.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority: P3 May be fixed according to the position in the backlog Severity: S3 Affects non-critical data or functionality and does not force users to employ a workaround
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants