[Sync] Propagate XMPP auth errors to SyncNotifierObservers

jingle/notifier/communicator/single_login_attempt_unittest.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 "jingle/notifier/communicator/single_login_attempt.h"
 
 #include <cstddef>
 
 #include "base/compiler_specific.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/message_loop.h"
 #include "jingle/notifier/base/const_communicator.h"
 #include "jingle/notifier/base/fake_base_task.h"
 #include "jingle/notifier/communicator/login_settings.h"
 #include "net/base/mock_host_resolver.h"
 #include "net/url_request/url_request_test_util.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "talk/xmllite/xmlelement.h"
 #include "talk/xmpp/constants.h"
 #include "talk/xmpp/xmppengine.h"
 
 namespace buzz {
 class XmppTaskParentInterface;
 }  // namespace buzz
 
 namespace notifier {
 
 namespace {
 
-enum DelegateState { IDLE, CONNECTED, NEED_RECONNECT, REDIRECTED };
+enum DelegateState {
+  IDLE, CONNECTED, NEED_RECONNECT, REDIRECTED, REJECTED_CREDENTIALS
+};
 
 class FakeDelegate : public SingleLoginAttempt::Delegate {
  public:
   FakeDelegate() : state_(IDLE) {}
 
-  void OnConnect(base::WeakPtr<buzz::XmppTaskParentInterface> base_task) {
+  void OnConnect(
+      base::WeakPtr<buzz::XmppTaskParentInterface> base_task) OVERRIDE {
     state_ = CONNECTED;
     base_task_ = base_task;
   }
 
-  virtual void OnNeedReconnect() {
+  virtual void OnNeedReconnect() OVERRIDE {
     state_ = NEED_RECONNECT;
   }
 
   virtual void OnRedirect(const ServerInformation& redirect_server) OVERRIDE {
     state_ = REDIRECTED;
     redirect_server_ = redirect_server;
   }
 
+  virtual void OnRejectedCredentials() OVERRIDE {
+    state_ = REJECTED_CREDENTIALS;
+  }
+
   DelegateState state() const { return state_; }
 
   base::WeakPtr<buzz::XmppTaskParentInterface> base_task() const {
     return base_task_;
   }
 
   const ServerInformation& redirect_server() const {
     return redirect_server_;
   }
 
@@ skipping 166 matching lines @@
 
 // Fire a redirect with a missing see-other-host element and make sure
 // the delegate does not get a redirect.
 TEST_F(SingleLoginAttemptTest, RedirectMissingSeeOtherHost) {
   scoped_ptr<buzz::XmlElement> redirect_error(MakeRedirectError(""));
   redirect_error->RemoveChildAfter(NULL);
   FireRedirect(redirect_error.get());
   EXPECT_EQ(IDLE, fake_delegate_.state());
 }
 
+// Fire 'Unauthorized' errors and make sure the delegate gets the
+// OnRejectedCredentials() event.
+TEST_F(SingleLoginAttemptTest, RejectedCredentials) {
+  attempt_.OnError(buzz::XmppEngine::ERROR_UNAUTHORIZED, 0, NULL);
+  EXPECT_EQ(REJECTED_CREDENTIALS, fake_delegate_.state());
+}
+
 }  // namespace
 
 }  // namespace notifier