Skip to content
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

[17] Read API :: Integration tests :: ConfigurableProduct.* #57

Closed
mslabko opened this issue Jun 9, 2020 · 2 comments
Closed

[17] Read API :: Integration tests :: ConfigurableProduct.* #57

mslabko opened this issue Jun 9, 2020 · 2 comments
Assignees
Milestone

Comments

@mslabko
Copy link
Member

mslabko commented Jun 9, 2020

In the scope of delivery #17 need to fix integration tests Magento.ConfigurableProduct*:

Needed branches:

Dev notes:

  • some classes in catalog-storefront repo are autogenerated (marked as "Generated by the Magento PHP proto generator. DO NOT EDIT!") If you still need to edit them to fix test, please mention it in the comment

Example of error:
Stacktrace
Magento\ConfigurableProduct\Block\Product\View\Type\ConfigurableProductPriceTest::testConfigurablePrice
Failed asserting that '

As low as $0.00
' matches PCRE pattern "/As low as</span>.*<span.data-price-amount="10".$10.00</span></span>/".

/var/www/html/dev/tests/integration/testsuite/Magento/ConfigurableProduct/Block/Product/View/Type/ConfigurableProductPriceTest.php:193
/var/www/html/dev/tests/integration/testsuite/Magento/ConfigurableProduct/Block/Product/View/Type/ConfigurableProductPriceTest.php:81

@m2-assistant
Copy link

m2-assistant bot commented Jun 9, 2020

Hi @mslabko. Thank you for your report.
To help us process this issue please make sure that you provided sufficient information.

Please, add a comment to assign the issue: @magento I am working on this


@mslabko
Copy link
Member Author

mslabko commented Jun 11, 2020

This test was fixed in the scope of #55 (comment)

Please, fill free to review related PR. In case you found another root cause of the test failure, please mention it and reopen this ticket.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants