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  | 
| 
3123.6.5
by Aaron Bentley
 Symlink to real files if possible  | 
18  | 
import os.path  | 
| 
1558.15.2
by Aaron Bentley
 Implemented binary file handling for diff  | 
19  | 
from cStringIO import StringIO  | 
| 
1692.8.7
by James Henstridge
 changes suggested by John Meinel  | 
20  | 
import errno  | 
| 
1920.1.1
by John Arbash Meinel
 fix bug #56307, handle binary files even when LANG is not english  | 
21  | 
import subprocess  | 
| 
3287.18.6
by Matt McClure
 Snapshot of unfinished work.  | 
22  | 
import sys  | 
| 
1920.1.3
by John Arbash Meinel
 Remove spurious import  | 
23  | 
from tempfile import TemporaryFile  | 
| 
1558.15.2
by Aaron Bentley
 Implemented binary file handling for diff  | 
24  | 
|
| 
3146.4.3
by Aaron Bentley
 Add missing Symlink requirement  | 
25  | 
from bzrlib import tests  | 
| 
3009.2.9
by Aaron Bentley
 Add tests for Differ  | 
26  | 
from bzrlib.diff import (  | 
| 
3123.6.2
by Aaron Bentley
 Implement diff --using natively  | 
27  | 
DiffFromTool,  | 
| 
3009.2.22
by Aaron Bentley
 Update names & docstring  | 
28  | 
DiffPath,  | 
29  | 
DiffSymlink,  | 
|
30  | 
DiffTree,  | 
|
31  | 
DiffText,  | 
|
| 
3123.6.2
by Aaron Bentley
 Implement diff --using natively  | 
32  | 
external_diff,  | 
33  | 
internal_diff,  | 
|
34  | 
show_diff_trees,  | 
|
| 
3009.2.9
by Aaron Bentley
 Add tests for Differ  | 
35  | 
    )
 | 
| 
3145.1.1
by Aaron Bentley
 Handle missing tools gracefully in diff --using  | 
36  | 
from bzrlib.errors import BinaryFile, NoDiff, ExecutableMissing  | 
| 
2321.2.5
by Alexander Belchenko
 external diff: no need for special code path for win32 (suggested by John Meinel)  | 
37  | 
import bzrlib.osutils as osutils  | 
| 
3668.5.1
by Jelmer Vernooij
 Use NULL_REVISION rather than None for Repository.revision_tree().  | 
38  | 
import bzrlib.revision as _mod_revision  | 
| 
3268.1.1
by C Miller
 Describe the property changes in diffs. Currently, this is the executable-bit  | 
39  | 
import bzrlib.transform as transform  | 
| 
1711.2.20
by John Arbash Meinel
 Late bind to patiencediff objects to make it easier to plug-in  | 
40  | 
import bzrlib.patiencediff  | 
| 
2781.1.1
by Martin Pool
 merge cpatiencediff from Lukas  | 
41  | 
import bzrlib._patiencediff_py  | 
| 
2592.2.4
by Jonathan Lange
 Skip the unicode filename test if the platform doesn't support unicode  | 
42  | 
from bzrlib.tests import (Feature, TestCase, TestCaseWithTransport,  | 
| 
1711.2.54
by John Arbash Meinel
 Use mkstemp instead of NamedTemporary file for external diff.  | 
43  | 
TestCaseInTempDir, TestSkipped)  | 
| 
1558.15.2
by Aaron Bentley
 Implemented binary file handling for diff  | 
44  | 
|
45  | 
||
| 
3287.18.20
by Matt McClure
 Introduces a Feature subclass to encapsulate the availability of 'attrib'.  | 
46  | 
class _AttribFeature(Feature):  | 
47  | 
||
48  | 
def _probe(self):  | 
|
49  | 
if (sys.platform not in ('cygwin', 'win32')):  | 
|
| 
3287.18.21
by Matt McClure
 Fixes capitalization of False.  | 
50  | 
return False  | 
| 
3287.18.20
by Matt McClure
 Introduces a Feature subclass to encapsulate the availability of 'attrib'.  | 
51  | 
try:  | 
52  | 
proc = subprocess.Popen(['attrib', '.'], stdout=subprocess.PIPE)  | 
|
53  | 
except OSError, e:  | 
|
| 
3287.18.21
by Matt McClure
 Fixes capitalization of False.  | 
54  | 
return False  | 
| 
3287.18.20
by Matt McClure
 Introduces a Feature subclass to encapsulate the availability of 'attrib'.  | 
55  | 
return (0 == proc.wait())  | 
| 
3287.18.24
by Matt McClure
 Cleans up for submission. Removes comments.  | 
56  | 
|
| 
3287.18.20
by Matt McClure
 Introduces a Feature subclass to encapsulate the availability of 'attrib'.  | 
57  | 
def feature_name(self):  | 
58  | 
return 'attrib Windows command-line tool'  | 
|
59  | 
||
60  | 
AttribFeature = _AttribFeature()  | 
|
61  | 
||
62  | 
||
| 
2781.1.1
by Martin Pool
 merge cpatiencediff from Lukas  | 
63  | 
class _CompiledPatienceDiffFeature(Feature):  | 
64  | 
||
65  | 
def _probe(self):  | 
|
66  | 
try:  | 
|
67  | 
import bzrlib._patiencediff_c  | 
|
68  | 
except ImportError:  | 
|
69  | 
return False  | 
|
70  | 
return True  | 
|
71  | 
||
72  | 
def feature_name(self):  | 
|
73  | 
return 'bzrlib._patiencediff_c'  | 
|
74  | 
||
75  | 
CompiledPatienceDiffFeature = _CompiledPatienceDiffFeature()  | 
|
76  | 
||
77  | 
||
| 
1558.15.11
by Aaron Bentley
 Apply merge review suggestions  | 
78  | 
def udiff_lines(old, new, allow_binary=False):  | 
| 
974.1.6
by Aaron Bentley
 Added unit tests  | 
79  | 
output = StringIO()  | 
| 
1558.15.11
by Aaron Bentley
 Apply merge review suggestions  | 
80  | 
internal_diff('old', old, 'new', new, output, allow_binary)  | 
| 
974.1.6
by Aaron Bentley
 Added unit tests  | 
81  | 
output.seek(0, 0)  | 
82  | 
return output.readlines()  | 
|
83  | 
||
| 
1711.2.54
by John Arbash Meinel
 Use mkstemp instead of NamedTemporary file for external diff.  | 
84  | 
|
| 
1711.2.57
by John Arbash Meinel
 Allow external diff to write to a file without a fileno.  | 
85  | 
def external_udiff_lines(old, new, use_stringio=False):  | 
86  | 
if use_stringio:  | 
|
87  | 
        # StringIO has no fileno, so it tests a different codepath
 | 
|
88  | 
output = StringIO()  | 
|
89  | 
else:  | 
|
90  | 
output = TemporaryFile()  | 
|
| 
1692.8.7
by James Henstridge
 changes suggested by John Meinel  | 
91  | 
try:  | 
92  | 
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  | 
93  | 
except NoDiff:  | 
| 
1711.2.56
by John Arbash Meinel
 Raise NoDiff if 'diff' not present.  | 
94  | 
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  | 
95  | 
output.seek(0, 0)  | 
96  | 
lines = output.readlines()  | 
|
97  | 
output.close()  | 
|
98  | 
return lines  | 
|
99  | 
||
100  | 
||
| 
1102
by Martin Pool
 - merge test refactoring from robertc  | 
101  | 
class TestDiff(TestCase):  | 
| 
1185.81.25
by Aaron Bentley
 Clean up test_diff  | 
102  | 
|
| 
1102
by Martin Pool
 - merge test refactoring from robertc  | 
103  | 
def test_add_nl(self):  | 
104  | 
"""diff generates a valid diff for patches that add a newline"""  | 
|
| 
974.1.6
by Aaron Bentley
 Added unit tests  | 
105  | 
lines = udiff_lines(['boo'], ['boo\n'])  | 
| 
1185.16.145
by Martin Pool
 Remove all assert statements from test cases.  | 
106  | 
self.check_patch(lines)  | 
107  | 
self.assertEquals(lines[4], '\\ No newline at end of file\n')  | 
|
108  | 
            ## "expected no-nl, got %r" % lines[4]
 | 
|
| 
974.1.6
by Aaron Bentley
 Added unit tests  | 
109  | 
|
| 
1102
by Martin Pool
 - merge test refactoring from robertc  | 
110  | 
def test_add_nl_2(self):  | 
111  | 
"""diff generates a valid diff for patches that change last line and  | 
|
112  | 
        add a newline.
 | 
|
113  | 
        """
 | 
|
| 
974.1.6
by Aaron Bentley
 Added unit tests  | 
114  | 
lines = udiff_lines(['boo'], ['goo\n'])  | 
| 
1185.16.145
by Martin Pool
 Remove all assert statements from test cases.  | 
115  | 
self.check_patch(lines)  | 
116  | 
self.assertEquals(lines[4], '\\ No newline at end of file\n')  | 
|
117  | 
            ## "expected no-nl, got %r" % lines[4]
 | 
|
| 
974.1.6
by Aaron Bentley
 Added unit tests  | 
118  | 
|
| 
1102
by Martin Pool
 - merge test refactoring from robertc  | 
119  | 
def test_remove_nl(self):  | 
120  | 
"""diff generates a valid diff for patches that change last line and  | 
|
121  | 
        add a newline.
 | 
|
122  | 
        """
 | 
|
| 
974.1.6
by Aaron Bentley
 Added unit tests  | 
123  | 
lines = udiff_lines(['boo\n'], ['boo'])  | 
| 
1185.16.145
by Martin Pool
 Remove all assert statements from test cases.  | 
124  | 
self.check_patch(lines)  | 
125  | 
self.assertEquals(lines[5], '\\ No newline at end of file\n')  | 
|
126  | 
            ## "expected no-nl, got %r" % lines[5]
 | 
|
127  | 
||
128  | 
def check_patch(self, lines):  | 
|
129  | 
self.assert_(len(lines) > 1)  | 
|
130  | 
            ## "Not enough lines for a file header for patch:\n%s" % "".join(lines)
 | 
|
131  | 
self.assert_(lines[0].startswith ('---'))  | 
|
132  | 
            ## 'No orig line for patch:\n%s' % "".join(lines)
 | 
|
133  | 
self.assert_(lines[1].startswith ('+++'))  | 
|
134  | 
            ## 'No mod line for patch:\n%s' % "".join(lines)
 | 
|
135  | 
self.assert_(len(lines) > 2)  | 
|
136  | 
            ## "No hunks for patch:\n%s" % "".join(lines)
 | 
|
137  | 
self.assert_(lines[2].startswith('@@'))  | 
|
138  | 
            ## "No hunk header for patch:\n%s" % "".join(lines)
 | 
|
139  | 
self.assert_('@@' in lines[2][2:])  | 
|
140  | 
            ## "Unterminated hunk header for patch:\n%s" % "".join(lines)
 | 
|
141  | 
||
| 
1558.15.2
by Aaron Bentley
 Implemented binary file handling for diff  | 
142  | 
def test_binary_lines(self):  | 
143  | 
self.assertRaises(BinaryFile, udiff_lines, [1023 * 'a' + '\x00'], [])  | 
|
144  | 
self.assertRaises(BinaryFile, udiff_lines, [], [1023 * 'a' + '\x00'])  | 
|
| 
1558.15.11
by Aaron Bentley
 Apply merge review suggestions  | 
145  | 
udiff_lines([1023 * 'a' + '\x00'], [], allow_binary=True)  | 
146  | 
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  | 
147  | 
|
148  | 
def test_external_diff(self):  | 
|
149  | 
lines = external_udiff_lines(['boo\n'], ['goo\n'])  | 
|
150  | 
self.check_patch(lines)  | 
|
| 
1899.1.6
by John Arbash Meinel
 internal_diff always adds a trailing \n, make sure external_diff does too  | 
151  | 
self.assertEqual('\n', lines[-1])  | 
| 
1711.2.57
by John Arbash Meinel
 Allow external diff to write to a file without a fileno.  | 
152  | 
|
153  | 
def test_external_diff_no_fileno(self):  | 
|
154  | 
        # Make sure that we can handle not having a fileno, even
 | 
|
155  | 
        # if the diff is large
 | 
|
156  | 
lines = external_udiff_lines(['boo\n']*10000,  | 
|
157  | 
['goo\n']*10000,  | 
|
158  | 
use_stringio=True)  | 
|
159  | 
self.check_patch(lines)  | 
|
| 
1899.1.1
by John Arbash Meinel
 Fix the bug in the NoDiff exception class, and add a test  | 
160  | 
|
| 
1920.1.1
by John Arbash Meinel
 fix bug #56307, handle binary files even when LANG is not english  | 
161  | 
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)  | 
162  | 
old_env = {}  | 
163  | 
for lang in ('LANG', 'LC_ALL', 'LANGUAGE'):  | 
|
164  | 
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  | 
165  | 
try:  | 
166  | 
lines = external_udiff_lines(['\x00foobar\n'], ['foo\x00bar\n'])  | 
|
| 
1959.1.1
by Marien Zwart
 merge.  | 
167  | 
            # Older versions of diffutils say "Binary files", newer
 | 
168  | 
            # versions just say "Files".
 | 
|
169  | 
self.assertContainsRe(lines[0],  | 
|
170  | 
'(Binary f|F)iles old and new differ\n')  | 
|
171  | 
self.assertEquals(lines[1:], ['\n'])  | 
|
| 
1920.1.1
by John Arbash Meinel
 fix bug #56307, handle binary files even when LANG is not english  | 
172  | 
finally:  | 
| 
2321.2.5
by Alexander Belchenko
 external diff: no need for special code path for win32 (suggested by John Meinel)  | 
173  | 
for lang, old_val in old_env.iteritems():  | 
174  | 
osutils.set_or_unset_env(lang, old_val)  | 
|
| 
1899.1.4
by John Arbash Meinel
 Just swallow a return code of 2  | 
175  | 
|
| 
1899.1.1
by John Arbash Meinel
 Fix the bug in the NoDiff exception class, and add a test  | 
176  | 
def test_no_external_diff(self):  | 
177  | 
"""Check that NoDiff is raised when diff is not available"""  | 
|
178  | 
        # Use os.environ['PATH'] to make sure no 'diff' command is available
 | 
|
179  | 
orig_path = os.environ['PATH']  | 
|
180  | 
try:  | 
|
181  | 
os.environ['PATH'] = ''  | 
|
182  | 
self.assertRaises(NoDiff, external_diff,  | 
|
183  | 
'old', ['boo\n'], 'new', ['goo\n'],  | 
|
184  | 
StringIO(), diff_opts=['-u'])  | 
|
185  | 
finally:  | 
|
186  | 
os.environ['PATH'] = orig_path  | 
|
| 
3943.8.1
by Marius Kruger
 remove all trailing whitespace from bzr source  | 
