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

Unified Diff: chrome/browser/chromeos/gdata/gdata_parser_unittest.cc

Issue 10381155: Added support for Google Drive's external app document type (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 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
Index: chrome/browser/chromeos/gdata/gdata_parser_unittest.cc
diff --git a/chrome/browser/chromeos/gdata/gdata_parser_unittest.cc b/chrome/browser/chromeos/gdata/gdata_parser_unittest.cc
index c6dd6745ee04d93584dcb25d684ab7c431f127e2..053f4dcb858f0b44a463d73845de0c8c833fdeea 100644
--- a/chrome/browser/chromeos/gdata/gdata_parser_unittest.cc
+++ b/chrome/browser/chromeos/gdata/gdata_parser_unittest.cc
@@ -106,7 +106,7 @@ TEST_F(GDataParserTest, DocumentFeedJsonParser) {
EXPECT_EQ("application/atom+xml", resumable_link->mime_type());
// Check entries.
- ASSERT_EQ(3U, feed->entries().size());
+ ASSERT_EQ(4U, feed->entries().size());
// Check a folder entry.
const DocumentEntry* folder_entry = feed->entries()[0];
@@ -169,6 +169,15 @@ TEST_F(GDataParserTest, DocumentFeedJsonParser) {
const DocumentEntry* document_entry = feed->entries()[2];
ASSERT_TRUE(document_entry);
EXPECT_EQ(DocumentEntry::DOCUMENT, document_entry->kind());
+ EXPECT_TRUE(document_entry->is_hosted_document());
+ EXPECT_TRUE(document_entry->is_google_document());
satorux1 2012/05/16 20:07:09 might want to add EXPECT_FALSE(app_entry->is_exte
zel 2012/05/16 20:56:14 Done.
+
+ // Check an external document entry.
+ const DocumentEntry* app_entry = feed->entries()[3];
+ ASSERT_TRUE(app_entry);
+ EXPECT_EQ(DocumentEntry::EXTERNAL_APP, app_entry->kind());
+ EXPECT_TRUE(app_entry->is_hosted_document());
+ EXPECT_TRUE(app_entry->is_external_app());
satorux1 2012/05/16 20:07:09 might want to add EXPECT_FALSE(document_entry->is
zel 2012/05/16 20:56:14 Done.
}

Powered by Google App Engine
This is Rietveld 408576698