Revert 154457 - Switch from SignedSettings to DeviceSettingsService.

chrome/browser/chromeos/settings/cros_settings_unittest.cc

Index: chrome/browser/chromeos/settings/cros_settings_unittest.cc
===================================================================
--- chrome/browser/chromeos/settings/cros_settings_unittest.cc	(revision 154482)
+++ chrome/browser/chromeos/settings/cros_settings_unittest.cc	(working copy)
@@ -2,19 +2,21 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
+#include "chrome/browser/chromeos/settings/signed_settings.h"
+
 #include <map>
 #include <string>
 
 #include "base/bind.h"
-#include "base/memory/scoped_ptr.h"
 #include "base/memory/weak_ptr.h"
 #include "base/message_loop.h"
 #include "base/stl_util.h"
 #include "base/values.h"
+#include "chrome/browser/chromeos/cros/cros_library.h"
+#include "chrome/browser/chromeos/login/mock_user_manager.h"
 #include "chrome/browser/chromeos/settings/cros_settings.h"
 #include "chrome/browser/chromeos/settings/cros_settings_names.h"
-#include "chrome/browser/chromeos/settings/device_settings_test_helper.h"
-#include "chrome/browser/policy/cloud_policy_constants.h"
+#include "chrome/browser/chromeos/settings/signed_settings_cache.h"
 #include "chrome/browser/policy/proto/chrome_device_policy.pb.h"
 #include "chrome/browser/policy/proto/device_management_backend.pb.h"
 #include "chrome/test/base/testing_browser_process.h"
@@ -22,8 +24,10 @@
 #include "content/public/test/test_browser_thread.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
+using ::testing::AnyNumber;
+using ::testing::Return;
+
 namespace em = enterprise_management;
