-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Change sylius.behat.context.ui.email context to sylius.behat.context.api.email #13767
Conversation
The base of this pull-request was changed, you need fetch and reset your local branch Unless you added new commits (to this branch) locally that you did not push yet, Feel free to ask for assistance when you get stuck 👍 |
7b1a619
to
0ab3231
Compare
There were some steps missing in API Context, so when #13829 PR will be merged the pipelines will be green :) |
fccd9b2
to
db02e8c
Compare
47d1260
to
5581210
Compare
@@ -263,7 +263,6 @@ | |||
<argument type="service" id="sylius.behat.current_page_resolver" /> | |||
<argument type="service" id="sylius.behat.notification_checker" /> | |||
<argument type="service" id="sylius.behat.java_script_test_helper" /> | |||
<argument type="service" id="behat.mink.default_session" /> |
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.
ManagingTaxonsContext has only 7 arguments ;p
Thank you, Zbigniew! 🥇 |
I don't know why, but the API resources had
sylius.behat.context.ui.email
suites defined, not sure whether they are misconfiguration or feature. Could not find reason for it in the PRs.