Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(3767)

Unified Diff: chrome/browser/policy/configuration_policy_pref_store_unittest.cc

Issue 5915004: Introduce incognito preference settings. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Addressed Mattias' feedback Created 9 years, 12 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/policy/configuration_policy_pref_store_unittest.cc
diff --git a/chrome/browser/policy/configuration_policy_pref_store_unittest.cc b/chrome/browser/policy/configuration_policy_pref_store_unittest.cc
index 100c77487bd3aecdc6813efde4e329da08a53318..9161e6414e60ba4ddade631033b53354dc5c20bb 100644
--- a/chrome/browser/policy/configuration_policy_pref_store_unittest.cc
+++ b/chrome/browser/policy/configuration_policy_pref_store_unittest.cc
@@ -1,8 +1,9 @@
-// Copyright (c) 2010 The Chromium Authors. All rights reserved.
+// 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 "base/file_path.h"
+#include "base/ref_counted.h"
#include "chrome/browser/policy/configuration_policy_pref_store.h"
#include "chrome/browser/policy/mock_configuration_policy_provider.h"
#include "chrome/browser/prefs/proxy_prefs.h"
@@ -38,10 +39,10 @@ class ConfigurationPolicyPrefStoreTestBase : public TESTBASE {
protected:
ConfigurationPolicyPrefStoreTestBase()
: provider_(),
- store_(&provider_) {}
+ store_(new ConfigurationPolicyPrefStore(&provider_)) {}
MockConfigurationPolicyProvider provider_;
- ConfigurationPolicyPrefStore store_;
+ scoped_refptr<ConfigurationPolicyPrefStore> store_;
};
// Test cases for list-valued policy settings.
@@ -52,7 +53,7 @@ class ConfigurationPolicyPrefStoreListTest
TEST_P(ConfigurationPolicyPrefStoreListTest, GetDefault) {
EXPECT_EQ(PrefStore::READ_NO_VALUE,
- store_.GetValue(GetParam().pref_name(), NULL));
+ store_->GetValue(GetParam().pref_name(), NULL));
}
TEST_P(ConfigurationPolicyPrefStoreListTest, SetValue) {
@@ -60,10 +61,10 @@ TEST_P(ConfigurationPolicyPrefStoreListTest, SetValue) {
in_value->Append(Value::CreateStringValue("test1"));
in_value->Append(Value::CreateStringValue("test2,"));
provider_.AddPolicy(GetParam().type(), in_value);
- store_.OnUpdatePolicy();
+ store_->OnUpdatePolicy();
Value* value;
EXPECT_EQ(PrefStore::READ_OK,
- store_.GetValue(GetParam().pref_name(), &value));
+ store_->GetValue(GetParam().pref_name(), &value));
EXPECT_TRUE(in_value->Equals(value));
}
@@ -88,16 +89,16 @@ class ConfigurationPolicyPrefStoreStringTest
TEST_P(ConfigurationPolicyPrefStoreStringTest, GetDefault) {
EXPECT_EQ(PrefStore::READ_NO_VALUE,
- store_.GetValue(GetParam().pref_name(), NULL));
+ store_->GetValue(GetParam().pref_name(), NULL));
}
TEST_P(ConfigurationPolicyPrefStoreStringTest, SetValue) {
provider_.AddPolicy(GetParam().type(),
Value::CreateStringValue("http://chromium.org"));
- store_.OnUpdatePolicy();
+ store_->OnUpdatePolicy();
Value* value;
EXPECT_EQ(PrefStore::READ_OK,
- store_.GetValue(GetParam().pref_name(), &value));
+ store_->GetValue(GetParam().pref_name(), &value));
EXPECT_TRUE(StringValue("http://chromium.org").Equals(value));
}
@@ -128,23 +129,23 @@ class ConfigurationPolicyPrefStoreBooleanTest
TEST_P(ConfigurationPolicyPrefStoreBooleanTest, GetDefault) {
EXPECT_EQ(PrefStore::READ_NO_VALUE,
- store_.GetValue(GetParam().pref_name(), NULL));
+ store_->GetValue(GetParam().pref_name(), NULL));
}
TEST_P(ConfigurationPolicyPrefStoreBooleanTest, SetValue) {
provider_.AddPolicy(GetParam().type(), Value::CreateBooleanValue(false));
- store_.OnUpdatePolicy();
+ store_->OnUpdatePolicy();
Value* value;
bool result = true;
EXPECT_EQ(PrefStore::READ_OK,
- store_.GetValue(GetParam().pref_name(), &value));
+ store_->GetValue(GetParam().pref_name(), &value));
EXPECT_TRUE(FundamentalValue(false).Equals(value));
provider_.AddPolicy(GetParam().type(), Value::CreateBooleanValue(true));
- store_.OnUpdatePolicy();
+ store_->OnUpdatePolicy();
result = false;
EXPECT_EQ(PrefStore::READ_OK,
- store_.GetValue(GetParam().pref_name(), &value));
+ store_->GetValue(GetParam().pref_name(), &value));
EXPECT_TRUE(FundamentalValue(true).Equals(value));
}
@@ -204,15 +205,15 @@ class ConfigurationPolicyPrefStoreIntegerTest
TEST_P(ConfigurationPolicyPrefStoreIntegerTest, GetDefault) {
EXPECT_EQ(PrefStore::READ_NO_VALUE,
- store_.GetValue(GetParam().pref_name(), NULL));
+ store_->GetValue(GetParam().pref_name(), NULL));
}
TEST_P(ConfigurationPolicyPrefStoreIntegerTest, SetValue) {
provider_.AddPolicy(GetParam().type(), Value::CreateIntegerValue(2));
- store_.OnUpdatePolicy();
+ store_->OnUpdatePolicy();
Value* value = NULL;
EXPECT_EQ(PrefStore::READ_OK,
- store_.GetValue(GetParam().pref_name(), &value));
+ store_->GetValue(GetParam().pref_name(), &value));
EXPECT_TRUE(FundamentalValue(2).Equals(value));
}
@@ -274,9 +275,10 @@ TEST_F(ConfigurationPolicyPrefStoreProxyTest, ManualOptions) {
Value::CreateIntegerValue(
kPolicyManuallyConfiguredProxyMode));
- ConfigurationPolicyPrefStore store(&provider);
+ scoped_refptr<ConfigurationPolicyPrefStore> store(
+ new ConfigurationPolicyPrefStore(&provider));
VerifyProxyPrefs(
- store, "chromium.org", "", "http://chromium.org/override",
+ *store, "chromium.org", "", "http://chromium.org/override",
ProxyPrefs::MODE_FIXED_SERVERS);
}
@@ -289,10 +291,10 @@ TEST_F(ConfigurationPolicyPrefStoreProxyTest, ManualOptionsReversedApplyOrder) {
Value::CreateStringValue("http://chromium.org/override"));
provider.AddPolicy(kPolicyProxyServer,
Value::CreateStringValue("chromium.org"));
-
- ConfigurationPolicyPrefStore store(&provider);
+ scoped_refptr<ConfigurationPolicyPrefStore> store(
+ new ConfigurationPolicyPrefStore(&provider));
VerifyProxyPrefs(
- store, "chromium.org", "", "http://chromium.org/override",
+ *store, "chromium.org", "", "http://chromium.org/override",
ProxyPrefs::MODE_FIXED_SERVERS);
}
@@ -301,8 +303,9 @@ TEST_F(ConfigurationPolicyPrefStoreProxyTest, NoProxy) {
provider.AddPolicy(kPolicyProxyMode,
Value::CreateIntegerValue(kPolicyNoProxyServerMode));
- ConfigurationPolicyPrefStore store(&provider);
- VerifyProxyPrefs(store, "", "", "", ProxyPrefs::MODE_DIRECT);
+ scoped_refptr<ConfigurationPolicyPrefStore> store(
+ new ConfigurationPolicyPrefStore(&provider));
+ VerifyProxyPrefs(*store, "", "", "", ProxyPrefs::MODE_DIRECT);
}
TEST_F(ConfigurationPolicyPrefStoreProxyTest, AutoDetect) {
@@ -310,8 +313,9 @@ TEST_F(ConfigurationPolicyPrefStoreProxyTest, AutoDetect) {
provider.AddPolicy(kPolicyProxyMode,
Value::CreateIntegerValue(kPolicyAutoDetectProxyMode));
- ConfigurationPolicyPrefStore store(&provider);
- VerifyProxyPrefs(store, "", "", "", ProxyPrefs::MODE_AUTO_DETECT);
+ scoped_refptr<ConfigurationPolicyPrefStore> store(
+ new ConfigurationPolicyPrefStore(&provider));
+ VerifyProxyPrefs(*store, "", "", "", ProxyPrefs::MODE_AUTO_DETECT);
}
TEST_F(ConfigurationPolicyPrefStoreProxyTest, AutoDetectPac) {
@@ -322,9 +326,10 @@ TEST_F(ConfigurationPolicyPrefStoreProxyTest, AutoDetectPac) {
kPolicyProxyMode,
Value::CreateIntegerValue(kPolicyManuallyConfiguredProxyMode));
- ConfigurationPolicyPrefStore store(&provider);
- VerifyProxyPrefs(
- store, "", "http://short.org/proxy.pac", "", ProxyPrefs::MODE_PAC_SCRIPT);
+ scoped_refptr<ConfigurationPolicyPrefStore> store(
+ new ConfigurationPolicyPrefStore(&provider));
+ VerifyProxyPrefs(*store, "", "http://short.org/proxy.pac", "",
+ ProxyPrefs::MODE_PAC_SCRIPT);
}
TEST_F(ConfigurationPolicyPrefStoreProxyTest, UseSystem) {
@@ -332,8 +337,9 @@ TEST_F(ConfigurationPolicyPrefStoreProxyTest, UseSystem) {
provider.AddPolicy(kPolicyProxyMode,
Value::CreateIntegerValue(kPolicyUseSystemProxyMode));
- ConfigurationPolicyPrefStore store(&provider);
- VerifyProxyPrefs(store, "", "", "", ProxyPrefs::MODE_SYSTEM);
+ scoped_refptr<ConfigurationPolicyPrefStore> store(
+ new ConfigurationPolicyPrefStore(&provider));
+ VerifyProxyPrefs(*store, "", "", "", ProxyPrefs::MODE_SYSTEM);
}
TEST_F(ConfigurationPolicyPrefStoreProxyTest, ProxyInvalid) {
@@ -349,9 +355,10 @@ TEST_F(ConfigurationPolicyPrefStoreProxyTest, ProxyInvalid) {
provider.AddPolicy(kPolicyProxyServer,
Value::CreateStringValue("chromium.org"));
- ConfigurationPolicyPrefStore store(&provider);
+ scoped_refptr<ConfigurationPolicyPrefStore> store(
+ new ConfigurationPolicyPrefStore(&provider));
EXPECT_EQ(PrefStore::READ_NO_VALUE,
- store.GetValue(prefs::kProxyMode, NULL));
+ store->GetValue(prefs::kProxyMode, NULL));
}
}
@@ -368,31 +375,32 @@ TEST_F(ConfigurationPolicyPrefStoreDefaultSearchTest, MinimallyDefined) {
provider.AddPolicy(kPolicyDefaultSearchProviderSearchURL,
Value::CreateStringValue(search_url));
- ConfigurationPolicyPrefStore store(&provider);
+ scoped_refptr<ConfigurationPolicyPrefStore> store(
+ new ConfigurationPolicyPrefStore(&provider));
Value* value = NULL;
EXPECT_EQ(PrefStore::READ_OK,
- store.GetValue(prefs::kDefaultSearchProviderSearchURL, &value));
+ store->GetValue(prefs::kDefaultSearchProviderSearchURL, &value));
EXPECT_TRUE(StringValue(search_url).Equals(value));
EXPECT_EQ(PrefStore::READ_OK,
- store.GetValue(prefs::kDefaultSearchProviderName, &value));
+ store->GetValue(prefs::kDefaultSearchProviderName, &value));
EXPECT_TRUE(StringValue("test.com").Equals(value));
EXPECT_EQ(PrefStore::READ_OK,
- store.GetValue(prefs::kDefaultSearchProviderKeyword, &value));
+ store->GetValue(prefs::kDefaultSearchProviderKeyword, &value));
EXPECT_TRUE(StringValue(std::string()).Equals(value));
EXPECT_EQ(PrefStore::READ_OK,
- store.GetValue(prefs::kDefaultSearchProviderSuggestURL, &value));
+ store->GetValue(prefs::kDefaultSearchProviderSuggestURL, &value));
EXPECT_TRUE(StringValue(std::string()).Equals(value));
EXPECT_EQ(PrefStore::READ_OK,
- store.GetValue(prefs::kDefaultSearchProviderIconURL, &value));
+ store->GetValue(prefs::kDefaultSearchProviderIconURL, &value));
EXPECT_TRUE(StringValue(std::string()).Equals(value));
EXPECT_EQ(PrefStore::READ_OK,
- store.GetValue(prefs::kDefaultSearchProviderEncodings, &value));
+ store->GetValue(prefs::kDefaultSearchProviderEncodings, &value));
EXPECT_TRUE(StringValue(std::string()).Equals(value));
}
@@ -421,31 +429,32 @@ TEST_F(ConfigurationPolicyPrefStoreDefaultSearchTest, FullyDefined) {
provider.AddPolicy(kPolicyDefaultSearchProviderEncodings,
Value::CreateStringValue(encodings));
- ConfigurationPolicyPrefStore store(&provider);
+ scoped_refptr<ConfigurationPolicyPrefStore> store(
+ new ConfigurationPolicyPrefStore(&provider));
Value* value = NULL;
EXPECT_EQ(PrefStore::READ_OK,
- store.GetValue(prefs::kDefaultSearchProviderSearchURL, &value));
+ store->GetValue(prefs::kDefaultSearchProviderSearchURL, &value));
EXPECT_TRUE(StringValue(search_url).Equals(value));
EXPECT_EQ(PrefStore::READ_OK,
- store.GetValue(prefs::kDefaultSearchProviderName, &value));
+ store->GetValue(prefs::kDefaultSearchProviderName, &value));
EXPECT_TRUE(StringValue(name).Equals(value));
EXPECT_EQ(PrefStore::READ_OK,
- store.GetValue(prefs::kDefaultSearchProviderKeyword, &value));
+ store->GetValue(prefs::kDefaultSearchProviderKeyword, &value));
EXPECT_TRUE(StringValue(keyword).Equals(value));
EXPECT_EQ(PrefStore::READ_OK,
- store.GetValue(prefs::kDefaultSearchProviderSuggestURL, &value));
+ store->GetValue(prefs::kDefaultSearchProviderSuggestURL, &value));
EXPECT_TRUE(StringValue(suggest_url).Equals(value));
EXPECT_EQ(PrefStore::READ_OK,
- store.GetValue(prefs::kDefaultSearchProviderIconURL, &value));
+ store->GetValue(prefs::kDefaultSearchProviderIconURL, &value));
EXPECT_TRUE(StringValue(icon_url).Equals(value));
EXPECT_EQ(PrefStore::READ_OK,
- store.GetValue(prefs::kDefaultSearchProviderEncodings, &value));
+ store->GetValue(prefs::kDefaultSearchProviderEncodings, &value));
EXPECT_TRUE(StringValue(encodings).Equals(value));
}
@@ -471,20 +480,21 @@ TEST_F(ConfigurationPolicyPrefStoreDefaultSearchTest, MissingUrl) {
provider.AddPolicy(kPolicyDefaultSearchProviderEncodings,
Value::CreateStringValue(encodings));
- ConfigurationPolicyPrefStore store(&provider);
+ scoped_refptr<ConfigurationPolicyPrefStore> store(
+ new ConfigurationPolicyPrefStore(&provider));
EXPECT_EQ(PrefStore::READ_NO_VALUE,
- store.GetValue(prefs::kDefaultSearchProviderSearchURL, NULL));
+ store->GetValue(prefs::kDefaultSearchProviderSearchURL, NULL));
EXPECT_EQ(PrefStore::READ_NO_VALUE,
- store.GetValue(prefs::kDefaultSearchProviderName, NULL));
+ store->GetValue(prefs::kDefaultSearchProviderName, NULL));
EXPECT_EQ(PrefStore::READ_NO_VALUE,
- store.GetValue(prefs::kDefaultSearchProviderKeyword, NULL));
+ store->GetValue(prefs::kDefaultSearchProviderKeyword, NULL));
EXPECT_EQ(PrefStore::READ_NO_VALUE,
- store.GetValue(prefs::kDefaultSearchProviderSuggestURL, NULL));
+ store->GetValue(prefs::kDefaultSearchProviderSuggestURL, NULL));
EXPECT_EQ(PrefStore::READ_NO_VALUE,
- store.GetValue(prefs::kDefaultSearchProviderIconURL, NULL));
+ store->GetValue(prefs::kDefaultSearchProviderIconURL, NULL));
EXPECT_EQ(PrefStore::READ_NO_VALUE,
- store.GetValue(prefs::kDefaultSearchProviderEncodings, NULL));
+ store->GetValue(prefs::kDefaultSearchProviderEncodings, NULL));
}
// Checks that if the default search policy is invalid, that no elements of the
@@ -512,20 +522,21 @@ TEST_F(ConfigurationPolicyPrefStoreDefaultSearchTest, Invalid) {
provider.AddPolicy(kPolicyDefaultSearchProviderEncodings,
Value::CreateStringValue(encodings));
- ConfigurationPolicyPrefStore store(&provider);
+ scoped_refptr<ConfigurationPolicyPrefStore> store(
+ new ConfigurationPolicyPrefStore(&provider));
EXPECT_EQ(PrefStore::READ_NO_VALUE,
- store.GetValue(prefs::kDefaultSearchProviderSearchURL, NULL));
+ store->GetValue(prefs::kDefaultSearchProviderSearchURL, NULL));
EXPECT_EQ(PrefStore::READ_NO_VALUE,
- store.GetValue(prefs::kDefaultSearchProviderName, NULL));
+ store->GetValue(prefs::kDefaultSearchProviderName, NULL));
EXPECT_EQ(PrefStore::READ_NO_VALUE,
- store.GetValue(prefs::kDefaultSearchProviderKeyword, NULL));
+ store->GetValue(prefs::kDefaultSearchProviderKeyword, NULL));
EXPECT_EQ(PrefStore::READ_NO_VALUE,
- store.GetValue(prefs::kDefaultSearchProviderSuggestURL, NULL));
+ store->GetValue(prefs::kDefaultSearchProviderSuggestURL, NULL));
EXPECT_EQ(PrefStore::READ_NO_VALUE,
- store.GetValue(prefs::kDefaultSearchProviderIconURL, NULL));
+ store->GetValue(prefs::kDefaultSearchProviderIconURL, NULL));
EXPECT_EQ(PrefStore::READ_NO_VALUE,
- store.GetValue(prefs::kDefaultSearchProviderEncodings, NULL));
+ store->GetValue(prefs::kDefaultSearchProviderEncodings, NULL));
}
// Test cases for the Sync policy setting.
@@ -535,23 +546,23 @@ class ConfigurationPolicyPrefStoreSyncTest
TEST_F(ConfigurationPolicyPrefStoreSyncTest, Default) {
EXPECT_EQ(PrefStore::READ_NO_VALUE,
- store_.GetValue(prefs::kSyncManaged, NULL));
+ store_->GetValue(prefs::kSyncManaged, NULL));
}
TEST_F(ConfigurationPolicyPrefStoreSyncTest, Enabled) {
provider_.AddPolicy(kPolicySyncDisabled, Value::CreateBooleanValue(false));
- store_.OnUpdatePolicy();
+ store_->OnUpdatePolicy();
// Enabling Sync should not set the pref.
EXPECT_EQ(PrefStore::READ_NO_VALUE,
- store_.GetValue(prefs::kSyncManaged, NULL));
+ store_->GetValue(prefs::kSyncManaged, NULL));
}
TEST_F(ConfigurationPolicyPrefStoreSyncTest, Disabled) {
provider_.AddPolicy(kPolicySyncDisabled, Value::CreateBooleanValue(true));
- store_.OnUpdatePolicy();
+ store_->OnUpdatePolicy();
// Sync should be flagged as managed.
Value* value = NULL;
- EXPECT_EQ(PrefStore::READ_OK, store_.GetValue(prefs::kSyncManaged, &value));
+ EXPECT_EQ(PrefStore::READ_OK, store_->GetValue(prefs::kSyncManaged, &value));
ASSERT_TRUE(value != NULL);
EXPECT_TRUE(FundamentalValue(true).Equals(value));
}
@@ -563,24 +574,24 @@ class ConfigurationPolicyPrefStoreAutoFillTest
TEST_F(ConfigurationPolicyPrefStoreAutoFillTest, Default) {
EXPECT_EQ(PrefStore::READ_NO_VALUE,
- store_.GetValue(prefs::kSyncManaged, NULL));
+ store_->GetValue(prefs::kSyncManaged, NULL));
}
TEST_F(ConfigurationPolicyPrefStoreAutoFillTest, Enabled) {
provider_.AddPolicy(kPolicyAutoFillEnabled, Value::CreateBooleanValue(true));
- store_.OnUpdatePolicy();
+ store_->OnUpdatePolicy();
// Enabling AutoFill should not set the pref.
EXPECT_EQ(PrefStore::READ_NO_VALUE,
- store_.GetValue(prefs::kSyncManaged, NULL));
+ store_->GetValue(prefs::kSyncManaged, NULL));
}
TEST_F(ConfigurationPolicyPrefStoreAutoFillTest, Disabled) {
provider_.AddPolicy(kPolicyAutoFillEnabled, Value::CreateBooleanValue(false));
- store_.OnUpdatePolicy();
+ store_->OnUpdatePolicy();
// Disabling AutoFill should switch the pref to managed.
Value* value = NULL;
EXPECT_EQ(PrefStore::READ_OK,
- store_.GetValue(prefs::kAutoFillEnabled, &value));
+ store_->GetValue(prefs::kAutoFillEnabled, &value));
EXPECT_TRUE(FundamentalValue(false).Equals(value));
}
@@ -589,11 +600,11 @@ class ConfigurationPolicyPrefStoreRefreshTest
: public ConfigurationPolicyPrefStoreTestBase<testing::Test> {
protected:
virtual void SetUp() {
- store_.AddObserver(&observer_);
+ store_->AddObserver(&observer_);
}
virtual void TearDown() {
- store_.RemoveObserver(&observer_);
+ store_->RemoveObserver(&observer_);
}
PrefStoreObserverMock observer_;
@@ -602,40 +613,40 @@ class ConfigurationPolicyPrefStoreRefreshTest
TEST_F(ConfigurationPolicyPrefStoreRefreshTest, Refresh) {
Value* value = NULL;
EXPECT_EQ(PrefStore::READ_NO_VALUE,
- store_.GetValue(prefs::kHomePage, NULL));
+ store_->GetValue(prefs::kHomePage, NULL));
EXPECT_CALL(observer_, OnPrefValueChanged(prefs::kHomePage)).Times(1);
provider_.AddPolicy(kPolicyHomePage,
Value::CreateStringValue("http://www.chromium.org"));
- store_.OnUpdatePolicy();
+ store_->OnUpdatePolicy();
Mock::VerifyAndClearExpectations(&observer_);
EXPECT_EQ(PrefStore::READ_OK,
- store_.GetValue(prefs::kHomePage, &value));
+ store_->GetValue(prefs::kHomePage, &value));
EXPECT_TRUE(StringValue("http://www.chromium.org").Equals(value));
EXPECT_CALL(observer_, OnPrefValueChanged(_)).Times(0);
- store_.OnUpdatePolicy();
+ store_->OnUpdatePolicy();
Mock::VerifyAndClearExpectations(&observer_);
EXPECT_CALL(observer_, OnPrefValueChanged(prefs::kHomePage)).Times(1);
provider_.RemovePolicy(kPolicyHomePage);
- store_.OnUpdatePolicy();
+ store_->OnUpdatePolicy();
Mock::VerifyAndClearExpectations(&observer_);
EXPECT_EQ(PrefStore::READ_NO_VALUE,
- store_.GetValue(prefs::kHomePage, NULL));
+ store_->GetValue(prefs::kHomePage, NULL));
}
TEST_F(ConfigurationPolicyPrefStoreRefreshTest, Initialization) {
- EXPECT_FALSE(store_.IsInitializationComplete());
+ EXPECT_FALSE(store_->IsInitializationComplete());
EXPECT_CALL(observer_, OnInitializationCompleted()).Times(1);
provider_.SetInitializationComplete(true);
- EXPECT_FALSE(store_.IsInitializationComplete());
+ EXPECT_FALSE(store_->IsInitializationComplete());
- store_.OnUpdatePolicy();
+ store_->OnUpdatePolicy();
Mock::VerifyAndClearExpectations(&observer_);
- EXPECT_TRUE(store_.IsInitializationComplete());
+ EXPECT_TRUE(store_->IsInitializationComplete());
}
} // namespace policy

Powered by Google App Engine
This is Rietveld 408576698