OLD | NEW |
| (Empty) |
1 // Copyright 2014 The Chromium Authors. All rights reserved. | |
2 // Use of this source code is governed by a BSD-style license that can be | |
3 // found in the LICENSE file. | |
4 | |
5 #include "chrome/browser/drive/drive_app_registry.h" | |
6 | |
7 #include "base/files/file_path.h" | |
8 #include "base/macros.h" | |
9 #include "base/message_loop/message_loop.h" | |
10 #include "base/run_loop.h" | |
11 #include "base/values.h" | |
12 #include "chrome/browser/drive/drive_app_registry_observer.h" | |
13 #include "chrome/browser/drive/fake_drive_service.h" | |
14 #include "google_apis/drive/drive_api_parser.h" | |
15 #include "google_apis/drive/test_util.h" | |
16 #include "testing/gtest/include/gtest/gtest.h" | |
17 | |
18 namespace drive { | |
19 | |
20 class TestDriveAppRegistryObserver : public DriveAppRegistryObserver { | |
21 public: | |
22 explicit TestDriveAppRegistryObserver(DriveAppRegistry* registry) | |
23 : registry_(registry), | |
24 update_count_(0) { | |
25 registry_->AddObserver(this); | |
26 } | |
27 ~TestDriveAppRegistryObserver() override { registry_->RemoveObserver(this); } | |
28 | |
29 int update_count() const { return update_count_; } | |
30 | |
31 private: | |
32 // DriveAppRegistryObserver overrides: | |
33 void OnDriveAppRegistryUpdated() override { ++update_count_; } | |
34 | |
35 DriveAppRegistry* registry_; | |
36 int update_count_; | |
37 DISALLOW_COPY_AND_ASSIGN(TestDriveAppRegistryObserver); | |
38 }; | |
39 | |
40 class DriveAppRegistryTest : public testing::Test { | |
41 protected: | |
42 void SetUp() override { | |
43 fake_drive_service_.reset(new FakeDriveService); | |
44 fake_drive_service_->LoadAppListForDriveApi("drive/applist.json"); | |
45 | |
46 apps_registry_.reset(new DriveAppRegistry(fake_drive_service_.get())); | |
47 } | |
48 | |
49 bool VerifyApp(const std::vector<DriveAppInfo>& list, | |
50 const std::string& app_id, | |
51 const std::string& app_name) { | |
52 bool found = false; | |
53 for (size_t i = 0; i < list.size(); ++i) { | |
54 const DriveAppInfo& app = list[i]; | |
55 if (app_id == app.app_id) { | |
56 EXPECT_EQ(app_name, app.app_name); | |
57 found = true; | |
58 break; | |
59 } | |
60 } | |
61 EXPECT_TRUE(found) << "Unable to find app with app_id " << app_id; | |
62 return found; | |
63 } | |
64 | |
65 base::MessageLoop message_loop_; | |
66 scoped_ptr<FakeDriveService> fake_drive_service_; | |
67 scoped_ptr<DriveAppRegistry> apps_registry_; | |
68 }; | |
69 | |
70 TEST_F(DriveAppRegistryTest, BasicParse) { | |
71 TestDriveAppRegistryObserver observer(apps_registry_.get()); | |
72 | |
73 apps_registry_->Update(); | |
74 base::RunLoop().RunUntilIdle(); | |
75 EXPECT_EQ(1, observer.update_count()); | |
76 | |
77 std::vector<DriveAppInfo> apps; | |
78 apps_registry_->GetAppList(&apps); | |
79 | |
80 ASSERT_EQ(2u, apps.size()); | |
81 EXPECT_EQ("123456788192", apps[0].app_id); | |
82 EXPECT_EQ("Drive app 1", apps[0].app_name); | |
83 EXPECT_EQ("https://www.example.com/createForApp1", | |
84 apps[0].create_url.spec()); | |
85 EXPECT_EQ("abcdefghabcdefghabcdefghabcdefgh", apps[0].product_id); | |
86 EXPECT_TRUE(apps[0].is_removable); | |
87 } | |
88 | |
89 TEST_F(DriveAppRegistryTest, LoadAndFindDriveApps) { | |
90 TestDriveAppRegistryObserver observer(apps_registry_.get()); | |
91 | |
92 apps_registry_->Update(); | |
93 base::RunLoop().RunUntilIdle(); | |
94 EXPECT_EQ(1, observer.update_count()); | |
95 | |
96 // Find by primary extension 'exe'. | |
97 std::vector<DriveAppInfo> ext_results; | |
98 base::FilePath ext_file(FILE_PATH_LITERAL("drive/file.exe")); | |
99 apps_registry_->GetAppsForFile(ext_file.Extension(), "", &ext_results); | |
100 ASSERT_EQ(1U, ext_results.size()); | |
101 VerifyApp(ext_results, "123456788192", "Drive app 1"); | |
102 | |
103 // Find by primary MIME type. | |
104 std::vector<DriveAppInfo> primary_app; | |
105 apps_registry_->GetAppsForFile(base::FilePath::StringType(), | |
106 "application/vnd.google-apps.drive-sdk.123456788192", &primary_app); | |
107 ASSERT_EQ(1U, primary_app.size()); | |
108 VerifyApp(primary_app, "123456788192", "Drive app 1"); | |
109 | |
110 // Find by secondary MIME type. | |
111 std::vector<DriveAppInfo> secondary_app; | |
112 apps_registry_->GetAppsForFile( | |
113 base::FilePath::StringType(), "text/html", &secondary_app); | |
114 ASSERT_EQ(1U, secondary_app.size()); | |
115 VerifyApp(secondary_app, "123456788192", "Drive app 1"); | |
116 } | |
117 | |
118 TEST_F(DriveAppRegistryTest, UpdateFromAppList) { | |
119 scoped_ptr<base::Value> app_info_value = | |
120 google_apis::test_util::LoadJSONFile("drive/applist.json"); | |
121 scoped_ptr<google_apis::AppList> app_list( | |
122 google_apis::AppList::CreateFrom(*app_info_value)); | |
123 | |
124 TestDriveAppRegistryObserver observer(apps_registry_.get()); | |
125 apps_registry_->UpdateFromAppList(*app_list); | |
126 EXPECT_EQ(1, observer.update_count()); | |
127 | |
128 // Confirm that something was loaded from applist.json. | |
129 std::vector<DriveAppInfo> ext_results; | |
130 base::FilePath ext_file(FILE_PATH_LITERAL("drive/file.exe")); | |
131 apps_registry_->GetAppsForFile(ext_file.Extension(), "", &ext_results); | |
132 ASSERT_EQ(1U, ext_results.size()); | |
133 } | |
134 | |
135 TEST_F(DriveAppRegistryTest, MultipleUpdate) { | |
136 TestDriveAppRegistryObserver observer(apps_registry_.get()); | |
137 | |
138 // Call Update(). | |
139 apps_registry_->Update(); | |
140 EXPECT_EQ(0, observer.update_count()); | |
141 | |
142 // Call Update() again. | |
143 // This call should be ignored because there is already an ongoing update. | |
144 apps_registry_->Update(); | |
145 EXPECT_EQ(0, observer.update_count()); | |
146 | |
147 // The app list should be loaded only once. | |
148 base::RunLoop().RunUntilIdle(); | |
149 EXPECT_EQ(1, fake_drive_service_->app_list_load_count()); | |
150 EXPECT_EQ(1, observer.update_count()); | |
151 } | |
152 | |
153 TEST(DriveAppRegistryUtilTest, FindPreferredIcon_Empty) { | |
154 DriveAppInfo::IconList icons; | |
155 EXPECT_EQ("", | |
156 util::FindPreferredIcon(icons, util::kPreferredIconSize).spec()); | |
157 } | |
158 | |
159 TEST(DriveAppRegistryUtilTest, FindPreferredIcon_) { | |
160 const char kSmallerIconUrl[] = "http://example.com/smaller.png"; | |
161 const char kMediumIconUrl[] = "http://example.com/medium.png"; | |
162 const char kBiggerIconUrl[] = "http://example.com/bigger.png"; | |
163 const int kMediumSize = 16; | |
164 | |
165 DriveAppInfo::IconList icons; | |
166 // The icons are not sorted by the size. | |
167 icons.push_back(std::make_pair(kMediumSize, | |
168 GURL(kMediumIconUrl))); | |
169 icons.push_back(std::make_pair(kMediumSize + 2, | |
170 GURL(kBiggerIconUrl))); | |
171 icons.push_back(std::make_pair(kMediumSize - 2, | |
172 GURL(kSmallerIconUrl))); | |
173 | |
174 // Exact match. | |
175 EXPECT_EQ(kMediumIconUrl, | |
176 util::FindPreferredIcon(icons, kMediumSize).spec()); | |
177 // The requested size is in-between of smaller.png and | |
178 // medium.png. medium.png should be returned. | |
179 EXPECT_EQ(kMediumIconUrl, | |
180 util::FindPreferredIcon(icons, kMediumSize - 1).spec()); | |
181 // The requested size is smaller than the smallest icon. The smallest icon | |
182 // should be returned. | |
183 EXPECT_EQ(kSmallerIconUrl, | |
184 util::FindPreferredIcon(icons, kMediumSize - 3).spec()); | |
185 // The requested size is larger than the largest icon. The largest icon | |
186 // should be returned. | |
187 EXPECT_EQ(kBiggerIconUrl, | |
188 util::FindPreferredIcon(icons, kMediumSize + 3).spec()); | |
189 } | |
190 | |
191 TEST_F(DriveAppRegistryTest, UninstallDriveApp) { | |
192 apps_registry_->Update(); | |
193 base::RunLoop().RunUntilIdle(); | |
194 | |
195 std::vector<DriveAppInfo> apps; | |
196 apps_registry_->GetAppList(&apps); | |
197 size_t original_count = apps.size(); | |
198 | |
199 // Uninstall an existing app. | |
200 google_apis::DriveApiErrorCode error = google_apis::DRIVE_OTHER_ERROR; | |
201 apps_registry_->UninstallApp( | |
202 "123456788192", | |
203 google_apis::test_util::CreateCopyResultCallback(&error)); | |
204 base::RunLoop().RunUntilIdle(); | |
205 EXPECT_EQ(error, google_apis::HTTP_NO_CONTENT); | |
206 | |
207 // Check that the number of apps is decreased by one. | |
208 apps_registry_->GetAppList(&apps); | |
209 EXPECT_EQ(original_count - 1, apps.size()); | |
210 | |
211 // Try to uninstall a non-existing app. | |
212 error = google_apis::DRIVE_OTHER_ERROR; | |
213 apps_registry_->UninstallApp( | |
214 "non-existing-app-id", | |
215 google_apis::test_util::CreateCopyResultCallback(&error)); | |
216 base::RunLoop().RunUntilIdle(); | |
217 EXPECT_EQ(error, google_apis::HTTP_NOT_FOUND); | |
218 | |
219 // Check that the number is not changed this time. | |
220 apps_registry_->GetAppList(&apps); | |
221 EXPECT_EQ(original_count - 1, apps.size()); | |
222 } | |
223 | |
224 } // namespace drive | |
OLD | NEW |