/brz/remove-bazaar

To get this branch, use:
bzr branch http://gegoxaren.bato24.eu/bzr/brz/remove-bazaar
2052.3.1 by John Arbash Meinel
Add tests to cleanup the copyright of all source files
1
# Copyright (C) 2005, 2006 Canonical Ltd
1711.2.16 by John Arbash Meinel
test_diff needs a copyright statement
2
#
3
# This program is free software; you can redistribute it and/or modify
4
# it under the terms of the GNU General Public License as published by
5
# the Free Software Foundation; either version 2 of the License, or
6
# (at your option) any later version.
7
#
8
# This program is distributed in the hope that it will be useful,
9
# but WITHOUT ANY WARRANTY; without even the implied warranty of
10
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
11
# GNU General Public License for more details.
2052.3.1 by John Arbash Meinel
Add tests to cleanup the copyright of all source files
12
#
1711.2.16 by John Arbash Meinel
test_diff needs a copyright statement
13
# You should have received a copy of the GNU General Public License
14
# along with this program; if not, write to the Free Software
15
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
16
1740.2.5 by Aaron Bentley
Merge from bzr.dev
17
import os
1558.15.2 by Aaron Bentley
Implemented binary file handling for diff
18
from cStringIO import StringIO
1692.8.7 by James Henstridge
changes suggested by John Meinel
19
import errno
1920.1.1 by John Arbash Meinel
fix bug #56307, handle binary files even when LANG is not english
20
import subprocess
1920.1.3 by John Arbash Meinel
Remove spurious import
21
from tempfile import TemporaryFile
1558.15.2 by Aaron Bentley
Implemented binary file handling for diff
22
3009.2.9 by Aaron Bentley
Add tests for Differ
23
from bzrlib.diff import (
24
    internal_diff,
25
    external_diff,
3009.2.22 by Aaron Bentley
Update names & docstring
26
    DiffPath,
3009.2.9 by Aaron Bentley
Add tests for Differ
27
    show_diff_trees,
3009.2.22 by Aaron Bentley
Update names & docstring
28
    DiffSymlink,
29
    DiffTree,
30
    DiffText,
3009.2.9 by Aaron Bentley
Add tests for Differ
31
    )
1711.2.56 by John Arbash Meinel
Raise NoDiff if 'diff' not present.
32
from bzrlib.errors import BinaryFile, NoDiff
2321.2.5 by Alexander Belchenko
external diff: no need for special code path for win32 (suggested by John Meinel)
33
import bzrlib.osutils as osutils
1711.2.20 by John Arbash Meinel
Late bind to patiencediff objects to make it easier to plug-in
34
import bzrlib.patiencediff
2781.1.1 by Martin Pool
merge cpatiencediff from Lukas
35
import bzrlib._patiencediff_py
2592.2.4 by Jonathan Lange
Skip the unicode filename test if the platform doesn't support unicode
36
from bzrlib.tests import (Feature, TestCase, TestCaseWithTransport,
1711.2.54 by John Arbash Meinel
Use mkstemp instead of NamedTemporary file for external diff.
37
                          TestCaseInTempDir, TestSkipped)
1558.15.2 by Aaron Bentley
Implemented binary file handling for diff
38
39
2781.1.1 by Martin Pool
merge cpatiencediff from Lukas
40
class _CompiledPatienceDiffFeature(Feature):
41
42
    def _probe(self):
43
        try:
44
            import bzrlib._patiencediff_c
45
        except ImportError:
46
            return False
47
        return True
48
49
    def feature_name(self):
50
        return 'bzrlib._patiencediff_c'
51
52
CompiledPatienceDiffFeature = _CompiledPatienceDiffFeature()
53
54
2592.2.4 by Jonathan Lange
Skip the unicode filename test if the platform doesn't support unicode
55
class _UnicodeFilename(Feature):
56
    """Does the filesystem support Unicode filenames?"""
57
58
    def _probe(self):
59
        try:
2653.1.1 by John Arbash Meinel
(Jonathan Lange) Fix bug #110092, when displaying a diff that includes a binary file, make sure the binary filenames are encoded
60
            os.stat(u'\u03b1')
2592.2.4 by Jonathan Lange
Skip the unicode filename test if the platform doesn't support unicode
61
        except UnicodeEncodeError:
62
            return False
2592.2.5 by Jonathan Lange
Make UnicodeFilename feature less insane. Add a simple test for it too.
63
        except (IOError, OSError):
64
            # The filesystem allows the Unicode filename but the file doesn't
2653.1.1 by John Arbash Meinel
(Jonathan Lange) Fix bug #110092, when displaying a diff that includes a binary file, make sure the binary filenames are encoded
65
            # exist.
2592.2.5 by Jonathan Lange
Make UnicodeFilename feature less insane. Add a simple test for it too.
66
            return True
67
        else:
68
            # The filesystem allows the Unicode filename and the file exists,
69
            # for some reason.
70
            return True
2592.2.4 by Jonathan Lange
Skip the unicode filename test if the platform doesn't support unicode
71
72
UnicodeFilename = _UnicodeFilename()
73
74
2592.2.5 by Jonathan Lange
Make UnicodeFilename feature less insane. Add a simple test for it too.
75
class TestUnicodeFilename(TestCase):
76
77
    def test_probe_passes(self):
78
        """UnicodeFilename._probe passes."""
79
        # We can't test much more than that because the behaviour depends
80
        # on the platform.
81
        UnicodeFilename._probe()
82
        
83
1558.15.11 by Aaron Bentley
Apply merge review suggestions
84
def udiff_lines(old, new, allow_binary=False):
974.1.6 by Aaron Bentley
Added unit tests
85
    output = StringIO()
1558.15.11 by Aaron Bentley
Apply merge review suggestions
86
    internal_diff('old', old, 'new', new, output, allow_binary)
974.1.6 by Aaron Bentley
Added unit tests
87
    output.seek(0, 0)
88
    return output.readlines()
89
1711.2.54 by John Arbash Meinel
Use mkstemp instead of NamedTemporary file for external diff.
90
1711.2.57 by John Arbash Meinel
Allow external diff to write to a file without a fileno.
91
def external_udiff_lines(old, new, use_stringio=False):
92
    if use_stringio:
93
        # StringIO has no fileno, so it tests a different codepath
94
        output = StringIO()
95
    else:
96
        output = TemporaryFile()
1692.8.7 by James Henstridge
changes suggested by John Meinel
97
    try:
98
        external_diff('old', old, 'new', new, output, diff_opts=['-u'])
1711.2.58 by John Arbash Meinel
Use osutils.pumpfile so we don't have to buffer everything in ram
99
    except NoDiff:
1711.2.56 by John Arbash Meinel
Raise NoDiff if 'diff' not present.
100
        raise TestSkipped('external "diff" not present to test')
1692.8.2 by James Henstridge
add a test for sending external diff output to a file
101
    output.seek(0, 0)
102
    lines = output.readlines()
103
    output.close()
104
    return lines
105
106
1102 by Martin Pool
- merge test refactoring from robertc
107
class TestDiff(TestCase):
1185.81.25 by Aaron Bentley
Clean up test_diff
108
1102 by Martin Pool
- merge test refactoring from robertc
109
    def test_add_nl(self):
110
        """diff generates a valid diff for patches that add a newline"""
974.1.6 by Aaron Bentley
Added unit tests
111
        lines = udiff_lines(['boo'], ['boo\n'])
1185.16.145 by Martin Pool
Remove all assert statements from test cases.
112
        self.check_patch(lines)
113
        self.assertEquals(lines[4], '\\ No newline at end of file\n')
114
            ## "expected no-nl, got %r" % lines[4]
974.1.6 by Aaron Bentley
Added unit tests
115
1102 by Martin Pool
- merge test refactoring from robertc
116
    def test_add_nl_2(self):
117
        """diff generates a valid diff for patches that change last line and
118
        add a newline.
119
        """
974.1.6 by Aaron Bentley
Added unit tests
120
        lines = udiff_lines(['boo'], ['goo\n'])
1185.16.145 by Martin Pool
Remove all assert statements from test cases.
121
        self.check_patch(lines)
122
        self.assertEquals(lines[4], '\\ No newline at end of file\n')
123
            ## "expected no-nl, got %r" % lines[4]
974.1.6 by Aaron Bentley
Added unit tests
124
1102 by Martin Pool
- merge test refactoring from robertc
125
    def test_remove_nl(self):
126
        """diff generates a valid diff for patches that change last line and
127
        add a newline.
128
        """
974.1.6 by Aaron Bentley
Added unit tests
129
        lines = udiff_lines(['boo\n'], ['boo'])
1185.16.145 by Martin Pool
Remove all assert statements from test cases.
130
        self.check_patch(lines)
131
        self.assertEquals(lines[5], '\\ No newline at end of file\n')
132
            ## "expected no-nl, got %r" % lines[5]
133
134
    def check_patch(self, lines):
135
        self.assert_(len(lines) > 1)
136
            ## "Not enough lines for a file header for patch:\n%s" % "".join(lines)
137
        self.assert_(lines[0].startswith ('---'))
138
            ## 'No orig line for patch:\n%s' % "".join(lines)
139
        self.assert_(lines[1].startswith ('+++'))
140
            ## 'No mod line for patch:\n%s' % "".join(lines)
141
        self.assert_(len(lines) > 2)
142
            ## "No hunks for patch:\n%s" % "".join(lines)
143
        self.assert_(lines[2].startswith('@@'))
144
            ## "No hunk header for patch:\n%s" % "".join(lines)
145
        self.assert_('@@' in lines[2][2:])
146
            ## "Unterminated hunk header for patch:\n%s" % "".join(lines)
147
1558.15.2 by Aaron Bentley
Implemented binary file handling for diff
148
    def test_binary_lines(self):
149
        self.assertRaises(BinaryFile, udiff_lines, [1023 * 'a' + '\x00'], [])
