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

Unified Diff: components/autofill/core/browser/autofill_field_unittest.cc

Issue 2819183004: [Autofill] Change MasterCard to Mastercard in UI and related tests (Closed)
Patch Set: Resolved conflict 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/autofill_field_unittest.cc
diff --git a/components/autofill/core/browser/autofill_field_unittest.cc b/components/autofill/core/browser/autofill_field_unittest.cc
index 5731f376ace6dbe29a999cd1e5bc7b8ff3944ffe..ef6b2e87bea1ac7e3ef9fa64bde24ed648f0415f 100644
--- a/components/autofill/core/browser/autofill_field_unittest.cc
+++ b/components/autofill/core/browser/autofill_field_unittest.cc
@@ -799,7 +799,7 @@ TEST_F(AutofillFieldTest, FillSelectControlWithTwoDigitCreditCardYear) {
}
TEST_F(AutofillFieldTest, FillSelectControlWithCreditCardType) {
- std::vector<const char*> kCreditCardTypes = {"Visa", "Master Card", "AmEx",
+ std::vector<const char*> kCreditCardTypes = {"Visa", "Mastercard", "AmEx",
"discover"};
AutofillField field;
test::CreateTestSelectField(kCreditCardTypes, &field);
@@ -812,8 +812,17 @@ TEST_F(AutofillFieldTest, FillSelectControlWithCreditCardType) {
// Filling should be able to handle intervening whitespace:
AutofillField::FillFormField(
+ field, ASCIIToUTF16("Master card"), "en-US", "en-US", &field);
+ EXPECT_EQ(ASCIIToUTF16("Mastercard"), field.value);
+
+ // Mastercard is sometimes shown as MasterCard or Master Card:
+ AutofillField::FillFormField(
field, ASCIIToUTF16("MasterCard"), "en-US", "en-US", &field);
- EXPECT_EQ(ASCIIToUTF16("Master Card"), field.value);
+ EXPECT_EQ(ASCIIToUTF16("Mastercard"), field.value);
+
+ AutofillField::FillFormField(
+ field, ASCIIToUTF16("Master Card"), "en-US", "en-US", &field);
+ EXPECT_EQ(ASCIIToUTF16("Mastercard"), field.value);
// American Express is sometimes abbreviated as AmEx:
AutofillField::FillFormField(
« no previous file with comments | « components/autofill/core/browser/autofill_field.cc ('k') | components/autofill/core/browser/autofill_manager_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698