Add SearchOperationTest for drive::file_system::SearchOperation.

chrome/browser/chromeos/drive/file_system/search_operation_unittest.cc

Index: chrome/browser/chromeos/drive/file_system/search_operation_unittest.cc
diff --git a/chrome/browser/chromeos/drive/file_system/search_operation_unittest.cc b/chrome/browser/chromeos/drive/file_system/search_operation_unittest.cc
new file mode 100644
index 0000000000000000000000000000000000000000..8dc508418758dd61f153a47316986912b9cf4c0e
--- /dev/null
+++ b/chrome/browser/chromeos/drive/file_system/search_operation_unittest.cc
@@ -0,0 +1,114 @@
+// Copyright 2013 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/file_system/search_operation.h"
+
+#include "chrome/browser/chromeos/drive/file_system/operation_test_base.h"
+#include "chrome/browser/google_apis/fake_drive_service.h"
+#include "chrome/browser/google_apis/gdata_wapi_parser.h"
+#include "chrome/browser/google_apis/test_util.h"
+#include "testing/gtest/include/gtest/gtest.h"
+
+namespace drive {
+namespace file_system {
+
+namespace {
+
+struct SearchResultPair {
+  const char* path;
+  const bool is_directory;
+};
+
+}  // namespace
+
+typedef OperationTestBase SearchOperationTest;
+
+TEST_F(SearchOperationTest, ContentSearch) {
+  SearchOperation operation(blocking_task_runner(), scheduler(), metadata());
+
+  const SearchResultPair kExpectedResults[] = {
+    { "drive/root/Directory 1/Sub Directory Folder/Sub Sub Directory Folder",
+      true },
+    { "drive/root/Directory 1/Sub Directory Folder", true },
+    { "drive/root/Directory 1/SubDirectory File 1.txt", false },
+    { "drive/root/Directory 1", true },
+    { "drive/root/Directory 2 excludeDir-test", true },
+  };
+
+  FileError error = FILE_ERROR_FAILED;
+  GURL next_feed;
+  scoped_ptr<std::vector<SearchResultInfo> > results;
+
+  operation.Search("Directory", GURL(),
+                   google_apis::test_util::CreateCopyResultCallback(
+                       &error, &next_feed, &results));
+  google_apis::test_util::RunBlockingPoolTask();
+
+  EXPECT_EQ(FILE_ERROR_OK, error);
+  EXPECT_EQ(GURL(), next_feed);
+  EXPECT_EQ(ARRAYSIZE_UNSAFE(kExpectedResults), results->size());
+  for (size_t i = 0; i < results->size(); i++) {
+    EXPECT_EQ(base::FilePath(kExpectedResults[i].path),

[hashimoto, 2013/05/27 03:15:51]

Shouldn't this be FromUTF8Unsafe()?
Also, how about comparing paths as string, instead of FilePath?
gtest is not good at printing FilePath.

[kinaba, 2013/05/27 03:35:17]

Sounds good. Done.

+              results->at(i).path);
+    EXPECT_EQ(kExpectedResults[i].is_directory,
+              results->at(i).entry.file_info().is_directory());
+  }
+}
+
+TEST_F(SearchOperationTest, ContentSearchWithNewEntry) {
+  SearchOperation operation(blocking_task_runner(), scheduler(), metadata());
+
+  // Create a new directory in the drive service.
+  google_apis::GDataErrorCode gdata_error = google_apis::GDATA_OTHER_ERROR;
+  scoped_ptr<google_apis::ResourceEntry> resource_entry;
+  fake_service()->AddNewDirectory(
+      fake_service()->GetRootResourceId(),
+      "New Directory 1!",
+      google_apis::test_util::CreateCopyResultCallback(
+          &gdata_error, &resource_entry));
+  google_apis::test_util::RunBlockingPoolTask();
+  ASSERT_EQ(google_apis::HTTP_CREATED, gdata_error);
+
+  // As the result of the first Search(), only entries in the current file
+  // system snapshot are expected to be returned (i.e. "New Directory 1!"
+  // shouldn't be included in the search result even though it matches
+  // "Directory 1".
+  const SearchResultPair kExpectedResults[] = {
+      { "drive/root/Directory 1", true }
+  };
+
+  FileError error = FILE_ERROR_FAILED;
+  GURL next_feed;
+  scoped_ptr<std::vector<SearchResultInfo> > results;
+
+  operation.Search("\"Directory 1\"", GURL(),
+                   google_apis::test_util::CreateCopyResultCallback(
+                       &error, &next_feed, &results));
+  google_apis::test_util::RunBlockingPoolTask();
+
+  EXPECT_EQ(FILE_ERROR_OK, error);
+  EXPECT_EQ(GURL(), next_feed);
+  ASSERT_EQ(1U, results->size());
+  EXPECT_EQ(base::FilePath(kExpectedResults[0].path), results->at(0).path);

[hashimoto, 2013/05/27 03:15:51]

ditto.

[kinaba, 2013/05/27 03:35:17]

Done.

+}
+
+TEST_F(SearchOperationTest, ContentSearchEmptyResult) {
+  SearchOperation operation(blocking_task_runner(), scheduler(), metadata());
+
+  FileError error = FILE_ERROR_FAILED;
+  GURL next_feed;
+  scoped_ptr<std::vector<SearchResultInfo> > results;
+
+  operation.Search("\"no-match query\"", GURL(),
+                   google_apis::test_util::CreateCopyResultCallback(
+                       &error, &next_feed, &results));
+  google_apis::test_util::RunBlockingPoolTask();
+
+  EXPECT_EQ(FILE_ERROR_OK, error);
+  EXPECT_EQ(GURL(), next_feed);
+  EXPECT_EQ(0U, results->size());
+}
+
+}  // namespace file_system
+}  // namespace drive