-
 namespace chromeos {
 
 class CrosSettingsTest : public testing::Test {
@@ -31,15 +35,28 @@
   CrosSettingsTest()
       : message_loop_(MessageLoop::TYPE_UI),
         ui_thread_(content::BrowserThread::UI, &message_loop_),
-        local_state_(static_cast<TestingBrowserProcess*>(g_browser_process)),
-        weak_factory_(this) {}
+        file_thread_(content::BrowserThread::FILE, &message_loop_),
+        pointer_factory_(this),
+        local_state_(static_cast<TestingBrowserProcess*>(g_browser_process)) {
+  }
 
-  virtual ~CrosSettingsTest() {}
+  virtual ~CrosSettingsTest() {
+  }
 
-  virtual void TearDown() OVERRIDE {
+  virtual void SetUp() {
+    EXPECT_CALL(*mock_user_manager_.user_manager(), IsCurrentUserOwner())
+        .Times(AnyNumber())
+        .WillRepeatedly(Return(true));
+    // Reset the cache between tests.
+    ApplyEmptyPolicy();
+  }
+
+  virtual void TearDown() {
+    message_loop_.RunAllPending();
     ASSERT_TRUE(expected_props_.empty());
+    // Reset the cache between tests.
+    ApplyEmptyPolicy();
     STLDeleteValues(&expected_props_);
-    expected_props_.clear();
   }
 
   void FetchPref(const std::string& pref) {
@@ -48,12 +65,12 @@
       return;
 
     if (CrosSettingsProvider::TRUSTED ==
-            settings_.PrepareTrustedValues(
-                base::Bind(&CrosSettingsTest::FetchPref,
-                           weak_factory_.GetWeakPtr(), pref))) {
+        CrosSettings::Get()->PrepareTrustedValues(
+            base::Bind(&CrosSettingsTest::FetchPref,
+                       pointer_factory_.GetWeakPtr(), pref))) {
       scoped_ptr<base::Value> expected_value(
           expected_props_.find(pref)->second);
-      const base::Value* pref_value = settings_.GetPref(pref);
+      const base::Value* pref_value = CrosSettings::Get()->GetPref(pref);
       if (expected_value.get()) {
         ASSERT_TRUE(pref_value);
         ASSERT_TRUE(expected_value->Equals(pref_value));
@@ -66,7 +83,7 @@
 
   void SetPref(const std::string& pref_name, const base::Value* value) {
     DCHECK(content::BrowserThread::CurrentlyOn(content::BrowserThread::UI));
-    settings_.Set(pref_name, *value);
+    CrosSettings::Get()->Set(pref_name, *value);
   }
 
   void AddExpectation(const std::string& pref_name, base::Value* value) {
@@ -79,7 +96,7 @@
     // Prepare some policy blob.
     em::PolicyFetchResponse response;
     em::ChromeDeviceSettingsProto pol;
-    policy->set_policy_type(policy::dm_protocol::kChromeDevicePolicyType);
+    policy->set_policy_type(chromeos::kDevicePolicyType);
     policy->set_username("me@owner");
     policy->set_policy_value(pol.SerializeAsString());
     // Wipe the signed settings store.
@@ -87,16 +104,25 @@
     response.set_policy_data_signature("false");
   }
 
+  void ApplyEmptyPolicy() {
+    em::PolicyData fake_pol;
+    PrepareEmptyPolicy(&fake_pol);
+    signed_settings_cache::Store(fake_pol, local_state_.Get());
+    CrosSettings::Get()->ReloadProviders();
+  }
+
+  std::map<std::string, base::Value*> expected_props_;
+
   MessageLoop message_loop_;
   content::TestBrowserThread ui_thread_;
+  content::TestBrowserThread file_thread_;
 
+  base::WeakPtrFactory<CrosSettingsTest> pointer_factory_;
+
   ScopedTestingLocalState local_state_;
-  ScopedDeviceSettingsTestHelper device_settings_test_helper_;
-  CrosSettings settings_;
 
-  base::WeakPtrFactory<CrosSettingsTest> weak_factory_;
-
-  std::map<std::string, base::Value*> expected_props_;
+  ScopedMockUserManagerEnabler mock_user_manager_;
+  ScopedStubCrosEnabler stub_cros_enabler_;
 };
 
 TEST_F(CrosSettingsTest, SetPref) {
@@ -105,6 +131,7 @@
                  base::Value::CreateBooleanValue(false));
   SetPref(kAccountsPrefAllowGuest, expected_props_[kAccountsPrefAllowGuest]);
   FetchPref(kAccountsPrefAllowGuest);
+  message_loop_.RunAllPending();
   ASSERT_TRUE(expected_props_.empty());
 }
 
@@ -136,7 +163,7 @@
                  base::Value::CreateBooleanValue(false));
   AddExpectation(kAccountsPrefUsers, whitelist);
   // Add some user to the whitelist.
-  settings_.AppendToList(kAccountsPrefUsers, &hacky_user);
+  CrosSettings::Get()->AppendToList(kAccountsPrefUsers, &hacky_user);
   FetchPref(kAccountsPrefAllowNewUser);
   FetchPref(kAccountsPrefUsers);
 }
@@ -154,10 +181,11 @@
   SetPref(kAccountsPrefUsers, &whitelist);
   FetchPref(kAccountsPrefAllowNewUser);
   FetchPref(kAccountsPrefUsers);
+  message_loop_.RunAllPending();
   ASSERT_TRUE(expected_props_.empty());
   // Now try to remove one element from that list.
   AddExpectation(kAccountsPrefUsers, expected_list);
-  settings_.RemoveFromList(kAccountsPrefUsers, &lamy_user);
+  CrosSettings::Get()->RemoveFromList(kAccountsPrefUsers, &lamy_user);
   FetchPref(kAccountsPrefAllowNewUser);
   FetchPref(kAccountsPrefUsers);
 }
@@ -199,6 +227,13 @@
   FetchPref(kAccountsPrefAllowNewUser);
 }
 
+TEST_F(CrosSettingsTest, SetOwner) {
+  base::StringValue hacky_owner("h@xxor");
+  AddExpectation(kDeviceOwner, base::Value::CreateStringValue("h@xxor"));
+  SetPref(kDeviceOwner, &hacky_owner);
+  FetchPref(kDeviceOwner);
+}
+
 TEST_F(CrosSettingsTest, SetEphemeralUsersEnabled) {
   base::FundamentalValue ephemeral_users_enabled(true);
   AddExpectation(kAccountsPrefEphemeralUsersEnabled,
@@ -214,7 +249,7 @@
   list.Append(base::Value::CreateStringValue("with.dots@gmail.com"));
   list.Append(base::Value::CreateStringValue("Upper@example.com"));
 
-  CrosSettings* cs = &settings_;
+  CrosSettings* cs = CrosSettings::Get();
   cs->Set(kAccountsPrefUsers, list);
 
   EXPECT_TRUE(cs->FindEmailInList(kAccountsPrefUsers, "user@example.com"));