Implement the UI logic behind the Sign In promo in the password bubble

chrome/browser/ui/passwords/manage_passwords_bubble_model_unittest.cc

 // Copyright 2014 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 "chrome/browser/ui/passwords/manage_passwords_bubble_model.h"
 
 #include <utility>
 #include <vector>
 
 #include "base/memory/ptr_util.h"
@@ skipping 10 matching lines @@
 #include "components/browser_sync/browser/profile_sync_service_mock.h"
 #include "components/password_manager/core/browser/mock_password_store.h"
 #include "components/password_manager/core/browser/password_bubble_experiment.h"
 #include "components/password_manager/core/browser/password_manager_metrics_util.h"
 #include "components/password_manager/core/browser/password_manager_test_utils.h"
 #include "components/password_manager/core/browser/statistics_table.h"
 #include "components/password_manager/core/common/credential_manager_types.h"
 #include "components/password_manager/core/common/password_manager_pref_names.h"
 #include "components/password_manager/core/common/password_manager_ui.h"
 #include "components/prefs/pref_service.h"
+#include "components/variations/variations_associated_data.h"
 #include "content/public/test/test_browser_thread_bundle.h"
 #include "content/public/test/web_contents_tester.h"
 #include "testing/gmock/include/gmock/gmock.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 using password_bubble_experiment::kBrandingExperimentName;
+using password_bubble_experiment::kChromeSignInPasswordPromoExperimentName;
+using password_bubble_experiment::kChromeSignInPasswordPromoThresholdParam;
 using password_bubble_experiment::kSmartLockBrandingGroupName;
 using password_bubble_experiment::kSmartLockBrandingSavePromptOnlyGroupName;
 using ::testing::AnyNumber;
 using ::testing::Return;
 using ::testing::ReturnRef;
 using ::testing::_;
 
 namespace {
 
+const char kFakeGroup[] = "FakeGroup";
+const char kSignInPromoDismissalReasonMetric[] = "PasswordManager.SignInPromo";
 const char kSiteOrigin[] = "http://example.com/login";
 const char kUsername[] = "Admin";
 const char kUIDismissalReasonMetric[] = "PasswordManager.UIDismissalReason";
 
 class TestSyncService : public ProfileSyncServiceMock {
  public:
   explicit TestSyncService(Profile* profile)
       : ProfileSyncServiceMock(CreateProfileSyncServiceParamsForTest(profile)),
         smartlock_enabled_(false) {}
   ~TestSyncService() override {}
@@ skipping 43 matching lines @@
         profile(),
         password_manager::BuildPasswordStore<
             content::BrowserContext,
             testing::StrictMock<password_manager::MockPasswordStore>>);
   }
 
   void TearDown() override {
     // Reset WebContents first. It can happen if the user closes the tab.
     test_web_contents_.reset();
     model_.reset();
+    variations::testing::ClearAllVariationIDs();
+    variations::testing::ClearAllVariationParams();
   }
 
   PrefService* prefs() { return profile_.GetPrefs(); }
 
   TestingProfile* profile() { return &profile_; }
 
   password_manager::MockPasswordStore* GetStore() {
     return static_cast<password_manager::MockPasswordStore*>(
         PasswordStoreFactory::GetInstance()
             ->GetForProfile(profile(), ServiceAccessType::EXPLICIT_ACCESS)
@@ skipping 234 matching lines @@
       password_manager::prefs::kWasSavePrompFirstRunExperienceShown));
 }
 
 TEST_F(ManagePasswordsBubbleModelTest, OnBrandLinkClicked) {
   PretendPasswordWaiting();
 
   EXPECT_CALL(*controller(), NavigateToSmartLockHelpPage());
   model()->OnBrandLinkClicked();
 }
 
