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

[Api][Checkout][Order] Add tests for accessing order right after completing checkout #11715

Merged
merged 1 commit into from
Aug 5, 2020

Conversation

Tomanhez
Copy link
Contributor

@Tomanhez Tomanhez commented Aug 4, 2020

Q A
Branch? master
Bug fix? no
New feature? yes
BC breaks? no
Deprecations? no
License MIT

@Tomanhez Tomanhez requested a review from a team as a code owner August 4, 2020 11:20
@GSadee GSadee added the API APIs related issues and PRs. label Aug 4, 2020
src/Sylius/Behat/Context/Api/Shop/CheckoutContext.php Outdated Show resolved Hide resolved

$this->sharedStorage->set(
'order_number',
$this->responseChecker->getValue($this->client->getResponse(), 'number')
Copy link
Member

Choose a reason for hiding this comment

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

Is the number not null here? I guess that it should be because the order number is assigned after checkout

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The number is already available here:
Screenshot 2020-08-04 at 14 44 26

@Tomanhez Tomanhez force-pushed the order-summary-after-checkout branch 2 times, most recently from b553f9b to 8d96efc Compare August 4, 2020 12:46
@Tomanhez Tomanhez force-pushed the order-summary-after-checkout branch from 8d96efc to 6b5bd16 Compare August 4, 2020 19:36
@Tomanhez Tomanhez force-pushed the order-summary-after-checkout branch from 6b5bd16 to 3a2492c Compare August 5, 2020 06:31
@GSadee GSadee merged commit 193903c into Sylius:master Aug 5, 2020
@GSadee
Copy link
Member

GSadee commented Aug 5, 2020

Thanks, Tomasz! 🥇

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.

2 participants