chromeos: Refactor D-Bus client type enum and stub vs. fake naming

chromeos/dbus/dbus_client_bundle_unittest.cc

Index: chromeos/dbus/dbus_client_bundle_unittest.cc
diff --git a/chromeos/dbus/dbus_client_bundle_unittest.cc b/chromeos/dbus/dbus_client_bundle_unittest.cc
index bfc439b59504e481d359e73b253134942724a139..6d346e730164d1236b3262ae25c38e755c50351a 100644
--- a/chromeos/dbus/dbus_client_bundle_unittest.cc
+++ b/chromeos/dbus/dbus_client_bundle_unittest.cc
@@ -4,35 +4,60 @@
 
 #include "chromeos/dbus/dbus_client_bundle.h"
 
-#include "base/compiler_specific.h"
+#include "chromeos/dbus/dbus_client_types.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 namespace chromeos {
+namespace {
+
+// Makes mask operations more readable.
+DBusClientTypeMask AsClientTypeMask(DBusClientType type) {
+  return static_cast<DBusClientTypeMask>(type);
+}
+
+}  // namespace
+
+// Tests that real and fake clients can be created.
+TEST(DBusClientBundleTest, CreateClientBundle) {
+  const DBusClientTypeMask real_clients =
+      AsClientTypeMask(DBusClientType::CRAS) |
+      AsClientTypeMask(DBusClientType::CROS_DISKS);
+  DBusClientBundle bundle(real_clients);
+
+  // Real clients are real.
+  EXPECT_TRUE(bundle.cras_audio_client());
+  EXPECT_TRUE(bundle.cros_disks_client());
+  EXPECT_TRUE(bundle.IsUsingReal(DBusClientType::CRAS));
+  EXPECT_TRUE(bundle.IsUsingReal(DBusClientType::CROS_DISKS));
+
+  // Fake clients are fake.
+  EXPECT_TRUE(bundle.permission_broker_client());
+  EXPECT_TRUE(bundle.power_manager_client());
+  EXPECT_FALSE(bundle.IsUsingReal(DBusClientType::PERMISSION_BROKER));
+  EXPECT_FALSE(bundle.IsUsingReal(DBusClientType::POWER_MANAGER));
+}
 
 TEST(DBusClientBundleTest, UnstubFlagParser) {
-  EXPECT_EQ(0, DBusClientBundle::ParseUnstubList("foo"));
+  EXPECT_EQ(0, ParseDBusUnstubClientsList("foo"));
 
-  EXPECT_EQ(DBusClientBundle::BLUETOOTH,
-            DBusClientBundle::ParseUnstubList("BLUETOOTH"));
+  EXPECT_EQ(AsClientTypeMask(DBusClientType::BLUETOOTH),
+            ParseDBusUnstubClientsList("BLUETOOTH"));
 
-  EXPECT_EQ(DBusClientBundle::BLUETOOTH,
-            DBusClientBundle::ParseUnstubList("bluetooth"));
+  EXPECT_EQ(AsClientTypeMask(DBusClientType::BLUETOOTH),
+            ParseDBusUnstubClientsList("bluetooth"));
 
-  EXPECT_EQ(
-      DBusClientBundle::CRAS |
-          DBusClientBundle::CROS_DISKS |
-          DBusClientBundle::DEBUG_DAEMON |
-          DBusClientBundle::SHILL,
-      DBusClientBundle::ParseUnstubList(
-            "Cras,Cros_Disks,debug_daemon,Shill"));
+  EXPECT_EQ(AsClientTypeMask(DBusClientType::CRAS) |
+                AsClientTypeMask(DBusClientType::CROS_DISKS) |
+                AsClientTypeMask(DBusClientType::DEBUG_DAEMON) |
+                AsClientTypeMask(DBusClientType::SHILL),
+            ParseDBusUnstubClientsList("Cras,Cros_Disks,debug_daemon,Shill"));
 
   EXPECT_EQ(
-      DBusClientBundle::CRAS |
-          DBusClientBundle::CROS_DISKS |
-          DBusClientBundle::DEBUG_DAEMON |
-          DBusClientBundle::SHILL,
-      DBusClientBundle::ParseUnstubList(
-            "foo,Cras,Cros_Disks,debug_daemon,Shill"));
+      AsClientTypeMask(DBusClientType::CRAS) |
+          AsClientTypeMask(DBusClientType::CROS_DISKS) |
+          AsClientTypeMask(DBusClientType::DEBUG_DAEMON) |
+          AsClientTypeMask(DBusClientType::SHILL),
+      ParseDBusUnstubClientsList("foo,Cras,Cros_Disks,debug_daemon,Shill"));
 }
 
 }  // namespace chromeos