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

Unified Diff: chrome/browser/google_apis/gdata_wapi_operations_unittest.cc

Issue 11577002: Converted ResumeUploadOperation to use JSON in response instead of XML. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Cleaned up. Created 8 years 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/google_apis/gdata_wapi_operations_unittest.cc
diff --git a/chrome/browser/google_apis/gdata_wapi_operations_unittest.cc b/chrome/browser/google_apis/gdata_wapi_operations_unittest.cc
index 8225658430198983b884f36a5f6761869eff4fb9..74f6eeaa05eb02588fad344bc3ca3a8723a4aae3 100644
--- a/chrome/browser/google_apis/gdata_wapi_operations_unittest.cc
+++ b/chrome/browser/google_apis/gdata_wapi_operations_unittest.cc
@@ -351,11 +351,11 @@ class GDataWapiOperationsTest : public testing::Test {
return scoped_ptr<test_server::HttpResponse>();
}
- // TODO(satorux): We should create a correct entry for the uploaded file,
- // but for now, just return entry.xml.
+ // TODO(satorux): We should create a correct JSON data for the uploaded
+ // file, but for now, just return file_entry.json.
scoped_ptr<test_server::HttpResponse> response =
CreateHttpResponseFromFile(
- test_util::GetTestFilePath("gdata/entry.xml"));
+ test_util::GetTestFilePath("gdata/file_entry.json"));
// response.code() is set to SUCCESS. Change it to CREATED if it's a new
// file.
if (absolute_url.path() == "/upload_new_file")
@@ -915,7 +915,8 @@ TEST_F(GDataWapiOperationsTest, UploadNewFile) {
EXPECT_EQ(test_server_.GetURL("/upload_new_file"), upload_url);
EXPECT_EQ(test_server::METHOD_POST, http_request_.method);
// convert=false should be passed as files should be uploaded as-is.
- EXPECT_EQ("/feeds/upload/create-session/default/private/full?convert=false",
+ EXPECT_EQ("/feeds/upload/create-session/default/private/full"
+ "?convert=false&v=3&alt=json",
http_request_.relative_url);
EXPECT_EQ("text/plain", http_request_.headers["X-Upload-Content-Type"]);
EXPECT_EQ("application/atom+xml", http_request_.headers["Content-Type"]);
@@ -1013,7 +1014,8 @@ TEST_F(GDataWapiOperationsTest, UploadNewLargeFile) {
EXPECT_EQ(test_server_.GetURL("/upload_new_file"), upload_url);
EXPECT_EQ(test_server::METHOD_POST, http_request_.method);
// convert=false should be passed as files should be uploaded as-is.
- EXPECT_EQ("/feeds/upload/create-session/default/private/full?convert=false",
+ EXPECT_EQ("/feeds/upload/create-session/default/private/full?convert=false"
+ "&v=3&alt=json",
http_request_.relative_url);
EXPECT_EQ("text/plain", http_request_.headers["X-Upload-Content-Type"]);
EXPECT_EQ("application/atom+xml", http_request_.headers["Content-Type"]);
@@ -1136,7 +1138,8 @@ TEST_F(GDataWapiOperationsTest, UploadNewEmptyFile) {
EXPECT_EQ(test_server_.GetURL("/upload_new_file"), upload_url);
EXPECT_EQ(test_server::METHOD_POST, http_request_.method);
// convert=false should be passed as files should be uploaded as-is.
- EXPECT_EQ("/feeds/upload/create-session/default/private/full?convert=false",
+ EXPECT_EQ("/feeds/upload/create-session/default/private/full?convert=false"
+ "&v=3&alt=json",
http_request_.relative_url);
EXPECT_EQ("text/plain", http_request_.headers["X-Upload-Content-Type"]);
EXPECT_EQ("application/atom+xml", http_request_.headers["Content-Type"]);
@@ -1231,7 +1234,7 @@ TEST_F(GDataWapiOperationsTest, UploadExistingFile) {
EXPECT_EQ(test_server::METHOD_PUT, http_request_.method);
// convert=false should be passed as files should be uploaded as-is.
EXPECT_EQ("/feeds/upload/create-session/default/private/full/file:foo"
- "?convert=false",
+ "?convert=false&v=3&alt=json",
http_request_.relative_url);
// Even though the body is empty, the content type should be set to
// "text/plain".

Powered by Google App Engine
This is Rietveld 408576698