editor: work around svn 1.7 closing the output fp in apply_textdelta() Calling close() on the output cStringIO when done with patching prevents us to retrieve the result with getvalue(). Note: I found this bug has been fixed already by 794f3e41ec1e. I feel ignoring the close() call is a little cleaner than storing the value before the handler() call for there is no guarantee the handler does not modify the value in its last call. Looking at text_delta.c, it does not in 1.7.1. So, do what you want with this patch.

import test_util

import unittest

class TestFetchBinaryFiles(test_util.TestBase):
    def test_binaryfiles(self, stupid=False):
        repo = self._load_fixture_and_fetch('binaryfiles.svndump', stupid=stupid)
        self.assertEqual('cce7fe400d8d', str(repo['tip']))

    def test_binaryfiles_stupid(self):
        self.test_binaryfiles(True)

def suite():
    all_tests = [unittest.TestLoader().loadTestsFromTestCase(TestFetchBinaryFiles),
          ]
    return unittest.TestSuite(all_tests)