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

Unified Diff: chrome/browser/chromeos/extensions/device_local_account_management_policy_provider_unittest.cc

Issue 24261010: Allow explicitly whitelisted apps/extensions in public sessions (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 3 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/chromeos/extensions/device_local_account_management_policy_provider_unittest.cc
diff --git a/chrome/browser/chromeos/extensions/device_local_account_management_policy_provider_unittest.cc b/chrome/browser/chromeos/extensions/device_local_account_management_policy_provider_unittest.cc
new file mode 100644
index 0000000000000000000000000000000000000000..2352aea9e177b3570bdc33d04a7854b68661c8cb
--- /dev/null
+++ b/chrome/browser/chromeos/extensions/device_local_account_management_policy_provider_unittest.cc
@@ -0,0 +1,84 @@
+// Copyright 2013 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/chromeos/extensions/device_local_account_management_policy_provider.h"
+
+#include <string>
+
+#include "base/files/file_path.h"
+#include "base/memory/ref_counted.h"
+#include "base/values.h"
+#include "chrome/common/extensions/extension.h"
+#include "extensions/common/manifest.h"
+#include "extensions/common/manifest_constants.h"
+#include "testing/gtest/include/gtest/gtest.h"
+
+namespace chromeos {
+
+namespace {
+
+const char kWhitelistedId[] = "bpmcpldpdmajfigpchkicefoigmkfalc";
+
+scoped_refptr<const extensions::Extension> CreateExtensionFromValues(
+ const std::string& id,
+ base::DictionaryValue* values) {
+ values->SetString(extensions::manifest_keys::kName, "test");
+ values->SetString(extensions::manifest_keys::kVersion, "0.1");
+ std::string error;
+ return extensions::Extension::Create(base::FilePath(),
+ extensions::Manifest::INTERNAL,
+ *values,
+ extensions::Extension::NO_FLAGS,
+ id,
+ &error);
+}
+
+scoped_refptr<const extensions::Extension> CreateExtension(
+ const std::string& id) {
+ base::DictionaryValue values;
+ return CreateExtensionFromValues(id, &values);
+}
+
+scoped_refptr<const extensions::Extension> CreateHostedApp() {
+ base::DictionaryValue values;
+ values.Set(extensions::manifest_keys::kApp, new base::DictionaryValue);
+ values.Set(extensions::manifest_keys::kWebURLs, new base::ListValue);
+ return CreateExtensionFromValues(std::string(), &values);
+}
+
+} // namespace
+
+typedef testing::Test DeviceLocalAccountManagementPolicyProviderTest;
+
+TEST_F(DeviceLocalAccountManagementPolicyProviderTest, CheckPolicy) {
+ DeviceLocalAccountManagementPolicyProvider provider;
+
+ // Verify that if an extension's type has been whitelisted for use in
+ // device-local accounts, the extension can be installed.
+ scoped_refptr<const extensions::Extension> extension = CreateHostedApp();
+ ASSERT_TRUE(extension);
+ string16 error;
+ EXPECT_TRUE(provider.UserMayLoad(extension.get(), &error));
+ EXPECT_EQ(string16(), error);
+ error.clear();
+
+ // Verify that if an extension's ID has been explicitly whitelisted for use in
+ // device-local accounts, the extension can be installed.
+ extension = CreateExtension(kWhitelistedId);
+ ASSERT_TRUE(extension);
+ EXPECT_TRUE(provider.UserMayLoad(extension.get(), &error));
+ EXPECT_EQ(string16(), error);
+ error.clear();
+
+ // Verify that if neither the type nor the ID of an extension have been
+ // whitelisted for use in device-local accounts, the extension cannot be
+ // installed.
+ extension = CreateExtension(std::string());
+ ASSERT_TRUE(extension);
+ EXPECT_FALSE(provider.UserMayLoad(extension.get(), &error));
+ EXPECT_NE(string16(), error);
+ error.clear();
+}
+
+} // namespace chromeos

Powered by Google App Engine
This is Rietveld 408576698