Force at least first and last name for CREDIT_CARD_NAME when using Wallet.

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 "base/guid.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/message_loop.h"
 #include "base/prefs/pref_service.h"
 #include "base/utf_string_conversions.h"
 #include "chrome/browser/ui/autofill/autofill_dialog_controller_impl.h"
@@ skipping 412 matching lines @@
       controller()->InputValidityMessage(iter->type, string16());
     }
   }
 }
 
 // Test for phone number validation.
 TEST_F(AutofillDialogControllerTest, PhoneNumberValidation) {
   // Construct DetailOutputMap from existing data.
   SwitchToAutofill();
 
-  EXPECT_CALL(*controller()->GetView(), ModelChanged()).Times(1);
-
   AutofillProfile full_profile(test::GetVerifiedProfile());
   controller()->GetTestingManager()->AddTestingProfile(&full_profile);
   controller()->EditClickedForSection(SECTION_SHIPPING);
 
   DetailOutputMap outputs;
   const DetailInputs& inputs =
       controller()->RequestedFieldsForSection(SECTION_SHIPPING);
 
   // Make sure country is United States.
   SetOutputValue(inputs, &outputs, ADDRESS_HOME_COUNTRY, "United States");
@@ skipping 47 matching lines @@
 
   // Input an invalid international phone number.
   SetOutputValue(inputs, &outputs, PHONE_HOME_WHOLE_NUMBER,
                  "+112333 892 70 12 39");
   validity_data =
       controller()->InputsAreValid(outputs,
                                    AutofillDialogController::VALIDATE_EDIT);
   EXPECT_EQ(1U, validity_data.count(PHONE_HOME_WHOLE_NUMBER));
 }
 
+TEST_F(AutofillDialogControllerTest, CardHolderNameValidation) {
+  // Construct DetailOutputMap from AutofillProfile data.
+  SwitchToAutofill();
+
+  AutofillProfile full_profile(test::GetVerifiedProfile());
+  controller()->GetTestingManager()->AddTestingProfile(&full_profile);
+  controller()->EditClickedForSection(SECTION_SHIPPING);
+
+  DetailOutputMap outputs;
+  const DetailInputs& inputs =
+      controller()->RequestedFieldsForSection(SECTION_CC);
+
+  // Input an empty card holder name with VALIDATE_FINAL.
+  SetOutputValue(inputs, &outputs, CREDIT_CARD_NAME, "");
+  ValidityData validity_data =
+      controller()->InputsAreValid(outputs,
+                                   AutofillDialogController::VALIDATE_FINAL);
+  EXPECT_EQ(1U, validity_data.count(CREDIT_CARD_NAME));
+
+  // Input an empty card holder name with VALIDATE_EDIT.
+  validity_data =
+      controller()->InputsAreValid(outputs,
+                                   AutofillDialogController::VALIDATE_EDIT);
+  EXPECT_EQ(0U, validity_data.count(CREDIT_CARD_NAME));
+
+  // Input a non-empty card holder name.
+  SetOutputValue(inputs, &outputs, CREDIT_CARD_NAME, "Bob");
+  validity_data =
+      controller()->InputsAreValid(outputs,
+                                   AutofillDialogController::VALIDATE_FINAL);
+  EXPECT_EQ(0U, validity_data.count(CREDIT_CARD_NAME));
+
+  // Switch to Wallet which only considers names with with at least two names to
+  // be valid.
+  SwitchToWallet();
+
+  // Setup some wallet state.
+  scoped_ptr<wallet::WalletItems> wallet_items = wallet::GetTestWalletItems();
+  wallet_items->AddAddress(wallet::GetTestShippingAddress());
+  wallet_items->AddInstrument(wallet::GetTestMaskedInstrument());
+  controller()->OnDidGetWalletItems(wallet_items.Pass());
+
+  controller()->EditClickedForSection(SECTION_CC_BILLING);
+
+  DetailOutputMap wallet_outputs;
+  const DetailInputs& wallet_inputs =
+      controller()->RequestedFieldsForSection(SECTION_CC_BILLING);
+
+  // Input an empty card holder name with VALIDATE_FINAL. Data source should not
+  // change this behavior.
+  SetOutputValue(wallet_inputs, &wallet_outputs, CREDIT_CARD_NAME, "");
+  validity_data =
+      controller()->InputsAreValid(wallet_outputs,
+                                   AutofillDialogController::VALIDATE_FINAL);
+  EXPECT_EQ(1U, validity_data.count(CREDIT_CARD_NAME));
+
+  // Input an empty card holder name with VALIDATE_EDIT. Data source should not
+  // change this behavior.
+  validity_data =
+      controller()->InputsAreValid(wallet_outputs,
+                                   AutofillDialogController::VALIDATE_EDIT);
+  EXPECT_EQ(0U, validity_data.count(CREDIT_CARD_NAME));
+
+  // Input a one name card holder name. Wallet does not currently support this.
+  SetOutputValue(wallet_inputs, &wallet_outputs, CREDIT_CARD_NAME, "Bob");
+  validity_data =
+      controller()->InputsAreValid(wallet_outputs,
+                                   AutofillDialogController::VALIDATE_FINAL);
+  EXPECT_EQ(1U, validity_data.count(CREDIT_CARD_NAME));
+
+  // Input a two name card holder name.
+  SetOutputValue(wallet_inputs, &wallet_outputs, CREDIT_CARD_NAME,
+                 "Bob Barker");
+  validity_data =
+      controller()->InputsAreValid(wallet_outputs,
+                                   AutofillDialogController::VALIDATE_FINAL);
+  EXPECT_EQ(0U, validity_data.count(CREDIT_CARD_NAME));
+
+  // Input a more than two name card holder name.
+  SetOutputValue(wallet_inputs, &wallet_outputs, CREDIT_CARD_NAME,
+                 "John Jacob Jingleheimer Schmidt");
+  validity_data =
+      controller()->InputsAreValid(wallet_outputs,
+                                   AutofillDialogController::VALIDATE_FINAL);
+  EXPECT_EQ(0U, validity_data.count(CREDIT_CARD_NAME));
+
+  // Input a card holder name with lots of crazy whitespace.
+  SetOutputValue(wallet_inputs, &wallet_outputs, CREDIT_CARD_NAME,
+                 "     \\n\\r John \\n  Jacob Jingleheimer \\t Schmidt  ");
+  validity_data =
+      controller()->InputsAreValid(wallet_outputs,
+                                   AutofillDialogController::VALIDATE_FINAL);
+  EXPECT_EQ(0U, validity_data.count(CREDIT_CARD_NAME));
+}
+
 TEST_F(AutofillDialogControllerTest, AutofillProfiles) {
   ui::MenuModel* shipping_model =
       controller()->MenuModelForSection(SECTION_SHIPPING);
   // Since the PersonalDataManager is empty, this should only have the
   // "use billing", "add new" and "manage" menu items.
   ASSERT_TRUE(shipping_model);
   EXPECT_EQ(3, shipping_model->GetItemCount());
   // On the other hand, the other models should be NULL when there's no
   // suggestion.
   EXPECT_FALSE(controller()->MenuModelForSection(SECTION_CC));
@@ skipping 1065 matching lines @@
   controller()->OnDidGetWalletItems(
       wallet::WalletItems::CreateWalletItems(dict).Pass());
 
   EXPECT_FALSE(controller()->MenuModelForSection(SECTION_CC_BILLING));
   // "Same as billing" and "Add address...".
   EXPECT_EQ(
       2, controller()->MenuModelForSection(SECTION_SHIPPING)->GetItemCount());
 }
 
 }  // namespace autofill