150
        self.assertRaises(BinaryFile, udiff_lines, [], [1023 * 'a' + '\x00'])
1558.15.11 by Aaron Bentley
Apply merge review suggestions
151
        udiff_lines([1023 * 'a' + '\x00'], [], allow_binary=True)
152
        udiff_lines([], [1023 * 'a' + '\x00'], allow_binary=True)
1692.8.2 by James Henstridge
add a test for sending external diff output to a file
153
154
    def test_external_diff(self):
155
        lines = external_udiff_lines(['boo\n'], ['goo\n'])
156
        self.check_patch(lines)
1899.1.6 by John Arbash Meinel
internal_diff always adds a trailing \n, make sure external_diff does too
157
        self.assertEqual('\n', lines[-1])
1711.2.57 by John Arbash Meinel
Allow external diff to write to a file without a fileno.
158
159
    def test_external_diff_no_fileno(self):
160
        # Make sure that we can handle not having a fileno, even
161
        # if the diff is large
162
        lines = external_udiff_lines(['boo\n']*10000,
163
                                     ['goo\n']*10000,
164
                                     use_stringio=True)
165
        self.check_patch(lines)
1899.1.1 by John Arbash Meinel
Fix the bug in the NoDiff exception class, and add a test
166
1920.1.1 by John Arbash Meinel
fix bug #56307, handle binary files even when LANG is not english
167
    def test_external_diff_binary_lang_c(self):
2321.2.5 by Alexander Belchenko
external diff: no need for special code path for win32 (suggested by John Meinel)
168
        old_env = {}
169
        for lang in ('LANG', 'LC_ALL', 'LANGUAGE'):
170
            old_env[lang] = osutils.set_or_unset_env(lang, 'C')
1920.1.1 by John Arbash Meinel
fix bug #56307, handle binary files even when LANG is not english
171
        try:
172
            lines = external_udiff_lines(['\x00foobar\n'], ['foo\x00bar\n'])
1959.1.1 by Marien Zwart
merge.
173
            # Older versions of diffutils say "Binary files", newer
174
            # versions just say "Files".
175
            self.assertContainsRe(lines[0],
176
                                  '(Binary f|F)iles old and new differ\n')
177
            self.assertEquals(lines[1:], ['\n'])
1920.1.1 by John Arbash Meinel
fix bug #56307, handle binary files even when LANG is not english
178
        finally:
2321.2.5 by Alexander Belchenko
external diff: no need for special code path for win32 (suggested by John Meinel)
179
            for lang, old_val in old_env.iteritems():
180
                osutils.set_or_unset_env(lang, old_val)
1899.1.4 by John Arbash Meinel
Just swallow a return code of 2
181
1899.1.1 by John Arbash Meinel
Fix the bug in the NoDiff exception class, and add a test
182
    def test_no_external_diff(self):
183
        """Check that NoDiff is raised when diff is not available"""
184
        # Use os.environ['PATH'] to make sure no 'diff' command is available
185
        orig_path = os.environ['PATH']
186
        try:
187
            os.environ['PATH'] = ''
188
            self.assertRaises(NoDiff, external_diff,
189
                              'old', ['boo\n'], 'new', ['goo\n'],
190
                              StringIO(), diff_opts=['-u'])
191
        finally:
192
            os.environ['PATH'] = orig_path
1692.8.2 by James Henstridge
add a test for sending external diff output to a file
193
        
1711.2.30 by John Arbash Meinel
Fix bug in internal_diff handling of unicode paths
194
    def test_internal_diff_default(self):
195
        # Default internal diff encoding is utf8
196
        output = StringIO()
197
        internal_diff(u'old_\xb5', ['old_text\n'],
198
                    u'new_\xe5', ['new_text\n'], output)
199
        lines = output.getvalue().splitlines(True)
200
        self.check_patch(lines)
1740.2.5 by Aaron Bentley
Merge from bzr.dev
201
        self.assertEquals(['--- old_\xc2\xb5\n',
202
                           '+++ new_\xc3\xa5\n',
1711.2.30 by John Arbash Meinel
Fix bug in internal_diff handling of unicode paths
203
                           '@@ -1,1 +1,1 @@\n',
204
                           '-old_text\n',
205
                           '+new_text\n',
206
                           '\n',
207
                          ]
208
                          , lines)
209
210
    def test_internal_diff_utf8(self):
211
        output = StringIO()
212
        internal_diff(u'old_\xb5', ['old_text\n'],
213
                    u'new_\xe5', ['new_text\n'], output,
214
                    path_encoding='utf8')
215
        lines = output.getvalue().splitlines(True)
216
        self.check_patch(lines)
1740.2.5 by Aaron Bentley
Merge from bzr.dev
217
        self.assertEquals(['--- old_\xc2\xb5\n',
218
                           '+++ new_\xc3\xa5\n',
1711.2.30 by John Arbash Meinel
Fix bug in internal_diff handling of unicode paths
219
                           '@@ -1,1 +1,1 @@\n',
220
                           '-old_text\n',
221
                           '+new_text\n',
222
                           '\n',
223
                          ]
224
                          , lines)
225
226
    def test_internal_diff_iso_8859_1(self):
227
        output = StringIO()
228
        internal_diff(u'old_\xb5', ['old_text\n'],
229
                    u'new_\xe5', ['new_text\n'], output,
230
                    path_encoding='iso-8859-1')
231
        lines = output.getvalue().splitlines(True)
232
        self.check_patch(lines)
1740.2.5 by Aaron Bentley
Merge from bzr.dev
233
        self.assertEquals(['--- old_\xb5\n',
234
                           '+++ new_\xe5\n',
1711.2.30 by John Arbash Meinel
Fix bug in internal_diff handling of unicode paths
235
                           '@@ -1,1 +1,1 @@\n',
236
                           '-old_text\n',
237
                           '+new_text\n',
238
                           '\n',
239
                          ]
240
                          , lines)
241
3085.1.1 by John Arbash Meinel
Fix internal_diff to not fail when the texts are identical.
242
    def test_internal_diff_no_content(self):
243
        output = StringIO()
244
        internal_diff(u'old', [], u'new', [], output)
245
        self.assertEqual('', output.getvalue())
246
247
    def test_internal_diff_no_changes(self):
248
        output = StringIO()
249
        internal_diff(u'old', ['text\n', 'contents\n'],
250
                      u'new', ['text\n', 'contents\n'],
251
                      output)
252
        self.assertEqual('', output.getvalue())
253
1711.2.30 by John Arbash Meinel
Fix bug in internal_diff handling of unicode paths
254
    def test_internal_diff_returns_bytes(self):
255
        import StringIO
256
        output = StringIO.StringIO()
257
        internal_diff(u'old_\xb5', ['old_text\n'],
258
                    u'new_\xe5', ['new_text\n'], output)
259
        self.failUnless(isinstance(output.getvalue(), str),
260
            'internal_diff should return bytestrings')
261
1185.81.25 by Aaron Bentley
Clean up test_diff
262
1920.1.1 by John Arbash Meinel
fix bug #56307, handle binary files even when LANG is not english
263
class TestDiffFiles(TestCaseInTempDir):
264
265
    def test_external_diff_binary(self):
266
        """The output when using external diff should use diff's i18n error"""
267
        # Make sure external_diff doesn't fail in the current LANG
268
        lines = external_udiff_lines(['\x00foobar\n'], ['foo\x00bar\n'])
269
2240.1.1 by Alexander Belchenko
test_external_diff_binary: run external diff with --binary flag
270
        cmd = ['diff', '-u', '--binary', 'old', 'new']
1920.1.1 by John Arbash Meinel
fix bug #56307, handle binary files even when LANG is not english
271
        open('old', 'wb').write('\x00foobar\n')
272
        open('new', 'wb').write('foo\x00bar\n')
273
        pipe = subprocess.Popen(cmd, stdout=subprocess.PIPE,
274
                                     stdin=subprocess.PIPE)
275
        out, err = pipe.communicate()
276
        # Diff returns '2' on Binary files.
277
        self.assertEqual(2, pipe.returncode)
278
        # We should output whatever diff tells us, plus a trailing newline
279
        self.assertEqual(out.splitlines(True) + ['\n'], lines)
280
281
2405.1.1 by John Arbash Meinel
Add a bunch of direct tests for 'show_diff_trees'
282
class TestShowDiffTreesHelper(TestCaseWithTransport):
283
    """Has a helper for running show_diff_trees"""
284
285
    def get_diff(self, tree1, tree2, specific_files=None, working_tree=None):
286
        output = StringIO()
287
        if working_tree is not None:
288
            extra_trees = (working_tree,)
289
        else:
290
            extra_trees = ()
291
        show_diff_trees(tree1, tree2, output, specific_files=specific_files,
292
                        extra_trees=extra_trees, old_label='old/',
293
                        new_label='new/')
294
        return output.getvalue()
295
296
297
class TestDiffDates(TestShowDiffTreesHelper):
1740.2.5 by Aaron Bentley
Merge from bzr.dev
298
299
    def setUp(self):
300
        super(TestDiffDates, self).setUp()
301
        self.wt = self.make_branch_and_tree('.')
302
        self.b = self.wt.branch
303
        self.build_tree_contents([
304
            ('file1', 'file1 contents at rev 1\n'),
305
            ('file2', 'file2 contents at rev 1\n')
306
            ])
307
        self.wt.add(['file1', 'file2'])
308
        self.wt.commit(
309
            message='Revision 1',
310
            timestamp=1143849600, # 2006-04-01 00:00:00 UTC
311
            timezone=0,
312
            rev_id='rev-1')
313
        self.build_tree_contents([('file1', 'file1 contents at rev 2\n')])
314
        self.wt.commit(
315
            message='Revision 2',
316
            timestamp=1143936000, # 2006-04-02 00:00:00 UTC
317
            timezone=28800,
318
            rev_id='rev-2')
319
        self.build_tree_contents([('file2', 'file2 contents at rev 3\n')])