187  | 
|
| 
1711.2.30
by John Arbash Meinel
 Fix bug in internal_diff handling of unicode paths  | 
188  | 
def test_internal_diff_default(self):  | 
189  | 
        # Default internal diff encoding is utf8
 | 
|
190  | 
output = StringIO()  | 
|
191  | 
internal_diff(u'old_\xb5', ['old_text\n'],  | 
|
192  | 
u'new_\xe5', ['new_text\n'], output)  | 
|
193  | 
lines = output.getvalue().splitlines(True)  | 
|
194  | 
self.check_patch(lines)  | 
|
| 
1740.2.5
by Aaron Bentley
 Merge from bzr.dev  | 
195  | 
self.assertEquals(['--- old_\xc2\xb5\n',  | 
196  | 
'+++ new_\xc3\xa5\n',  | 
|
| 
1711.2.30
by John Arbash Meinel
 Fix bug in internal_diff handling of unicode paths  | 
197  | 
'@@ -1,1 +1,1 @@\n',  | 
198  | 
'-old_text\n',  | 
|
199  | 
'+new_text\n',  | 
|
200  | 
'\n',  | 
|
201  | 
                          ]
 | 
|
202  | 
, lines)  | 
|
203  | 
||
204  | 
def test_internal_diff_utf8(self):  | 
|
205  | 
output = StringIO()  | 
|
206  | 
internal_diff(u'old_\xb5', ['old_text\n'],  | 
|
207  | 
u'new_\xe5', ['new_text\n'], output,  | 
|
208  | 
path_encoding='utf8')  | 
|
209  | 
lines = output.getvalue().splitlines(True)  | 
|
210  | 
self.check_patch(lines)  | 
|
| 
1740.2.5
by Aaron Bentley
 Merge from bzr.dev  | 
211  | 
self.assertEquals(['--- old_\xc2\xb5\n',  | 
212  | 
'+++ new_\xc3\xa5\n',  | 
|
| 
1711.2.30
by John Arbash Meinel
 Fix bug in internal_diff handling of unicode paths  | 
213  | 
'@@ -1,1 +1,1 @@\n',  | 
214  | 
'-old_text\n',  | 
|
215  | 
'+new_text\n',  | 
|
216  | 
'\n',  | 
|
217  | 
                          ]
 | 
|
218  | 
, lines)  | 
|
219  | 
||
220  | 
def test_internal_diff_iso_8859_1(self):  | 
|
221  | 
output = StringIO()  | 
|
222  | 
internal_diff(u'old_\xb5', ['old_text\n'],  | 
|
223  | 
u'new_\xe5', ['new_text\n'], output,  | 
|
224  | 
path_encoding='iso-8859-1')  | 
|
225  | 
lines = output.getvalue().splitlines(True)  | 
|
226  | 
self.check_patch(lines)  | 
|
| 
1740.2.5
by Aaron Bentley
 Merge from bzr.dev  | 
227  | 
self.assertEquals(['--- old_\xb5\n',  | 
228  | 
'+++ new_\xe5\n',  | 
|
| 
1711.2.30
by John Arbash Meinel
 Fix bug in internal_diff handling of unicode paths  | 
229  | 
'@@ -1,1 +1,1 @@\n',  | 
230  | 
'-old_text\n',  | 
|
231  | 
'+new_text\n',  | 
|
232  | 
'\n',  | 
|
233  | 
                          ]
 | 
|
234  | 
, lines)  | 
|
235  | 
||
| 
3085.1.1
by John Arbash Meinel
 Fix internal_diff to not fail when the texts are identical.  | 
236  | 
def test_internal_diff_no_content(self):  | 
237  | 
output = StringIO()  | 
|
238  | 
internal_diff(u'old', [], u'new', [], output)  | 
|
239  | 
self.assertEqual('', output.getvalue())  | 
|
240  | 
||
241  | 
def test_internal_diff_no_changes(self):  | 
|
242  | 
output = StringIO()  | 
|
243  | 
internal_diff(u'old', ['text\n', 'contents\n'],  | 
|
244  | 
u'new', ['text\n', 'contents\n'],  | 
|
245  | 
output)  | 
|
246  | 
self.assertEqual('', output.getvalue())  | 
|
247  | 
||
| 
1711.2.30
by John Arbash Meinel
 Fix bug in internal_diff handling of unicode paths  | 
248  | 
def test_internal_diff_returns_bytes(self):  | 
249  | 
import StringIO  | 
|
250  | 
output = StringIO.StringIO()  | 
|
251  | 
internal_diff(u'old_\xb5', ['old_text\n'],  | 
|
252  | 
u'new_\xe5', ['new_text\n'], output)  | 
|
253  | 
self.failUnless(isinstance(output.getvalue(), str),  | 
|
254  | 
'internal_diff should return bytestrings')  | 
|
255  | 
||
| 
1185.81.25
by Aaron Bentley
 Clean up test_diff  | 
256  | 
|
| 
1920.1.1
by John Arbash Meinel
 fix bug #56307, handle binary files even when LANG is not english  | 
257  | 
class TestDiffFiles(TestCaseInTempDir):  | 
258  | 
||
259  | 
def test_external_diff_binary(self):  | 
|
260  | 
"""The output when using external diff should use diff's i18n error"""  | 
|
261  | 
        # Make sure external_diff doesn't fail in the current LANG
 | 
|
262  | 
lines = external_udiff_lines(['\x00foobar\n'], ['foo\x00bar\n'])  | 
|
263  | 
||
| 
2240.1.1
by Alexander Belchenko
 test_external_diff_binary: run external diff with --binary flag  | 
264  | 
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  | 
265  | 
open('old', 'wb').write('\x00foobar\n')  | 
266  | 
open('new', 'wb').write('foo\x00bar\n')  | 
|
267  | 
pipe = subprocess.Popen(cmd, stdout=subprocess.PIPE,  | 
|
268  | 
stdin=subprocess.PIPE)  | 
|
269  | 
out, err = pipe.communicate()  | 
|
270  | 
        # Diff returns '2' on Binary files.
 | 
|
271  | 
self.assertEqual(2, pipe.returncode)  | 
|
272  | 
        # We should output whatever diff tells us, plus a trailing newline
 | 
|
273  | 
self.assertEqual(out.splitlines(True) + ['\n'], lines)  | 
|
274  | 
||
275  | 
||
| 
2405.1.1
by John Arbash Meinel
 Add a bunch of direct tests for 'show_diff_trees'  | 
276  | 
class TestShowDiffTreesHelper(TestCaseWithTransport):  | 
277  | 
"""Has a helper for running show_diff_trees"""  | 
|
278  | 
||
279  | 
def get_diff(self, tree1, tree2, specific_files=None, working_tree=None):  | 
|
280  | 
output = StringIO()  | 
|
281  | 
if working_tree is not None:  | 
|
282  | 
extra_trees = (working_tree,)  | 
|
283  | 
else:  | 
|
284  | 
extra_trees = ()  | 
|
285  | 
show_diff_trees(tree1, tree2, output, specific_files=specific_files,  | 
|
286  | 
extra_trees=extra_trees, old_label='old/',  | 
|
287  | 
new_label='new/')  | 
|
288  | 
return output.getvalue()  | 
|
289  | 
||
290  | 
||
291  | 
class TestDiffDates(TestShowDiffTreesHelper):  | 
|
| 
1740.2.5
by Aaron Bentley
 Merge from bzr.dev  | 
292  | 
|
293  | 
def setUp(self):  | 
|
294  | 
super(TestDiffDates, self).setUp()  | 
|
295  | 
self.wt = self.make_branch_and_tree('.')  | 
|
296  | 
self.b = self.wt.branch  | 
|
297  | 
self.build_tree_contents([  | 
|
298  | 
('file1', 'file1 contents at rev 1\n'),  | 
|
299  | 
('file2', 'file2 contents at rev 1\n')  | 
|
300  | 
            ])
 | 
|
301  | 
self.wt.add(['file1', 'file2'])  | 
|
302  | 
self.wt.commit(  | 
|
303  | 
message='Revision 1',  | 
|
304  | 
timestamp=1143849600, # 2006-04-01 00:00:00 UTC  | 
|
305  | 
timezone=0,  | 
|
306  | 
rev_id='rev-1')  | 
|
307  | 
self.build_tree_contents([('file1', 'file1 contents at rev 2\n')])  | 
|
308  | 
self.wt.commit(  | 
|
309  | 
message='Revision 2',  | 
|
310  | 
timestamp=1143936000, # 2006-04-02 00:00:00 UTC  | 
|
311  | 
timezone=28800,  | 
|
312  | 
rev_id='rev-2')  | 
|
313  | 
self.build_tree_contents([('file2', 'file2 contents at rev 3\n')])  | 
|
314  | 
self.wt.commit(  | 
|
315  | 
message='Revision 3',  | 
|
316  | 
timestamp=1144022400, # 2006-04-03 00:00:00 UTC  | 
|
317  | 
timezone=-3600,  | 
|
318  | 
rev_id='rev-3')  | 
|
319  | 
self.wt.remove(['file2'])  | 
|
320  | 
self.wt.commit(  | 
|
321  | 
message='Revision 4',  | 
|
322  | 
timestamp=1144108800, # 2006-04-04 00:00:00 UTC  | 
|
323  | 
timezone=0,  | 
|
324  | 
rev_id='rev-4')  | 
|
325  | 
self.build_tree_contents([  | 
|
326  | 
('file1', 'file1 contents in working tree\n')  | 
|
327  | 
            ])
 | 
|
328  | 
        # set the date stamps for files in the working tree to known values
 | 
|
329  | 
os.utime('file1', (1144195200, 1144195200)) # 2006-04-05 00:00:00 UTC  | 
|
330  | 
||
331  | 
def test_diff_rev_tree_working_tree(self):  | 
|
332  | 
output = self.get_diff(self.wt.basis_tree(), self.wt)  | 
|
333  | 
        # note that the date for old/file1 is from rev 2 rather than from
 | 
|
334  | 
        # the basis revision (rev 4)
 | 
|
335  | 
self.assertEqualDiff(output, '''\  | 
|
336  | 
=== modified file 'file1'
 | 
|
337  | 
--- old/file1\t2006-04-02 00:00:00 +0000  | 
|
338  | 
+++ new/file1\t2006-04-05 00:00:00 +0000  | 
|
339  | 
@@ -1,1 +1,1 @@
 | 
|
340  | 
-file1 contents at rev 2
 | 
|
341  | 
+file1 contents in working tree
 | 
|
342  | 
||
343  | 
''')  | 
|
344  | 
||
345  | 
def test_diff_rev_tree_rev_tree(self):  | 
|
346  | 
tree1 = self.b.repository.revision_tree('rev-2')  | 
|
347  | 
tree2 = self.b.repository.revision_tree('rev-3')  | 
|
348  | 
output = self.get_diff(tree1, tree2)  | 
|
349  | 
self.assertEqualDiff(output, '''\  | 
|
350  | 
=== modified file 'file2'
 | 
|
351  | 
--- old/file2\t2006-04-01 00:00:00 +0000  | 
|
352  | 
+++ new/file2\t2006-04-03 00:00:00 +0000  | 
|
353  | 
@@ -1,1 +1,1 @@
 | 
|
354  | 
-file2 contents at rev 1
 | 
|
355  | 
+file2 contents at rev 3
 | 
|
356  | 
||
357  | 
''')  | 
|
| 
3943.8.1
by Marius Kruger
 remove all trailing whitespace from bzr source  | 
358  | 
|
| 
1740.2.5
by Aaron Bentley
 Merge from bzr.dev  | 
359  | 
def test_diff_add_files(self):  | 
| 
3668.5.1
by Jelmer Vernooij
 Use NULL_REVISION rather than None for Repository.revision_tree().  | 
360  | 
tree1 = self.b.repository.revision_tree(_mod_revision.NULL_REVISION)  | 
| 
1740.2.5
by Aaron Bentley
 Merge from bzr.dev  | 
361  | 
tree2 = self.b.repository.revision_tree('rev-1')  | 
362  | 
output = self.get_diff(tree1, tree2)  | 
|
363  | 
        # the files have the epoch time stamp for the tree in which
 | 
|
364  | 
        # they don't exist.
 | 
|
365  | 
self.assertEqualDiff(output, '''\  | 
|
366  | 
=== added file 'file1'
 | 
|
367  | 
--- old/file1\t1970-01-01 00:00:00 +0000  | 
|
368  | 
+++ new/file1\t2006-04-01 00:00:00 +0000  | 
|
369  | 
@@ -0,0 +1,1 @@
 | 
|
370  | 
+file1 contents at rev 1
 | 
|
371  | 
||
372  | 
=== added file 'file2'
 | 
|
373  | 
--- old/file2\t1970-01-01 00:00:00 +0000  | 
|
374  | 
+++ new/file2\t2006-04-01 00:00:00 +0000  | 
|
375  | 
@@ -0,0 +1,1 @@
 | 
|
376  | 
+file2 contents at rev 1
 | 
|
377  | 
||
378  | 
''')  | 
|
379  | 
||
380  | 
def test_diff_remove_files(self):  | 
|
381  | 
tree1 = self.b.repository.revision_tree('rev-3')  | 
|
382  | 
tree2 = self.b.repository.revision_tree('rev-4')  | 
|
383  | 
output = self.get_diff(tree1, tree2)  | 
|
384  | 
        # the file has the epoch time stamp for the tree in which
 | 
|
385  | 
        # it doesn't exist.
 | 
|
386  | 
self.assertEqualDiff(output, '''\  | 
|
387  | 
=== removed file 'file2'
 | 
|
388  | 
--- old/file2\t2006-04-03 00:00:00 +0000  | 
|
389  | 
+++ new/file2\t1970-01-01 00:00:00 +0000  | 
|
390  | 
@@ -1,1 +0,0 @@
 | 
|
391  | 
-file2 contents at rev 3
 | 
|
392  | 
||
393  | 
''')  | 
|
394  | 
||
| 
1551.7.17
by Aaron Bentley
 Switch to PathsNotVersioned, accept extra_trees  | 
395  | 
def test_show_diff_specified(self):  | 
| 
1551.7.22
by Aaron Bentley
 Changes from review  | 
396  | 
"""A working tree filename can be used to identify a file"""  | 
| 
1551.7.17
by Aaron Bentley
 Switch to PathsNotVersioned, accept extra_trees  | 
397  | 
self.wt.rename_one('file1', 'file1b')  | 
398  | 
old_tree = self.b.repository.revision_tree('rev-1')  | 
|
399  | 
new_tree = self.b.repository.revision_tree('rev-4')  | 
|
| 
3943.8.1
by Marius Kruger
 remove all trailing whitespace from bzr source  | 
