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

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

Issue 2849523003: Add billing address as a mandatory field of Payments credit cards. (Closed)
Patch Set: Components Unittests fix 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: 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 a9455bee66bb53fe2c74cc2e63f505e51196c348..5d01e9b84e10bc7072e12648d104f9b3c6610abe 100644
--- a/components/autofill/core/browser/payments/full_card_request_unittest.cc
+++ b/components/autofill/core/browser/payments/full_card_request_unittest.cc
@@ -167,7 +167,7 @@ TEST_F(FullCardRequestTest, GetFullCardPanAndCvcForLocalCard) {
OnUnmaskVerificationResult(AutofillClient::SUCCESS));
CreditCard card;
- test::SetCreditCardInfo(&card, nullptr, "4111", "12", "2050");
+ test::SetCreditCardInfo(&card, nullptr, "4111", "12", "2050", "1");
request()->GetFullCard(card, AutofillClient::UNMASK_FOR_AUTOFILL,
result_delegate()->AsWeakPtr(),
ui_delegate()->AsWeakPtr());
@@ -188,7 +188,8 @@ TEST_F(FullCardRequestTest, GetFullCardPanAndCvcForFullServerCard) {
OnUnmaskVerificationResult(AutofillClient::SUCCESS));
CreditCard full_server_card(CreditCard::FULL_SERVER_CARD, "server_id");
- test::SetCreditCardInfo(&full_server_card, nullptr, "4111", "12", "2050");
+ test::SetCreditCardInfo(&full_server_card, nullptr, "4111", "12", "2050",
+ "1");
request()->GetFullCard(full_server_card, AutofillClient::UNMASK_FOR_AUTOFILL,
result_delegate()->AsWeakPtr(),
ui_delegate()->AsWeakPtr());
@@ -212,7 +213,8 @@ TEST_F(FullCardRequestTest,
OnUnmaskVerificationResult(AutofillClient::SUCCESS));
CreditCard full_server_card(CreditCard::FULL_SERVER_CARD, "server_id");
- test::SetCreditCardInfo(&full_server_card, nullptr, "4111", "12", "2050");
+ test::SetCreditCardInfo(&full_server_card, nullptr, "4111", "12", "2050",
+ "1");
full_server_card.SetServerStatus(CreditCard::EXPIRED);
request()->GetFullCard(full_server_card, AutofillClient::UNMASK_FOR_AUTOFILL,
result_delegate()->AsWeakPtr(),
@@ -242,7 +244,8 @@ TEST_F(FullCardRequestTest, GetFullCardPanAndCvcForExpiredFullServerCard) {
base::Time::Now().LocalExplode(&today);
CreditCard full_server_card(CreditCard::FULL_SERVER_CARD, "server_id");
test::SetCreditCardInfo(&full_server_card, nullptr, "4111", "12",
- base::StringPrintf("%d", today.year - 1).c_str());
+ base::StringPrintf("%d", today.year - 1).c_str(),
+ "1");
full_server_card.SetServerStatus(CreditCard::OK);
request()->GetFullCard(full_server_card, AutofillClient::UNMASK_FOR_AUTOFILL,
result_delegate()->AsWeakPtr(),
@@ -286,7 +289,7 @@ TEST_F(FullCardRequestTest, SecondRequestOkAfterFirstFinished) {
.Times(2);
CreditCard card;
- test::SetCreditCardInfo(&card, nullptr, "4111", "12", "2050");
+ test::SetCreditCardInfo(&card, nullptr, "4111", "12", "2050", "1");
request()->GetFullCard(card, AutofillClient::UNMASK_FOR_AUTOFILL,
result_delegate()->AsWeakPtr(),
ui_delegate()->AsWeakPtr());
@@ -435,7 +438,8 @@ TEST_F(FullCardRequestTest, UpdateExpDateForFullServerCard) {
OnUnmaskVerificationResult(AutofillClient::SUCCESS));
CreditCard full_server_card(CreditCard::FULL_SERVER_CARD, "server_id");
- test::SetCreditCardInfo(&full_server_card, nullptr, "4111", "10", "2000");
+ test::SetCreditCardInfo(&full_server_card, nullptr, "4111", "10", "2000",
+ "1");
request()->GetFullCard(full_server_card, AutofillClient::UNMASK_FOR_AUTOFILL,
result_delegate()->AsWeakPtr(),
ui_delegate()->AsWeakPtr());
@@ -465,7 +469,8 @@ TEST_F(FullCardRequestTest, UpdateExpDateForLocalCard) {
base::Time::Now().LocalExplode(&today);
CreditCard card;
test::SetCreditCardInfo(&card, nullptr, "4111", "10",
- base::StringPrintf("%d", today.year - 1).c_str());
+ base::StringPrintf("%d", today.year - 1).c_str(),
+ "1");
request()->GetFullCard(card, AutofillClient::UNMASK_FOR_AUTOFILL,
result_delegate()->AsWeakPtr(),
ui_delegate()->AsWeakPtr());
@@ -574,7 +579,7 @@ TEST_F(FullCardRequestTest, IsGettingFullCardForLocalCard) {
EXPECT_FALSE(request()->IsGettingFullCard());
CreditCard card;
- test::SetCreditCardInfo(&card, nullptr, "4111", "12", "2050");
+ test::SetCreditCardInfo(&card, nullptr, "4111", "12", "2050", "1");
request()->GetFullCard(card, AutofillClient::UNMASK_FOR_AUTOFILL,
result_delegate()->AsWeakPtr(),
ui_delegate()->AsWeakPtr());

Powered by Google App Engine
This is Rietveld 408576698