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

Side by Side Diff: extensions/common/manifest_handlers/shared_module_manifest_unittest.cc

Issue 263703002: Allow shared modules to whitelist extensions that import them. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: feedback Created 6 years, 7 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 unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « extensions/common/manifest_handlers/shared_module_info.cc ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "base/version.h" 5 #include "base/version.h"
6 #include "chrome/common/extensions/manifest_tests/extension_manifest_test.h" 6 #include "chrome/common/extensions/manifest_tests/extension_manifest_test.h"
7 #include "extensions/common/extension.h" 7 #include "extensions/common/extension.h"
8 #include "extensions/common/manifest_handlers/shared_module_info.h" 8 #include "extensions/common/manifest_handlers/shared_module_info.h"
9 #include "testing/gtest/include/gtest/gtest.h" 9 #include "testing/gtest/include/gtest/gtest.h"
10 10
(...skipping 21 matching lines...) Expand all
32 scoped_refptr<Extension> extension = LoadAndExpectSuccess(manifest); 32 scoped_refptr<Extension> extension = LoadAndExpectSuccess(manifest);
33 33
34 EXPECT_TRUE(SharedModuleInfo::IsSharedModule(extension.get())) 34 EXPECT_TRUE(SharedModuleInfo::IsSharedModule(extension.get()))
35 << manifest.name(); 35 << manifest.name();
36 EXPECT_FALSE(SharedModuleInfo::ImportsModules(extension.get())) 36 EXPECT_FALSE(SharedModuleInfo::ImportsModules(extension.get()))
37 << manifest.name(); 37 << manifest.name();
38 EXPECT_TRUE(SharedModuleInfo::IsExportAllowed(extension.get(), "foo")) 38 EXPECT_TRUE(SharedModuleInfo::IsExportAllowed(extension.get(), "foo"))
39 << manifest.name(); 39 << manifest.name();
40 EXPECT_TRUE(SharedModuleInfo::IsExportAllowed(extension.get(), "foo/bar")) 40 EXPECT_TRUE(SharedModuleInfo::IsExportAllowed(extension.get(), "foo/bar"))
41 << manifest.name(); 41 << manifest.name();
42
43 EXPECT_TRUE(SharedModuleInfo::IsExportAllowedByWhitelist(extension.get(),
44 kImportId1)) << manifest.name();
45 EXPECT_TRUE(SharedModuleInfo::IsExportAllowedByWhitelist(extension.get(),
46 kImportId2)) << manifest.name();
47 EXPECT_FALSE(SharedModuleInfo::IsExportAllowedByWhitelist(extension.get(),
48 kNoImport)) << manifest.name();
49 }
50
51 TEST_F(SharedModuleManifestTest, ExportWhitelistAll) {
52 Manifest manifest("shared_module_export_no_whitelist.json");
53
54 scoped_refptr<Extension> extension = LoadAndExpectSuccess(manifest);
55
56 EXPECT_TRUE(SharedModuleInfo::IsExportAllowedByWhitelist(extension.get(),
57 kImportId1)) << manifest.name();
58 EXPECT_TRUE(SharedModuleInfo::IsExportAllowedByWhitelist(extension.get(),
59 kImportId2)) << manifest.name();
60 EXPECT_TRUE(SharedModuleInfo::IsExportAllowedByWhitelist(extension.get(),
61 kNoImport)) << manifest.name();
42 } 62 }
43 63
44 TEST_F(SharedModuleManifestTest, ExportFoo) { 64 TEST_F(SharedModuleManifestTest, ExportFoo) {
45 Manifest manifest("shared_module_export_foo.json"); 65 Manifest manifest("shared_module_export_foo.json");
46 66
47 scoped_refptr<Extension> extension = LoadAndExpectSuccess(manifest); 67 scoped_refptr<Extension> extension = LoadAndExpectSuccess(manifest);
48 68
49 EXPECT_TRUE(SharedModuleInfo::IsSharedModule(extension.get())) 69 EXPECT_TRUE(SharedModuleInfo::IsSharedModule(extension.get()))
50 << manifest.name(); 70 << manifest.name();
51 EXPECT_FALSE(SharedModuleInfo::ImportsModules(extension.get())) 71 EXPECT_FALSE(SharedModuleInfo::ImportsModules(extension.get()))
52 << manifest.name(); 72 << manifest.name();
53 EXPECT_TRUE(SharedModuleInfo::IsExportAllowed(extension.get(), "foo")) 73 EXPECT_TRUE(SharedModuleInfo::IsExportAllowed(extension.get(), "foo"))
54 << manifest.name(); 74 << manifest.name();
55 EXPECT_FALSE(SharedModuleInfo::IsExportAllowed(extension.get(), "foo/bar")) 75 EXPECT_FALSE(SharedModuleInfo::IsExportAllowed(extension.get(), "foo/bar"))
56 << manifest.name(); 76 << manifest.name();
57 } 77 }
58 78
59 TEST_F(SharedModuleManifestTest, ExportParseErrors) { 79 TEST_F(SharedModuleManifestTest, ExportParseErrors) {
60 Testcase testcases[] = { 80 Testcase testcases[] = {
61 Testcase("shared_module_export_and_import.json", 81 Testcase("shared_module_export_and_import.json",
62 "Simultaneous 'import' and 'export' are not allowed."), 82 "Simultaneous 'import' and 'export' are not allowed."),
63 Testcase("shared_module_export_not_dict.json", 83 Testcase("shared_module_export_not_dict.json",
64 "Invalid value for 'export'."), 84 "Invalid value for 'export'."),
65 Testcase("shared_module_export_resources_not_list.json", 85 Testcase("shared_module_export_resources_not_list.json",
66 "Invalid value for 'export.resources'."), 86 "Invalid value for 'export.resources'."),
67 Testcase("shared_module_export_resource_not_string.json", 87 Testcase("shared_module_export_resource_not_string.json",
68 "Invalid value for 'export.resources[1]'."), 88 "Invalid value for 'export.resources[1]'."),
89 Testcase("shared_module_export_whitelist_item_not_id.json",
90 "Invalid value for 'export.whitelist[0]'."),
91 Testcase("shared_module_export_whitelist_item_not_string.json",
92 "Invalid value for 'export.whitelist[0]'."),
93 Testcase("shared_module_export_whitelist_not_list.json",
94 "Invalid value for 'export.whitelist'."),
69 }; 95 };
70 RunTestcases(testcases, arraysize(testcases), EXPECT_TYPE_ERROR); 96 RunTestcases(testcases, arraysize(testcases), EXPECT_TYPE_ERROR);
71 } 97 }
72 98
73 TEST_F(SharedModuleManifestTest, SharedModuleStaticFunctions) { 99 TEST_F(SharedModuleManifestTest, SharedModuleStaticFunctions) {
74 EXPECT_TRUE(SharedModuleInfo::IsImportedPath(kValidImportPath)); 100 EXPECT_TRUE(SharedModuleInfo::IsImportedPath(kValidImportPath));
75 EXPECT_FALSE(SharedModuleInfo::IsImportedPath(kInvalidImportPath)); 101 EXPECT_FALSE(SharedModuleInfo::IsImportedPath(kInvalidImportPath));
76 102
77 std::string id; 103 std::string id;
78 std::string relative; 104 std::string relative;
(...skipping 32 matching lines...) Expand 10 before | Expand all | Expand 10 after
111 "Invalid value for 'import'."), 137 "Invalid value for 'import'."),
112 Testcase("shared_module_import_invalid_id.json", 138 Testcase("shared_module_import_invalid_id.json",
113 "Invalid value for 'import[0].id'."), 139 "Invalid value for 'import[0].id'."),
114 Testcase("shared_module_import_invalid_version.json", 140 Testcase("shared_module_import_invalid_version.json",
115 "Invalid value for 'import[0].minimum_version'."), 141 "Invalid value for 'import[0].minimum_version'."),
116 }; 142 };
117 RunTestcases(testcases, arraysize(testcases), EXPECT_TYPE_ERROR); 143 RunTestcases(testcases, arraysize(testcases), EXPECT_TYPE_ERROR);
118 } 144 }
119 145
120 } // namespace extensions 146 } // namespace extensions
OLDNEW
« no previous file with comments | « extensions/common/manifest_handlers/shared_module_info.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698