400  | 
out = self.get_diff(old_tree, new_tree, specific_files=['file1b'],  | 
| 
1551.7.22
by Aaron Bentley
 Changes from review  | 
401  | 
working_tree=self.wt)  | 
402  | 
self.assertContainsRe(out, 'file1\t')  | 
|
| 
1551.7.17
by Aaron Bentley
 Switch to PathsNotVersioned, accept extra_trees  | 
403  | 
|
| 
1551.7.22
by Aaron Bentley
 Changes from review  | 
404  | 
def test_recursive_diff(self):  | 
405  | 
"""Children of directories are matched"""  | 
|
406  | 
os.mkdir('dir1')  | 
|
407  | 
os.mkdir('dir2')  | 
|
408  | 
self.wt.add(['dir1', 'dir2'])  | 
|
409  | 
self.wt.rename_one('file1', 'dir1/file1')  | 
|
410  | 
old_tree = self.b.repository.revision_tree('rev-1')  | 
|
411  | 
new_tree = self.b.repository.revision_tree('rev-4')  | 
|
| 
3943.8.1
by Marius Kruger
 remove all trailing whitespace from bzr source  | 
412  | 
out = self.get_diff(old_tree, new_tree, specific_files=['dir1'],  | 
| 
1551.7.22
by Aaron Bentley
 Changes from review  | 
413  | 
working_tree=self.wt)  | 
414  | 
self.assertContainsRe(out, 'file1\t')  | 
|
| 
3943.8.1
by Marius Kruger
 remove all trailing whitespace from bzr source  | 
415  | 
out = self.get_diff(old_tree, new_tree, specific_files=['dir2'],  | 
| 
1551.7.22
by Aaron Bentley
 Changes from review  | 
416  | 
working_tree=self.wt)  | 
417  | 
self.assertNotContainsRe(out, 'file1\t')  | 
|
| 
1740.2.5
by Aaron Bentley
 Merge from bzr.dev  | 
418  | 
|
| 
1899.1.1
by John Arbash Meinel
 Fix the bug in the NoDiff exception class, and add a test  | 
419  | 
|
| 
2405.1.1
by John Arbash Meinel
 Add a bunch of direct tests for 'show_diff_trees'  | 
420  | 
|
421  | 
class TestShowDiffTrees(TestShowDiffTreesHelper):  | 
|
422  | 
"""Direct tests for show_diff_trees"""  | 
|
423  | 
||
424  | 
def test_modified_file(self):  | 
|
425  | 
"""Test when a file is modified."""  | 
|
426  | 
tree = self.make_branch_and_tree('tree')  | 
|
427  | 
self.build_tree_contents([('tree/file', 'contents\n')])  | 
|
428  | 
tree.add(['file'], ['file-id'])  | 
|
429  | 
tree.commit('one', rev_id='rev-1')  | 
|
430  | 
||
431  | 
self.build_tree_contents([('tree/file', 'new contents\n')])  | 
|
432  | 
diff = self.get_diff(tree.basis_tree(), tree)  | 
|
433  | 
self.assertContainsRe(diff, "=== modified file 'file'\n")  | 
|
434  | 
self.assertContainsRe(diff, '--- old/file\t')  | 
|
435  | 
self.assertContainsRe(diff, '\\+\\+\\+ new/file\t')  | 
|
436  | 
self.assertContainsRe(diff, '-contents\n'  | 
|
437  | 
'\\+new contents\n')  | 
|
438  | 
||
| 
2405.1.2
by John Arbash Meinel
 Fix bug #103870 by passing None instead of a (sometimes wrong) path  | 
439  | 
def test_modified_file_in_renamed_dir(self):  | 
440  | 
"""Test when a file is modified in a renamed directory."""  | 
|
441  | 
tree = self.make_branch_and_tree('tree')  | 
|
442  | 
self.build_tree(['tree/dir/'])  | 
|
443  | 
self.build_tree_contents([('tree/dir/file', 'contents\n')])  | 
|
444  | 
tree.add(['dir', 'dir/file'], ['dir-id', 'file-id'])  | 
|
445  | 
tree.commit('one', rev_id='rev-1')  | 
|
446  | 
||
447  | 
tree.rename_one('dir', 'other')  | 
|
448  | 
self.build_tree_contents([('tree/other/file', 'new contents\n')])  | 
|
449  | 
diff = self.get_diff(tree.basis_tree(), tree)  | 
|
450  | 
self.assertContainsRe(diff, "=== renamed directory 'dir' => 'other'\n")  | 
|
451  | 
self.assertContainsRe(diff, "=== modified file 'other/file'\n")  | 
|
452  | 
        # XXX: This is technically incorrect, because it used to be at another
 | 
|
453  | 
        # 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.  | 
454  | 
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  | 
455  | 
self.assertContainsRe(diff, '\\+\\+\\+ new/other/file\t')  | 
456  | 
self.assertContainsRe(diff, '-contents\n'  | 
|
457  | 
'\\+new contents\n')  | 
|
458  | 
||
| 
2405.1.1
by John Arbash Meinel
 Add a bunch of direct tests for 'show_diff_trees'  | 
459  | 
def test_renamed_directory(self):  | 
460  | 
"""Test when only a directory is only renamed."""  | 
|
461  | 
tree = self.make_branch_and_tree('tree')  | 
|
462  | 
self.build_tree(['tree/dir/'])  | 
|
463  | 
self.build_tree_contents([('tree/dir/file', 'contents\n')])  | 
|
464  | 
tree.add(['dir', 'dir/file'], ['dir-id', 'file-id'])  | 
|
465  | 
tree.commit('one', rev_id='rev-1')  | 
|
466  | 
||
467  | 
tree.rename_one('dir', 'newdir')  | 
|
468  | 
diff = self.get_diff(tree.basis_tree(), tree)  | 
|
469  | 
        # Renaming a directory should be a single "you renamed this dir" even
 | 
|
470  | 
        # when there are files inside.
 | 
|
471  | 
self.assertEqual("=== renamed directory 'dir' => 'newdir'\n", diff)  | 
|
472  | 
||
473  | 
def test_renamed_file(self):  | 
|
474  | 
"""Test when a file is only renamed."""  | 
|
475  | 
tree = self.make_branch_and_tree('tree')  | 
|
476  | 
self.build_tree_contents([('tree/file', 'contents\n')])  | 
|
477  | 
tree.add(['file'], ['file-id'])  | 
|
478  | 
tree.commit('one', rev_id='rev-1')  | 
|
479  | 
||
480  | 
tree.rename_one('file', 'newname')  | 
|
481  | 
diff = self.get_diff(tree.basis_tree(), tree)  | 
|
482  | 
self.assertContainsRe(diff, "=== renamed file 'file' => 'newname'\n")  | 
|
483  | 
        # We shouldn't have a --- or +++ line, because there is no content
 | 
|
484  | 
        # change
 | 
|
485  | 
self.assertNotContainsRe(diff, '---')  | 
|
486  | 
||
487  | 
def test_renamed_and_modified_file(self):  | 
|
488  | 
"""Test when a file is only renamed."""  | 
|
489  | 
tree = self.make_branch_and_tree('tree')  | 
|
490  | 
self.build_tree_contents([('tree/file', 'contents\n')])  | 
|
491  | 
tree.add(['file'], ['file-id'])  | 
|
492  | 
tree.commit('one', rev_id='rev-1')  | 
|
493  | 
||
494  | 
tree.rename_one('file', 'newname')  | 
|
495  | 
self.build_tree_contents([('tree/newname', 'new contents\n')])  | 
|
496  | 
diff = self.get_diff(tree.basis_tree(), tree)  | 
|
497  | 
self.assertContainsRe(diff, "=== renamed file 'file' => 'newname'\n")  | 
|
498  | 
self.assertContainsRe(diff, '--- old/file\t')  | 
|
499  | 
self.assertContainsRe(diff, '\\+\\+\\+ new/newname\t')  | 
|
500  | 
self.assertContainsRe(diff, '-contents\n'  | 
|
501  | 
'\\+new contents\n')  | 
|
502  | 
||
| 
3268.1.1
by C Miller
 Describe the property changes in diffs. Currently, this is the executable-bit  | 
503  | 
|
504  | 
def test_internal_diff_exec_property(self):  | 
|
505  | 
tree = self.make_branch_and_tree('tree')  | 
|
506  | 
||
507  | 
tt = transform.TreeTransform(tree)  | 
|
508  | 
tt.new_file('a', tt.root, 'contents\n', 'a-id', True)  | 
|
509  | 
tt.new_file('b', tt.root, 'contents\n', 'b-id', False)  | 
|
510  | 
tt.new_file('c', tt.root, 'contents\n', 'c-id', True)  | 
|
511  | 
tt.new_file('d', tt.root, 'contents\n', 'd-id', False)  | 
|
512  | 
tt.new_file('e', tt.root, 'contents\n', 'control-e-id', True)  | 
|
513  | 
tt.new_file('f', tt.root, 'contents\n', 'control-f-id', False)  | 
|
514  | 
tt.apply()  | 
|
515  | 
tree.commit('one', rev_id='rev-1')  | 
|
516  | 
||
517  | 
tt = transform.TreeTransform(tree)  | 
|
518  | 
tt.set_executability(False, tt.trans_id_file_id('a-id'))  | 
|
519  | 
tt.set_executability(True, tt.trans_id_file_id('b-id'))  | 
|
520  | 
tt.set_executability(False, tt.trans_id_file_id('c-id'))  | 
|
521  | 
tt.set_executability(True, tt.trans_id_file_id('d-id'))  | 
|
522  | 
tt.apply()  | 
|
523  | 
tree.rename_one('c', 'new-c')  | 
|
524  | 
tree.rename_one('d', 'new-d')  | 
|
525  | 
||
526  | 
diff = self.get_diff(tree.basis_tree(), tree)  | 
|
527  | 
||
528  | 
self.assertContainsRe(diff, r"file 'a'.*\(properties changed:.*\+x to -x.*\)")  | 
|
529  | 
self.assertContainsRe(diff, r"file 'b'.*\(properties changed:.*-x to \+x.*\)")  | 
|
530  | 
self.assertContainsRe(diff, r"file 'c'.*\(properties changed:.*\+x to -x.*\)")  | 
|
531  | 
self.assertContainsRe(diff, r"file 'd'.*\(properties changed:.*-x to \+x.*\)")  | 
|
532  | 
self.assertNotContainsRe(diff, r"file 'e'")  | 
|
533  | 
self.assertNotContainsRe(diff, r"file 'f'")  | 
|
534  | 
||
535  | 
||
| 
2592.2.1
by Jonathan Lange
 Reproduce and fix bug 110092.  | 
536  | 
def test_binary_unicode_filenames(self):  | 
| 
2592.2.2
by Jonathan Lange
 Apply jam's comments to test_binary_unicode_filenames. Change the  | 
537  | 
"""Test that contents of files are *not* encoded in UTF-8 when there  | 
538  | 
        is a binary file in the diff.
 | 
|
| 
2592.2.1
by Jonathan Lange
 Reproduce and fix bug 110092.  | 
539  | 
        """
 | 
540  | 
        # See https://bugs.launchpad.net/bugs/110092.
 | 
|
| 
3477.1.2
by John Arbash Meinel
 Rename UnicodeFilename => UnicodeFilenameFeature  | 
541  | 
self.requireFeature(tests.UnicodeFilenameFeature)  | 
| 
2592.2.1
by Jonathan Lange
 Reproduce and fix bug 110092.  | 
542  | 
|
543  | 
        # This bug isn't triggered with cStringIO.
 | 
|
544  | 
from StringIO import StringIO  | 
|
545  | 
tree = self.make_branch_and_tree('tree')  | 
|
| 
2592.2.2
by Jonathan Lange
 Apply jam's comments to test_binary_unicode_filenames. Change the  | 
546  | 
alpha, omega = u'\u03b1', u'\u03c9'  | 
547  | 
alpha_utf8, omega_utf8 = alpha.encode('utf8'), omega.encode('utf8')  | 
|
| 
2592.2.1
by Jonathan Lange
 Reproduce and fix bug 110092.  | 
548  | 
self.build_tree_contents(  | 
| 
2592.2.2
by Jonathan Lange
 Apply jam's comments to test_binary_unicode_filenames. Change the  | 
549  | 
[('tree/' + alpha, chr(0)),  | 
550  | 
('tree/' + omega,  | 
|
551  | 
('The %s and the %s\n' % (alpha_utf8, omega_utf8)))])  | 
|
552  | 
tree.add([alpha], ['file-id'])  | 
|
553  | 
tree.add([omega], ['file-id-2'])  | 
|
| 
2592.2.1
by Jonathan Lange
 Reproduce and fix bug 110092.  | 
554  | 
diff_content = StringIO()  | 
555  | 
show_diff_trees(tree.basis_tree(), tree, diff_content)  | 
|
556  | 
diff = diff_content.getvalue()  | 
|
| 
2592.2.2
by Jonathan Lange
 Apply jam's comments to test_binary_unicode_filenames. Change the  | 
557  | 
self.assertContainsRe(diff, r"=== added file '%s'" % alpha_utf8)  | 
| 
2592.2.1
by Jonathan Lange
 Reproduce and fix bug 110092.  | 
558  | 
self.assertContainsRe(  | 
| 
2592.2.2
by Jonathan Lange
 Apply jam's comments to test_binary_unicode_filenames. Change the  | 
559  | 
diff, "Binary files a/%s.*and b/%s.* differ\n" % (alpha_utf8, alpha_utf8))  | 
560  | 
self.assertContainsRe(diff, r"=== added file '%s'" % omega_utf8)  | 
|
561  | 
self.assertContainsRe(diff, r"--- a/%s" % (omega_utf8,))  | 
|
562  | 
self.assertContainsRe(diff, r"\+\+\+ b/%s" % (omega_utf8,))  | 
|
| 
2592.2.1
by Jonathan Lange
 Reproduce and fix bug 110092.  | 
563  | 
|
| 
2725.2.1
by ghigo
 When a unicode filename is renamed, in the diff is showed a wrong result  | 
564  | 
def test_unicode_filename(self):  | 
565  | 
"""Test when the filename are unicode."""  | 
|
| 
3477.1.2
by John Arbash Meinel
 Rename UnicodeFilename => UnicodeFilenameFeature  | 
566  | 
self.requireFeature(tests.UnicodeFilenameFeature)  | 
| 
2725.2.1
by ghigo
 When a unicode filename is renamed, in the diff is showed a wrong result  | 
