Native Messaging!

chrome/browser/extensions/api/messaging/native_message_process_host_unittest.cc

Index: chrome/browser/extensions/api/messaging/native_message_process_host_unittest.cc
diff --git a/chrome/browser/extensions/api/messaging/native_message_process_host_unittest.cc b/chrome/browser/extensions/api/messaging/native_message_process_host_unittest.cc
new file mode 100644
index 0000000000000000000000000000000000000000..52c9a3158a1defb464f168eae9aacbb873ece3e9
--- /dev/null
+++ b/chrome/browser/extensions/api/messaging/native_message_process_host_unittest.cc
@@ -0,0 +1,143 @@
+// 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/bind.h"
+#include "base/file_path.h"
+#include "base/file_util.h"
+#include "base/memory/scoped_ptr.h"
+#include "base/memory/weak_ptr.h"
+#include "base/message_loop.h"
+#include "base/path_service.h"
+#include "base/platform_file.h"
+#include "base/threading/sequenced_worker_pool.h"
+#include "chrome/browser/extensions/api/messaging/native_message_process_host.h"
+#include "chrome/browser/extensions/api/messaging/native_process_launcher.h"
+#include "chrome/common/chrome_paths.h"
+#include "content/public/browser/browser_thread.h"
+#include "content/public/test/test_browser_thread.h"
+#include "testing/gtest/include/gtest/gtest.h"
+
+using content::BrowserThread;
+
+namespace {
+
+FilePath GetTestDir() {
+  FilePath test_dir;
+  PathService::Get(chrome::DIR_TEST_DATA, &test_dir);
+  test_dir = test_dir.AppendASCII("native_messaging");
+  return test_dir;
+}
+
+}  // namespace
+
+namespace extensions {
+
+class FakeLauncher : public NativeProcessLauncher {
+ public:
+  FakeLauncher(FilePath read_file, FilePath write_file) {
+    read_file_ = base::CreatePlatformFile(
+        read_file,
+        base::PLATFORM_FILE_OPEN | base::PLATFORM_FILE_READ,
+        NULL, NULL);
+    write_file_ = base::CreatePlatformFile(
+        write_file,
+        base::PLATFORM_FILE_OPEN | base::PLATFORM_FILE_WRITE,
+        NULL, NULL);
+  }
+
+  virtual bool LaunchNativeProcess(
+      const FilePath& path,
+      base::ProcessHandle* native_process_handle,
+      NativeMessageProcessHost::FileHandle* read_file,
+      NativeMessageProcessHost::FileHandle* write_file) const OVERRIDE {
+    *native_process_handle = base::kNullProcessHandle;
+    *read_file = read_file_;
+    *write_file = write_file_;
+    return true;
+  }
+
+ private:
+  base::PlatformFile read_file_;
+  base::PlatformFile write_file_;
+};
+
+class NativeMessagingTest : public ::testing::Test,
+                            public NativeMessageProcessHost::Client,
+                            public base::SupportsWeakPtr<NativeMessagingTest> {
+ public:
+  virtual void SetUp() {
+    // Change the user data dir so native apps will be looked for in the test
+    // directory.
+    ASSERT_TRUE(PathService::Get(chrome::DIR_USER_DATA, &user_data_dir_));
+    ASSERT_TRUE(PathService::Override(chrome::DIR_USER_DATA, GetTestDir()));
+    ui_thread_.reset(new content::TestBrowserThread(BrowserThread::UI,
+                                                    &message_loop_));
+    file_thread_.reset(new content::TestBrowserThread(BrowserThread::FILE,
+                                                      &message_loop_));
+  }
+
+  virtual void TearDown() {
+    // Change the user data dir back for other tests.
+    ASSERT_TRUE(PathService::Override(chrome::DIR_USER_DATA, user_data_dir_));
+    BrowserThread::DeleteSoon(BrowserThread::FILE, FROM_HERE,
+                              native_message_process_host_);
+    message_loop_.RunAllPending();
+  }
+
+  void PostMessageFromNativeProcess(int port_id, const std::string& message) {
+    last_posted_message_ = message;
+  }
+
+  void AcquireProcess(NativeMessageProcessHost* process) {
+    native_message_process_host_ = process;
+  }
+
+ protected:
+  NativeMessageProcessHost* native_message_process_host_;
+  FilePath user_data_dir_;
+  MessageLoopForIO message_loop_;
+  scoped_ptr<content::TestBrowserThread> ui_thread_;
+  scoped_ptr<content::TestBrowserThread> file_thread_;
+  std::string last_posted_message_;
+};
+
+TEST_F(NativeMessagingTest, SingleSendMessageRead) {

[Matt Perry, 2012/08/22 00:02:42]

Can you add comments briefly explaining what these tests are testing?

[eaugusti, 2012/08/31 23:47:13]

Done.

+  FilePath temp_file;
+  file_util::CreateTemporaryFile(&temp_file);
+  FakeLauncher launcher(GetTestDir().AppendASCII("single_message_response.msg"),
+                        temp_file);
+  NativeMessageProcessHost::CreateWithLauncher(
+      AsWeakPtr(), "empty_app", "{}", 0,
+      NativeMessageProcessHost::TYPE_SEND_MESSAGE_REQUEST, base::Bind(
+          &NativeMessagingTest::AcquireProcess, AsWeakPtr()),
+      launcher);
+  message_loop_.RunAllPending();
+  ASSERT_TRUE(native_message_process_host_);
+  native_message_process_host_->ReadNowForTesting();
+  message_loop_.RunAllPending();
+  ASSERT_EQ(last_posted_message_, "{\"text\": \"Hi There!.\"}");
+  file_util::Delete(temp_file, false /* non-recursive */);
+}
+
+TEST_F(NativeMessagingTest, SingleSendMessageWrite) {
+  FilePath temp_file;
+  file_util::CreateTemporaryFile(&temp_file);
+  FakeLauncher launcher(GetTestDir().AppendASCII("single_message_response.msg"),
+                        temp_file);
+  NativeMessageProcessHost::CreateWithLauncher(
+      AsWeakPtr(), "empty_app", "{\"text\": \"Hello.\"}", 0,
+      NativeMessageProcessHost::TYPE_SEND_MESSAGE_REQUEST, base::Bind(
+          &NativeMessagingTest::AcquireProcess, AsWeakPtr()),
+      launcher);
+  message_loop_.RunAllPending();
+  ASSERT_TRUE(native_message_process_host_);
+
+  ASSERT_TRUE(file_util::ContentsEqual(
+      temp_file, GetTestDir().AppendASCII("single_message_request.msg")));
+
+  file_util::Delete(temp_file, false /* non-recursive */);
+}
+
+
+}  // namespace extensions