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

Side by Side Diff: client/tests/run_isolated_smoke_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/logging_utils_test.py ('k') | client/tests/run_isolated_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 2012 The Swarming Authors. All rights reserved. 2 # Copyright 2012 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 json 6 import json
7 import logging 7 import logging
8 import os 8 import os
9 import subprocess 9 import subprocess
10 import sys 10 import sys
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 15
16 import isolated_format 16 import isolated_format
17 import run_isolated 17 import run_isolated
18 from depot_tools import fix_encoding
18 from utils import file_path 19 from utils import file_path
19 20
20 import isolateserver_mock 21 import isolateserver_mock
21 import test_utils 22 import test_utils
22 23
23 24
24 CONTENTS = { 25 CONTENTS = {
25 'check_files.py': """if True: 26 'check_files.py': """if True:
26 import os, sys 27 import os, sys
27 ROOT_DIR = os.path.dirname(os.path.abspath(__file__)) 28 ROOT_DIR = os.path.dirname(os.path.abspath(__file__))
(...skipping 311 matching lines...) Expand 10 before | Expand all | Expand 10 after
339 # Test that an entry with an invalid file content but same size is NOT 340 # Test that an entry with an invalid file content but same size is NOT
340 # detected property. 341 # detected property.
341 cached_file_path = self._test_corruption_common( 342 cached_file_path = self._test_corruption_common(
342 CONTENTS['file1.txt'][:-1] + ' ') 343 CONTENTS['file1.txt'][:-1] + ' ')
343 # TODO(maruel): This corruption is NOT detected. 344 # TODO(maruel): This corruption is NOT detected.
344 # This needs to be fixed. 345 # This needs to be fixed.
345 self.assertNotEqual(CONTENTS['file1.txt'], read_content(cached_file_path)) 346 self.assertNotEqual(CONTENTS['file1.txt'], read_content(cached_file_path))
346 347
347 348
348 if __name__ == '__main__': 349 if __name__ == '__main__':
350 fix_encoding.fix_encoding()
349 test_utils.main() 351 test_utils.main()
OLDNEW
« no previous file with comments | « client/tests/logging_utils_test.py ('k') | client/tests/run_isolated_test.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698