567  | 
|
568  | 
alpha, omega = u'\u03b1', u'\u03c9'  | 
|
569  | 
autf8, outf8 = alpha.encode('utf8'), omega.encode('utf8')  | 
|
570  | 
||
571  | 
tree = self.make_branch_and_tree('tree')  | 
|
572  | 
self.build_tree_contents([('tree/ren_'+alpha, 'contents\n')])  | 
|
573  | 
tree.add(['ren_'+alpha], ['file-id-2'])  | 
|
574  | 
self.build_tree_contents([('tree/del_'+alpha, 'contents\n')])  | 
|
575  | 
tree.add(['del_'+alpha], ['file-id-3'])  | 
|
576  | 
self.build_tree_contents([('tree/mod_'+alpha, 'contents\n')])  | 
|
577  | 
tree.add(['mod_'+alpha], ['file-id-4'])  | 
|
578  | 
||
579  | 
tree.commit('one', rev_id='rev-1')  | 
|
580  | 
||
581  | 
tree.rename_one('ren_'+alpha, 'ren_'+omega)  | 
|
582  | 
tree.remove('del_'+alpha)  | 
|
583  | 
self.build_tree_contents([('tree/add_'+alpha, 'contents\n')])  | 
|
584  | 
tree.add(['add_'+alpha], ['file-id'])  | 
|
585  | 
self.build_tree_contents([('tree/mod_'+alpha, 'contents_mod\n')])  | 
|
586  | 
||
587  | 
diff = self.get_diff(tree.basis_tree(), tree)  | 
|
588  | 
self.assertContainsRe(diff,  | 
|
589  | 
"=== renamed file 'ren_%s' => 'ren_%s'\n"%(autf8, outf8))  | 
|
590  | 
self.assertContainsRe(diff, "=== added file 'add_%s'"%autf8)  | 
|
591  | 
self.assertContainsRe(diff, "=== modified file 'mod_%s'"%autf8)  | 
|
592  | 
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'  | 
593  | 
|
| 
3009.2.9
by Aaron Bentley
 Add tests for Differ  | 
594  | 
|
| 
3009.2.22
by Aaron Bentley
 Update names & docstring  | 
595  | 
class DiffWasIs(DiffPath):  | 
| 
3009.2.15
by Aaron Bentley
 Test differ registration  | 
596  | 
|
597  | 
def diff(self, file_id, old_path, new_path, old_kind, new_kind):  | 
|
598  | 
self.to_file.write('was: ')  | 
|
599  | 
self.to_file.write(self.old_tree.get_file(file_id).read())  | 
|
600  | 
self.to_file.write('is: ')  | 
|
601  | 
self.to_file.write(self.new_tree.get_file(file_id).read())  | 
|
602  | 
        pass
 | 
|
603  | 
||
604  | 
||
| 
3009.2.22
by Aaron Bentley
 Update names & docstring  | 
605  | 
class TestDiffTree(TestCaseWithTransport):  | 
| 
3009.2.9
by Aaron Bentley
 Add tests for Differ  | 
606  | 
|
607  | 
def setUp(self):  | 
|
608  | 
TestCaseWithTransport.setUp(self)  | 
|
609  | 
self.old_tree = self.make_branch_and_tree('old-tree')  | 
|
610  | 
self.old_tree.lock_write()  | 
|
611  | 
self.addCleanup(self.old_tree.unlock)  | 
|
612  | 
self.new_tree = self.make_branch_and_tree('new-tree')  | 
|
613  | 
self.new_tree.lock_write()  | 
|
614  | 
self.addCleanup(self.new_tree.unlock)  | 
|
| 
3009.2.22
by Aaron Bentley
 Update names & docstring  | 
615  | 
self.differ = DiffTree(self.old_tree, self.new_tree, StringIO())  | 
| 
3009.2.9
by Aaron Bentley
 Add tests for Differ  | 
616  | 
|
617  | 
def test_diff_text(self):  | 
|
618  | 
self.build_tree_contents([('old-tree/olddir/',),  | 
|
619  | 
('old-tree/olddir/oldfile', 'old\n')])  | 
|
620  | 
self.old_tree.add('olddir')  | 
|
621  | 
self.old_tree.add('olddir/oldfile', 'file-id')  | 
|
622  | 
self.build_tree_contents([('new-tree/newdir/',),  | 
|
623  | 
('new-tree/newdir/newfile', 'new\n')])  | 
|
624  | 
self.new_tree.add('newdir')  | 
|
625  | 
self.new_tree.add('newdir/newfile', 'file-id')  | 
|
| 
3009.2.22
by Aaron Bentley
 Update names & docstring  | 
626  | 
differ = DiffText(self.old_tree, self.new_tree, StringIO())  | 
| 
3009.2.11
by Aaron Bentley
 Refactor diff to be more pluggable  | 
627  | 
differ.diff_text('file-id', None, 'old label', 'new label')  | 
| 
3009.2.9
by Aaron Bentley
 Add tests for Differ  | 
628  | 
self.assertEqual(  | 
629  | 
'--- 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  | 
630  | 
differ.to_file.getvalue())  | 
631  | 
differ.to_file.seek(0)  | 
|
632  | 
differ.diff_text(None, 'file-id', 'old label', 'new label')  | 
|
| 
3009.2.9
by Aaron Bentley
 Add tests for Differ  | 
633  | 
self.assertEqual(  | 
634  | 
'--- 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  | 
635  | 
differ.to_file.getvalue())  | 
636  | 
differ.to_file.seek(0)  | 
|
637  | 
differ.diff_text('file-id', 'file-id', 'old label', 'new label')  | 
|
| 
3009.2.9
by Aaron Bentley
 Add tests for Differ  | 
638  | 
self.assertEqual(  | 
639  | 
'--- 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  | 
640  | 
differ.to_file.getvalue())  | 
| 
3009.2.9
by Aaron Bentley
 Add tests for Differ  | 
641  | 
|
| 
3087.1.1
by Aaron Bentley
 Diff handles missing files correctly, with no tracebacks  | 
642  | 
def test_diff_deletion(self):  | 
643  | 
self.build_tree_contents([('old-tree/file', 'contents'),  | 
|
644  | 
('new-tree/file', 'contents')])  | 
|
645  | 
self.old_tree.add('file', 'file-id')  | 
|
646  | 
self.new_tree.add('file', 'file-id')  | 
|
647  | 
os.unlink('new-tree/file')  | 
|
648  | 
self.differ.show_diff(None)  | 
|
649  | 
self.assertContainsRe(self.differ.to_file.getvalue(), '-contents')  | 
|
650  | 
||
651  | 
def test_diff_creation(self):  | 
|
652  | 
self.build_tree_contents([('old-tree/file', 'contents'),  | 
|
653  | 
('new-tree/file', 'contents')])  | 
|
654  | 
self.old_tree.add('file', 'file-id')  | 
|
655  | 
self.new_tree.add('file', 'file-id')  | 
|
656  | 
os.unlink('old-tree/file')  | 
|
657  | 
self.differ.show_diff(None)  | 
|
658  | 
self.assertContainsRe(self.differ.to_file.getvalue(), '\+contents')  | 
|
659  | 
||
| 
3009.2.9
by Aaron Bentley
 Add tests for Differ  | 
660  | 
def test_diff_symlink(self):  | 
| 
3009.2.22
by Aaron Bentley
 Update names & docstring  | 
661  | 
differ = DiffSymlink(self.old_tree, self.new_tree, StringIO())  | 
| 
3009.2.11
by Aaron Bentley
 Refactor diff to be more pluggable  | 
662  | 
differ.diff_symlink('old target', None)  | 
| 
3009.2.9
by Aaron Bentley
 Add tests for Differ  | 
663  | 
self.assertEqual("=== target was 'old target'\n",  | 
| 
3009.2.11
by Aaron Bentley
 Refactor diff to be more pluggable  | 
664  | 
differ.to_file.getvalue())  | 
| 
3009.2.9
by Aaron Bentley
 Add tests for Differ  | 
665  | 
|
| 
3009.2.22
by Aaron Bentley
 Update names & docstring  | 
666  | 
differ = DiffSymlink(self.old_tree, self.new_tree, StringIO())  | 
| 
3009.2.11
by Aaron Bentley
 Refactor diff to be more pluggable  | 
667  | 
differ.diff_symlink(None, 'new target')  | 
| 
3009.2.9
by Aaron Bentley
 Add tests for Differ  | 
668  | 
self.assertEqual("=== target is 'new target'\n",  | 
| 
3009.2.11
by Aaron Bentley
 Refactor diff to be more pluggable  | 
669  | 
differ.to_file.getvalue())  | 
670  | 
||
| 
3009.2.22
by Aaron Bentley
 Update names & docstring  | 
671  | 
differ = DiffSymlink(self.old_tree, self.new_tree, StringIO())  | 
| 
3009.2.11
by Aaron Bentley
 Refactor diff to be more pluggable  | 
672  | 
differ.diff_symlink('old target', 'new target')  | 
| 
3009.2.9
by Aaron Bentley
 Add tests for Differ  | 
673  | 
self.assertEqual("=== target changed 'old target' => 'new target'\n",  | 
| 
3009.2.11
by Aaron Bentley
 Refactor diff to be more pluggable  | 
674  | 
differ.to_file.getvalue())  | 
| 
3009.2.9
by Aaron Bentley
 Add tests for Differ  | 
675  | 
|
676  | 
def test_diff(self):  | 
|
677  | 
self.build_tree_contents([('old-tree/olddir/',),  | 
|
678  | 
('old-tree/olddir/oldfile', 'old\n')])  | 
|
679  | 
self.old_tree.add('olddir')  | 
|
680  | 
self.old_tree.add('olddir/oldfile', 'file-id')  | 
|
681  | 
self.build_tree_contents([('new-tree/newdir/',),  | 
|
682  | 
('new-tree/newdir/newfile', 'new\n')])  | 
|
683  | 
self.new_tree.add('newdir')  | 
|
684  | 
self.new_tree.add('newdir/newfile', 'file-id')  | 
|
| 
3009.2.12
by Aaron Bentley
 Associate labels with text diffing only  | 
685  | 
self.differ.diff('file-id', 'olddir/oldfile', 'newdir/newfile')  | 
| 
3009.2.9
by Aaron Bentley
 Add tests for Differ  | 
686  | 
self.assertContainsRe(  | 
687  | 
self.differ.to_file.getvalue(),  | 
|
688  | 
r'--- olddir/oldfile.*\n\+\+\+ newdir/newfile.*\n\@\@ -1,1 \+1,1'  | 
|
689  | 
' \@\@\n-old\n\+new\n\n')  | 
|
690  | 
||
691  | 
def test_diff_kind_change(self):  | 
|
| 
3146.4.3
by Aaron Bentley
 Add missing Symlink requirement  | 
692  | 
self.requireFeature(tests.SymlinkFeature)  | 
| 
3009.2.9
by Aaron Bentley
 Add tests for Differ  | 
693  | 
self.build_tree_contents([('old-tree/olddir/',),  | 
694  | 
('old-tree/olddir/oldfile', 'old\n')])  | 
|
695  | 
self.old_tree.add('olddir')  | 
|
696  | 
self.old_tree.add('olddir/oldfile', 'file-id')  | 
|
697  | 
self.build_tree(['new-tree/newdir/'])  | 
|
698  | 
os.symlink('new', 'new-tree/newdir/newfile')  | 
|
699  | 
self.new_tree.add('newdir')  | 
|
700  | 
self.new_tree.add('newdir/newfile', 'file-id')  | 
|
| 
3009.2.12
by Aaron Bentley
 Associate labels with text diffing only  | 
701  | 
self.differ.diff('file-id', 'olddir/oldfile', 'newdir/newfile')  | 
| 
3009.2.9
by Aaron Bentley
 Add tests for Differ  | 
702  | 
self.assertContainsRe(  | 
703  | 
self.differ.to_file.getvalue(),  | 
|
704  | 
r'--- olddir/oldfile.*\n\+\+\+ newdir/newfile.*\n\@\@ -1,1 \+0,0'  | 
|
705  | 
' \@\@\n-old\n\n')  | 
|
706  | 
self.assertContainsRe(self.differ.to_file.getvalue(),  | 
|
707  | 
"=== target is 'new'\n")  | 
|
708  | 
||
| 
3009.2.19
by Aaron Bentley
 Implement directory diffing  | 
709  | 
def test_diff_directory(self):  | 
710  | 
self.build_tree(['new-tree/new-dir/'])  | 
|
711  | 
self.new_tree.add('new-dir', 'new-dir-id')  | 
|
712  | 
self.differ.diff('new-dir-id', None, 'new-dir')  | 
|
713  | 
self.assertEqual(self.differ.to_file.getvalue(), '')  | 
|
714  | 
||
| 
3009.2.16
by Aaron Bentley
 Test support for extra differs  | 
715  | 
def create_old_new(self):  | 
716  | 
self.build_tree_contents([('old-tree/olddir/',),  | 
|
717  | 
('old-tree/olddir/oldfile', 'old\n')])  | 
|
718  | 
self.old_tree.add('olddir')  | 
|
719  | 
self.old_tree.add('olddir/oldfile', 'file-id')  | 
|
720  | 
self.build_tree_contents([('new-tree/newdir/',),  | 
|
721  | 
('new-tree/newdir/newfile', 'new\n')])  | 
|
722  | 
self.new_tree.add('newdir')  | 
|
723  | 
self.new_tree.add('newdir/newfile', 'file-id')  | 
|
724  | 
||
| 
3009.2.27
by Aaron Bentley
 Use extra_factories instead of extra_diffs  | 
725  | 
def test_register_diff(self):  | 
| 
3009.2.16
by Aaron Bentley
 Test support for extra differs  | 
726  | 
self.create_old_new()  | 
| 
3009.2.27
by Aaron Bentley
 Use extra_factories instead of extra_diffs  | 
727  | 
old_diff_factories = DiffTree.diff_factories  | 
728  | 
DiffTree.diff_factories=old_diff_factories[:]  | 
|
| 
3009.2.28
by Aaron Bentley
 Add from_diff_tree factories  | 
729  | 
DiffTree.diff_factories.insert(0, DiffWasIs.from_diff_tree)  | 
| 
3009.2.16
by Aaron Bentley
 Test support for extra differs  | 
730  | 
try:  | 
| 
3009.2.22
by Aaron Bentley
 Update names & docstring  | 
731  | 
differ = DiffTree(self.old_tree, self.new_tree, StringIO())  | 
| 
3009.2.16
by Aaron Bentley
 Test support for extra differs  | 
732  | 
finally:  | 
| 
3009.2.27
by Aaron Bentley
 Use extra_factories instead of extra_diffs  | 
733  | 
DiffTree.diff_factories = old_diff_factories  | 
| 
3009.2.16
by Aaron Bentley
 Test support for extra differs  | 
734  | 
differ.diff('file-id', 'olddir/oldfile', 'newdir/newfile')  | 
735  | 
self.assertNotContainsRe(  | 
|
736  | 
differ.to_file.getvalue(),  | 
|
737  | 
r'--- olddir/oldfile.*\n\+\+\+ newdir/newfile.*\n\@\@ -1,1 \+1,1'  | 
|
738  | 
' \@\@\n-old\n\+new\n\n')  | 
|
739  | 
self.assertContainsRe(differ.to_file.getvalue(),  | 
|
740  | 
'was: old\nis: new\n')  | 
|
741  | 
||
| 
3009.2.27
by Aaron Bentley
 Use extra_factories instead of extra_diffs  | 