320
        self.wt.commit(
321
            message='Revision 3',
322
            timestamp=1144022400, # 2006-04-03 00:00:00 UTC
323
            timezone=-3600,
324
            rev_id='rev-3')
325
        self.wt.remove(['file2'])
326
        self.wt.commit(
327
            message='Revision 4',
328
            timestamp=1144108800, # 2006-04-04 00:00:00 UTC
329
            timezone=0,
330
            rev_id='rev-4')
331
        self.build_tree_contents([
332
            ('file1', 'file1 contents in working tree\n')
333
            ])
334
        # set the date stamps for files in the working tree to known values
335
        os.utime('file1', (1144195200, 1144195200)) # 2006-04-05 00:00:00 UTC
336
337
    def test_diff_rev_tree_working_tree(self):
338
        output = self.get_diff(self.wt.basis_tree(), self.wt)
339
        # note that the date for old/file1 is from rev 2 rather than from
340
        # the basis revision (rev 4)
341
        self.assertEqualDiff(output, '''\
342
=== modified file 'file1'
343
--- old/file1\t2006-04-02 00:00:00 +0000
344
+++ new/file1\t2006-04-05 00:00:00 +0000
345
@@ -1,1 +1,1 @@
346
-file1 contents at rev 2
347
+file1 contents in working tree
348
349
''')
350
351
    def test_diff_rev_tree_rev_tree(self):
352
        tree1 = self.b.repository.revision_tree('rev-2')
353
        tree2 = self.b.repository.revision_tree('rev-3')
354
        output = self.get_diff(tree1, tree2)
355
        self.assertEqualDiff(output, '''\
356
=== modified file 'file2'
357
--- old/file2\t2006-04-01 00:00:00 +0000
358
+++ new/file2\t2006-04-03 00:00:00 +0000
359
@@ -1,1 +1,1 @@
360
-file2 contents at rev 1
361
+file2 contents at rev 3
362
363
''')
364
        
365
    def test_diff_add_files(self):
366
        tree1 = self.b.repository.revision_tree(None)
367
        tree2 = self.b.repository.revision_tree('rev-1')
368
        output = self.get_diff(tree1, tree2)
369
        # the files have the epoch time stamp for the tree in which
370
        # they don't exist.
371
        self.assertEqualDiff(output, '''\
372
=== added file 'file1'
373
--- old/file1\t1970-01-01 00:00:00 +0000
374
+++ new/file1\t2006-04-01 00:00:00 +0000
375
@@ -0,0 +1,1 @@
376
+file1 contents at rev 1
377
378
=== added file 'file2'
379
--- old/file2\t1970-01-01 00:00:00 +0000
380
+++ new/file2\t2006-04-01 00:00:00 +0000
381
@@ -0,0 +1,1 @@
382
+file2 contents at rev 1
383
384
''')
385
386
    def test_diff_remove_files(self):
387
        tree1 = self.b.repository.revision_tree('rev-3')
388
        tree2 = self.b.repository.revision_tree('rev-4')
389
        output = self.get_diff(tree1, tree2)
390
        # the file has the epoch time stamp for the tree in which
391
        # it doesn't exist.
392
        self.assertEqualDiff(output, '''\
393
=== removed file 'file2'
394
--- old/file2\t2006-04-03 00:00:00 +0000
395
+++ new/file2\t1970-01-01 00:00:00 +0000
396
@@ -1,1 +0,0 @@
397
-file2 contents at rev 3
398
399
''')
400
1551.7.17 by Aaron Bentley
Switch to PathsNotVersioned, accept extra_trees
401
    def test_show_diff_specified(self):
1551.7.22 by Aaron Bentley
Changes from review
402
        """A working tree filename can be used to identify a file"""
1551.7.17 by Aaron Bentley
Switch to PathsNotVersioned, accept extra_trees
403
        self.wt.rename_one('file1', 'file1b')
404
        old_tree = self.b.repository.revision_tree('rev-1')
405
        new_tree = self.b.repository.revision_tree('rev-4')
1551.7.22 by Aaron Bentley
Changes from review
406
        out = self.get_diff(old_tree, new_tree, specific_files=['file1b'], 
407
                            working_tree=self.wt)
408
        self.assertContainsRe(out, 'file1\t')
1551.7.17 by Aaron Bentley
Switch to PathsNotVersioned, accept extra_trees
409
1551.7.22 by Aaron Bentley
Changes from review
410
    def test_recursive_diff(self):
411
        """Children of directories are matched"""
412
        os.mkdir('dir1')
413
        os.mkdir('dir2')
414
        self.wt.add(['dir1', 'dir2'])
415
        self.wt.rename_one('file1', 'dir1/file1')
416
        old_tree = self.b.repository.revision_tree('rev-1')
417
        new_tree = self.b.repository.revision_tree('rev-4')
418
        out = self.get_diff(old_tree, new_tree, specific_files=['dir1'], 
419
                            working_tree=self.wt)
420
        self.assertContainsRe(out, 'file1\t')
421
        out = self.get_diff(old_tree, new_tree, specific_files=['dir2'], 
422
                            working_tree=self.wt)
423
        self.assertNotContainsRe(out, 'file1\t')
1740.2.5 by Aaron Bentley
Merge from bzr.dev
424
1899.1.1 by John Arbash Meinel
Fix the bug in the NoDiff exception class, and add a test
425
2405.1.1 by John Arbash Meinel
Add a bunch of direct tests for 'show_diff_trees'
426
427
class TestShowDiffTrees(TestShowDiffTreesHelper):
428
    """Direct tests for show_diff_trees"""
429
430
    def test_modified_file(self):
431
        """Test when a file is modified."""
432
        tree = self.make_branch_and_tree('tree')
433
        self.build_tree_contents([('tree/file', 'contents\n')])
434
        tree.add(['file'], ['file-id'])
435
        tree.commit('one', rev_id='rev-1')
436
437
        self.build_tree_contents([('tree/file', 'new contents\n')])
438
        diff = self.get_diff(tree.basis_tree(), tree)
439
        self.assertContainsRe(diff, "=== modified file 'file'\n")
440
        self.assertContainsRe(diff, '--- old/file\t')
441
        self.assertContainsRe(diff, '\\+\\+\\+ new/file\t')
442
        self.assertContainsRe(diff, '-contents\n'
443
                                    '\\+new contents\n')
444
2405.1.2 by John Arbash Meinel
Fix bug #103870 by passing None instead of a (sometimes wrong) path
445
    def test_modified_file_in_renamed_dir(self):
446
        """Test when a file is modified in a renamed directory."""
447
        tree = self.make_branch_and_tree('tree')
448
        self.build_tree(['tree/dir/'])
449
        self.build_tree_contents([('tree/dir/file', 'contents\n')])
450
        tree.add(['dir', 'dir/file'], ['dir-id', 'file-id'])
451
        tree.commit('one', rev_id='rev-1')
452
453
        tree.rename_one('dir', 'other')
454
        self.build_tree_contents([('tree/other/file', 'new contents\n')])
455
        diff = self.get_diff(tree.basis_tree(), tree)
456
        self.assertContainsRe(diff, "=== renamed directory 'dir' => 'other'\n")
457
        self.assertContainsRe(diff, "=== modified file 'other/file'\n")
458
        # XXX: This is technically incorrect, because it used to be at another
459
        # location. What to do?
2405.1.3 by John Arbash Meinel
Do a better fix, which recognizes that we should pass the correct old path.
460
        self.assertContainsRe(diff, '--- old/dir/file\t')
2405.1.2 by John Arbash Meinel
Fix bug #103870 by passing None instead of a (sometimes wrong) path
461
        self.assertContainsRe(diff, '\\+\\+\\+ new/other/file\t')
462
        self.assertContainsRe(diff, '-contents\n'
463
                                    '\\+new contents\n')
464
2405.1.1 by John Arbash Meinel
Add a bunch of direct tests for 'show_diff_trees'
465
    def test_renamed_directory(self):
466
        """Test when only a directory is only renamed."""
467
        tree = self.make_branch_and_tree('tree')
468
        self.build_tree(['tree/dir/'])
469
        self.build_tree_contents([('tree/dir/file', 'contents\n')])
470
        tree.add(['dir', 'dir/file'], ['dir-id', 'file-id'])
471
        tree.commit('one', rev_id='rev-1')
472
473
        tree.rename_one('dir', 'newdir')
474
        diff = self.get_diff(tree.basis_tree(), tree)
475
        # Renaming a directory should be a single "you renamed this dir" even
476
        # when there are files inside.
477
        self.assertEqual("=== renamed directory 'dir' => 'newdir'\n", diff)
478
479
    def test_renamed_file(self):
480
        """Test when a file is only renamed."""
481
        tree = self.make_branch_and_tree('tree')
482
        self.build_tree_contents([('tree/file', 'contents\n')])
483
        tree.add(['file'], ['file-id'])
484
        tree.commit('one', rev_id='rev-1')
485
486
        tree.rename_one('file', 'newname')
487
        diff = self.get_diff(tree.basis_tree(), tree)
488
        self.assertContainsRe(diff, "=== renamed file 'file' => 'newname'\n")
489
        # We shouldn't have a --- or +++ line, because there is no content
490
        # change
491
        self.assertNotContainsRe(diff, '---')
492
493
    def test_renamed_and_modified_file(self):
494
        """Test when a file is only renamed."""
495
        tree = self.make_branch_and_tree('tree')
496
        self.build_tree_contents([('tree/file', 'contents\n')])
497
        tree.add(['file'], ['file-id'])
498
        tree.commit('one', rev_id='rev-1')
499
500
        tree.rename_one('file', 'newname')
501
        self.build_tree_contents([('tree/newname', 'new contents\n')])
502
        diff = self.get_diff(tree.basis_tree(), tree)
503
        self.assertContainsRe(diff, "=== renamed file 'file' => 'newname'\n")
504
        self.assertContainsRe(diff, '--- old/file\t')
