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

Side by Side Diff: presubmit_canned_checks.py

Issue 118525: Update CheckChangeSvnEolStyle() to be upload-friendly. (Closed)
Patch Set: Created 11 years, 6 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 | « no previous file | 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/env python 1 #!/usr/bin/env python
2 # Copyright (c) 2006-2009 The Chromium Authors. All rights reserved. 2 # Copyright (c) 2006-2009 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 """Generic presubmit checks that can be reused by other presubmit checks.""" 6 """Generic presubmit checks that can be reused by other presubmit checks."""
7 7
8 8
9 ### Description checks 9 ### Description checks
10 10
(...skipping 160 matching lines...) Expand 10 before | Expand all | Expand 10 after
171 return [output_api.PresubmitPromptWarning(msg, items=bad)] 171 return [output_api.PresubmitPromptWarning(msg, items=bad)]
172 else: 172 else:
173 return [] 173 return []
174 174
175 175
176 def CheckChangeSvnEolStyle(input_api, output_api, source_file_filter=None): 176 def CheckChangeSvnEolStyle(input_api, output_api, source_file_filter=None):
177 """Checks that the source files have svn:eol-style=LF.""" 177 """Checks that the source files have svn:eol-style=LF."""
178 bad = filter(lambda f: f.scm == 'svn' and f.Property('svn:eol-style') != 'LF', 178 bad = filter(lambda f: f.scm == 'svn' and f.Property('svn:eol-style') != 'LF',
179 input_api.AffectedSourceFiles(source_file_filter)) 179 input_api.AffectedSourceFiles(source_file_filter))
180 if bad: 180 if bad:
181 return [output_api.PresubmitError( 181 if input_api.is_committing:
182 "Fix these files with svn svn:eol-style=LF", items=bad)] 182 return [output_api.PresubmitError(
183 "Fix these files with svn svn:eol-style=LF", items=bad)]
184 else:
185 return [output_api.PresubmitNotifyResult(
186 "Fix these files with svn svn:eol-style=LF", items=bad)]
183 return [] 187 return []
184 188
185 189
186 ### Other checks 190 ### Other checks
187 191
188 def CheckDoNotSubmit(input_api, output_api): 192 def CheckDoNotSubmit(input_api, output_api):
189 return ( 193 return (
190 CheckDoNotSubmitInDescription(input_api, output_api) + 194 CheckDoNotSubmitInDescription(input_api, output_api) +
191 CheckDoNotSubmitInFiles(input_api, output_api) 195 CheckDoNotSubmitInFiles(input_api, output_api)
192 ) 196 )
(...skipping 41 matching lines...) Expand 10 before | Expand all | Expand 10 after
234 long_text=input_api.traceback.format_exc())) 238 long_text=input_api.traceback.format_exc()))
235 239
236 buffer = input_api.cStringIO.StringIO() 240 buffer = input_api.cStringIO.StringIO()
237 results = input_api.unittest.TextTestRunner(stream=buffer, verbosity=0).run( 241 results = input_api.unittest.TextTestRunner(stream=buffer, verbosity=0).run(
238 input_api.unittest.TestSuite(tests_suite)) 242 input_api.unittest.TestSuite(tests_suite))
239 if not results.wasSuccessful(): 243 if not results.wasSuccessful():
240 outputs.append(message_type("%d unit tests failed." % 244 outputs.append(message_type("%d unit tests failed." %
241 (len(results.failures) + len(results.errors)), 245 (len(results.failures) + len(results.errors)),
242 long_text=buffer.getvalue())) 246 long_text=buffer.getvalue()))
243 return outputs 247 return outputs
OLDNEW
« no previous file with comments | « no previous file | tests/presubmit_unittest.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698