Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[Behat][API] Fixed adding simple product scenarios#13616

Merged
lchrusciel merged 1 commit into
Sylius:1.11from
ernestWarwas:api-fix-adding-simple-products
Feb 9, 2022
Merged

[Behat][API] Fixed adding simple product scenarios#13616
lchrusciel merged 1 commit into
Sylius:1.11from
ernestWarwas:api-fix-adding-simple-products

Conversation

@ernestWarwas
Copy link
Copy Markdown
Contributor

@ernestWarwas ernestWarwas commented Feb 8, 2022

Q A
Branch? 1.11
Bug fix? no
New feature? no
BC breaks? no
License MIT

Simple product concept does not exists in API so we need to tag all behat scenarios creating simple product as no-api

@ernestWarwas ernestWarwas requested a review from a team as a code owner February 8, 2022 15:11
@ernestWarwas ernestWarwas added the API APIs related issues and PRs. label Feb 8, 2022
@ernestWarwas ernestWarwas changed the title [WIP][API] Fixed adding simple product scenarios [WIP][Behat][API] Fixed adding simple product scenarios Feb 8, 2022
@ernestWarwas ernestWarwas changed the title [WIP][Behat][API] Fixed adding simple product scenarios [Behat][API] Fixed adding simple product scenarios Feb 8, 2022
Comment thread features/product/managing_products/adding_product_with_associations.feature Outdated
@ernestWarwas ernestWarwas force-pushed the api-fix-adding-simple-products branch from 509dc14 to 5b920a4 Compare February 9, 2022 10:58
Comment thread features/product/managing_products/editing_product_slug.feature
@ernestWarwas ernestWarwas force-pushed the api-fix-adding-simple-products branch from 43c0447 to 5b920a4 Compare February 9, 2022 11:44
[Behat] add no-api tag to simple product create tests

[Behat] add no-api tag to simple product create tests

[Behat] review fixes

[Behat] added/removed no-api tags from some scenarios

[Behat] added/removed no-api tags from some scenarios

[Behat] added/removed no-api tags from some scenarios

[Behat] creating product with an autogenerated slug now creates configurable product instead of simple

[Behat] suggested changes
@ernestWarwas ernestWarwas force-pushed the api-fix-adding-simple-products branch from f1961ad to 568bb0a Compare February 9, 2022 13:11
@lchrusciel lchrusciel merged commit 4954f1d into Sylius:1.11 Feb 9, 2022
@lchrusciel
Copy link
Copy Markdown
Contributor

Thanks, @ernestWarwas! 🥇

lchrusciel added a commit that referenced this pull request Feb 9, 2022
…ation in api tests (ernestWarwas)

This PR was merged into the 1.11 branch.

Discussion
----------

| Q               | A
| --------------- | -----
| Branch?         | 1.11
| Bug fix?        | no
| New feature?    | no
| BC breaks?      | no
| Related tickets | second part of #13616 
| License         | MIT

<!--
 - Bug fixes must be submitted against the 1.10 or 1.11 branch(the lowest possible)
 - Features and deprecations must be submitted against the master branch
 - Make sure that the correct base branch is set

 To be sure you are not breaking any Backward Compatibilities, check the documentation:
 https://docs.sylius.com/en/latest/book/organization/backward-compatibility-promise.html
-->


Commits
-------

1d3f168 [Behat] add no-api tag to simple product create tests part 2
35f4a6a [Behat] completly remove simple product concept from api tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

API APIs related issues and PRs.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants