Parse TeamDrive resource inside ChangeList.

google_apis/drive/drive_api_parser_unittest.cc

Index: google_apis/drive/drive_api_parser_unittest.cc
diff --git a/google_apis/drive/drive_api_parser_unittest.cc b/google_apis/drive/drive_api_parser_unittest.cc
index 77579bc2b68fcf5fd642b0190409d0f2f083aa2a..3b14e0b3b3a37de2f7b2a3961c21591d14559812 100644
--- a/google_apis/drive/drive_api_parser_unittest.cc
+++ b/google_apis/drive/drive_api_parser_unittest.cc
@@ -252,10 +252,11 @@ TEST(DriveAPIParserTest, ChangeListParser) {
             changelist->next_link().spec());
   EXPECT_EQ(13664, changelist->largest_change_id());
 
-  ASSERT_EQ(4U, changelist->items().size());
+  ASSERT_EQ(5U, changelist->items().size());
 
   const ChangeResource& change1 = *changelist->items()[0];
   EXPECT_EQ(8421, change1.change_id());
+  EXPECT_EQ(ChangeResource::FILE, change1.type());
   EXPECT_FALSE(change1.is_deleted());
   EXPECT_EQ("1Pc8jzfU1ErbN_eucMMqdqzY3eBm0v8sxXm_1CtLxABC", change1.file_id());
   EXPECT_EQ(change1.file_id(), change1.file()->file_id());
@@ -264,6 +265,7 @@ TEST(DriveAPIParserTest, ChangeListParser) {
 
   const ChangeResource& change2 = *changelist->items()[1];
   EXPECT_EQ(8424, change2.change_id());
+  EXPECT_EQ(ChangeResource::FILE, change2.type());
   EXPECT_FALSE(change2.is_deleted());
   EXPECT_EQ("0B4v7G8yEYAWHUmRrU2lMS2hLABC", change2.file_id());
   EXPECT_EQ(change2.file_id(), change2.file()->file_id());
@@ -272,6 +274,7 @@ TEST(DriveAPIParserTest, ChangeListParser) {
 
   const ChangeResource& change3 = *changelist->items()[2];
   EXPECT_EQ(8429, change3.change_id());
+  EXPECT_EQ(ChangeResource::FILE, change3.type());
   EXPECT_FALSE(change3.is_deleted());
   EXPECT_EQ("0B4v7G8yEYAWHYW1OcExsUVZLABC", change3.file_id());
   EXPECT_EQ(change3.file_id(), change3.file()->file_id());
@@ -281,12 +284,39 @@ TEST(DriveAPIParserTest, ChangeListParser) {
   // Deleted entry.
   const ChangeResource& change4 = *changelist->items()[3];
   EXPECT_EQ(8430, change4.change_id());
+  EXPECT_EQ(ChangeResource::FILE, change4.type());
   EXPECT_EQ("ABCv7G8yEYAWHc3Y5X0hMSkJYXYZ", change4.file_id());
   EXPECT_TRUE(change4.is_deleted());
   base::Time modification_time;
   ASSERT_TRUE(util::GetTimeFromString("2012-07-27T12:34:56.789Z",
                                       &modification_time));
   EXPECT_EQ(modification_time, change4.modification_date());
+
+  // Team Drive entry.
+  const ChangeResource& change5 = *changelist->items()[4];
+  EXPECT_EQ(8431, change5.change_id());
+  EXPECT_EQ(ChangeResource::TEAM_DRIVE, change5.type());
+  EXPECT_EQ("id-of-team-drive-test-data", change5.team_drive()->id());
+  EXPECT_EQ("id-of-team-drive-test-data", change5.team_drive_id());
+  EXPECT_FALSE(change5.is_deleted());
+  ASSERT_TRUE(
+      util::GetTimeFromString("2017-07-27T12:34:56.789Z", &modification_time));
+  EXPECT_EQ(modification_time, change5.modification_date());
+  const TeamDriveCapabilities& capabilities =

[hashimoto, 2017/03/22 06:40:30]

Lines after this look redundant as DriveAPIParserTest.TeamDriveResourceParser is
already testing the same thing.

[yamaguchi, 2017/03/22 07:30:36]

Done.

+      change5.team_drive()->capabilities();
+  EXPECT_TRUE(capabilities.can_add_children());
+  EXPECT_TRUE(capabilities.can_comment());
+  EXPECT_TRUE(capabilities.can_copy());
+  EXPECT_TRUE(capabilities.can_delete_team_drive());
+  EXPECT_TRUE(capabilities.can_download());
+  EXPECT_TRUE(capabilities.can_edit());
+  EXPECT_TRUE(capabilities.can_list_children());
+  EXPECT_TRUE(capabilities.can_manage_members());
+  EXPECT_TRUE(capabilities.can_read_revisions());
+  EXPECT_TRUE(capabilities.can_remove_children());
+  EXPECT_TRUE(capabilities.can_rename());
+  EXPECT_TRUE(capabilities.can_rename_team_drive());
+  EXPECT_TRUE(capabilities.can_share());
 }
 
 TEST(DriveAPIParserTest, HasKind) {