| OLD | NEW |
| 1 // Copyright 2013 The Chromium Authors. All rights reserved. | 1 // Copyright 2013 The Chromium Authors. All rights reserved. |
| 2 // Use of this source code is governed by a BSD-style license that can be | 2 // Use of this source code is governed by a BSD-style license that can be |
| 3 // found in the LICENSE file. | 3 // found in the LICENSE file. |
| 4 | 4 |
| 5 #include "base/prefs/pref_service.h" | 5 #include "base/prefs/pref_service.h" |
| 6 #include "chrome/browser/ui/autofill/autofill_dialog_models.h" | 6 #include "base/utf_string_conversions.h" |
| 7 #include "chrome/browser/ui/autofill/account_chooser_model.h" |
| 7 #include "chrome/common/pref_names.h" | 8 #include "chrome/common/pref_names.h" |
| 8 #include "chrome/test/base/testing_profile.h" | 9 #include "chrome/test/base/testing_profile.h" |
| 9 #include "components/autofill/browser/autofill_metrics.h" | 10 #include "components/autofill/browser/autofill_metrics.h" |
| 10 #include "testing/gmock/include/gmock/gmock.h" | 11 #include "testing/gmock/include/gmock/gmock.h" |
| 11 #include "testing/gtest/include/gtest/gtest.h" | 12 #include "testing/gtest/include/gtest/gtest.h" |
| 12 | 13 |
| 13 namespace autofill { | 14 namespace autofill { |
| 14 | 15 |
| 15 namespace { | 16 namespace { |
| 16 | 17 |
| 17 class TestAccountChooserModel : public AccountChooserModel { | 18 class TestAccountChooserModel : public AccountChooserModel { |
| 18 public: | 19 public: |
| 19 TestAccountChooserModel(AccountChooserModelDelegate* delegate, | 20 TestAccountChooserModel(AccountChooserModelDelegate* delegate, |
| 20 PrefService* prefs, | 21 PrefService* prefs, |
| 21 const AutofillMetrics& metric_logger) | 22 const AutofillMetrics& metric_logger) |
| 22 : AccountChooserModel(delegate, prefs, metric_logger, | 23 : AccountChooserModel(delegate, prefs, metric_logger, |
| 23 DIALOG_TYPE_REQUEST_AUTOCOMPLETE) {} | 24 DIALOG_TYPE_REQUEST_AUTOCOMPLETE) {} |
| 24 virtual ~TestAccountChooserModel() {} | 25 virtual ~TestAccountChooserModel() {} |
| 25 | 26 |
| 27 using AccountChooserModel::kActiveWalletItemId; |
| 28 using AccountChooserModel::kAutofillItemId; |
| 29 |
| 26 private: | 30 private: |
| 27 DISALLOW_COPY_AND_ASSIGN(TestAccountChooserModel); | 31 DISALLOW_COPY_AND_ASSIGN(TestAccountChooserModel); |
| 28 }; | 32 }; |
| 29 | 33 |
| 30 class MockAccountChooserModelDelegate : public AccountChooserModelDelegate { | 34 class MockAccountChooserModelDelegate : public AccountChooserModelDelegate { |
| 31 public: | 35 public: |
| 32 MockAccountChooserModelDelegate() {} | 36 MockAccountChooserModelDelegate() {} |
| 33 virtual ~MockAccountChooserModelDelegate() {} | 37 virtual ~MockAccountChooserModelDelegate() {} |
| 34 | 38 |
| 35 MOCK_METHOD0(AccountChoiceChanged, void()); | 39 MOCK_METHOD0(AccountChoiceChanged, void()); |
| 40 MOCK_METHOD0(UpdateAccountChooserView, void()); |
| 36 }; | 41 }; |
| 37 | 42 |
| 38 class AccountChooserModelTest : public testing::Test { | 43 class AccountChooserModelTest : public testing::Test { |
| 39 public: | 44 public: |
| 40 AccountChooserModelTest() | 45 AccountChooserModelTest() |
| 41 : model_(&delegate_, profile_.GetPrefs(), metric_logger_) {} | 46 : model_(&delegate_, profile_.GetPrefs(), metric_logger_) {} |
| 42 virtual ~AccountChooserModelTest() {} | 47 virtual ~AccountChooserModelTest() {} |
| 43 | 48 |
| 44 Profile* profile() { return &profile_; } | 49 Profile* profile() { return &profile_; } |
| 45 MockAccountChooserModelDelegate* delegate() { return &delegate_; } | 50 MockAccountChooserModelDelegate* delegate() { return &delegate_; } |
| (...skipping 36 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 82 // Check that nothing changes while this dialog is running if a pref changes | 87 // Check that nothing changes while this dialog is running if a pref changes |
| 83 // (this could cause subtle bugs or annoyances if a user closes another | 88 // (this could cause subtle bugs or annoyances if a user closes another |
| 84 // running dialog). | 89 // running dialog). |
| 85 profile()->GetPrefs()->SetBoolean( | 90 profile()->GetPrefs()->SetBoolean( |
| 86 ::prefs::kAutofillDialogPayWithoutWallet, true); | 91 ::prefs::kAutofillDialogPayWithoutWallet, true); |
| 87 EXPECT_TRUE(model()->WalletIsSelected()); | 92 EXPECT_TRUE(model()->WalletIsSelected()); |
| 88 } | 93 } |
| 89 | 94 |
| 90 TEST_F(AccountChooserModelTest, HandlesError) { | 95 TEST_F(AccountChooserModelTest, HandlesError) { |
| 91 EXPECT_CALL(*delegate(), AccountChoiceChanged()).Times(1); | 96 EXPECT_CALL(*delegate(), AccountChoiceChanged()).Times(1); |
| 97 EXPECT_CALL(*delegate(), UpdateAccountChooserView()).Times(1); |
| 92 | 98 |
| 93 ASSERT_TRUE(model()->WalletIsSelected()); | 99 ASSERT_TRUE(model()->WalletIsSelected()); |
| 94 ASSERT_TRUE(model()->IsCommandIdEnabled(AccountChooserModel::kWalletItemId)); | 100 ASSERT_TRUE(model()->IsCommandIdEnabled( |
| 101 TestAccountChooserModel::kActiveWalletItemId)); |
| 95 | 102 |
| 96 model()->SetHadWalletError(); | 103 model()->SetHadWalletError(); |
| 97 EXPECT_FALSE(model()->WalletIsSelected()); | 104 EXPECT_FALSE(model()->WalletIsSelected()); |
| 98 EXPECT_FALSE(model()->IsCommandIdEnabled(AccountChooserModel::kWalletItemId)); | 105 EXPECT_FALSE(model()->IsCommandIdEnabled( |
| 106 TestAccountChooserModel::kActiveWalletItemId)); |
| 99 } | 107 } |
| 100 | 108 |
| 101 TEST_F(AccountChooserModelTest, HandlesSigninError) { | 109 TEST_F(AccountChooserModelTest, HandlesSigninError) { |
| 102 EXPECT_CALL(*delegate(), AccountChoiceChanged()).Times(1); | 110 EXPECT_CALL(*delegate(), AccountChoiceChanged()).Times(1); |
| 111 EXPECT_CALL(*delegate(), UpdateAccountChooserView()).Times(2); |
| 103 | 112 |
| 113 // 0. "Unknown" wallet account, we don't know if the user is signed-in yet. |
| 104 ASSERT_TRUE(model()->WalletIsSelected()); | 114 ASSERT_TRUE(model()->WalletIsSelected()); |
| 105 ASSERT_TRUE(model()->IsCommandIdEnabled(AccountChooserModel::kWalletItemId)); | 115 ASSERT_TRUE(model()->IsCommandIdEnabled( |
| 116 TestAccountChooserModel::kActiveWalletItemId)); |
| 117 ASSERT_TRUE(model()->IsActiveWalletAccountSelected()); |
| 118 ASSERT_FALSE(model()->HasAccountsToChoose()); |
| 119 ASSERT_EQ(2, model()->GetItemCount()); |
| 120 EXPECT_EQ(string16(), model()->active_wallet_account_name()); |
| 106 | 121 |
| 122 // 1. "Known" wallet account (e.g. after active/passive/automatic sign-in). |
| 123 // Calls UpdateAccountChooserView. |
| 124 const string16 kAccount1 = ASCIIToUTF16("john.doe@gmail.com"); |
| 125 model()->SetActiveWalletAccountName(kAccount1); |
| 126 ASSERT_TRUE(model()->WalletIsSelected()); |
| 127 ASSERT_TRUE(model()->IsCommandIdEnabled( |
| 128 TestAccountChooserModel::kActiveWalletItemId)); |
| 129 ASSERT_TRUE(model()->IsActiveWalletAccountSelected()); |
| 130 ASSERT_TRUE(model()->HasAccountsToChoose()); |
| 131 EXPECT_EQ(2, model()->GetItemCount()); |
| 132 EXPECT_EQ(kAccount1, model()->active_wallet_account_name()); |
| 133 |
| 134 // 2. Sign-in failure. |
| 135 // Autofill data should be selected and be the only valid choice. |
| 136 // Calls UpdateAccountChooserView. |
| 137 // Calls AccountChoiceChanged. |
| 107 model()->SetHadWalletSigninError(); | 138 model()->SetHadWalletSigninError(); |
| 108 EXPECT_FALSE(model()->WalletIsSelected()); | 139 EXPECT_FALSE(model()->WalletIsSelected()); |
| 109 EXPECT_TRUE(model()->IsCommandIdEnabled(AccountChooserModel::kWalletItemId)); | 140 EXPECT_TRUE(model()->IsCommandIdEnabled( |
| 141 TestAccountChooserModel::kActiveWalletItemId)); |
| 142 EXPECT_FALSE(model()->IsActiveWalletAccountSelected()); |
| 143 EXPECT_FALSE(model()->HasAccountsToChoose()); |
| 144 EXPECT_EQ(1, model()->GetItemCount()); |
| 145 EXPECT_EQ(string16(), model()->active_wallet_account_name()); |
| 110 } | 146 } |
| 111 | 147 |
| 112 TEST_F(AccountChooserModelTest, RespectsUserChoice) { | 148 TEST_F(AccountChooserModelTest, RespectsUserChoice) { |
| 113 EXPECT_CALL(*delegate(), AccountChoiceChanged()).Times(2); | 149 EXPECT_CALL(*delegate(), AccountChoiceChanged()).Times(2); |
| 114 | 150 |
| 115 model()->ExecuteCommand(AccountChooserModel::kAutofillItemId, 0); | 151 model()->ExecuteCommand(TestAccountChooserModel::kAutofillItemId, 0); |
| 116 EXPECT_FALSE(model()->WalletIsSelected()); | 152 EXPECT_FALSE(model()->WalletIsSelected()); |
| 117 | 153 |
| 118 model()->ExecuteCommand(AccountChooserModel::kWalletItemId, 0); | 154 model()->ExecuteCommand(TestAccountChooserModel::kActiveWalletItemId, 0); |
| 119 EXPECT_TRUE(model()->WalletIsSelected()); | 155 EXPECT_TRUE(model()->WalletIsSelected()); |
| 120 } | 156 } |
| 121 | 157 |
| 122 } // namespace autofill | 158 } // namespace autofill |
| OLD | NEW |