505
        self.assertContainsRe(diff, '\\+\\+\\+ new/newname\t')
506
        self.assertContainsRe(diff, '-contents\n'
507
                                    '\\+new contents\n')
508
2592.2.1 by Jonathan Lange
Reproduce and fix bug 110092.
509
    def test_binary_unicode_filenames(self):
2592.2.2 by Jonathan Lange
Apply jam's comments to test_binary_unicode_filenames. Change the
510
        """Test that contents of files are *not* encoded in UTF-8 when there
511
        is a binary file in the diff.
2592.2.1 by Jonathan Lange
Reproduce and fix bug 110092.
512
        """
513
        # See https://bugs.launchpad.net/bugs/110092.
2592.2.4 by Jonathan Lange
Skip the unicode filename test if the platform doesn't support unicode
514
        self.requireFeature(UnicodeFilename)
2592.2.1 by Jonathan Lange
Reproduce and fix bug 110092.
515
516
        # This bug isn't triggered with cStringIO.
517
        from StringIO import StringIO
518
        tree = self.make_branch_and_tree('tree')
2592.2.2 by Jonathan Lange
Apply jam's comments to test_binary_unicode_filenames. Change the
519
        alpha, omega = u'\u03b1', u'\u03c9'
520
        alpha_utf8, omega_utf8 = alpha.encode('utf8'), omega.encode('utf8')
2592.2.1 by Jonathan Lange
Reproduce and fix bug 110092.
521
        self.build_tree_contents(
2592.2.2 by Jonathan Lange
Apply jam's comments to test_binary_unicode_filenames. Change the
522
            [('tree/' + alpha, chr(0)),
523
             ('tree/' + omega,
524
              ('The %s and the %s\n' % (alpha_utf8, omega_utf8)))])
525
        tree.add([alpha], ['file-id'])
526
        tree.add([omega], ['file-id-2'])
2592.2.1 by Jonathan Lange
Reproduce and fix bug 110092.
527
        diff_content = StringIO()
528
        show_diff_trees(tree.basis_tree(), tree, diff_content)
529
        diff = diff_content.getvalue()
2592.2.2 by Jonathan Lange
Apply jam's comments to test_binary_unicode_filenames. Change the
530
        self.assertContainsRe(diff, r"=== added file '%s'" % alpha_utf8)
2592.2.1 by Jonathan Lange
Reproduce and fix bug 110092.
531
        self.assertContainsRe(
2592.2.2 by Jonathan Lange
Apply jam's comments to test_binary_unicode_filenames. Change the
532
            diff, "Binary files a/%s.*and b/%s.* differ\n" % (alpha_utf8, alpha_utf8))
533
        self.assertContainsRe(diff, r"=== added file '%s'" % omega_utf8)
534
        self.assertContainsRe(diff, r"--- a/%s" % (omega_utf8,))
535
        self.assertContainsRe(diff, r"\+\+\+ b/%s" % (omega_utf8,))
2592.2.1 by Jonathan Lange
Reproduce and fix bug 110092.
536
2725.2.1 by ghigo
When a unicode filename is renamed, in the diff is showed a wrong result
537
    def test_unicode_filename(self):
538
        """Test when the filename are unicode."""
539
        self.requireFeature(UnicodeFilename)
540
541
        alpha, omega = u'\u03b1', u'\u03c9'
542
        autf8, outf8 = alpha.encode('utf8'), omega.encode('utf8')
543
544
        tree = self.make_branch_and_tree('tree')
545
        self.build_tree_contents([('tree/ren_'+alpha, 'contents\n')])
546
        tree.add(['ren_'+alpha], ['file-id-2'])
547
        self.build_tree_contents([('tree/del_'+alpha, 'contents\n')])
548
        tree.add(['del_'+alpha], ['file-id-3'])
549
        self.build_tree_contents([('tree/mod_'+alpha, 'contents\n')])
550
        tree.add(['mod_'+alpha], ['file-id-4'])
551
552
        tree.commit('one', rev_id='rev-1')
553
554
        tree.rename_one('ren_'+alpha, 'ren_'+omega)
555
        tree.remove('del_'+alpha)
556
        self.build_tree_contents([('tree/add_'+alpha, 'contents\n')])
557
        tree.add(['add_'+alpha], ['file-id'])
558
        self.build_tree_contents([('tree/mod_'+alpha, 'contents_mod\n')])
559
560
        diff = self.get_diff(tree.basis_tree(), tree)
561
        self.assertContainsRe(diff,
562
                "=== renamed file 'ren_%s' => 'ren_%s'\n"%(autf8, outf8))
563
        self.assertContainsRe(diff, "=== added file 'add_%s'"%autf8)
564
        self.assertContainsRe(diff, "=== modified file 'mod_%s'"%autf8)
565
        self.assertContainsRe(diff, "=== removed file 'del_%s'"%autf8)
2405.1.1 by John Arbash Meinel
Add a bunch of direct tests for 'show_diff_trees'
566
3009.2.9 by Aaron Bentley
Add tests for Differ
567
3009.2.22 by Aaron Bentley
Update names & docstring
568
class DiffWasIs(DiffPath):
3009.2.15 by Aaron Bentley
Test differ registration
569
570
    def diff(self, file_id, old_path, new_path, old_kind, new_kind):
571
        self.to_file.write('was: ')
572
        self.to_file.write(self.old_tree.get_file(file_id).read())
573
        self.to_file.write('is: ')
574
        self.to_file.write(self.new_tree.get_file(file_id).read())
575
        pass
576
577
3009.2.22 by Aaron Bentley
Update names & docstring
578
class TestDiffTree(TestCaseWithTransport):
3009.2.9 by Aaron Bentley
Add tests for Differ
579
580
    def setUp(self):
581
        TestCaseWithTransport.setUp(self)
582
        self.old_tree = self.make_branch_and_tree('old-tree')
583
        self.old_tree.lock_write()
584
        self.addCleanup(self.old_tree.unlock)
585
        self.new_tree = self.make_branch_and_tree('new-tree')
586
        self.new_tree.lock_write()
587
        self.addCleanup(self.new_tree.unlock)
3009.2.22 by Aaron Bentley
Update names & docstring
588
        self.differ = DiffTree(self.old_tree, self.new_tree, StringIO())
3009.2.9 by Aaron Bentley
Add tests for Differ
589
590
    def test_diff_text(self):
591
        self.build_tree_contents([('old-tree/olddir/',),
592
                                  ('old-tree/olddir/oldfile', 'old\n')])
593
        self.old_tree.add('olddir')
594
        self.old_tree.add('olddir/oldfile', 'file-id')
595
        self.build_tree_contents([('new-tree/newdir/',),
596
                                  ('new-tree/newdir/newfile', 'new\n')])
597
        self.new_tree.add('newdir')
598
        self.new_tree.add('newdir/newfile', 'file-id')
3009.2.22 by Aaron Bentley
Update names & docstring
599
        differ = DiffText(self.old_tree, self.new_tree, StringIO())
3009.2.11 by Aaron Bentley
Refactor diff to be more pluggable
600
        differ.diff_text('file-id', None, 'old label', 'new label')
3009.2.9 by Aaron Bentley
Add tests for Differ
601
        self.assertEqual(
602
            '--- old label\n+++ new label\n@@ -1,1 +0,0 @@\n-old\n\n',
3009.2.11 by Aaron Bentley
Refactor diff to be more pluggable
603
            differ.to_file.getvalue())
604
        differ.to_file.seek(0)
605
        differ.diff_text(None, 'file-id', 'old label', 'new label')
3009.2.9 by Aaron Bentley
Add tests for Differ
606
        self.assertEqual(
607
            '--- old label\n+++ new label\n@@ -0,0 +1,1 @@\n+new\n\n',
3009.2.11 by Aaron Bentley
Refactor diff to be more pluggable
608
            differ.to_file.getvalue())
609
        differ.to_file.seek(0)
610
        differ.diff_text('file-id', 'file-id', 'old label', 'new label')
3009.2.9 by Aaron Bentley
Add tests for Differ
611
        self.assertEqual(
612
            '--- old label\n+++ new label\n@@ -1,1 +1,1 @@\n-old\n+new\n\n',
3009.2.11 by Aaron Bentley
Refactor diff to be more pluggable
613
            differ.to_file.getvalue())
3009.2.9 by Aaron Bentley
Add tests for Differ
614
615
    def test_diff_symlink(self):
3009.2.22 by Aaron Bentley
Update names & docstring
616
        differ = DiffSymlink(self.old_tree, self.new_tree, StringIO())
3009.2.11 by Aaron Bentley
Refactor diff to be more pluggable
617
        differ.diff_symlink('old target', None)
3009.2.9 by Aaron Bentley
Add tests for Differ
618
        self.assertEqual("=== target was 'old target'\n",
3009.2.11 by Aaron Bentley
Refactor diff to be more pluggable
619
                         differ.to_file.getvalue())
3009.2.9 by Aaron Bentley
Add tests for Differ
620
3009.2.22 by Aaron Bentley
Update names & docstring
621
        differ = DiffSymlink(self.old_tree, self.new_tree, StringIO())
3009.2.11 by Aaron Bentley
Refactor diff to be more pluggable
622
        differ.diff_symlink(None, 'new target')
3009.2.9 by Aaron Bentley
Add tests for Differ
623
        self.assertEqual("=== target is 'new target'\n",
3009.2.11 by Aaron Bentley
Refactor diff to be more pluggable
624
                         differ.to_file.getvalue())
625
3009.2.22 by Aaron Bentley
Update names & docstring
626
        differ = DiffSymlink(self.old_tree, self.new_tree, StringIO())
3009.2.11 by Aaron Bentley
Refactor diff to be more pluggable
627
        differ.diff_symlink('old target', 'new target')
3009.2.9 by Aaron Bentley
Add tests for Differ
628
        self.assertEqual("=== target changed 'old target' => 'new target'\n",
3009.2.11 by Aaron Bentley
Refactor diff to be more pluggable
629
                         differ.to_file.getvalue())
