Add a presubmit check to prevent committing .rej/.orig patch files

PRESUBMIT_test.py

 #!/usr/bin/env python
 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 import os
 import re
 import unittest
 
 import PRESUBMIT
 
 
 class MockInputApi(object):
   def __init__(self):
     self.re = re
     self.os_path = os.path
+    self.files = []
+
+  def AddFile(self, f):

[M-A Ruel, 2012/12/05 19:30:17]

I don't think it's necessary.

+    self.files.append(f)
+
+  def AffectedFiles(self):
+    return self.files
+
+
+class MockOutputApi(object):
+  class PresubmitResult(object):
+    def __init__(self, message, items=None, long_text=''):
+      self.message = message
+      self.items = items
+      self.long_text = long_text
+
+  class PresubmitError(PresubmitResult):
+    pass
+
+  class PresubmitPromptWarning(PresubmitResult):
+    pass
+
+  class PresubmitNotifyResult(PresubmitResult):
+    pass
 
 
 class MockFile(object):
   def __init__(self, local_path, new_contents):
     self._local_path = local_path
     self._new_contents = new_contents
     self._changed_contents = [(i + 1, l) for i, l in enumerate(new_contents)]
 
   def ChangedContents(self):
     return self._changed_contents
@@ skipping 167 matching lines @@
              '  ScopedTempDir temp_dir_;',
              '>>>>>>> master']
     errors = PRESUBMIT._CheckForVersionControlConflictsInFile(
         MockInputApi(), MockFile('some/path/foo_platform.cc', lines))
     self.assertEqual(3, len(errors))
     self.assertTrue('1' in errors[0])
     self.assertTrue('3' in errors[1])
     self.assertTrue('5' in errors[2])
 
 
+class BadExtensionsTest(unittest.TestCase):
+  def testBadRejFile(self):
+    mock_input_api = MockInputApi()
+    mock_input_api.AddFile(MockFile('some/path/foo.cc', ''))

[M-A Ruel, 2012/12/05 19:30:17]

mock_input_api.files = [
    MockFile(...),
    ...
]

+    mock_input_api.AddFile(MockFile('some/path/foo.cc.rej', ''))
+    mock_input_api.AddFile(MockFile('some/path2/bar.h.rej', ''))
+
+    results = PRESUBMIT._CheckPatchFiles(mock_input_api, MockOutputApi())
+    self.assertEqual(1, len(results))
+    self.assertEqual(2, len(results[0].items))
+    self.assertTrue('foo.cc.rej' in results[0].items[0])
+    self.assertTrue('bar.h.rej' in results[0].items[1])
+
+  def testBadOrigFile(self):
+    mock_input_api = MockInputApi()
+    mock_input_api.AddFile(MockFile('other/path/qux.h.orig', ''))
+    mock_input_api.AddFile(MockFile('other/path/qux.h', ''))
+    mock_input_api.AddFile(MockFile('other/path/qux.cc', ''))
+
+    results = PRESUBMIT._CheckPatchFiles(mock_input_api, MockOutputApi())
+    self.assertEqual(1, len(results))
+    self.assertEqual(1, len(results[0].items))
+    self.assertTrue('qux.h.orig' in results[0].items[0])
+
+  def testGoodFiles(self):
+    mock_input_api = MockInputApi()
+    mock_input_api.AddFile(MockFile('other/path/qux.h', ''))
+    mock_input_api.AddFile(MockFile('other/path/qux.cc', ''))
+    results = PRESUBMIT._CheckPatchFiles(mock_input_api, MockOutputApi())
+    self.assertEqual(0, len(results))
+
+
 if __name__ == '__main__':
   unittest.main()