OLD | NEW |
1 # Copyright (c) 2012 The Chromium Authors. All rights reserved. | 1 # Copyright (c) 2012 The Chromium Authors. All rights reserved. |
2 # Use of this source code is governed by a BSD-style license that can be | 2 # Use of this source code is governed by a BSD-style license that can be |
3 # found in the LICENSE file. | 3 # found in the LICENSE file. |
4 | 4 |
5 """Class for running instrumentation tests on a single device.""" | 5 """Class for running instrumentation tests on a single device.""" |
6 | 6 |
7 import collections | 7 import collections |
8 import logging | 8 import logging |
9 import os | 9 import os |
10 import re | 10 import re |
(...skipping 211 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
222 if not results: | 222 if not results: |
223 return | 223 return |
224 if results.DidRunPass(): | 224 if results.DidRunPass(): |
225 self.TearDownPerfMonitoring(test) | 225 self.TearDownPerfMonitoring(test) |
226 | 226 |
227 if self.coverage_dir: | 227 if self.coverage_dir: |
228 self.device.PullFile( | 228 self.device.PullFile( |
229 self.coverage_device_file, self.coverage_host_file) | 229 self.coverage_device_file, self.coverage_host_file) |
230 self.device.RunShellCommand( | 230 self.device.RunShellCommand( |
231 'rm -f %s' % self.coverage_device_file) | 231 'rm -f %s' % self.coverage_device_file) |
232 elif self.package_info: | 232 elif self.package_info and not self.options.skip_clear_data: |
233 apk_under_test = self.test_pkg.GetApkUnderTest() | 233 apk_under_test = self.test_pkg.GetApkUnderTest() |
234 permissions = apk_under_test.GetPermissions() if apk_under_test else None | 234 permissions = apk_under_test.GetPermissions() if apk_under_test else None |
235 self.device.ClearApplicationState( | 235 self.device.ClearApplicationState( |
236 self.package_info.package, permissions=permissions) | 236 self.package_info.package, permissions=permissions) |
| 237 self.device.ClearApplicationState(self.package_info.package) |
237 | 238 |
238 def TearDownPerfMonitoring(self, test): | 239 def TearDownPerfMonitoring(self, test): |
239 """Cleans up performance monitoring if the specified test required it. | 240 """Cleans up performance monitoring if the specified test required it. |
240 | 241 |
241 Args: | 242 Args: |
242 test: The name of the test that was just run. | 243 test: The name of the test that was just run. |
243 Raises: | 244 Raises: |
244 Exception: if there's anything wrong with the perf data. | 245 Exception: if there's anything wrong with the perf data. |
245 """ | 246 """ |
246 if not self._IsPerfTest(test): | 247 if not self._IsPerfTest(test): |
(...skipping 168 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
415 result_name, ' '.join(flag_modifiers.add)) | 416 result_name, ' '.join(flag_modifiers.add)) |
416 if flag_modifiers.remove: | 417 if flag_modifiers.remove: |
417 result_name = '%s without {%s}' % ( | 418 result_name = '%s without {%s}' % ( |
418 result_name, ' '.join(flag_modifiers.remove)) | 419 result_name, ' '.join(flag_modifiers.remove)) |
419 result.SetName(result_name) | 420 result.SetName(result_name) |
420 results.AddResult(result) | 421 results.AddResult(result) |
421 | 422 |
422 self.TestTeardown(test, results) | 423 self.TestTeardown(test, results) |
423 | 424 |
424 return (results, None if results.DidRunPass() else test) | 425 return (results, None if results.DidRunPass() else test) |
OLD | NEW |