ChromeOs Power Emulation Impl

chromeos/dbus/fake_power_manager_client_unittest.cc

Index: chromeos/dbus/fake_power_manager_client_unittest.cc
diff --git a/chromeos/dbus/fake_power_manager_client_unittest.cc b/chromeos/dbus/fake_power_manager_client_unittest.cc
new file mode 100644
index 0000000000000000000000000000000000000000..71b61aeeeb49f9e3245eaebb95b7d9cd4d5b162c
--- /dev/null
+++ b/chromeos/dbus/fake_power_manager_client_unittest.cc
@@ -0,0 +1,130 @@
+// Copyright 2015 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 "chromeos/dbus/fake_power_manager_client.h"
+
+#include "testing/gtest/include/gtest/gtest.h"
+
+namespace chromeos {
+namespace {
+class TestObserver : public PowerManagerClient::Observer {
+ public:
+  TestObserver() : num_power_changed_(0) {}
+  ~TestObserver() override {}
+
+  const power_manager::PowerSupplyProperties& props() const { return props_; }
+  int num_power_changed() const { return num_power_changed_; }

[oshima, 2015/07/10 22:39:41]

new line

[mozartalouis, 2015/07/14 20:01:29]

Done.

+  void ClearProps() { props_.Clear(); }
+
+  void PowerChanged(
+      const power_manager::PowerSupplyProperties& proto) override {
+    props_ = proto;
+    ++num_power_changed_;
+  }
+
+ private:
+  int num_power_changed_;
+  power_manager::PowerSupplyProperties props_;
+
+  DISALLOW_COPY_AND_ASSIGN(TestObserver);
+};
+
+void SetTestProperties(power_manager::PowerSupplyProperties& props) {

[oshima, 2015/07/10 22:39:41]

out parameter should be pointer per style guide.

[mozartalouis, 2015/07/14 20:01:29]

Done.

+  props.set_battery_percent(85);
+  props.set_is_calculating_battery_time(true);
+  props.set_battery_state(
+      power_manager::PowerSupplyProperties_BatteryState_DISCHARGING);
+  props.set_external_power(
+      power_manager::PowerSupplyProperties_ExternalPower_USB);
+}
+
+}  // namespace
+
+TEST(FakePowerManagerClientTest, UpdatePowerPropertiesTest) {
+  // Checking to verify when UpdatePowerProperties is called,
+  // |props_| values are updated.
+  FakePowerManagerClient client;
+  power_manager::PowerSupplyProperties props;
+
+  SetTestProperties(props);
+  client.UpdatePowerProperties(props);
+
+  EXPECT_EQ(85, client.props().battery_percent());
+  EXPECT_TRUE(client.props().is_calculating_battery_time());
+  EXPECT_EQ(power_manager::PowerSupplyProperties_BatteryState_DISCHARGING,
+            client.props().battery_state());
+  EXPECT_EQ(power_manager::PowerSupplyProperties_ExternalPower_USB,
+            client.props().external_power());
+
+  // Test if when the values are changed, the correct data is set in the
+  // FakePowerManagerClient.
+  props = client.props();
+  props.set_battery_percent(70);
+  client.UpdatePowerProperties(props);
+
+  EXPECT_EQ(70, client.props().battery_percent());
+  EXPECT_TRUE(client.props().is_calculating_battery_time());
+  EXPECT_EQ(power_manager::PowerSupplyProperties_BatteryState_DISCHARGING,
+            client.props().battery_state());
+  EXPECT_EQ(power_manager::PowerSupplyProperties_ExternalPower_USB,
+            client.props().external_power());
+};
+
+TEST(FakePowerManagerClientTest, NotifyObserversTest) {
+  FakePowerManagerClient client;
+  power_manager::PowerSupplyProperties props;

[oshima, 2015/07/10 22:39:41]

move this to line 82 just before it's first used.

[mozartalouis, 2015/07/14 20:01:29]

Done.

+  TestObserver test_observer;
+
+  // Test adding observer.
+  client.AddObserver(&test_observer);
+  EXPECT_TRUE(client.HasObserver(&test_observer));
+
+  // Test if NotifyObservers sends the correct values to |observer|.
+  // Check number of times NotifyObservers is called.
+  SetTestProperties(props);
+  client.UpdatePowerProperties(props);
+
+  EXPECT_EQ(85, test_observer.props().battery_percent());
+  EXPECT_TRUE(test_observer.props().is_calculating_battery_time());
+  EXPECT_EQ(power_manager::PowerSupplyProperties_BatteryState_DISCHARGING,
+            test_observer.props().battery_state());
+  EXPECT_EQ(power_manager::PowerSupplyProperties_ExternalPower_USB,
+            test_observer.props().external_power());
+  EXPECT_EQ(1, test_observer.num_power_changed());
+
+  // Test if RequestStatusUpdate will propergate the data to the observer.
+  // Check number of times NotifyObservers is called.
+  test_observer.ClearProps();
+  client.RequestStatusUpdate();
+
+  EXPECT_EQ(85, test_observer.props().battery_percent());
+  EXPECT_TRUE(test_observer.props().is_calculating_battery_time());
+  EXPECT_EQ(power_manager::PowerSupplyProperties_BatteryState_DISCHARGING,
+            test_observer.props().battery_state());
+  EXPECT_EQ(power_manager::PowerSupplyProperties_ExternalPower_USB,
+            test_observer.props().external_power());
+  EXPECT_EQ(2, test_observer.num_power_changed());
+
+  // Check when values are changed, the correct values are propergated to the
+  // Obserever
+  props = client.props();
+  props.set_battery_percent(90);
+  props.set_external_power(
+      power_manager::PowerSupplyProperties_ExternalPower_AC);
+  client.UpdatePowerProperties(props);
+
+  EXPECT_EQ(90, test_observer.props().battery_percent());
+  EXPECT_TRUE(test_observer.props().is_calculating_battery_time());
+  EXPECT_EQ(power_manager::PowerSupplyProperties_BatteryState_DISCHARGING,
+            test_observer.props().battery_state());
+  EXPECT_EQ(power_manager::PowerSupplyProperties_ExternalPower_AC,
+            test_observer.props().external_power());
+  EXPECT_EQ(3, test_observer.num_power_changed());
+
+  // Test removing observer.
+  client.RemoveObserver(&test_observer);
+  EXPECT_FALSE(client.HasObserver(&test_observer));
+};
+
+}  // namespace chromeos