No Easy unlock if bluetooth is not available.

chrome/browser/signin/easy_unlock_service_browsertest_chromeos.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 "base/command_line.h"
 #include "base/macros.h"
 #include "base/run_loop.h"
 #include "base/values.h"
 #include "chrome/browser/chromeos/login/login_manager_test.h"
 #include "chrome/browser/chromeos/login/startup_utils.h"
 #include "chrome/browser/chromeos/login/ui/user_adding_screen.h"
 #include "chrome/browser/chromeos/login/users/user_manager.h"
 #include "chrome/browser/chromeos/profiles/profile_helper.h"
 #include "chrome/browser/extensions/extension_service.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/signin/easy_unlock_service.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/common/extensions/extension_constants.h"
 #include "chrome/test/base/in_process_browser_test.h"
 #include "components/policy/core/browser/browser_policy_connector.h"
 #include "components/policy/core/common/mock_configuration_policy_provider.h"
 #include "components/policy/core/common/policy_map.h"
 #include "components/policy/core/common/policy_types.h"
 #include "content/public/common/content_switches.h"
+#include "device/bluetooth/bluetooth_adapter_factory.h"
+#include "device/bluetooth/test/mock_bluetooth_adapter.h"
 #include "extensions/browser/extension_system.h"
 #include "policy/policy_constants.h"
 #include "testing/gmock/include/gmock/gmock.h"
 
 using chromeos::ProfileHelper;
 using chromeos::LoginManagerTest;
 using chromeos::StartupUtils;
 using chromeos::UserAddingScreen;
 using chromeos::UserManager;
+using device::MockBluetoothAdapter;
 using testing::_;
 using testing::Return;
 
 namespace {
 
 const char kTestUser1[] = "primary.user@example.com";
 const char kTestUser2[] = "secondary.user@example.com";
 
 #if defined(GOOGLE_CHROME_BUILD)
 bool HasEasyUnlockAppForProfile(Profile* profile) {
   extensions::ExtensionSystem* extension_system =
       extensions::ExtensionSystem::Get(profile);
   ExtensionService* extension_service = extension_system->extension_service();
   return !!extension_service->GetExtensionById(
       extension_misc::kEasyUnlockAppId, false);
 }
 #endif
 
-} //namespace
+}  // namespace
 
 class EasyUnlockServiceTest : public InProcessBrowserTest {
  public:
-  EasyUnlockServiceTest() {}
+  EasyUnlockServiceTest() : is_bluetooth_adapter_present_(true) {}
   virtual ~EasyUnlockServiceTest() {}
 
   void SetEasyUnlockAllowedPolicy(bool allowed) {
     policy::PolicyMap policy;
     policy.Set(policy::key::kEasyUnlockAllowed,
                policy::POLICY_LEVEL_MANDATORY,
                policy::POLICY_SCOPE_USER,
                new base::FundamentalValue(allowed),
                NULL);
     provider_.UpdateChromePolicy(policy);
     base::RunLoop().RunUntilIdle();
   }
 
+  void SetupBluetoothMock(bool is_present) {

[Ilya Sherman, 2014/08/14 21:38:28]

nit: "Setup" -> "SetUp"

[xiyuan, 2014/08/14 22:56:46]

Done.

+    mock_adapter_ = new testing::NiceMock<MockBluetoothAdapter>();
+    device::BluetoothAdapterFactory::SetAdapterForTesting(mock_adapter_);
+
+    EXPECT_CALL(*mock_adapter_, AddObserver(_))
+        .WillRepeatedly(testing::Return());
+    EXPECT_CALL(*mock_adapter_, RemoveObserver(_))
+        .WillRepeatedly(testing::Return());

[Ilya Sherman, 2014/08/14 21:38:28]

Hmm, why are these four lines needed?

[xiyuan, 2014/08/14 22:56:46]

It was because I was desperately trying to figure out why gmock fails on bots.
Thank you for helping me investigate. IsInitialized is not mocked though. I now
suspect IsPowered and GetDevices, which are called from system tray. Maybe the
bots is too slow and somehow they are called.

+    EXPECT_CALL(*mock_adapter_, IsPresent())
+        .WillRepeatedly(testing::Return(is_present));
+  }
+
 #if defined(GOOGLE_CHROME_BUILD)
   bool HasEasyUnlockApp() const {
     return HasEasyUnlockAppForProfile(profile());
   }
 #endif
 
   // InProcessBrowserTest:
   virtual void SetUpInProcessBrowserTestFixture() OVERRIDE {
     EXPECT_CALL(provider_, IsInitializationComplete(_))
         .WillRepeatedly(Return(true));
     policy::BrowserPolicyConnector::SetPolicyProviderForTesting(&provider_);
+    SetupBluetoothMock(is_bluetooth_adapter_present_);
   }
 
   Profile* profile() const { return browser()->profile(); }
 
   EasyUnlockService* service() const {
     return EasyUnlockService::Get(profile());
   }
 
+  void set_is_bluetooth_adapter_present(bool is_present) {
+    is_bluetooth_adapter_present_ = is_present;
+  }
+
  private:
   policy::MockConfigurationPolicyProvider provider_;
+  scoped_refptr<testing::NiceMock<MockBluetoothAdapter> > mock_adapter_;

[Ilya Sherman, 2014/08/14 21:38:28]

nit: Does this really need to be a class member, or can it just be a local
variable?

[xiyuan, 2014/08/14 22:56:46]

It turns out that we have to do this. Otherwise, it would be released when
SetupBluetoothMock returns and default adapter gets used again.

+  bool is_bluetooth_adapter_present_;
 
   DISALLOW_COPY_AND_ASSIGN(EasyUnlockServiceTest);
 };
 
 // Tests that EasyUnlock is on by default.
 IN_PROC_BROWSER_TEST_F(EasyUnlockServiceTest, DefaultOn) {
   EXPECT_TRUE(service()->IsAllowed());
 #if defined(GOOGLE_CHROME_BUILD)
   EXPECT_TRUE(HasEasyUnlockApp());
 #endif
 }
 
