[chromeos] Limit the data samples stored by PowerDataCollector.

chromeos/power/power_data_collector_unittest.cc

Index: chromeos/power/power_data_collector_unittest.cc
diff --git a/chromeos/power/power_data_collector_unittest.cc b/chromeos/power/power_data_collector_unittest.cc
index a04a4d894595855868e764315c7bc2051a799005..a1a3dcc05a2e8e24a8cc8a65a024eef75c0faa64 100644
--- a/chromeos/power/power_data_collector_unittest.cc
+++ b/chromeos/power/power_data_collector_unittest.cc
@@ -38,9 +38,8 @@ TEST_F(PowerDataCollectorTest, PowerChanged) {
 
   prop1.set_external_power(power_manager::PowerSupplyProperties::DISCONNECTED);
   prop1.set_battery_percent(20.00);
-
   power_data_collector_->PowerChanged(prop1);
-  const std::vector<PowerDataCollector::PowerSupplySnapshot>& data1 =
+  const std::deque<PowerDataCollector::PowerSupplySnapshot>& data1 =
       power_data_collector_->power_supply_data();
   ASSERT_EQ(static_cast<size_t>(1), data1.size());
   EXPECT_DOUBLE_EQ(prop1.battery_percent(), data1[0].battery_percent);
@@ -48,13 +47,30 @@ TEST_F(PowerDataCollectorTest, PowerChanged) {
 
   prop2.set_external_power(power_manager::PowerSupplyProperties::AC);
   prop2.set_battery_percent(100.00);
-
   power_data_collector_->PowerChanged(prop2);
-  const std::vector<PowerDataCollector::PowerSupplySnapshot>& data2 =
+  const std::deque<PowerDataCollector::PowerSupplySnapshot>& data2 =
       power_data_collector_->power_supply_data();
   ASSERT_EQ(static_cast<size_t>(2), data2.size());
-  EXPECT_DOUBLE_EQ(prop2.battery_percent(), data1[1].battery_percent);
+  EXPECT_DOUBLE_EQ(prop2.battery_percent(), data2[1].battery_percent);
   EXPECT_TRUE(data2[1].external_power);
 }
 
+TEST_F(PowerDataCollectorTest, AddSnapshot) {
+  PowerDataCollector::PowerSupplySnapshot snapshot1, snapshot2;
+
+  snapshot1.time = base::TimeTicks::Now();
+  snapshot2.time = base::TimeTicks::Now() + base::TimeDelta::FromSeconds(
+      PowerDataCollector::kSampleTimeLimitSec);

[Daniel Erat, 2014/01/10 22:28:47]

nit: this is a bit racy now; you're depending on the fact that the two
base::TimeTicks::Now() calls return different values (since otherwise, the first
snapshot would be retained). something like this would be better:

snapshot1.time = base::TimeTicks::FromInternalValue(1000);  // abritrary
snapshot2.time = snapshot1.time +
    base::TimeDelta::FromSeconds(PowerDataCollector::kSampleTimeLimitSec + 1);

[Siva Chandra, 2014/01/10 22:54:53]

Done.

+
+  power_data_collector_->AddSnapshot(snapshot1);
+  const std::deque<PowerDataCollector::PowerSupplySnapshot>& data1 =
+      power_data_collector_->power_supply_data();
+  ASSERT_EQ(static_cast<size_t>(1), data1.size());
+
+  power_data_collector_->AddSnapshot(snapshot2);
+  const std::deque<PowerDataCollector::PowerSupplySnapshot>& data2 =
+      power_data_collector_->power_supply_data();
+  ASSERT_EQ(static_cast<size_t>(1), data2.size());

[Daniel Erat, 2014/01/10 22:28:47]

nit: also do:

  EXPECT_EQ(snapshot2.time.ToInternalValue(),
            data2[0].time.ToInternalValue());

to verify that the second snapshot was the one that was retained?

[Siva Chandra, 2014/01/10 22:54:53]

Done.

+}
+
 }  // namespace chromeos