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

Unified Diff: chrome/browser/extensions/convert_user_script_unittest.cc

Issue 340057: Add first-class support for user scripts (Closed)
Patch Set: newness Created 11 years, 1 month 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/extensions/convert_user_script_unittest.cc
diff --git a/chrome/browser/extensions/convert_user_script_unittest.cc b/chrome/browser/extensions/convert_user_script_unittest.cc
new file mode 100644
index 0000000000000000000000000000000000000000..882c16b52bafa6cda2ccc3a028b55f3067ca970f
--- /dev/null
+++ b/chrome/browser/extensions/convert_user_script_unittest.cc
@@ -0,0 +1,89 @@
+// Copyright (c) 2009 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 <string>
+#include <vector>
+
+#include "base/file_path.h"
+#include "base/file_util.h"
+#include "base/path_service.h"
+#include "base/scoped_ptr.h"
+#include "chrome/browser/extensions/convert_user_script.h"
+#include "chrome/common/chrome_paths.h"
+#include "chrome/common/extensions/extension.h"
+#include "testing/gtest/include/gtest/gtest.h"
+
+TEST(ExtensionFromUserScript, Basic) {
+ FilePath test_file;
+ ASSERT_TRUE(PathService::Get(chrome::DIR_TEST_DATA, &test_file));
+ test_file = test_file.AppendASCII("extensions")
+ .AppendASCII("user_script_basic.user.js");
+
+ std::string error;
+ scoped_ptr<Extension> extension(ConvertUserScriptToExtension(
+ test_file, GURL("http://www.google.com/foo"), &error));
+
+ ASSERT_TRUE(extension.get());
+ EXPECT_EQ("", error);
+
+ // Validate generated extension metadata.
+ EXPECT_EQ("My user script", extension->name());
+ EXPECT_EQ("1.0", extension->VersionString());
+ EXPECT_EQ("Does totally awesome stuff.", extension->description());
+ EXPECT_EQ("IhCFCg9PMQTAcJdc9ytUP99WME+4yh6aMnM1uupkovrMzMzMzMzMzA==",
+ extension->public_key());
+
+ ASSERT_EQ(1, extension->content_scripts().size());
+ const UserScript& script = extension->content_scripts()[0];
+ ASSERT_EQ(2, script.globs().size());
+ EXPECT_EQ("http://www.google.com/*", script.globs().at(0));
+ EXPECT_EQ("http://www.yahoo.com/*", script.globs().at(1));
+ ASSERT_EQ(1, script.exclude_globs().size());
+ EXPECT_EQ("*foo*", script.exclude_globs().at(0));
+ ASSERT_EQ(1, script.url_patterns().size());
+ EXPECT_EQ("http://www.google.com/*", script.url_patterns()[0].GetAsString());
+
+ // Make sure the files actually exist on disk.
+ EXPECT_TRUE(file_util::PathExists(
+ extension->path().Append(script.js_scripts()[0].relative_path())));
+ EXPECT_TRUE(file_util::PathExists(
+ extension->path().AppendASCII(Extension::kManifestFilename)));
+}
+
+TEST(ExtensionFromUserScript, NoMetdata) {
+ FilePath test_file;
+ ASSERT_TRUE(PathService::Get(chrome::DIR_TEST_DATA, &test_file));
+ test_file = test_file.AppendASCII("extensions")
+ .AppendASCII("user_script_no_metadata.user.js");
+
+ std::string error;
+ scoped_ptr<Extension> extension(ConvertUserScriptToExtension(
+ test_file, GURL("http://www.google.com/foo/bar.user.js?monkey"), &error));
+
+ ASSERT_TRUE(extension.get());
+ EXPECT_EQ("", error);
+
+ // Validate generated extension metadata.
+ EXPECT_EQ("bar.user.js", extension->name());
+ EXPECT_EQ("1.0", extension->VersionString());
+ EXPECT_EQ("", extension->description());
+ EXPECT_EQ("k1WxKx54hX6tfl5gQaXD/m4d9QUMwRdXWM4RW+QkWcbMzMzMzMzMzA==",
+ extension->public_key());
+
+ ASSERT_EQ(1, extension->content_scripts().size());
+ const UserScript& script = extension->content_scripts()[0];
+ ASSERT_EQ(1, script.globs().size());
+ EXPECT_EQ("*", script.globs()[0]);
+ EXPECT_EQ(0, script.exclude_globs().size());
+ ASSERT_EQ(3, script.url_patterns().size());
+ EXPECT_EQ("http://*/*", script.url_patterns()[0].GetAsString());
+ EXPECT_EQ("https://*/*", script.url_patterns()[1].GetAsString());
+ EXPECT_EQ("file:///*", script.url_patterns()[2].GetAsString());
+
+ // Make sure the files actually exist on disk.
+ EXPECT_TRUE(file_util::PathExists(
+ extension->path().Append(script.js_scripts()[0].relative_path())));
+ EXPECT_TRUE(file_util::PathExists(
+ extension->path().AppendASCII(Extension::kManifestFilename)));
+}

Powered by Google App Engine
This is Rietveld 408576698