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

Side by Side Diff: build/android/pylib/device/adb_wrapper_test.py

Issue 132463007: Enable presubmit pylint in build/android. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase vs tot and only disabling F0401 in specific spots Created 6 years, 10 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
« no previous file with comments | « build/android/pylib/device/adb_wrapper.py ('k') | build/android/pylib/flag_changer.py » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Copyright 2013 The Chromium Authors. All rights reserved. 1 # Copyright 2013 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 """Tests for the AdbWrapper class.""" 5 """Tests for the AdbWrapper class."""
6 6
7 import os 7 import os
8 import socket
9 import tempfile 8 import tempfile
10 import time 9 import time
11 import unittest 10 import unittest
12 11
13 import adb_wrapper 12 from pylib.device import adb_wrapper
14 13
15 14
16 class TestAdbWrapper(unittest.TestCase): 15 class TestAdbWrapper(unittest.TestCase):
17 16
18 def setUp(self): 17 def setUp(self):
19 devices = adb_wrapper.AdbWrapper.GetDevices() 18 devices = adb_wrapper.AdbWrapper.GetDevices()
20 assert devices, 'A device must be attached' 19 assert devices, 'A device must be attached'
21 self._adb = devices[0] 20 self._adb = devices[0]
22 self._adb.WaitForDevice() 21 self._adb.WaitForDevice()
23 22
24 def _MakeTempFile(self, contents): 23 @staticmethod
24 def _MakeTempFile(contents):
25 """Make a temporary file with the given contents. 25 """Make a temporary file with the given contents.
26 26
27 Args: 27 Args:
28 contents: string to write to the temporary file. 28 contents: string to write to the temporary file.
29 29
30 Returns: 30 Returns:
31 The absolute path to the file. 31 The absolute path to the file.
32 """ 32 """
33 fi, path = tempfile.mkstemp() 33 fi, path = tempfile.mkstemp()
34 with os.fdopen(fi, 'wb') as f: 34 with os.fdopen(fi, 'wb') as f:
35 f.write('foo') 35 f.write(contents)
36 return path 36 return path
37 37
38 def testShell(self): 38 def testShell(self):
39 output = self._adb.Shell('echo test', expect_rc=0) 39 output = self._adb.Shell('echo test', expect_rc=0)
40 self.assertEqual(output.strip(), 'test') 40 self.assertEqual(output.strip(), 'test')
41 output = self._adb.Shell('echo test') 41 output = self._adb.Shell('echo test')
42 self.assertEqual(output.strip(), 'test') 42 self.assertEqual(output.strip(), 'test')
43 self.assertRaises(adb_wrapper.CommandFailedError, self._adb.Shell, 43 self.assertRaises(adb_wrapper.CommandFailedError, self._adb.Shell,
44 'echo test', expect_rc=1) 44 'echo test', expect_rc=1)
45 45
(...skipping 35 matching lines...) Expand 10 before | Expand all | Expand 10 after
81 try: 81 try:
82 self._adb.Shell('start') 82 self._adb.Shell('start')
83 break 83 break
84 except adb_wrapper.CommandFailedError: 84 except adb_wrapper.CommandFailedError:
85 time.sleep(1) 85 time.sleep(1)
86 self._adb.Remount() 86 self._adb.Remount()
87 87
88 88
89 if __name__ == '__main__': 89 if __name__ == '__main__':
90 unittest.main() 90 unittest.main()
OLDNEW
« no previous file with comments | « build/android/pylib/device/adb_wrapper.py ('k') | build/android/pylib/flag_changer.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698