[Payments] Allow international phone from different country in shipping editor.

components/payments/core/payment_request_data_util_unittest.cc

 // Copyright 2017 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 "components/payments/core/payment_request_data_util.h"
 
 #include <memory>
 
 #include "base/json/json_writer.h"
+#include "base/macros.h"
 #include "base/strings/utf_string_conversions.h"
 #include "base/values.h"
 #include "components/autofill/core/browser/autofill_profile.h"
 #include "components/autofill/core/browser/autofill_test_utils.h"
 #include "components/autofill/core/browser/credit_card.h"
 #include "components/payments/core/basic_card_response.h"
 #include "components/payments/core/payment_address.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 namespace payments {
@@ skipping 63 matching lines @@
 }
 
 // Tests that the phone numbers are correctly formatted to display to the user.
 TEST(PaymentRequestDataUtilTest, FormatPhoneForDisplay) {
   EXPECT_EQ("+1 515-123-1234",
             payments::data_util::FormatPhoneForDisplay("5151231234", "US"));
   EXPECT_EQ("+33 1 42 68 53 00",
             payments::data_util::FormatPhoneForDisplay("142685300", "FR"));
 }
 
+// Test for the GetFormattedPhoneNumberForDisplay method.
+struct PhoneNumberFormatCase {
+  PhoneNumberFormatCase(const char* phone,
+                        const char* country,
+                        const char* locale,
+                        const char* expected_format)
+      : phone(phone),
+        country(country),
+        locale(locale),
+        expected_format(expected_format) {}
+
+  const char* phone;
+  const char* country;
+  const char* locale;

[Mathieu, 2017/05/29 16:34:00]

are you using this?

[sebsg, 2017/05/29 19:48:39]

Done.

+  const char* expected_format;
+};
+
+class GetFormattedPhoneNumberForDisplayTest
+    : public testing::TestWithParam<PhoneNumberFormatCase> {};
+
+TEST_P(GetFormattedPhoneNumberForDisplayTest,
+       GetFormattedPhoneNumberForDisplay) {
+  autofill::AutofillProfile profile;
+  profile.SetRawInfo(autofill::PHONE_HOME_WHOLE_NUMBER,
+                     base::UTF8ToUTF16(GetParam().phone));
+  profile.SetRawInfo(autofill::ADDRESS_HOME_COUNTRY,
+                     base::UTF8ToUTF16(GetParam().country));
+  EXPECT_EQ(GetParam().expected_format,
+            base::UTF16ToUTF8(
+                GetFormattedPhoneNumberForDisplay(profile, GetParam().locale)));
+}
+
+INSTANTIATE_TEST_CASE_P(

[Mathieu, 2017/05/29 16:34:00]

let's have a US phone in MX and MX phone in US block

[sebsg, 2017/05/29 19:48:39]

Done.

+    GetFormattedPhoneNumberForDisplay,
+    GetFormattedPhoneNumberForDisplayTest,
+    testing::Values(
+        /////////////////////
+        // US phone in US.

[Mathieu, 2017/05/29 16:34:00]

US phone in CA?

Let's also have a US phone in US block

[sebsg, 2017/05/29 19:48:39]

Done.

+        /////////////////////
+        // Formatted phone numbers.
+        PhoneNumberFormatCase("+1 415-555-5555", "CA", "", "+1 415-555-5555"),
+        PhoneNumberFormatCase("1 415-555-5555", "CA", "", "+1 415-555-5555"),
+        PhoneNumberFormatCase("415-555-5555", "CA", "", "+1 415-555-5555"),
+        // Raw phone numbers.
+        PhoneNumberFormatCase("+14155555555", "CA", "", "+1 415-555-5555"),
+        PhoneNumberFormatCase("14155555555", "CA", "", "+1 415-555-5555"),
+        PhoneNumberFormatCase("4155555555", "CA", "", "+1 415-555-5555"),
+
+        /////////////////////
+        // US phone in AU.
+        /////////////////////
+        // A US phone with the country code is correctly formatted as an US
+        // number.
+        PhoneNumberFormatCase("+1 415-555-5555", "AU", "", "+1 415-555-5555"),
+        PhoneNumberFormatCase("1 415-555-5555", "AU", "", "+1 415-555-5555"),
+        // Without a country code, the phone is formatted for the profile's
+        // country, even if it's invalid.
+        PhoneNumberFormatCase("415-555-5555", "AU", "", "+61 4155555555"),
+
+        /////////////////////
+        // AU phone in AU.
+        /////////////////////
+        // Formatted phone numbers.
+        PhoneNumberFormatCase("+61 2 9374 4000", "AU", "", "+61 2 9374 4000"),
+        PhoneNumberFormatCase("61 2 9374 4000", "AU", "", "+61 2 9374 4000"),
+        PhoneNumberFormatCase("02 9374 4000", "AU", "", "+61 2 9374 4000"),
+        PhoneNumberFormatCase("2 9374 4000", "AU", "", "+61 2 9374 4000"),
+        // Raw phone numbers.
+        PhoneNumberFormatCase("+61293744000", "AU", "", "+61 2 9374 4000"),
+        PhoneNumberFormatCase("61293744000", "AU", "", "+61 2 9374 4000"),
+        PhoneNumberFormatCase("0293744000", "AU", "", "+61 2 9374 4000"),
+        PhoneNumberFormatCase("293744000", "AU", "", "+61 2 9374 4000"),
+
+        /////////////////////
+        // AU phone in US.
+        /////////////////////
+        // An AU phone with the country code is correctly formatted as an AU
+        // number.
+        PhoneNumberFormatCase("+61 2 9374 4000", "US", "", "+61 2 9374 4000"),
+        PhoneNumberFormatCase("61 2 9374 4000", "US", "", "+61 2 9374 4000"),
+        // Without a country code, the phone is formatted for the profile's
+        // country.
+        // This local AU number fits the length of a US number, so it's
+        // formatted for US.
+        PhoneNumberFormatCase("02 9374 4000", "US", "", "+1 029-374-4000"),
+        // This local AU number is formatted as an US number, even if it's
+        // invlaid.
+        PhoneNumberFormatCase("2 9374 4000", "US", "", "+1 293744000")));
+
 }  // namespace data_util
 }  // namespace payments