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

Side by Side Diff: tests/gcl_unittest.py

Issue 6717025: Revert r79006: "Add code to 'fix' python encoding and it's unit test." (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/tools/depot_tools
Patch Set: Created 9 years, 9 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 | Annotate | Revision Log
« no previous file with comments | « tests/fix_encoding_test.py ('k') | tests/presubmit_unittest.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/python 1 #!/usr/bin/python
2 # Copyright (c) 2010 The Chromium Authors. All rights reserved. 2 # Copyright (c) 2010 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 """Unit tests for gcl.py.""" 6 """Unit tests for gcl.py."""
7 7
8 # pylint is too confused. 8 # pylint is too confused.
9 # pylint: disable=E1101,E1103,E1120,W0212,W0403 9 # pylint: disable=E1101,E1103,E1120,W0212,W0403
10 10
(...skipping 72 matching lines...) Expand 10 before | Expand all | Expand 10 after
83 'DEFAULT_LINT_REGEX', 'CheckHomeForFile', 'DoPresubmitChecks', 83 'DEFAULT_LINT_REGEX', 'CheckHomeForFile', 'DoPresubmitChecks',
84 'ErrorExit', 'FILES_CACHE', 'FilterFlag', 'GenUsage', 84 'ErrorExit', 'FILES_CACHE', 'FilterFlag', 'GenUsage',
85 'GenerateChangeName', 'GenerateDiff', 'GetCLs', 'GetCacheDir', 85 'GenerateChangeName', 'GenerateDiff', 'GetCLs', 'GetCacheDir',
86 'GetCachedFile', 'GetChangelistInfoFile', 'GetChangesDir', 86 'GetCachedFile', 'GetChangelistInfoFile', 'GetChangesDir',
87 'GetCodeReviewSetting', 'GetEditor', 'GetFilesNotInCL', 'GetInfoDir', 87 'GetCodeReviewSetting', 'GetEditor', 'GetFilesNotInCL', 'GetInfoDir',
88 'GetModifiedFiles', 'GetRepositoryRoot', 'ListFiles', 88 'GetModifiedFiles', 'GetRepositoryRoot', 'ListFiles',
89 'LoadChangelistInfoForMultiple', 'MISSING_TEST_MSG', 89 'LoadChangelistInfoForMultiple', 'MISSING_TEST_MSG',
90 'OptionallyDoPresubmitChecks', 'REPOSITORY_ROOT', 'REVIEWERS_REGEX', 90 'OptionallyDoPresubmitChecks', 'REPOSITORY_ROOT', 'REVIEWERS_REGEX',
91 'RunShell', 'RunShellWithReturnCode', 'SVN', 91 'RunShell', 'RunShellWithReturnCode', 'SVN',
92 'TryChange', 'UnknownFiles', 'Warn', 92 'TryChange', 'UnknownFiles', 'Warn',
93 'attrs', 'breakpad', 'defer_attributes', 'fix_encoding', 93 'attrs', 'breakpad', 'defer_attributes', 'gclient_utils', 'getpass',
94 'gclient_utils', 'getpass',
95 'json', 'main', 'need_change', 'need_change_and_args', 'no_args', 94 'json', 'main', 'need_change', 'need_change_and_args', 'no_args',
96 'optparse', 'os', 'owners', 'presubmit_support', 'random', 're', 95 'optparse', 'os', 'owners', 'presubmit_support', 'random', 're',
97 'string', 'subprocess', 'suggest_reviewers', 'sys', 'tempfile', 96 'string', 'subprocess', 'suggest_reviewers', 'sys', 'tempfile',
98 'time', 'upload', 'urllib2', 97 'time', 'upload', 'urllib2',
99 ] 98 ]
100 # If this test fails, you should add the relevant test. 99 # If this test fails, you should add the relevant test.
101 self.compareMembers(gcl, members) 100 self.compareMembers(gcl, members)
102 101
103 def testIsSVNMoved(self): 102 def testIsSVNMoved(self):
104 # TODO(maruel): TEST ME 103 # TODO(maruel): TEST ME
(...skipping 456 matching lines...) Expand 10 before | Expand all | Expand 10 after
561 self.assertEquals(retval, 0) 560 self.assertEquals(retval, 0)
562 self.assertEquals(change_info.description, 561 self.assertEquals(change_info.description,
563 'deescription\n\nCommitted: http://view/12345') 562 'deescription\n\nCommitted: http://view/12345')
564 self.assertTrue(change_info._deleted) 563 self.assertTrue(change_info._deleted)
565 self.assertTrue(change_info._closed) 564 self.assertTrue(change_info._closed)
566 565
567 566
568 if __name__ == '__main__': 567 if __name__ == '__main__':
569 import unittest 568 import unittest
570 unittest.main() 569 unittest.main()
OLDNEW
« no previous file with comments | « tests/fix_encoding_test.py ('k') | tests/presubmit_unittest.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698