+class EasyUnlockServiceNoBluetoothTest : public EasyUnlockServiceTest {
+ public:
+  EasyUnlockServiceNoBluetoothTest() {}
+  virtual ~EasyUnlockServiceNoBluetoothTest() {}
+
+  // InProcessBrowserTest:
+  virtual void SetUpInProcessBrowserTestFixture() OVERRIDE {
+    set_is_bluetooth_adapter_present(false);
+    EasyUnlockServiceTest::SetUpInProcessBrowserTestFixture();
+  }
+
+ private:
+  DISALLOW_COPY_AND_ASSIGN(EasyUnlockServiceNoBluetoothTest);
+};
+
+IN_PROC_BROWSER_TEST_F(EasyUnlockServiceNoBluetoothTest, NoService) {
+  EXPECT_FALSE(service()->IsAllowed());
+#if defined(GOOGLE_CHROME_BUILD)
+  EXPECT_FALSE(HasEasyUnlockApp());
+#endif
+}
+
 class EasyUnlockServiceFinchEnabledTest : public EasyUnlockServiceTest {
  public:
   EasyUnlockServiceFinchEnabledTest() {}
   virtual ~EasyUnlockServiceFinchEnabledTest() {}
 
   // InProcessBrowserTest:
   virtual void SetUpCommandLine(CommandLine* command_line) OVERRIDE {
     command_line->AppendSwitchASCII(switches::kForceFieldTrials,
                                     "EasyUnlock/Enable/");
   }
@@ skipping 52 matching lines @@
 #if defined(GOOGLE_CHROME_BUILD)
   EXPECT_FALSE(HasEasyUnlockApp());
 #endif
 }
 
 class EasyUnlockServiceMultiProfileTest : public LoginManagerTest {
  public:
   EasyUnlockServiceMultiProfileTest() : LoginManagerTest(false) {}
   virtual ~EasyUnlockServiceMultiProfileTest() {}
 
+  // InProcessBrowserTest:
+  virtual void SetUpInProcessBrowserTestFixture() OVERRIDE {
+    LoginManagerTest::SetUpInProcessBrowserTestFixture();
+
+    mock_adapter_ = new testing::NiceMock<MockBluetoothAdapter>();
+    device::BluetoothAdapterFactory::SetAdapterForTesting(mock_adapter_);
+    EXPECT_CALL(*mock_adapter_, IsPresent())
+        .WillRepeatedly(testing::Return(true));
+  }
+
  private:
+  scoped_refptr<testing::NiceMock<MockBluetoothAdapter> > mock_adapter_;

[Ilya Sherman, 2014/08/14 21:38:28]

nit: Does this really need to be a class member, or can it just be a local
variable?

[xiyuan, 2014/08/14 22:56:46]

ditto.

   DISALLOW_COPY_AND_ASSIGN(EasyUnlockServiceMultiProfileTest);
 };
 
 IN_PROC_BROWSER_TEST_F(EasyUnlockServiceMultiProfileTest,
                        PRE_DisallowedOnSecondaryProfile) {
   RegisterUser(kTestUser1);
   RegisterUser(kTestUser2);
   StartupUtils::MarkOobeCompleted();
 }
 
@@ skipping 13 matching lines @@
   Profile* secondary_profile = ProfileHelper::Get()->GetProfileByUserIdHash(
       secondary_user->username_hash());
 
   EXPECT_TRUE(EasyUnlockService::Get(primary_profile)->IsAllowed());
   EXPECT_FALSE(EasyUnlockService::Get(secondary_profile)->IsAllowed());
 #if defined(GOOGLE_CHROME_BUILD)
   EXPECT_TRUE(HasEasyUnlockAppForProfile(primary_profile));
   EXPECT_FALSE(HasEasyUnlockAppForProfile(secondary_profile));
 #endif
 }