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

Unified Diff: chrome/browser/chromeos/file_system_provider/fileapi/provider_async_file_util_unittest.cc

Issue 240483008: Revert of [fsp] Add an initial AsyncFileUtil. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 8 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/file_system_provider/fileapi/provider_async_file_util_unittest.cc
diff --git a/chrome/browser/chromeos/file_system_provider/fileapi/provider_async_file_util_unittest.cc b/chrome/browser/chromeos/file_system_provider/fileapi/provider_async_file_util_unittest.cc
deleted file mode 100644
index 507217974679c029a637a9e5000f0dcae4fcbbb9..0000000000000000000000000000000000000000
--- a/chrome/browser/chromeos/file_system_provider/fileapi/provider_async_file_util_unittest.cc
+++ /dev/null
@@ -1,373 +0,0 @@
-// 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 <string>
-#include <vector>
-
-#include "base/files/file.h"
-#include "base/files/file_path.h"
-#include "base/files/scoped_temp_dir.h"
-#include "base/memory/scoped_ptr.h"
-#include "base/memory/weak_ptr.h"
-#include "base/platform_file.h"
-#include "chrome/browser/chromeos/file_system_provider/fileapi/provider_async_file_util.h"
-#include "chrome/browser/chromeos/file_system_provider/mount_path_util.h"
-#include "chrome/test/base/testing_profile.h"
-#include "content/public/test/test_browser_thread_bundle.h"
-#include "content/public/test/test_file_system_context.h"
-#include "testing/gtest/include/gtest/gtest.h"
-#include "webkit/browser/fileapi/async_file_util.h"
-#include "webkit/browser/fileapi/file_system_context.h"
-#include "webkit/browser/fileapi/file_system_url.h"
-#include "webkit/common/blob/shareable_file_reference.h"
-
-namespace chromeos {
-namespace file_system_provider {
-namespace {
-
-// Logs callbacks invocations on the tested operations.
-// TODO(mtomasz): Store and verify more arguments, once the operations return
-// anything else than just an error.
-class EventLogger {
- public:
- EventLogger() : weak_ptr_factory_(this) {}
- virtual ~EventLogger() {}
-
- void OnStatus(base::File::Error error) {
- error_.reset(new base::File::Error(error));
- }
-
- void OnCreateOrOpen(base::File::Error error,
- base::PassPlatformFile platform_file,
- const base::Closure& on_close_callback) {
- error_.reset(new base::File::Error(error));
- }
-
- void OnEnsureFileExists(base::File::Error error, bool created) {
- error_.reset(new base::File::Error(error));
- }
-
- void OnGetFileInfo(base::File::Error error,
- const base::File::Info& file_info) {
- error_.reset(new base::File::Error(error));
- }
-
- void OnReadDirectory(base::File::Error error,
- const fileapi::AsyncFileUtil::EntryList& file_list,
- bool has_more) {
- error_.reset(new base::File::Error(error));
- }
-
- void OnCreateSnapshotFile(
- base::File::Error error,
- const base::File::Info& file_info,
- const base::FilePath& platform_path,
- const scoped_refptr<webkit_blob::ShareableFileReference>& file_ref) {
- error_.reset(new base::File::Error(error));
- }
-
- void OnCopyFileProgress(int64 size) {}
-
- base::WeakPtr<EventLogger> GetWeakPtr() {
- return weak_ptr_factory_.GetWeakPtr();
- }
-
- base::File::Error* error() { return error_.get(); }
-
- private:
- scoped_ptr<base::File::Error> error_;
- base::WeakPtrFactory<EventLogger> weak_ptr_factory_;
-
- DISALLOW_COPY_AND_ASSIGN(EventLogger);
-};
-
-// Creates a FileSystemURL for tests.
-fileapi::FileSystemURL CreateFileSystemURL(Profile* profile,
- const base::FilePath& file_path) {
- const std::string kExtensionId = "mbflcebpggnecokmikipoihdbecnjfoj";
- const int kFileSystemId = 1;
- const std::string origin = std::string("chrome-extension://") + kExtensionId;
- const base::FilePath mount_path =
- util::GetMountPointPath(profile, kExtensionId, kFileSystemId);
- return fileapi::FileSystemURL::CreateForTest(GURL(origin),
- fileapi::kFileSystemTypeProvided,
- mount_path.Append(file_path));
-}
-
-} // namespace
-
-// Tests in this file are very lightweight and just test integration between
-// AsyncFileUtil and ProvideFileSystemInterface. Currently it tests if not
-// implemented operations return a correct error code. For not allowed
-// operations it is FILE_ERROR_SECURITY, and for not implemented the error is
-// FILE_ERROR_NOT_FOUND.
-class FileSystemProviderProviderAsyncFileUtilTest : public testing::Test {
- protected:
- FileSystemProviderProviderAsyncFileUtilTest() {}
- virtual ~FileSystemProviderProviderAsyncFileUtilTest() {}
-
- virtual void SetUp() OVERRIDE {
- ASSERT_TRUE(data_dir_.CreateUniqueTempDir());
- profile_.reset(new TestingProfile);
- async_file_util_.reset(new internal::ProviderAsyncFileUtil);
- file_system_context_ =
- content::CreateFileSystemContextForTesting(NULL, data_dir_.path());
- file_url_ = CreateFileSystemURL(
- profile_.get(), base::FilePath::FromUTF8Unsafe("hello/world.txt"));
- directory_url_ = CreateFileSystemURL(
- profile_.get(), base::FilePath::FromUTF8Unsafe("hello"));
- root_url_ = CreateFileSystemURL(profile_.get(), base::FilePath());
- LOG(ERROR) << file_url_.DebugString();
- }
-
- scoped_ptr<fileapi::FileSystemOperationContext> CreateOperationContext() {
- return make_scoped_ptr(
- new fileapi::FileSystemOperationContext(file_system_context_.get()));
- }
-
- content::TestBrowserThreadBundle thread_bundle_;
- base::ScopedTempDir data_dir_;
- scoped_ptr<TestingProfile> profile_;
- scoped_ptr<fileapi::AsyncFileUtil> async_file_util_;
- scoped_refptr<fileapi::FileSystemContext> file_system_context_;
- fileapi::FileSystemURL file_url_;
- fileapi::FileSystemURL directory_url_;
- fileapi::FileSystemURL root_url_;
-};
-
-TEST_F(FileSystemProviderProviderAsyncFileUtilTest, CreateOrOpen_Create) {
- EventLogger logger;
-
- async_file_util_->CreateOrOpen(
- CreateOperationContext(),
- file_url_,
- base::PLATFORM_FILE_CREATE,
- base::Bind(&EventLogger::OnCreateOrOpen, logger.GetWeakPtr()));
-
- ASSERT_TRUE(logger.error());
- EXPECT_EQ(base::File::FILE_ERROR_SECURITY, *logger.error());
-}
-
-TEST_F(FileSystemProviderProviderAsyncFileUtilTest, CreateOrOpen_CreateAlways) {
- EventLogger logger;
-
- async_file_util_->CreateOrOpen(
- CreateOperationContext(),
- file_url_,
- base::PLATFORM_FILE_CREATE_ALWAYS,
- base::Bind(&EventLogger::OnCreateOrOpen, logger.GetWeakPtr()));
-
- ASSERT_TRUE(logger.error());
- EXPECT_EQ(base::File::FILE_ERROR_SECURITY, *logger.error());
-}
-
-TEST_F(FileSystemProviderProviderAsyncFileUtilTest, CreateOrOpen_OpenAlways) {
- EventLogger logger;
-
- async_file_util_->CreateOrOpen(
- CreateOperationContext(),
- file_url_,
- base::PLATFORM_FILE_OPEN_ALWAYS,
- base::Bind(&EventLogger::OnCreateOrOpen, logger.GetWeakPtr()));
-
- ASSERT_TRUE(logger.error());
- EXPECT_EQ(base::File::FILE_ERROR_SECURITY, *logger.error());
-}
-
-TEST_F(FileSystemProviderProviderAsyncFileUtilTest,
- CreateOrOpen_OpenTruncated) {
- EventLogger logger;
-
- async_file_util_->CreateOrOpen(
- CreateOperationContext(),
- file_url_,
- base::PLATFORM_FILE_OPEN_TRUNCATED,
- base::Bind(&EventLogger::OnCreateOrOpen, logger.GetWeakPtr()));
-
- ASSERT_TRUE(logger.error());
- EXPECT_EQ(base::File::FILE_ERROR_SECURITY, *logger.error());
-}
-
-TEST_F(FileSystemProviderProviderAsyncFileUtilTest, CreateOrOpen_Open) {
- EventLogger logger;
-
- async_file_util_->CreateOrOpen(
- CreateOperationContext(),
- file_url_,
- base::PLATFORM_FILE_OPEN,
- base::Bind(&EventLogger::OnCreateOrOpen, logger.GetWeakPtr()));
-
- ASSERT_TRUE(logger.error());
- EXPECT_EQ(base::File::FILE_ERROR_NOT_FOUND, *logger.error());
-}
-
-TEST_F(FileSystemProviderProviderAsyncFileUtilTest, EnsureFileExists) {
- EventLogger logger;
-
- async_file_util_->EnsureFileExists(
- CreateOperationContext(),
- file_url_,
- base::Bind(&EventLogger::OnEnsureFileExists, logger.GetWeakPtr()));
-
- ASSERT_TRUE(logger.error());
- EXPECT_EQ(base::File::FILE_ERROR_SECURITY, *logger.error());
-}
-
-TEST_F(FileSystemProviderProviderAsyncFileUtilTest, CreateDirectory) {
- EventLogger logger;
-
- async_file_util_->CreateDirectory(
- CreateOperationContext(),
- directory_url_,
- false, // exclusive
- false, // recursive
- base::Bind(&EventLogger::OnStatus, logger.GetWeakPtr()));
-
- ASSERT_TRUE(logger.error());
- EXPECT_EQ(base::File::FILE_ERROR_SECURITY, *logger.error());
-}
-
-TEST_F(FileSystemProviderProviderAsyncFileUtilTest, GetFileInfo) {
- EventLogger logger;
-
- async_file_util_->GetFileInfo(
- CreateOperationContext(),
- file_url_,
- base::Bind(&EventLogger::OnGetFileInfo, logger.GetWeakPtr()));
-
- ASSERT_TRUE(logger.error());
- EXPECT_EQ(base::File::FILE_ERROR_NOT_FOUND, *logger.error());
-}
-
-TEST_F(FileSystemProviderProviderAsyncFileUtilTest, ReadDirectory) {
- EventLogger logger;
-
- async_file_util_->ReadDirectory(
- CreateOperationContext(),
- root_url_,
- base::Bind(&EventLogger::OnReadDirectory, logger.GetWeakPtr()));
-
- ASSERT_TRUE(logger.error());
- EXPECT_EQ(base::File::FILE_ERROR_NOT_FOUND, *logger.error());
-}
-
-TEST_F(FileSystemProviderProviderAsyncFileUtilTest, Touch) {
- EventLogger logger;
-
- async_file_util_->CreateDirectory(
- CreateOperationContext(),
- file_url_,
- false, // exclusive
- false, // recursive
- base::Bind(&EventLogger::OnStatus, logger.GetWeakPtr()));
-
- ASSERT_TRUE(logger.error());
- EXPECT_EQ(base::File::FILE_ERROR_SECURITY, *logger.error());
-}
-
-TEST_F(FileSystemProviderProviderAsyncFileUtilTest, Truncate) {
- EventLogger logger;
-
- async_file_util_->Truncate(
- CreateOperationContext(),
- file_url_,
- 0, // length
- base::Bind(&EventLogger::OnStatus, logger.GetWeakPtr()));
-
- ASSERT_TRUE(logger.error());
- EXPECT_EQ(base::File::FILE_ERROR_SECURITY, *logger.error());
-}
-
-TEST_F(FileSystemProviderProviderAsyncFileUtilTest, CopyFileLocal) {
- EventLogger logger;
-
- async_file_util_->CopyFileLocal(
- CreateOperationContext(),
- file_url_, // src_url
- file_url_, // dst_url
- fileapi::FileSystemOperation::OPTION_NONE,
- base::Bind(&EventLogger::OnCopyFileProgress, logger.GetWeakPtr()),
- base::Bind(&EventLogger::OnStatus, logger.GetWeakPtr()));
-
- ASSERT_TRUE(logger.error());
- EXPECT_EQ(base::File::FILE_ERROR_SECURITY, *logger.error());
-}
-
-TEST_F(FileSystemProviderProviderAsyncFileUtilTest, MoveFileLocal) {
- EventLogger logger;
-
- async_file_util_->MoveFileLocal(
- CreateOperationContext(),
- file_url_, // src_url
- file_url_, // dst_url
- fileapi::FileSystemOperation::OPTION_NONE,
- base::Bind(&EventLogger::OnStatus, logger.GetWeakPtr()));
-
- ASSERT_TRUE(logger.error());
- EXPECT_EQ(base::File::FILE_ERROR_SECURITY, *logger.error());
-}
-
-TEST_F(FileSystemProviderProviderAsyncFileUtilTest, CopyInForeignFile) {
- EventLogger logger;
-
- async_file_util_->CopyInForeignFile(
- CreateOperationContext(),
- base::FilePath(), // src_file_path
- file_url_, // dst_url
- base::Bind(&EventLogger::OnStatus, logger.GetWeakPtr()));
-
- ASSERT_TRUE(logger.error());
- EXPECT_EQ(base::File::FILE_ERROR_SECURITY, *logger.error());
-}
-
-TEST_F(FileSystemProviderProviderAsyncFileUtilTest, DeleteFile) {
- EventLogger logger;
-
- async_file_util_->DeleteFile(
- CreateOperationContext(),
- file_url_,
- base::Bind(&EventLogger::OnStatus, logger.GetWeakPtr()));
-
- ASSERT_TRUE(logger.error());
- EXPECT_EQ(base::File::FILE_ERROR_SECURITY, *logger.error());
-}
-
-TEST_F(FileSystemProviderProviderAsyncFileUtilTest, DeleteDirectory) {
- EventLogger logger;
-
- async_file_util_->DeleteDirectory(
- CreateOperationContext(),
- directory_url_,
- base::Bind(&EventLogger::OnStatus, logger.GetWeakPtr()));
-
- ASSERT_TRUE(logger.error());
- EXPECT_EQ(base::File::FILE_ERROR_SECURITY, *logger.error());
-}
-
-TEST_F(FileSystemProviderProviderAsyncFileUtilTest, DeleteRecursively) {
- EventLogger logger;
-
- async_file_util_->DeleteRecursively(
- CreateOperationContext(),
- directory_url_,
- base::Bind(&EventLogger::OnStatus, logger.GetWeakPtr()));
-
- ASSERT_TRUE(logger.error());
- EXPECT_EQ(base::File::FILE_ERROR_SECURITY, *logger.error());
-}
-
-TEST_F(FileSystemProviderProviderAsyncFileUtilTest, CreateSnapshotFile) {
- EventLogger logger;
-
- async_file_util_->CreateSnapshotFile(
- CreateOperationContext(),
- file_url_,
- base::Bind(&EventLogger::OnCreateSnapshotFile, logger.GetWeakPtr()));
-
- ASSERT_TRUE(logger.error());
- EXPECT_EQ(base::File::FILE_ERROR_NOT_FOUND, *logger.error());
-}
-
-} // namespace file_system_provider
-} // namespace chromeos

Powered by Google App Engine
This is Rietveld 408576698