[Telemetry] Fix C-State stat reading for Note 3 and S5.

tools/telemetry/telemetry/core/platform/power_monitor/sysfs_power_monitor_unittest.py

 # 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.
 
 import unittest
 
 from telemetry.core.platform.power_monitor import sysfs_power_monitor
 
 
 class SysfsPowerMonitorMonitorTest(unittest.TestCase):
@@ skipping 175 matching lines @@
     self.assertDictEqual(initial, self.expected_initial_freq)
     self.assertDictEqual(final, self.expected_final_freq)
 
   def testComputeCpuStats(self):
     results = sysfs_power_monitor.SysfsPowerMonitor.ComputeCpuStats(
         self.expected_initial_freq, self.expected_final_freq)
     for cpu in self.expected_freq_percents:
       for freq in results[cpu]:
         self.assertAlmostEqual(results[cpu][freq],
                                self.expected_freq_percents[cpu][freq])
+
+  def testGetCpuState(self):
+    class PlatformStub(object):
+      def __init__(self, run_command_return_value):
+        self._run_command_return_value = run_command_return_value
+      def RunCommand(self, _cmd):
+        return self._run_command_return_value
+
+    cpu_state_from_samsung_note3 = (
+        "C0\n\nC1\n\nC2\n\nC3\n\n"
+        "53658520886\n1809072\n7073\n1722554\n"
+        "1\n35\n300\n500\n"
+        "1412949256\n")
+    numOfCStates = 4
+
+    sysfsmon = sysfs_power_monitor.SysfsPowerMonitor(
+      PlatformStub(cpu_state_from_samsung_note3))
+    # pylint: disable=W0212
+    sysfsmon._cpus = ["cpu%d" % cpu for cpu in range(2)]
+    state = sysfsmon.GetCpuState()
+

[nednguyen, 2014/10/10 16:11:52]

Instead of the assertions below, how about asserting that
sysfsmon.ParseFreqSample(state) sysfsmon.ParseCStateSample(state) returns the
expected results ?

+    self.assertSetEqual(set(state.iterkeys()), set(sysfsmon._cpus))
+    for cpuState in state.itervalues():
+      lines = cpuState.splitlines()
+      self.assertEquals(len(lines), numOfCStates * 3 + 1)
+      try:
+        [int(line) for line in lines[numOfCStates:]]
+      except ValueError:
+        self.fail("Invalid integer in CPU state string: '%s'" % line)