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

Unified Diff: chrome/browser/extensions/api/file_system/file_system_apitest_chromeos.cc

Issue 137463006: Add tests for Google Drive integration in chrome.fileSystem API on Chrome OS. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Review fix. Created 6 years, 11 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
« no previous file with comments | « no previous file | chrome/chrome_tests.gypi » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/extensions/api/file_system/file_system_apitest_chromeos.cc
diff --git a/chrome/browser/extensions/api/file_system/file_system_apitest_chromeos.cc b/chrome/browser/extensions/api/file_system/file_system_apitest_chromeos.cc
new file mode 100644
index 0000000000000000000000000000000000000000..16abe55d56fd3bc345e9d06629047067b0c9cbba
--- /dev/null
+++ b/chrome/browser/extensions/api/file_system/file_system_apitest_chromeos.cc
@@ -0,0 +1,124 @@
+// Copyright 2014 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/apps/app_browsertest_util.h"
+#include "chrome/browser/chromeos/drive/drive_integration_service.h"
+#include "chrome/browser/chromeos/drive/file_system_interface.h"
+#include "chrome/browser/chromeos/drive/file_system_util.h"
+#include "chrome/browser/chromeos/drive/test_util.h"
+#include "chrome/browser/drive/fake_drive_service.h"
+#include "chrome/browser/extensions/api/file_system/file_system_api.h"
+#include "content/public/test/test_utils.h"
+#include "google_apis/drive/test_util.h"
+
+namespace extensions {
+
+// This class contains chrome.filesystem API test specific to Chrome OS, namely,
+// the integrated Google Drive support.
+class FileSystemApiTestForDrive : public PlatformAppBrowserTest {
+ public:
+ FileSystemApiTestForDrive()
+ : fake_drive_service_(NULL),
+ integration_service_(NULL) {
+ }
+
+ // Sets up fake Drive service for tests (this has to be injected before the
+ // real DriveIntegrationService instance is created.)
+ virtual void SetUpInProcessBrowserTestFixture() OVERRIDE {
+ PlatformAppBrowserTest::SetUpInProcessBrowserTestFixture();
+
+ ASSERT_TRUE(test_cache_root_.CreateUniqueTempDir());
+
+ create_drive_integration_service_ =
+ base::Bind(&FileSystemApiTestForDrive::CreateDriveIntegrationService,
+ base::Unretained(this));
+ service_factory_for_test_.reset(
+ new drive::DriveIntegrationServiceFactory::ScopedFactoryForTest(
+ &create_drive_integration_service_));
+ }
+
+ // Ensure the fake service's data is fetch in the local file system. This is
+ // necessary because the fetch starts lazily upon the first read operation.
+ virtual void SetUpOnMainThread() OVERRIDE {
+ PlatformAppBrowserTest::SetUpOnMainThread();
+
+ scoped_ptr<drive::ResourceEntry> entry;
+ drive::FileError error = drive::FILE_ERROR_FAILED;
+ integration_service_->file_system()->GetResourceEntry(
+ base::FilePath::FromUTF8Unsafe("drive/root"), // whatever
+ google_apis::test_util::CreateCopyResultCallback(&error, &entry));
+ drive::test_util::RunBlockingPoolTask();
+ ASSERT_EQ(drive::FILE_ERROR_OK, error);
+ }
+
+ virtual void TearDown() OVERRIDE {
+ FileSystemChooseEntryFunction::StopSkippingPickerForTest();
+ PlatformAppBrowserTest::TearDown();
+ };
+
+ private:
+ drive::DriveIntegrationService* CreateDriveIntegrationService(
+ Profile* profile) {
+ fake_drive_service_ = new drive::FakeDriveService;
+ fake_drive_service_->LoadResourceListForWapi(
+ "gdata/empty_feed.json");
+ fake_drive_service_->LoadAccountMetadataForWapi(
+ "gdata/account_metadata.json");
+ fake_drive_service_->LoadAppListForDriveApi("drive/applist.json");
+
+ SetUpTestFileHierarchy();
+
+ integration_service_ = new drive::DriveIntegrationService(
+ profile, NULL, fake_drive_service_, test_cache_root_.path(), NULL);
+ return integration_service_;
+ }
+
+ void SetUpTestFileHierarchy() {
+ const std::string root = fake_drive_service_->GetRootResourceId();
+ ASSERT_TRUE(AddTestFile("open_existing.txt", "Can you see me?", root));
+ }
+
+ bool AddTestFile(const std::string& title,
+ const std::string& data,
+ const std::string& parent_id) {
+ scoped_ptr<google_apis::ResourceEntry> resource_entry;
+ google_apis::GDataErrorCode error = google_apis::GDATA_OTHER_ERROR;
+ fake_drive_service_->AddNewFile(
+ "text/plain", data, parent_id, title, false,
+ google_apis::test_util::CreateCopyResultCallback(&error,
+ &resource_entry));
+ content::RunAllPendingInMessageLoop();
+ return error == google_apis::HTTP_CREATED && resource_entry;
+ }
+
+ base::ScopedTempDir test_cache_root_;
+ drive::FakeDriveService* fake_drive_service_;
+ drive::DriveIntegrationService* integration_service_;
+ drive::DriveIntegrationServiceFactory::FactoryCallback
+ create_drive_integration_service_;
+ scoped_ptr<drive::DriveIntegrationServiceFactory::ScopedFactoryForTest>
+ service_factory_for_test_;
+};
+
+IN_PROC_BROWSER_TEST_F(FileSystemApiTestForDrive,
+ FileSystemApiOpenExistingFileTest) {
+ base::FilePath test_file = drive::util::GetDriveMountPointPath().AppendASCII(
+ "root/open_existing.txt");
+ FileSystemChooseEntryFunction::SkipPickerAndAlwaysSelectPathForTest(
+ &test_file);
+ ASSERT_TRUE(RunPlatformAppTest("api_test/file_system/open_existing"))
+ << message_;
+}
+
+IN_PROC_BROWSER_TEST_F(FileSystemApiTestForDrive,
+ FileSystemApiOpenExistingFileWithWriteTest) {
+ base::FilePath test_file = drive::util::GetDriveMountPointPath().AppendASCII(
+ "root/open_existing.txt");
+ FileSystemChooseEntryFunction::SkipPickerAndAlwaysSelectPathForTest(
+ &test_file);
+ ASSERT_TRUE(RunPlatformAppTest(
+ "api_test/file_system/open_existing_with_write")) << message_;
+}
+
+} // namespace extensions
« no previous file with comments | « no previous file | chrome/chrome_tests.gypi » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698