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

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

Issue 5562002: Refactor FileBasedPolicyProvider, introduce AsynchronousPolicyProvider. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: merge with TOT Created 10 years 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/asynchronous_policy_loader_unittest.cc
diff --git a/chrome/browser/policy/asynchronous_policy_loader_unittest.cc b/chrome/browser/policy/asynchronous_policy_loader_unittest.cc
new file mode 100644
index 0000000000000000000000000000000000000000..499543158bec88377a608bd479ca68dd418c2829
--- /dev/null
+++ b/chrome/browser/policy/asynchronous_policy_loader_unittest.cc
@@ -0,0 +1,132 @@
+// Copyright (c) 2010 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/asynchronous_policy_loader.h"
+#include "chrome/browser/policy/asynchronous_policy_provider.h"
+#include "chrome/browser/policy/asynchronous_policy_test_base.h"
+#include "chrome/browser/policy/mock_configuration_policy_provider.h"
+#include "chrome/common/notification_observer_mock.h"
+#include "chrome/common/notification_registrar.h"
+#include "chrome/common/notification_service.h"
+#include "chrome/common/notification_type.h"
+
+using ::testing::_;
+using ::testing::InSequence;
+using ::testing::Return;
+
+namespace policy {
+
+class AsynchronousPolicyLoaderTest : public AsynchronousPolicyTestBase {
+ public:
+ AsynchronousPolicyLoaderTest() {}
+ virtual ~AsynchronousPolicyLoaderTest() {}
+
+ virtual void SetUp() {
+ AsynchronousPolicyTestBase::SetUp();
+ mock_provider_.reset(new MockConfigurationPolicyProvider());
+ }
+
+ protected:
+ scoped_ptr<MockConfigurationPolicyProvider> mock_provider_;
+
+ private:
+ DISALLOW_COPY_AND_ASSIGN(AsynchronousPolicyLoaderTest);
+};
+
+ACTION(CreateTestDictionary) {
+ return new DictionaryValue();
+}
+
+ACTION_P(CreateSequencedTestDictionary, number) {
+ DictionaryValue* test_dictionary = new DictionaryValue();
+ test_dictionary->SetInteger("id", ++(*number));
+ return test_dictionary;
+}
+
+ACTION(RescheduleImmediatePolicyReload) {
+ *arg1 = base::TimeDelta();
+ return false;
+}
+
+TEST_F(AsynchronousPolicyLoaderTest, InitialLoad) {
+ DictionaryValue* template_dict(new DictionaryValue());
+ EXPECT_CALL(*delegate_, Load()).WillOnce(Return(template_dict));
+ scoped_refptr<AsynchronousPolicyLoader> loader =
+ new AsynchronousPolicyLoader(delegate_.release());
+ loader->Init();
+ const DictionaryValue* loaded_dict(loader->policy());
+ EXPECT_TRUE(loaded_dict->Equals(template_dict));
+}
+
+// Verify that the fallback policy requests are made.
+TEST_F(AsynchronousPolicyLoaderTest, InitialLoadWithFallback) {
+ int dictionary_number = 0;
+ InSequence s;
+ EXPECT_CALL(*delegate_, Load()).WillOnce(
+ CreateSequencedTestDictionary(&dictionary_number));
+ EXPECT_CALL(*delegate_, Load()).WillOnce(
+ CreateSequencedTestDictionary(&dictionary_number));
+ scoped_refptr<AsynchronousPolicyLoader> loader =
+ new AsynchronousPolicyLoader(delegate_.release());
+ loader->Init();
+ loop_.RunAllPending();
+ loader->Reload();
+ loop_.RunAllPending();
+
+ const DictionaryValue* loaded_dict(loader->policy());
+ int loaded_number;
+ EXPECT_TRUE(loaded_dict->GetInteger("id", &loaded_number));
+ EXPECT_EQ(dictionary_number, loaded_number);
+}
+
+// Ensure that calling stop on the loader stops subsequent reloads from
+// happening.
+TEST_F(AsynchronousPolicyLoaderTest, Stop) {
+ ON_CALL(*delegate_, Load()).WillByDefault(CreateTestDictionary());
+ EXPECT_CALL(*delegate_, Load()).Times(1);
+ scoped_refptr<AsynchronousPolicyLoader> loader =
+ new AsynchronousPolicyLoader(delegate_.release());
+ loader->Init();
+ loop_.RunAllPending();
+ loader->Stop();
+ loop_.RunAllPending();
+ loader->Reload();
+ loop_.RunAllPending();
+}
+
+// Verifies that the provider is notified upon policy reload, but only
+// if the policy changed.
+TEST_F(AsynchronousPolicyLoaderTest, ProviderNotificationOnPolicyChange) {
+ InSequence s;
+ NotificationObserverMock observer;
+ // NotificationService service;
+ NotificationRegistrar registrar;
+ registrar.Add(&observer,
+ NotificationType::POLICY_CHANGED,
+ NotificationService::AllSources());
+ int dictionary_number_1 = 0;
+ int dictionary_number_2 = 0;
+ EXPECT_CALL(*delegate_, Load()).WillOnce(
+ CreateSequencedTestDictionary(&dictionary_number_1));
+ EXPECT_CALL(*delegate_, Load()).WillOnce(
+ CreateSequencedTestDictionary(&dictionary_number_2));
+ EXPECT_CALL(observer, Observe(_, _, _)).Times(0);
+ EXPECT_CALL(*delegate_, Load()).WillOnce(
+ CreateSequencedTestDictionary(&dictionary_number_2));
+ EXPECT_CALL(observer, Observe(_, _, _)).Times(1);
+ EXPECT_CALL(*delegate_, Load()).WillOnce(
+ CreateSequencedTestDictionary(&dictionary_number_1));
+ scoped_refptr<AsynchronousPolicyLoader> loader =
+ new AsynchronousPolicyLoader(delegate_.release());
+ AsynchronousPolicyProvider provider(NULL, loader);
+ loop_.RunAllPending();
+ loader->Reload();
+ loop_.RunAllPending();
+ loader->Reload();
+ loop_.RunAllPending();
+ loader->Reload();
+ loop_.RunAllPending();
+}
+
+} // namespace policy
« no previous file with comments | « chrome/browser/policy/asynchronous_policy_loader.cc ('k') | chrome/browser/policy/asynchronous_policy_provider.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698