[BattOr] Add real device smoke test

common/battor/battor/battor_wrapper_devicetest.py

Index: common/battor/battor/battor_wrapper_devicetest.py
diff --git a/common/battor/battor/battor_wrapper_devicetest.py b/common/battor/battor/battor_wrapper_devicetest.py
new file mode 100644
index 0000000000000000000000000000000000000000..5c65e5f0e020eab73fff19d911b33160d1379883
--- /dev/null
+++ b/common/battor/battor/battor_wrapper_devicetest.py
@@ -0,0 +1,87 @@
+# Copyright 2016 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 logging
+import platform
+import os
+import sys
+import time
+import unittest
+
+if __name__ == '__main__':
+  sys.path.append(
+      os.path.join(os.path.dirname(__file__), '..'))
+
+from battor import battor_wrapper
+from devil.utils import battor_device_mapping
+from devil.utils import find_usb_devices
+
+
+# TODO(rnephew): Expand to work on mac when the CQ has a mac with a BattOr.
+_SUPPORTED_CQ_PLATFORMS = ['win', 'linux']
+
+class BattorWrapperDeviceTest(unittest.TestCase):
+  def setUp(self):
+    test_platform = platform.system()
+    self._battor_list = None
+
+    if 'Win' in test_platform:
+      self._platform = 'win'
+    elif 'Linux' in test_platform:
+      self._platform = 'linux'
+      device_tree  = find_usb_devices.GetBusNumberToDeviceTreeMap()
+      self._battor_list = battor_device_mapping.GetBattorList(device_tree)
+    elif 'Darwin' in test_platform:
+      self._platform = 'mac'
+
+  def testFullRun(self):
+    if self._platform not in _SUPPORTED_CQ_PLATFORMS:
+      logging.critical('Platform %s is not supported on CQ.' % self._platform)
+      return
+
+    # If battor_list is an empty list, a BattOr was expected but not found.
+    if self._battor_list is not None and not self._battor_list:
+      logging.critical('No BattOrs attached. Cannot run tests.')
+      return
+
+    battor_path = (None if not self._battor_list
+                   else '/dev/%s' % self._battor_list[0])
+    battor = battor_wrapper.BattorWrapper(self._platform,
+                                          battor_path=battor_path)
+    battor.StartShell()
+    battor.StartTracing()
+    # TODO(rnephew): This sleep is required for now because crbug.com/602266
+    # causes the BattOr to crash when the trace time is too short. Once that bug
+    # is fixed, we should remove this delay.
+    time.sleep(1)
+    battor.RecordClockSyncMarker('abc')
+    # Sleep here because clock sync marker will be flaky if not.
+    time.sleep(1)
+    battor.StopTracing()
+
+    # Below is a work around for crbug.com/603309. On this short of a trace, 5
+    # seconds is enough to ensure that the trace will finish flushing to the
+    # file. The process is then killed so that BattorWrapper knows that the
+    # process has been closed after tracing stops.
+    if self._platform == 'win':
+      time.sleep(5)
+      battor._battor_shell.kill()
+    results = battor.CollectTraceData()
+    self.assertTrue('# BattOr' in results[0])
+    self.assertTrue('# voltage_range' in results[1])
+    self.assertTrue('# current_range' in results[2])
+    self.assertTrue('# sample_rate' in results[3])
+    # First line with results. Should be 3 'words'.
+    self.assertTrue(len(results[4].split()) == 3)
+    clock_sync_found = False
+    for entry in results:
+      if '<abc>' in entry:
+        clock_sync_found = True
+        break
+    self.assertTrue(clock_sync_found, 'BattOr Data:%s\n' % repr(results))
+
+
+if __name__ == '__main__':
+  logging.getLogger().setLevel(logging.DEBUG)
+  unittest.main(verbosity=2)