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

Side by Side Diff: tests/gclient_utils_test.py

Issue 14759006: Kill subprocesses on KeyboardInterrupt. (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/depot_tools.git@master
Patch Set: Fix nit. Created 7 years, 7 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 | « gclient_utils.py ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 #!/usr/bin/env python 1 #!/usr/bin/env python
2 # Copyright (c) 2012 The Chromium Authors. All rights reserved. 2 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
3 # Use of this source code is governed by a BSD-style license that can be 3 # Use of this source code is governed by a BSD-style license that can be
4 # found in the LICENSE file. 4 # found in the LICENSE file.
5 5
6 import os 6 import os
7 import StringIO 7 import StringIO
8 import sys 8 import sys
9 9
10 sys.path.insert(0, os.path.dirname(os.path.dirname(os.path.abspath(__file__)))) 10 sys.path.insert(0, os.path.dirname(os.path.dirname(os.path.abspath(__file__))))
(...skipping 16 matching lines...) Expand all
27 class GclientUtilsUnittest(GclientUtilBase): 27 class GclientUtilsUnittest(GclientUtilBase):
28 """General gclient_utils.py tests.""" 28 """General gclient_utils.py tests."""
29 def testMembersChanged(self): 29 def testMembersChanged(self):
30 members = [ 30 members = [
31 'Annotated', 'AutoFlush', 'CheckCallAndFilter', 31 'Annotated', 'AutoFlush', 'CheckCallAndFilter',
32 'CheckCallAndFilterAndHeader', 'Error', 'ExecutionQueue', 'FileRead', 32 'CheckCallAndFilterAndHeader', 'Error', 'ExecutionQueue', 'FileRead',
33 'FileWrite', 'FindFileUpwards', 'FindGclientRoot', 33 'FileWrite', 'FindFileUpwards', 'FindGclientRoot',
34 'GetGClientRootAndEntries', 'GetEditor', 'IsDateRevision', 34 'GetGClientRootAndEntries', 'GetEditor', 'IsDateRevision',
35 'MakeDateRevision', 'MakeFileAutoFlush', 'MakeFileAnnotated', 35 'MakeDateRevision', 'MakeFileAutoFlush', 'MakeFileAnnotated',
36 'PathDifference', 'ParseCodereviewSettingsContent', 'NumLocalCpus', 36 'PathDifference', 'ParseCodereviewSettingsContent', 'NumLocalCpus',
37 'PrintableObject', 'RunEditor', 37 'PrintableObject', 'RunEditor', 'GCLIENT_CHILDREN',
38 'GCLIENT_CHILDREN_LOCK', 'GClientChildren',
38 'SplitUrlRevision', 'SyntaxErrorToError', 'UpgradeToHttps', 'Wrapper', 39 'SplitUrlRevision', 'SyntaxErrorToError', 'UpgradeToHttps', 'Wrapper',
39 'WorkItem', 'codecs', 'lockedmethod', 'logging', 'os', 'Queue', 're', 40 'WorkItem', 'codecs', 'lockedmethod', 'logging', 'os', 'Queue', 're',
40 'rmtree', 'safe_makedirs', 'stat', 'subprocess', 'subprocess2', 'sys', 41 'rmtree', 'safe_makedirs', 'stat', 'subprocess', 'subprocess2', 'sys',
41 'tempfile', 'threading', 'time', 'urlparse', 42 'tempfile', 'threading', 'time', 'urlparse',
42 ] 43 ]
43 # If this test fails, you should add the relevant test. 44 # If this test fails, you should add the relevant test.
44 self.compareMembers(gclient_utils, members) 45 self.compareMembers(gclient_utils, members)
45 46
46 47
47 48
48 class CheckCallAndFilterTestCase(GclientUtilBase): 49 class CheckCallAndFilterTestCase(GclientUtilBase):
49 class ProcessIdMock(object): 50 class ProcessIdMock(object):
50 def __init__(self, test_string): 51 def __init__(self, test_string):
51 self.stdout = StringIO.StringIO(test_string) 52 self.stdout = StringIO.StringIO(test_string)
53 self.pid = 9284
52 def wait(self): 54 def wait(self):
53 pass 55 pass
54 56
55 def _inner(self, args, test_string): 57 def _inner(self, args, test_string):
56 cwd = 'bleh' 58 cwd = 'bleh'
57 gclient_utils.sys.stdout.write( 59 gclient_utils.sys.stdout.write(
58 '\n________ running \'boo foo bar\' in \'bleh\'\n') 60 '\n________ running \'boo foo bar\' in \'bleh\'\n')
59 for i in test_string: 61 for i in test_string:
60 gclient_utils.sys.stdout.write(i) 62 gclient_utils.sys.stdout.write(i)
61 # pylint: disable=E1101 63 # pylint: disable=E1101
(...skipping 148 matching lines...) Expand 10 before | Expand all | Expand 10 after
210 for content, expected in values: 212 for content, expected in values:
211 self.assertEquals( 213 self.assertEquals(
212 expected, gclient_utils.ParseCodereviewSettingsContent(content)) 214 expected, gclient_utils.ParseCodereviewSettingsContent(content))
213 215
214 216
215 if __name__ == '__main__': 217 if __name__ == '__main__':
216 import unittest 218 import unittest
217 unittest.main() 219 unittest.main()
218 220
219 # vim: ts=2:sw=2:tw=80:et: 221 # vim: ts=2:sw=2:tw=80:et:
OLDNEW
« no previous file with comments | « gclient_utils.py ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698