Credential Manager: Stub out the browser-side IPC handlers.

chrome/browser/password_manager/chrome_password_manager_client_unittest.cc

 // Copyright 2014 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/command_line.h"

[vabr (Chromium), 2014/08/22 14:04:32]

nit: Since you touch this anyway, please move the base/command_line.h #include
to the #include block below.

-
 #include "chrome/browser/password_manager/chrome_password_manager_client.h"
 
+#include "base/strings/string16.h"
+#include "base/strings/utf_string_conversions.h"
 #include "chrome/common/chrome_version_info.h"
 #include "chrome/test/base/chrome_render_view_host_test_harness.h"
 #include "chrome/test/base/testing_profile.h"
 #include "components/autofill/content/common/autofill_messages.h"
 #include "components/password_manager/content/browser/password_manager_internals_service_factory.h"
+#include "components/password_manager/content/common/credential_manager_messages.h"
+#include "components/password_manager/content/common/credential_manager_types.h"
 #include "components/password_manager/core/browser/log_receiver.h"
 #include "components/password_manager/core/browser/password_manager_internals_service.h"
 #include "components/password_manager/core/common/password_manager_switches.h"
 #include "content/public/browser/browser_context.h"
 #include "content/public/browser/web_contents.h"
 #include "content/public/test/mock_render_process_host.h"
 #include "testing/gmock/include/gmock/gmock.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 using content::BrowserContext;
 using content::WebContents;
 
 namespace {
 
 const char kTestText[] = "abcd1234";
+const int kRequestId = 4;
 
 class MockLogReceiver : public password_manager::LogReceiver {
  public:
   MOCK_METHOD1(LogSavePasswordProgress, void(const std::string&));
 };
 
 class TestChromePasswordManagerClient : public ChromePasswordManagerClient {
  public:
   explicit TestChromePasswordManagerClient(content::WebContents* web_contents)
       : ChromePasswordManagerClient(web_contents, NULL),
@@ skipping 220 matching lines @@
 
   // Adding disallow switch should cause sync credential to be filtered.
   CommandLine* command_line = CommandLine::ForCurrentProcess();
   command_line->AppendSwitch(
       password_manager::switches::kDisallowAutofillSyncCredential);
   client.reset(new TestChromePasswordManagerClient(web_contents()));
   client->set_is_sync_account_credential(true);
   NavigateAndCommit(GURL("https://accounts.google.com/Login"));
   EXPECT_TRUE(client->ShouldFilterAutofillResult(form));
 }
+
+TEST_F(ChromePasswordManagerClientTest, CredentialManagerOnNotifyFailedSignIn) {
+  scoped_ptr<TestChromePasswordManagerClient> client(
+      new TestChromePasswordManagerClient(web_contents()));
+
+  password_manager::CredentialInfo info(base::ASCIIToUTF16("id"),
+                                        base::ASCIIToUTF16("name"),
+                                        GURL("https://example.com/image.png"));
+  client->OnNotifyFailedSignIn(kRequestId, info);
+
+  const uint32 kMsgID = CredentialManagerMsg_AcknowledgeFailedSignIn::ID;
+  const IPC::Message* message =
+      process()->sink().GetFirstMessageMatching(kMsgID);
+  EXPECT_TRUE(!!message);

[vabr (Chromium), 2014/08/22 14:04:32]

nit: I'm surprised you need to "!!" this -- pointers used to work as bools for
EXPECT_TRUE. Did some compiler scream at you?

+  process()->sink().ClearMessages();
+}
+
+TEST_F(ChromePasswordManagerClientTest, CredentialManagerOnNotifySignedIn) {
+  scoped_ptr<TestChromePasswordManagerClient> client(
+      new TestChromePasswordManagerClient(web_contents()));
+
+  password_manager::CredentialInfo info(base::ASCIIToUTF16("id"),
+                                        base::ASCIIToUTF16("name"),
+                                        GURL("https://example.com/image.png"));
+  client->OnNotifySignedIn(kRequestId, info);
+
+  const uint32 kMsgID = CredentialManagerMsg_AcknowledgeSignedIn::ID;
+  const IPC::Message* message =
+      process()->sink().GetFirstMessageMatching(kMsgID);
+  EXPECT_TRUE(!!message);
+  process()->sink().ClearMessages();
+}
+
+TEST_F(ChromePasswordManagerClientTest, CredentialManagerOnNotifySignedOut) {
+  scoped_ptr<TestChromePasswordManagerClient> client(
+      new TestChromePasswordManagerClient(web_contents()));
+
+  client->OnNotifySignedOut(kRequestId);
+
+  const uint32 kMsgID = CredentialManagerMsg_AcknowledgeSignedOut::ID;
+  const IPC::Message* message =
+      process()->sink().GetFirstMessageMatching(kMsgID);
+  EXPECT_TRUE(!!message);
+  process()->sink().ClearMessages();
+}
+
+TEST_F(ChromePasswordManagerClientTest, CredentialManagerOnRequestCredential) {
+  scoped_ptr<TestChromePasswordManagerClient> client(
+      new TestChromePasswordManagerClient(web_contents()));
+
+  std::vector<GURL> federations;
+  client->OnRequestCredential(kRequestId, false, federations);
+
+  const uint32 kMsgID = CredentialManagerMsg_SendCredential::ID;
+  const IPC::Message* message =
+      process()->sink().GetFirstMessageMatching(kMsgID);
+  EXPECT_TRUE(!!message);
+  process()->sink().ClearMessages();
+}