Add automated test for chrome.onFileSynced JavsScript event.

chrome/browser/extensions/api/sync_file_system/sync_file_system_apitest.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 "base/file_util.h"
 #include "base/path_service.h"
 #include "base/run_loop.h"
 #include "chrome/browser/extensions/event_names.h"
 #include "chrome/browser/extensions/extension_apitest.h"
 #include "chrome/browser/sync_file_system/local_change_processor.h"
 #include "chrome/browser/sync_file_system/mock_remote_file_sync_service.h"
 #include "chrome/browser/sync_file_system/sync_file_system_service.h"
 #include "chrome/common/chrome_version_info.h"
 #include "chrome/common/extensions/features/feature.h"
 #include "testing/gtest/include/gtest/gtest.h"
+#include "webkit/fileapi/file_system_url.h"
 #include "webkit/fileapi/syncable/sync_status_code.h"
 #include "webkit/quota/quota_manager.h"
 
 using sync_file_system::MockRemoteFileSyncService;
 using sync_file_system::RemoteFileSyncService;
 using sync_file_system::SyncFileSystemServiceFactory;
 using ::testing::_;
 
 namespace chrome {
 
@@ skipping 33 matching lines @@
   int64 real_default_quota_;
 };
 
 ACTION_P(NotifyOkStateAndCallback, mock_remote_service) {
   mock_remote_service->NotifyRemoteServiceStateUpdated(
       sync_file_system::REMOTE_SERVICE_OK, "Test event description.");
   base::MessageLoopProxy::current()->PostTask(
       FROM_HERE, base::Bind(arg1, fileapi::SYNC_STATUS_OK));
 }
 
+ACTION_P(CaptureRemoteSyncObserver, observer_list) {
+  observer_list->AddObserver(arg0);
+}
+
+ACTION_P2(UpdateRemoteChangeQueue, origin, observer_list) {
+  *origin = arg0;
+  FOR_EACH_OBSERVER(
+      RemoteFileSyncService::Observer, *observer_list,
+      OnRemoteChangeQueueUpdated(1));
+}
+
+ACTION_P3(ProcessRemoteChangeAndCallBack,
+          origin, mock_remote_servicem, observer_list) {
+  fileapi::FileSystemURL mockUrl(*origin,

[kinuko, 2013/01/08 06:02:38]

naming: mockUrl -> mock_url (we're in c++!)

[calvinlo, 2013/01/08 08:32:21]

Done.

+                                 fileapi::kFileSystemTypeTest,
+                                 FilePath(FILE_PATH_LITERAL("foo")));
+  FOR_EACH_OBSERVER(
+      RemoteFileSyncService::Observer, *observer_list,
+      OnRemoteChangeQueueUpdated(0));
+  base::MessageLoopProxy::current()->PostTask(
+      FROM_HERE, base::Bind(arg1,
+                            fileapi::SYNC_STATUS_OK,
+                            mockUrl,
+                            fileapi::SYNC_OPERATION_ADDED));
+}
+
 }  // namespace
 
 // TODO(calvinlo): Add Chrome OS support for syncable file system
 // (http://crbug.com/160693)
 #if !defined(OS_CHROMEOS)
 
 IN_PROC_BROWSER_TEST_F(SyncFileSystemApiTest, DeleteFileSystem) {
   ASSERT_TRUE(RunPlatformAppTest("sync_file_system/delete_file_system"))
       << message_;
 }
 
 IN_PROC_BROWSER_TEST_F(SyncFileSystemApiTest, GetUsageAndQuota) {
   ASSERT_TRUE(RunExtensionTest("sync_file_system/get_usage_and_quota"))
       << message_;
 }
 
+IN_PROC_BROWSER_TEST_F(SyncFileSystemApiTest, OnFileSynced) {
+  // Mock a pending remote change to be synced.
+  ObserverList<RemoteFileSyncService::Observer> observer_list;
+  EXPECT_CALL(*mock_remote_service(), AddObserver(_))
+      .WillOnce(CaptureRemoteSyncObserver(&observer_list));

[kinuko, 2013/01/08 06:02:38]

I think you can omit this observer_list part but can use mock_remote_service's
NotifyRemoteChangeQueueUpdated() method in line 76 and line 86.

[calvinlo, 2013/01/08 08:32:21]

Done. Sorry, I didn't know you could do that before.

+  GURL origin;
+  EXPECT_CALL(*mock_remote_service(), RegisterOriginForTrackingChanges(_, _))
+      .WillOnce(UpdateRemoteChangeQueue(&origin, &observer_list));
+  EXPECT_CALL(*mock_remote_service(), ProcessRemoteChange(_, _))
+      .WillOnce(ProcessRemoteChangeAndCallBack(&origin, mock_remote_service(),
+                                               &observer_list));
+  ASSERT_TRUE(RunPlatformAppTest("sync_file_system/on_file_synced"))
+      << message_;
+}
+
 IN_PROC_BROWSER_TEST_F(SyncFileSystemApiTest, OnSyncStateChanged) {
-  EXPECT_CALL(*mock_remote_service(),
-              RegisterOriginForTrackingChanges(_, _))
+  EXPECT_CALL(*mock_remote_service(), RegisterOriginForTrackingChanges(_, _))
       .WillOnce(NotifyOkStateAndCallback(mock_remote_service()));
   ASSERT_TRUE(RunPlatformAppTest("sync_file_system/on_sync_state_changed"))
       << message_;
 }
 
 IN_PROC_BROWSER_TEST_F(SyncFileSystemApiTest, RequestFileSystem) {
   EXPECT_CALL(*mock_remote_service(),
               RegisterOriginForTrackingChanges(_, _)).Times(1);
   ASSERT_TRUE(RunPlatformAppTest("sync_file_system/request_file_system"))
       << message_;
 }
 
 IN_PROC_BROWSER_TEST_F(SyncFileSystemApiTest, WriteFileThenGetUsage) {
   ASSERT_TRUE(RunPlatformAppTest("sync_file_system/write_file_then_get_usage"))
       << message_;
 }
 
 #endif  // !defined(OS_CHROMEOS)
 
 }  // namespace chrome