Consolidate data storage and notifications in the cloud policy subsystem

chrome/browser/policy/device_token_fetcher_unittest.cc

 // Copyright (c) 2011 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/policy/device_token_fetcher.h"
 
 #include "base/file_util.h"

[Joao da Silva, 2011/07/06 16:45:14]

Nit: file_util.h not used

[gfeher, 2011/07/07 13:51:00]

Done.

 #include "base/message_loop.h"
 #include "base/scoped_temp_dir.h"
 #include "chrome/browser/net/gaia/token_service.h"

[Joao da Silva, 2011/07/06 16:45:14]

Nit: token_service.h not used

[gfeher, 2011/07/07 13:51:00]

Done.

+#include "chrome/browser/policy/cloud_policy_data.h"
 #include "chrome/browser/policy/device_management_service.h"

[Joao da Silva, 2011/07/06 16:45:14]

Nit: device_management_service.h not used

[gfeher, 2011/07/07 13:51:00]

Done.

 #include "chrome/browser/policy/logging_work_scheduler.h"
 #include "chrome/browser/policy/mock_device_management_backend.h"
 #include "chrome/browser/policy/mock_device_management_service.h"
 #include "chrome/browser/policy/policy_notifier.h"
 #include "chrome/browser/policy/proto/device_management_backend.pb.h"

[Joao da Silva, 2011/07/06 16:45:14]

Nit: device_management_backend.pb.h not used

 #include "chrome/browser/policy/user_policy_cache.h"
 #include "chrome/common/net/gaia/gaia_constants.h"
 #include "chrome/test/testing_profile.h"

[Joao da Silva, 2011/07/06 16:45:14]

Nit: gaia_constants.h and testing_profile.h not used

[gfeher, 2011/07/07 13:51:00]

Done.

 #include "content/browser/browser_thread.h"
 #include "testing/gmock/include/gmock/gmock.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 namespace policy {
 
 const char kTestToken[] = "device_token_fetcher_test_auth_token";
 
 using testing::_;
 using testing::Mock;
 
