drive: Fills no-parent entiries in "/drive/other".

chrome/browser/chromeos/drive/resource_entry_conversion_unittest.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/chromeos/drive/resource_entry_conversion.h"
 
 #include "base/files/file_path.h"
 #include "base/values.h"
 #include "chrome/browser/chromeos/drive/drive.pb.h"
 #include "chrome/browser/chromeos/drive/drive_test_util.h"
@@ skipping 14 matching lines @@
 
   DriveEntryProto entry_proto =
       ConvertResourceEntryToDriveEntryProto(*resource_entry);
 
   EXPECT_EQ("File 1.mp3",  entry_proto.title());
   EXPECT_EQ("File 1.mp3",  entry_proto.base_name());
   EXPECT_EQ("file:2_file_resource_id",  entry_proto.resource_id());
   EXPECT_EQ("https://file_content_url/",  entry_proto.download_url());
   EXPECT_EQ("https://file1_link_self/file:2_file_resource_id",
             entry_proto.edit_url());
-  EXPECT_EQ("",  entry_proto.parent_resource_id());
+  EXPECT_EQ("<other>",  entry_proto.parent_resource_id());

[hidehiko, 2013/04/03 07:55:40]

nit: using util::kDriveOtherDirSpecialResourceId may be better?

[Haruki Sato, 2013/04/04 06:16:06]

Done. Thanks.

 
   EXPECT_FALSE(entry_proto.deleted());
   EXPECT_FALSE(entry_proto.shared_with_me());
 
   base::Time expected_creation_time;
   base::Time expected_modified_time;
 
   {
     // 2011-12-14T00:40:47.330Z
     base::Time::Exploded exploded;
@@ skipping 62 matching lines @@
   DriveEntryProto entry_proto =
       ConvertResourceEntryToDriveEntryProto(*resource_entry);
 
   EXPECT_EQ("Document 1",  entry_proto.title());
   EXPECT_EQ("Document 1.gdoc",  entry_proto.base_name());  // The suffix added.
   EXPECT_EQ(".gdoc", entry_proto.file_specific_info().document_extension());
   EXPECT_EQ("document:5_document_resource_id",  entry_proto.resource_id());
   EXPECT_EQ("https://3_document_content/",  entry_proto.download_url());
   EXPECT_EQ("https://3_document_self_link/document:5_document_resource_id",
             entry_proto.edit_url());
-  EXPECT_EQ("",  entry_proto.parent_resource_id());
+  EXPECT_EQ("<other>",  entry_proto.parent_resource_id());
 
   EXPECT_FALSE(entry_proto.deleted());
   EXPECT_FALSE(entry_proto.shared_with_me());
 
   // 2011-12-12T23:28:52.783Z
   base::Time::Exploded exploded;
   exploded.year = 2011;
   exploded.month = 12;
   exploded.day_of_month = 12;
   exploded.day_of_week = 1;  // Monday
@@ skipping 141 matching lines @@
 
   DriveEntryProto entry_proto =
       ConvertResourceEntryToDriveEntryProto(*resource_entry);
 
   EXPECT_EQ("Deleted document",  entry_proto.title());
   EXPECT_EQ("Deleted document.gdoc",  entry_proto.base_name());
   EXPECT_EQ("document:deleted_in_root_id",  entry_proto.resource_id());
   EXPECT_EQ("https://content_url/",  entry_proto.download_url());
   EXPECT_EQ("https://edit_url/document%3Adeleted_in_root_id",
             entry_proto.edit_url());
-  EXPECT_EQ("",  entry_proto.parent_resource_id());
+  EXPECT_EQ("<other>",  entry_proto.parent_resource_id());
 
   EXPECT_TRUE(entry_proto.deleted());  // The document was deleted.
   EXPECT_FALSE(entry_proto.shared_with_me());
 
   // 2012-04-10T22:50:55.797Z
   base::Time::Exploded exploded;
   exploded.year = 2012;
   exploded.month = 04;
   exploded.day_of_month = 10;
   exploded.day_of_week = 2;  // Tuesday
@@ skipping 59 matching lines @@
       google_apis::ResourceEntry::ExtractAndParse(*value));
   ASSERT_TRUE(resource_entry.get());
 
   DriveEntryProto entry_proto =
       ConvertResourceEntryToDriveEntryProto(*resource_entry);
 
   EXPECT_TRUE(entry_proto.shared_with_me());
 }
 
 }  // namespace drive