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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: components/autofill/core/browser/payments/full_card_request_unittest.cc

Issue 2249773002: Scan card holder name. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 4 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
« components/autofill/core/browser/credit_card.h ('K') | « components/autofill/core/browser/credit_card_unittest.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/autofill/core/browser/payments/full_card_request_unittest.cc
diff --git a/components/autofill/core/browser/payments/full_card_request_unittest.cc b/components/autofill/core/browser/payments/full_card_request_unittest.cc
index 8f93cd765f8b1578e6895842459338e0c7b7914c..aa4edf581862a1b132d2b8bab58382b500e6b268 100644
--- a/components/autofill/core/browser/payments/full_card_request_unittest.cc
+++ b/components/autofill/core/browser/payments/full_card_request_unittest.cc
@@ -156,7 +156,8 @@ TEST_F(FullCardRequestTest, GetFullCardPanAndCvcForLocalCard) {
EXPECT_CALL(*client(), ShowUnmaskPrompt(_, _, _));
EXPECT_CALL(*client(), OnUnmaskVerificationResult(AutofillClient::SUCCESS));
- request()->GetFullCard(CreditCard(base::ASCIIToUTF16("4111"), 12, 2050),
+ request()->GetFullCard(CreditCard(base::ASCIIToUTF16("Alice"),
+ base::ASCIIToUTF16("4111"), 12, 2050),
AutofillClient::UNMASK_FOR_AUTOFILL,
delegate()->AsWeakPtr());
CardUnmaskDelegate::UnmaskResponse response;
@@ -174,7 +175,8 @@ TEST_F(FullCardRequestTest, GetFullCardPanAndCvcForFullServerCard) {
EXPECT_CALL(*client(), ShowUnmaskPrompt(_, _, _));
EXPECT_CALL(*client(), OnUnmaskVerificationResult(AutofillClient::SUCCESS));
- CreditCard full_server_card(base::ASCIIToUTF16("4111"), 12, 2050);
+ CreditCard full_server_card(base::ASCIIToUTF16("Alice"),
+ base::ASCIIToUTF16("4111"), 12, 2050);
full_server_card.set_record_type(CreditCard::FULL_SERVER_CARD);
request()->GetFullCard(full_server_card, AutofillClient::UNMASK_FOR_AUTOFILL,
delegate()->AsWeakPtr());
@@ -196,7 +198,8 @@ TEST_F(FullCardRequestTest,
EXPECT_CALL(*personal_data(), UpdateServerCreditCard(_)).Times(0);
EXPECT_CALL(*client(), OnUnmaskVerificationResult(AutofillClient::SUCCESS));
- CreditCard full_server_card(base::ASCIIToUTF16("4111"), 12, 2050);
+ CreditCard full_server_card(base::ASCIIToUTF16("Alice"),
+ base::ASCIIToUTF16("4111"), 12, 2050);
full_server_card.set_record_type(CreditCard::FULL_SERVER_CARD);
full_server_card.SetServerStatus(CreditCard::EXPIRED);
request()->GetFullCard(full_server_card, AutofillClient::UNMASK_FOR_AUTOFILL,
@@ -223,7 +226,8 @@ TEST_F(FullCardRequestTest, GetFullCardPanAndCvcForExpiredFullServerCard) {
base::Time::Exploded today;
base::Time::Now().LocalExplode(&today);
- CreditCard full_server_card(base::ASCIIToUTF16("4111"), 12, today.year - 1);
+ CreditCard full_server_card(base::ASCIIToUTF16("Alice"),
+ base::ASCIIToUTF16("4111"), 12, today.year - 1);
full_server_card.set_record_type(CreditCard::FULL_SERVER_CARD);
full_server_card.SetServerStatus(CreditCard::OK);
request()->GetFullCard(full_server_card, AutofillClient::UNMASK_FOR_AUTOFILL,
@@ -262,7 +266,8 @@ TEST_F(FullCardRequestTest, SecondRequestOkAfterFirstFinished) {
EXPECT_CALL(*client(), OnUnmaskVerificationResult(AutofillClient::SUCCESS))
.Times(2);
- request()->GetFullCard(CreditCard(base::ASCIIToUTF16("4111"), 12, 2050),
+ request()->GetFullCard(CreditCard(base::ASCIIToUTF16("Alice"),
+ base::ASCIIToUTF16("4111"), 12, 2050),
AutofillClient::UNMASK_FOR_AUTOFILL,
delegate()->AsWeakPtr());
CardUnmaskDelegate::UnmaskResponse response;
@@ -270,7 +275,8 @@ TEST_F(FullCardRequestTest, SecondRequestOkAfterFirstFinished) {
ui_delegate()->OnUnmaskResponse(response);
ui_delegate()->OnUnmaskPromptClosed();
- request()->GetFullCard(CreditCard(base::ASCIIToUTF16("4111"), 12, 2050),
+ request()->GetFullCard(CreditCard(base::ASCIIToUTF16("Alice"),
+ base::ASCIIToUTF16("4111"), 12, 2050),
AutofillClient::UNMASK_FOR_AUTOFILL,
delegate()->AsWeakPtr());
ui_delegate()->OnUnmaskResponse(response);
@@ -400,7 +406,8 @@ TEST_F(FullCardRequestTest, UpdateExpDateForFullServerCard) {
EXPECT_CALL(*client(), ShowUnmaskPrompt(_, _, _));
EXPECT_CALL(*client(), OnUnmaskVerificationResult(AutofillClient::SUCCESS));
- CreditCard full_server_card(base::ASCIIToUTF16("4111"), 10, 2000);
+ CreditCard full_server_card(base::ASCIIToUTF16("Alice"),
+ base::ASCIIToUTF16("4111"), 10, 2000);
full_server_card.set_record_type(CreditCard::FULL_SERVER_CARD);
request()->GetFullCard(full_server_card, AutofillClient::UNMASK_FOR_AUTOFILL,
delegate()->AsWeakPtr());
@@ -427,7 +434,8 @@ TEST_F(FullCardRequestTest, UpdateExpDateForLocalCard) {
base::Time::Exploded today;
base::Time::Now().LocalExplode(&today);
request()->GetFullCard(
- CreditCard(base::ASCIIToUTF16("4111"), 12, today.year - 1),
+ CreditCard(base::ASCIIToUTF16("Alice"), base::ASCIIToUTF16("4111"), 12,
+ today.year - 1),
AutofillClient::UNMASK_FOR_AUTOFILL, delegate()->AsWeakPtr());
CardUnmaskDelegate::UnmaskResponse response;
response.cvc = base::ASCIIToUTF16("123");
@@ -525,7 +533,8 @@ TEST_F(FullCardRequestTest, IsGettingFullCardForLocalCard) {
EXPECT_FALSE(request()->IsGettingFullCard());
- request()->GetFullCard(CreditCard(base::ASCIIToUTF16("4111"), 12, 2050),
+ request()->GetFullCard(CreditCard(base::ASCIIToUTF16("Alice"),
+ base::ASCIIToUTF16("4111"), 12, 2050),
AutofillClient::UNMASK_FOR_AUTOFILL,
delegate()->AsWeakPtr());
« components/autofill/core/browser/credit_card.h ('K') | « components/autofill/core/browser/credit_card_unittest.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698