-class MockTokenAvailableObserver : public DeviceTokenFetcher::Observer {
+class MockTokenAvailableObserver : public CloudPolicyData::Observer {
  public:
   MockTokenAvailableObserver() {}
   virtual ~MockTokenAvailableObserver() {}
 
-  MOCK_METHOD0(OnDeviceTokenAvailable, void());
+  MOCK_METHOD0(OnDeviceTokenChanged, void());
+  MOCK_METHOD0(OnCredentialsChanged, void());
+  MOCK_METHOD0(OnPolicyDataGoingAway, void());
 
  private:
   DISALLOW_COPY_AND_ASSIGN(MockTokenAvailableObserver);
 };
 
 class DeviceTokenFetcherTest : public testing::Test {
  protected:
   DeviceTokenFetcherTest()
       : ui_thread_(BrowserThread::UI, &loop_),
         file_thread_(BrowserThread::FILE, &loop_) {
     EXPECT_TRUE(temp_user_data_dir_.CreateUniqueTempDir());
   }
 
   virtual void SetUp() {
     cache_.reset(new UserPolicyCache(
         temp_user_data_dir_.path().AppendASCII("DeviceTokenFetcherTest")));
     service_.set_backend(&backend_);
+    cloud_policy_data_.reset(CloudPolicyData::CreateForUserPolicies());
+    cloud_policy_data_->AddObserver(&observer_);
   }
 
   virtual void TearDown() {
     loop_.RunAllPending();
+    cloud_policy_data_->RemoveObserver(&observer_);
+  }
+
+  void FetchToken(DeviceTokenFetcher* fetcher) {
+    cloud_policy_data_->SetupForTesting("", "fake_device_id", "fake_user_name",
+                                        "fake_auth_token", true);
+    fetcher->FetchToken();
   }
 
   MessageLoop loop_;
   MockDeviceManagementBackend backend_;
   MockDeviceManagementService service_;
   scoped_ptr<CloudPolicyCacheBase> cache_;
+  scoped_ptr<CloudPolicyData> cloud_policy_data_;
+  MockTokenAvailableObserver observer_;
   PolicyNotifier notifier_;
   ScopedTempDir temp_user_data_dir_;
 
  private:
   BrowserThread ui_thread_;
   BrowserThread file_thread_;
 };
 
 TEST_F(DeviceTokenFetcherTest, FetchToken) {
   testing::InSequence s;
   EXPECT_CALL(backend_, ProcessRegisterRequest(_, _, _, _)).WillOnce(
       MockDeviceManagementBackendSucceedRegister());
-  DeviceTokenFetcher fetcher(&service_, cache_.get(), &notifier_);
-  MockTokenAvailableObserver observer;
-  EXPECT_CALL(observer, OnDeviceTokenAvailable());
-  fetcher.AddObserver(&observer);
-  EXPECT_EQ("", fetcher.GetDeviceToken());
-  fetcher.FetchToken("fake_auth_token", "fake_device_id",
-                     em::DeviceRegisterRequest::USER,
-                     "fake_machine_id", "fake_machine_model");
+  DeviceTokenFetcher fetcher(&service_, cache_.get(), cloud_policy_data_.get(),
+                             &notifier_);
+  EXPECT_CALL(observer_, OnDeviceTokenChanged());
+  EXPECT_EQ("", cloud_policy_data_->device_token());
+  FetchToken(&fetcher);
   loop_.RunAllPending();
-  Mock::VerifyAndClearExpectations(&observer);
-  std::string token = fetcher.GetDeviceToken();
+  Mock::VerifyAndClearExpectations(&observer_);
+  std::string token = cloud_policy_data_->device_token();
   EXPECT_NE("", token);
 
   // Calling FetchToken() again should result in a new token being fetched.
   EXPECT_CALL(backend_, ProcessRegisterRequest(_, _, _, _)).WillOnce(
       MockDeviceManagementBackendSucceedRegister());
-  EXPECT_CALL(observer, OnDeviceTokenAvailable());
-  fetcher.FetchToken("fake_auth_token", "fake_device_id",
-                     em::DeviceRegisterRequest::USER,
-                     "fake_machine_id", "fake_machine_model");
+  EXPECT_CALL(observer_, OnDeviceTokenChanged());
+  FetchToken(&fetcher);
   loop_.RunAllPending();
-  Mock::VerifyAndClearExpectations(&observer);
-  std::string token2 = fetcher.GetDeviceToken();
+  Mock::VerifyAndClearExpectations(&observer_);
+  std::string token2 = cloud_policy_data_->device_token();
   EXPECT_NE("", token2);
   EXPECT_NE(token, token2);
-  fetcher.RemoveObserver(&observer);
 }
 
 TEST_F(DeviceTokenFetcherTest, RetryOnError) {
   testing::InSequence s;
   EXPECT_CALL(backend_, ProcessRegisterRequest(_, _, _, _)).WillOnce(
       MockDeviceManagementBackendFailRegister(
           DeviceManagementBackend::kErrorRequestFailed)).WillOnce(
       MockDeviceManagementBackendSucceedRegister());
-  DeviceTokenFetcher fetcher(&service_, cache_.get(), &notifier_,
-                             new DummyWorkScheduler);
-  MockTokenAvailableObserver observer;
-  EXPECT_CALL(observer, OnDeviceTokenAvailable());
-  fetcher.AddObserver(&observer);
-  fetcher.FetchToken("fake_auth_token", "fake_device_id",
-                     em::DeviceRegisterRequest::USER,
-                     "fake_machine_id", "fake_machine_model");
+  DeviceTokenFetcher fetcher(&service_, cache_.get(), cloud_policy_data_.get(),
+                             &notifier_, new DummyWorkScheduler);
+  EXPECT_CALL(observer_, OnDeviceTokenChanged());
+  FetchToken(&fetcher);
   loop_.RunAllPending();
-  Mock::VerifyAndClearExpectations(&observer);
-  EXPECT_NE("", fetcher.GetDeviceToken());
-  fetcher.RemoveObserver(&observer);
+  Mock::VerifyAndClearExpectations(&observer_);
+  EXPECT_NE("", cloud_policy_data_->device_token());
 }
 
 TEST_F(DeviceTokenFetcherTest, UnmanagedDevice) {
   EXPECT_CALL(backend_, ProcessRegisterRequest(_, _, _, _)).WillOnce(
       MockDeviceManagementBackendFailRegister(
           DeviceManagementBackend::kErrorServiceManagementNotSupported));
   EXPECT_FALSE(cache_->is_unmanaged());
-  DeviceTokenFetcher fetcher(&service_, cache_.get(), &notifier_);
-  MockTokenAvailableObserver observer;
-  EXPECT_CALL(observer, OnDeviceTokenAvailable()).Times(0);
-  fetcher.AddObserver(&observer);
-  fetcher.FetchToken("fake_auth_token", "fake_device_id",
-                     em::DeviceRegisterRequest::USER,
-                     "fake_machine_id", "fake_machine_model");
+  DeviceTokenFetcher fetcher(&service_, cache_.get(), cloud_policy_data_.get(),
+                             &notifier_);
+  EXPECT_CALL(observer_, OnDeviceTokenChanged()).Times(0);
+  FetchToken(&fetcher);
   loop_.RunAllPending();
-  Mock::VerifyAndClearExpectations(&observer);
-  EXPECT_EQ("", fetcher.GetDeviceToken());
+  Mock::VerifyAndClearExpectations(&observer_);
+  EXPECT_EQ("", cloud_policy_data_->device_token());
   EXPECT_TRUE(cache_->is_unmanaged());
-  fetcher.RemoveObserver(&observer);
 }
 
 }  // namespace policy