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

Unified Diff: webkit/fileapi/local_file_system_file_util_unittest.cc

Issue 7470037: [Refactor] to rename and re-layer the file_util stack layers. (Closed) Base URL: http://git.chromium.org/git/chromium.git@trunk
Patch Set: Rebased on the svn tree. Created 9 years, 3 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_system_file_util.cc ('k') | webkit/fileapi/local_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_system_file_util_unittest.cc
diff --git a/webkit/fileapi/local_file_system_file_util_unittest.cc b/webkit/fileapi/local_file_system_file_util_unittest.cc
deleted file mode 100644
index 5f3906ede56bdaba8baabb8ecb6f5f355d5db9f6..0000000000000000000000000000000000000000
--- a/webkit/fileapi/local_file_system_file_util_unittest.cc
+++ /dev/null
@@ -1,275 +0,0 @@
-// Copyright (c) 2011 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_path.h"
-#include "base/memory/scoped_callback_factory.h"
-#include "base/message_loop_proxy.h"
-#include "base/platform_file.h"
-#include "base/scoped_temp_dir.h"
-#include "base/sys_string_conversions.h"
-#include "base/utf_string_conversions.h"
-#include "testing/gtest/include/gtest/gtest.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_path_manager.h"
-#include "webkit/fileapi/file_system_test_helper.h"
-#include "webkit/fileapi/file_system_types.h"
-#include "webkit/fileapi/local_file_system_file_util.h"
-
-namespace fileapi {
-
-// TODO(dmikurube): Cover all public methods in LocalFileSystemFileUtil.
-class LocalFileSystemFileUtilTest : public testing::Test {
- public:
- LocalFileSystemFileUtilTest()
- : local_file_util_(
- new LocalFileSystemFileUtil(new FileSystemFileUtil())),
- callback_factory_(ALLOW_THIS_IN_INITIALIZER_LIST(this)) {
- }
-
- void SetUp() {
- ASSERT_TRUE(data_dir_.CreateUniqueTempDir());
- test_helper_.SetUp(data_dir_.path(), FileUtil());
- }
-
- void TearDown() {
- test_helper_.TearDown();
- }
-
- protected:
- FileSystemOperationContext* NewContext() {
- FileSystemOperationContext* context = test_helper_.NewOperationContext();
- return context;
- }
-
- LocalFileSystemFileUtil* FileUtil() {
- return local_file_util_.get();
- }
-
- static FilePath Path(const std::string& file_name) {
- return FilePath().AppendASCII(file_name);
- }
-
- 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);
- }
-
- private:
- scoped_ptr<LocalFileSystemFileUtil> local_file_util_;
- ScopedTempDir data_dir_;
- FileSystemTestOriginHelper test_helper_;
-
- base::ScopedCallbackFactory<LocalFileSystemFileUtilTest> callback_factory_;
-
- DISALLOW_COPY_AND_ASSIGN(LocalFileSystemFileUtilTest);
-};
-
-TEST_F(LocalFileSystemFileUtilTest, 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));
-}
-
-TEST_F(LocalFileSystemFileUtilTest, 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(LocalFileSystemFileUtilTest, 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(LocalFileSystemFileUtilTest, 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));
-
- context.reset(NewContext());
- ASSERT_EQ(base::PLATFORM_FILE_OK,
- FileUtil()->Copy(context.get(), Path(from_file), Path(to_file1)));
-
- context.reset(NewContext());
- ASSERT_EQ(base::PLATFORM_FILE_OK,
- FileUtil()->Copy(context.get(), 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(LocalFileSystemFileUtilTest, 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,
- FileUtil()->Copy(context.get(), 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(LocalFileSystemFileUtilTest, 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,
- FileUtil()->Move(context.get(), Path(from_file), Path(to_file)));
-
- EXPECT_FALSE(FileExists(from_file));
- EXPECT_TRUE(FileExists(to_file));
- EXPECT_EQ(1020, GetSize(to_file));
-}
-
-TEST_F(LocalFileSystemFileUtilTest, 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,
- FileUtil()->Move(context.get(), 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_system_file_util.cc ('k') | webkit/fileapi/local_file_util.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698