Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1050)

Unified Diff: remoting/host/server_log_entry_unittest.cc

Issue 8468015: The host sends simple log entries to the server. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Review. Created 9 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « remoting/host/server_log_entry.cc ('k') | remoting/host/simple_host_process.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/host/server_log_entry_unittest.cc
diff --git a/remoting/host/server_log_entry_unittest.cc b/remoting/host/server_log_entry_unittest.cc
new file mode 100644
index 0000000000000000000000000000000000000000..11bf8634e071f884371930c6be77cdc7e10e0615
--- /dev/null
+++ b/remoting/host/server_log_entry_unittest.cc
@@ -0,0 +1,105 @@
+// Copyright (c) 2011 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/memory/scoped_ptr.h"
+#include "remoting/host/server_log_entry.h"
+#include "testing/gtest/include/gtest/gtest.h"
+#include "third_party/libjingle/source/talk/xmllite/xmlelement.h"
+
+using buzz::XmlAttr;
+using buzz::XmlElement;
+
+namespace remoting {
+
+class ServerLogEntryTest : public testing::Test {
+ protected:
+ // Verifies a logging stanza.
+ // |keyValuePairs| lists the keys that must have specified values, and |keys|
+ // lists the keys that must be present, but may have arbitrary values.
+ // There must be no other keys.
+ static bool VerifyStanza(
+ const std::map<std::string, std::string>& key_value_pairs,
+ const std::set<std::string> keys,
+ const XmlElement* elem,
+ std::string* error) {
+ int attrCount = 0;
+ for (const XmlAttr* attr = elem->FirstAttr(); attr != NULL;
+ attr = attr->NextAttr(), attrCount++) {
+ if (attr->Name().Namespace().length() != 0) {
+ *error = "attribute has non-empty namespace " +
+ attr->Name().Namespace();
+ return false;
+ }
+ const std::string& key = attr->Name().LocalPart();
+ const std::string& value = attr->Value();
+ std::map<std::string, std::string>::const_iterator iter =
+ key_value_pairs.find(key);
+ if (iter == key_value_pairs.end()) {
+ if (keys.find(key) == keys.end()) {
+ *error = "unexpected attribute " + key;
+ return false;
+ }
+ } else {
+ if (iter->second != value) {
+ *error = "attribute " + key + " has value " + iter->second +
+ ": expected " + value;
+ return false;
+ }
+ }
+ }
+ int attr_count_expected = key_value_pairs.size() + keys.size();
+ if (attrCount != attr_count_expected) {
+ std::stringstream s;
+ s << "stanza has " << attrCount << " keys: expected "
+ << attr_count_expected;
+ *error = s.str();
+ return false;
+ }
+ return true;
+ }
+};
+
+TEST_F(ServerLogEntryTest, MakeSessionStateChange) {
+ scoped_ptr<ServerLogEntry> entry(
+ ServerLogEntry::MakeSessionStateChange(true));
+ scoped_ptr<XmlElement> stanza(entry->ToStanza());
+ std::string error;
+ std::map<std::string, std::string> key_value_pairs;
+ key_value_pairs["role"] = "host";
+ key_value_pairs["event-name"] = "session-state";
+ key_value_pairs["session-state"] = "connected";
+ std::set<std::string> keys;
+ ASSERT_TRUE(VerifyStanza(key_value_pairs, keys, stanza.get(), &error)) <<
+ error;
+}
+
+TEST_F(ServerLogEntryTest, AddHostFields) {
+ scoped_ptr<ServerLogEntry> entry(
+ ServerLogEntry::MakeSessionStateChange(true));
+ entry->AddHostFields();
+ scoped_ptr<XmlElement> stanza(entry->ToStanza());
+ std::string error;
+ std::map<std::string, std::string> key_value_pairs;
+ key_value_pairs["role"] = "host";
+ key_value_pairs["event-name"] = "session-state";
+ key_value_pairs["session-state"] = "connected";
+ std::set<std::string> keys;
+ keys.insert("cpu");
+#if defined(OS_WIN)
+ key_value_pairs["os-name"] = "Windows";
+ keys.insert("os-version");
+#elif defined(OS_MACOSX)
+ key_value_pairs["os-name"] = "Mac";
+ keys.insert("os-version");
+#elif defined(OS_CHROMEOS)
+ key_value_pairs["os-name"] = "ChromeOS";
+ keys.insert("os-version");
+#elif defined(OS_LINUX)
+ key_value_pairs["os-name"] = "Linux";
+#endif
+ ASSERT_TRUE(VerifyStanza(key_value_pairs, keys, stanza.get(), &error)) <<
+ error;
+}
+
+} // namespace remoting
« no previous file with comments | « remoting/host/server_log_entry.cc ('k') | remoting/host/simple_host_process.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698