3009.2.9 by Aaron Bentley
Add tests for Differ
630
631
    def test_diff(self):
632
        self.build_tree_contents([('old-tree/olddir/',),
633
                                  ('old-tree/olddir/oldfile', 'old\n')])
634
        self.old_tree.add('olddir')
635
        self.old_tree.add('olddir/oldfile', 'file-id')
636
        self.build_tree_contents([('new-tree/newdir/',),
637
                                  ('new-tree/newdir/newfile', 'new\n')])
638
        self.new_tree.add('newdir')
639
        self.new_tree.add('newdir/newfile', 'file-id')
3009.2.12 by Aaron Bentley
Associate labels with text diffing only
640
        self.differ.diff('file-id', 'olddir/oldfile', 'newdir/newfile')
3009.2.9 by Aaron Bentley
Add tests for Differ
641
        self.assertContainsRe(
642
            self.differ.to_file.getvalue(),
643
            r'--- olddir/oldfile.*\n\+\+\+ newdir/newfile.*\n\@\@ -1,1 \+1,1'
644
             ' \@\@\n-old\n\+new\n\n')
645
646
    def test_diff_kind_change(self):
647
        self.build_tree_contents([('old-tree/olddir/',),
648
                                  ('old-tree/olddir/oldfile', 'old\n')])
649
        self.old_tree.add('olddir')
650
        self.old_tree.add('olddir/oldfile', 'file-id')
651
        self.build_tree(['new-tree/newdir/'])
652
        os.symlink('new', 'new-tree/newdir/newfile')
653
        self.new_tree.add('newdir')
654
        self.new_tree.add('newdir/newfile', 'file-id')
3009.2.12 by Aaron Bentley
Associate labels with text diffing only
655
        self.differ.diff('file-id', 'olddir/oldfile', 'newdir/newfile')
3009.2.9 by Aaron Bentley
Add tests for Differ
656
        self.assertContainsRe(
657
            self.differ.to_file.getvalue(),
658
            r'--- olddir/oldfile.*\n\+\+\+ newdir/newfile.*\n\@\@ -1,1 \+0,0'
659
             ' \@\@\n-old\n\n')
660
        self.assertContainsRe(self.differ.to_file.getvalue(),
661
                              "=== target is 'new'\n")
662
3009.2.19 by Aaron Bentley
Implement directory diffing
663
    def test_diff_directory(self):
664
        self.build_tree(['new-tree/new-dir/'])
665
        self.new_tree.add('new-dir', 'new-dir-id')
666
        self.differ.diff('new-dir-id', None, 'new-dir')
667
        self.assertEqual(self.differ.to_file.getvalue(), '')
668
3009.2.16 by Aaron Bentley
Test support for extra differs
669
    def create_old_new(self):
670
        self.build_tree_contents([('old-tree/olddir/',),
671
                                  ('old-tree/olddir/oldfile', 'old\n')])
672
        self.old_tree.add('olddir')
673
        self.old_tree.add('olddir/oldfile', 'file-id')
674
        self.build_tree_contents([('new-tree/newdir/',),
675
                                  ('new-tree/newdir/newfile', 'new\n')])
676
        self.new_tree.add('newdir')
677
        self.new_tree.add('newdir/newfile', 'file-id')
678
3009.2.27 by Aaron Bentley
Use extra_factories instead of extra_diffs
679
    def test_register_diff(self):
3009.2.16 by Aaron Bentley
Test support for extra differs
680
        self.create_old_new()
3009.2.27 by Aaron Bentley
Use extra_factories instead of extra_diffs
681
        old_diff_factories = DiffTree.diff_factories
682
        DiffTree.diff_factories=old_diff_factories[:]
3009.2.28 by Aaron Bentley
Add from_diff_tree factories
683
        DiffTree.diff_factories.insert(0, DiffWasIs.from_diff_tree)
3009.2.16 by Aaron Bentley
Test support for extra differs
684
        try:
3009.2.22 by Aaron Bentley
Update names & docstring
685
            differ = DiffTree(self.old_tree, self.new_tree, StringIO())
3009.2.16 by Aaron Bentley
Test support for extra differs
686
        finally:
3009.2.27 by Aaron Bentley
Use extra_factories instead of extra_diffs
687
            DiffTree.diff_factories = old_diff_factories
3009.2.16 by Aaron Bentley
Test support for extra differs
688
        differ.diff('file-id', 'olddir/oldfile', 'newdir/newfile')
689
        self.assertNotContainsRe(
690
            differ.to_file.getvalue(),
691
            r'--- olddir/oldfile.*\n\+\+\+ newdir/newfile.*\n\@\@ -1,1 \+1,1'
692
             ' \@\@\n-old\n\+new\n\n')
693
        self.assertContainsRe(differ.to_file.getvalue(),
694
                              'was: old\nis: new\n')
695
3009.2.27 by Aaron Bentley
Use extra_factories instead of extra_diffs
696
    def test_extra_factories(self):
3009.2.16 by Aaron Bentley
Test support for extra differs
697
        self.create_old_new()
3009.2.27 by Aaron Bentley
Use extra_factories instead of extra_diffs
698
        differ = DiffTree(self.old_tree, self.new_tree, StringIO(),
3009.2.28 by Aaron Bentley
Add from_diff_tree factories
699
                            extra_factories=[DiffWasIs.from_diff_tree])
3009.2.16 by Aaron Bentley
Test support for extra differs
700
        differ.diff('file-id', 'olddir/oldfile', 'newdir/newfile')
701
        self.assertNotContainsRe(
702
            differ.to_file.getvalue(),
703
            r'--- olddir/oldfile.*\n\+\+\+ newdir/newfile.*\n\@\@ -1,1 \+1,1'
704
             ' \@\@\n-old\n\+new\n\n')
705
        self.assertContainsRe(differ.to_file.getvalue(),
706
                              'was: old\nis: new\n')
707
3009.2.9 by Aaron Bentley
Add tests for Differ
708
1711.2.15 by John Arbash Meinel
Found a couple CDV left
709
class TestPatienceDiffLib(TestCase):
1185.81.1 by John Arbash Meinel
Adding nofrillsprecisemerge's diff algorithm, wrapped in difflib.
710
2781.1.1 by Martin Pool
merge cpatiencediff from Lukas
711
    def setUp(self):
712
        super(TestPatienceDiffLib, self).setUp()
713
        self._unique_lcs = bzrlib._patiencediff_py.unique_lcs_py
714
        self._recurse_matches = bzrlib._patiencediff_py.recurse_matches_py
715
        self._PatienceSequenceMatcher = \
716
            bzrlib._patiencediff_py.PatienceSequenceMatcher_py
717
1185.81.9 by John Arbash Meinel
Added (failing) tests for cdv.recurse_matches with common sections,
718
    def test_unique_lcs(self):
2781.1.1 by Martin Pool
merge cpatiencediff from Lukas
719
        unique_lcs = self._unique_lcs
1185.81.9 by John Arbash Meinel
Added (failing) tests for cdv.recurse_matches with common sections,
720
        self.assertEquals(unique_lcs('', ''), [])
2781.1.1 by Martin Pool
merge cpatiencediff from Lukas
721
        self.assertEquals(unique_lcs('', 'a'), [])
722
        self.assertEquals(unique_lcs('a', ''), [])
1185.81.9 by John Arbash Meinel
Added (failing) tests for cdv.recurse_matches with common sections,
723
        self.assertEquals(unique_lcs('a', 'a'), [(0,0)])
724
        self.assertEquals(unique_lcs('a', 'b'), [])
725
        self.assertEquals(unique_lcs('ab', 'ab'), [(0,0), (1,1)])
726
        self.assertEquals(unique_lcs('abcde', 'cdeab'), [(2,0), (3,1), (4,2)])
727
        self.assertEquals(unique_lcs('cdeab', 'abcde'), [(0,2), (1,3), (2,4)])
728
        self.assertEquals(unique_lcs('abXde', 'abYde'), [(0,0), (1,1), 
729
                                                         (3,3), (4,4)])
730
        self.assertEquals(unique_lcs('acbac', 'abc'), [(2,1)])
731
732
    def test_recurse_matches(self):
733
        def test_one(a, b, matches):
734
            test_matches = []
2781.1.1 by Martin Pool
merge cpatiencediff from Lukas
735
            self._recurse_matches(
736
                a, b, 0, 0, len(a), len(b), test_matches, 10)
1185.81.9 by John Arbash Meinel
Added (failing) tests for cdv.recurse_matches with common sections,
737
            self.assertEquals(test_matches, matches)
738
1711.2.17 by John Arbash Meinel
Small cleanups to patience_diff code.
739
        test_one(['a', '', 'b', '', 'c'], ['a', 'a', 'b', 'c', 'c'],
1185.81.9 by John Arbash Meinel
Added (failing) tests for cdv.recurse_matches with common sections,
740
                 [(0, 0), (2, 2), (4, 4)])
741
        test_one(['a', 'c', 'b', 'a', 'c'], ['a', 'b', 'c'],
742
                 [(0, 0), (2, 1), (4, 2)])
2781.1.1 by Martin Pool
merge cpatiencediff from Lukas
743
        # Even though 'bc' is not unique globally, and is surrounded by
744
        # non-matching lines, we should still match, because they are locally
745
        # unique
746
        test_one('abcdbce', 'afbcgdbce', [(0,0), (1, 2), (2, 3), (3, 5),
747
                                          (4, 6), (5, 7), (6, 8)])
1185.81.9 by John Arbash Meinel
Added (failing) tests for cdv.recurse_matches with common sections,
748
1185.81.14 by John Arbash Meinel
Added a main function for running cdvdifflib manually, included tests for unified_diff interfaces
749
        # recurse_matches doesn't match non-unique 
750
        # lines surrounded by bogus text.
