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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: chrome/browser/drive/fake_drive_service_unittest.cc

Issue 17140009: Support contents upload on FakeDriveService (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: style fix Created 7 years, 6 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
« chrome/browser/drive/fake_drive_service.cc ('K') | « chrome/browser/drive/fake_drive_service.cc ('k') | chrome/browser/google_apis/test_util.h » ('j') | chrome/browser/google_apis/test_util.h » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/drive/fake_drive_service_unittest.cc
diff --git a/chrome/browser/drive/fake_drive_service_unittest.cc b/chrome/browser/drive/fake_drive_service_unittest.cc
index 957dc813176820579ff4b64075707e96a43d7263..6c95a06782b6b347284a3410a870771f5c1533b7 100644
--- a/chrome/browser/drive/fake_drive_service_unittest.cc
+++ b/chrome/browser/drive/fake_drive_service_unittest.cc
@@ -9,6 +9,7 @@
#include "base/file_util.h"
#include "base/files/scoped_temp_dir.h"
+#include "base/md5.h"
#include "base/run_loop.h"
#include "base/stl_util.h"
#include "base/strings/stringprintf.h"
@@ -1710,6 +1711,13 @@ TEST_F(FakeDriveServiceTest, ResumeUpload_NotFound) {
}
TEST_F(FakeDriveServiceTest, ResumeUpload_ExistingFile) {
+ base::ScopedTempDir temp_dir;
+ ASSERT_TRUE(temp_dir.CreateUniqueTempDir());
+ base::FilePath local_file_path =
+ temp_dir.path().Append(FILE_PATH_LITERAL("File 1.txt"));
+ std::string contents("hogefugapiyo");
+ test_util::WriteStringToFile(local_file_path, contents);
hashimoto 2013/06/20 10:17:42 nit: Should better EXPECT_TRUE/ASSERT_TRUE the res
tzik 2013/06/20 13:37:22 Done.
+
ASSERT_TRUE(fake_service_.LoadResourceListForWapi(
"chromeos/gdata/root_feed.json"));
@@ -1718,7 +1726,7 @@ TEST_F(FakeDriveServiceTest, ResumeUpload_ExistingFile) {
fake_service_.InitiateUploadExistingFile(
base::FilePath(FILE_PATH_LITERAL("drive/File 1.txt")),
"text/plain",
- 15,
+ contents.size(),
"file:2_file_resource_id",
"\"HhMOFgxXHit7ImBr\"",
test_util::CreateCopyResultCallback(&error, &upload_location));
@@ -1732,8 +1740,8 @@ TEST_F(FakeDriveServiceTest, ResumeUpload_ExistingFile) {
fake_service_.ResumeUpload(
base::FilePath(FILE_PATH_LITERAL("drive/File 1.txt")),
upload_location,
- 0, 13, 15, "text/plain",
- base::FilePath(),
+ 0, contents.size() / 2, contents.size(), "text/plain",
+ local_file_path,
test_util::CreateCopyResultCallback(&response, &entry),
base::Bind(&test_util::AppendProgressCallbackResult,
&upload_progress_values));
@@ -1744,14 +1752,15 @@ TEST_F(FakeDriveServiceTest, ResumeUpload_ExistingFile) {
ASSERT_TRUE(!upload_progress_values.empty());
EXPECT_TRUE(base::STLIsSorted(upload_progress_values));
EXPECT_LE(0, upload_progress_values.front().first);
- EXPECT_GE(13, upload_progress_values.back().first);
+ EXPECT_GE(static_cast<int64>(contents.size() / 2),
+ upload_progress_values.back().first);
upload_progress_values.clear();
fake_service_.ResumeUpload(
base::FilePath(FILE_PATH_LITERAL("drive/File 1.txt")),
upload_location,
- 13, 15, 15, "text/plain",
- base::FilePath(),
+ contents.size() / 2, contents.size(), contents.size(), "text/plain",
+ local_file_path,
test_util::CreateCopyResultCallback(&response, &entry),
base::Bind(&test_util::AppendProgressCallbackResult,
&upload_progress_values));
@@ -1759,15 +1768,25 @@ TEST_F(FakeDriveServiceTest, ResumeUpload_ExistingFile) {
EXPECT_EQ(HTTP_SUCCESS, response.code);
EXPECT_TRUE(entry.get());
- EXPECT_EQ(15L, entry->file_size());
+ EXPECT_EQ(static_cast<int64>(contents.size()),
+ entry->file_size());
EXPECT_TRUE(Exists(entry->resource_id()));
ASSERT_TRUE(!upload_progress_values.empty());
EXPECT_TRUE(base::STLIsSorted(upload_progress_values));
EXPECT_LE(0, upload_progress_values.front().first);
- EXPECT_GE(2, upload_progress_values.back().first);
+ EXPECT_GE(static_cast<int64>(contents.size() - contents.size() / 2),
+ upload_progress_values.back().first);
+ EXPECT_EQ(base::MD5String(contents), entry->file_md5());
}
TEST_F(FakeDriveServiceTest, ResumeUpload_NewFile) {
+ base::ScopedTempDir temp_dir;
+ ASSERT_TRUE(temp_dir.CreateUniqueTempDir());
+ base::FilePath local_file_path =
+ temp_dir.path().Append(FILE_PATH_LITERAL("new file.foo"));
+ std::string contents("hogefugapiyo");
+ test_util::WriteStringToFile(local_file_path, contents);
+
ASSERT_TRUE(fake_service_.LoadResourceListForWapi(
"chromeos/gdata/root_feed.json"));
@@ -1776,7 +1795,7 @@ TEST_F(FakeDriveServiceTest, ResumeUpload_NewFile) {
fake_service_.InitiateUploadNewFile(
base::FilePath(FILE_PATH_LITERAL("drive/Directory 1/new file.foo")),
"test/foo",
- 15,
+ contents.size(),
"folder:1_folder_resource_id",
"new file.foo",
test_util::CreateCopyResultCallback(&error, &upload_location));
@@ -1793,8 +1812,8 @@ TEST_F(FakeDriveServiceTest, ResumeUpload_NewFile) {
fake_service_.ResumeUpload(
base::FilePath(FILE_PATH_LITERAL("drive/Directory 1/new file.foo")),
upload_location,
- 0, 13, 15, "test/foo",
- base::FilePath(),
+ 0, contents.size() / 2, contents.size(), "test/foo",
+ local_file_path,
test_util::CreateCopyResultCallback(&response, &entry),
base::Bind(&test_util::AppendProgressCallbackResult,
&upload_progress_values));
@@ -1805,14 +1824,15 @@ TEST_F(FakeDriveServiceTest, ResumeUpload_NewFile) {
ASSERT_TRUE(!upload_progress_values.empty());
EXPECT_TRUE(base::STLIsSorted(upload_progress_values));
EXPECT_LE(0, upload_progress_values.front().first);
- EXPECT_GE(13, upload_progress_values.back().first);
+ EXPECT_GE(static_cast<int64>(contents.size() / 2),
+ upload_progress_values.back().first);
upload_progress_values.clear();
fake_service_.ResumeUpload(
base::FilePath(FILE_PATH_LITERAL("drive/Directory 1/new file.foo")),
upload_location,
- 13, 15, 15, "test/foo",
- base::FilePath(),
+ contents.size() / 2, contents.size(), contents.size(), "test/foo",
+ local_file_path,
test_util::CreateCopyResultCallback(&response, &entry),
base::Bind(&test_util::AppendProgressCallbackResult,
&upload_progress_values));
@@ -1820,12 +1840,14 @@ TEST_F(FakeDriveServiceTest, ResumeUpload_NewFile) {
EXPECT_EQ(HTTP_CREATED, response.code);
EXPECT_TRUE(entry.get());
- EXPECT_EQ(15L, entry->file_size());
+ EXPECT_EQ(static_cast<int64>(contents.size()), entry->file_size());
EXPECT_TRUE(Exists(entry->resource_id()));
ASSERT_TRUE(!upload_progress_values.empty());
EXPECT_TRUE(base::STLIsSorted(upload_progress_values));
EXPECT_LE(0, upload_progress_values.front().first);
- EXPECT_GE(2, upload_progress_values.back().first);
+ EXPECT_GE(static_cast<int64>(contents.size() - contents.size() / 2),
+ upload_progress_values.back().first);
+ EXPECT_EQ(base::MD5String(contents), entry->file_md5());
}
TEST_F(FakeDriveServiceTest, AddNewFile_ToRootDirectory) {
« chrome/browser/drive/fake_drive_service.cc ('K') | « chrome/browser/drive/fake_drive_service.cc ('k') | chrome/browser/google_apis/test_util.h » ('j') | chrome/browser/google_apis/test_util.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698