Fix handling of file renames.

tests/rietveld_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 rietveld.py."""
 
 import logging
 import os
 import sys
@@ skipping 64 matching lines @@
     self.assertEquals(p.filename, filename)
     self.assertEquals(p.source_filename, source_filename)
     self.assertEquals(p.is_binary, is_binary)
     self.assertEquals(p.is_delete, is_delete)
     if hasattr(p, 'is_git_diff'):
       self.assertEquals(p.is_git_diff, is_git_diff)
     self.assertEquals(p.is_new, is_new)
     if hasattr(p, 'patchlevel'):
       self.assertEquals(p.patchlevel, patchlevel)
     if diff:
-      self.assertEquals(p.get(), diff)
+      self.assertEquals(p.get(True), diff)
     if hasattr(p, 'svn_properties'):
       self.assertEquals(p.svn_properties, svn_properties)
 
   def test_get_patch_no_status(self):
     self.requests = [('/api/123/456', _api({'file_a': {}}))]
     try:
       self.rietveld.get_patch(123, 456)
       self.fail()
     except patch.UnsupportedPatchFormat, e:
       self.assertEquals('file_a', e.filename)
 
   def test_get_patch_2_files(self):
     self.requests = [
         ('/api/123/456',
           _api({'foo': _file('A'), 'file_a': _file('M', chunk_id=790)})),
         ('/download/issue123_456_789.diff', RAW.NEW),
         ('/download/issue123_456_790.diff', RAW.NEW_NOT_NULL),
     ]
     patches = self.rietveld.get_patch(123, 456)
     self.assertEquals(2, len(patches.patches))
-    self._check_patch(patches.patches[0], 'foo', RAW.NEW, is_new=True)
     # TODO(maruel): svn sucks.
-    self._check_patch(patches.patches[1], 'file_a', RAW.NEW_NOT_NULL)
+    self._check_patch(patches.patches[0], 'file_a', RAW.NEW_NOT_NULL)
+    self._check_patch(patches.patches[1], 'foo', RAW.NEW, is_new=True)
 
   def test_get_patch_add(self):
     self.requests = [
         ('/api/123/456', _api({'foo': _file('A')})),
         ('/download/issue123_456_789.diff', RAW.NEW),
     ]
     patches = self.rietveld.get_patch(123, 456)
     self.assertEquals(1, len(patches.patches))
     self._check_patch(patches.patches[0], 'foo', RAW.NEW, is_new=True)
 
@@ skipping 223 matching lines @@
         {'foo': 'prout'},
     ]
     for i in self.rietveld.search(base='base'):
       self.assertEquals(expected.pop(0), i)
     self.assertEquals([], expected)
 
 
 if __name__ == '__main__':
   logging.basicConfig(level=logging.ERROR)
   unittest.main()