742  | 
def test_extra_factories(self):  | 
| 
3009.2.16
by Aaron Bentley
 Test support for extra differs  | 
743  | 
self.create_old_new()  | 
| 
3009.2.27
by Aaron Bentley
 Use extra_factories instead of extra_diffs  | 
744  | 
differ = DiffTree(self.old_tree, self.new_tree, StringIO(),  | 
| 
3009.2.28
by Aaron Bentley
 Add from_diff_tree factories  | 
745  | 
extra_factories=[DiffWasIs.from_diff_tree])  | 
| 
3009.2.16
by Aaron Bentley
 Test support for extra differs  | 
746  | 
differ.diff('file-id', 'olddir/oldfile', 'newdir/newfile')  | 
747  | 
self.assertNotContainsRe(  | 
|
748  | 
differ.to_file.getvalue(),  | 
|
749  | 
r'--- olddir/oldfile.*\n\+\+\+ newdir/newfile.*\n\@\@ -1,1 \+1,1'  | 
|
750  | 
' \@\@\n-old\n\+new\n\n')  | 
|
751  | 
self.assertContainsRe(differ.to_file.getvalue(),  | 
|
752  | 
'was: old\nis: new\n')  | 
|
753  | 
||
| 
3123.4.1
by Aaron Bentley
 Diff sorts files in alphabetical order  | 
754  | 
def test_alphabetical_order(self):  | 
755  | 
self.build_tree(['new-tree/a-file'])  | 
|
756  | 
self.new_tree.add('a-file')  | 
|
757  | 
self.build_tree(['old-tree/b-file'])  | 
|
758  | 
self.old_tree.add('b-file')  | 
|
759  | 
self.differ.show_diff(None)  | 
|
760  | 
self.assertContainsRe(self.differ.to_file.getvalue(),  | 
|
761  | 
'.*a-file(.|\n)*b-file')  | 
|
762  | 
||
| 
3009.2.9
by Aaron Bentley
 Add tests for Differ  | 
763  | 
|
| 
1711.2.15
by John Arbash Meinel
 Found a couple CDV left  | 
764  | 
class TestPatienceDiffLib(TestCase):  | 
| 
1185.81.1
by John Arbash Meinel
 Adding nofrillsprecisemerge's diff algorithm, wrapped in difflib.  | 
765  | 
|
| 
2781.1.1
by Martin Pool
 merge cpatiencediff from Lukas  | 
766  | 
def setUp(self):  | 
767  | 
super(TestPatienceDiffLib, self).setUp()  | 
|
768  | 
self._unique_lcs = bzrlib._patiencediff_py.unique_lcs_py  | 
|
769  | 
self._recurse_matches = bzrlib._patiencediff_py.recurse_matches_py  | 
|
770  | 
self._PatienceSequenceMatcher = \  | 
|
771  | 
bzrlib._patiencediff_py.PatienceSequenceMatcher_py  | 
|
772  | 
||
| 
3628.1.3
by Lukáš Lalinský
 Add a test  | 
773  | 
def test_diff_unicode_string(self):  | 
774  | 
a = ''.join([unichr(i) for i in range(4000, 4500, 3)])  | 
|
775  | 
b = ''.join([unichr(i) for i in range(4300, 4800, 2)])  | 
|
776  | 
sm = self._PatienceSequenceMatcher(None, a, b)  | 
|
777  | 
mb = sm.get_matching_blocks()  | 
|
778  | 
self.assertEquals(35, len(mb))  | 
|
779  | 
||
| 
1185.81.9
by John Arbash Meinel
 Added (failing) tests for cdv.recurse_matches with common sections,  | 
780  | 
def test_unique_lcs(self):  | 
| 
2781.1.1
by Martin Pool
 merge cpatiencediff from Lukas  | 
781  | 
unique_lcs = self._unique_lcs  | 
| 
1185.81.9
by John Arbash Meinel
 Added (failing) tests for cdv.recurse_matches with common sections,  | 
782  | 
self.assertEquals(unique_lcs('', ''), [])  | 
| 
2781.1.1
by Martin Pool
 merge cpatiencediff from Lukas  | 
783  | 
self.assertEquals(unique_lcs('', 'a'), [])  | 
784  | 
self.assertEquals(unique_lcs('a', ''), [])  | 
|
| 
1185.81.9
by John Arbash Meinel
 Added (failing) tests for cdv.recurse_matches with common sections,  | 
785  | 
self.assertEquals(unique_lcs('a', 'a'), [(0,0)])  | 
786  | 
self.assertEquals(unique_lcs('a', 'b'), [])  | 
|
787  | 
self.assertEquals(unique_lcs('ab', 'ab'), [(0,0), (1,1)])  | 
|
788  | 
self.assertEquals(unique_lcs('abcde', 'cdeab'), [(2,0), (3,1), (4,2)])  | 
|
789  | 
self.assertEquals(unique_lcs('cdeab', 'abcde'), [(0,2), (1,3), (2,4)])  | 
|
| 
3943.8.1
by Marius Kruger
 remove all trailing whitespace from bzr source  | 
790  | 
self.assertEquals(unique_lcs('abXde', 'abYde'), [(0,0), (1,1),  | 
| 
1185.81.9
by John Arbash Meinel
 Added (failing) tests for cdv.recurse_matches with common sections,  | 
791  | 
(3,3), (4,4)])  | 
792  | 
self.assertEquals(unique_lcs('acbac', 'abc'), [(2,1)])  | 
|
793  | 
||
794  | 
def test_recurse_matches(self):  | 
|
795  | 
def test_one(a, b, matches):  | 
|
796  | 
test_matches = []  | 
|
| 
2781.1.1
by Martin Pool
 merge cpatiencediff from Lukas  | 
797  | 
self._recurse_matches(  | 
798  | 
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,  | 
799  | 
self.assertEquals(test_matches, matches)  | 
800  | 
||
| 
1711.2.17
by John Arbash Meinel
 Small cleanups to patience_diff code.  | 
801  | 
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,  | 
802  | 
[(0, 0), (2, 2), (4, 4)])  | 
803  | 
test_one(['a', 'c', 'b', 'a', 'c'], ['a', 'b', 'c'],  | 
|
804  | 
[(0, 0), (2, 1), (4, 2)])  | 
|
| 
2781.1.1
by Martin Pool
 merge cpatiencediff from Lukas  | 
805  | 
        # Even though 'bc' is not unique globally, and is surrounded by
 | 
806  | 
        # non-matching lines, we should still match, because they are locally
 | 
|
807  | 
        # unique
 | 
|
808  | 
test_one('abcdbce', 'afbcgdbce', [(0,0), (1, 2), (2, 3), (3, 5),  | 
|
809  | 
(4, 6), (5, 7), (6, 8)])  | 
|
| 
1185.81.9
by John Arbash Meinel
 Added (failing) tests for cdv.recurse_matches with common sections,  | 
810  | 
|
| 
3943.8.1
by Marius Kruger
 remove all trailing whitespace from bzr source  | 
811  | 
        # recurse_matches doesn't match non-unique
 | 
| 
1185.81.14
by John Arbash Meinel
 Added a main function for running cdvdifflib manually, included tests for unified_diff interfaces  | 
812  | 
        # lines surrounded by bogus text.
 | 
| 
1185.81.24
by Aaron Bentley
 Reoganize patience-related code  | 
813  | 
        # 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  | 
814  | 
|
815  | 
        # This is what it could be
 | 
|
| 
1185.81.9
by John Arbash Meinel
 Added (failing) tests for cdv.recurse_matches with common sections,  | 
816  | 
        #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  | 
817  | 
|
| 
1185.81.9
by John Arbash Meinel
 Added (failing) tests for cdv.recurse_matches with common sections,  | 
818  | 
        # This is what it currently gives:
 | 
819  | 
test_one('aBccDe', 'abccde', [(0,0), (5,5)])  | 
|
820  | 
||
| 
3074.2.10
by John Arbash Meinel
 Cleanup the test cases (Andrew)  | 
821  | 
def assertDiffBlocks(self, a, b, expected_blocks):  | 
822  | 
"""Check that the sequence matcher returns the correct blocks.  | 
|
823  | 
||
824  | 
        :param a: A sequence to match
 | 
|
825  | 
        :param b: Another sequence to match
 | 
|
826  | 
        :param expected_blocks: The expected output, not including the final
 | 
|
827  | 
            matching block (len(a), len(b), 0)
 | 
|
828  | 
        """
 | 
|
829  | 
matcher = self._PatienceSequenceMatcher(None, a, b)  | 
|
830  | 
blocks = matcher.get_matching_blocks()  | 
|
831  | 
last = blocks.pop()  | 
|
832  | 
self.assertEqual((len(a), len(b), 0), last)  | 
|
833  | 
self.assertEqual(expected_blocks, blocks)  | 
|
834  | 
||
| 
1185.81.1
by John Arbash Meinel
 Adding nofrillsprecisemerge's diff algorithm, wrapped in difflib.  | 
835  | 
def test_matching_blocks(self):  | 
| 
1185.81.2
by John Arbash Meinel
 A couple small tests.  | 
836  | 
        # Some basic matching tests
 | 
| 
3074.2.10
by John Arbash Meinel
 Cleanup the test cases (Andrew)  | 
837  | 
self.assertDiffBlocks('', '', [])  | 
838  | 
self.assertDiffBlocks([], [], [])  | 
|
839  | 
self.assertDiffBlocks('abc', '', [])  | 
|
840  | 
self.assertDiffBlocks('', 'abc', [])  | 
|
841  | 
self.assertDiffBlocks('abcd', 'abcd', [(0, 0, 4)])  | 
|
842  | 
self.assertDiffBlocks('abcd', 'abce', [(0, 0, 3)])  | 
|
843  | 
self.assertDiffBlocks('eabc', 'abce', [(1, 0, 3)])  | 
|
844  | 
self.assertDiffBlocks('eabce', 'abce', [(1, 0, 4)])  | 
|
845  | 
self.assertDiffBlocks('abcde', 'abXde', [(0, 0, 2), (3, 3, 2)])  | 
|
846  | 
self.assertDiffBlocks('abcde', 'abXYZde', [(0, 0, 2), (3, 5, 2)])  | 
|
847  | 
self.assertDiffBlocks('abde', 'abXYZde', [(0, 0, 2), (2, 5, 2)])  | 
|
848  | 
        # This may check too much, but it checks to see that
 | 
|
| 
1185.81.3
by John Arbash Meinel
 Adding tests for checking opcodes.  | 
849  | 
        # a copied block stays attached to the previous section,
 | 
850  | 
        # not the later one.
 | 
|
851  | 
        # difflib would tend to grab the trailing longest match
 | 
|
852  | 
        # which would make the diff not look right
 | 
|
| 
3074.2.10
by John Arbash Meinel
 Cleanup the test cases (Andrew)  | 
853  | 
self.assertDiffBlocks('abcdefghijklmnop', 'abcdefxydefghijklmnop',  | 
854  | 
[(0, 0, 6), (6, 11, 10)])  | 
|
| 
1185.81.1
by John Arbash Meinel
 Adding nofrillsprecisemerge's diff algorithm, wrapped in difflib.  | 
855  | 
|
| 
1185.81.2
by John Arbash Meinel
 A couple small tests.  | 
856  | 
        # make sure it supports passing in lists
 | 
| 
3074.2.10
by John Arbash Meinel
 Cleanup the test cases (Andrew)  | 
857  | 
self.assertDiffBlocks(  | 
| 
1185.81.29
by Aaron Bentley
 Fix style issues and duplicated tests  | 
858  | 
['hello there\n',  | 
859  | 
'world\n',  | 
|
860  | 
'how are you today?\n'],  | 
|
861  | 
['hello there\n',  | 
|
862  | 
'how are you today?\n'],  | 
|
| 
1185.81.2
by John Arbash Meinel
 A couple small tests.  | 
863  | 
[(0, 0, 1), (2, 1, 1)])  | 
| 
1185.81.1
by John Arbash Meinel
 Adding nofrillsprecisemerge's diff algorithm, wrapped in difflib.  | 
864  | 
|
| 
1711.2.21
by John Arbash Meinel
 Cleanup patiencediff, remove the use of difflib.SequenceMatcher.  | 
865  | 
        # non unique lines surrounded by non-matching lines
 | 
866  | 
        # won't be found
 | 
|
| 
3074.2.10
by John Arbash Meinel
 Cleanup the test cases (Andrew)  | 
867  | 
self.assertDiffBlocks('aBccDe', 'abccde', [(0,0,1), (5,5,1)])  | 
| 
1711.2.21
by John Arbash Meinel
 Cleanup patiencediff, remove the use of difflib.SequenceMatcher.  | 
868  | 
|
869  | 
        # But they only need to be locally unique
 | 
|
| 
3074.2.10
by John Arbash Meinel
 Cleanup the test cases (Andrew)  | 
870  | 
self.assertDiffBlocks('aBcDec', 'abcdec', [(0,0,1), (2,2,1), (4,4,2)])  | 
| 
1711.2.21
by John Arbash Meinel
 Cleanup patiencediff, remove the use of difflib.SequenceMatcher.  | 
871  | 
|
872  | 
        # non unique blocks won't be matched
 | 
|
| 
3074.2.10
by John Arbash Meinel
 Cleanup the test cases (Andrew)  | 
873  | 
self.assertDiffBlocks('aBcdEcdFg', 'abcdecdfg', [(0,0,1), (8,8,1)])  | 
| 
1711.2.21
by John Arbash Meinel
 Cleanup patiencediff, remove the use of difflib.SequenceMatcher.  | 
874  | 
|
875  | 
        # but locally unique ones will
 | 
|
| 
3074.2.10
by John Arbash Meinel
 Cleanup the test cases (Andrew)  | 
876  | 
self.assertDiffBlocks('aBcdEeXcdFg', 'abcdecdfg', [(0,0,1), (2,2,2),  | 
| 
1711.2.21
by John Arbash Meinel
 Cleanup patiencediff, remove the use of difflib.SequenceMatcher.  | 
877  | 
(5,4,1), (7,5,2), (10,8,1)])  | 
878  | 
||
| 
3074.2.10
by John Arbash Meinel
 Cleanup the test cases (Andrew)  | 
879  | 
self.assertDiffBlocks('abbabbXd', 'cabbabxd', [(7,7,1)])  | 
880  | 
self.assertDiffBlocks('abbabbbb', 'cabbabbc', [])  | 
|
881  | 
self.assertDiffBlocks('bbbbbbbb', 'cbbbbbbc', [])  | 
|
| 
1185.81.11
by John Arbash Meinel
 Found some edge cases that weren't being matched.  | 
