[CrOS Tether] Open the Tether settings page when a tether notification body is clicked.

chrome/browser/chromeos/net/tether_notification_presenter_unittest.cc

 // Copyright 2017 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/chromeos/net/tether_notification_presenter.h"
 
 #include <memory>
 
 #include "base/memory/ptr_util.h"
 #include "base/observer_list.h"
+#include "chrome/test/base/testing_profile.h"
 #include "components/cryptauth/remote_device_test_util.h"
+#include "content/public/test/test_browser_thread_bundle.h"
 #include "testing/gmock/include/gmock/gmock.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "ui/message_center/fake_message_center.h"
 #include "ui/message_center/message_center_observer.h"
 
 namespace chromeos {
 
 namespace tether {
 
 namespace {
@@ skipping 107 matching lines @@
                              bool shared) override {}
 
     void ConnectToNetworkId(const std::string& network_id) override {
       network_id_to_connect_ = network_id;
     }
 
    private:
     std::string network_id_to_connect_;
   };
 
+  class TestSettingsUiDelegate
+      : public TetherNotificationPresenter::SettingsUiDelegate {
+   public:
+    TestSettingsUiDelegate() : last_profile_(nullptr) {}

[stevenjb, 2017/05/03 23:07:44]

nit: set to nullptr inline in declaration.

[Kyle Horimoto, 2017/05/03 23:25:48]

Done.

+    ~TestSettingsUiDelegate() override {}
+
+    Profile* last_profile() { return last_profile_; }
+    std::string last_settings_subpage() { return last_settings_subpage_; }
+
+    // TetherNotificationPresenter::SettingsUiDelegate:
+    void ShowSettingsSubPageForProfile(Profile* profile,
+                                       const std::string& sub_page) override {
+      last_profile_ = profile;
+      last_settings_subpage_ = sub_page;
+    }
+
+   private:
+    Profile* last_profile_;
+    std::string last_settings_subpage_;
+  };
+
  protected:
   TetherNotificationPresenterTest() : test_device_(CreateTestRemoteDevice()) {}
 
   void SetUp() override {
+    TestingProfile::Builder builder;
+    profile_ = builder.Build();
     test_message_center_ = base::WrapUnique(new TestMessageCenter());
     test_network_connect_ = base::WrapUnique(new TestNetworkConnect());
     notification_presenter_ = base::WrapUnique(new TetherNotificationPresenter(
-        test_message_center_.get(), test_network_connect_.get()));
+        profile_.get(), test_message_center_.get(),
+        test_network_connect_.get()));
+
+    test_settings_ui_delegate_ = new TestSettingsUiDelegate();
+    notification_presenter_->SetSettingsUiDelegateForTesting(
+        base::WrapUnique(test_settings_ui_delegate_));
   }
 
+  void TearDown() override { profile_.reset(); }
+
   std::string GetActiveHostNotificationId() {
     return std::string(TetherNotificationPresenter::kActiveHostNotificationId);
   }
 
   std::string GetPotentialHotspotNotificationId() {
     return std::string(
         TetherNotificationPresenter::kPotentialHotspotNotificationId);
   }
 
+  void VerifySettingsOpened() {
+    EXPECT_EQ(profile_.get(), test_settings_ui_delegate_->last_profile());
+    EXPECT_EQ("networks?type=Tether",
+              test_settings_ui_delegate_->last_settings_subpage());
+  }
+
+  void VerifySettingsNotOpened() {
+    EXPECT_FALSE(test_settings_ui_delegate_->last_profile());
+    EXPECT_TRUE(test_settings_ui_delegate_->last_settings_subpage().empty());
+  }
+
+  const content::TestBrowserThreadBundle thread_bundle_;
   const cryptauth::RemoteDevice test_device_;
 
+  std::unique_ptr<TestingProfile> profile_;
   std::unique_ptr<TestMessageCenter> test_message_center_;
   std::unique_ptr<TestNetworkConnect> test_network_connect_;
