Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(9)

Side by Side Diff: systrace/systrace/tracing_agents/battor_trace_agent_unittest.py

Issue 2736123003: [Systrace][BattOr] Simplify BattOr options in systrace. (Closed)
Patch Set: Created 3 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 #!/usr/bin/env python 1 #!/usr/bin/env python
2 2
3 # Copyright 2016 The Chromium Authors. All rights reserved. 3 # Copyright 2016 The Chromium Authors. All rights reserved.
4 # Use of this source code is governed by a BSD-style license that can be 4 # Use of this source code is governed by a BSD-style license that can be
5 # found in the LICENSE file. 5 # found in the LICENSE file.
6 6
7 from collections import namedtuple 7 from collections import namedtuple
8 import unittest 8 import unittest
9 import logging 9 import logging
10 10
11 from systrace import decorators 11 from systrace import decorators
12 from systrace.tracing_agents import battor_trace_agent 12 from systrace.tracing_agents import battor_trace_agent
13 from battor import battor_wrapper 13 from battor import battor_wrapper
14 from devil.android import battery_utils 14 from devil.android import battery_utils
15 from devil.utils import battor_device_mapping 15 from devil.utils import battor_device_mapping
16 from devil.utils import find_usb_devices
16 17
17 18
18 mock_opts = namedtuple('mock_opts', ['target', 'device_serial_number', 19 mock_opts = namedtuple('mock_opts', ['target', 'device_serial_number',
charliea (OOO until 10-5) 2017/03/14 18:35:54 Looks like there might be some stuff in this secti
rnephew (Reviews Here) 2017/03/15 16:22:35 Done.
19 'hub_types', 'battor_path', 20 'hub_types', 'battor_path',
20 'update_map', 'serial_map']) 21 'update_map', 'serial_map'])
21 OPTIONS = mock_opts('android', 'Phn2', ['plugable_7port'], 22 OPTIONS = mock_opts('android', 'Phn2', ['plugable_7port'],
22 None, False, __file__) 23 None, False, __file__)
23 CATEGORIES = None 24 CATEGORIES = None
24 25
25 26
26 def raise_error(*args, **kwargs): 27 def raise_error(*args, **kwargs):
27 del args 28 del args
28 del kwargs 29 del kwargs
29 raise RuntimeError('Should not call this function in the test') 30 raise RuntimeError('Should not call this function in the test')
30 31
31 battor_device_mapping.GenerateSerialMapFile = raise_error 32 battor_device_mapping.GenerateSerialMapFile = raise_error
32 33
33 def setup_battor_test(StartShell_error, StartTracing_error, 34 def setup_battor_test(StartShell_error, StartTracing_error,
34 StopTracing_error, CollectTraceData_error): 35 StopTracing_error, CollectTraceData_error):
35 wrapper = MockBattOrWrapper(StartShell_error, StartTracing_error, 36 wrapper = MockBattOrWrapper(StartShell_error, StartTracing_error,
36 StopTracing_error, CollectTraceData_error) 37 StopTracing_error, CollectTraceData_error)
37 def wrapper_maker(*args, **kwargs): 38 def wrapper_maker(*args, **kwargs):
38 del args 39 del args
39 del kwargs 40 del kwargs
40 return wrapper 41 return wrapper
41 battor_wrapper.BattOrWrapper = wrapper_maker 42 battor_wrapper.BattOrWrapper = wrapper_maker
43 find_usb_devices.GetBusNumberToDeviceTreeMap = lambda: None
44 battor_device_mapping.GetBattOrList = lambda x: ['/dev/ttyUSB0']
42 45
43 46
44 class MockBattOrWrapper(object): 47 class MockBattOrWrapper(object):
45 def __init__(self, StartShell_error=False, StartTracing_error=False, 48 def __init__(self, StartShell_error=False, StartTracing_error=False,
46 StopTracing_error=False, CollectTraceData_error=False): 49 StopTracing_error=False, CollectTraceData_error=False):
47 self._StartShell_error = StartShell_error 50 self._StartShell_error = StartShell_error
48 self._StartTracing_error = StartTracing_error 51 self._StartTracing_error = StartTracing_error
49 self._StopTracing_error = StopTracing_error 52 self._StopTracing_error = StopTracing_error
50 self._CollectTraceData_error = CollectTraceData_error 53 self._CollectTraceData_error = CollectTraceData_error
51 self._running = False 54 self._running = False
(...skipping 97 matching lines...) Expand 10 before | Expand all | Expand 10 after
149 agent = battor_trace_agent.BattOrTraceAgent() 152 agent = battor_trace_agent.BattOrTraceAgent()
150 agent.StartAgentTracing(OPTIONS, CATEGORIES) 153 agent.StartAgentTracing(OPTIONS, CATEGORIES)
151 agent.StopAgentTracing() 154 agent.StopAgentTracing()
152 x = agent.GetResults() 155 x = agent.GetResults()
153 self.assertEqual(x.raw_data, 'traceout1\ntraceout2') 156 self.assertEqual(x.raw_data, 'traceout1\ntraceout2')
154 157
155 158
156 if __name__ == "__main__": 159 if __name__ == "__main__":
157 logging.getLogger().setLevel(logging.DEBUG) 160 logging.getLogger().setLevel(logging.DEBUG)
158 unittest.main(verbosity=2) 161 unittest.main(verbosity=2)
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698