Improve patch handling and tests.

tests/patch_test.py

 #!/usr/bin/env python
 # Copyright (c) 2011 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.
 
 """Unit tests for patch.py."""
 
 import os
 import sys
 import unittest
@@ skipping 235 matching lines @@
         'e\n'
         '+FOO!\n'
         'ff\n'
         'ggg\n'
         'hh\n'),
         [])
       self.fail()
     except patch.UnsupportedPatchFormat:
       pass
 
-  def testInvalidFilePatchDiffSvn(self):
-    try:
-      patch.FilePatchDiff('svn_utils_test.txt', (
-        '--- svn_utils_test.txt2\n'
-        '+++ svn_utils_test.txt\n'
-        '@@ -3,6 +3,7 @@ bb\n'
-        'ccc\n'
-        'dd\n'
-        'e\n'
-        '+FOO!\n'
-        'ff\n'
-        'ggg\n'
-        'hh\n'),
-        [])
-      self.fail()
-    except patch.UnsupportedPatchFormat:
-      pass
-
   def testValidSvn(self):
     # pylint: disable=R0201
     # Method could be a function
     # Should not throw.
     p = patch.FilePatchDiff('chrome/file.cc', SVN_PATCH, [])
     lines = SVN_PATCH.splitlines(True)
     header = ''.join(lines[:4])
     hunks = ''.join(lines[4:])
     self.assertEquals(header, p.diff_header)
     self.assertEquals(hunks, p.diff_hunks)
@@ skipping 74 matching lines @@
     # Shouldn't throw.
     patch.FilePatchDiff('tools/clang_check/README.chromium', GIT_DELETE, [])
     patch.FilePatchDiff('tools/run_local_server.sh', GIT_RENAME, [])
     patch.FilePatchDiff(
         'chrome/browser/chromeos/views/webui_menu_widget.h',
         GIT_RENAME_PARTIAL, [])
     patch.FilePatchDiff('pp', GIT_COPY, [])
     patch.FilePatchDiff('foo', GIT_NEW, [])
     self.assertTrue(True)
 
+  def testOnlyHeader(self):
+    p = patch.FilePatchDiff('file_a', '--- file_a\n+++ file_a\n', [])
+    self.assertTrue(p)
+
+  def testSmallest(self):
+    p = patch.FilePatchDiff(
+        'file_a', '--- file_a\n+++ file_a\n@@ -0,0 +1 @@\n+foo\n', [])
+    self.assertTrue(p)
+
+  def testInverted(self):
+    try:
+      patch.FilePatchDiff(
+        'file_a', '+++ file_a\n--- file_a\n@@ -0,0 +1 @@\n+foo\n', [])
+      self.fail()
+    except patch.UnsupportedPatchFormat:
+      pass
+
+  def testInvertedOnlyHeader(self):
+    try:
+      patch.FilePatchDiff('file_a', '+++ file_a\n--- file_a\n', [])
+      self.fail()
+    except patch.UnsupportedPatchFormat:
+      pass
+
+  def testRenameOnlyHeader(self):
+    p = patch.FilePatchDiff('file_b', '--- file_a\n+++ file_b\n', [])
+    self.assertTrue(p)
+
+  def testGitCopy(self):
+    diff = (
+        'diff --git a/wtf b/wtf2\n'
+        'similarity index 98%\n'
+        'copy from wtf\n'
+        'copy to wtf2\n'
+        'index 79fbaf3..3560689 100755\n'
+        '--- a/wtf\n'
+        '+++ b/wtf2\n'
+        '@@ -1,4 +1,4 @@\n'
+        '-#!/usr/bin/env python\n'
+        '+#!/usr/bin/env python1.3\n'
+        ' # Copyright (c) 2010 The Chromium Authors. All rights reserved.\n'
+        ' # blah blah blah as\n'
+        ' # found in the LICENSE file.\n')
+    p = patch.FilePatchDiff('wtf2', diff, [])
+    self.assertTrue(p)
+
 
 if __name__ == '__main__':
   unittest.main()