882  | 
|
| 
3074.2.1
by John Arbash Meinel
 Change the C PatienceDiff implementation to support arbitrary objects.  | 
883  | 
def test_matching_blocks_tuples(self):  | 
884  | 
        # Some basic matching tests
 | 
|
| 
3074.2.10
by John Arbash Meinel
 Cleanup the test cases (Andrew)  | 
885  | 
self.assertDiffBlocks([], [], [])  | 
886  | 
self.assertDiffBlocks([('a',), ('b',), ('c,')], [], [])  | 
|
887  | 
self.assertDiffBlocks([], [('a',), ('b',), ('c,')], [])  | 
|
888  | 
self.assertDiffBlocks([('a',), ('b',), ('c,')],  | 
|
889  | 
[('a',), ('b',), ('c,')],  | 
|
890  | 
[(0, 0, 3)])  | 
|
891  | 
self.assertDiffBlocks([('a',), ('b',), ('c,')],  | 
|
892  | 
[('a',), ('b',), ('d,')],  | 
|
893  | 
[(0, 0, 2)])  | 
|
894  | 
self.assertDiffBlocks([('d',), ('b',), ('c,')],  | 
|
895  | 
[('a',), ('b',), ('c,')],  | 
|
896  | 
[(1, 1, 2)])  | 
|
897  | 
self.assertDiffBlocks([('d',), ('a',), ('b',), ('c,')],  | 
|
898  | 
[('a',), ('b',), ('c,')],  | 
|
899  | 
[(1, 0, 3)])  | 
|
900  | 
self.assertDiffBlocks([('a', 'b'), ('c', 'd'), ('e', 'f')],  | 
|
901  | 
[('a', 'b'), ('c', 'X'), ('e', 'f')],  | 
|
902  | 
[(0, 0, 1), (2, 2, 1)])  | 
|
903  | 
self.assertDiffBlocks([('a', 'b'), ('c', 'd'), ('e', 'f')],  | 
|
904  | 
[('a', 'b'), ('c', 'dX'), ('e', 'f')],  | 
|
905  | 
[(0, 0, 1), (2, 2, 1)])  | 
|
| 
3074.2.1
by John Arbash Meinel
 Change the C PatienceDiff implementation to support arbitrary objects.  | 
906  | 
|
| 
1185.81.3
by John Arbash Meinel
 Adding tests for checking opcodes.  | 
907  | 
def test_opcodes(self):  | 
| 
1711.2.10
by John Arbash Meinel
 Clarify the patience tests a little bit.  | 
908  | 
def chk_ops(a, b, expected_codes):  | 
| 
2781.1.1
by Martin Pool
 merge cpatiencediff from Lukas  | 
909  | 
s = self._PatienceSequenceMatcher(None, a, b)  | 
| 
1711.2.10
by John Arbash Meinel
 Clarify the patience tests a little bit.  | 
910  | 
self.assertEquals(expected_codes, s.get_opcodes())  | 
| 
1185.81.3
by John Arbash Meinel
 Adding tests for checking opcodes.  | 
911  | 
|
912  | 
chk_ops('', '', [])  | 
|
913  | 
chk_ops([], [], [])  | 
|
| 
2781.1.1
by Martin Pool
 merge cpatiencediff from Lukas  | 
914  | 
chk_ops('abc', '', [('delete', 0,3, 0,0)])  | 
915  | 
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,  | 
916  | 
chk_ops('abcd', 'abcd', [('equal', 0,4, 0,4)])  | 
| 
1185.81.29
by Aaron Bentley
 Fix style issues and duplicated tests  | 
917  | 
chk_ops('abcd', 'abce', [('equal', 0,3, 0,3),  | 
918  | 
('replace', 3,4, 3,4)  | 
|
919  | 
                                ])
 | 
|
920  | 
chk_ops('eabc', 'abce', [('delete', 0,1, 0,0),  | 
|
921  | 
('equal', 1,4, 0,3),  | 
|
922  | 
('insert', 4,4, 3,4)  | 
|
923  | 
                                ])
 | 
|
924  | 
chk_ops('eabce', 'abce', [('delete', 0,1, 0,0),  | 
|
925  | 
('equal', 1,5, 0,4)  | 
|
| 
1185.81.3
by John Arbash Meinel
 Adding tests for checking opcodes.  | 
926  | 
                                 ])
 | 
| 
1185.81.29
by Aaron Bentley
 Fix style issues and duplicated tests  | 
927  | 
chk_ops('abcde', 'abXde', [('equal', 0,2, 0,2),  | 
928  | 
('replace', 2,3, 2,3),  | 
|
929  | 
('equal', 3,5, 3,5)  | 
|
| 
1185.81.3
by John Arbash Meinel
 Adding tests for checking opcodes.  | 
930  | 
                                  ])
 | 
| 
1185.81.29
by Aaron Bentley
 Fix style issues and duplicated tests  | 
931  | 
chk_ops('abcde', 'abXYZde', [('equal', 0,2, 0,2),  | 
932  | 
('replace', 2,3, 2,5),  | 
|
933  | 
('equal', 3,5, 5,7)  | 
|
| 
1185.81.3
by John Arbash Meinel
 Adding tests for checking opcodes.  | 
934  | 
                                    ])
 | 
| 
1185.81.29
by Aaron Bentley
 Fix style issues and duplicated tests  | 
935  | 
chk_ops('abde', 'abXYZde', [('equal', 0,2, 0,2),  | 
936  | 
('insert', 2,2, 2,5),  | 
|
937  | 
('equal', 2,4, 5,7)  | 
|
| 
1185.81.3
by John Arbash Meinel
 Adding tests for checking opcodes.  | 
938  | 
                                   ])
 | 
939  | 
chk_ops('abcdefghijklmnop', 'abcdefxydefghijklmnop',  | 
|
| 
1185.81.29
by Aaron Bentley
 Fix style issues and duplicated tests  | 
940  | 
[('equal', 0,6, 0,6),  | 
941  | 
('insert', 6,6, 6,11),  | 
|
942  | 
('equal', 6,16, 11,21)  | 
|
| 
1185.81.3
by John Arbash Meinel
 Adding tests for checking opcodes.  | 
943  | 
                ])
 | 
944  | 
chk_ops(  | 
|
945  | 
[ 'hello there\n'  | 
|
946  | 
, 'world\n'  | 
|
947  | 
, 'how are you today?\n'],  | 
|
948  | 
[ 'hello there\n'  | 
|
949  | 
, 'how are you today?\n'],  | 
|
| 
1185.81.29
by Aaron Bentley
 Fix style issues and duplicated tests  | 
950  | 
[('equal', 0,1, 0,1),  | 
951  | 
('delete', 1,2, 1,1),  | 
|
| 
1711.2.21
by John Arbash Meinel
 Cleanup patiencediff, remove the use of difflib.SequenceMatcher.  | 
952  | 
('equal', 2,3, 1,2),  | 
| 
1185.81.9
by John Arbash Meinel
 Added (failing) tests for cdv.recurse_matches with common sections,  | 
953  | 
                ])
 | 
| 
3943.8.1
by Marius Kruger
 remove all trailing whitespace from bzr source  | 
954  | 
chk_ops('aBccDe', 'abccde',  | 
| 
1185.81.29
by Aaron Bentley
 Fix style issues and duplicated tests  | 
955  | 
[('equal', 0,1, 0,1),  | 
| 
1711.2.21
by John Arbash Meinel
 Cleanup patiencediff, remove the use of difflib.SequenceMatcher.  | 
956  | 
('replace', 1,5, 1,5),  | 
957  | 
('equal', 5,6, 5,6),  | 
|
958  | 
                ])
 | 
|
| 
3943.8.1
by Marius Kruger
 remove all trailing whitespace from bzr source  | 
959  | 
chk_ops('aBcDec', 'abcdec',  | 
| 
1711.2.21
by John Arbash Meinel
 Cleanup patiencediff, remove the use of difflib.SequenceMatcher.  | 
960  | 
[('equal', 0,1, 0,1),  | 
| 
1185.81.29
by Aaron Bentley
 Fix style issues and duplicated tests  | 
961  | 
('replace', 1,2, 1,2),  | 
| 
1711.2.21
by John Arbash Meinel
 Cleanup patiencediff, remove the use of difflib.SequenceMatcher.  | 
962  | 
('equal', 2,3, 2,3),  | 
963  | 
('replace', 3,4, 3,4),  | 
|
964  | 
('equal', 4,6, 4,6),  | 
|
| 
1185.81.3
by John Arbash Meinel
 Adding tests for checking opcodes.  | 
965  | 
                ])
 | 
| 
3943.8.1
by Marius Kruger
 remove all trailing whitespace from bzr source  | 
966  | 
chk_ops('aBcdEcdFg', 'abcdecdfg',  | 
| 
1185.81.29
by Aaron Bentley
 Fix style issues and duplicated tests  | 
967  | 
[('equal', 0,1, 0,1),  | 
| 
1711.2.21
by John Arbash Meinel
 Cleanup patiencediff, remove the use of difflib.SequenceMatcher.  | 
968  | 
('replace', 1,8, 1,8),  | 
| 
1185.81.29
by Aaron Bentley
 Fix style issues and duplicated tests  | 
969  | 
('equal', 8,9, 8,9)  | 
| 
1185.81.10
by John Arbash Meinel
 Added some more test cases.  | 
970  | 
                ])
 | 
| 
3943.8.1
by Marius Kruger
 remove all trailing whitespace from bzr source  | 
971  | 
chk_ops('aBcdEeXcdFg', 'abcdecdfg',  | 
| 
1711.2.21
by John Arbash Meinel
 Cleanup patiencediff, remove the use of difflib.SequenceMatcher.  | 
972  | 
[('equal', 0,1, 0,1),  | 
973  | 
('replace', 1,2, 1,2),  | 
|
974  | 
('equal', 2,4, 2,4),  | 
|
975  | 
('delete', 4,5, 4,4),  | 
|
976  | 
('equal', 5,6, 4,5),  | 
|
977  | 
('delete', 6,7, 5,5),  | 
|
978  | 
('equal', 7,9, 5,7),  | 
|
979  | 
('replace', 9,10, 7,8),  | 
|
980  | 
('equal', 10,11, 8,9)  | 
|
981  | 
                ])
 | 
|
| 
1185.81.10
by John Arbash Meinel
 Added some more test cases.  | 
982  | 
|
| 
2781.1.1
by Martin Pool
 merge cpatiencediff from Lukas  | 
983  | 
def test_grouped_opcodes(self):  | 
984  | 
def chk_ops(a, b, expected_codes, n=3):  | 
|
985  | 
s = self._PatienceSequenceMatcher(None, a, b)  | 
|
986  | 
self.assertEquals(expected_codes, list(s.get_grouped_opcodes(n)))  | 
|
987  | 
||
988  | 
chk_ops('', '', [])  | 
|
989  | 
chk_ops([], [], [])  | 
|
990  | 
chk_ops('abc', '', [[('delete', 0,3, 0,0)]])  | 
|
991  | 
chk_ops('', 'abc', [[('insert', 0,0, 0,3)]])  | 
|
992  | 
chk_ops('abcd', 'abcd', [])  | 
|
993  | 
chk_ops('abcd', 'abce', [[('equal', 0,3, 0,3),  | 
|
994  | 
('replace', 3,4, 3,4)  | 
|
995  | 
                                 ]])
 | 
|
996  | 
chk_ops('eabc', 'abce', [[('delete', 0,1, 0,0),  | 
|
997  | 
('equal', 1,4, 0,3),  | 
|
998  | 
('insert', 4,4, 3,4)  | 
|
999  | 
                                ]])
 | 
|
1000  | 
chk_ops('abcdefghijklmnop', 'abcdefxydefghijklmnop',  | 
|
1001  | 
[[('equal', 3,6, 3,6),  | 
|
1002  | 
('insert', 6,6, 6,11),  | 
|
1003  | 
('equal', 6,9, 11,14)  | 
|
1004  | 
                  ]])
 | 
|
1005  | 
chk_ops('abcdefghijklmnop', 'abcdefxydefghijklmnop',  | 
|
1006  | 
[[('equal', 2,6, 2,6),  | 
|
1007  | 
('insert', 6,6, 6,11),  | 
|
1008  | 
('equal', 6,10, 11,15)  | 
|
1009  | 
]], 4)  | 
|
1010  | 
chk_ops('Xabcdef', 'abcdef',  | 
|
1011  | 
[[('delete', 0,1, 0,0),  | 
|
1012  | 
('equal', 1,4, 0,3)  | 
|
1013  | 
                  ]])
 | 
|
1014  | 
chk_ops('abcdef', 'abcdefX',  | 
|
1015  | 
[[('equal', 3,6, 3,6),  | 
|
1016  | 
('insert', 6,6, 6,7)  | 
|
1017  | 
                  ]])
 | 
|
1018  | 
||
1019  | 
||
| 
1185.81.16
by John Arbash Meinel
 Added tests, and an assert check to make sure ranges are always increasing.  | 
1020  | 
def test_multiple_ranges(self):  | 
1021  | 
        # There was an earlier bug where we used a bad set of ranges,
 | 
|
1022  | 
        # this triggers that specific bug, to make sure it doesn't regress
 | 
|
| 
3074.2.10
by John Arbash Meinel
 Cleanup the test cases (Andrew)  | 
