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
[CatalogPromotion][API] Add validation for creating catalog promotion with actions #13071
[CatalogPromotion][API] Add validation for creating catalog promotion with actions #13071
Conversation
GSadee
commented
Sep 8, 2021
Q | A |
---|---|
Branch? | master |
Bug fix? | no |
New feature? | yes |
BC breaks? | no |
Deprecations? | no |
Related tickets | based on #13037 |
License | MIT |
340352e
to
b68da67
Compare
json_encode([ | ||
'name' => 'T-Shirts discount', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we should consider something for our contract tests that will allow us to use json files with request body. Because those tests are getting enormously huge with just this funky 💃 code.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We can probably use json files without problems, but then these tests will become less readable, so I'm not 100% convinced. We should think about that and probably refactor our contracts tests in a separate PR
$response = $this->client->getResponse(); | ||
|
||
$this->assertResponse( | ||
$response, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
$response = $this->client->getResponse(); | |
$this->assertResponse( | |
$response, | |
$this->assertResponse( | |
$this->client->getResponse(), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll change that in another PR
…(GSadee) This PR was merged into the 1.11-dev branch. Discussion ---------- | Q | A | --------------- | ----- | Branch? | master | Bug fix? | no | New feature? | no | BC breaks? | no | Deprecations? | no | Related tickets | #13071 | License | MIT Commits ------- 2c8cc59 [CatalogPromotion][API] Refactor validation of rules