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

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

Issue 12861002: drive: Remove "/-/mine" from WAPI resource list urls. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Modified the test feed to include an not-mine entry. Created 7 years, 9 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/google_apis/gdata_wapi_url_generator_unittest.cc
diff --git a/chrome/browser/google_apis/gdata_wapi_url_generator_unittest.cc b/chrome/browser/google_apis/gdata_wapi_url_generator_unittest.cc
index ba5f041032b0b574d96cd58e21f08fb9ca9f85ae..bdd8b20c785962f59d3aa84d36a2a010ed0d1071 100644
--- a/chrome/browser/google_apis/gdata_wapi_url_generator_unittest.cc
+++ b/chrome/browser/google_apis/gdata_wapi_url_generator_unittest.cc
@@ -70,7 +70,7 @@ TEST_F(GDataWapiUrlGeneratorTest, AddFeedUrlParams) {
TEST_F(GDataWapiUrlGeneratorTest, GenerateResourceListUrl) {
// This is the very basic URL for the GetResourceList operation.
EXPECT_EQ(
- "https://docs.google.com/feeds/default/private/full/-/mine"
+ "https://docs.google.com/feeds/default/private/full"
"?v=3&alt=json&showroot=true&showfolders=true&max-results=500"
"&include-installed-apps=true",
url_generator_.GenerateResourceListUrl(GURL(), // override_url,
@@ -95,7 +95,7 @@ TEST_F(GDataWapiUrlGeneratorTest, GenerateResourceListUrl) {
).spec());
// With a non-zero start_changestamp provided, the base URL is changed from
- // "full/-/mine" to "changes", and "start-index" parameter is added.
+ // "full" to "changes", and "start-index" parameter is added.
EXPECT_EQ(
"https://docs.google.com/feeds/default/private/changes"
"?v=3&alt=json&showroot=true&showfolders=true&max-results=500"
@@ -111,7 +111,7 @@ TEST_F(GDataWapiUrlGeneratorTest, GenerateResourceListUrl) {
// With a non-empty search string provided, "max-results" value is changed,
// and "q" parameter is added.
EXPECT_EQ(
- "https://docs.google.com/feeds/default/private/full/-/mine"
+ "https://docs.google.com/feeds/default/private/full"
"?v=3&alt=json&showroot=true&showfolders=true&max-results=50"
"&include-installed-apps=true&q=foo",
url_generator_.GenerateResourceListUrl(GURL(), // override_url,
@@ -137,7 +137,7 @@ TEST_F(GDataWapiUrlGeneratorTest, GenerateResourceListUrl) {
// With a non-empty directory resource ID provided, the base URL is
// changed, but the default parameters remain.
EXPECT_EQ(
- "https://docs.google.com/feeds/default/private/full/XXX/contents/-/mine"
+ "https://docs.google.com/feeds/default/private/full/XXX/contents"
"?v=3&alt=json&showroot=true&showfolders=true&max-results=500"
"&include-installed-apps=true",
url_generator_.GenerateResourceListUrl(GURL(), // override_url,
« no previous file with comments | « chrome/browser/google_apis/gdata_wapi_url_generator.cc ('k') | chrome/test/data/chromeos/gdata/root_feed.json » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698