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

Unified Diff: chrome/browser/autofill/phone_number_unittest.cc

Issue 12434004: Move remaining Autofill code to //components/autofill. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix long lines Created 7 years, 9 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 | « chrome/browser/autofill/phone_number_i18n_unittest.cc ('k') | chrome/browser/autofill/risk/fingerprint.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/autofill/phone_number_unittest.cc
diff --git a/chrome/browser/autofill/phone_number_unittest.cc b/chrome/browser/autofill/phone_number_unittest.cc
deleted file mode 100644
index 5c96aabb1e4843f673a13b6d953e9d59397b7112..0000000000000000000000000000000000000000
--- a/chrome/browser/autofill/phone_number_unittest.cc
+++ /dev/null
@@ -1,203 +0,0 @@
-// Copyright (c) 2011 The Chromium Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-#include "base/string16.h"
-#include "base/utf_string_conversions.h"
-#include "chrome/browser/autofill/autofill_profile.h"
-#include "chrome/browser/autofill/field_types.h"
-#include "chrome/browser/autofill/phone_number.h"
-#include "chrome/browser/autofill/phone_number_i18n.h"
-#include "testing/gtest/include/gtest/gtest.h"
-
-TEST(PhoneNumberTest, Matcher) {
- AutofillProfile profile;
- profile.SetCountryCode("US");
- // Set phone number so country_code == 1, city_code = 650, number = 2345678.
- string16 phone(ASCIIToUTF16("1 [650] 234-5678"));
- PhoneNumber phone_number(&profile);
- phone_number.SetInfo(PHONE_HOME_WHOLE_NUMBER, phone, "US");
-
- FieldTypeSet matching_types;
- phone_number.GetMatchingTypes(string16(), "US", &matching_types);
- EXPECT_EQ(1U, matching_types.size());
- EXPECT_TRUE(matching_types.find(EMPTY_TYPE) != matching_types.end());
- matching_types.clear();
- phone_number.GetMatchingTypes(ASCIIToUTF16("1"), "US", &matching_types);
- EXPECT_EQ(1U, matching_types.size());
- EXPECT_TRUE(matching_types.find(PHONE_HOME_COUNTRY_CODE) !=
- matching_types.end());
- matching_types.clear();
- phone_number.GetMatchingTypes(ASCIIToUTF16("16"), "US", &matching_types);
- EXPECT_EQ(0U, matching_types.size());
- phone_number.GetMatchingTypes(ASCIIToUTF16("165"), "US", &matching_types);
- EXPECT_EQ(0U, matching_types.size());
- phone_number.GetMatchingTypes(ASCIIToUTF16("1650"), "US", &matching_types);
- EXPECT_EQ(0U, matching_types.size());
- phone_number.GetMatchingTypes(ASCIIToUTF16("16502"), "US", &matching_types);
- EXPECT_EQ(0U, matching_types.size());
- phone_number.GetMatchingTypes(ASCIIToUTF16("165023"), "US", &matching_types);
- EXPECT_EQ(0U, matching_types.size());
- phone_number.GetMatchingTypes(ASCIIToUTF16("1650234"), "US", &matching_types);
- EXPECT_EQ(0U, matching_types.size());
- matching_types.clear();
- phone_number.GetMatchingTypes(ASCIIToUTF16("16502345678"), "US",
- &matching_types);
- EXPECT_EQ(1U, matching_types.size());
- EXPECT_TRUE(matching_types.find(PHONE_HOME_WHOLE_NUMBER) !=
- matching_types.end());
- matching_types.clear();
- phone_number.GetMatchingTypes(ASCIIToUTF16("650"), "US", &matching_types);
- EXPECT_EQ(1U, matching_types.size());
- EXPECT_TRUE(matching_types.find(PHONE_HOME_CITY_CODE) !=
- matching_types.end());
- matching_types.clear();
- phone_number.GetMatchingTypes(ASCIIToUTF16("2345678"), "US", &matching_types);
- EXPECT_EQ(1U, matching_types.size());
- EXPECT_TRUE(matching_types.find(PHONE_HOME_NUMBER) != matching_types.end());
- matching_types.clear();
- phone_number.GetMatchingTypes(ASCIIToUTF16("234"), "US", &matching_types);
- EXPECT_EQ(1U, matching_types.size());
- EXPECT_TRUE(matching_types.find(PHONE_HOME_NUMBER) != matching_types.end());
- matching_types.clear();
- phone_number.GetMatchingTypes(ASCIIToUTF16("5678"), "US", &matching_types);
- EXPECT_EQ(1U, matching_types.size());
- EXPECT_TRUE(matching_types.find(PHONE_HOME_NUMBER) != matching_types.end());
- matching_types.clear();
- phone_number.GetMatchingTypes(ASCIIToUTF16("2345"), "US", &matching_types);
- EXPECT_EQ(0U, matching_types.size());
- matching_types.clear();
- phone_number.GetMatchingTypes(ASCIIToUTF16("6502345678"), "US",
- &matching_types);
- EXPECT_EQ(1U, matching_types.size());
- EXPECT_TRUE(matching_types.find(PHONE_HOME_CITY_AND_NUMBER) !=
- matching_types.end());
- matching_types.clear();
- phone_number.GetMatchingTypes(ASCIIToUTF16("(650)2345678"), "US",
- &matching_types);
- EXPECT_EQ(1U, matching_types.size());
- EXPECT_TRUE(matching_types.find(PHONE_HOME_CITY_AND_NUMBER) !=
- matching_types.end());
-}
-
-// Verify that PhoneNumber::SetInfo() correctly formats the incoming number.
-TEST(PhoneNumberTest, SetInfo) {
- AutofillProfile profile;
- profile.SetCountryCode("US");
-
- PhoneNumber phone(&profile);
- EXPECT_EQ(string16(), phone.GetRawInfo(PHONE_HOME_WHOLE_NUMBER));
-
- // Set the formatted info directly.
- EXPECT_TRUE(phone.SetInfo(PHONE_HOME_WHOLE_NUMBER,
- ASCIIToUTF16("(650) 234-5678"), "US"));
- EXPECT_EQ(ASCIIToUTF16("(650) 234-5678"),
- phone.GetRawInfo(PHONE_HOME_WHOLE_NUMBER));
-
- // Unformatted numbers should be formatted.
- EXPECT_TRUE(phone.SetInfo(PHONE_HOME_WHOLE_NUMBER,
- ASCIIToUTF16("8887776666"), "US"));
- EXPECT_EQ(ASCIIToUTF16("(888) 777-6666"),
- phone.GetRawInfo(PHONE_HOME_WHOLE_NUMBER));
-
- // Differently formatted numbers should be re-formatted.
- EXPECT_TRUE(phone.SetInfo(PHONE_HOME_WHOLE_NUMBER,
- ASCIIToUTF16("800-432-8765"), "US"));
- EXPECT_EQ(ASCIIToUTF16("(800) 432-8765"),
- phone.GetRawInfo(PHONE_HOME_WHOLE_NUMBER));
-
- // Invalid numbers should not be stored. In the US, phone numbers cannot
- // start with the digit '1'.
- EXPECT_FALSE(phone.SetInfo(PHONE_HOME_WHOLE_NUMBER,
- ASCIIToUTF16("650111111"), "US"));
- EXPECT_EQ(string16(), phone.GetRawInfo(PHONE_HOME_WHOLE_NUMBER));
-}
-
-// Test that cached phone numbers are correctly invalidated and updated.
-TEST(PhoneNumberTest, UpdateCachedPhoneNumber) {
- AutofillProfile profile;
- profile.SetCountryCode("US");
-
- PhoneNumber phone(&profile);
- phone.SetRawInfo(PHONE_HOME_WHOLE_NUMBER, ASCIIToUTF16("6502345678"));
- EXPECT_EQ(ASCIIToUTF16("650"), phone.GetInfo(PHONE_HOME_CITY_CODE, "US"));
-
- // Update the area code.
- phone.SetRawInfo(PHONE_HOME_WHOLE_NUMBER, ASCIIToUTF16("8322345678"));
- EXPECT_EQ(ASCIIToUTF16("832"), phone.GetInfo(PHONE_HOME_CITY_CODE, "US"));
-
- // Change the phone number to have a UK format, but try to parse with the
- // wrong locale.
- phone.SetRawInfo(PHONE_HOME_WHOLE_NUMBER, ASCIIToUTF16("07023456789"));
- EXPECT_EQ(string16(), phone.GetInfo(PHONE_HOME_CITY_CODE, "US"));
-
- // Now try parsing using the correct locale. Note that the profile's country
- // code should override the app locale, which is still set to "US".
- profile.SetCountryCode("GB");
- phone.SetRawInfo(PHONE_HOME_WHOLE_NUMBER, ASCIIToUTF16("07023456789"));
- EXPECT_EQ(ASCIIToUTF16("70"), phone.GetInfo(PHONE_HOME_CITY_CODE, "US"));
-}
-
-TEST(PhoneNumberTest, PhoneCombineHelper) {
- AutofillProfile profile;
- profile.SetCountryCode("US");
-
- PhoneNumber::PhoneCombineHelper number1;
- EXPECT_FALSE(number1.SetInfo(ADDRESS_BILLING_CITY,
- ASCIIToUTF16("1")));
- EXPECT_TRUE(number1.SetInfo(PHONE_HOME_COUNTRY_CODE,
- ASCIIToUTF16("1")));
- EXPECT_TRUE(number1.SetInfo(PHONE_HOME_CITY_CODE,
- ASCIIToUTF16("650")));
- EXPECT_TRUE(number1.SetInfo(PHONE_HOME_NUMBER,
- ASCIIToUTF16("2345678")));
- string16 parsed_phone;
- EXPECT_TRUE(number1.ParseNumber(profile, "en-US", &parsed_phone));
- // International format as it has a country code.
- EXPECT_EQ(ASCIIToUTF16("+1 650-234-5678"), parsed_phone);
-
- PhoneNumber::PhoneCombineHelper number3;
- EXPECT_TRUE(number3.SetInfo(PHONE_HOME_CITY_CODE,
- ASCIIToUTF16("650")));
- EXPECT_TRUE(number3.SetInfo(PHONE_HOME_NUMBER,
- ASCIIToUTF16("2345680")));
- EXPECT_TRUE(number3.ParseNumber(profile, "en-US", &parsed_phone));
- // National format as it does not have a country code.
- EXPECT_EQ(ASCIIToUTF16("(650) 234-5680"), parsed_phone);
-
- PhoneNumber::PhoneCombineHelper number4;
- EXPECT_TRUE(number4.SetInfo(PHONE_HOME_CITY_CODE,
- ASCIIToUTF16("123"))); // Incorrect city code.
- EXPECT_TRUE(number4.SetInfo(PHONE_HOME_NUMBER,
- ASCIIToUTF16("2345680")));
- EXPECT_FALSE(number4.ParseNumber(profile, "en-US", &parsed_phone));
- EXPECT_EQ(string16(), parsed_phone);
-
- PhoneNumber::PhoneCombineHelper number5;
- EXPECT_TRUE(number5.SetInfo(PHONE_HOME_CITY_AND_NUMBER,
- ASCIIToUTF16("6502345681")));
- EXPECT_TRUE(number5.ParseNumber(profile, "en-US", &parsed_phone));
- EXPECT_EQ(ASCIIToUTF16("(650) 234-5681"), parsed_phone);
-
- PhoneNumber::PhoneCombineHelper number6;
- EXPECT_TRUE(number6.SetInfo(PHONE_HOME_CITY_CODE,
- ASCIIToUTF16("650")));
- EXPECT_TRUE(number6.SetInfo(PHONE_HOME_NUMBER,
- ASCIIToUTF16("234")));
- EXPECT_TRUE(number6.SetInfo(PHONE_HOME_NUMBER,
- ASCIIToUTF16("5682")));
- EXPECT_TRUE(number6.ParseNumber(profile, "en-US", &parsed_phone));
- EXPECT_EQ(ASCIIToUTF16("(650) 234-5682"), parsed_phone);
-
- // Ensure parsing is possible when falling back to detecting the country code
- // based on the app locale.
- PhoneNumber::PhoneCombineHelper number7;
- EXPECT_TRUE(number7.SetInfo(PHONE_HOME_CITY_CODE,
- ASCIIToUTF16("650")));
- EXPECT_TRUE(number7.SetInfo(PHONE_HOME_NUMBER,
- ASCIIToUTF16("234")));
- EXPECT_TRUE(number7.SetInfo(PHONE_HOME_NUMBER,
- ASCIIToUTF16("5682")));
- EXPECT_TRUE(number7.ParseNumber(AutofillProfile(), "en-US", &parsed_phone));
- EXPECT_EQ(ASCIIToUTF16("(650) 234-5682"), parsed_phone);
-}
« no previous file with comments | « chrome/browser/autofill/phone_number_i18n_unittest.cc ('k') | chrome/browser/autofill/risk/fingerprint.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698