+TEST_F(ManagePasswordsBubbleModelTest, SuppressSignInPromo) {
+  base::HistogramTester histogram_tester;
+  PretendPasswordWaiting();
+  EXPECT_CALL(*GetStore(), RemoveSiteStatsImpl(GURL(kSiteOrigin).GetOrigin()));
+  EXPECT_CALL(*controller(), SavePassword());
+  model()->OnSaveClicked();
+
+  EXPECT_FALSE(model()->ReplaceToShowSignInPromoIfNeeded());
+  DestroyModel();
+  histogram_tester.ExpectTotalCount(kSignInPromoDismissalReasonMetric, 0);
+}
+
+TEST_F(ManagePasswordsBubbleModelTest, SignInPromoOK) {
+  ASSERT_TRUE(base::FieldTrialList::CreateFieldTrial(
+      kChromeSignInPasswordPromoExperimentName, kFakeGroup));
+  variations::AssociateVariationParams(
+      kChromeSignInPasswordPromoExperimentName, kFakeGroup,
+      {{kChromeSignInPasswordPromoThresholdParam, "3"}});
+  base::HistogramTester histogram_tester;
+  PretendPasswordWaiting();
+  EXPECT_CALL(*GetStore(), RemoveSiteStatsImpl(GURL(kSiteOrigin).GetOrigin()));
+  EXPECT_CALL(*controller(), SavePassword());
+  model()->OnSaveClicked();
+
+  EXPECT_TRUE(model()->ReplaceToShowSignInPromoIfNeeded());
+  EXPECT_CALL(*controller(), NavigateToChromeSignIn());
+  model()->OnSignInToChromeClicked();
+  DestroyModel();
+  histogram_tester.ExpectUniqueSample(
+      kUIDismissalReasonMetric,
+      password_manager::metrics_util::CLICKED_SAVE, 1);
+  histogram_tester.ExpectUniqueSample(
+      kSignInPromoDismissalReasonMetric,
+      password_manager::metrics_util::CHROME_SIGNIN_OK, 1);
+  EXPECT_TRUE(prefs()->GetBoolean(
+      password_manager::prefs::kWasSignInPasswordPromoClicked));
+}
+
+TEST_F(ManagePasswordsBubbleModelTest, SignInPromoCancel) {
+  ASSERT_TRUE(base::FieldTrialList::CreateFieldTrial(
+      kChromeSignInPasswordPromoExperimentName, kFakeGroup));
+  variations::AssociateVariationParams(
+      kChromeSignInPasswordPromoExperimentName, kFakeGroup,
+      {{kChromeSignInPasswordPromoThresholdParam, "3"}});
+  base::HistogramTester histogram_tester;
+  PretendPasswordWaiting();
+  EXPECT_CALL(*GetStore(), RemoveSiteStatsImpl(GURL(kSiteOrigin).GetOrigin()));
+  EXPECT_CALL(*controller(), SavePassword());
+  model()->OnSaveClicked();
+
+  EXPECT_TRUE(model()->ReplaceToShowSignInPromoIfNeeded());
+  model()->OnSkipSignInClicked();
+  DestroyModel();
+  histogram_tester.ExpectUniqueSample(
+      kUIDismissalReasonMetric,
+      password_manager::metrics_util::CLICKED_SAVE, 1);
+  histogram_tester.ExpectUniqueSample(
+      kSignInPromoDismissalReasonMetric,
+      password_manager::metrics_util::CHROME_SIGNIN_CANCEL, 1);
+  EXPECT_TRUE(prefs()->GetBoolean(
+      password_manager::prefs::kWasSignInPasswordPromoClicked));
+}
+
+TEST_F(ManagePasswordsBubbleModelTest, SignInPromoDismiss) {
+  ASSERT_TRUE(base::FieldTrialList::CreateFieldTrial(
+      kChromeSignInPasswordPromoExperimentName, kFakeGroup));
+  variations::AssociateVariationParams(
+      kChromeSignInPasswordPromoExperimentName, kFakeGroup,
+      {{kChromeSignInPasswordPromoThresholdParam, "3"}});
+  base::HistogramTester histogram_tester;
+  PretendPasswordWaiting();
+  EXPECT_CALL(*GetStore(), RemoveSiteStatsImpl(GURL(kSiteOrigin).GetOrigin()));
+  EXPECT_CALL(*controller(), SavePassword());
+  model()->OnSaveClicked();
+
+  EXPECT_TRUE(model()->ReplaceToShowSignInPromoIfNeeded());
+  DestroyModel();
+  histogram_tester.ExpectUniqueSample(
+      kUIDismissalReasonMetric,
+      password_manager::metrics_util::CLICKED_SAVE, 1);
+  histogram_tester.ExpectUniqueSample(
+      kSignInPromoDismissalReasonMetric,
+      password_manager::metrics_util::CHROME_SIGNIN_DISMISSED, 1);
+  EXPECT_FALSE(prefs()->GetBoolean(
+      password_manager::prefs::kWasSignInPasswordPromoClicked));
+}
+
 namespace {
 
 enum class SmartLockStatus { ENABLE, DISABLE };
 
 struct TitleTestCase {
   const char* experiment_group;
   SmartLockStatus smartlock_status;
   const char* expected_title;
 };
 
@@ skipping 101 matching lines @@
      ManageLinkTarget::SETTINGS_PAGE},
     {nullptr, SmartLockStatus::DISABLE, ManageLinkTarget::SETTINGS_PAGE},
     {"Default", SmartLockStatus::DISABLE, ManageLinkTarget::SETTINGS_PAGE},
 };
 
 }  // namespace
 
 INSTANTIATE_TEST_CASE_P(Default,
                         ManagePasswordsBubbleModelManageLinkTest,
                         ::testing::ValuesIn(kManageLinkTestCases));