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

Side by Side Diff: client/tests/isolate_format_test.py

Issue 1390773002: Add an assert to ensure UTF-8 locale when handling file paths. (Closed) Base URL: git@github.com:luci/luci-py.git@master
Patch Set: More fixes Created 5 years, 2 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 | « client/tests/file_path_test.py ('k') | client/tests/isolate_smoke_test.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 #!/usr/bin/env python 1 #!/usr/bin/env python
2 # Copyright 2014 The Swarming Authors. All rights reserved. 2 # Copyright 2014 The Swarming Authors. All rights reserved.
3 # Use of this source code is governed under the Apache License, Version 2.0 that 3 # Use of this source code is governed under the Apache License, Version 2.0 that
4 # can be found in the LICENSE file. 4 # can be found in the LICENSE file.
5 5
6 import cStringIO 6 import cStringIO
7 import logging 7 import logging
8 import os 8 import os
9 import sys 9 import sys
10 import tempfile 10 import tempfile
11 import unittest 11 import unittest
12 12
13 ROOT_DIR = os.path.dirname(os.path.dirname(os.path.abspath(__file__))) 13 ROOT_DIR = os.path.dirname(os.path.dirname(os.path.abspath(__file__)))
14 sys.path.insert(0, ROOT_DIR) 14 sys.path.insert(0, ROOT_DIR)
15 sys.path.insert(0, os.path.join(ROOT_DIR, 'third_party')) 15 sys.path.insert(0, os.path.join(ROOT_DIR, 'third_party'))
16 16
17 import isolate_format 17 import isolate_format
18 from depot_tools import auto_stub 18 from depot_tools import auto_stub
19 from depot_tools import fix_encoding
19 from utils import file_path 20 from utils import file_path
20 21
21 22
22 # Access to a protected member XXX of a client class 23 # Access to a protected member XXX of a client class
23 # pylint: disable=W0212 24 # pylint: disable=W0212
24 25
25 26
26 FAKE_DIR = ( 27 FAKE_DIR = (
27 u'z:\\path\\to\\non_existing' 28 u'z:\\path\\to\\non_existing'
28 if sys.platform == 'win32' else u'/path/to/non_existing') 29 if sys.platform == 'win32' else u'/path/to/non_existing')
(...skipping 958 matching lines...) Expand 10 before | Expand all | Expand 10 after
987 'files': [ 988 'files': [
988 '<(PATH)/file_non_linux', 989 '<(PATH)/file_non_linux',
989 ], 990 ],
990 'isolate_dir': dir_1, 991 'isolate_dir': dir_1,
991 }, 992 },
992 } 993 }
993 self.assertEqual(expected, actual.flatten()) 994 self.assertEqual(expected, actual.flatten())
994 995
995 996
996 if __name__ == '__main__': 997 if __name__ == '__main__':
998 fix_encoding.fix_encoding()
997 logging.basicConfig( 999 logging.basicConfig(
998 level=logging.DEBUG if '-v' in sys.argv else logging.ERROR, 1000 level=logging.DEBUG if '-v' in sys.argv else logging.ERROR,
999 format='%(levelname)5s %(filename)15s(%(lineno)3d): %(message)s') 1001 format='%(levelname)5s %(filename)15s(%(lineno)3d): %(message)s')
1000 if '-v' in sys.argv: 1002 if '-v' in sys.argv:
1001 unittest.TestCase.maxDiff = None 1003 unittest.TestCase.maxDiff = None
1002 unittest.main() 1004 unittest.main()
OLDNEW
« no previous file with comments | « client/tests/file_path_test.py ('k') | client/tests/isolate_smoke_test.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698