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

Unified Diff: chrome/browser/ui/autofill/autofill_dialog_controller_unittest.cc

Issue 23512003: new animation for rAc card generation (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fix unit tests Created 7 years, 3 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
Index: chrome/browser/ui/autofill/autofill_dialog_controller_unittest.cc
diff --git a/chrome/browser/ui/autofill/autofill_dialog_controller_unittest.cc b/chrome/browser/ui/autofill/autofill_dialog_controller_unittest.cc
index 00b17ffde6d9e2730087e6fd88b7154b05896d66..107e2f0e5e8706c89c69c36cf13bdc495bf3f242 100644
--- a/chrome/browser/ui/autofill/autofill_dialog_controller_unittest.cc
+++ b/chrome/browser/ui/autofill/autofill_dialog_controller_unittest.cc
@@ -271,6 +271,13 @@ class TestAutofillDialogController
OnWalletSigninError();
}
+ // Skips past the 2 second wait between FinishSubmit and DoFinishSubmit.
+ void ForceFinishSubmit() {
+#if defined(TOOLKIT_VIEWS)
+ DoFinishSubmit();
+#endif
+ }
+
MOCK_METHOD0(LoadRiskFingerprintData, void());
using AutofillDialogControllerImpl::OnDidLoadRiskFingerprintData;
using AutofillDialogControllerImpl::IsEditingExistingData;
@@ -1900,13 +1907,11 @@ TEST_F(AutofillDialogControllerTest, SubmitWithSigninErrorDoesntSetPref) {
::prefs::kAutofillDialogPayWithoutWallet));
}
-// Tests that there's an overlay shown while waiting for full wallet items,
-// and on first run an additional expository wallet overlay shown after full
-// wallet items are returned.
+// Tests that there's an overlay shown while waiting for full wallet items.
// TODO(estade): enable on other platforms when overlays are supported there.
#if defined(TOOLKIT_VIEWS)
TEST_F(AutofillDialogControllerTest, WalletFirstRun) {
- // Simulate first run.
+ // Simulate fist run.
PrefService* prefs = profile()->GetPrefs();
prefs->SetBoolean(::prefs::kAutofillDialogHasPaidWithWallet, false);
SetUpControllerWithFormData(DefaultFormData());
@@ -1923,28 +1928,12 @@ TEST_F(AutofillDialogControllerTest, WalletFirstRun) {
EXPECT_FALSE(controller()->GetDialogOverlay().image.IsEmpty());
EXPECT_FALSE(form_structure());
- controller()->OverlayButtonPressed();
- EXPECT_TRUE(prefs->GetBoolean(::prefs::kAutofillDialogHasPaidWithWallet));
+ // Don't wait for 2 seconds.
+ controller()->ForceFinishSubmit();
EXPECT_TRUE(form_structure());
}
#endif
-// On second run, the second overlay doesn't show.
-TEST_F(AutofillDialogControllerTest, WalletSecondRun) {
- SwitchToWallet();
- EXPECT_TRUE(controller()->GetDialogOverlay().image.IsEmpty());
-
- SubmitWithWalletItems(CompleteAndValidWalletItems());
- EXPECT_FALSE(controller()->GetDialogOverlay().image.IsEmpty());
-
- EXPECT_TRUE(profile()->GetPrefs()->GetBoolean(
- ::prefs::kAutofillDialogHasPaidWithWallet));
- controller()->OnDidGetFullWallet(wallet::GetTestFullWallet());
- EXPECT_TRUE(profile()->GetPrefs()->GetBoolean(
- ::prefs::kAutofillDialogHasPaidWithWallet));
- EXPECT_TRUE(form_structure());
-}
-
TEST_F(AutofillDialogControllerTest, ViewSubmitSetsPref) {
ASSERT_FALSE(profile()->GetPrefs()->HasPrefPath(
::prefs::kAutofillDialogPayWithoutWallet));
@@ -1984,6 +1973,7 @@ TEST_F(AutofillDialogControllerTest, ViewSubmitSetsPref) {
controller()->OnDidGetWalletItems(wallet_items.Pass());
controller()->OnAccept();
controller()->OnDidGetFullWallet(wallet::GetTestFullWallet());
+ controller()->ForceFinishSubmit();
EXPECT_TRUE(profile()->GetPrefs()->HasPrefPath(
::prefs::kAutofillDialogPayWithoutWallet));
@@ -2014,7 +2004,9 @@ TEST_F(AutofillDialogControllerTest, HideWalletEmail) {
controller()->OnAccept();
controller()->OnDidGetFullWallet(wallet::GetTestFullWallet());
+ controller()->ForceFinishSubmit();
+ ASSERT_TRUE(form_structure());
size_t i = 0;
for (; i < form_structure()->field_count(); ++i) {
if (form_structure()->field(i)->Type().GetStorableType() == EMAIL_ADDRESS) {
@@ -2031,6 +2023,8 @@ TEST_F(AutofillDialogControllerTest, AutofillTypes) {
controller()->OnDidGetWalletItems(CompleteAndValidWalletItems());
controller()->OnAccept();
controller()->OnDidGetFullWallet(wallet::GetTestFullWallet());
+ controller()->ForceFinishSubmit();
+ ASSERT_TRUE(form_structure());
ASSERT_EQ(20U, form_structure()->field_count());
EXPECT_EQ(EMAIL_ADDRESS,
form_structure()->field(0)->Type().GetStorableType());
@@ -2192,6 +2186,7 @@ TEST_F(AutofillDialogControllerTest, ShippingSectionCanBeHiddenForWallet) {
wallet_items->AddInstrument(wallet::GetTestMaskedInstrument());
SubmitWithWalletItems(wallet_items.Pass());
controller()->OnDidGetFullWallet(wallet::GetTestFullWalletInstrumentOnly());
+ controller()->ForceFinishSubmit();
EXPECT_TRUE(form_structure());
}
@@ -2280,6 +2275,7 @@ TEST_F(AutofillDialogControllerTest, GeneratedCardBubbleShown) {
SubmitWithWalletItems(CompleteAndValidWalletItems());
controller()->OnDidGetFullWallet(wallet::GetTestFullWallet());
+ controller()->ForceFinishSubmit();
controller()->ViewClosed();
EXPECT_EQ(0, mock_new_card_bubble_controller()->bubbles_shown());
« no previous file with comments | « chrome/browser/ui/autofill/autofill_dialog_controller_impl.cc ('k') | chrome/browser/ui/autofill/autofill_dialog_types.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698