Remote service state handling

chrome/browser/sync_file_system/drive_file_sync_service_unittest.cc

Index: chrome/browser/sync_file_system/drive_file_sync_service_unittest.cc
diff --git a/chrome/browser/sync_file_system/drive_file_sync_service_unittest.cc b/chrome/browser/sync_file_system/drive_file_sync_service_unittest.cc
index 8d80c87b24d8a0d167afa8ebc06d69e206136434..782a5dec7ea1b02b584d75cc1061f9f604289f2c 100644
--- a/chrome/browser/sync_file_system/drive_file_sync_service_unittest.cc
+++ b/chrome/browser/sync_file_system/drive_file_sync_service_unittest.cc
@@ -20,6 +20,7 @@
 
 using ::testing::StrictMock;
 using ::testing::InSequence;
+using ::testing::AtLeast;

[tzik, 2012/11/16 05:21:51]

sort?

[kinuko, 2012/11/16 05:45:02]

Done.

 using ::testing::_;
 
 using google_apis::DriveServiceInterface;
@@ -53,6 +54,19 @@ void ExpectEqStatus(bool* done,
 
 }  // namespace
 
+class MockRemoteServiceObserver : public RemoteFileSyncService::Observer {
+ public:
+  MockRemoteServiceObserver() {}
+  virtual ~MockRemoteServiceObserver() {}
+
+  // LocalChangeProcessor override.
+  MOCK_METHOD1(OnRemoteChangeAvailable,
+               void(int64 pending_changes));
+  MOCK_METHOD2(OnRemoteServiceStateUpdated,
+               void(RemoteServiceState state,
+                    const std::string& description));
+};
+
 class DriveFileSyncServiceTest : public testing::Test {
  public:
   DriveFileSyncServiceTest()
@@ -87,6 +101,7 @@ class DriveFileSyncServiceTest : public testing::Test {
   void SetUpDriveSyncService() {
     sync_service_ = DriveFileSyncService::CreateForTesting(
         sync_client_.Pass(), metadata_store_.Pass()).Pass();
+    sync_service_->AddObserver(&mock_remote_observer_);
   }
 
   virtual void TearDown() OVERRIDE {
@@ -125,6 +140,10 @@ class DriveFileSyncServiceTest : public testing::Test {
     return mock_drive_uploader_;
   }
 
+  StrictMock<MockRemoteServiceObserver>* mock_remote_observer() {
+    return &mock_remote_observer_;
+  }
+
   MessageLoop* message_loop() { return &message_loop_; }
   DriveFileSyncService* sync_service() { return sync_service_.get(); }
 
@@ -147,6 +166,7 @@ class DriveFileSyncServiceTest : public testing::Test {
   // Owned by |sync_client_|.
   StrictMock<google_apis::MockDriveService>* mock_drive_service_;
   StrictMock<google_apis::MockDriveUploader>* mock_drive_uploader_;
+  StrictMock<MockRemoteServiceObserver> mock_remote_observer_;
 
   scoped_ptr<DriveFileSyncClient> sync_client_;
   scoped_ptr<DriveMetadataStore> metadata_store_;
@@ -198,6 +218,10 @@ TEST_F(DriveFileSyncServiceTest, GetSyncRoot) {
           google_apis::HTTP_SUCCESS,
           base::Passed(&sync_root_found)));
 
+  EXPECT_CALL(*mock_remote_observer(),
+              OnRemoteServiceStateUpdated(REMOTE_SERVICE_OK, _))
+      .Times(1);
+
   SetUpDriveSyncService();
   message_loop()->RunUntilIdle();
 
@@ -236,11 +260,16 @@ TEST_F(DriveFileSyncServiceTest, BatchSyncOnInitialization) {
   scoped_ptr<Value> listing_files_in_directory(LoadJSONFile(
       "sync_file_system/listing_files_in_directory.json"));
   EXPECT_CALL(*mock_drive_service(),
-              GetDocuments(GURL(), 0, std::string(), kDirectoryResourceId1, _))
+              GetDocuments(GURL(), 0, std::string(),
+                           kDirectoryResourceId1, _))
       .WillOnce(InvokeGetDataCallback4(
           google_apis::HTTP_SUCCESS,
           base::Passed(&listing_files_in_directory)));
 
+  EXPECT_CALL(*mock_remote_observer(),
+              OnRemoteServiceStateUpdated(REMOTE_SERVICE_OK, _))
+      .Times(1);
+
   SetUpDriveSyncService();
   message_loop()->RunUntilIdle();
 
@@ -261,6 +290,10 @@ TEST_F(DriveFileSyncServiceTest, RegisterNewOrigin) {
 
   metadata_store()->SetSyncRootDirectory(kSyncRootResourceId);
 
+  EXPECT_CALL(*mock_remote_observer(),
+              OnRemoteServiceStateUpdated(REMOTE_SERVICE_OK, _))
+      .Times(AtLeast(1));
+
   InSequence sequence;
 
   scoped_ptr<Value> origin_directory_not_found(LoadJSONFile(
@@ -334,6 +367,10 @@ TEST_F(DriveFileSyncServiceTest, RegisterExistingOrigin) {
 
   metadata_store()->SetSyncRootDirectory(kSyncRootResourceId);
 
+  EXPECT_CALL(*mock_remote_observer(),
+              OnRemoteServiceStateUpdated(REMOTE_SERVICE_OK, _))
+      .Times(AtLeast(1));
+
   InSequence sequence;
 
   scoped_ptr<Value> origin_directory_found(LoadJSONFile(
@@ -392,6 +429,10 @@ TEST_F(DriveFileSyncServiceTest, UnregisterOrigin) {
   metadata_store()->AddBatchSyncOrigin(kOrigin2, kDirectoryResourceId2);
   metadata_store()->MoveBatchSyncOriginToIncremental(kOrigin2);
 
+  EXPECT_CALL(*mock_remote_observer(),
+              OnRemoteServiceStateUpdated(REMOTE_SERVICE_OK, _))
+      .Times(AtLeast(1));
+
   InSequence sequence;
 
   scoped_ptr<Value> account_metadata(LoadJSONFile(