-
-
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
[Shop] Change payment method on an order #5419
Conversation
tuka217
commented
Jul 4, 2016
Q | A |
---|---|
Bug fix? | no |
New feature? | yes |
BC breaks? | no |
Related tickets | #5415 |
License | MIT |
0e262d1
to
fbd8de9
Compare
@@ -910,6 +910,22 @@ public function iCompleteAddressingStepWithEmail($email, AddressInterface $addre | |||
} | |||
|
|||
/** | |||
* @When I choose payment method :paymentMethodName | |||
*/ | |||
public function iChoosePaymentMethod1($paymentMethodName) |
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.
iChoosePaymentMethod
@tuka217 rebase required ;) And there is a typo in PR title |
And I am redirected back to the order payment page | ||
When I try to pay again with offline method | ||
Then I should be redirected back to the thank you page | ||
And I proceed selecting "PayPal Express Checkout" payment method |
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.
proceed
-> proceeded
236ca85
to
7e929e7
Compare
8a88962
to
ebf0c87
Compare
154deb7
to
5106ee1
Compare
@@ -32,4 +32,6 @@ public function mockCollaborator($className); | |||
* @return MockInterface | |||
*/ | |||
public function mockService($serviceId, $className); | |||
|
|||
public function unmockAllServices(); |
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.
Is it needed? I think it has been removed eventually.
@tuka217 Last fixes + rebase and I think it'll be ready to go ;) |
Modify scenario about changing payment method on order
@@ -881,6 +889,14 @@ private function createDefaultAddress() | |||
} | |||
|
|||
/** | |||
* @Given I confirm my changes |
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.
Given
-> When
Thank you Ania, nice work! |
[Shop] Change payment method on an order