1023  | 
self.assertDiffBlocks('abcdefghijklmnop',  | 
1024  | 
'abcXghiYZQRSTUVWXYZijklmnop',  | 
|
1025  | 
[(0, 0, 3), (6, 4, 3), (9, 20, 7)])  | 
|
1026  | 
||
1027  | 
self.assertDiffBlocks('ABCd efghIjk L',  | 
|
1028  | 
'AxyzBCn mo pqrstuvwI1 2 L',  | 
|
1029  | 
[(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.  | 
1030  | 
|
| 
1711.2.8
by John Arbash Meinel
 rot13 the code snippet to help with clarity.  | 
1031  | 
        # These are rot13 code snippets.
 | 
| 
3074.2.10
by John Arbash Meinel
 Cleanup the test cases (Andrew)  | 
1032  | 
self.assertDiffBlocks('''\  | 
| 
1711.2.8
by John Arbash Meinel
 rot13 the code snippet to help with clarity.  | 
1033  | 
    trg nqqrq jura lbh nqq n svyr va gur qverpgbel.
 | 
1034  | 
    """
 | 
|
1035  | 
    gnxrf_netf = ['svyr*']
 | 
|
1036  | 
    gnxrf_bcgvbaf = ['ab-erphefr']
 | 
|
| 
3943.8.1
by Marius Kruger
 remove all trailing whitespace from bzr source  | 
1037  | 
|
| 
1711.2.8
by John Arbash Meinel
 rot13 the code snippet to help with clarity.  | 
1038  | 
    qrs eha(frys, svyr_yvfg, ab_erphefr=Snyfr):
 | 
1039  | 
        sebz omeyvo.nqq vzcbeg fzneg_nqq, nqq_ercbegre_cevag, nqq_ercbegre_ahyy
 | 
|
1040  | 
        vs vf_dhvrg():
 | 
|
1041  | 
            ercbegre = nqq_ercbegre_ahyy
 | 
|
1042  | 
        ryfr:
 | 
|
1043  | 
            ercbegre = nqq_ercbegre_cevag
 | 
|
1044  | 
        fzneg_nqq(svyr_yvfg, abg ab_erphefr, ercbegre)
 | 
|
1045  | 
||
1046  | 
||
1047  | 
pynff pzq_zxqve(Pbzznaq):
 | 
|
1048  | 
'''.splitlines(True), '''\  | 
|
1049  | 
    trg nqqrq jura lbh nqq n svyr va gur qverpgbel.
 | 
|
1050  | 
||
| 
3943.8.1
by Marius Kruger
 remove all trailing whitespace from bzr source  | 
1051  | 
    --qel-eha jvyy fubj juvpu svyrf jbhyq or nqqrq, ohg abg npghnyyl
 | 
| 
1711.2.8
by John Arbash Meinel
 rot13 the code snippet to help with clarity.  | 
1052  | 
    nqq gurz.
 | 
1053  | 
    """
 | 
|
1054  | 
    gnxrf_netf = ['svyr*']
 | 
|
1055  | 
    gnxrf_bcgvbaf = ['ab-erphefr', 'qel-eha']
 | 
|
1056  | 
||
1057  | 
    qrs eha(frys, svyr_yvfg, ab_erphefr=Snyfr, qel_eha=Snyfr):
 | 
|
1058  | 
        vzcbeg omeyvo.nqq
 | 
|
1059  | 
||
1060  | 
        vs qel_eha:
 | 
|
1061  | 
            vs vf_dhvrg():
 | 
|
1062  | 
                # Guvf vf cbvagyrff, ohg V'q engure abg envfr na reebe
 | 
|
1063  | 
                npgvba = omeyvo.nqq.nqq_npgvba_ahyy
 | 
|
1064  | 
            ryfr:
 | 
|
1065  | 
  npgvba = omeyvo.nqq.nqq_npgvba_cevag
 | 
|
1066  | 
        ryvs vf_dhvrg():
 | 
|
1067  | 
            npgvba = omeyvo.nqq.nqq_npgvba_nqq
 | 
|
1068  | 
        ryfr:
 | 
|
1069  | 
       npgvba = omeyvo.nqq.nqq_npgvba_nqq_naq_cevag
 | 
|
1070  | 
||
1071  | 
        omeyvo.nqq.fzneg_nqq(svyr_yvfg, abg ab_erphefr, npgvba)
 | 
|
1072  | 
||
1073  | 
||
1074  | 
pynff pzq_zxqve(Pbzznaq):
 | 
|
| 
1185.81.16
by John Arbash Meinel
 Added tests, and an assert check to make sure ranges are always increasing.  | 
1075  | 
'''.splitlines(True)  | 
1076  | 
, [(0,0,1), (1, 4, 2), (9, 19, 1), (12, 23, 3)])  | 
|
1077  | 
||
| 
1711.2.9
by John Arbash Meinel
 Rename cdv => patience  | 
1078  | 
def test_patience_unified_diff(self):  | 
| 
1185.81.29
by Aaron Bentley
 Fix style issues and duplicated tests  | 
1079  | 
txt_a = ['hello there\n',  | 
1080  | 
'world\n',  | 
|
1081  | 
'how are you today?\n']  | 
|
1082  | 
txt_b = ['hello there\n',  | 
|
1083  | 
'how are you today?\n']  | 
|
| 
1711.2.20
by John Arbash Meinel
 Late bind to patiencediff objects to make it easier to plug-in  | 
1084  | 
unified_diff = bzrlib.patiencediff.unified_diff  | 
| 
2781.1.1
by Martin Pool
 merge cpatiencediff from Lukas  | 
1085  | 
psm = self._PatienceSequenceMatcher  | 
| 
3922.1.3
by John Arbash Meinel
 fix some odd spacing.  | 
1086  | 
self.assertEquals(['--- \n',  | 
| 
3922.1.2
by John Arbash Meinel
 Update the test cases for the new patience diff code.  | 
1087  | 
'+++ \n',  | 
| 
1185.81.29
by Aaron Bentley
 Fix style issues and duplicated tests  | 
1088  | 
'@@ -1,3 +1,2 @@\n',  | 
1089  | 
' hello there\n',  | 
|
1090  | 
'-world\n',  | 
|
1091  | 
' 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  | 
1092  | 
                          ]
 | 
| 
1711.2.20
by John Arbash Meinel
 Late bind to patiencediff objects to make it easier to plug-in  | 
1093  | 
, list(unified_diff(txt_a, txt_b,  | 
1094  | 
sequencematcher=psm)))  | 
|
| 
1185.81.14
by John Arbash Meinel
 Added a main function for running cdvdifflib manually, included tests for unified_diff interfaces  | 
1095  | 
txt_a = map(lambda x: x+'\n', 'abcdefghijklmnop')  | 
1096  | 
txt_b = map(lambda x: x+'\n', 'abcdefxydefghijklmnop')  | 
|
1097  | 
        # This is the result with LongestCommonSubstring matching
 | 
|
| 
3922.1.2
by John Arbash Meinel
 Update the test cases for the new patience diff code.  | 
1098  | 
self.assertEquals(['--- \n',  | 
1099  | 
'+++ \n',  | 
|
| 
1185.81.29
by Aaron Bentley
 Fix style issues and duplicated tests  | 
1100  | 
'@@ -1,6 +1,11 @@\n',  | 
1101  | 
' a\n',  | 
|
1102  | 
' b\n',  | 
|
1103  | 
' c\n',  | 
|
1104  | 
'+d\n',  | 
|
1105  | 
'+e\n',  | 
|
1106  | 
'+f\n',  | 
|
1107  | 
'+x\n',  | 
|
1108  | 
'+y\n',  | 
|
1109  | 
' d\n',  | 
|
1110  | 
' e\n',  | 
|
1111  | 
' f\n']  | 
|
| 
1185.81.14
by John Arbash Meinel
 Added a main function for running cdvdifflib manually, included tests for unified_diff interfaces  | 
1112  | 
, list(unified_diff(txt_a, txt_b)))  | 
| 
1711.2.9
by John Arbash Meinel
 Rename cdv => patience  | 
1113  | 
        # And the patience diff
 | 
| 
3922.1.2
by John Arbash Meinel
 Update the test cases for the new patience diff code.  | 
1114  | 
self.assertEquals(['--- \n',  | 
1115  | 
'+++ \n',  | 
|
| 
1185.81.29
by Aaron Bentley
 Fix style issues and duplicated tests  | 
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(txt_a, txt_b,  | 
| 
1711.2.20
by John Arbash Meinel
 Late bind to patiencediff objects to make it easier to plug-in  | 
1130  | 
sequencematcher=psm)))  | 
| 
1185.81.25
by Aaron Bentley
 Clean up test_diff  | 
1131  | 
|
| 
3922.1.2
by John Arbash Meinel
 Update the test cases for the new patience diff code.  | 
1132  | 
def test_patience_unified_diff_with_dates(self):  | 
1133  | 
txt_a = ['hello there\n',  | 
|
1134  | 
'world\n',  | 
|
1135  | 
'how are you today?\n']  | 
|
1136  | 
txt_b = ['hello there\n',  | 
|
1137  | 
'how are you today?\n']  | 
|
1138  | 
unified_diff = bzrlib.patiencediff.unified_diff  | 
|
1139  | 
psm = self._PatienceSequenceMatcher  | 
|
| 
3922.1.4
by John Arbash Meinel
 It turns out that internal_diff worked around the trailing whitespace problem  | 
1140  | 
self.assertEquals(['--- a\t2008-08-08\n',  | 
1141  | 
'+++ b\t2008-09-09\n',  | 
|
| 
3922.1.2
by John Arbash Meinel
 Update the test cases for the new patience diff code.  | 
1142  | 
'@@ -1,3 +1,2 @@\n',  | 
1143  | 
' hello there\n',  | 
|
1144  | 
'-world\n',  | 
|
1145  | 
' how are you today?\n'  | 
|
1146  | 
                          ]
 | 
|
1147  | 
, list(unified_diff(txt_a, txt_b,  | 
|
1148  | 
fromfile='a', tofile='b',  | 
|
1149  | 
fromfiledate='2008-08-08',  | 
|
1150  | 
tofiledate='2008-09-09',  | 
|
1151  | 
sequencematcher=psm)))  | 
|
1152  | 
||
| 
1185.81.14
by John Arbash Meinel
 Added a main function for running cdvdifflib manually, included tests for unified_diff interfaces  | 
1153  | 
|
| 
2781.1.1
by Martin Pool
 merge cpatiencediff from Lukas  | 
1154  | 
class TestPatienceDiffLib_c(TestPatienceDiffLib):  | 
1155  | 
||
1156  | 
_test_needs_features = [CompiledPatienceDiffFeature]  | 
|
1157  | 
||
1158  | 
def setUp(self):  | 
|
1159  | 
super(TestPatienceDiffLib_c, self).setUp()  | 
|
1160  | 
import bzrlib._patiencediff_c  | 
|
1161  | 
self._unique_lcs = bzrlib._patiencediff_c.unique_lcs_c  | 
|
1162  | 
self._recurse_matches = bzrlib._patiencediff_c.recurse_matches_c  | 
|
1163  | 
self._PatienceSequenceMatcher = \  | 
|
1164  | 
bzrlib._patiencediff_c.PatienceSequenceMatcher_c  | 
|
1165  | 
||
| 
3074.2.3
by John Arbash Meinel
 Enable some error checking, and small amount of code cleanup.  | 
1166  | 
def test_unhashable(self):  | 
1167  | 
"""We should get a proper exception here."""  | 
|
| 
3074.2.10
by John Arbash Meinel
 Cleanup the test cases (Andrew)  | 
1168  | 
        # We need to be able to hash items in the sequence, lists are
 | 
1169  | 
        # unhashable, and thus cannot be diffed
 | 
|
| 
3074.2.3
by John Arbash Meinel
 Enable some error checking, and small amount of code cleanup.  | 
1170  | 
e = self.assertRaises(TypeError, self._PatienceSequenceMatcher,  | 
1171  | 
None, [[]], [])  | 
|
| 
3074.2.10
by John Arbash Meinel
 Cleanup the test cases (Andrew)  | 
1172  | 
e = self.assertRaises(TypeError, self._PatienceSequenceMatcher,  | 
1173  | 
None, ['valid', []], [])  | 
|
1174  | 
e = self.assertRaises(TypeError, self._PatienceSequenceMatcher,  | 
|
1175  | 
None, ['valid'], [[]])  | 
|
1176  | 
e = self.assertRaises(TypeError, self._PatienceSequenceMatcher,  | 
|
1177  | 
None, ['valid'], ['valid', []])  | 
|
| 
3074.2.3
by John Arbash Meinel
 Enable some error checking, and small amount of code cleanup.  | 
1178  | 
|
| 
2781.1.1
by Martin Pool
 merge cpatiencediff from Lukas  | 
1179  | 
|
| 
1711.2.15
by John Arbash Meinel
 Found a couple CDV left  | 
1180  | 
class TestPatienceDiffLibFiles(TestCaseInTempDir):  | 
| 
1185.81.14
by John Arbash Meinel
 Added a main function for running cdvdifflib manually, included tests for unified_diff interfaces  | 
1181  | 
|
| 
2781.1.1
by Martin Pool
 merge cpatiencediff from Lukas  | 
1182  | 
def setUp(self):  | 
1183  | 
super(TestPatienceDiffLibFiles, self).setUp()  | 
|
1184  | 
self._PatienceSequenceMatcher = \  | 
|
1185  | 
bzrlib._patiencediff_py.PatienceSequenceMatcher_py  | 
|
1186  | 
||
| 
1711.2.9
by John Arbash Meinel
 Rename cdv => patience  | 
1187  | 
def test_patience_unified_diff_files(self):  | 
| 
1185.81.29
by Aaron Bentley
 Fix style issues and duplicated tests  | 
1188  | 
txt_a = ['hello there\n',  | 
1189  | 
'world\n',  | 
|
1190  | 
'how are you today?\n']  | 
|
1191  | 
txt_b = ['hello there\n',  | 
|
1192  | 
'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  | 
1193  | 
open('a1', 'wb').writelines(txt_a)  | 
1194  | 
open('b1', 'wb').writelines(txt_b)  | 
|
1195  | 
||
| 
1711.2.20
by John Arbash Meinel
 Late bind to patiencediff objects to make it easier to plug-in  | 
1196  | 
unified_diff_files = bzrlib.patiencediff.unified_diff_files  | 
| 
2781.1.1
by Martin Pool
 merge cpatiencediff from Lukas  | 
1197  | 
psm = self._PatienceSequenceMatcher  | 
| 
3922.1.2
by John Arbash Meinel
 Update the test cases for the new patience diff code.  | 
1198  | 
self.assertEquals(['--- a1\n',  | 
1199  | 
'+++ b1\n',  | 
|
| 
1185.81.29
by Aaron Bentley
 Fix style issues and duplicated tests  | 
1200  | 
'@@ -1,3 +1,2 @@\n',  | 
1201  | 
' hello there\n',  | 
|
1202  | 
'-world\n',  | 
|
1203  | 
' 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  | 
1204  | 
                          ]
 | 
| 
1185.81.25
by Aaron Bentley
 Clean up test_diff  | 
1205  | 
, 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  | 
1206  | 
sequencematcher=psm)))  | 
| 
1185.81.14
by John Arbash Meinel
 Added a main function for running cdvdifflib manually, included tests for unified_diff interfaces  | 
1207  | 
|
1208  | 
txt_a = map(lambda x: x+'\n', 'abcdefghijklmnop')  | 
|
1209  | 
txt_b = map(lambda x: x+'\n', 'abcdefxydefghijklmnop')  | 
|
1210  | 
open('a2', 'wb').writelines(txt_a)  | 
|
1211  | 
open('b2', 'wb').writelines(txt_b)  | 
|
1212  | 
||
1213  | 
        # This is the result with LongestCommonSubstring matching
 | 
|
| 
3922.1.2
by John Arbash Meinel
 Update the test cases for the new patience diff code.  | 
1214  | 
self.assertEquals(['--- a2\n',  | 
1215  | 
'+++ b2\n',  | 
|
| 
1185.81.29
by Aaron Bentley
 Fix style issues and duplicated tests  | 
1216  | 
'@@ -1,6 +1,11 @@\n',  | 
1217  | 
' a\n',  | 
|
1218  | 
' b\n',  | 
|
1219  | 
' c\n',  | 
|
1220  | 
'+d\n',  | 
|
1221  | 
'+e\n',  | 
|
1222  | 
'+f\n',  | 
|
1223  | 
'+x\n',  | 
|
1224  | 
'+y\n',  | 
|
1225  | 
' d\n',  | 
|
1226  | 
' e\n',  | 
|
1227  | 
' f\n']  | 
|
| 
1185.81.14
by John Arbash Meinel
 Added a main function for running cdvdifflib manually, included tests for unified_diff interfaces  | 
1228  | 
, list(unified_diff_files('a2', 'b2')))  | 
1229  | 
||
| 
1711.2.9
by John Arbash Meinel
 Rename cdv => patience  | 
1230  | 
        # And the patience diff
 | 
| 
3922.1.2
by John Arbash Meinel
 Update the test cases for the new patience diff code.  | 
1231  | 
self.assertEquals(['--- a2\n',  | 
1232  | 
'+++ b2\n',  | 
|
| 
1185.81.29
by Aaron Bentley
 Fix style issues and duplicated tests  | 
1233  | 
'@@ -4,6 +4,11 @@\n',  | 
1234  | 
' d\n',  | 
|
1235  | 
' e\n',  | 
|
1236  | 
' f\n',  | 
|
1237  | 
'+x\n',  | 
|
1238  | 
'+y\n',  | 
|
1239  | 
'+d\n',  | 
|
1240  | 
'+e\n',  | 
|
1241  | 
'+f\n',  | 
|
1242  | 
' g\n',  | 
|
1243  | 
' h\n',  | 
|
1244  | 
' i\n',  | 
|
| 
1185.81.14
by John Arbash Meinel
 Added a main function for running cdvdifflib manually, included tests for unified_diff interfaces  | 
1245  | 
                          ]
 | 
| 
1185.81.25
by Aaron Bentley
 Clean up test_diff  | 
1246  | 
, 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  | 
1247  | 
sequencematcher=psm)))  | 
| 
2781.1.1
by Martin Pool
 merge cpatiencediff from Lukas  | 
