Add support for actions for multiple file selection.

chrome/browser/chromeos/file_system_provider/operations/get_actions_unittest.cc

 // Copyright 2015 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/file_system_provider/operations/get_actions.h"
 
 #include <string>
+#include <vector>
 
 #include "base/files/file.h"
 #include "base/files/file_path.h"
 #include "base/json/json_reader.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/memory/scoped_vector.h"
 #include "base/values.h"
 #include "chrome/browser/chromeos/file_system_provider/operations/get_metadata.h"
 #include "chrome/browser/chromeos/file_system_provider/operations/test_util.h"
 #include "chrome/common/extensions/api/file_system_provider.h"
 #include "chrome/common/extensions/api/file_system_provider_capabilities/file_system_provider_capabilities_handler.h"
 #include "chrome/common/extensions/api/file_system_provider_internal.h"
 #include "extensions/browser/event_router.h"
 #include "storage/browser/fileapi/async_file_util.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 namespace chromeos {
 namespace file_system_provider {
 namespace operations {
 namespace {
 
 const char kExtensionId[] = "mbflcebpggnecokmikipoihdbecnjfoj";
 const char kFileSystemId[] = "testing-file-system";
 const int kRequestId = 2;
-const base::FilePath::CharType kEntryPath[] = FILE_PATH_LITERAL("/directory");
+const base::FilePath::CharType kDirectoryPath[] =
+    FILE_PATH_LITERAL("/directory");
+const base::FilePath::CharType kFilePath[] = FILE_PATH_LITERAL("/file");
 
 // Callback invocation logger. Acts as a fileapi end-point.
 class CallbackLogger {
  public:
   class Event {
    public:
     Event(const Actions& actions, base::File::Error result)
         : actions_(actions), result_(result) {}
     virtual ~Event() {}
 
@@ skipping 47 matching lines @@
 class FileSystemProviderOperationsGetActionsTest : public testing::Test {
  protected:
   FileSystemProviderOperationsGetActionsTest() {}
   ~FileSystemProviderOperationsGetActionsTest() override {}
 
   void SetUp() override {
     file_system_info_ = ProvidedFileSystemInfo(
         kExtensionId, MountOptions(kFileSystemId, "" /* display_name */),
         base::FilePath(), false /* configurable */, true /* watchable */,
         extensions::SOURCE_FILE);
+    entry_paths_.clear();
+    entry_paths_.push_back(base::FilePath(kDirectoryPath));
+    entry_paths_.push_back(base::FilePath(kFilePath));
   }
 
   ProvidedFileSystemInfo file_system_info_;
+  std::vector<base::FilePath> entry_paths_;
 };
 
 TEST_F(FileSystemProviderOperationsGetActionsTest, Execute) {
   using extensions::api::file_system_provider::GetActionsRequestedOptions;
 
   util::LoggingDispatchEventImpl dispatcher(true /* dispatch_reply */);
   CallbackLogger callback_logger;
 
-  GetActions get_actions(NULL, file_system_info_, base::FilePath(kEntryPath),
+  GetActions get_actions(NULL, file_system_info_, entry_paths_,
                          base::Bind(&CallbackLogger::OnGetActions,
                                     base::Unretained(&callback_logger)));
   get_actions.SetDispatchEventImplForTesting(
       base::Bind(&util::LoggingDispatchEventImpl::OnDispatchEventImpl,
                  base::Unretained(&dispatcher)));
 
   EXPECT_TRUE(get_actions.Execute(kRequestId));
 
   ASSERT_EQ(1u, dispatcher.events().size());
   extensions::Event* event = dispatcher.events()[0];
   EXPECT_EQ(
       extensions::api::file_system_provider::OnGetActionsRequested::kEventName,
       event->event_name);
   base::ListValue* event_args = event->event_args.get();
   ASSERT_EQ(1u, event_args->GetSize());
 
   const base::DictionaryValue* options_as_value = NULL;
   ASSERT_TRUE(event_args->GetDictionary(0, &options_as_value));
 
   GetActionsRequestedOptions options;
   ASSERT_TRUE(
       GetActionsRequestedOptions::Populate(*options_as_value, &options));
   EXPECT_EQ(kFileSystemId, options.file_system_id);
   EXPECT_EQ(kRequestId, options.request_id);
-  EXPECT_EQ(kEntryPath, options.entry_path);
+  ASSERT_EQ(entry_paths_.size(), options.entry_paths.size());
+  EXPECT_EQ(entry_paths_[0].value(), options.entry_paths[0]);
+  EXPECT_EQ(entry_paths_[1].value(), options.entry_paths[1]);
 }
 
 TEST_F(FileSystemProviderOperationsGetActionsTest, Execute_NoListener) {
   util::LoggingDispatchEventImpl dispatcher(false /* dispatch_reply */);
   CallbackLogger callback_logger;
 
-  GetActions get_actions(NULL, file_system_info_, base::FilePath(kEntryPath),
+  GetActions get_actions(NULL, file_system_info_, entry_paths_,
                          base::Bind(&CallbackLogger::OnGetActions,
                                     base::Unretained(&callback_logger)));
   get_actions.SetDispatchEventImplForTesting(
       base::Bind(&util::LoggingDispatchEventImpl::OnDispatchEventImpl,
                  base::Unretained(&dispatcher)));
 
   EXPECT_FALSE(get_actions.Execute(kRequestId));
 }
 
 TEST_F(FileSystemProviderOperationsGetActionsTest, OnSuccess) {
   util::LoggingDispatchEventImpl dispatcher(true /* dispatch_reply */);
   CallbackLogger callback_logger;
 
-  GetActions get_actions(NULL, file_system_info_, base::FilePath(kEntryPath),
+  GetActions get_actions(NULL, file_system_info_, entry_paths_,
                          base::Bind(&CallbackLogger::OnGetActions,
                                     base::Unretained(&callback_logger)));
   get_actions.SetDispatchEventImplForTesting(
       base::Bind(&util::LoggingDispatchEventImpl::OnDispatchEventImpl,
                  base::Unretained(&dispatcher)));
 
   EXPECT_TRUE(get_actions.Execute(kRequestId));
 
   // Sample input as JSON. Keep in sync with file_system_provider_api.idl.
   // As for now, it is impossible to create *::Params class directly, not from
@@ skipping 38 matching lines @@
 
   const Action action_custom = event->actions()[2];
   EXPECT_EQ("SomeCustomActionId", action_custom.id);
   EXPECT_EQ("Custom action.", action_custom.title);
 }
 
 TEST_F(FileSystemProviderOperationsGetActionsTest, OnError) {
   util::LoggingDispatchEventImpl dispatcher(true /* dispatch_reply */);
   CallbackLogger callback_logger;
 
-  GetActions get_actions(NULL, file_system_info_, base::FilePath(kEntryPath),
+  GetActions get_actions(NULL, file_system_info_, entry_paths_,
                          base::Bind(&CallbackLogger::OnGetActions,
                                     base::Unretained(&callback_logger)));
   get_actions.SetDispatchEventImplForTesting(
       base::Bind(&util::LoggingDispatchEventImpl::OnDispatchEventImpl,
                  base::Unretained(&dispatcher)));
 
   EXPECT_TRUE(get_actions.Execute(kRequestId));
 
   get_actions.OnError(kRequestId, scoped_ptr<RequestValue>(new RequestValue()),
                       base::File::FILE_ERROR_TOO_MANY_OPENED);
 
   ASSERT_EQ(1u, callback_logger.events().size());
   CallbackLogger::Event* event = callback_logger.events()[0];
   EXPECT_EQ(base::File::FILE_ERROR_TOO_MANY_OPENED, event->result());
   ASSERT_EQ(0u, event->actions().size());
 }
 
 }  // namespace operations
 }  // namespace file_system_provider
 }  // namespace chromeos