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

Unified Diff: webkit/fileapi/local_file_util_unittest.cc

Issue 15371005: Move browser-specific FileAPI code from webkit/fileapi to webkit/browser/fileapi (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « webkit/fileapi/local_file_util.cc ('k') | webkit/fileapi/native_file_util.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webkit/fileapi/local_file_util_unittest.cc
diff --git a/webkit/fileapi/local_file_util_unittest.cc b/webkit/fileapi/local_file_util_unittest.cc
deleted file mode 100644
index 87904672b32969d934f49da1b0aed62c8d87b81f..0000000000000000000000000000000000000000
--- a/webkit/fileapi/local_file_util_unittest.cc
+++ /dev/null
@@ -1,370 +0,0 @@
-// Copyright (c) 2012 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 "base/file_util.h"
-#include "base/files/file_path.h"
-#include "base/files/scoped_temp_dir.h"
-#include "base/message_loop.h"
-#include "base/message_loop_proxy.h"
-#include "base/platform_file.h"
-#include "base/strings/sys_string_conversions.h"
-#include "base/utf_string_conversions.h"
-#include "testing/gtest/include/gtest/gtest.h"
-#include "webkit/fileapi/async_file_test_helper.h"
-#include "webkit/fileapi/file_system_context.h"
-#include "webkit/fileapi/file_system_file_util.h"
-#include "webkit/fileapi/file_system_operation_context.h"
-#include "webkit/fileapi/file_system_types.h"
-#include "webkit/fileapi/local_file_system_test_helper.h"
-#include "webkit/fileapi/local_file_util.h"
-#include "webkit/fileapi/native_file_util.h"
-
-namespace fileapi {
-
-// TODO(dmikurube): Cover all public methods in LocalFileUtil.
-class LocalFileUtilTest : public testing::Test {
- public:
- LocalFileUtilTest()
- : test_helper_(GURL("http://foo/"), kFileSystemTypeTest) {}
-
- virtual void SetUp() {
- ASSERT_TRUE(data_dir_.CreateUniqueTempDir());
- test_helper_.SetUp(data_dir_.path());
- }
-
- virtual void TearDown() {
- test_helper_.TearDown();
- }
-
- protected:
- FileSystemOperationContext* NewContext() {
- FileSystemOperationContext* context = test_helper_.NewOperationContext();
- return context;
- }
-
- LocalFileUtil* FileUtil() {
- return static_cast<LocalFileUtil*>(test_helper_.file_util());
- }
-
- FileSystemURL Path(const std::string& file_name) {
- return test_helper_.CreateURLFromUTF8(file_name);
- }
-
- base::FilePath LocalPath(const char *file_name) {
- return test_helper_.GetLocalPathFromASCII(file_name);
- }
-
- bool FileExists(const char *file_name) {
- return file_util::PathExists(LocalPath(file_name)) &&
- !file_util::DirectoryExists(LocalPath(file_name));
- }
-
- bool DirectoryExists(const char *file_name) {
- return file_util::DirectoryExists(LocalPath(file_name));
- }
-
- int64 GetSize(const char *file_name) {
- base::PlatformFileInfo info;
- file_util::GetFileInfo(LocalPath(file_name), &info);
- return info.size;
- }
-
- base::PlatformFileError CreateFile(const char* file_name,
- base::PlatformFile* file_handle,
- bool* created) {
- int file_flags = base::PLATFORM_FILE_CREATE |
- base::PLATFORM_FILE_WRITE | base::PLATFORM_FILE_ASYNC;
-
- scoped_ptr<FileSystemOperationContext> context(NewContext());
- return FileUtil()->CreateOrOpen(
- context.get(),
- Path(file_name),
- file_flags, file_handle, created);
- }
-
- base::PlatformFileError EnsureFileExists(const char* file_name,
- bool* created) {
- scoped_ptr<FileSystemOperationContext> context(NewContext());
- return FileUtil()->EnsureFileExists(
- context.get(),
- Path(file_name), created);
- }
-
- const LocalFileSystemTestOriginHelper& test_helper() const {
- return test_helper_;
- }
-
- FileSystemContext* file_system_context() {
- return test_helper_.file_system_context();
- }
-
- private:
- scoped_ptr<LocalFileUtil> local_file_util_;
- base::ScopedTempDir data_dir_;
- base::MessageLoop message_loop_;
- LocalFileSystemTestOriginHelper test_helper_;
-
- DISALLOW_COPY_AND_ASSIGN(LocalFileUtilTest);
-};
-
-TEST_F(LocalFileUtilTest, CreateAndClose) {
- const char *file_name = "test_file";
- base::PlatformFile file_handle;
- bool created;
- ASSERT_EQ(base::PLATFORM_FILE_OK,
- CreateFile(file_name, &file_handle, &created));
- ASSERT_TRUE(created);
-
- EXPECT_TRUE(FileExists(file_name));
- EXPECT_EQ(0, GetSize(file_name));
-
- scoped_ptr<FileSystemOperationContext> context(NewContext());
- EXPECT_EQ(base::PLATFORM_FILE_OK,
- FileUtil()->Close(context.get(), file_handle));
-}
-
-// file_util::CreateSymbolicLink is only supported on POSIX.
-#if defined(OS_POSIX)
-TEST_F(LocalFileUtilTest, CreateFailForSymlink) {
- // Create symlink target file.
- const char *target_name = "symlink_target";
- base::PlatformFile target_handle;
- bool symlink_target_created = false;
- ASSERT_EQ(base::PLATFORM_FILE_OK,
- CreateFile(target_name, &target_handle, &symlink_target_created));
- ASSERT_TRUE(symlink_target_created);
- base::FilePath target_path = LocalPath(target_name);
-
- // Create symlink where target must be real file.
- const char *symlink_name = "symlink_file";
- base::FilePath symlink_path = LocalPath(symlink_name);
- ASSERT_TRUE(file_util::CreateSymbolicLink(target_path, symlink_path));
- ASSERT_TRUE(FileExists(symlink_name));
-
- // Try to open the symlink file which should fail.
- scoped_ptr<FileSystemOperationContext> context(NewContext());
- FileSystemURL url = test_helper().CreateURLFromUTF8(symlink_name);
- int file_flags = base::PLATFORM_FILE_OPEN | base::PLATFORM_FILE_READ;
- base::PlatformFile file_handle;
- bool created = false;
- EXPECT_EQ(base::PLATFORM_FILE_ERROR_NOT_FOUND, FileUtil()->CreateOrOpen(
- context.get(), url, file_flags, &file_handle, &created));
- EXPECT_FALSE(created);
-}
-#endif
-
-TEST_F(LocalFileUtilTest, EnsureFileExists) {
- const char *file_name = "foobar";
- bool created;
- ASSERT_EQ(base::PLATFORM_FILE_OK, EnsureFileExists(file_name, &created));
- ASSERT_TRUE(created);
-
- EXPECT_TRUE(FileExists(file_name));
- EXPECT_EQ(0, GetSize(file_name));
-
- ASSERT_EQ(base::PLATFORM_FILE_OK, EnsureFileExists(file_name, &created));
- EXPECT_FALSE(created);
-}
-
-TEST_F(LocalFileUtilTest, TouchFile) {
- const char *file_name = "test_file";
- base::PlatformFile file_handle;
- bool created;
- ASSERT_EQ(base::PLATFORM_FILE_OK,
- CreateFile(file_name, &file_handle, &created));
- ASSERT_TRUE(created);
-
- scoped_ptr<FileSystemOperationContext> context(NewContext());
-
- base::PlatformFileInfo info;
- ASSERT_TRUE(file_util::GetFileInfo(LocalPath(file_name), &info));
- const base::Time new_accessed =
- info.last_accessed + base::TimeDelta::FromHours(10);
- const base::Time new_modified =
- info.last_modified + base::TimeDelta::FromHours(5);
-
- EXPECT_EQ(base::PLATFORM_FILE_OK,
- FileUtil()->Touch(context.get(), Path(file_name),
- new_accessed, new_modified));
-
- ASSERT_TRUE(file_util::GetFileInfo(LocalPath(file_name), &info));
- EXPECT_EQ(new_accessed, info.last_accessed);
- EXPECT_EQ(new_modified, info.last_modified);
-
- EXPECT_EQ(base::PLATFORM_FILE_OK,
- FileUtil()->Close(context.get(), file_handle));
-}
-
-TEST_F(LocalFileUtilTest, TouchDirectory) {
- const char *dir_name = "test_dir";
- scoped_ptr<FileSystemOperationContext> context(NewContext());
- ASSERT_EQ(base::PLATFORM_FILE_OK,
- FileUtil()->CreateDirectory(context.get(),
- Path(dir_name),
- false /* exclusive */,
- false /* recursive */));
-
- base::PlatformFileInfo info;
- ASSERT_TRUE(file_util::GetFileInfo(LocalPath(dir_name), &info));
- const base::Time new_accessed =
- info.last_accessed + base::TimeDelta::FromHours(10);
- const base::Time new_modified =
- info.last_modified + base::TimeDelta::FromHours(5);
-
- EXPECT_EQ(base::PLATFORM_FILE_OK,
- FileUtil()->Touch(context.get(), Path(dir_name),
- new_accessed, new_modified));
-
- ASSERT_TRUE(file_util::GetFileInfo(LocalPath(dir_name), &info));
- EXPECT_EQ(new_accessed, info.last_accessed);
- EXPECT_EQ(new_modified, info.last_modified);
-}
-
-TEST_F(LocalFileUtilTest, Truncate) {
- const char *file_name = "truncated";
- bool created;
- ASSERT_EQ(base::PLATFORM_FILE_OK, EnsureFileExists(file_name, &created));
- ASSERT_TRUE(created);
-
- scoped_ptr<FileSystemOperationContext> context;
-
- context.reset(NewContext());
- ASSERT_EQ(base::PLATFORM_FILE_OK,
- FileUtil()->Truncate(context.get(), Path(file_name), 1020));
-
- EXPECT_TRUE(FileExists(file_name));
- EXPECT_EQ(1020, GetSize(file_name));
-}
-
-TEST_F(LocalFileUtilTest, CopyFile) {
- const char *from_file = "fromfile";
- const char *to_file1 = "tofile1";
- const char *to_file2 = "tofile2";
- bool created;
- ASSERT_EQ(base::PLATFORM_FILE_OK, EnsureFileExists(from_file, &created));
- ASSERT_TRUE(created);
-
- scoped_ptr<FileSystemOperationContext> context;
- context.reset(NewContext());
- ASSERT_EQ(base::PLATFORM_FILE_OK,
- FileUtil()->Truncate(context.get(), Path(from_file), 1020));
-
- EXPECT_TRUE(FileExists(from_file));
- EXPECT_EQ(1020, GetSize(from_file));
-
- ASSERT_EQ(base::PLATFORM_FILE_OK,
- AsyncFileTestHelper::Copy(file_system_context(),
- Path(from_file), Path(to_file1)));
-
- context.reset(NewContext());
- ASSERT_EQ(base::PLATFORM_FILE_OK,
- AsyncFileTestHelper::Copy(file_system_context(),
- Path(from_file), Path(to_file2)));
-
- EXPECT_TRUE(FileExists(from_file));
- EXPECT_EQ(1020, GetSize(from_file));
- EXPECT_TRUE(FileExists(to_file1));
- EXPECT_EQ(1020, GetSize(to_file1));
- EXPECT_TRUE(FileExists(to_file2));
- EXPECT_EQ(1020, GetSize(to_file2));
-}
-
-TEST_F(LocalFileUtilTest, CopyDirectory) {
- const char *from_dir = "fromdir";
- const char *from_file = "fromdir/fromfile";
- const char *to_dir = "todir";
- const char *to_file = "todir/fromfile";
- bool created;
- scoped_ptr<FileSystemOperationContext> context;
-
- context.reset(NewContext());
- ASSERT_EQ(base::PLATFORM_FILE_OK,
- FileUtil()->CreateDirectory(context.get(), Path(from_dir), false, false));
- ASSERT_EQ(base::PLATFORM_FILE_OK, EnsureFileExists(from_file, &created));
- ASSERT_TRUE(created);
-
- context.reset(NewContext());
- ASSERT_EQ(base::PLATFORM_FILE_OK,
- FileUtil()->Truncate(context.get(), Path(from_file), 1020));
-
- EXPECT_TRUE(DirectoryExists(from_dir));
- EXPECT_TRUE(FileExists(from_file));
- EXPECT_EQ(1020, GetSize(from_file));
- EXPECT_FALSE(DirectoryExists(to_dir));
-
- context.reset(NewContext());
- ASSERT_EQ(base::PLATFORM_FILE_OK,
- AsyncFileTestHelper::Copy(file_system_context(),
- Path(from_dir), Path(to_dir)));
-
- EXPECT_TRUE(DirectoryExists(from_dir));
- EXPECT_TRUE(FileExists(from_file));
- EXPECT_EQ(1020, GetSize(from_file));
- EXPECT_TRUE(DirectoryExists(to_dir));
- EXPECT_TRUE(FileExists(to_file));
- EXPECT_EQ(1020, GetSize(to_file));
-}
-
-TEST_F(LocalFileUtilTest, MoveFile) {
- const char *from_file = "fromfile";
- const char *to_file = "tofile";
- bool created;
- ASSERT_EQ(base::PLATFORM_FILE_OK, EnsureFileExists(from_file, &created));
- ASSERT_TRUE(created);
- scoped_ptr<FileSystemOperationContext> context;
-
- context.reset(NewContext());
- ASSERT_EQ(base::PLATFORM_FILE_OK,
- FileUtil()->Truncate(context.get(), Path(from_file), 1020));
-
- EXPECT_TRUE(FileExists(from_file));
- EXPECT_EQ(1020, GetSize(from_file));
-
- context.reset(NewContext());
- ASSERT_EQ(base::PLATFORM_FILE_OK,
- AsyncFileTestHelper::Move(file_system_context(),
- Path(from_file), Path(to_file)));
-
- EXPECT_FALSE(FileExists(from_file));
- EXPECT_TRUE(FileExists(to_file));
- EXPECT_EQ(1020, GetSize(to_file));
-}
-
-TEST_F(LocalFileUtilTest, MoveDirectory) {
- const char *from_dir = "fromdir";
- const char *from_file = "fromdir/fromfile";
- const char *to_dir = "todir";
- const char *to_file = "todir/fromfile";
- bool created;
- scoped_ptr<FileSystemOperationContext> context;
-
- context.reset(NewContext());
- ASSERT_EQ(base::PLATFORM_FILE_OK,
- FileUtil()->CreateDirectory(context.get(), Path(from_dir), false, false));
- ASSERT_EQ(base::PLATFORM_FILE_OK, EnsureFileExists(from_file, &created));
- ASSERT_TRUE(created);
-
- context.reset(NewContext());
- ASSERT_EQ(base::PLATFORM_FILE_OK,
- FileUtil()->Truncate(context.get(), Path(from_file), 1020));
-
- EXPECT_TRUE(DirectoryExists(from_dir));
- EXPECT_TRUE(FileExists(from_file));
- EXPECT_EQ(1020, GetSize(from_file));
- EXPECT_FALSE(DirectoryExists(to_dir));
-
- context.reset(NewContext());
- ASSERT_EQ(base::PLATFORM_FILE_OK,
- AsyncFileTestHelper::Move(file_system_context(),
- Path(from_dir), Path(to_dir)));
-
- EXPECT_FALSE(DirectoryExists(from_dir));
- EXPECT_TRUE(DirectoryExists(to_dir));
- EXPECT_TRUE(FileExists(to_file));
- EXPECT_EQ(1020, GetSize(to_file));
-}
-
-} // namespace fileapi
« no previous file with comments | « webkit/fileapi/local_file_util.cc ('k') | webkit/fileapi/native_file_util.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698