1248  | 
|
1249  | 
||
1250  | 
class TestPatienceDiffLibFiles_c(TestPatienceDiffLibFiles):  | 
|
1251  | 
||
1252  | 
_test_needs_features = [CompiledPatienceDiffFeature]  | 
|
1253  | 
||
1254  | 
def setUp(self):  | 
|
1255  | 
super(TestPatienceDiffLibFiles_c, self).setUp()  | 
|
1256  | 
import bzrlib._patiencediff_c  | 
|
1257  | 
self._PatienceSequenceMatcher = \  | 
|
1258  | 
bzrlib._patiencediff_c.PatienceSequenceMatcher_c  | 
|
1259  | 
||
1260  | 
||
1261  | 
class TestUsingCompiledIfAvailable(TestCase):  | 
|
1262  | 
||
1263  | 
def test_PatienceSequenceMatcher(self):  | 
|
1264  | 
if CompiledPatienceDiffFeature.available():  | 
|
1265  | 
from bzrlib._patiencediff_c import PatienceSequenceMatcher_c  | 
|
1266  | 
self.assertIs(PatienceSequenceMatcher_c,  | 
|
1267  | 
bzrlib.patiencediff.PatienceSequenceMatcher)  | 
|
1268  | 
else:  | 
|
1269  | 
from bzrlib._patiencediff_py import PatienceSequenceMatcher_py  | 
|
1270  | 
self.assertIs(PatienceSequenceMatcher_py,  | 
|
1271  | 
bzrlib.patiencediff.PatienceSequenceMatcher)  | 
|
1272  | 
||
1273  | 
def test_unique_lcs(self):  | 
|
1274  | 
if CompiledPatienceDiffFeature.available():  | 
|
1275  | 
from bzrlib._patiencediff_c import unique_lcs_c  | 
|
1276  | 
self.assertIs(unique_lcs_c,  | 
|
1277  | 
bzrlib.patiencediff.unique_lcs)  | 
|
1278  | 
else:  | 
|
1279  | 
from bzrlib._patiencediff_py import unique_lcs_py  | 
|
1280  | 
self.assertIs(unique_lcs_py,  | 
|
1281  | 
bzrlib.patiencediff.unique_lcs)  | 
|
1282  | 
||
1283  | 
def test_recurse_matches(self):  | 
|
1284  | 
if CompiledPatienceDiffFeature.available():  | 
|
1285  | 
from bzrlib._patiencediff_c import recurse_matches_c  | 
|
1286  | 
self.assertIs(recurse_matches_c,  | 
|
1287  | 
bzrlib.patiencediff.recurse_matches)  | 
|
1288  | 
else:  | 
|
1289  | 
from bzrlib._patiencediff_py import recurse_matches_py  | 
|
1290  | 
self.assertIs(recurse_matches_py,  | 
|
1291  | 
bzrlib.patiencediff.recurse_matches)  | 
|
| 
3123.6.2
by Aaron Bentley
 Implement diff --using natively  | 
1292  | 
|
1293  | 
||
1294  | 
class TestDiffFromTool(TestCaseWithTransport):  | 
|
1295  | 
||
1296  | 
def test_from_string(self):  | 
|
1297  | 
diff_obj = DiffFromTool.from_string('diff', None, None, None)  | 
|
1298  | 
self.addCleanup(diff_obj.finish)  | 
|
1299  | 
self.assertEqual(['diff', '%(old_path)s', '%(new_path)s'],  | 
|
1300  | 
diff_obj.command_template)  | 
|
| 
3199.1.6
by Vincent Ladeuil
 Fiz last leaking tmp dir.  | 
1301  | 
|
1302  | 
def test_from_string_u5(self):  | 
|
| 
3123.6.2
by Aaron Bentley
 Implement diff --using natively  | 
1303  | 
diff_obj = DiffFromTool.from_string('diff -u\\ 5', None, None, None)  | 
| 
3199.1.6
by Vincent Ladeuil
 Fiz last leaking tmp dir.  | 
1304  | 
self.addCleanup(diff_obj.finish)  | 
| 
3123.6.2
by Aaron Bentley
 Implement diff --using natively  | 
1305  | 
self.assertEqual(['diff', '-u 5', '%(old_path)s', '%(new_path)s'],  | 
1306  | 
diff_obj.command_template)  | 
|
1307  | 
self.assertEqual(['diff', '-u 5', 'old-path', 'new-path'],  | 
|
1308  | 
diff_obj._get_command('old-path', 'new-path'))  | 
|
1309  | 
||
1310  | 
def test_execute(self):  | 
|
1311  | 
output = StringIO()  | 
|
1312  | 
diff_obj = DiffFromTool(['python', '-c',  | 
|
1313  | 
'print "%(old_path)s %(new_path)s"'],  | 
|
1314  | 
None, None, output)  | 
|
1315  | 
self.addCleanup(diff_obj.finish)  | 
|
1316  | 
diff_obj._execute('old', 'new')  | 
|
| 
3146.4.2
by Aaron Bentley
 Avoid assuming unix newline on output  | 
1317  | 
self.assertEqual(output.getvalue().rstrip(), 'old new')  | 
| 
3123.6.2
by Aaron Bentley
 Implement diff --using natively  | 
1318  | 
|
| 
3145.1.1
by Aaron Bentley
 Handle missing tools gracefully in diff --using  | 
1319  | 
def test_excute_missing(self):  | 
1320  | 
diff_obj = DiffFromTool(['a-tool-which-is-unlikely-to-exist'],  | 
|
1321  | 
None, None, None)  | 
|
1322  | 
self.addCleanup(diff_obj.finish)  | 
|
1323  | 
e = self.assertRaises(ExecutableMissing, diff_obj._execute, 'old',  | 
|
1324  | 
'new')  | 
|
1325  | 
self.assertEqual('a-tool-which-is-unlikely-to-exist could not be found'  | 
|
1326  | 
' on this machine', str(e))  | 
|
1327  | 
||
| 
3287.18.22
by Matt McClure
 Reverts to prior decomposition of exercise and verification, as suggested  | 
1328  | 
def test_prepare_files_creates_paths_readable_by_windows_tool(self):  | 
| 
3287.18.20
by Matt McClure
 Introduces a Feature subclass to encapsulate the availability of 'attrib'.  | 
1329  | 
self.requireFeature(AttribFeature)  | 
| 
3287.18.10
by Matt McClure
 Uses TestSkipped for test_execute_windows_tool on non-Windows platforms.  | 
1330  | 
output = StringIO()  | 
1331  | 
tree = self.make_branch_and_tree('tree')  | 
|
1332  | 
self.build_tree_contents([('tree/file', 'content')])  | 
|
1333  | 
tree.add('file', 'file-id')  | 
|
| 
3287.18.11
by Matt McClure
 Removed unnecessary timestamp parameter.  | 
1334  | 
tree.commit('old tree')  | 
| 
3287.18.10
by Matt McClure
 Uses TestSkipped for test_execute_windows_tool on non-Windows platforms.  | 
1335  | 
tree.lock_read()  | 
1336  | 
self.addCleanup(tree.unlock)  | 
|
1337  | 
diff_obj = DiffFromTool(['python', '-c',  | 
|
| 
3287.18.22
by Matt McClure
 Reverts to prior decomposition of exercise and verification, as suggested  | 
1338  | 
'print "%(old_path)s %(new_path)s"'],  | 
1339  | 
tree, tree, output)  | 
|
1340  | 
diff_obj._prepare_files('file-id', 'file', 'file')  | 
|
1341  | 
self.assertReadableByAttrib(diff_obj._root, 'old\\file', r'old\\file')  | 
|
1342  | 
self.assertReadableByAttrib(diff_obj._root, 'new\\file', r'new\\file')  | 
|
1343  | 
||
1344  | 
def assertReadableByAttrib(self, cwd, relpath, regex):  | 
|
1345  | 
proc = subprocess.Popen(['attrib', relpath],  | 
|
1346  | 
stdout=subprocess.PIPE,  | 
|
1347  | 
cwd=cwd)  | 
|
1348  | 
proc.wait()  | 
|
| 
3287.18.26
by Matt McClure
 Addresses concerns raised in  | 
1349  | 
result = proc.stdout.read()  | 
| 
3287.18.22
by Matt McClure
 Reverts to prior decomposition of exercise and verification, as suggested  | 
1350  | 
self.assertContainsRe(result, regex)  | 
| 
3287.18.9
by Matt McClure
 Adds a test asserting that a Windows tool that understands forward slashes  | 
1351  | 
|
| 
3123.6.2
by Aaron Bentley
 Implement diff --using natively  | 
1352  | 
def test_prepare_files(self):  | 
1353  | 
output = StringIO()  | 
|
1354  | 
tree = self.make_branch_and_tree('tree')  | 
|
| 
3123.6.5
by Aaron Bentley
 Symlink to real files if possible  | 
1355  | 
self.build_tree_contents([('tree/oldname', 'oldcontent')])  | 
| 
3287.18.23
by Matt McClure
 Adds comments that document my understanding of  | 
1356  | 
self.build_tree_contents([('tree/oldname2', 'oldcontent2')])  | 
| 
3123.6.5
by Aaron Bentley
 Symlink to real files if possible  | 
1357  | 
tree.add('oldname', 'file-id')  | 
| 
3287.18.23
by Matt McClure
 Adds comments that document my understanding of  | 
1358  | 
tree.add('oldname2', 'file2-id')  | 
| 
3123.6.4
by Aaron Bentley
 Set mtime (and atime) on files for --using  | 
1359  | 
tree.commit('old tree', timestamp=0)  | 
| 
3123.6.5
by Aaron Bentley
 Symlink to real files if possible  | 
1360  | 
tree.rename_one('oldname', 'newname')  | 
| 
3287.18.23
by Matt McClure
 Adds comments that document my understanding of  | 
1361  | 
tree.rename_one('oldname2', 'newname2')  | 
| 
3123.6.5
by Aaron Bentley
 Symlink to real files if possible  | 
1362  | 
self.build_tree_contents([('tree/newname', 'newcontent')])  | 
| 
3287.19.1
by Matt McClure
 Fixes https://bugs.launchpad.net/bzr/+bug/212289. I submitted this patch.  | 
1363  | 
self.build_tree_contents([('tree/newname2', 'newcontent2')])  | 
| 
3123.6.2
by Aaron Bentley
 Implement diff --using natively  | 
1364  | 
old_tree = tree.basis_tree()  | 
1365  | 
old_tree.lock_read()  | 
|
1366  | 
self.addCleanup(old_tree.unlock)  | 
|
| 
3123.6.4
by Aaron Bentley
 Set mtime (and atime) on files for --using  | 
1367  | 
tree.lock_read()  | 
1368  | 
self.addCleanup(tree.unlock)  | 
|
| 
3123.6.2
by Aaron Bentley
 Implement diff --using natively  | 
1369  | 
diff_obj = DiffFromTool(['python', '-c',  | 
1370  | 
'print "%(old_path)s %(new_path)s"'],  | 
|
1371  | 
old_tree, tree, output)  | 
|
1372  | 
self.addCleanup(diff_obj.finish)  | 
|
1373  | 
self.assertContainsRe(diff_obj._root, 'bzr-diff-[^/]*')  | 
|
1374  | 
old_path, new_path = diff_obj._prepare_files('file-id', 'oldname',  | 
|
1375  | 
'newname')  | 
|
1376  | 
self.assertContainsRe(old_path, 'old/oldname$')  | 
|
| 
3123.6.4
by Aaron Bentley
 Set mtime (and atime) on files for --using  | 
1377  | 
self.assertEqual(0, os.stat(old_path).st_mtime)  | 
| 
3123.6.2
by Aaron Bentley
 Implement diff --using natively  | 
1378  | 
self.assertContainsRe(new_path, 'new/newname$')  | 
1379  | 
self.assertFileEqual('oldcontent', old_path)  | 
|
1380  | 
self.assertFileEqual('newcontent', new_path)  | 
|
| 
3287.18.14
by Matt McClure
 Extracted a host_os_dereferences_symlinks method.  | 
1381  | 
if osutils.host_os_dereferences_symlinks():  | 
| 
3123.6.5
by Aaron Bentley
 Symlink to real files if possible  | 
1382  | 
self.assertTrue(os.path.samefile('tree/newname', new_path))  | 
| 
3123.6.2
by Aaron Bentley
 Implement diff --using natively  | 
1383  | 
        # make sure we can create files with the same parent directories
 | 
| 
3287.18.25
by Matt McClure
 Uses the correct file_id as the argument to _prepare_files.  | 
1384  | 
diff_obj._prepare_files('file2-id', 'oldname2', 'newname2')  |