ios/chrome/browser/ui/payments/payment_request_coordinator_unittest.mm - Issue 2887123002: Adds address normalization to iOS.

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

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

Issue 2887123002: Adds address normalization to iOS. (Closed)

Patch Set: Addresses nits from mahmadi@. 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

« no previous file with comments | « ios/chrome/browser/ui/payments/payment_request_coordinator.mm ('k') | ios/chrome/browser/ui/payments/payment_request_manager.mm » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

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 f053f263b1e753e0157d6c8feba8e35fca9565c8..1b2ce034d3ad19277e80e8411f8cadf4042b99b8 100644

--- a/ios/chrome/browser/ui/payments/payment_request_coordinator_unittest.mm

+++ b/ios/chrome/browser/ui/payments/payment_request_coordinator_unittest.mm

@@ -38,7 +38,6 @@

typedef void (^mock_coordinator_cancel)(PaymentRequestCoordinator*);

typedef void (^mock_coordinator_complete)(PaymentRequestCoordinator*,

- PaymentRequest*,

const autofill::CreditCard&,

const base::string16&);

typedef void (^mock_coordinator_select_shipping_address)(

@@ -55,11 +54,10 @@ typedef void (^mock_coordinator_select_shipping_option)(

}

- (void)paymentRequestCoordinator:(PaymentRequestCoordinator*)coordinator

- didCompletePaymentRequest:(PaymentRequest*)paymentRequest

- card:(const autofill::CreditCard&)card

- verificationCode:(const base::string16&)verificationCode {

+ didCompletePaymentRequestWithCard:(const autofill::CreditCard&)card

+ verificationCode:(const base::string16&)verificationCode {

return static_cast<mock_coordinator_complete>([self blockForSelector:_cmd])(

- coordinator, paymentRequest, card, verificationCode);

+ coordinator, card, verificationCode);

}

- (void)paymentRequestCoordinator:(PaymentRequestCoordinator*)coordinator

@@ -151,11 +149,11 @@ TEST_F(PaymentRequestCoordinatorTest, FullCardRequestDidSucceed) {

mockForProtocol:@protocol(PaymentMethodSelectionCoordinatorDelegate)];

id delegate_mock([[PaymentRequestCoordinatorDelegateMock alloc]

initWithRepresentedObject:delegate]);

- SEL selector = @selector(paymentRequestCoordinator:didCompletePaymentRequest:

- card:verificationCode:);

+ SEL selector =

+ @selector(paymentRequestCoordinator:didCompletePaymentRequestWithCard

+ :verificationCode:);

[delegate_mock onSelector:selector

callBlockExpectation:^(PaymentRequestCoordinator* callerCoordinator,

- PaymentRequest* paymentRequest,

const autofill::CreditCard& card,

const base::string16& verificationCode) {

EXPECT_EQ(credit_card_, card);