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

Unified Diff: components/wifi_sync/wifi_security_class_unittest.cc

Issue 675993005: components: add wifi_sync component (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@local-master
Patch Set: fix style issues, and unnecessary virtual functions Created 6 years, 2 months 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
Index: components/wifi_sync/wifi_security_class_unittest.cc
diff --git a/components/wifi_sync/wifi_security_class_unittest.cc b/components/wifi_sync/wifi_security_class_unittest.cc
new file mode 100644
index 0000000000000000000000000000000000000000..08785d23900532cec5493fb26016a064c026c615
--- /dev/null
+++ b/components/wifi_sync/wifi_security_class_unittest.cc
@@ -0,0 +1,52 @@
+// 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 "components/wifi_sync/wifi_security_class.h"
+
+#include "testing/gtest/include/gtest/gtest.h"
+#if defined(OS_CHROMEOS)
erikwright (departed) 2014/10/29 18:18:25 same question about file suffix
mukesh agrawal 2014/10/30 18:41:29 Done.
+#include "third_party/cros_system_api/dbus/service_constants.h"
+#endif
+
+namespace wifi_sync {
+
+#if defined(OS_CHROMEOS)
+TEST(WifiSecurityClassTest, FromShillStringToEnum) {
+ EXPECT_EQ(SECURITY_CLASS_NONE,
+ WifiSecurityClassFromShillString(shill::kSecurityNone));
+ EXPECT_EQ(SECURITY_CLASS_WEP,
+ WifiSecurityClassFromShillString(shill::kSecurityWep));
+ EXPECT_EQ(SECURITY_CLASS_PSK,
+ WifiSecurityClassFromShillString(shill::kSecurityPsk));
+ EXPECT_EQ(SECURITY_CLASS_802_1X,
+ WifiSecurityClassFromShillString(shill::kSecurity8021x));
+ EXPECT_EQ(SECURITY_CLASS_INVALID,
+ WifiSecurityClassFromShillString("bogus-security-class"));
+}
+
+TEST(WifiSecurityClassTest, FromEnumToShillString) {
+ std::string security_class_string;
+
+ EXPECT_TRUE(WifiSecurityClassToShillString(SECURITY_CLASS_NONE,
+ &security_class_string));
+ EXPECT_EQ(shill::kSecurityNone, security_class_string);
+
+ EXPECT_TRUE(WifiSecurityClassToShillString(SECURITY_CLASS_WEP,
+ &security_class_string));
+ EXPECT_EQ(shill::kSecurityWep, security_class_string);
+
+ EXPECT_TRUE(WifiSecurityClassToShillString(SECURITY_CLASS_PSK,
+ &security_class_string));
+ EXPECT_EQ(shill::kSecurityPsk, security_class_string);
+
+ EXPECT_TRUE(WifiSecurityClassToShillString(SECURITY_CLASS_802_1X,
+ &security_class_string));
+ EXPECT_EQ(shill::kSecurity8021x, security_class_string);
+
+ EXPECT_FALSE(WifiSecurityClassToShillString(SECURITY_CLASS_INVALID,
+ &security_class_string));
+}
+#endif // OS_CHROMEOS
+
+} // namespace wifi_sync
« components/wifi_sync/wifi_security_class.cc ('K') | « components/wifi_sync/wifi_security_class.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698