+  TestSettingsUiDelegate* test_settings_ui_delegate_;
 
   std::unique_ptr<TetherNotificationPresenter> notification_presenter_;
 
  private:
   DISALLOW_COPY_AND_ASSIGN(TetherNotificationPresenterTest);
 };
 
 TEST_F(TetherNotificationPresenterTest,
        TestHostConnectionFailedNotification_RemoveProgrammatically) {
   EXPECT_FALSE(test_message_center_->FindVisibleNotificationById(
       GetActiveHostNotificationId()));
   notification_presenter_->NotifyConnectionToHostFailed();
 
   message_center::Notification* notification =
       test_message_center_->FindVisibleNotificationById(
           GetActiveHostNotificationId());
   EXPECT_TRUE(notification);
   EXPECT_EQ(GetActiveHostNotificationId(), notification->id());
 
   EXPECT_EQ(1u, test_message_center_->GetNumNotifications());
   notification_presenter_->RemoveConnectionToHostFailedNotification();
   EXPECT_FALSE(test_message_center_->FindVisibleNotificationById(
       GetActiveHostNotificationId()));
   EXPECT_EQ(0u, test_message_center_->GetNumNotifications());
+
+  VerifySettingsNotOpened();
 }
 
 TEST_F(TetherNotificationPresenterTest,
        TestHostConnectionFailedNotification_TapNotification) {
   EXPECT_FALSE(test_message_center_->FindVisibleNotificationById(
       GetActiveHostNotificationId()));
   notification_presenter_->NotifyConnectionToHostFailed();
 
   message_center::Notification* notification =
       test_message_center_->FindVisibleNotificationById(
           GetActiveHostNotificationId());
   EXPECT_TRUE(notification);
   EXPECT_EQ(GetActiveHostNotificationId(), notification->id());
 
   // Tap the notification.
   test_message_center_->NotifyNotificationTapped(GetActiveHostNotificationId());
-  // TODO(khorimoto): Test that the tethering settings page is opened.
+  VerifySettingsOpened();
 }
 
 TEST_F(TetherNotificationPresenterTest,
        TestSinglePotentialHotspotNotification_RemoveProgrammatically) {
   EXPECT_FALSE(test_message_center_->FindVisibleNotificationById(
       GetPotentialHotspotNotificationId()));
   notification_presenter_->NotifyPotentialHotspotNearby(test_device_);
 
   message_center::Notification* notification =
       test_message_center_->FindVisibleNotificationById(
           GetPotentialHotspotNotificationId());
   EXPECT_TRUE(notification);
   EXPECT_EQ(GetPotentialHotspotNotificationId(), notification->id());
 
   notification_presenter_->RemovePotentialHotspotNotification();
   EXPECT_FALSE(test_message_center_->FindVisibleNotificationById(
       GetPotentialHotspotNotificationId()));
+
+  VerifySettingsNotOpened();
 }
 
 TEST_F(TetherNotificationPresenterTest,
        TestSinglePotentialHotspotNotification_TapNotification) {
   EXPECT_FALSE(test_message_center_->FindVisibleNotificationById(
       GetPotentialHotspotNotificationId()));
   notification_presenter_->NotifyPotentialHotspotNearby(test_device_);
 
   message_center::Notification* notification =
       test_message_center_->FindVisibleNotificationById(
           GetPotentialHotspotNotificationId());
   EXPECT_TRUE(notification);
   EXPECT_EQ(GetPotentialHotspotNotificationId(), notification->id());
 
   // Tap the notification.
   test_message_center_->NotifyNotificationTapped(
       GetPotentialHotspotNotificationId());
-  // TODO(khorimoto): Test that the tethering settings page is opened.
+  VerifySettingsOpened();
 }
 
 TEST_F(TetherNotificationPresenterTest,
        TestSinglePotentialHotspotNotification_TapNotificationButton) {
   EXPECT_FALSE(test_message_center_->FindVisibleNotificationById(
       GetPotentialHotspotNotificationId()));
   notification_presenter_->NotifyPotentialHotspotNearby(test_device_);
 
   message_center::Notification* notification =
       test_message_center_->FindVisibleNotificationById(
           GetPotentialHotspotNotificationId());
   EXPECT_TRUE(notification);
   EXPECT_EQ(GetPotentialHotspotNotificationId(), notification->id());
 
   // Tap the notification's button.
   test_message_center_->NotifyNotificationButtonTapped(
       GetPotentialHotspotNotificationId(), 0 /* button_index */);
 
   EXPECT_EQ(test_device_.GetDeviceId(),
             test_network_connect_->network_id_to_connect());
-
-  // TODO(hansberry): Test for the case of the user not yet going through
-  // the connection dialog.
 }
 
 TEST_F(TetherNotificationPresenterTest,
        TestMultiplePotentialHotspotNotification_RemoveProgrammatically) {
   EXPECT_FALSE(test_message_center_->FindVisibleNotificationById(
       GetPotentialHotspotNotificationId()));
   notification_presenter_->NotifyMultiplePotentialHotspotsNearby();
 
   message_center::Notification* notification =
       test_message_center_->FindVisibleNotificationById(
           GetPotentialHotspotNotificationId());
   EXPECT_TRUE(notification);
   EXPECT_EQ(GetPotentialHotspotNotificationId(), notification->id());
 
   notification_presenter_->RemovePotentialHotspotNotification();
   EXPECT_FALSE(test_message_center_->FindVisibleNotificationById(
       GetPotentialHotspotNotificationId()));
+
+  VerifySettingsNotOpened();
 }
 
 TEST_F(TetherNotificationPresenterTest,
        TestMultiplePotentialHotspotNotification_TapNotification) {
   EXPECT_FALSE(test_message_center_->FindVisibleNotificationById(
       GetPotentialHotspotNotificationId()));
   notification_presenter_->NotifyMultiplePotentialHotspotsNearby();
 
   message_center::Notification* notification =
       test_message_center_->FindVisibleNotificationById(
           GetPotentialHotspotNotificationId());
   EXPECT_TRUE(notification);
   EXPECT_EQ(GetPotentialHotspotNotificationId(), notification->id());
 
   // Tap the notification.
   test_message_center_->NotifyNotificationTapped(
       GetPotentialHotspotNotificationId());
-  // TODO(khorimoto): Test that the tethering settings page is opened.
+  VerifySettingsOpened();
 }
 
 TEST_F(TetherNotificationPresenterTest,
        TestPotentialHotspotNotifications_UpdatesOneNotification) {
   EXPECT_FALSE(test_message_center_->FindVisibleNotificationById(
       GetPotentialHotspotNotificationId()));
   notification_presenter_->NotifyPotentialHotspotNearby(test_device_);
 
   message_center::Notification* notification =
       test_message_center_->FindVisibleNotificationById(
@@ skipping 10 matching lines @@
   // new one.
   notification = test_message_center_->FindVisibleNotificationById(
       GetPotentialHotspotNotificationId());
   EXPECT_TRUE(notification);
   EXPECT_EQ(GetPotentialHotspotNotificationId(), notification->id());
   EXPECT_EQ(1u, test_message_center_->GetNumNotifications());
 
   notification_presenter_->RemovePotentialHotspotNotification();
   EXPECT_FALSE(test_message_center_->FindVisibleNotificationById(
       GetPotentialHotspotNotificationId()));
+
+  VerifySettingsNotOpened();
 }
 
 }  // namespace tether
 
 }  // namespace chromeos