1185.81.24 by Aaron Bentley
Reoganize patience-related code
751
        # The update has been done in patiencediff.SequenceMatcher instead
1185.81.14 by John Arbash Meinel
Added a main function for running cdvdifflib manually, included tests for unified_diff interfaces
752
753
        # This is what it could be
1185.81.9 by John Arbash Meinel
Added (failing) tests for cdv.recurse_matches with common sections,
754
        #test_one('aBccDe', 'abccde', [(0,0), (2,2), (3,3), (5,5)])
1185.81.14 by John Arbash Meinel
Added a main function for running cdvdifflib manually, included tests for unified_diff interfaces
755
1185.81.9 by John Arbash Meinel
Added (failing) tests for cdv.recurse_matches with common sections,
756
        # This is what it currently gives:
757
        test_one('aBccDe', 'abccde', [(0,0), (5,5)])
758
1185.81.1 by John Arbash Meinel
Adding nofrillsprecisemerge's diff algorithm, wrapped in difflib.
759
    def test_matching_blocks(self):
1711.2.10 by John Arbash Meinel
Clarify the patience tests a little bit.
760
        def chk_blocks(a, b, expected_blocks):
1185.81.1 by John Arbash Meinel
Adding nofrillsprecisemerge's diff algorithm, wrapped in difflib.
761
            # difflib always adds a signature of the total
762
            # length, with no matching entries at the end
2781.1.1 by Martin Pool
merge cpatiencediff from Lukas
763
            s = self._PatienceSequenceMatcher(None, a, b)
1185.81.11 by John Arbash Meinel
Found some edge cases that weren't being matched.
764
            blocks = s.get_matching_blocks()
1711.2.10 by John Arbash Meinel
Clarify the patience tests a little bit.
765
            self.assertEquals((len(a), len(b), 0), blocks[-1])
766
            self.assertEquals(expected_blocks, blocks[:-1])
1185.81.1 by John Arbash Meinel
Adding nofrillsprecisemerge's diff algorithm, wrapped in difflib.
767
1185.81.2 by John Arbash Meinel
A couple small tests.
768
        # Some basic matching tests
1185.81.1 by John Arbash Meinel
Adding nofrillsprecisemerge's diff algorithm, wrapped in difflib.
769
        chk_blocks('', '', [])
770
        chk_blocks([], [], [])
2781.1.1 by Martin Pool
merge cpatiencediff from Lukas
771
        chk_blocks('abc', '', [])
772
        chk_blocks('', 'abc', [])
1185.81.1 by John Arbash Meinel
Adding nofrillsprecisemerge's diff algorithm, wrapped in difflib.
773
        chk_blocks('abcd', 'abcd', [(0, 0, 4)])
774
        chk_blocks('abcd', 'abce', [(0, 0, 3)])
775
        chk_blocks('eabc', 'abce', [(1, 0, 3)])
776
        chk_blocks('eabce', 'abce', [(1, 0, 4)])
777
        chk_blocks('abcde', 'abXde', [(0, 0, 2), (3, 3, 2)])
1185.81.3 by John Arbash Meinel
Adding tests for checking opcodes.
778
        chk_blocks('abcde', 'abXYZde', [(0, 0, 2), (3, 5, 2)])
779
        chk_blocks('abde', 'abXYZde', [(0, 0, 2), (2, 5, 2)])
780
        # This may check too much, but it checks to see that 
781
        # a copied block stays attached to the previous section,
782
        # not the later one.
783
        # difflib would tend to grab the trailing longest match
784
        # which would make the diff not look right
1185.81.9 by John Arbash Meinel
Added (failing) tests for cdv.recurse_matches with common sections,
785
        chk_blocks('abcdefghijklmnop', 'abcdefxydefghijklmnop',
786
                   [(0, 0, 6), (6, 11, 10)])
1185.81.1 by John Arbash Meinel
Adding nofrillsprecisemerge's diff algorithm, wrapped in difflib.
787
1185.81.2 by John Arbash Meinel
A couple small tests.
788
        # make sure it supports passing in lists
789
        chk_blocks(
1185.81.29 by Aaron Bentley
Fix style issues and duplicated tests
790
                   ['hello there\n',
791
                    'world\n',
792
                    'how are you today?\n'],
793
                   ['hello there\n',
794
                    'how are you today?\n'],
1185.81.2 by John Arbash Meinel
A couple small tests.
795
                [(0, 0, 1), (2, 1, 1)])
1185.81.1 by John Arbash Meinel
Adding nofrillsprecisemerge's diff algorithm, wrapped in difflib.
796
1711.2.21 by John Arbash Meinel
Cleanup patiencediff, remove the use of difflib.SequenceMatcher.
797
        # non unique lines surrounded by non-matching lines
798
        # won't be found
799
        chk_blocks('aBccDe', 'abccde', [(0,0,1), (5,5,1)])
800
801
        # But they only need to be locally unique
802
        chk_blocks('aBcDec', 'abcdec', [(0,0,1), (2,2,1), (4,4,2)])
803
804
        # non unique blocks won't be matched
805
        chk_blocks('aBcdEcdFg', 'abcdecdfg', [(0,0,1), (8,8,1)])
806
807
        # but locally unique ones will
808
        chk_blocks('aBcdEeXcdFg', 'abcdecdfg', [(0,0,1), (2,2,2),
809
                                              (5,4,1), (7,5,2), (10,8,1)])
810
811
        chk_blocks('abbabbXd', 'cabbabxd', [(7,7,1)])
812
        chk_blocks('abbabbbb', 'cabbabbc', [])
813
        chk_blocks('bbbbbbbb', 'cbbbbbbc', [])
1185.81.11 by John Arbash Meinel
Found some edge cases that weren't being matched.
814
1185.81.3 by John Arbash Meinel
Adding tests for checking opcodes.
815
    def test_opcodes(self):
1711.2.10 by John Arbash Meinel
Clarify the patience tests a little bit.
816
        def chk_ops(a, b, expected_codes):
2781.1.1 by Martin Pool
merge cpatiencediff from Lukas
817
            s = self._PatienceSequenceMatcher(None, a, b)
1711.2.10 by John Arbash Meinel
Clarify the patience tests a little bit.
818
            self.assertEquals(expected_codes, s.get_opcodes())
1185.81.3 by John Arbash Meinel
Adding tests for checking opcodes.
819
820
        chk_ops('', '', [])
821
        chk_ops([], [], [])
2781.1.1 by Martin Pool
merge cpatiencediff from Lukas
822
        chk_ops('abc', '', [('delete', 0,3, 0,0)])
823
        chk_ops('', 'abc', [('insert', 0,0, 0,3)])
1185.81.9 by John Arbash Meinel
Added (failing) tests for cdv.recurse_matches with common sections,
824
        chk_ops('abcd', 'abcd', [('equal',    0,4, 0,4)])
1185.81.29 by Aaron Bentley
Fix style issues and duplicated tests
825
        chk_ops('abcd', 'abce', [('equal',   0,3, 0,3),
826
                                 ('replace', 3,4, 3,4)
827
                                ])
828
        chk_ops('eabc', 'abce', [('delete', 0,1, 0,0),
829
                                 ('equal',  1,4, 0,3),
830
                                 ('insert', 4,4, 3,4)
831
                                ])
832
        chk_ops('eabce', 'abce', [('delete', 0,1, 0,0),
833
                                  ('equal',  1,5, 0,4)
1185.81.3 by John Arbash Meinel
Adding tests for checking opcodes.
834
                                 ])
1185.81.29 by Aaron Bentley
Fix style issues and duplicated tests
835
        chk_ops('abcde', 'abXde', [('equal',   0,2, 0,2),
836
                                   ('replace', 2,3, 2,3),
837
                                   ('equal',   3,5, 3,5)
1185.81.3 by John Arbash Meinel
Adding tests for checking opcodes.
838
                                  ])
1185.81.29 by Aaron Bentley
Fix style issues and duplicated tests
839
        chk_ops('abcde', 'abXYZde', [('equal',   0,2, 0,2),
840
                                     ('replace', 2,3, 2,5),
841
                                     ('equal',   3,5, 5,7)
1185.81.3 by John Arbash Meinel
Adding tests for checking opcodes.
842
                                    ])
1185.81.29 by Aaron Bentley
Fix style issues and duplicated tests
843
        chk_ops('abde', 'abXYZde', [('equal',  0,2, 0,2),
844
                                    ('insert', 2,2, 2,5),
845
                                    ('equal',  2,4, 5,7)
1185.81.3 by John Arbash Meinel
Adding tests for checking opcodes.
846
                                   ])
847
        chk_ops('abcdefghijklmnop', 'abcdefxydefghijklmnop',
1185.81.29 by Aaron Bentley
Fix style issues and duplicated tests
848
                [('equal',  0,6,  0,6),
849
                 ('insert', 6,6,  6,11),
850
                 ('equal',  6,16, 11,21)
1185.81.3 by John Arbash Meinel
Adding tests for checking opcodes.
851
                ])
852
        chk_ops(
853
                [ 'hello there\n'
854
                , 'world\n'
855
                , 'how are you today?\n'],
856
                [ 'hello there\n'
857
                , 'how are you today?\n'],
1185.81.29 by Aaron Bentley
Fix style issues and duplicated tests
858
                [('equal',  0,1, 0,1),
859
                 ('delete', 1,2, 1,1),
1711.2.21 by John Arbash Meinel
Cleanup patiencediff, remove the use of difflib.SequenceMatcher.
860
                 ('equal',  2,3, 1,2),
1185.81.9 by John Arbash Meinel
Added (failing) tests for cdv.recurse_matches with common sections,
861
                ])
862
        chk_ops('aBccDe', 'abccde', 
1185.81.29 by Aaron Bentley
Fix style issues and duplicated tests
863
                [('equal',   0,1, 0,1),
1711.2.21 by John Arbash Meinel
Cleanup patiencediff, remove the use of difflib.SequenceMatcher.
864
                 ('replace', 1,5, 1,5),
865
                 ('equal',   5,6, 5,6),
866
                ])
