Adding XMPP ping functionality to CLoudPrint. XMPP ping and timeout is controlled thorugh Service S…

jingle/notifier/listener/send_ping_task_unittest.cc

Index: jingle/notifier/listener/send_ping_task_unittest.cc
diff --git a/jingle/notifier/listener/send_ping_task_unittest.cc b/jingle/notifier/listener/send_ping_task_unittest.cc
new file mode 100644
index 0000000000000000000000000000000000000000..1a2dd4bae83e8be446020266b6809236dbeba1a6
--- /dev/null
+++ b/jingle/notifier/listener/send_ping_task_unittest.cc
@@ -0,0 +1,41 @@
+// 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/listener/send_ping_task.h"
+
+#include "base/base64.h"
+#include "base/memory/scoped_ptr.h"
+#include "jingle/notifier/listener/xml_element_util.h"
+#include "talk/xmpp/jid.h"
+#include "testing/gtest/include/gtest/gtest.h"
+
+namespace buzz {
+class XmlElement;
+}
+
+namespace notifier {
+
+class SendPingTaskTest : public testing::Test {
+ public:
+  SendPingTaskTest() {}
+
+ private:
+  DISALLOW_COPY_AND_ASSIGN(SendPingTaskTest);
+};
+
+TEST_F(SendPingTaskTest, MakePingStanza) {
+  std::string task_id = "42";
+
+  scoped_ptr<buzz::XmlElement> message(SendPingTask::MakePingStanza(task_id));
+
+  std::string expected_xml_string("<cli:iq type=\"get\" id=\"");
+  expected_xml_string += task_id;
+  expected_xml_string +=
+      "\" xmlns:cli=\"jabber:client\">"
+      "<ping:ping xmlns:ping=\"urn:xmpp:ping\"/></cli:iq>";
+
+  EXPECT_EQ(expected_xml_string, XmlElementToString(*message));
+}
+

[akalin, 2012/10/25 00:51:10]

still writing the test for ProcessStanza / OnResponse?

[gene, 2012/10/25 01:06:22]

Sorry, no.
I was in the process when upload, but I think it is too much overhead. I think
PushNotificationSendUpdateTask only tests static member, while
PushNotificationsListenTask lack them for the same reason.

+}  // namespace notifier