Skip to content

MSI-969: [Configuration-Catalog-Products-Configurable product] Manage Stock on Configuration page turned off in admin for Configurable Product on Default stock#1030

Merged
maghamed merged 3 commits into2.3-developfrom
MSI-969
May 10, 2018
Merged

Conversation

@nmalevanec
Copy link
Copy Markdown
Contributor

@nmalevanec nmalevanec commented Apr 26, 2018

Description

Functional test, which covers managing configurable product variation stock and how it affects configurable product appearance on storefront.

Fixed Issues (if relevant)

  1. [Configuration-Catalog-Products-Configurable product] Manage Stock on Configuration page turned off in admin for Configurable Product on Default stock #969: [Configuration-Catalog-Products-Configurable product] Manage Stock on Configuration page turned off in admin for Configurable Product on Default stock

Manual testing scenarios

  1. none.

Currently this test scenario is blocked by #1029


Original HipTest ID: 1695735

… Stock on Configuration page turned off in admin for Configurable Product on Default stock
@maghamed maghamed added the MFTF Functional Test Coverage label May 3, 2018
@okolesnyk okolesnyk self-assigned this May 4, 2018
@okolesnyk okolesnyk self-requested a review May 4, 2018 09:02
Copy link
Copy Markdown
Member

@okolesnyk okolesnyk left a comment

Choose a reason for hiding this comment

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

Please avoid using numbers as stepKey="loginAsAdmin1" or stepKey="fillNameForNewAttribute2"

Try to make them more descriptive like
stepKey="fillNameForRedColorAttribute"
stepKey="fillNameRedColorAttrForConfigurable"

It will help when some other module will want to extend your test.

The only place I see where numbers can be used is here: https://github.com/magento-engcom/msi/pull/1040/files

… Stock on Configuration page turned off in admin for Configurable Product on Default stock
@okolesnyk okolesnyk removed their assignment May 10, 2018
@maghamed maghamed merged commit d7b8bb4 into 2.3-develop May 10, 2018
@maghamed maghamed deleted the MSI-969 branch February 5, 2019 17:18
sidolov pushed a commit that referenced this pull request Nov 4, 2019
 - Merge Pull Request magento/graphql-ce#1030 from TomashKhamlai/graphql-ce:cover-customer-address-customizable-attribute
 - Merged commits:
   1. 56382f1
   2. a97ef07
   3. 461763e
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

MFTF Functional Test Coverage

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants