Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(113)

Unified Diff: ios/chrome/browser/ui/payments/payment_request_coordinator_unittest.mm

Issue 2866793002: [Payment Request] Refactors PaymentRequestCoordinator. (Closed)
Patch Set: Rebase. Created 3 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: ios/chrome/browser/ui/payments/payment_request_coordinator_unittest.mm
diff --git a/ios/chrome/browser/ui/payments/payment_request_coordinator_unittest.mm b/ios/chrome/browser/ui/payments/payment_request_coordinator_unittest.mm
index f1071127c3d563d3d03001858c7aa31340eca01e..a1f6fee38551a45585a0f7ee43306ef3548fac69 100644
--- a/ios/chrome/browser/ui/payments/payment_request_coordinator_unittest.mm
+++ b/ios/chrome/browser/ui/payments/payment_request_coordinator_unittest.mm
@@ -37,8 +37,10 @@
@implementation PaymentRequestCoordinatorDelegateMock
typedef void (^mock_coordinator_cancel)(PaymentRequestCoordinator*);
-typedef void (^mock_coordinator_confirm)(PaymentRequestCoordinator*,
- web::PaymentResponse);
+typedef void (^mock_coordinator_complete)(PaymentRequestCoordinator*,
+ PaymentRequest*,
+ const autofill::CreditCard&,
+ const base::string16&);
typedef void (^mock_coordinator_select_shipping_address)(
PaymentRequestCoordinator*,
payments::PaymentAddress);
@@ -53,9 +55,11 @@ typedef void (^mock_coordinator_select_shipping_option)(
}
- (void)paymentRequestCoordinator:(PaymentRequestCoordinator*)coordinator
- didConfirmWithPaymentResponse:(web::PaymentResponse)paymentResponse {
- return static_cast<mock_coordinator_confirm>([self blockForSelector:_cmd])(
- coordinator, paymentResponse);
+ didCompletePaymentRequest:(PaymentRequest*)paymentRequest
+ card:(const autofill::CreditCard&)card
+ cvc:(const base::string16&)cvc {
+ return static_cast<mock_coordinator_complete>([self blockForSelector:_cmd])(
+ coordinator, paymentRequest, card, cvc);
}
- (void)paymentRequestCoordinator:(PaymentRequestCoordinator*)coordinator
@@ -148,20 +152,14 @@ TEST_F(PaymentRequestCoordinatorTest, FullCardRequestDidSucceed) {
id delegate_mock([[PaymentRequestCoordinatorDelegateMock alloc]
initWithRepresentedObject:delegate]);
SEL selector =
- @selector(paymentRequestCoordinator:didConfirmWithPaymentResponse:);
+ @selector(paymentRequestCoordinator:didCompletePaymentRequest:card:cvc:);
[delegate_mock onSelector:selector
callBlockExpectation:^(PaymentRequestCoordinator* callerCoordinator,
- web::PaymentResponse paymentResponse) {
- EXPECT_EQ(base::ASCIIToUTF16("4111111111111111"),
- paymentResponse.details.card_number);
- EXPECT_EQ(base::ASCIIToUTF16("Test User"),
- paymentResponse.details.cardholder_name);
- EXPECT_EQ(base::ASCIIToUTF16("11"),
- paymentResponse.details.expiry_month);
- EXPECT_EQ(base::ASCIIToUTF16("2022"),
- paymentResponse.details.expiry_year);
- EXPECT_EQ(base::ASCIIToUTF16("123"),
- paymentResponse.details.card_security_code);
+ PaymentRequest* paymentRequest,
+ const autofill::CreditCard& card,
+ const base::string16& cvc) {
+ EXPECT_EQ(credit_card_, card);
+ EXPECT_EQ(base::ASCIIToUTF16("123"), cvc);
EXPECT_EQ(coordinator, callerCoordinator);
}];
[coordinator setDelegate:delegate_mock];

Powered by Google App Engine
This is Rietveld 408576698