867
        chk_ops('aBcDec', 'abcdec', 
868
                [('equal',   0,1, 0,1),
1185.81.29 by Aaron Bentley
Fix style issues and duplicated tests
869
                 ('replace', 1,2, 1,2),
1711.2.21 by John Arbash Meinel
Cleanup patiencediff, remove the use of difflib.SequenceMatcher.
870
                 ('equal',   2,3, 2,3),
871
                 ('replace', 3,4, 3,4),
872
                 ('equal',   4,6, 4,6),
1185.81.3 by John Arbash Meinel
Adding tests for checking opcodes.
873
                ])
1185.81.10 by John Arbash Meinel
Added some more test cases.
874
        chk_ops('aBcdEcdFg', 'abcdecdfg', 
1185.81.29 by Aaron Bentley
Fix style issues and duplicated tests
875
                [('equal',   0,1, 0,1),
1711.2.21 by John Arbash Meinel
Cleanup patiencediff, remove the use of difflib.SequenceMatcher.
876
                 ('replace', 1,8, 1,8),
1185.81.29 by Aaron Bentley
Fix style issues and duplicated tests
877
                 ('equal',   8,9, 8,9)
1185.81.10 by John Arbash Meinel
Added some more test cases.
878
                ])
1711.2.21 by John Arbash Meinel
Cleanup patiencediff, remove the use of difflib.SequenceMatcher.
879
        chk_ops('aBcdEeXcdFg', 'abcdecdfg', 
880
                [('equal',   0,1, 0,1),
881
                 ('replace', 1,2, 1,2),
882
                 ('equal',   2,4, 2,4),
883
                 ('delete', 4,5, 4,4),
884
                 ('equal',   5,6, 4,5),
885
                 ('delete', 6,7, 5,5),
886
                 ('equal',   7,9, 5,7),
887
                 ('replace', 9,10, 7,8),
888
                 ('equal',   10,11, 8,9)
889
                ])
1185.81.10 by John Arbash Meinel
Added some more test cases.
890
2781.1.1 by Martin Pool
merge cpatiencediff from Lukas
891
    def test_grouped_opcodes(self):
892
        def chk_ops(a, b, expected_codes, n=3):
893
            s = self._PatienceSequenceMatcher(None, a, b)
894
            self.assertEquals(expected_codes, list(s.get_grouped_opcodes(n)))
895
896
        chk_ops('', '', [])
897
        chk_ops([], [], [])
898
        chk_ops('abc', '', [[('delete', 0,3, 0,0)]])
899
        chk_ops('', 'abc', [[('insert', 0,0, 0,3)]])
900
        chk_ops('abcd', 'abcd', [])
901
        chk_ops('abcd', 'abce', [[('equal',   0,3, 0,3),
902
                                  ('replace', 3,4, 3,4)
903
                                 ]])
904
        chk_ops('eabc', 'abce', [[('delete', 0,1, 0,0),
905
                                 ('equal',  1,4, 0,3),
906
                                 ('insert', 4,4, 3,4)
907
                                ]])
908
        chk_ops('abcdefghijklmnop', 'abcdefxydefghijklmnop',
909
                [[('equal',  3,6, 3,6),
910
                  ('insert', 6,6, 6,11),
911
                  ('equal',  6,9, 11,14)
912
                  ]])
913
        chk_ops('abcdefghijklmnop', 'abcdefxydefghijklmnop',
914
                [[('equal',  2,6, 2,6),
915
                  ('insert', 6,6, 6,11),
916
                  ('equal',  6,10, 11,15)
917
                  ]], 4)
918
        chk_ops('Xabcdef', 'abcdef',
919
                [[('delete', 0,1, 0,0),
920
                  ('equal',  1,4, 0,3)
921
                  ]])
922
        chk_ops('abcdef', 'abcdefX',
923
                [[('equal',  3,6, 3,6),
924
                  ('insert', 6,6, 6,7)
925
                  ]])
926
927
1185.81.16 by John Arbash Meinel
Added tests, and an assert check to make sure ranges are always increasing.
928
    def test_multiple_ranges(self):
929
        # There was an earlier bug where we used a bad set of ranges,
930
        # this triggers that specific bug, to make sure it doesn't regress
1711.2.10 by John Arbash Meinel
Clarify the patience tests a little bit.
931
        def chk_blocks(a, b, expected_blocks):
1185.81.16 by John Arbash Meinel
Added tests, and an assert check to make sure ranges are always increasing.
932
            # difflib always adds a signature of the total
933
            # length, with no matching entries at the end
2781.1.1 by Martin Pool
merge cpatiencediff from Lukas
934
            s = self._PatienceSequenceMatcher(None, a, b)
1185.81.16 by John Arbash Meinel
Added tests, and an assert check to make sure ranges are always increasing.
935
            blocks = s.get_matching_blocks()
936
            x = blocks.pop()
937
            self.assertEquals(x, (len(a), len(b), 0))
1711.2.10 by John Arbash Meinel
Clarify the patience tests a little bit.
938
            self.assertEquals(expected_blocks, blocks)
1185.81.16 by John Arbash Meinel
Added tests, and an assert check to make sure ranges are always increasing.
939
940
        chk_blocks('abcdefghijklmnop'
941
                 , 'abcXghiYZQRSTUVWXYZijklmnop'
942
                 , [(0, 0, 3), (6, 4, 3), (9, 20, 7)])
943
944
        chk_blocks('ABCd efghIjk  L'
945
                 , 'AxyzBCn mo pqrstuvwI1 2  L'
1711.2.21 by John Arbash Meinel
Cleanup patiencediff, remove the use of difflib.SequenceMatcher.
946
                 , [(0,0,1), (1, 4, 2), (9, 19, 1), (12, 23, 3)])
1185.81.16 by John Arbash Meinel
Added tests, and an assert check to make sure ranges are always increasing.
947
1711.2.8 by John Arbash Meinel
rot13 the code snippet to help with clarity.
948
        # These are rot13 code snippets.
1185.81.16 by John Arbash Meinel
Added tests, and an assert check to make sure ranges are always increasing.
949
        chk_blocks('''\
1711.2.8 by John Arbash Meinel
rot13 the code snippet to help with clarity.
950
    trg nqqrq jura lbh nqq n svyr va gur qverpgbel.
951
    """
952
    gnxrf_netf = ['svyr*']
953
    gnxrf_bcgvbaf = ['ab-erphefr']
954
  
955
    qrs eha(frys, svyr_yvfg, ab_erphefr=Snyfr):
956
        sebz omeyvo.nqq vzcbeg fzneg_nqq, nqq_ercbegre_cevag, nqq_ercbegre_ahyy
957
        vs vf_dhvrg():
958
            ercbegre = nqq_ercbegre_ahyy
959
        ryfr:
960
            ercbegre = nqq_ercbegre_cevag
961
        fzneg_nqq(svyr_yvfg, abg ab_erphefr, ercbegre)
962
963
964
pynff pzq_zxqve(Pbzznaq):
965
'''.splitlines(True), '''\
966
    trg nqqrq jura lbh nqq n svyr va gur qverpgbel.
967
968
    --qel-eha jvyy fubj juvpu svyrf jbhyq or nqqrq, ohg abg npghnyyl 
969
    nqq gurz.
970
    """
971
    gnxrf_netf = ['svyr*']
972
    gnxrf_bcgvbaf = ['ab-erphefr', 'qel-eha']
973
974
    qrs eha(frys, svyr_yvfg, ab_erphefr=Snyfr, qel_eha=Snyfr):
975
        vzcbeg omeyvo.nqq
976
977
        vs qel_eha:
978
            vs vf_dhvrg():
979
                # Guvf vf cbvagyrff, ohg V'q engure abg envfr na reebe
980
                npgvba = omeyvo.nqq.nqq_npgvba_ahyy
981
            ryfr:
982
  npgvba = omeyvo.nqq.nqq_npgvba_cevag
983
        ryvs vf_dhvrg():
984
            npgvba = omeyvo.nqq.nqq_npgvba_nqq
985
        ryfr:
986
       npgvba = omeyvo.nqq.nqq_npgvba_nqq_naq_cevag
987
988
        omeyvo.nqq.fzneg_nqq(svyr_yvfg, abg ab_erphefr, npgvba)
989
990
991
pynff pzq_zxqve(Pbzznaq):
1185.81.16 by John Arbash Meinel
Added tests, and an assert check to make sure ranges are always increasing.
992
'''.splitlines(True)
993
, [(0,0,1), (1, 4, 2), (9, 19, 1), (12, 23, 3)])
994
1711.2.9 by John Arbash Meinel
Rename cdv => patience
995
    def test_patience_unified_diff(self):
1185.81.29 by Aaron Bentley
Fix style issues and duplicated tests
996
        txt_a = ['hello there\n',
997
                 'world\n',
998
                 'how are you today?\n']
999
        txt_b = ['hello there\n',
1000
                 'how are you today?\n']
1711.2.20 by John Arbash Meinel
Late bind to patiencediff objects to make it easier to plug-in
1001
        unified_diff = bzrlib.patiencediff.unified_diff
2781.1.1 by Martin Pool
merge cpatiencediff from Lukas
1002
        psm = self._PatienceSequenceMatcher
1185.81.29 by Aaron Bentley
Fix style issues and duplicated tests
1003
        self.assertEquals([ '---  \n',
1004
                           '+++  \n',
1005
                           '@@ -1,3 +1,2 @@\n',
1006
                           ' hello there\n',
1007
                           '-world\n',
1008
                           ' how are you today?\n'
1185.81.14 by John Arbash Meinel
Added a main function for running cdvdifflib manually, included tests for unified_diff interfaces
1009
                          ]
1711.2.20 by John Arbash Meinel
Late bind to patiencediff objects to make it easier to plug-in
1010
                          , list(unified_diff(txt_a, txt_b,
1011
                                 sequencematcher=psm)))
