rAc: support street-address.

chrome/browser/ui/autofill/autofill_dialog_controller_unittest.cc

 // Copyright (c) 2012 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 <map>
 
 #include "base/bind.h"
 #include "base/command_line.h"
 #include "base/guid.h"
 #include "base/memory/scoped_ptr.h"
@@ skipping 1161 matching lines @@
             form_structure()->field(1)->Type().GetStorableType());
   EXPECT_EQ(PHONE_BILLING, form_structure()->field(1)->Type().group());
   EXPECT_EQ(shipping_profile.GetRawInfo(PHONE_HOME_WHOLE_NUMBER),
             form_structure()->field(0)->value);
   EXPECT_EQ(billing_profile.GetRawInfo(PHONE_HOME_WHOLE_NUMBER),
             form_structure()->field(1)->value);
   EXPECT_NE(form_structure()->field(1)->value,
             form_structure()->field(0)->value);
 }
 
+// Similar to the above, but tests that street-address (i.e. all lines of the
+// street address) is successfully filled for both shipping and billing
+// sections.
+TEST_F(AutofillDialogControllerTest, BillingVsShippingStreetAddress) {
+  FormFieldData shipping_address;
+  shipping_address.autocomplete_attribute = "shipping street-address";
+  FormFieldData billing_address;
+  billing_address.autocomplete_attribute = "billing street-address";
+  FormFieldData shipping_address_textarea;
+  shipping_address.autocomplete_attribute = "shipping street-address";
+  shipping_address.form_control_type = "textarea";

[Ilya Sherman, 2014/01/08 00:09:35]

Hmm, did you mean these to be modifying shipping_address_textarea?

[Evan Stade, 2014/01/08 22:16:56]

ah yes, I did mean to. Good thing the tests didn't pass with this mistake
present!

+  FormFieldData billing_address_textarea;
+  billing_address.autocomplete_attribute = "billing street-address";
+  billing_address.form_control_type = "textarea";

[Ilya Sherman, 2014/01/08 00:09:35]

Likewise, did you mean these to be modifying billing_address_textarea?

[Evan Stade, 2014/01/08 22:16:56]

Done.

+
+  FormData form_data;
+  form_data.fields.push_back(shipping_address);
+  form_data.fields.push_back(billing_address);
+  form_data.fields.push_back(shipping_address_textarea);
+  form_data.fields.push_back(billing_address_textarea);
+  SetUpControllerWithFormData(form_data);
+
+  SwitchToAutofill();
+
+  // The profile that will be chosen for the shipping section.
+  AutofillProfile shipping_profile(test::GetVerifiedProfile());
+  // The profile that will be chosen for the billing section.
+  AutofillProfile billing_profile(test::GetVerifiedProfile2());
+  CreditCard credit_card(test::GetVerifiedCreditCard());
+  controller()->GetTestingManager()->AddTestingProfile(&shipping_profile);
+  controller()->GetTestingManager()->AddTestingProfile(&billing_profile);
+  controller()->GetTestingManager()->AddTestingCreditCard(&credit_card);
+  ui::MenuModel* billing_model =
+      controller()->MenuModelForSection(SECTION_BILLING);
+  billing_model->ActivatedAt(1);
+
+  controller()->OnAccept();
+  ASSERT_EQ(4U, form_structure()->field_count());
+  EXPECT_EQ(ADDRESS_HOME_STREET_ADDRESS,
+            form_structure()->field(0)->Type().GetStorableType());

[Ilya Sherman, 2014/01/08 00:09:35]

Why do you check this for fields 0 and 1, but not 2 and 3?

[Evan Stade, 2014/01/08 22:16:56]

Because that would be redundant, imo. That should only depend on the
autocomplete attribute, which is not different between |*_address| and
|*_address_textarea|. Redundancy doesn't hurt necessarily, but it does make the
test longer, harder to read, more unwieldy to change or refactor, etc., so
generally I prefer to avoid redundancy.

However in this case, I do not feel particularly strongly about it if you want
me to change it.

+  EXPECT_EQ(ADDRESS_HOME, form_structure()->field(0)->Type().group());
+  EXPECT_EQ(ADDRESS_HOME_STREET_ADDRESS,
+            form_structure()->field(1)->Type().GetStorableType());
+  EXPECT_EQ(ADDRESS_BILLING, form_structure()->field(1)->Type().group());
+  // Inexact matching; single-line inputs get the address data concatenated but
+  // separated by commas.
+  EXPECT_TRUE(StartsWith(form_structure()->field(0)->value,
+                         shipping_profile.GetRawInfo(ADDRESS_HOME_LINE1),
+                         true));
+  EXPECT_TRUE(EndsWith(form_structure()->field(0)->value,
+                       shipping_profile.GetRawInfo(ADDRESS_HOME_LINE2),
+                       true));
+  EXPECT_TRUE(StartsWith(form_structure()->field(1)->value,
+                         billing_profile.GetRawInfo(ADDRESS_HOME_LINE1),
+                         true));
+  EXPECT_TRUE(EndsWith(form_structure()->field(1)->value,
+                       billing_profile.GetRawInfo(ADDRESS_HOME_LINE2),
+                       true));
+  // The textareas should be an exact match.
+  EXPECT_EQ(shipping_profile.GetRawInfo(ADDRESS_HOME_STREET_ADDRESS),
+            form_structure()->field(2)->value);
+  EXPECT_EQ(billing_profile.GetRawInfo(ADDRESS_HOME_STREET_ADDRESS),
+            form_structure()->field(3)->value);
+
+  EXPECT_NE(form_structure()->field(1)->value,
+            form_structure()->field(0)->value);
+  EXPECT_NE(form_structure()->field(3)->value,
+            form_structure()->field(2)->value);
+}
+
 TEST_F(AutofillDialogControllerTest, AcceptLegalDocuments) {
   for (size_t i = 0; i < 2; ++i) {
     SCOPED_TRACE(testing::Message() << "Case " << i);
 
     EXPECT_CALL(*controller()->GetTestingWalletClient(),
                 AcceptLegalDocuments(_, _));
     EXPECT_CALL(*controller()->GetTestingWalletClient(), GetFullWallet(_));
     EXPECT_CALL(*controller(), LoadRiskFingerprintData());
 
     EXPECT_TRUE(controller()->LegalDocumentLinks().empty());
@@ skipping 1571 matching lines @@
   ui::ResourceBundle& rb = ui::ResourceBundle::GetSharedInstance();
   for (size_t i = 0; i < arraysize(kSupportedCardIdrs); ++i) {
     SCOPED_TRACE(base::IntToString(i));
     gfx::Image supported_card_icon = rb.GetImageNamed(kSupportedCardIdrs[i]);
     EXPECT_GE(placeholder_icon.Width(), supported_card_icon.Width());
     EXPECT_GE(placeholder_icon.Height(), supported_card_icon.Height());
   }
 }
 
 }  // namespace autofill