1185.81.14 by John Arbash Meinel
Added a main function for running cdvdifflib manually, included tests for unified_diff interfaces
1012
        txt_a = map(lambda x: x+'\n', 'abcdefghijklmnop')
1013
        txt_b = map(lambda x: x+'\n', 'abcdefxydefghijklmnop')
1014
        # This is the result with LongestCommonSubstring matching
1185.81.29 by Aaron Bentley
Fix style issues and duplicated tests
1015
        self.assertEquals(['---  \n',
1016
                           '+++  \n',
1017
                           '@@ -1,6 +1,11 @@\n',
1018
                           ' a\n',
1019
                           ' b\n',
1020
                           ' c\n',
1021
                           '+d\n',
1022
                           '+e\n',
1023
                           '+f\n',
1024
                           '+x\n',
1025
                           '+y\n',
1026
                           ' d\n',
1027
                           ' e\n',
1028
                           ' f\n']
1185.81.14 by John Arbash Meinel
Added a main function for running cdvdifflib manually, included tests for unified_diff interfaces
1029
                          , list(unified_diff(txt_a, txt_b)))
1711.2.9 by John Arbash Meinel
Rename cdv => patience
1030
        # And the patience diff
1185.81.29 by Aaron Bentley
Fix style issues and duplicated tests
1031
        self.assertEquals(['---  \n',
1032
                           '+++  \n',
1033
                           '@@ -4,6 +4,11 @@\n',
1034
                           ' d\n',
1035
                           ' e\n',
1036
                           ' f\n',
1037
                           '+x\n',
1038
                           '+y\n',
1039
                           '+d\n',
1040
                           '+e\n',
1041
                           '+f\n',
1042
                           ' g\n',
1043
                           ' h\n',
1044
                           ' i\n',
1185.81.14 by John Arbash Meinel
Added a main function for running cdvdifflib manually, included tests for unified_diff interfaces
1045
                          ]
1185.81.25 by Aaron Bentley
Clean up test_diff
1046
                          , list(unified_diff(txt_a, txt_b,
1711.2.20 by John Arbash Meinel
Late bind to patiencediff objects to make it easier to plug-in
1047
                                 sequencematcher=psm)))
1185.81.25 by Aaron Bentley
Clean up test_diff
1048
1185.81.14 by John Arbash Meinel
Added a main function for running cdvdifflib manually, included tests for unified_diff interfaces
1049
2781.1.1 by Martin Pool
merge cpatiencediff from Lukas
1050
class TestPatienceDiffLib_c(TestPatienceDiffLib):
1051
1052
    _test_needs_features = [CompiledPatienceDiffFeature]
1053
1054
    def setUp(self):
1055
        super(TestPatienceDiffLib_c, self).setUp()
1056
        import bzrlib._patiencediff_c
1057
        self._unique_lcs = bzrlib._patiencediff_c.unique_lcs_c
1058
        self._recurse_matches = bzrlib._patiencediff_c.recurse_matches_c
1059
        self._PatienceSequenceMatcher = \
1060
            bzrlib._patiencediff_c.PatienceSequenceMatcher_c
1061
1062
1711.2.15 by John Arbash Meinel
Found a couple CDV left
1063
class TestPatienceDiffLibFiles(TestCaseInTempDir):
1185.81.14 by John Arbash Meinel
Added a main function for running cdvdifflib manually, included tests for unified_diff interfaces
1064
2781.1.1 by Martin Pool
merge cpatiencediff from Lukas
1065
    def setUp(self):
1066
        super(TestPatienceDiffLibFiles, self).setUp()
1067
        self._PatienceSequenceMatcher = \
1068
            bzrlib._patiencediff_py.PatienceSequenceMatcher_py
1069
1711.2.9 by John Arbash Meinel
Rename cdv => patience
1070
    def test_patience_unified_diff_files(self):
1185.81.29 by Aaron Bentley
Fix style issues and duplicated tests
1071
        txt_a = ['hello there\n',
1072
                 'world\n',
1073
                 'how are you today?\n']
1074
        txt_b = ['hello there\n',
1075
                 'how are you today?\n']
1185.81.14 by John Arbash Meinel
Added a main function for running cdvdifflib manually, included tests for unified_diff interfaces
1076
        open('a1', 'wb').writelines(txt_a)
1077
        open('b1', 'wb').writelines(txt_b)
1078
1711.2.20 by John Arbash Meinel
Late bind to patiencediff objects to make it easier to plug-in
1079
        unified_diff_files = bzrlib.patiencediff.unified_diff_files
2781.1.1 by Martin Pool
merge cpatiencediff from Lukas
1080
        psm = self._PatienceSequenceMatcher
1185.81.29 by Aaron Bentley
Fix style issues and duplicated tests
1081
        self.assertEquals(['--- a1 \n',
1082
                           '+++ b1 \n',
1083
                           '@@ -1,3 +1,2 @@\n',
1084
                           ' hello there\n',
1085
                           '-world\n',
1086
                           ' how are you today?\n',
1185.81.14 by John Arbash Meinel
Added a main function for running cdvdifflib manually, included tests for unified_diff interfaces
1087
                          ]
1185.81.25 by Aaron Bentley
Clean up test_diff
1088
                          , list(unified_diff_files('a1', 'b1',
1711.2.20 by John Arbash Meinel
Late bind to patiencediff objects to make it easier to plug-in
1089
                                 sequencematcher=psm)))
1185.81.14 by John Arbash Meinel
Added a main function for running cdvdifflib manually, included tests for unified_diff interfaces
1090
1091
        txt_a = map(lambda x: x+'\n', 'abcdefghijklmnop')
1092
        txt_b = map(lambda x: x+'\n', 'abcdefxydefghijklmnop')
1093
        open('a2', 'wb').writelines(txt_a)
1094
        open('b2', 'wb').writelines(txt_b)
1095
1096
        # This is the result with LongestCommonSubstring matching
1185.81.29 by Aaron Bentley
Fix style issues and duplicated tests
1097
        self.assertEquals(['--- a2 \n',
1098
                           '+++ b2 \n',
1099
                           '@@ -1,6 +1,11 @@\n',
1100
                           ' a\n',
1101
                           ' b\n',
1102
                           ' c\n',
1103
                           '+d\n',
1104
                           '+e\n',
1105
                           '+f\n',
1106
                           '+x\n',
1107
                           '+y\n',
1108
                           ' d\n',
1109
                           ' e\n',
1110
                           ' f\n']
1185.81.14 by John Arbash Meinel
Added a main function for running cdvdifflib manually, included tests for unified_diff interfaces
1111
                          , list(unified_diff_files('a2', 'b2')))
1112
1711.2.9 by John Arbash Meinel
Rename cdv => patience
1113
        # And the patience diff
1185.81.29 by Aaron Bentley
Fix style issues and duplicated tests
1114
        self.assertEquals(['--- a2 \n',
1115
                           '+++ b2 \n',
1116
                           '@@ -4,6 +4,11 @@\n',
1117
                           ' d\n',
1118
                           ' e\n',
1119
                           ' f\n',
1120
                           '+x\n',
1121
                           '+y\n',
1122
                           '+d\n',
1123
                           '+e\n',
1124
                           '+f\n',
1125
                           ' g\n',
1126
                           ' h\n',
1127
                           ' i\n',
1185.81.14 by John Arbash Meinel
Added a main function for running cdvdifflib manually, included tests for unified_diff interfaces
1128
                          ]
1185.81.25 by Aaron Bentley
Clean up test_diff
1129
                          , list(unified_diff_files('a2', 'b2',
1711.2.20 by John Arbash Meinel
Late bind to patiencediff objects to make it easier to plug-in
1130
                                 sequencematcher=psm)))
2781.1.1 by Martin Pool
merge cpatiencediff from Lukas
1131
1132
1133
class TestPatienceDiffLibFiles_c(TestPatienceDiffLibFiles):
1134
1135
    _test_needs_features = [CompiledPatienceDiffFeature]
1136
1137
    def setUp(self):
1138
        super(TestPatienceDiffLibFiles_c, self).setUp()
1139
        import bzrlib._patiencediff_c
1140
        self._PatienceSequenceMatcher = \
1141
            bzrlib._patiencediff_c.PatienceSequenceMatcher_c
1142
1143
1144
class TestUsingCompiledIfAvailable(TestCase):
1145
1146
    def test_PatienceSequenceMatcher(self):
1147
        if CompiledPatienceDiffFeature.available():
1148
            from bzrlib._patiencediff_c import PatienceSequenceMatcher_c
1149
            self.assertIs(PatienceSequenceMatcher_c,
1150
                          bzrlib.patiencediff.PatienceSequenceMatcher)
1151
        else:
1152
            from bzrlib._patiencediff_py import PatienceSequenceMatcher_py
1153
            self.assertIs(PatienceSequenceMatcher_py,
1154
                          bzrlib.patiencediff.PatienceSequenceMatcher)
1155
1156
    def test_unique_lcs(self):
1157
        if CompiledPatienceDiffFeature.available():
1158
            from bzrlib._patiencediff_c import unique_lcs_c
1159
            self.assertIs(unique_lcs_c,
1160
                          bzrlib.patiencediff.unique_lcs)
1161
        else:
1162
            from bzrlib._patiencediff_py import unique_lcs_py
1163
            self.assertIs(unique_lcs_py,
1164
                          bzrlib.patiencediff.unique_lcs)
1165
1166
    def test_recurse_matches(self):
1167
        if CompiledPatienceDiffFeature.available():
1168
            from bzrlib._patiencediff_c import recurse_matches_c
1169
            self.assertIs(recurse_matches_c,
1170
                          bzrlib.patiencediff.recurse_matches)
1171
        else:
1172
            from bzrlib._patiencediff_py import recurse_matches_py
1173
            self.assertIs(recurse_matches_py,
1174
                          bzrlib.patiencediff.recurse_matches)