bzr branch
http://gegoxaren.bato24.eu/bzr/brz/remove-bazaar
|
6619.2.1
by Vincent Ladeuil
Fix test failure for recent versions of diff |
1 |
# Copyright (C) 2005-2012, 2014, 2016, 2017 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
|
|
|
4183.7.1
by Sabin Iacob
update FSF mailing address |
15 |
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
|
|
1711.2.16
by John Arbash Meinel
test_diff needs a copyright statement |
16 |
|
|
1740.2.5
by Aaron Bentley
Merge from bzr.dev |
17 |
import os |
|
7031.1.1
by Jelmer Vernooij
Fix breezy.tests.test_diff. |
18 |
import re |
|
1920.1.1
by John Arbash Meinel
fix bug #56307, handle binary files even when LANG is not english |
19 |
import subprocess |
|
7141.1.1
by Jelmer Vernooij
Use sys.executable rather than python for ad-hoc tests. |
20 |
import sys |
|
5168.1.2
by Vincent Ladeuil
Ckeanup some more imports. |
21 |
import tempfile |
|
1558.15.2
by Aaron Bentley
Implemented binary file handling for diff |
22 |
|
|
6624
by Jelmer Vernooij
Merge Python3 porting work ('py3 pokes') |
23 |
from .. import ( |
|
5168.1.4
by Vincent Ladeuil
Final import fixes for bt.test_diff. |
24 |
diff, |
|
5168.1.2
by Vincent Ladeuil
Ckeanup some more imports. |
25 |
errors, |
26 |
osutils, |
|
|
5168.1.3
by Vincent Ladeuil
Even more import fixes. |
27 |
patiencediff, |
28 |
_patiencediff_py, |
|
|
5168.1.2
by Vincent Ladeuil
Ckeanup some more imports. |
29 |
revision as _mod_revision, |
30 |
revisionspec, |
|
31 |
revisiontree, |
|
32 |
tests, |
|
33 |
transform, |
|
34 |
)
|
|
|
6624
by Jelmer Vernooij
Merge Python3 porting work ('py3 pokes') |
35 |
from ..sixish import ( |
|
6621.22.2
by Martin
Use BytesIO or StringIO from bzrlib.sixish |
36 |
BytesIO, |
|
7031.1.1
by Jelmer Vernooij
Fix breezy.tests.test_diff. |
37 |
unichr, |
|
6621.22.2
by Martin
Use BytesIO or StringIO from bzrlib.sixish |
38 |
)
|
|
6624
by Jelmer Vernooij
Merge Python3 porting work ('py3 pokes') |
39 |
from ..tests import ( |
|
5967.12.1
by Martin Pool
Move all test features into bzrlib.tests.features |
40 |
features, |
|
6597.2.2
by Vincent Ladeuil
Split the diff tests to get finer grained failures. Also cleaned up some unused imports. |
41 |
EncodingAdapter, |
|
6621.22.2
by Martin
Use BytesIO or StringIO from bzrlib.sixish |
42 |
)
|
|
6624
by Jelmer Vernooij
Merge Python3 porting work ('py3 pokes') |
43 |
from ..tests.scenarios import load_tests_apply_scenarios |
|
6597.2.2
by Vincent Ladeuil
Split the diff tests to get finer grained failures. Also cleaned up some unused imports. |
44 |
|
45 |
||
46 |
load_tests = load_tests_apply_scenarios |
|
|
2781.1.1
by Martin Pool
merge cpatiencediff from Lukas |
47 |
|
48 |
||
|
7031.1.1
by Jelmer Vernooij
Fix breezy.tests.test_diff. |
49 |
def subst_dates(string): |
50 |
"""Replace date strings with constant values.""" |
|
51 |
return re.sub(br'\d{4}-\d{2}-\d{2} \d{2}:\d{2}:\d{2} [-\+]\d{4}', |
|
52 |
b'YYYY-MM-DD HH:MM:SS +ZZZZ', string) |
|
53 |
||
54 |
||
|
1558.15.11
by Aaron Bentley
Apply merge review suggestions |
55 |
def udiff_lines(old, new, allow_binary=False): |
|
6621.22.2
by Martin
Use BytesIO or StringIO from bzrlib.sixish |
56 |
output = BytesIO() |
|
5168.1.4
by Vincent Ladeuil
Final import fixes for bt.test_diff. |
57 |
diff.internal_diff('old', old, 'new', new, output, allow_binary) |
|
974.1.6
by Aaron Bentley
Added unit tests |
58 |
output.seek(0, 0) |
59 |
return output.readlines() |
|
60 |
||
|
1711.2.54
by John Arbash Meinel
Use mkstemp instead of NamedTemporary file for external diff. |
61 |
|
|
1711.2.57
by John Arbash Meinel
Allow external diff to write to a file without a fileno. |
62 |
def external_udiff_lines(old, new, use_stringio=False): |
63 |
if use_stringio: |
|
|
6621.22.2
by Martin
Use BytesIO or StringIO from bzrlib.sixish |
64 |
# BytesIO has no fileno, so it tests a different codepath
|
65 |
output = BytesIO() |
|
|
1711.2.57
by John Arbash Meinel
Allow external diff to write to a file without a fileno. |
66 |
else: |
|
5168.1.2
by Vincent Ladeuil
Ckeanup some more imports. |
67 |
output = tempfile.TemporaryFile() |
|
1692.8.7
by James Henstridge
changes suggested by John Meinel |
68 |
try: |
|
5168.1.4
by Vincent Ladeuil
Final import fixes for bt.test_diff. |
69 |
diff.external_diff('old', old, 'new', new, output, diff_opts=['-u']) |
|
5168.1.2
by Vincent Ladeuil
Ckeanup some more imports. |
70 |
except errors.NoDiff: |
71 |
raise tests.TestSkipped('external "diff" not present to test') |
|
|
1692.8.2
by James Henstridge
add a test for sending external diff output to a file |
72 |
output.seek(0, 0) |
73 |
lines = output.readlines() |
|
74 |
output.close() |
|
75 |
return lines |
|
76 |
||
77 |
||
|
6621.22.2
by Martin
Use BytesIO or StringIO from bzrlib.sixish |
78 |
class StubO(object): |
79 |
"""Simple file-like object that allows writes with any type and records.""" |
|
80 |
||
81 |
def __init__(self): |
|
82 |
self.write_record = [] |
|
83 |
||
84 |
def write(self, data): |
|
85 |
self.write_record.append(data) |
|
86 |
||
87 |
def check_types(self, testcase, expected_type): |
|
88 |
testcase.assertFalse( |
|
89 |
any(not isinstance(o, expected_type) for o in self.write_record), |
|
90 |
"Not all writes of type %s: %r" % ( |
|
91 |
expected_type.__name__, self.write_record)) |
|
92 |
||
93 |
||
|
6597.2.2
by Vincent Ladeuil
Split the diff tests to get finer grained failures. Also cleaned up some unused imports. |
94 |
class TestDiffOptions(tests.TestCase): |
95 |
||
96 |
def test_unified_added(self): |
|
97 |
"""Check for default style '-u' only if no other style specified |
|
98 |
in 'diff-options'.
|
|
99 |
"""
|
|
100 |
# Verify that style defaults to unified, id est '-u' appended
|
|
101 |
# to option list, in the absence of an alternative style.
|
|
102 |
self.assertEqual(['-a', '-u'], diff.default_style_unified(['-a'])) |
|
103 |
||
104 |
||
105 |
class TestDiffOptionsScenarios(tests.TestCase): |
|
106 |
||
107 |
scenarios = [(s, dict(style=s)) for s in diff.style_option_list] |
|
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
108 |
style = None # Set by load_tests_apply_scenarios from scenarios |
|
6597.2.2
by Vincent Ladeuil
Split the diff tests to get finer grained failures. Also cleaned up some unused imports. |
109 |
|
110 |
def test_unified_not_added(self): |
|
111 |
# Verify that for all valid style options, '-u' is not
|
|
112 |
# appended to option list.
|
|
113 |
ret_opts = diff.default_style_unified(diff_opts=["%s" % (self.style,)]) |
|
114 |
self.assertEqual(["%s" % (self.style,)], ret_opts) |
|
115 |
||
116 |
||
|
5168.1.2
by Vincent Ladeuil
Ckeanup some more imports. |
117 |
class TestDiff(tests.TestCase): |
|
1185.81.25
by Aaron Bentley
Clean up test_diff |
118 |
|
|
1102
by Martin Pool
- merge test refactoring from robertc |
119 |
def test_add_nl(self): |
120 |
"""diff generates a valid diff for patches that add a newline""" |
|
|
7031.1.1
by Jelmer Vernooij
Fix breezy.tests.test_diff. |
121 |
lines = udiff_lines([b'boo'], [b'boo\n']) |
|
1185.16.145
by Martin Pool
Remove all assert statements from test cases. |
122 |
self.check_patch(lines) |
|
7031.1.1
by Jelmer Vernooij
Fix breezy.tests.test_diff. |
123 |
self.assertEqual(lines[4], b'\\ No newline at end of file\n') |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
124 |
## "expected no-nl, got %r" % lines[4]
|
|
974.1.6
by Aaron Bentley
Added unit tests |
125 |
|
|
1102
by Martin Pool
- merge test refactoring from robertc |
126 |
def test_add_nl_2(self): |
127 |
"""diff generates a valid diff for patches that change last line and |
|
128 |
add a newline.
|
|
129 |
"""
|
|
|
7031.1.1
by Jelmer Vernooij
Fix breezy.tests.test_diff. |
130 |
lines = udiff_lines([b'boo'], [b'goo\n']) |
|
1185.16.145
by Martin Pool
Remove all assert statements from test cases. |
131 |
self.check_patch(lines) |
|
7031.1.1
by Jelmer Vernooij
Fix breezy.tests.test_diff. |
132 |
self.assertEqual(lines[4], b'\\ No newline at end of file\n') |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
133 |
## "expected no-nl, got %r" % lines[4]
|
|
974.1.6
by Aaron Bentley
Added unit tests |
134 |
|
|
1102
by Martin Pool
- merge test refactoring from robertc |
135 |
def test_remove_nl(self): |
136 |
"""diff generates a valid diff for patches that change last line and |
|
137 |
add a newline.
|
|
138 |
"""
|
|
|
7031.1.1
by Jelmer Vernooij
Fix breezy.tests.test_diff. |
139 |
lines = udiff_lines([b'boo\n'], [b'boo']) |
|
1185.16.145
by Martin Pool
Remove all assert statements from test cases. |
140 |
self.check_patch(lines) |
|
7031.1.1
by Jelmer Vernooij
Fix breezy.tests.test_diff. |
141 |
self.assertEqual(lines[5], b'\\ No newline at end of file\n') |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
142 |
## "expected no-nl, got %r" % lines[5]
|
|
1185.16.145
by Martin Pool
Remove all assert statements from test cases. |
143 |
|
144 |
def check_patch(self, lines): |
|
|
6614.1.1
by Vincent Ladeuil
Fix assert_ being deprecated by using assertTrue. |
145 |
self.assertTrue(len(lines) > 1) |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
146 |
## "Not enough lines for a file header for patch:\n%s" % "".join(lines)
|
147 |
self.assertTrue(lines[0].startswith(b'---')) |
|
148 |
## 'No orig line for patch:\n%s' % "".join(lines)
|
|
149 |
self.assertTrue(lines[1].startswith(b'+++')) |
|
150 |
## 'No mod line for patch:\n%s' % "".join(lines)
|
|
|
6614.1.1
by Vincent Ladeuil
Fix assert_ being deprecated by using assertTrue. |
151 |
self.assertTrue(len(lines) > 2) |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
152 |
## "No hunks for patch:\n%s" % "".join(lines)
|
|
7031.1.1
by Jelmer Vernooij
Fix breezy.tests.test_diff. |
153 |
self.assertTrue(lines[2].startswith(b'@@')) |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
154 |
## "No hunk header for patch:\n%s" % "".join(lines)
|
|
7031.1.1
by Jelmer Vernooij
Fix breezy.tests.test_diff. |
155 |
self.assertTrue(b'@@' in lines[2][2:]) |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
156 |
## "Unterminated hunk header for patch:\n%s" % "".join(lines)
|
|
1185.16.145
by Martin Pool
Remove all assert statements from test cases. |
157 |
|
|
1558.15.2
by Aaron Bentley
Implemented binary file handling for diff |
158 |
def test_binary_lines(self): |
|
5168.1.2
by Vincent Ladeuil
Ckeanup some more imports. |
159 |
empty = [] |
|
7031.1.1
by Jelmer Vernooij
Fix breezy.tests.test_diff. |
160 |
uni_lines = [1023 * b'a' + b'\x00'] |
|
6809.1.1
by Martin
Apply 2to3 ws_comma fixer |
161 |
self.assertRaises(errors.BinaryFile, udiff_lines, uni_lines, empty) |
|
5168.1.2
by Vincent Ladeuil
Ckeanup some more imports. |
162 |
self.assertRaises(errors.BinaryFile, udiff_lines, empty, uni_lines) |
|
6809.1.1
by Martin
Apply 2to3 ws_comma fixer |
163 |
udiff_lines(uni_lines, empty, allow_binary=True) |
|
5168.1.2
by Vincent Ladeuil
Ckeanup some more imports. |
164 |
udiff_lines(empty, uni_lines, allow_binary=True) |
|
1692.8.2
by James Henstridge
add a test for sending external diff output to a file |
165 |
|
166 |
def test_external_diff(self): |
|
|
7031.1.1
by Jelmer Vernooij
Fix breezy.tests.test_diff. |
167 |
lines = external_udiff_lines([b'boo\n'], [b'goo\n']) |
|
1692.8.2
by James Henstridge
add a test for sending external diff output to a file |
168 |
self.check_patch(lines) |
|
7031.1.1
by Jelmer Vernooij
Fix breezy.tests.test_diff. |
169 |
self.assertEqual(b'\n', lines[-1]) |
|
1711.2.57
by John Arbash Meinel
Allow external diff to write to a file without a fileno. |
170 |
|
171 |
def test_external_diff_no_fileno(self): |
|
172 |
# Make sure that we can handle not having a fileno, even
|
|
173 |
# if the diff is large
|
|
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
174 |
lines = external_udiff_lines([b'boo\n'] * 10000, |
175 |
[b'goo\n'] * 10000, |
|
|
1711.2.57
by John Arbash Meinel
Allow external diff to write to a file without a fileno. |
176 |
use_stringio=True) |
177 |
self.check_patch(lines) |
|
|
1899.1.1
by John Arbash Meinel
Fix the bug in the NoDiff exception class, and add a test |
178 |
|
|
1920.1.1
by John Arbash Meinel
fix bug #56307, handle binary files even when LANG is not english |
179 |
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) |
180 |
for lang in ('LANG', 'LC_ALL', 'LANGUAGE'): |
|
5570.3.9
by Vincent Ladeuil
More use cases for overrideEnv, _cleanEnvironment *may* contain too much variables now. |
181 |
self.overrideEnv(lang, 'C') |
|
7031.1.1
by Jelmer Vernooij
Fix breezy.tests.test_diff. |
182 |
lines = external_udiff_lines([b'\x00foobar\n'], [b'foo\x00bar\n']) |
|
5570.3.9
by Vincent Ladeuil
More use cases for overrideEnv, _cleanEnvironment *may* contain too much variables now. |
183 |
# Older versions of diffutils say "Binary files", newer
|
184 |
# versions just say "Files".
|
|
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
185 |
self.assertContainsRe( |
186 |
lines[0], b'(Binary f|F)iles old and new differ\n') |
|
|
7031.1.1
by Jelmer Vernooij
Fix breezy.tests.test_diff. |
187 |
self.assertEqual(lines[1:], [b'\n']) |
|
1899.1.4
by John Arbash Meinel
Just swallow a return code of 2 |
188 |
|
|
1899.1.1
by John Arbash Meinel
Fix the bug in the NoDiff exception class, and add a test |
189 |
def test_no_external_diff(self): |
190 |
"""Check that NoDiff is raised when diff is not available""" |
|
|
5570.3.9
by Vincent Ladeuil
More use cases for overrideEnv, _cleanEnvironment *may* contain too much variables now. |
191 |
# Make sure no 'diff' command is available
|
192 |
# XXX: Weird, using None instead of '' breaks the test -- vila 20101216
|
|
193 |
self.overrideEnv('PATH', '') |
|
194 |
self.assertRaises(errors.NoDiff, diff.external_diff, |
|
|
7031.1.1
by Jelmer Vernooij
Fix breezy.tests.test_diff. |
195 |
b'old', [b'boo\n'], b'new', [b'goo\n'], |
|
6621.22.2
by Martin
Use BytesIO or StringIO from bzrlib.sixish |
196 |
BytesIO(), diff_opts=['-u']) |
|
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
197 |
|
|
1711.2.30
by John Arbash Meinel
Fix bug in internal_diff handling of unicode paths |
198 |
def test_internal_diff_default(self): |
199 |
# Default internal diff encoding is utf8
|
|
|
6621.22.2
by Martin
Use BytesIO or StringIO from bzrlib.sixish |
200 |
output = BytesIO() |
|
7031.1.1
by Jelmer Vernooij
Fix breezy.tests.test_diff. |
201 |
diff.internal_diff(u'old_\xb5', [b'old_text\n'], |
202 |
u'new_\xe5', [b'new_text\n'], output) |
|
|
1711.2.30
by John Arbash Meinel
Fix bug in internal_diff handling of unicode paths |
203 |
lines = output.getvalue().splitlines(True) |
204 |
self.check_patch(lines) |
|
|
7031.1.1
by Jelmer Vernooij
Fix breezy.tests.test_diff. |
205 |
self.assertEqual([b'--- old_\xc2\xb5\n', |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
206 |
b'+++ new_\xc3\xa5\n', |
207 |
b'@@ -1,1 +1,1 @@\n', |
|
208 |
b'-old_text\n', |
|
209 |
b'+new_text\n', |
|
210 |
b'\n', |
|
211 |
], lines) |
|
|
1711.2.30
by John Arbash Meinel
Fix bug in internal_diff handling of unicode paths |
212 |
|
213 |
def test_internal_diff_utf8(self): |
|
|
6621.22.2
by Martin
Use BytesIO or StringIO from bzrlib.sixish |
214 |
output = BytesIO() |
|
7031.1.1
by Jelmer Vernooij
Fix breezy.tests.test_diff. |
215 |
diff.internal_diff(u'old_\xb5', [b'old_text\n'], |
216 |
u'new_\xe5', [b'new_text\n'], output, |
|
|
5168.1.4
by Vincent Ladeuil
Final import fixes for bt.test_diff. |
217 |
path_encoding='utf8') |
|
1711.2.30
by John Arbash Meinel
Fix bug in internal_diff handling of unicode paths |
218 |
lines = output.getvalue().splitlines(True) |
219 |
self.check_patch(lines) |
|
|
7031.1.1
by Jelmer Vernooij
Fix breezy.tests.test_diff. |
220 |
self.assertEqual([b'--- old_\xc2\xb5\n', |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
221 |
b'+++ new_\xc3\xa5\n', |
222 |
b'@@ -1,1 +1,1 @@\n', |
|
223 |
b'-old_text\n', |
|
224 |
b'+new_text\n', |
|
225 |
b'\n', |
|
226 |
], lines) |
|
|
1711.2.30
by John Arbash Meinel
Fix bug in internal_diff handling of unicode paths |
227 |
|
228 |
def test_internal_diff_iso_8859_1(self): |
|
|
6621.22.2
by Martin
Use BytesIO or StringIO from bzrlib.sixish |
229 |
output = BytesIO() |
|
6973.13.2
by Jelmer Vernooij
Fix some more tests. |
230 |
diff.internal_diff(u'old_\xb5', [b'old_text\n'], |
231 |
u'new_\xe5', [b'new_text\n'], output, |
|
|
5168.1.4
by Vincent Ladeuil
Final import fixes for bt.test_diff. |
232 |
path_encoding='iso-8859-1') |
|
1711.2.30
by John Arbash Meinel
Fix bug in internal_diff handling of unicode paths |
233 |
lines = output.getvalue().splitlines(True) |
234 |
self.check_patch(lines) |
|
|
6973.13.2
by Jelmer Vernooij
Fix some more tests. |
235 |
self.assertEqual([b'--- old_\xb5\n', |
236 |
b'+++ new_\xe5\n', |
|
237 |
b'@@ -1,1 +1,1 @@\n', |
|
238 |
b'-old_text\n', |
|
239 |
b'+new_text\n', |
|
240 |
b'\n', |
|
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
241 |
], lines) |
|
1711.2.30
by John Arbash Meinel
Fix bug in internal_diff handling of unicode paths |
242 |
|
|
3085.1.1
by John Arbash Meinel
Fix internal_diff to not fail when the texts are identical. |
243 |
def test_internal_diff_no_content(self): |
|
6621.22.2
by Martin
Use BytesIO or StringIO from bzrlib.sixish |
244 |
output = BytesIO() |
|
5168.1.4
by Vincent Ladeuil
Final import fixes for bt.test_diff. |
245 |
diff.internal_diff(u'old', [], u'new', [], output) |
|
6973.13.2
by Jelmer Vernooij
Fix some more tests. |
246 |
self.assertEqual(b'', output.getvalue()) |
|
3085.1.1
by John Arbash Meinel
Fix internal_diff to not fail when the texts are identical. |
247 |
|
248 |
def test_internal_diff_no_changes(self): |
|
|
6621.22.2
by Martin
Use BytesIO or StringIO from bzrlib.sixish |
249 |
output = BytesIO() |
|
6973.13.2
by Jelmer Vernooij
Fix some more tests. |
250 |
diff.internal_diff(u'old', [b'text\n', b'contents\n'], |
251 |
u'new', [b'text\n', b'contents\n'], |
|
|
5168.1.4
by Vincent Ladeuil
Final import fixes for bt.test_diff. |
252 |
output) |
|
6973.13.2
by Jelmer Vernooij
Fix some more tests. |
253 |
self.assertEqual(b'', output.getvalue()) |
|
3085.1.1
by John Arbash Meinel
Fix internal_diff to not fail when the texts are identical. |
254 |
|
|
1711.2.30
by John Arbash Meinel
Fix bug in internal_diff handling of unicode paths |
255 |
def test_internal_diff_returns_bytes(self): |
|
6621.22.2
by Martin
Use BytesIO or StringIO from bzrlib.sixish |
256 |
output = StubO() |
|
6973.13.2
by Jelmer Vernooij
Fix some more tests. |
257 |
diff.internal_diff(u'old_\xb5', [b'old_text\n'], |
|
7031.1.1
by Jelmer Vernooij
Fix breezy.tests.test_diff. |
258 |
u'new_\xe5', [b'new_text\n'], output) |
|
6621.22.2
by Martin
Use BytesIO or StringIO from bzrlib.sixish |
259 |
output.check_types(self, bytes) |
|
1711.2.30
by John Arbash Meinel
Fix bug in internal_diff handling of unicode paths |
260 |
|
|
6524.5.5
by Paul Nixon
Added tests of configurable context |
261 |
def test_internal_diff_default_context(self): |
|
6621.22.2
by Martin
Use BytesIO or StringIO from bzrlib.sixish |
262 |
output = BytesIO() |
|
7031.1.1
by Jelmer Vernooij
Fix breezy.tests.test_diff. |
263 |
diff.internal_diff('old', [b'same_text\n', b'same_text\n', b'same_text\n', |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
264 |
b'same_text\n', b'same_text\n', b'old_text\n'], |
|
7031.1.1
by Jelmer Vernooij
Fix breezy.tests.test_diff. |
265 |
'new', [b'same_text\n', b'same_text\n', b'same_text\n', |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
266 |
b'same_text\n', b'same_text\n', b'new_text\n'], output) |
|
6524.5.5
by Paul Nixon
Added tests of configurable context |
267 |
lines = output.getvalue().splitlines(True) |
268 |
self.check_patch(lines) |
|
|
7031.1.1
by Jelmer Vernooij
Fix breezy.tests.test_diff. |
269 |
self.assertEqual([b'--- old\n', |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
270 |
b'+++ new\n', |
271 |
b'@@ -3,4 +3,4 @@\n', |
|
272 |
b' same_text\n', |
|
273 |
b' same_text\n', |
|
274 |
b' same_text\n', |
|
275 |
b'-old_text\n', |
|
276 |
b'+new_text\n', |
|
277 |
b'\n', |
|
278 |
], lines) |
|
|
6524.5.5
by Paul Nixon
Added tests of configurable context |
279 |
|
280 |
def test_internal_diff_no_context(self): |
|
|
6621.22.2
by Martin
Use BytesIO or StringIO from bzrlib.sixish |
281 |
output = BytesIO() |
|
7031.1.1
by Jelmer Vernooij
Fix breezy.tests.test_diff. |
282 |
diff.internal_diff('old', [b'same_text\n', b'same_text\n', b'same_text\n', |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
283 |
b'same_text\n', b'same_text\n', b'old_text\n'], |
|
7031.1.1
by Jelmer Vernooij
Fix breezy.tests.test_diff. |
284 |
'new', [b'same_text\n', b'same_text\n', b'same_text\n', |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
285 |
b'same_text\n', b'same_text\n', b'new_text\n'], output, |
|
6524.5.5
by Paul Nixon
Added tests of configurable context |
286 |
context_lines=0) |
287 |
lines = output.getvalue().splitlines(True) |
|
288 |
self.check_patch(lines) |
|
|
7031.1.1
by Jelmer Vernooij
Fix breezy.tests.test_diff. |
289 |
self.assertEqual([b'--- old\n', |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
290 |
b'+++ new\n', |
291 |
b'@@ -6,1 +6,1 @@\n', |
|
292 |
b'-old_text\n', |
|
293 |
b'+new_text\n', |
|
294 |
b'\n', |
|
295 |
], lines) |
|
|
6524.5.5
by Paul Nixon
Added tests of configurable context |
296 |
|
297 |
def test_internal_diff_more_context(self): |
|
|
6621.22.2
by Martin
Use BytesIO or StringIO from bzrlib.sixish |
298 |
output = BytesIO() |
|
7031.1.1
by Jelmer Vernooij
Fix breezy.tests.test_diff. |
299 |
diff.internal_diff('old', [b'same_text\n', b'same_text\n', b'same_text\n', |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
300 |
b'same_text\n', b'same_text\n', b'old_text\n'], |
|
7031.1.1
by Jelmer Vernooij
Fix breezy.tests.test_diff. |
301 |
'new', [b'same_text\n', b'same_text\n', b'same_text\n', |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
302 |
b'same_text\n', b'same_text\n', b'new_text\n'], output, |
|
6524.5.5
by Paul Nixon
Added tests of configurable context |
303 |
context_lines=4) |
304 |
lines = output.getvalue().splitlines(True) |
|
305 |
self.check_patch(lines) |
|
|
7031.1.1
by Jelmer Vernooij
Fix breezy.tests.test_diff. |
306 |
self.assertEqual([b'--- old\n', |
307 |
b'+++ new\n', |
|
308 |
b'@@ -2,5 +2,5 @@\n', |
|
309 |
b' same_text\n', |
|
310 |
b' same_text\n', |
|
311 |
b' same_text\n', |
|
312 |
b' same_text\n', |
|
313 |
b'-old_text\n', |
|
314 |
b'+new_text\n', |
|
315 |
b'\n', |
|
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
316 |
], lines) |
|
6524.5.5
by Paul Nixon
Added tests of configurable context |
317 |
|
318 |
||
|
5168.1.2
by Vincent Ladeuil
Ckeanup some more imports. |
319 |
class TestDiffFiles(tests.TestCaseInTempDir): |
|
1920.1.1
by John Arbash Meinel
fix bug #56307, handle binary files even when LANG is not english |
320 |
|
321 |
def test_external_diff_binary(self): |
|
322 |
"""The output when using external diff should use diff's i18n error""" |
|
|
6792.1.2
by Jelmer Vernooij
Alternative approach. |
323 |
for lang in ('LANG', 'LC_ALL', 'LANGUAGE'): |
324 |
self.overrideEnv(lang, 'C') |
|
|
1920.1.1
by John Arbash Meinel
fix bug #56307, handle binary files even when LANG is not english |
325 |
# Make sure external_diff doesn't fail in the current LANG
|
|
7031.1.1
by Jelmer Vernooij
Fix breezy.tests.test_diff. |
326 |
lines = external_udiff_lines([b'\x00foobar\n'], [b'foo\x00bar\n']) |
|
1920.1.1
by John Arbash Meinel
fix bug #56307, handle binary files even when LANG is not english |
327 |
|
|
2240.1.1
by Alexander Belchenko
test_external_diff_binary: run external diff with --binary flag |
328 |
cmd = ['diff', '-u', '--binary', 'old', 'new'] |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
329 |
with open('old', 'wb') as f: |
330 |
f.write(b'\x00foobar\n') |
|
331 |
with open('new', 'wb') as f: |
|
332 |
f.write(b'foo\x00bar\n') |
|
|
6792.1.3
by Jelmer Vernooij
Alternative approach. |
333 |
pipe = subprocess.Popen(cmd, stdout=subprocess.PIPE, |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
334 |
stdin=subprocess.PIPE) |
|
1920.1.1
by John Arbash Meinel
fix bug #56307, handle binary files even when LANG is not english |
335 |
out, err = pipe.communicate() |
336 |
# We should output whatever diff tells us, plus a trailing newline
|
|
|
7031.1.1
by Jelmer Vernooij
Fix breezy.tests.test_diff. |
337 |
self.assertEqual(out.splitlines(True) + [b'\n'], lines) |
|
1920.1.1
by John Arbash Meinel
fix bug #56307, handle binary files even when LANG is not english |
338 |
|
339 |
||
|
5784.3.1
by Martin Pool
Remove unnecessary TestShowDiffTreesHelper and just use a function |
340 |
def get_diff_as_string(tree1, tree2, specific_files=None, working_tree=None): |
|
6621.22.2
by Martin
Use BytesIO or StringIO from bzrlib.sixish |
341 |
output = BytesIO() |
|
5784.3.1
by Martin Pool
Remove unnecessary TestShowDiffTreesHelper and just use a function |
342 |
if working_tree is not None: |
343 |
extra_trees = (working_tree,) |
|
344 |
else: |
|
345 |
extra_trees = () |
|
346 |
diff.show_diff_trees(tree1, tree2, output, |
|
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
347 |
specific_files=specific_files, |
348 |
extra_trees=extra_trees, old_label='old/', |
|
349 |
new_label='new/') |
|
|
5784.3.1
by Martin Pool
Remove unnecessary TestShowDiffTreesHelper and just use a function |
350 |
return output.getvalue() |
351 |
||
352 |
||
353 |
class TestDiffDates(tests.TestCaseWithTransport): |
|
|
1740.2.5
by Aaron Bentley
Merge from bzr.dev |
354 |
|
355 |
def setUp(self): |
|
356 |
super(TestDiffDates, self).setUp() |
|
357 |
self.wt = self.make_branch_and_tree('.') |
|
358 |
self.b = self.wt.branch |
|
359 |
self.build_tree_contents([ |
|
|
6855.4.1
by Jelmer Vernooij
Yet more bees. |
360 |
('file1', b'file1 contents at rev 1\n'), |
361 |
('file2', b'file2 contents at rev 1\n') |
|
|
1740.2.5
by Aaron Bentley
Merge from bzr.dev |
362 |
])
|
363 |
self.wt.add(['file1', 'file2']) |
|
364 |
self.wt.commit( |
|
365 |
message='Revision 1', |
|
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
366 |
timestamp=1143849600, # 2006-04-01 00:00:00 UTC |
|
1740.2.5
by Aaron Bentley
Merge from bzr.dev |
367 |
timezone=0, |
|
6855.4.1
by Jelmer Vernooij
Yet more bees. |
368 |
rev_id=b'rev-1') |
369 |
self.build_tree_contents([('file1', b'file1 contents at rev 2\n')]) |
|
|
1740.2.5
by Aaron Bentley
Merge from bzr.dev |
370 |
self.wt.commit( |
371 |
message='Revision 2', |
|
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
372 |
timestamp=1143936000, # 2006-04-02 00:00:00 UTC |
|
1740.2.5
by Aaron Bentley
Merge from bzr.dev |
373 |
timezone=28800, |
|
6855.4.1
by Jelmer Vernooij
Yet more bees. |
374 |
rev_id=b'rev-2') |
375 |
self.build_tree_contents([('file2', b'file2 contents at rev 3\n')]) |
|
|
1740.2.5
by Aaron Bentley
Merge from bzr.dev |
376 |
self.wt.commit( |
377 |
message='Revision 3', |
|
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
378 |
timestamp=1144022400, # 2006-04-03 00:00:00 UTC |
|
1740.2.5
by Aaron Bentley
Merge from bzr.dev |
379 |
timezone=-3600, |
|
6855.4.1
by Jelmer Vernooij
Yet more bees. |
380 |
rev_id=b'rev-3') |
|
1740.2.5
by Aaron Bentley
Merge from bzr.dev |
381 |
self.wt.remove(['file2']) |
382 |
self.wt.commit( |
|
383 |
message='Revision 4', |
|
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
384 |
timestamp=1144108800, # 2006-04-04 00:00:00 UTC |
|
1740.2.5
by Aaron Bentley
Merge from bzr.dev |
385 |
timezone=0, |
|
6855.4.1
by Jelmer Vernooij
Yet more bees. |
386 |
rev_id=b'rev-4') |
|
1740.2.5
by Aaron Bentley
Merge from bzr.dev |
387 |
self.build_tree_contents([ |
|
6855.4.1
by Jelmer Vernooij
Yet more bees. |
388 |
('file1', b'file1 contents in working tree\n') |
|
1740.2.5
by Aaron Bentley
Merge from bzr.dev |
389 |
])
|
390 |
# set the date stamps for files in the working tree to known values
|
|
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
391 |
os.utime('file1', (1144195200, 1144195200)) # 2006-04-05 00:00:00 UTC |
|
1740.2.5
by Aaron Bentley
Merge from bzr.dev |
392 |
|
393 |
def test_diff_rev_tree_working_tree(self): |
|
|
5784.3.1
by Martin Pool
Remove unnecessary TestShowDiffTreesHelper and just use a function |
394 |
output = get_diff_as_string(self.wt.basis_tree(), self.wt) |
|
1740.2.5
by Aaron Bentley
Merge from bzr.dev |
395 |
# note that the date for old/file1 is from rev 2 rather than from
|
396 |
# the basis revision (rev 4)
|
|
|
7031.1.1
by Jelmer Vernooij
Fix breezy.tests.test_diff. |
397 |
self.assertEqualDiff(output, b'''\ |
|
1740.2.5
by Aaron Bentley
Merge from bzr.dev |
398 |
=== modified file 'file1'
|
399 |
--- old/file1\t2006-04-02 00:00:00 +0000 |
|
400 |
+++ new/file1\t2006-04-05 00:00:00 +0000 |
|
401 |
@@ -1,1 +1,1 @@
|
|
402 |
-file1 contents at rev 2
|
|
403 |
+file1 contents in working tree
|
|
404 |
||
405 |
''') |
|
406 |
||
407 |
def test_diff_rev_tree_rev_tree(self): |
|
|
6973.5.2
by Jelmer Vernooij
Add more bees. |
408 |
tree1 = self.b.repository.revision_tree(b'rev-2') |
409 |
tree2 = self.b.repository.revision_tree(b'rev-3') |
|
|
5784.3.1
by Martin Pool
Remove unnecessary TestShowDiffTreesHelper and just use a function |
410 |
output = get_diff_as_string(tree1, tree2) |
|
7031.1.1
by Jelmer Vernooij
Fix breezy.tests.test_diff. |
411 |
self.assertEqualDiff(output, b'''\ |
|
1740.2.5
by Aaron Bentley
Merge from bzr.dev |
412 |
=== modified file 'file2'
|
413 |
--- old/file2\t2006-04-01 00:00:00 +0000 |
|
414 |
+++ new/file2\t2006-04-03 00:00:00 +0000 |
|
415 |
@@ -1,1 +1,1 @@
|
|
416 |
-file2 contents at rev 1
|
|
417 |
+file2 contents at rev 3
|
|
418 |
||
419 |
''') |
|
|
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
420 |
|
|
1740.2.5
by Aaron Bentley
Merge from bzr.dev |
421 |
def test_diff_add_files(self): |
|
3668.5.1
by Jelmer Vernooij
Use NULL_REVISION rather than None for Repository.revision_tree(). |
422 |
tree1 = self.b.repository.revision_tree(_mod_revision.NULL_REVISION) |
|
6973.5.2
by Jelmer Vernooij
Add more bees. |
423 |
tree2 = self.b.repository.revision_tree(b'rev-1') |
|
5784.3.1
by Martin Pool
Remove unnecessary TestShowDiffTreesHelper and just use a function |
424 |
output = get_diff_as_string(tree1, tree2) |
|
1740.2.5
by Aaron Bentley
Merge from bzr.dev |
425 |
# the files have the epoch time stamp for the tree in which
|
426 |
# they don't exist.
|
|
|
7031.1.1
by Jelmer Vernooij
Fix breezy.tests.test_diff. |
427 |
self.assertEqualDiff(output, b'''\ |
|
1740.2.5
by Aaron Bentley
Merge from bzr.dev |
428 |
=== added file 'file1'
|
429 |
--- old/file1\t1970-01-01 00:00:00 +0000 |
|
430 |
+++ new/file1\t2006-04-01 00:00:00 +0000 |
|
431 |
@@ -0,0 +1,1 @@
|
|
432 |
+file1 contents at rev 1
|
|
433 |
||
434 |
=== added file 'file2'
|
|
435 |
--- old/file2\t1970-01-01 00:00:00 +0000 |
|
436 |
+++ new/file2\t2006-04-01 00:00:00 +0000 |
|
437 |
@@ -0,0 +1,1 @@
|
|
438 |
+file2 contents at rev 1
|
|
439 |
||
440 |
''') |
|
441 |
||
442 |
def test_diff_remove_files(self): |
|
|
6973.5.2
by Jelmer Vernooij
Add more bees. |
443 |
tree1 = self.b.repository.revision_tree(b'rev-3') |
444 |
tree2 = self.b.repository.revision_tree(b'rev-4') |
|
|
5784.3.1
by Martin Pool
Remove unnecessary TestShowDiffTreesHelper and just use a function |
445 |
output = get_diff_as_string(tree1, tree2) |
|
1740.2.5
by Aaron Bentley
Merge from bzr.dev |
446 |
# the file has the epoch time stamp for the tree in which
|
447 |
# it doesn't exist.
|
|
|
7031.1.1
by Jelmer Vernooij
Fix breezy.tests.test_diff. |
448 |
self.assertEqualDiff(output, b'''\ |
|
1740.2.5
by Aaron Bentley
Merge from bzr.dev |
449 |
=== removed file 'file2'
|
450 |
--- old/file2\t2006-04-03 00:00:00 +0000 |
|
451 |
+++ new/file2\t1970-01-01 00:00:00 +0000 |
|
452 |
@@ -1,1 +0,0 @@
|
|
453 |
-file2 contents at rev 3
|
|
454 |
||
455 |
''') |
|
456 |
||
|
1551.7.17
by Aaron Bentley
Switch to PathsNotVersioned, accept extra_trees |
457 |
def test_show_diff_specified(self): |
|
1551.7.22
by Aaron Bentley
Changes from review |
458 |
"""A working tree filename can be used to identify a file""" |
|
1551.7.17
by Aaron Bentley
Switch to PathsNotVersioned, accept extra_trees |
459 |
self.wt.rename_one('file1', 'file1b') |
|
6973.5.2
by Jelmer Vernooij
Add more bees. |
460 |
old_tree = self.b.repository.revision_tree(b'rev-1') |
461 |
new_tree = self.b.repository.revision_tree(b'rev-4') |
|
|
5784.3.1
by Martin Pool
Remove unnecessary TestShowDiffTreesHelper and just use a function |
462 |
out = get_diff_as_string(old_tree, new_tree, specific_files=['file1b'], |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
463 |
working_tree=self.wt) |
|
7031.1.1
by Jelmer Vernooij
Fix breezy.tests.test_diff. |
464 |
self.assertContainsRe(out, b'file1\t') |
|
1551.7.17
by Aaron Bentley
Switch to PathsNotVersioned, accept extra_trees |
465 |
|
|
1551.7.22
by Aaron Bentley
Changes from review |
466 |
def test_recursive_diff(self): |
467 |
"""Children of directories are matched""" |
|
468 |
os.mkdir('dir1') |
|
469 |
os.mkdir('dir2') |
|
470 |
self.wt.add(['dir1', 'dir2']) |
|
471 |
self.wt.rename_one('file1', 'dir1/file1') |
|
|
6973.5.2
by Jelmer Vernooij
Add more bees. |
472 |
old_tree = self.b.repository.revision_tree(b'rev-1') |
473 |
new_tree = self.b.repository.revision_tree(b'rev-4') |
|
|
5784.3.1
by Martin Pool
Remove unnecessary TestShowDiffTreesHelper and just use a function |
474 |
out = get_diff_as_string(old_tree, new_tree, specific_files=['dir1'], |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
475 |
working_tree=self.wt) |
|
6973.14.7
by Jelmer Vernooij
Bees bees bees. |
476 |
self.assertContainsRe(out, b'file1\t') |
|
5784.3.1
by Martin Pool
Remove unnecessary TestShowDiffTreesHelper and just use a function |
477 |
out = get_diff_as_string(old_tree, new_tree, specific_files=['dir2'], |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
478 |
working_tree=self.wt) |
|
6973.14.7
by Jelmer Vernooij
Bees bees bees. |
479 |
self.assertNotContainsRe(out, b'file1\t') |
|
1740.2.5
by Aaron Bentley
Merge from bzr.dev |
480 |
|
|
1899.1.1
by John Arbash Meinel
Fix the bug in the NoDiff exception class, and add a test |
481 |
|
|
5784.3.1
by Martin Pool
Remove unnecessary TestShowDiffTreesHelper and just use a function |
482 |
class TestShowDiffTrees(tests.TestCaseWithTransport): |
|
2405.1.1
by John Arbash Meinel
Add a bunch of direct tests for 'show_diff_trees' |
483 |
"""Direct tests for show_diff_trees""" |
484 |
||
485 |
def test_modified_file(self): |
|
486 |
"""Test when a file is modified.""" |
|
487 |
tree = self.make_branch_and_tree('tree') |
|
|
6855.4.1
by Jelmer Vernooij
Yet more bees. |
488 |
self.build_tree_contents([('tree/file', b'contents\n')]) |
489 |
tree.add(['file'], [b'file-id']) |
|
490 |
tree.commit('one', rev_id=b'rev-1') |
|
|
2405.1.1
by John Arbash Meinel
Add a bunch of direct tests for 'show_diff_trees' |
491 |
|
|
6855.4.1
by Jelmer Vernooij
Yet more bees. |
492 |
self.build_tree_contents([('tree/file', b'new contents\n')]) |
|
5784.3.1
by Martin Pool
Remove unnecessary TestShowDiffTreesHelper and just use a function |
493 |
d = get_diff_as_string(tree.basis_tree(), tree) |
|
7031.1.1
by Jelmer Vernooij
Fix breezy.tests.test_diff. |
494 |
self.assertContainsRe(d, b"=== modified file 'file'\n") |
495 |
self.assertContainsRe(d, b'--- old/file\t') |
|
496 |
self.assertContainsRe(d, b'\\+\\+\\+ new/file\t') |
|
497 |
self.assertContainsRe(d, b'-contents\n' |
|
498 |
b'\\+new contents\n') |
|
|
2405.1.1
by John Arbash Meinel
Add a bunch of direct tests for 'show_diff_trees' |
499 |
|
|
2405.1.2
by John Arbash Meinel
Fix bug #103870 by passing None instead of a (sometimes wrong) path |
500 |
def test_modified_file_in_renamed_dir(self): |
501 |
"""Test when a file is modified in a renamed directory.""" |
|
502 |
tree = self.make_branch_and_tree('tree') |
|
503 |
self.build_tree(['tree/dir/']) |
|
|
6855.4.1
by Jelmer Vernooij
Yet more bees. |
504 |
self.build_tree_contents([('tree/dir/file', b'contents\n')]) |
505 |
tree.add(['dir', 'dir/file'], [b'dir-id', b'file-id']) |
|
506 |
tree.commit('one', rev_id=b'rev-1') |
|
|
2405.1.2
by John Arbash Meinel
Fix bug #103870 by passing None instead of a (sometimes wrong) path |
507 |
|
508 |
tree.rename_one('dir', 'other') |
|
|
6855.4.1
by Jelmer Vernooij
Yet more bees. |
509 |
self.build_tree_contents([('tree/other/file', b'new contents\n')]) |
|
5784.3.1
by Martin Pool
Remove unnecessary TestShowDiffTreesHelper and just use a function |
510 |
d = get_diff_as_string(tree.basis_tree(), tree) |
|
7031.1.1
by Jelmer Vernooij
Fix breezy.tests.test_diff. |
511 |
self.assertContainsRe(d, b"=== renamed directory 'dir' => 'other'\n") |
512 |
self.assertContainsRe(d, b"=== modified file 'other/file'\n") |
|
|
2405.1.2
by John Arbash Meinel
Fix bug #103870 by passing None instead of a (sometimes wrong) path |
513 |
# XXX: This is technically incorrect, because it used to be at another
|
514 |
# location. What to do?
|
|
|
7031.1.1
by Jelmer Vernooij
Fix breezy.tests.test_diff. |
515 |
self.assertContainsRe(d, b'--- old/dir/file\t') |
516 |
self.assertContainsRe(d, b'\\+\\+\\+ new/other/file\t') |
|
517 |
self.assertContainsRe(d, b'-contents\n' |
|
518 |
b'\\+new contents\n') |
|
|
2405.1.2
by John Arbash Meinel
Fix bug #103870 by passing None instead of a (sometimes wrong) path |
519 |
|
|
2405.1.1
by John Arbash Meinel
Add a bunch of direct tests for 'show_diff_trees' |
520 |
def test_renamed_directory(self): |
521 |
"""Test when only a directory is only renamed.""" |
|
522 |
tree = self.make_branch_and_tree('tree') |
|
523 |
self.build_tree(['tree/dir/']) |
|
|
6855.4.1
by Jelmer Vernooij
Yet more bees. |
524 |
self.build_tree_contents([('tree/dir/file', b'contents\n')]) |
525 |
tree.add(['dir', 'dir/file'], [b'dir-id', b'file-id']) |
|
526 |
tree.commit('one', rev_id=b'rev-1') |
|
|
2405.1.1
by John Arbash Meinel
Add a bunch of direct tests for 'show_diff_trees' |
527 |
|
528 |
tree.rename_one('dir', 'newdir') |
|
|
5784.3.1
by Martin Pool
Remove unnecessary TestShowDiffTreesHelper and just use a function |
529 |
d = get_diff_as_string(tree.basis_tree(), tree) |
|
2405.1.1
by John Arbash Meinel
Add a bunch of direct tests for 'show_diff_trees' |
530 |
# Renaming a directory should be a single "you renamed this dir" even
|
531 |
# when there are files inside.
|
|
|
7031.1.1
by Jelmer Vernooij
Fix breezy.tests.test_diff. |
532 |
self.assertEqual(d, b"=== renamed directory 'dir' => 'newdir'\n") |
|
2405.1.1
by John Arbash Meinel
Add a bunch of direct tests for 'show_diff_trees' |
533 |
|
534 |
def test_renamed_file(self): |
|
535 |
"""Test when a file is only renamed.""" |
|
536 |
tree = self.make_branch_and_tree('tree') |
|
|
6855.4.1
by Jelmer Vernooij
Yet more bees. |
537 |
self.build_tree_contents([('tree/file', b'contents\n')]) |
538 |
tree.add(['file'], [b'file-id']) |
|
539 |
tree.commit('one', rev_id=b'rev-1') |
|
|
2405.1.1
by John Arbash Meinel
Add a bunch of direct tests for 'show_diff_trees' |
540 |
|
541 |
tree.rename_one('file', 'newname') |
|
|
5784.3.1
by Martin Pool
Remove unnecessary TestShowDiffTreesHelper and just use a function |
542 |
d = get_diff_as_string(tree.basis_tree(), tree) |
|
7031.1.1
by Jelmer Vernooij
Fix breezy.tests.test_diff. |
543 |
self.assertContainsRe(d, b"=== renamed file 'file' => 'newname'\n") |
|
2405.1.1
by John Arbash Meinel
Add a bunch of direct tests for 'show_diff_trees' |
544 |
# We shouldn't have a --- or +++ line, because there is no content
|
545 |
# change
|
|
|
7031.1.1
by Jelmer Vernooij
Fix breezy.tests.test_diff. |
546 |
self.assertNotContainsRe(d, b'---') |
|
2405.1.1
by John Arbash Meinel
Add a bunch of direct tests for 'show_diff_trees' |
547 |
|
548 |
def test_renamed_and_modified_file(self): |
|
549 |
"""Test when a file is only renamed.""" |
|
550 |
tree = self.make_branch_and_tree('tree') |
|
|
6855.4.1
by Jelmer Vernooij
Yet more bees. |
551 |
self.build_tree_contents([('tree/file', b'contents\n')]) |
552 |
tree.add(['file'], [b'file-id']) |
|
553 |
tree.commit('one', rev_id=b'rev-1') |
|
|
2405.1.1
by John Arbash Meinel
Add a bunch of direct tests for 'show_diff_trees' |
554 |
|
555 |
tree.rename_one('file', 'newname') |
|
|
6855.4.1
by Jelmer Vernooij
Yet more bees. |
556 |
self.build_tree_contents([('tree/newname', b'new contents\n')]) |
|
5784.3.1
by Martin Pool
Remove unnecessary TestShowDiffTreesHelper and just use a function |
557 |
d = get_diff_as_string(tree.basis_tree(), tree) |
|
7031.1.1
by Jelmer Vernooij
Fix breezy.tests.test_diff. |
558 |
self.assertContainsRe(d, b"=== renamed file 'file' => 'newname'\n") |
559 |
self.assertContainsRe(d, b'--- old/file\t') |
|
560 |
self.assertContainsRe(d, b'\\+\\+\\+ new/newname\t') |
|
561 |
self.assertContainsRe(d, b'-contents\n' |
|
562 |
b'\\+new contents\n') |
|
|
2405.1.1
by John Arbash Meinel
Add a bunch of direct tests for 'show_diff_trees' |
563 |
|
|
3268.1.1
by C Miller
Describe the property changes in diffs. Currently, this is the executable-bit |
564 |
def test_internal_diff_exec_property(self): |
565 |
tree = self.make_branch_and_tree('tree') |
|
566 |
||
567 |
tt = transform.TreeTransform(tree) |
|
|
6973.13.2
by Jelmer Vernooij
Fix some more tests. |
568 |
tt.new_file('a', tt.root, [b'contents\n'], b'a-id', True) |
569 |
tt.new_file('b', tt.root, [b'contents\n'], b'b-id', False) |
|
570 |
tt.new_file('c', tt.root, [b'contents\n'], b'c-id', True) |
|
571 |
tt.new_file('d', tt.root, [b'contents\n'], b'd-id', False) |
|
572 |
tt.new_file('e', tt.root, [b'contents\n'], b'control-e-id', True) |
|
573 |
tt.new_file('f', tt.root, [b'contents\n'], b'control-f-id', False) |
|
|
3268.1.1
by C Miller
Describe the property changes in diffs. Currently, this is the executable-bit |
574 |
tt.apply() |
|
6855.4.1
by Jelmer Vernooij
Yet more bees. |
575 |
tree.commit('one', rev_id=b'rev-1') |
|
3268.1.1
by C Miller
Describe the property changes in diffs. Currently, this is the executable-bit |
576 |
|
577 |
tt = transform.TreeTransform(tree) |
|
|
6973.13.2
by Jelmer Vernooij
Fix some more tests. |
578 |
tt.set_executability(False, tt.trans_id_file_id(b'a-id')) |
579 |
tt.set_executability(True, tt.trans_id_file_id(b'b-id')) |
|
580 |
tt.set_executability(False, tt.trans_id_file_id(b'c-id')) |
|
581 |
tt.set_executability(True, tt.trans_id_file_id(b'd-id')) |
|
|
3268.1.1
by C Miller
Describe the property changes in diffs. Currently, this is the executable-bit |
582 |
tt.apply() |
583 |
tree.rename_one('c', 'new-c') |
|
584 |
tree.rename_one('d', 'new-d') |
|
585 |
||
|
5784.3.1
by Martin Pool
Remove unnecessary TestShowDiffTreesHelper and just use a function |
586 |
d = get_diff_as_string(tree.basis_tree(), tree) |
|
3268.1.1
by C Miller
Describe the property changes in diffs. Currently, this is the executable-bit |
587 |
|
|
7031.1.1
by Jelmer Vernooij
Fix breezy.tests.test_diff. |
588 |
self.assertContainsRe(d, br"file 'a'.*\(properties changed:" |
589 |
br".*\+x to -x.*\)") |
|
590 |
self.assertContainsRe(d, br"file 'b'.*\(properties changed:" |
|
591 |
br".*-x to \+x.*\)") |
|
592 |
self.assertContainsRe(d, br"file 'c'.*\(properties changed:" |
|
593 |
br".*\+x to -x.*\)") |
|
594 |
self.assertContainsRe(d, br"file 'd'.*\(properties changed:" |
|
595 |
br".*-x to \+x.*\)") |
|
596 |
self.assertNotContainsRe(d, br"file 'e'") |
|
597 |
self.assertNotContainsRe(d, br"file 'f'") |
|
|
3268.1.1
by C Miller
Describe the property changes in diffs. Currently, this is the executable-bit |
598 |
|
|
2592.2.1
by Jonathan Lange
Reproduce and fix bug 110092. |
599 |
def test_binary_unicode_filenames(self): |
|
2592.2.2
by Jonathan Lange
Apply jam's comments to test_binary_unicode_filenames. Change the |
600 |
"""Test that contents of files are *not* encoded in UTF-8 when there |
601 |
is a binary file in the diff.
|
|
|
2592.2.1
by Jonathan Lange
Reproduce and fix bug 110092. |
602 |
"""
|
603 |
# See https://bugs.launchpad.net/bugs/110092.
|
|
|
5967.12.1
by Martin Pool
Move all test features into bzrlib.tests.features |
604 |
self.requireFeature(features.UnicodeFilenameFeature) |
|
2592.2.1
by Jonathan Lange
Reproduce and fix bug 110092. |
605 |
|
606 |
tree = self.make_branch_and_tree('tree') |
|
|
2592.2.2
by Jonathan Lange
Apply jam's comments to test_binary_unicode_filenames. Change the |
607 |
alpha, omega = u'\u03b1', u'\u03c9' |
608 |
alpha_utf8, omega_utf8 = alpha.encode('utf8'), omega.encode('utf8') |
|
|
2592.2.1
by Jonathan Lange
Reproduce and fix bug 110092. |
609 |
self.build_tree_contents( |
|
7031.1.1
by Jelmer Vernooij
Fix breezy.tests.test_diff. |
610 |
[('tree/' + alpha, b'\0'), |
|
2592.2.2
by Jonathan Lange
Apply jam's comments to test_binary_unicode_filenames. Change the |
611 |
('tree/' + omega, |
|
7031.1.1
by Jelmer Vernooij
Fix breezy.tests.test_diff. |
612 |
(b'The %s and the %s\n' % (alpha_utf8, omega_utf8)))]) |
|
6855.4.1
by Jelmer Vernooij
Yet more bees. |
613 |
tree.add([alpha], [b'file-id']) |
614 |
tree.add([omega], [b'file-id-2']) |
|
|
6621.22.2
by Martin
Use BytesIO or StringIO from bzrlib.sixish |
615 |
diff_content = StubO() |
|
5168.1.4
by Vincent Ladeuil
Final import fixes for bt.test_diff. |
616 |
diff.show_diff_trees(tree.basis_tree(), tree, diff_content) |
|
6621.22.2
by Martin
Use BytesIO or StringIO from bzrlib.sixish |
617 |
diff_content.check_types(self, bytes) |
618 |
d = b''.join(diff_content.write_record) |
|
|
7031.1.1
by Jelmer Vernooij
Fix breezy.tests.test_diff. |
619 |
self.assertContainsRe(d, br"=== added file '%s'" % alpha_utf8) |
620 |
self.assertContainsRe(d, b"Binary files a/%s.*and b/%s.* differ\n" |
|
|
5168.1.4
by Vincent Ladeuil
Final import fixes for bt.test_diff. |
621 |
% (alpha_utf8, alpha_utf8)) |
|
7031.1.1
by Jelmer Vernooij
Fix breezy.tests.test_diff. |
622 |
self.assertContainsRe(d, br"=== added file '%s'" % omega_utf8) |
623 |
self.assertContainsRe(d, br"--- a/%s" % (omega_utf8,)) |
|
624 |
self.assertContainsRe(d, br"\+\+\+ b/%s" % (omega_utf8,)) |
|
|
2592.2.1
by Jonathan Lange
Reproduce and fix bug 110092. |
625 |
|
|
2725.2.1
by ghigo
When a unicode filename is renamed, in the diff is showed a wrong result |
626 |
def test_unicode_filename(self): |
627 |
"""Test when the filename are unicode.""" |
|
|
5967.12.1
by Martin Pool
Move all test features into bzrlib.tests.features |
628 |
self.requireFeature(features.UnicodeFilenameFeature) |
|
2725.2.1
by ghigo
When a unicode filename is renamed, in the diff is showed a wrong result |
629 |
|
630 |
alpha, omega = u'\u03b1', u'\u03c9' |
|
631 |
autf8, outf8 = alpha.encode('utf8'), omega.encode('utf8') |
|
632 |
||
633 |
tree = self.make_branch_and_tree('tree') |
|
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
634 |
self.build_tree_contents([('tree/ren_' + alpha, b'contents\n')]) |
635 |
tree.add(['ren_' + alpha], [b'file-id-2']) |
|
636 |
self.build_tree_contents([('tree/del_' + alpha, b'contents\n')]) |
|
637 |
tree.add(['del_' + alpha], [b'file-id-3']) |
|
638 |
self.build_tree_contents([('tree/mod_' + alpha, b'contents\n')]) |
|
639 |
tree.add(['mod_' + alpha], [b'file-id-4']) |
|
|
2725.2.1
by ghigo
When a unicode filename is renamed, in the diff is showed a wrong result |
640 |
|
|
6855.4.1
by Jelmer Vernooij
Yet more bees. |
641 |
tree.commit('one', rev_id=b'rev-1') |
|
2725.2.1
by ghigo
When a unicode filename is renamed, in the diff is showed a wrong result |
642 |
|
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
643 |
tree.rename_one('ren_' + alpha, 'ren_' + omega) |
644 |
tree.remove('del_' + alpha) |
|
645 |
self.build_tree_contents([('tree/add_' + alpha, b'contents\n')]) |
|
646 |
tree.add(['add_' + alpha], [b'file-id']) |
|
647 |
self.build_tree_contents([('tree/mod_' + alpha, b'contents_mod\n')]) |
|
|
2725.2.1
by ghigo
When a unicode filename is renamed, in the diff is showed a wrong result |
648 |
|
|
5784.3.1
by Martin Pool
Remove unnecessary TestShowDiffTreesHelper and just use a function |
649 |
d = get_diff_as_string(tree.basis_tree(), tree) |
|
5168.1.4
by Vincent Ladeuil
Final import fixes for bt.test_diff. |
650 |
self.assertContainsRe(d, |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
651 |
b"=== renamed file 'ren_%s' => 'ren_%s'\n" % (autf8, outf8)) |
652 |
self.assertContainsRe(d, b"=== added file 'add_%s'" % autf8) |
|
653 |
self.assertContainsRe(d, b"=== modified file 'mod_%s'" % autf8) |
|
654 |
self.assertContainsRe(d, b"=== removed file 'del_%s'" % autf8) |
|
|
5168.1.4
by Vincent Ladeuil
Final import fixes for bt.test_diff. |
655 |
|
|
4797.57.6
by Alexander Belchenko
added whitebox test for path_encoding in diff. |
656 |
def test_unicode_filename_path_encoding(self): |
657 |
"""Test for bug #382699: unicode filenames on Windows should be shown |
|
658 |
in user encoding.
|
|
659 |
"""
|
|
|
5967.12.1
by Martin Pool
Move all test features into bzrlib.tests.features |
660 |
self.requireFeature(features.UnicodeFilenameFeature) |
|
4797.57.6
by Alexander Belchenko
added whitebox test for path_encoding in diff. |
661 |
# The word 'test' in Russian
|
662 |
_russian_test = u'\u0422\u0435\u0441\u0442' |
|
663 |
directory = _russian_test + u'/' |
|
664 |
test_txt = _russian_test + u'.txt' |
|
665 |
u1234 = u'\u1234.txt' |
|
666 |
||
667 |
tree = self.make_branch_and_tree('.') |
|
668 |
self.build_tree_contents([ |
|
|
6855.4.1
by Jelmer Vernooij
Yet more bees. |
669 |
(test_txt, b'foo\n'), |
670 |
(u1234, b'foo\n'), |
|
|
4797.57.6
by Alexander Belchenko
added whitebox test for path_encoding in diff. |
671 |
(directory, None), |
672 |
])
|
|
673 |
tree.add([test_txt, u1234, directory]) |
|
674 |
||
|
6621.22.2
by Martin
Use BytesIO or StringIO from bzrlib.sixish |
675 |
sio = BytesIO() |
|
5258.1.1
by Alexander Belchenko
merge diff header work from my 2.1 branch |
676 |
diff.show_diff_trees(tree.basis_tree(), tree, sio, |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
677 |
path_encoding='cp1251') |
|
4797.57.6
by Alexander Belchenko
added whitebox test for path_encoding in diff. |
678 |
|
679 |
output = subst_dates(sio.getvalue()) |
|
|
7031.1.1
by Jelmer Vernooij
Fix breezy.tests.test_diff. |
680 |
shouldbe = (b'''\ |
|
4797.57.6
by Alexander Belchenko
added whitebox test for path_encoding in diff. |
681 |
=== added directory '%(directory)s' |
682 |
=== added file '%(test_txt)s' |
|
683 |
--- a/%(test_txt)s\tYYYY-MM-DD HH:MM:SS +ZZZZ |
|
684 |
+++ b/%(test_txt)s\tYYYY-MM-DD HH:MM:SS +ZZZZ |
|
685 |
@@ -0,0 +1,1 @@
|
|
686 |
+foo
|
|
687 |
||
688 |
=== added file '?.txt'
|
|
689 |
--- a/?.txt\tYYYY-MM-DD HH:MM:SS +ZZZZ |
|
690 |
+++ b/?.txt\tYYYY-MM-DD HH:MM:SS +ZZZZ |
|
691 |
@@ -0,0 +1,1 @@
|
|
692 |
+foo
|
|
693 |
||
|
7031.1.1
by Jelmer Vernooij
Fix breezy.tests.test_diff. |
694 |
''' % {b'directory': _russian_test.encode('cp1251'), |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
695 |
b'test_txt': test_txt.encode('cp1251'), |
696 |
})
|
|
|
4797.57.6
by Alexander Belchenko
added whitebox test for path_encoding in diff. |
697 |
self.assertEqualDiff(output, shouldbe) |
698 |
||
|
5168.1.4
by Vincent Ladeuil
Final import fixes for bt.test_diff. |
699 |
|
700 |
class DiffWasIs(diff.DiffPath): |
|
|
3009.2.15
by Aaron Bentley
Test differ registration |
701 |
|
|
7206.6.2
by Jelmer Vernooij
Remove file_id from diff API. |
702 |
def diff(self, old_path, new_path, old_kind, new_kind): |
|
7031.1.1
by Jelmer Vernooij
Fix breezy.tests.test_diff. |
703 |
self.to_file.write(b'was: ') |
|
6809.4.5
by Jelmer Vernooij
Swap arguments for get_file_*. |
704 |
self.to_file.write(self.old_tree.get_file(old_path).read()) |
|
7031.1.1
by Jelmer Vernooij
Fix breezy.tests.test_diff. |
705 |
self.to_file.write(b'is: ') |
|
6809.4.5
by Jelmer Vernooij
Swap arguments for get_file_*. |
706 |
self.to_file.write(self.new_tree.get_file(new_path).read()) |
|
3009.2.15
by Aaron Bentley
Test differ registration |
707 |
|
708 |
||
|
5168.1.2
by Vincent Ladeuil
Ckeanup some more imports. |
709 |
class TestDiffTree(tests.TestCaseWithTransport): |
|
3009.2.9
by Aaron Bentley
Add tests for Differ |
710 |
|
711 |
def setUp(self): |
|
|
5168.1.2
by Vincent Ladeuil
Ckeanup some more imports. |
712 |
super(TestDiffTree, self).setUp() |
|
3009.2.9
by Aaron Bentley
Add tests for Differ |
713 |
self.old_tree = self.make_branch_and_tree('old-tree') |
714 |
self.old_tree.lock_write() |
|
715 |
self.addCleanup(self.old_tree.unlock) |
|
716 |
self.new_tree = self.make_branch_and_tree('new-tree') |
|
717 |
self.new_tree.lock_write() |
|
718 |
self.addCleanup(self.new_tree.unlock) |
|
|
6621.22.2
by Martin
Use BytesIO or StringIO from bzrlib.sixish |
719 |
self.differ = diff.DiffTree(self.old_tree, self.new_tree, BytesIO()) |
|
3009.2.9
by Aaron Bentley
Add tests for Differ |
720 |
|
721 |
def test_diff_text(self): |
|
722 |
self.build_tree_contents([('old-tree/olddir/',), |
|
|
6855.4.1
by Jelmer Vernooij
Yet more bees. |
723 |
('old-tree/olddir/oldfile', b'old\n')]) |
|
3009.2.9
by Aaron Bentley
Add tests for Differ |
724 |
self.old_tree.add('olddir') |
|
6855.4.1
by Jelmer Vernooij
Yet more bees. |
725 |
self.old_tree.add('olddir/oldfile', b'file-id') |
|
3009.2.9
by Aaron Bentley
Add tests for Differ |
726 |
self.build_tree_contents([('new-tree/newdir/',), |
|
6855.4.1
by Jelmer Vernooij
Yet more bees. |
727 |
('new-tree/newdir/newfile', b'new\n')]) |
|
3009.2.9
by Aaron Bentley
Add tests for Differ |
728 |
self.new_tree.add('newdir') |
|
6855.4.1
by Jelmer Vernooij
Yet more bees. |
729 |
self.new_tree.add('newdir/newfile', b'file-id') |
|
6621.22.2
by Martin
Use BytesIO or StringIO from bzrlib.sixish |
730 |
differ = diff.DiffText(self.old_tree, self.new_tree, BytesIO()) |
|
7206.6.2
by Jelmer Vernooij
Remove file_id from diff API. |
731 |
differ.diff_text('olddir/oldfile', None, 'old label', 'new label') |
|
3009.2.9
by Aaron Bentley
Add tests for Differ |
732 |
self.assertEqual( |
|
7031.1.1
by Jelmer Vernooij
Fix breezy.tests.test_diff. |
733 |
b'--- 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 |
734 |
differ.to_file.getvalue()) |
735 |
differ.to_file.seek(0) |
|
|
6809.4.9
by Jelmer Vernooij
Fix some more tests. |
736 |
differ.diff_text(None, 'newdir/newfile', |
|
7206.6.2
by Jelmer Vernooij
Remove file_id from diff API. |
737 |
'old label', 'new label') |
|
3009.2.9
by Aaron Bentley
Add tests for Differ |
738 |
self.assertEqual( |
|
7031.1.1
by Jelmer Vernooij
Fix breezy.tests.test_diff. |
739 |
b'--- 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 |
740 |
differ.to_file.getvalue()) |
741 |
differ.to_file.seek(0) |
|
|
6809.4.9
by Jelmer Vernooij
Fix some more tests. |
742 |
differ.diff_text('olddir/oldfile', 'newdir/newfile', |
|
7206.6.2
by Jelmer Vernooij
Remove file_id from diff API. |
743 |
'old label', 'new label') |
|
3009.2.9
by Aaron Bentley
Add tests for Differ |
744 |
self.assertEqual( |
|
7031.1.1
by Jelmer Vernooij
Fix breezy.tests.test_diff. |
745 |
b'--- 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 |
746 |
differ.to_file.getvalue()) |
|
3009.2.9
by Aaron Bentley
Add tests for Differ |
747 |
|
|
3087.1.1
by Aaron Bentley
Diff handles missing files correctly, with no tracebacks |
748 |
def test_diff_deletion(self): |
|
6855.4.1
by Jelmer Vernooij
Yet more bees. |
749 |
self.build_tree_contents([('old-tree/file', b'contents'), |
750 |
('new-tree/file', b'contents')]) |
|
751 |
self.old_tree.add('file', b'file-id') |
|
752 |
self.new_tree.add('file', b'file-id') |
|
|
3087.1.1
by Aaron Bentley
Diff handles missing files correctly, with no tracebacks |
753 |
os.unlink('new-tree/file') |
754 |
self.differ.show_diff(None) |
|
|
7031.1.1
by Jelmer Vernooij
Fix breezy.tests.test_diff. |
755 |
self.assertContainsRe(self.differ.to_file.getvalue(), b'-contents') |
|
3087.1.1
by Aaron Bentley
Diff handles missing files correctly, with no tracebacks |
756 |
|
757 |
def test_diff_creation(self): |
|
|
6855.4.1
by Jelmer Vernooij
Yet more bees. |
758 |
self.build_tree_contents([('old-tree/file', b'contents'), |
759 |
('new-tree/file', b'contents')]) |
|
760 |
self.old_tree.add('file', b'file-id') |
|
761 |
self.new_tree.add('file', b'file-id') |
|
|
3087.1.1
by Aaron Bentley
Diff handles missing files correctly, with no tracebacks |
762 |
os.unlink('old-tree/file') |
763 |
self.differ.show_diff(None) |
|
|
7031.1.1
by Jelmer Vernooij
Fix breezy.tests.test_diff. |
764 |
self.assertContainsRe(self.differ.to_file.getvalue(), br'\+contents') |
|
3087.1.1
by Aaron Bentley
Diff handles missing files correctly, with no tracebacks |
765 |
|
|
3009.2.9
by Aaron Bentley
Add tests for Differ |
766 |
def test_diff_symlink(self): |
|
6621.22.2
by Martin
Use BytesIO or StringIO from bzrlib.sixish |
767 |
differ = diff.DiffSymlink(self.old_tree, self.new_tree, BytesIO()) |
|
3009.2.11
by Aaron Bentley
Refactor diff to be more pluggable |
768 |
differ.diff_symlink('old target', None) |
|
7031.1.1
by Jelmer Vernooij
Fix breezy.tests.test_diff. |
769 |
self.assertEqual(b"=== target was 'old target'\n", |
|
3009.2.11
by Aaron Bentley
Refactor diff to be more pluggable |
770 |
differ.to_file.getvalue()) |
|
3009.2.9
by Aaron Bentley
Add tests for Differ |
771 |
|
|
6621.22.2
by Martin
Use BytesIO or StringIO from bzrlib.sixish |
772 |
differ = diff.DiffSymlink(self.old_tree, self.new_tree, BytesIO()) |
|
3009.2.11
by Aaron Bentley
Refactor diff to be more pluggable |
773 |
differ.diff_symlink(None, 'new target') |
|
7031.1.1
by Jelmer Vernooij
Fix breezy.tests.test_diff. |
774 |
self.assertEqual(b"=== target is 'new target'\n", |
|
3009.2.11
by Aaron Bentley
Refactor diff to be more pluggable |
775 |
differ.to_file.getvalue()) |
776 |
||
|
6621.22.2
by Martin
Use BytesIO or StringIO from bzrlib.sixish |
777 |
differ = diff.DiffSymlink(self.old_tree, self.new_tree, BytesIO()) |
|
3009.2.11
by Aaron Bentley
Refactor diff to be more pluggable |
778 |
differ.diff_symlink('old target', 'new target') |
|
7031.1.1
by Jelmer Vernooij
Fix breezy.tests.test_diff. |
779 |
self.assertEqual(b"=== target changed 'old target' => 'new target'\n", |
|
3009.2.11
by Aaron Bentley
Refactor diff to be more pluggable |
780 |
differ.to_file.getvalue()) |
|
3009.2.9
by Aaron Bentley
Add tests for Differ |
781 |
|
782 |
def test_diff(self): |
|
783 |
self.build_tree_contents([('old-tree/olddir/',), |
|
|
6855.4.1
by Jelmer Vernooij
Yet more bees. |
784 |
('old-tree/olddir/oldfile', b'old\n')]) |
|
3009.2.9
by Aaron Bentley
Add tests for Differ |
785 |
self.old_tree.add('olddir') |
|
6855.4.1
by Jelmer Vernooij
Yet more bees. |
786 |
self.old_tree.add('olddir/oldfile', b'file-id') |
|
3009.2.9
by Aaron Bentley
Add tests for Differ |
787 |
self.build_tree_contents([('new-tree/newdir/',), |
|
6855.4.1
by Jelmer Vernooij
Yet more bees. |
788 |
('new-tree/newdir/newfile', b'new\n')]) |
|
3009.2.9
by Aaron Bentley
Add tests for Differ |
789 |
self.new_tree.add('newdir') |
|
6855.4.1
by Jelmer Vernooij
Yet more bees. |
790 |
self.new_tree.add('newdir/newfile', b'file-id') |
|
7206.6.2
by Jelmer Vernooij
Remove file_id from diff API. |
791 |
self.differ.diff('olddir/oldfile', 'newdir/newfile') |
|
3009.2.9
by Aaron Bentley
Add tests for Differ |
792 |
self.assertContainsRe( |
793 |
self.differ.to_file.getvalue(), |
|
|
7031.1.1
by Jelmer Vernooij
Fix breezy.tests.test_diff. |
794 |
br'--- olddir/oldfile.*\n\+\+\+ newdir/newfile.*\n\@\@ -1,1 \+1,1' |
795 |
br' \@\@\n-old\n\+new\n\n') |
|
|
3009.2.9
by Aaron Bentley
Add tests for Differ |
796 |
|
797 |
def test_diff_kind_change(self): |
|
|
5967.12.1
by Martin Pool
Move all test features into bzrlib.tests.features |
798 |
self.requireFeature(features.SymlinkFeature) |
|
3009.2.9
by Aaron Bentley
Add tests for Differ |
799 |
self.build_tree_contents([('old-tree/olddir/',), |
|
6855.4.1
by Jelmer Vernooij
Yet more bees. |
800 |
('old-tree/olddir/oldfile', b'old\n')]) |
|
3009.2.9
by Aaron Bentley
Add tests for Differ |
801 |
self.old_tree.add('olddir') |
|
6855.4.1
by Jelmer Vernooij
Yet more bees. |
802 |
self.old_tree.add('olddir/oldfile', b'file-id') |
|
3009.2.9
by Aaron Bentley
Add tests for Differ |
803 |
self.build_tree(['new-tree/newdir/']) |
804 |
os.symlink('new', 'new-tree/newdir/newfile') |
|
805 |
self.new_tree.add('newdir') |
|
|
6855.4.1
by Jelmer Vernooij
Yet more bees. |
806 |
self.new_tree.add('newdir/newfile', b'file-id') |
|
7206.6.2
by Jelmer Vernooij
Remove file_id from diff API. |
807 |
self.differ.diff('olddir/oldfile', 'newdir/newfile') |
|
3009.2.9
by Aaron Bentley
Add tests for Differ |
808 |
self.assertContainsRe( |
809 |
self.differ.to_file.getvalue(), |
|
|
7031.1.1
by Jelmer Vernooij
Fix breezy.tests.test_diff. |
810 |
br'--- olddir/oldfile.*\n\+\+\+ newdir/newfile.*\n\@\@ -1,1 \+0,0' |
811 |
br' \@\@\n-old\n\n') |
|
|
3009.2.9
by Aaron Bentley
Add tests for Differ |
812 |
self.assertContainsRe(self.differ.to_file.getvalue(), |
|
7031.1.1
by Jelmer Vernooij
Fix breezy.tests.test_diff. |
813 |
b"=== target is 'new'\n") |
|
3009.2.9
by Aaron Bentley
Add tests for Differ |
814 |
|
|
3009.2.19
by Aaron Bentley
Implement directory diffing |
815 |
def test_diff_directory(self): |
816 |
self.build_tree(['new-tree/new-dir/']) |
|
|
6855.4.1
by Jelmer Vernooij
Yet more bees. |
817 |
self.new_tree.add('new-dir', b'new-dir-id') |
|
7206.6.2
by Jelmer Vernooij
Remove file_id from diff API. |
818 |
self.differ.diff(None, 'new-dir') |
|
6973.12.3
by Jelmer Vernooij
Fixes. |
819 |
self.assertEqual(self.differ.to_file.getvalue(), b'') |
|
3009.2.19
by Aaron Bentley
Implement directory diffing |
820 |
|
|
3009.2.16
by Aaron Bentley
Test support for extra differs |
821 |
def create_old_new(self): |
822 |
self.build_tree_contents([('old-tree/olddir/',), |
|
|
6855.4.1
by Jelmer Vernooij
Yet more bees. |
823 |
('old-tree/olddir/oldfile', b'old\n')]) |
|
3009.2.16
by Aaron Bentley
Test support for extra differs |
824 |
self.old_tree.add('olddir') |
|
6855.4.1
by Jelmer Vernooij
Yet more bees. |
825 |
self.old_tree.add('olddir/oldfile', b'file-id') |
|
3009.2.16
by Aaron Bentley
Test support for extra differs |
826 |
self.build_tree_contents([('new-tree/newdir/',), |
|
6855.4.1
by Jelmer Vernooij
Yet more bees. |
827 |
('new-tree/newdir/newfile', b'new\n')]) |
|
3009.2.16
by Aaron Bentley
Test support for extra differs |
828 |
self.new_tree.add('newdir') |
|
6855.4.1
by Jelmer Vernooij
Yet more bees. |
829 |
self.new_tree.add('newdir/newfile', b'file-id') |
|
3009.2.16
by Aaron Bentley
Test support for extra differs |
830 |
|
|
3009.2.27
by Aaron Bentley
Use extra_factories instead of extra_diffs |
831 |
def test_register_diff(self): |
|
3009.2.16
by Aaron Bentley
Test support for extra differs |
832 |
self.create_old_new() |
|
5168.1.4
by Vincent Ladeuil
Final import fixes for bt.test_diff. |
833 |
old_diff_factories = diff.DiffTree.diff_factories |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
834 |
diff.DiffTree.diff_factories = old_diff_factories[:] |
|
5168.1.4
by Vincent Ladeuil
Final import fixes for bt.test_diff. |
835 |
diff.DiffTree.diff_factories.insert(0, DiffWasIs.from_diff_tree) |
|
3009.2.16
by Aaron Bentley
Test support for extra differs |
836 |
try: |
|
6621.22.2
by Martin
Use BytesIO or StringIO from bzrlib.sixish |
837 |
differ = diff.DiffTree(self.old_tree, self.new_tree, BytesIO()) |
|
3009.2.16
by Aaron Bentley
Test support for extra differs |
838 |
finally: |
|
5168.1.4
by Vincent Ladeuil
Final import fixes for bt.test_diff. |
839 |
diff.DiffTree.diff_factories = old_diff_factories |
|
7206.6.2
by Jelmer Vernooij
Remove file_id from diff API. |
840 |
differ.diff('olddir/oldfile', 'newdir/newfile') |
|
3009.2.16
by Aaron Bentley
Test support for extra differs |
841 |
self.assertNotContainsRe( |
842 |
differ.to_file.getvalue(), |
|
|
7031.1.1
by Jelmer Vernooij
Fix breezy.tests.test_diff. |
843 |
br'--- olddir/oldfile.*\n\+\+\+ newdir/newfile.*\n\@\@ -1,1 \+1,1' |
844 |
br' \@\@\n-old\n\+new\n\n') |
|
|
3009.2.16
by Aaron Bentley
Test support for extra differs |
845 |
self.assertContainsRe(differ.to_file.getvalue(), |
|
7031.1.1
by Jelmer Vernooij
Fix breezy.tests.test_diff. |
846 |
b'was: old\nis: new\n') |
|
3009.2.16
by Aaron Bentley
Test support for extra differs |
847 |
|
|
3009.2.27
by Aaron Bentley
Use extra_factories instead of extra_diffs |
848 |
def test_extra_factories(self): |
|
3009.2.16
by Aaron Bentley
Test support for extra differs |
849 |
self.create_old_new() |
|
6621.22.2
by Martin
Use BytesIO or StringIO from bzrlib.sixish |
850 |
differ = diff.DiffTree(self.old_tree, self.new_tree, BytesIO(), |
|
5168.1.4
by Vincent Ladeuil
Final import fixes for bt.test_diff. |
851 |
extra_factories=[DiffWasIs.from_diff_tree]) |
|
7206.6.2
by Jelmer Vernooij
Remove file_id from diff API. |
852 |
differ.diff('olddir/oldfile', 'newdir/newfile') |
|
3009.2.16
by Aaron Bentley
Test support for extra differs |
853 |
self.assertNotContainsRe( |
854 |
differ.to_file.getvalue(), |
|
|
7031.1.1
by Jelmer Vernooij
Fix breezy.tests.test_diff. |
855 |
br'--- olddir/oldfile.*\n\+\+\+ newdir/newfile.*\n\@\@ -1,1 \+1,1' |
856 |
br' \@\@\n-old\n\+new\n\n') |
|
|
3009.2.16
by Aaron Bentley
Test support for extra differs |
857 |
self.assertContainsRe(differ.to_file.getvalue(), |
|
7031.1.1
by Jelmer Vernooij
Fix breezy.tests.test_diff. |
858 |
b'was: old\nis: new\n') |
|
3009.2.16
by Aaron Bentley
Test support for extra differs |
859 |
|
|
3123.4.1
by Aaron Bentley
Diff sorts files in alphabetical order |
860 |
def test_alphabetical_order(self): |
861 |
self.build_tree(['new-tree/a-file']) |
|
862 |
self.new_tree.add('a-file') |
|
863 |
self.build_tree(['old-tree/b-file']) |
|
864 |
self.old_tree.add('b-file') |
|
865 |
self.differ.show_diff(None) |
|
866 |
self.assertContainsRe(self.differ.to_file.getvalue(), |
|
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
867 |
b'.*a-file(.|\n)*b-file') |
|
3123.4.1
by Aaron Bentley
Diff sorts files in alphabetical order |
868 |
|
|
3009.2.9
by Aaron Bentley
Add tests for Differ |
869 |
|
|
5168.1.2
by Vincent Ladeuil
Ckeanup some more imports. |
870 |
class TestPatienceDiffLib(tests.TestCase): |
|
1185.81.1
by John Arbash Meinel
Adding nofrillsprecisemerge's diff algorithm, wrapped in difflib. |
871 |
|
|
2781.1.1
by Martin Pool
merge cpatiencediff from Lukas |
872 |
def setUp(self): |
873 |
super(TestPatienceDiffLib, self).setUp() |
|
|
5168.1.3
by Vincent Ladeuil
Even more import fixes. |
874 |
self._unique_lcs = _patiencediff_py.unique_lcs_py |
875 |
self._recurse_matches = _patiencediff_py.recurse_matches_py |
|
|
2781.1.1
by Martin Pool
merge cpatiencediff from Lukas |
876 |
self._PatienceSequenceMatcher = \ |
|
5168.1.3
by Vincent Ladeuil
Even more import fixes. |
877 |
_patiencediff_py.PatienceSequenceMatcher_py |
|
2781.1.1
by Martin Pool
merge cpatiencediff from Lukas |
878 |
|
|
3628.1.3
by Lukáš Lalinský
Add a test |
879 |
def test_diff_unicode_string(self): |
880 |
a = ''.join([unichr(i) for i in range(4000, 4500, 3)]) |
|
881 |
b = ''.join([unichr(i) for i in range(4300, 4800, 2)]) |
|
882 |
sm = self._PatienceSequenceMatcher(None, a, b) |
|
883 |
mb = sm.get_matching_blocks() |
|
|
6614.1.3
by Vincent Ladeuil
Fix assertEquals being deprecated by using assertEqual. |
884 |
self.assertEqual(35, len(mb)) |
|
3628.1.3
by Lukáš Lalinský
Add a test |
885 |
|
|
1185.81.9
by John Arbash Meinel
Added (failing) tests for cdv.recurse_matches with common sections, |
886 |
def test_unique_lcs(self): |
|
2781.1.1
by Martin Pool
merge cpatiencediff from Lukas |
887 |
unique_lcs = self._unique_lcs |
|
6614.1.3
by Vincent Ladeuil
Fix assertEquals being deprecated by using assertEqual. |
888 |
self.assertEqual(unique_lcs('', ''), []) |
889 |
self.assertEqual(unique_lcs('', 'a'), []) |
|
890 |
self.assertEqual(unique_lcs('a', ''), []) |
|
|
6809.1.1
by Martin
Apply 2to3 ws_comma fixer |
891 |
self.assertEqual(unique_lcs('a', 'a'), [(0, 0)]) |
|
6614.1.3
by Vincent Ladeuil
Fix assertEquals being deprecated by using assertEqual. |
892 |
self.assertEqual(unique_lcs('a', 'b'), []) |
|
6809.1.1
by Martin
Apply 2to3 ws_comma fixer |
893 |
self.assertEqual(unique_lcs('ab', 'ab'), [(0, 0), (1, 1)]) |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
894 |
self.assertEqual(unique_lcs('abcde', 'cdeab'), |
895 |
[(2, 0), (3, 1), (4, 2)]) |
|
896 |
self.assertEqual(unique_lcs('cdeab', 'abcde'), |
|
897 |
[(0, 2), (1, 3), (2, 4)]) |
|
|
6809.1.1
by Martin
Apply 2to3 ws_comma fixer |
898 |
self.assertEqual(unique_lcs('abXde', 'abYde'), [(0, 0), (1, 1), |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
899 |
(3, 3), (4, 4)]) |
|
6809.1.1
by Martin
Apply 2to3 ws_comma fixer |
900 |
self.assertEqual(unique_lcs('acbac', 'abc'), [(2, 1)]) |
|
1185.81.9
by John Arbash Meinel
Added (failing) tests for cdv.recurse_matches with common sections, |
901 |
|
902 |
def test_recurse_matches(self): |
|
903 |
def test_one(a, b, matches): |
|
904 |
test_matches = [] |
|
|
2781.1.1
by Martin Pool
merge cpatiencediff from Lukas |
905 |
self._recurse_matches( |
906 |
a, b, 0, 0, len(a), len(b), test_matches, 10) |
|
|
6614.1.3
by Vincent Ladeuil
Fix assertEquals being deprecated by using assertEqual. |
907 |
self.assertEqual(test_matches, matches) |
|
1185.81.9
by John Arbash Meinel
Added (failing) tests for cdv.recurse_matches with common sections, |
908 |
|
|
1711.2.17
by John Arbash Meinel
Small cleanups to patience_diff code. |
909 |
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, |
910 |
[(0, 0), (2, 2), (4, 4)]) |
911 |
test_one(['a', 'c', 'b', 'a', 'c'], ['a', 'b', 'c'], |
|
912 |
[(0, 0), (2, 1), (4, 2)]) |
|
|
2781.1.1
by Martin Pool
merge cpatiencediff from Lukas |
913 |
# Even though 'bc' is not unique globally, and is surrounded by
|
914 |
# non-matching lines, we should still match, because they are locally
|
|
915 |
# unique
|
|
|
6809.1.1
by Martin
Apply 2to3 ws_comma fixer |
916 |
test_one('abcdbce', 'afbcgdbce', [(0, 0), (1, 2), (2, 3), (3, 5), |
|
2781.1.1
by Martin Pool
merge cpatiencediff from Lukas |
917 |
(4, 6), (5, 7), (6, 8)]) |
|
1185.81.9
by John Arbash Meinel
Added (failing) tests for cdv.recurse_matches with common sections, |
918 |
|
|
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
919 |
# 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 |
920 |
# lines surrounded by bogus text.
|
|
1185.81.24
by Aaron Bentley
Reoganize patience-related code |
921 |
# 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 |
922 |
|
923 |
# This is what it could be
|
|
|
1185.81.9
by John Arbash Meinel
Added (failing) tests for cdv.recurse_matches with common sections, |
924 |
#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 |
925 |
|
|
1185.81.9
by John Arbash Meinel
Added (failing) tests for cdv.recurse_matches with common sections, |
926 |
# This is what it currently gives:
|
|
6809.1.1
by Martin
Apply 2to3 ws_comma fixer |
927 |
test_one('aBccDe', 'abccde', [(0, 0), (5, 5)]) |
|
1185.81.9
by John Arbash Meinel
Added (failing) tests for cdv.recurse_matches with common sections, |
928 |
|
|
3074.2.10
by John Arbash Meinel
Cleanup the test cases (Andrew) |
929 |
def assertDiffBlocks(self, a, b, expected_blocks): |
930 |
"""Check that the sequence matcher returns the correct blocks. |
|
931 |
||
932 |
:param a: A sequence to match
|
|
933 |
:param b: Another sequence to match
|
|
934 |
:param expected_blocks: The expected output, not including the final
|
|
935 |
matching block (len(a), len(b), 0)
|
|
936 |
"""
|
|
937 |
matcher = self._PatienceSequenceMatcher(None, a, b) |
|
938 |
blocks = matcher.get_matching_blocks() |
|
939 |
last = blocks.pop() |
|
940 |
self.assertEqual((len(a), len(b), 0), last) |
|
941 |
self.assertEqual(expected_blocks, blocks) |
|
942 |
||
|
1185.81.1
by John Arbash Meinel
Adding nofrillsprecisemerge's diff algorithm, wrapped in difflib. |
943 |
def test_matching_blocks(self): |
|
1185.81.2
by John Arbash Meinel
A couple small tests. |
944 |
# Some basic matching tests
|
|
3074.2.10
by John Arbash Meinel
Cleanup the test cases (Andrew) |
945 |
self.assertDiffBlocks('', '', []) |
946 |
self.assertDiffBlocks([], [], []) |
|
947 |
self.assertDiffBlocks('abc', '', []) |
|
948 |
self.assertDiffBlocks('', 'abc', []) |
|
949 |
self.assertDiffBlocks('abcd', 'abcd', [(0, 0, 4)]) |
|
950 |
self.assertDiffBlocks('abcd', 'abce', [(0, 0, 3)]) |
|
951 |
self.assertDiffBlocks('eabc', 'abce', [(1, 0, 3)]) |
|
952 |
self.assertDiffBlocks('eabce', 'abce', [(1, 0, 4)]) |
|
953 |
self.assertDiffBlocks('abcde', 'abXde', [(0, 0, 2), (3, 3, 2)]) |
|
954 |
self.assertDiffBlocks('abcde', 'abXYZde', [(0, 0, 2), (3, 5, 2)]) |
|
955 |
self.assertDiffBlocks('abde', 'abXYZde', [(0, 0, 2), (2, 5, 2)]) |
|
956 |
# This may check too much, but it checks to see that
|
|
|
1185.81.3
by John Arbash Meinel
Adding tests for checking opcodes. |
957 |
# a copied block stays attached to the previous section,
|
958 |
# not the later one.
|
|
959 |
# difflib would tend to grab the trailing longest match
|
|
960 |
# which would make the diff not look right
|
|
|
3074.2.10
by John Arbash Meinel
Cleanup the test cases (Andrew) |
961 |
self.assertDiffBlocks('abcdefghijklmnop', 'abcdefxydefghijklmnop', |
962 |
[(0, 0, 6), (6, 11, 10)]) |
|
|
1185.81.1
by John Arbash Meinel
Adding nofrillsprecisemerge's diff algorithm, wrapped in difflib. |
963 |
|
|
1185.81.2
by John Arbash Meinel
A couple small tests. |
964 |
# make sure it supports passing in lists
|
|
3074.2.10
by John Arbash Meinel
Cleanup the test cases (Andrew) |
965 |
self.assertDiffBlocks( |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
966 |
['hello there\n', |
967 |
'world\n', |
|
968 |
'how are you today?\n'], |
|
969 |
['hello there\n', |
|
970 |
'how are you today?\n'], |
|
971 |
[(0, 0, 1), (2, 1, 1)]) |
|
|
1185.81.1
by John Arbash Meinel
Adding nofrillsprecisemerge's diff algorithm, wrapped in difflib. |
972 |
|
|
1711.2.21
by John Arbash Meinel
Cleanup patiencediff, remove the use of difflib.SequenceMatcher. |
973 |
# non unique lines surrounded by non-matching lines
|
974 |
# won't be found
|
|
|
6809.1.1
by Martin
Apply 2to3 ws_comma fixer |
975 |
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. |
976 |
|
977 |
# But they only need to be locally unique
|
|
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
978 |
self.assertDiffBlocks('aBcDec', 'abcdec', [ |
979 |
(0, 0, 1), (2, 2, 1), (4, 4, 2)]) |
|
|
1711.2.21
by John Arbash Meinel
Cleanup patiencediff, remove the use of difflib.SequenceMatcher. |
980 |
|
981 |
# non unique blocks won't be matched
|
|
|
6809.1.1
by Martin
Apply 2to3 ws_comma fixer |
982 |
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. |
983 |
|
984 |
# but locally unique ones will
|
|
|
6809.1.1
by Martin
Apply 2to3 ws_comma fixer |
985 |
self.assertDiffBlocks('aBcdEeXcdFg', 'abcdecdfg', [(0, 0, 1), (2, 2, 2), |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
986 |
(5, 4, 1), (7, 5, 2), (10, 8, 1)]) |
|
1711.2.21
by John Arbash Meinel
Cleanup patiencediff, remove the use of difflib.SequenceMatcher. |
987 |
|
|
6809.1.1
by Martin
Apply 2to3 ws_comma fixer |
988 |
self.assertDiffBlocks('abbabbXd', 'cabbabxd', [(7, 7, 1)]) |
|
3074.2.10
by John Arbash Meinel
Cleanup the test cases (Andrew) |
989 |
self.assertDiffBlocks('abbabbbb', 'cabbabbc', []) |
990 |
self.assertDiffBlocks('bbbbbbbb', 'cbbbbbbc', []) |
|
|
1185.81.11
by John Arbash Meinel
Found some edge cases that weren't being matched. |
991 |
|
|
3074.2.1
by John Arbash Meinel
Change the C PatienceDiff implementation to support arbitrary objects. |
992 |
def test_matching_blocks_tuples(self): |
993 |
# Some basic matching tests
|
|
|
3074.2.10
by John Arbash Meinel
Cleanup the test cases (Andrew) |
994 |
self.assertDiffBlocks([], [], []) |
995 |
self.assertDiffBlocks([('a',), ('b',), ('c,')], [], []) |
|
996 |
self.assertDiffBlocks([], [('a',), ('b',), ('c,')], []) |
|
997 |
self.assertDiffBlocks([('a',), ('b',), ('c,')], |
|
998 |
[('a',), ('b',), ('c,')], |
|
999 |
[(0, 0, 3)]) |
|
1000 |
self.assertDiffBlocks([('a',), ('b',), ('c,')], |
|
1001 |
[('a',), ('b',), ('d,')], |
|
1002 |
[(0, 0, 2)]) |
|
1003 |
self.assertDiffBlocks([('d',), ('b',), ('c,')], |
|
1004 |
[('a',), ('b',), ('c,')], |
|
1005 |
[(1, 1, 2)]) |
|
1006 |
self.assertDiffBlocks([('d',), ('a',), ('b',), ('c,')], |
|
1007 |
[('a',), ('b',), ('c,')], |
|
1008 |
[(1, 0, 3)]) |
|
1009 |
self.assertDiffBlocks([('a', 'b'), ('c', 'd'), ('e', 'f')], |
|
1010 |
[('a', 'b'), ('c', 'X'), ('e', 'f')], |
|
1011 |
[(0, 0, 1), (2, 2, 1)]) |
|
1012 |
self.assertDiffBlocks([('a', 'b'), ('c', 'd'), ('e', 'f')], |
|
1013 |
[('a', 'b'), ('c', 'dX'), ('e', 'f')], |
|
1014 |
[(0, 0, 1), (2, 2, 1)]) |
|
|
3074.2.1
by John Arbash Meinel
Change the C PatienceDiff implementation to support arbitrary objects. |
1015 |
|
|
1185.81.3
by John Arbash Meinel
Adding tests for checking opcodes. |
1016 |
def test_opcodes(self): |
|
1711.2.10
by John Arbash Meinel
Clarify the patience tests a little bit. |
1017 |
def chk_ops(a, b, expected_codes): |
|
2781.1.1
by Martin Pool
merge cpatiencediff from Lukas |
1018 |
s = self._PatienceSequenceMatcher(None, a, b) |
|
6614.1.3
by Vincent Ladeuil
Fix assertEquals being deprecated by using assertEqual. |
1019 |
self.assertEqual(expected_codes, s.get_opcodes()) |
|
1185.81.3
by John Arbash Meinel
Adding tests for checking opcodes. |
1020 |
|
1021 |
chk_ops('', '', []) |
|
1022 |
chk_ops([], [], []) |
|
|
6809.1.1
by Martin
Apply 2to3 ws_comma fixer |
1023 |
chk_ops('abc', '', [('delete', 0, 3, 0, 0)]) |
1024 |
chk_ops('', 'abc', [('insert', 0, 0, 0, 3)]) |
|
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
1025 |
chk_ops('abcd', 'abcd', [('equal', 0, 4, 0, 4)]) |
1026 |
chk_ops('abcd', 'abce', [('equal', 0, 3, 0, 3), |
|
|
6809.1.1
by Martin
Apply 2to3 ws_comma fixer |
1027 |
('replace', 3, 4, 3, 4) |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
1028 |
])
|
|
6809.1.1
by Martin
Apply 2to3 ws_comma fixer |
1029 |
chk_ops('eabc', 'abce', [('delete', 0, 1, 0, 0), |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
1030 |
('equal', 1, 4, 0, 3), |
|
6809.1.1
by Martin
Apply 2to3 ws_comma fixer |
1031 |
('insert', 4, 4, 3, 4) |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
1032 |
])
|
|
6809.1.1
by Martin
Apply 2to3 ws_comma fixer |
1033 |
chk_ops('eabce', 'abce', [('delete', 0, 1, 0, 0), |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
1034 |
('equal', 1, 5, 0, 4) |
1035 |
])
|
|
1036 |
chk_ops('abcde', 'abXde', [('equal', 0, 2, 0, 2), |
|
|
6809.1.1
by Martin
Apply 2to3 ws_comma fixer |
1037 |
('replace', 2, 3, 2, 3), |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
1038 |
('equal', 3, 5, 3, 5) |
1039 |
])
|
|
1040 |
chk_ops('abcde', 'abXYZde', [('equal', 0, 2, 0, 2), |
|
|
6809.1.1
by Martin
Apply 2to3 ws_comma fixer |
1041 |
('replace', 2, 3, 2, 5), |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
1042 |
('equal', 3, 5, 5, 7) |
1043 |
])
|
|
1044 |
chk_ops('abde', 'abXYZde', [('equal', 0, 2, 0, 2), |
|
1045 |
('insert', 2, 2, 2, 5), |
|
1046 |
('equal', 2, 4, 5, 7) |
|
|
1185.81.3
by John Arbash Meinel
Adding tests for checking opcodes. |
1047 |
])
|
1048 |
chk_ops('abcdefghijklmnop', 'abcdefxydefghijklmnop', |
|
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
1049 |
[('equal', 0, 6, 0, 6), |
1050 |
('insert', 6, 6, 6, 11), |
|
1051 |
('equal', 6, 16, 11, 21) |
|
1052 |
])
|
|
|
1185.81.3
by John Arbash Meinel
Adding tests for checking opcodes. |
1053 |
chk_ops( |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
1054 |
['hello there\n', 'world\n', 'how are you today?\n'], |
1055 |
['hello there\n', 'how are you today?\n'], |
|
1056 |
[('equal', 0, 1, 0, 1), |
|
1057 |
('delete', 1, 2, 1, 1), |
|
1058 |
('equal', 2, 3, 1, 2), |
|
1059 |
])
|
|
|
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
1060 |
chk_ops('aBccDe', 'abccde', |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
1061 |
[('equal', 0, 1, 0, 1), |
|
6809.1.1
by Martin
Apply 2to3 ws_comma fixer |
1062 |
('replace', 1, 5, 1, 5), |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
1063 |
('equal', 5, 6, 5, 6), |
1064 |
])
|
|
|
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
1065 |
chk_ops('aBcDec', 'abcdec', |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
1066 |
[('equal', 0, 1, 0, 1), |
|
6809.1.1
by Martin
Apply 2to3 ws_comma fixer |
1067 |
('replace', 1, 2, 1, 2), |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
1068 |
('equal', 2, 3, 2, 3), |
|
6809.1.1
by Martin
Apply 2to3 ws_comma fixer |
1069 |
('replace', 3, 4, 3, 4), |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
1070 |
('equal', 4, 6, 4, 6), |
1071 |
])
|
|
|
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
1072 |
chk_ops('aBcdEcdFg', 'abcdecdfg', |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
1073 |
[('equal', 0, 1, 0, 1), |
|
6809.1.1
by Martin
Apply 2to3 ws_comma fixer |
1074 |
('replace', 1, 8, 1, 8), |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
1075 |
('equal', 8, 9, 8, 9) |
1076 |
])
|
|
|
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
1077 |
chk_ops('aBcdEeXcdFg', 'abcdecdfg', |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
1078 |
[('equal', 0, 1, 0, 1), |
|
6809.1.1
by Martin
Apply 2to3 ws_comma fixer |
1079 |
('replace', 1, 2, 1, 2), |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
1080 |
('equal', 2, 4, 2, 4), |
|
6809.1.1
by Martin
Apply 2to3 ws_comma fixer |
1081 |
('delete', 4, 5, 4, 4), |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
1082 |
('equal', 5, 6, 4, 5), |
|
6809.1.1
by Martin
Apply 2to3 ws_comma fixer |
1083 |
('delete', 6, 7, 5, 5), |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
1084 |
('equal', 7, 9, 5, 7), |
|
6809.1.1
by Martin
Apply 2to3 ws_comma fixer |
1085 |
('replace', 9, 10, 7, 8), |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
1086 |
('equal', 10, 11, 8, 9) |
1087 |
])
|
|
|
1185.81.10
by John Arbash Meinel
Added some more test cases. |
1088 |
|
|
2781.1.1
by Martin Pool
merge cpatiencediff from Lukas |
1089 |
def test_grouped_opcodes(self): |
1090 |
def chk_ops(a, b, expected_codes, n=3): |
|
1091 |
s = self._PatienceSequenceMatcher(None, a, b) |
|
|
6614.1.3
by Vincent Ladeuil
Fix assertEquals being deprecated by using assertEqual. |
1092 |
self.assertEqual(expected_codes, list(s.get_grouped_opcodes(n))) |
|
2781.1.1
by Martin Pool
merge cpatiencediff from Lukas |
1093 |
|
1094 |
chk_ops('', '', []) |
|
1095 |
chk_ops([], [], []) |
|
|
6809.1.1
by Martin
Apply 2to3 ws_comma fixer |
1096 |
chk_ops('abc', '', [[('delete', 0, 3, 0, 0)]]) |
1097 |
chk_ops('', 'abc', [[('insert', 0, 0, 0, 3)]]) |
|
|
2781.1.1
by Martin Pool
merge cpatiencediff from Lukas |
1098 |
chk_ops('abcd', 'abcd', []) |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
1099 |
chk_ops('abcd', 'abce', [[('equal', 0, 3, 0, 3), |
|
6809.1.1
by Martin
Apply 2to3 ws_comma fixer |
1100 |
('replace', 3, 4, 3, 4) |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
1101 |
]])
|
|
6809.1.1
by Martin
Apply 2to3 ws_comma fixer |
1102 |
chk_ops('eabc', 'abce', [[('delete', 0, 1, 0, 0), |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
1103 |
('equal', 1, 4, 0, 3), |
1104 |
('insert', 4, 4, 3, 4) |
|
1105 |
]])
|
|
|
2781.1.1
by Martin Pool
merge cpatiencediff from Lukas |
1106 |
chk_ops('abcdefghijklmnop', 'abcdefxydefghijklmnop', |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
1107 |
[[('equal', 3, 6, 3, 6), |
|
6809.1.1
by Martin
Apply 2to3 ws_comma fixer |
1108 |
('insert', 6, 6, 6, 11), |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
1109 |
('equal', 6, 9, 11, 14) |
|
2781.1.1
by Martin Pool
merge cpatiencediff from Lukas |
1110 |
]])
|
1111 |
chk_ops('abcdefghijklmnop', 'abcdefxydefghijklmnop', |
|
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
1112 |
[[('equal', 2, 6, 2, 6), |
|
6809.1.1
by Martin
Apply 2to3 ws_comma fixer |
1113 |
('insert', 6, 6, 6, 11), |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
1114 |
('equal', 6, 10, 11, 15) |
|
2781.1.1
by Martin Pool
merge cpatiencediff from Lukas |
1115 |
]], 4) |
1116 |
chk_ops('Xabcdef', 'abcdef', |
|
|
6809.1.1
by Martin
Apply 2to3 ws_comma fixer |
1117 |
[[('delete', 0, 1, 0, 0), |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
1118 |
('equal', 1, 4, 0, 3) |
|
2781.1.1
by Martin Pool
merge cpatiencediff from Lukas |
1119 |
]])
|
1120 |
chk_ops('abcdef', 'abcdefX', |
|
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
1121 |
[[('equal', 3, 6, 3, 6), |
|
6809.1.1
by Martin
Apply 2to3 ws_comma fixer |
1122 |
('insert', 6, 6, 6, 7) |
|
2781.1.1
by Martin Pool
merge cpatiencediff from Lukas |
1123 |
]])
|
1124 |
||
|
1185.81.16
by John Arbash Meinel
Added tests, and an assert check to make sure ranges are always increasing. |
1125 |
def test_multiple_ranges(self): |
1126 |
# There was an earlier bug where we used a bad set of ranges,
|
|
1127 |
# this triggers that specific bug, to make sure it doesn't regress
|
|
|
3074.2.10
by John Arbash Meinel
Cleanup the test cases (Andrew) |
1128 |
self.assertDiffBlocks('abcdefghijklmnop', |
1129 |
'abcXghiYZQRSTUVWXYZijklmnop', |
|
1130 |
[(0, 0, 3), (6, 4, 3), (9, 20, 7)]) |
|
1131 |
||
1132 |
self.assertDiffBlocks('ABCd efghIjk L', |
|
1133 |
'AxyzBCn mo pqrstuvwI1 2 L', |
|
|
6809.1.1
by Martin
Apply 2to3 ws_comma fixer |
1134 |
[(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. |
1135 |
|
|
1711.2.8
by John Arbash Meinel
rot13 the code snippet to help with clarity. |
1136 |
# These are rot13 code snippets.
|
|
3074.2.10
by John Arbash Meinel
Cleanup the test cases (Andrew) |
1137 |
self.assertDiffBlocks('''\ |
|
1711.2.8
by John Arbash Meinel
rot13 the code snippet to help with clarity. |
1138 |
trg nqqrq jura lbh nqq n svyr va gur qverpgbel.
|
1139 |
"""
|
|
1140 |
gnxrf_netf = ['svyr*']
|
|
1141 |
gnxrf_bcgvbaf = ['ab-erphefr']
|
|
|
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
1142 |
|
|
1711.2.8
by John Arbash Meinel
rot13 the code snippet to help with clarity. |
1143 |
qrs eha(frys, svyr_yvfg, ab_erphefr=Snyfr):
|
1144 |
sebz omeyvo.nqq vzcbeg fzneg_nqq, nqq_ercbegre_cevag, nqq_ercbegre_ahyy
|
|
1145 |
vs vf_dhvrg():
|
|
1146 |
ercbegre = nqq_ercbegre_ahyy
|
|
1147 |
ryfr:
|
|
1148 |
ercbegre = nqq_ercbegre_cevag
|
|
1149 |
fzneg_nqq(svyr_yvfg, abg ab_erphefr, ercbegre)
|
|
1150 |
||
1151 |
||
1152 |
pynff pzq_zxqve(Pbzznaq):
|
|
1153 |
'''.splitlines(True), '''\ |
|
1154 |
trg nqqrq jura lbh nqq n svyr va gur qverpgbel.
|
|
1155 |
||
|
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
1156 |
--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. |
1157 |
nqq gurz.
|
1158 |
"""
|
|
1159 |
gnxrf_netf = ['svyr*']
|
|
1160 |
gnxrf_bcgvbaf = ['ab-erphefr', 'qel-eha']
|
|
1161 |
||
1162 |
qrs eha(frys, svyr_yvfg, ab_erphefr=Snyfr, qel_eha=Snyfr):
|
|
1163 |
vzcbeg omeyvo.nqq
|
|
1164 |
||
1165 |
vs qel_eha:
|
|
1166 |
vs vf_dhvrg():
|
|
1167 |
# Guvf vf cbvagyrff, ohg V'q engure abg envfr na reebe
|
|
1168 |
npgvba = omeyvo.nqq.nqq_npgvba_ahyy
|
|
1169 |
ryfr:
|
|
1170 |
npgvba = omeyvo.nqq.nqq_npgvba_cevag
|
|
1171 |
ryvs vf_dhvrg():
|
|
1172 |
npgvba = omeyvo.nqq.nqq_npgvba_nqq
|
|
1173 |
ryfr:
|
|
1174 |
npgvba = omeyvo.nqq.nqq_npgvba_nqq_naq_cevag
|
|
1175 |
||
1176 |
omeyvo.nqq.fzneg_nqq(svyr_yvfg, abg ab_erphefr, npgvba)
|
|
1177 |
||
1178 |
||
1179 |
pynff pzq_zxqve(Pbzznaq):
|
|
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
1180 |
'''.splitlines(True), [(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. |
1181 |
|
|
1711.2.9
by John Arbash Meinel
Rename cdv => patience |
1182 |
def test_patience_unified_diff(self): |
|
1185.81.29
by Aaron Bentley
Fix style issues and duplicated tests |
1183 |
txt_a = ['hello there\n', |
1184 |
'world\n', |
|
1185 |
'how are you today?\n'] |
|
1186 |
txt_b = ['hello there\n', |
|
1187 |
'how are you today?\n'] |
|
|
5168.1.3
by Vincent Ladeuil
Even more import fixes. |
1188 |
unified_diff = patiencediff.unified_diff |
|
2781.1.1
by Martin Pool
merge cpatiencediff from Lukas |
1189 |
psm = self._PatienceSequenceMatcher |
|
6614.1.3
by Vincent Ladeuil
Fix assertEquals being deprecated by using assertEqual. |
1190 |
self.assertEqual(['--- \n', |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
1191 |
'+++ \n', |
1192 |
'@@ -1,3 +1,2 @@\n', |
|
1193 |
' hello there\n', |
|
1194 |
'-world\n', |
|
1195 |
' how are you today?\n' |
|
1196 |
], list(unified_diff(txt_a, txt_b, |
|
1197 |
sequencematcher=psm))) |
|
1198 |
txt_a = [x + '\n' for x in 'abcdefghijklmnop'] |
|
1199 |
txt_b = [x + '\n' for x in 'abcdefxydefghijklmnop'] |
|
|
1185.81.14
by John Arbash Meinel
Added a main function for running cdvdifflib manually, included tests for unified_diff interfaces |
1200 |
# This is the result with LongestCommonSubstring matching
|
|
6614.1.3
by Vincent Ladeuil
Fix assertEquals being deprecated by using assertEqual. |
1201 |
self.assertEqual(['--- \n', |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
1202 |
'+++ \n', |
1203 |
'@@ -1,6 +1,11 @@\n', |
|
1204 |
' a\n', |
|
1205 |
' b\n', |
|
1206 |
' c\n', |
|
1207 |
'+d\n', |
|
1208 |
'+e\n', |
|
1209 |
'+f\n', |
|
1210 |
'+x\n', |
|
1211 |
'+y\n', |
|
1212 |
' d\n', |
|
1213 |
' e\n', |
|
1214 |
' f\n'], list(unified_diff(txt_a, txt_b))) |
|
|
1711.2.9
by John Arbash Meinel
Rename cdv => patience |
1215 |
# And the patience diff
|
|
6614.1.3
by Vincent Ladeuil
Fix assertEquals being deprecated by using assertEqual. |
1216 |
self.assertEqual(['--- \n', |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
1217 |
'+++ \n', |
1218 |
'@@ -4,6 +4,11 @@\n', |
|
1219 |
' d\n', |
|
1220 |
' e\n', |
|
1221 |
' f\n', |
|
1222 |
'+x\n', |
|
1223 |
'+y\n', |
|
1224 |
'+d\n', |
|
1225 |
'+e\n', |
|
1226 |
'+f\n', |
|
1227 |
' g\n', |
|
1228 |
' h\n', |
|
1229 |
' i\n', |
|
1230 |
], list(unified_diff(txt_a, txt_b, |
|
1231 |
sequencematcher=psm))) |
|
|
1185.81.25
by Aaron Bentley
Clean up test_diff |
1232 |
|
|
3922.1.2
by John Arbash Meinel
Update the test cases for the new patience diff code. |
1233 |
def test_patience_unified_diff_with_dates(self): |
1234 |
txt_a = ['hello there\n', |
|
1235 |
'world\n', |
|
1236 |
'how are you today?\n'] |
|
1237 |
txt_b = ['hello there\n', |
|
1238 |
'how are you today?\n'] |
|
|
5168.1.3
by Vincent Ladeuil
Even more import fixes. |
1239 |
unified_diff = patiencediff.unified_diff |
|
3922.1.2
by John Arbash Meinel
Update the test cases for the new patience diff code. |
1240 |
psm = self._PatienceSequenceMatcher |
|
6614.1.3
by Vincent Ladeuil
Fix assertEquals being deprecated by using assertEqual. |
1241 |
self.assertEqual(['--- a\t2008-08-08\n', |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
1242 |
'+++ b\t2008-09-09\n', |
1243 |
'@@ -1,3 +1,2 @@\n', |
|
1244 |
' hello there\n', |
|
1245 |
'-world\n', |
|
1246 |
' how are you today?\n' |
|
1247 |
], list(unified_diff(txt_a, txt_b, |
|
1248 |
fromfile='a', tofile='b', |
|
1249 |
fromfiledate='2008-08-08', |
|
1250 |
tofiledate='2008-09-09', |
|
1251 |
sequencematcher=psm))) |
|
|
3922.1.2
by John Arbash Meinel
Update the test cases for the new patience diff code. |
1252 |
|
|
1185.81.14
by John Arbash Meinel
Added a main function for running cdvdifflib manually, included tests for unified_diff interfaces |
1253 |
|
|
2781.1.1
by Martin Pool
merge cpatiencediff from Lukas |
1254 |
class TestPatienceDiffLib_c(TestPatienceDiffLib): |
1255 |
||
|
5967.12.1
by Martin Pool
Move all test features into bzrlib.tests.features |
1256 |
_test_needs_features = [features.compiled_patiencediff_feature] |
|
2781.1.1
by Martin Pool
merge cpatiencediff from Lukas |
1257 |
|
1258 |
def setUp(self): |
|
1259 |
super(TestPatienceDiffLib_c, self).setUp() |
|
|
6622.1.34
by Jelmer Vernooij
Rename brzlib => breezy. |
1260 |
from breezy import _patiencediff_c |
|
5168.1.3
by Vincent Ladeuil
Even more import fixes. |
1261 |
self._unique_lcs = _patiencediff_c.unique_lcs_c |
1262 |
self._recurse_matches = _patiencediff_c.recurse_matches_c |
|
|
2781.1.1
by Martin Pool
merge cpatiencediff from Lukas |
1263 |
self._PatienceSequenceMatcher = \ |
|
5168.1.3
by Vincent Ladeuil
Even more import fixes. |
1264 |
_patiencediff_c.PatienceSequenceMatcher_c |
|
2781.1.1
by Martin Pool
merge cpatiencediff from Lukas |
1265 |
|
|
3074.2.3
by John Arbash Meinel
Enable some error checking, and small amount of code cleanup. |
1266 |
def test_unhashable(self): |
1267 |
"""We should get a proper exception here.""" |
|
|
3074.2.10
by John Arbash Meinel
Cleanup the test cases (Andrew) |
1268 |
# We need to be able to hash items in the sequence, lists are
|
1269 |
# unhashable, and thus cannot be diffed
|
|
|
3074.2.3
by John Arbash Meinel
Enable some error checking, and small amount of code cleanup. |
1270 |
e = self.assertRaises(TypeError, self._PatienceSequenceMatcher, |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
1271 |
None, [[]], []) |
1272 |
e = self.assertRaises(TypeError, self._PatienceSequenceMatcher, |
|
1273 |
None, ['valid', []], []) |
|
1274 |
e = self.assertRaises(TypeError, self._PatienceSequenceMatcher, |
|
1275 |
None, ['valid'], [[]]) |
|
1276 |
e = self.assertRaises(TypeError, self._PatienceSequenceMatcher, |
|
1277 |
None, ['valid'], ['valid', []]) |
|
|
3074.2.3
by John Arbash Meinel
Enable some error checking, and small amount of code cleanup. |
1278 |
|
|
2781.1.1
by Martin Pool
merge cpatiencediff from Lukas |
1279 |
|
|
5168.1.2
by Vincent Ladeuil
Ckeanup some more imports. |
1280 |
class TestPatienceDiffLibFiles(tests.TestCaseInTempDir): |
|
1185.81.14
by John Arbash Meinel
Added a main function for running cdvdifflib manually, included tests for unified_diff interfaces |
1281 |
|
|
2781.1.1
by Martin Pool
merge cpatiencediff from Lukas |
1282 |
def setUp(self): |
1283 |
super(TestPatienceDiffLibFiles, self).setUp() |
|
1284 |
self._PatienceSequenceMatcher = \ |
|
|
5168.1.3
by Vincent Ladeuil
Even more import fixes. |
1285 |
_patiencediff_py.PatienceSequenceMatcher_py |
|
2781.1.1
by Martin Pool
merge cpatiencediff from Lukas |
1286 |
|
|
1711.2.9
by John Arbash Meinel
Rename cdv => patience |
1287 |
def test_patience_unified_diff_files(self): |
|
6973.12.3
by Jelmer Vernooij
Fixes. |
1288 |
txt_a = [b'hello there\n', |
1289 |
b'world\n', |
|
1290 |
b'how are you today?\n'] |
|
1291 |
txt_b = [b'hello there\n', |
|
1292 |
b'how are you today?\n'] |
|
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
1293 |
with open('a1', 'wb') as f: |
1294 |
f.writelines(txt_a) |
|
1295 |
with open('b1', 'wb') as f: |
|
1296 |
f.writelines(txt_b) |
|
|
1185.81.14
by John Arbash Meinel
Added a main function for running cdvdifflib manually, included tests for unified_diff interfaces |
1297 |
|
|
5168.1.3
by Vincent Ladeuil
Even more import fixes. |
1298 |
unified_diff_files = patiencediff.unified_diff_files |
|
2781.1.1
by Martin Pool
merge cpatiencediff from Lukas |
1299 |
psm = self._PatienceSequenceMatcher |
|
7031.1.1
by Jelmer Vernooij
Fix breezy.tests.test_diff. |
1300 |
self.assertEqual([b'--- a1\n', |
1301 |
b'+++ b1\n', |
|
1302 |
b'@@ -1,3 +1,2 @@\n', |
|
1303 |
b' hello there\n', |
|
1304 |
b'-world\n', |
|
1305 |
b' how are you today?\n', |
|
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
1306 |
], list(unified_diff_files(b'a1', b'b1', |
1307 |
sequencematcher=psm))) |
|
|
1185.81.14
by John Arbash Meinel
Added a main function for running cdvdifflib manually, included tests for unified_diff interfaces |
1308 |
|
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
1309 |
txt_a = [x + '\n' for x in 'abcdefghijklmnop'] |
1310 |
txt_b = [x + '\n' for x in 'abcdefxydefghijklmnop'] |
|
1311 |
with open('a2', 'wt') as f: |
|
1312 |
f.writelines(txt_a) |
|
1313 |
with open('b2', 'wt') as f: |
|
1314 |
f.writelines(txt_b) |
|
|
1185.81.14
by John Arbash Meinel
Added a main function for running cdvdifflib manually, included tests for unified_diff interfaces |
1315 |
|
1316 |
# This is the result with LongestCommonSubstring matching
|
|
|
7031.1.1
by Jelmer Vernooij
Fix breezy.tests.test_diff. |
1317 |
self.assertEqual([b'--- a2\n', |
1318 |
b'+++ b2\n', |
|
1319 |
b'@@ -1,6 +1,11 @@\n', |
|
1320 |
b' a\n', |
|
1321 |
b' b\n', |
|
1322 |
b' c\n', |
|
1323 |
b'+d\n', |
|
1324 |
b'+e\n', |
|
1325 |
b'+f\n', |
|
1326 |
b'+x\n', |
|
1327 |
b'+y\n', |
|
1328 |
b' d\n', |
|
1329 |
b' e\n', |
|
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
1330 |
b' f\n'], list(unified_diff_files(b'a2', b'b2'))) |
|
1185.81.14
by John Arbash Meinel
Added a main function for running cdvdifflib manually, included tests for unified_diff interfaces |
1331 |
|
|
1711.2.9
by John Arbash Meinel
Rename cdv => patience |
1332 |
# And the patience diff
|
|
7031.1.1
by Jelmer Vernooij
Fix breezy.tests.test_diff. |
1333 |
self.assertEqual([b'--- a2\n', |
1334 |
b'+++ b2\n', |
|
1335 |
b'@@ -4,6 +4,11 @@\n', |
|
1336 |
b' d\n', |
|
1337 |
b' e\n', |
|
1338 |
b' f\n', |
|
1339 |
b'+x\n', |
|
1340 |
b'+y\n', |
|
1341 |
b'+d\n', |
|
1342 |
b'+e\n', |
|
1343 |
b'+f\n', |
|
1344 |
b' g\n', |
|
1345 |
b' h\n', |
|
1346 |
b' i\n'], |
|
1347 |
list(unified_diff_files(b'a2', b'b2', |
|
|
6614.1.3
by Vincent Ladeuil
Fix assertEquals being deprecated by using assertEqual. |
1348 |
sequencematcher=psm))) |
|
2781.1.1
by Martin Pool
merge cpatiencediff from Lukas |
1349 |
|
1350 |
||
1351 |
class TestPatienceDiffLibFiles_c(TestPatienceDiffLibFiles): |
|
1352 |
||
|
5967.12.1
by Martin Pool
Move all test features into bzrlib.tests.features |
1353 |
_test_needs_features = [features.compiled_patiencediff_feature] |
|
2781.1.1
by Martin Pool
merge cpatiencediff from Lukas |
1354 |
|
1355 |
def setUp(self): |
|
1356 |
super(TestPatienceDiffLibFiles_c, self).setUp() |
|
|
6622.1.34
by Jelmer Vernooij
Rename brzlib => breezy. |
1357 |
from breezy import _patiencediff_c |
|
2781.1.1
by Martin Pool
merge cpatiencediff from Lukas |
1358 |
self._PatienceSequenceMatcher = \ |
|
5168.1.3
by Vincent Ladeuil
Even more import fixes. |
1359 |
_patiencediff_c.PatienceSequenceMatcher_c |
|
2781.1.1
by Martin Pool
merge cpatiencediff from Lukas |
1360 |
|
1361 |
||
|
5168.1.2
by Vincent Ladeuil
Ckeanup some more imports. |
1362 |
class TestUsingCompiledIfAvailable(tests.TestCase): |
|
2781.1.1
by Martin Pool
merge cpatiencediff from Lukas |
1363 |
|
1364 |
def test_PatienceSequenceMatcher(self): |
|
|
5967.12.1
by Martin Pool
Move all test features into bzrlib.tests.features |
1365 |
if features.compiled_patiencediff_feature.available(): |
|
6622.1.34
by Jelmer Vernooij
Rename brzlib => breezy. |
1366 |
from breezy._patiencediff_c import PatienceSequenceMatcher_c |
|
2781.1.1
by Martin Pool
merge cpatiencediff from Lukas |
1367 |
self.assertIs(PatienceSequenceMatcher_c, |
|
5168.1.3
by Vincent Ladeuil
Even more import fixes. |
1368 |
patiencediff.PatienceSequenceMatcher) |
|
2781.1.1
by Martin Pool
merge cpatiencediff from Lukas |
1369 |
else: |
|
6622.1.34
by Jelmer Vernooij
Rename brzlib => breezy. |
1370 |
from breezy._patiencediff_py import PatienceSequenceMatcher_py |
|
2781.1.1
by Martin Pool
merge cpatiencediff from Lukas |
1371 |
self.assertIs(PatienceSequenceMatcher_py, |
|
5168.1.3
by Vincent Ladeuil
Even more import fixes. |
1372 |
patiencediff.PatienceSequenceMatcher) |
|
2781.1.1
by Martin Pool
merge cpatiencediff from Lukas |
1373 |
|
1374 |
def test_unique_lcs(self): |
|
|
5967.12.1
by Martin Pool
Move all test features into bzrlib.tests.features |
1375 |
if features.compiled_patiencediff_feature.available(): |
|
6622.1.34
by Jelmer Vernooij
Rename brzlib => breezy. |
1376 |
from breezy._patiencediff_c import unique_lcs_c |
|
2781.1.1
by Martin Pool
merge cpatiencediff from Lukas |
1377 |
self.assertIs(unique_lcs_c, |
|
5168.1.3
by Vincent Ladeuil
Even more import fixes. |
1378 |
patiencediff.unique_lcs) |
|
2781.1.1
by Martin Pool
merge cpatiencediff from Lukas |
1379 |
else: |
|
6622.1.34
by Jelmer Vernooij
Rename brzlib => breezy. |
1380 |
from breezy._patiencediff_py import unique_lcs_py |
|
2781.1.1
by Martin Pool
merge cpatiencediff from Lukas |
1381 |
self.assertIs(unique_lcs_py, |
|
5168.1.3
by Vincent Ladeuil
Even more import fixes. |
1382 |
patiencediff.unique_lcs) |
|
2781.1.1
by Martin Pool
merge cpatiencediff from Lukas |
1383 |
|
1384 |
def test_recurse_matches(self): |
|
|
5967.12.1
by Martin Pool
Move all test features into bzrlib.tests.features |
1385 |
if features.compiled_patiencediff_feature.available(): |
|
6622.1.34
by Jelmer Vernooij
Rename brzlib => breezy. |
1386 |
from breezy._patiencediff_c import recurse_matches_c |
|
2781.1.1
by Martin Pool
merge cpatiencediff from Lukas |
1387 |
self.assertIs(recurse_matches_c, |
|
5168.1.3
by Vincent Ladeuil
Even more import fixes. |
1388 |
patiencediff.recurse_matches) |
|
2781.1.1
by Martin Pool
merge cpatiencediff from Lukas |
1389 |
else: |
|
6622.1.34
by Jelmer Vernooij
Rename brzlib => breezy. |
1390 |
from breezy._patiencediff_py import recurse_matches_py |
|
2781.1.1
by Martin Pool
merge cpatiencediff from Lukas |
1391 |
self.assertIs(recurse_matches_py, |
|
5168.1.3
by Vincent Ladeuil
Even more import fixes. |
1392 |
patiencediff.recurse_matches) |
|
3123.6.2
by Aaron Bentley
Implement diff --using natively |
1393 |
|
1394 |
||
|
5168.1.2
by Vincent Ladeuil
Ckeanup some more imports. |
1395 |
class TestDiffFromTool(tests.TestCaseWithTransport): |
|
3123.6.2
by Aaron Bentley
Implement diff --using natively |
1396 |
|
1397 |
def test_from_string(self): |
|
|
5168.1.4
by Vincent Ladeuil
Final import fixes for bt.test_diff. |
1398 |
diff_obj = diff.DiffFromTool.from_string('diff', None, None, None) |
|
3123.6.2
by Aaron Bentley
Implement diff --using natively |
1399 |
self.addCleanup(diff_obj.finish) |
|
4603.1.20
by Aaron Bentley
Use string.Template substitution with @ as delimiter. |
1400 |
self.assertEqual(['diff', '@old_path', '@new_path'], |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
1401 |
diff_obj.command_template) |
|
3199.1.6
by Vincent Ladeuil
Fiz last leaking tmp dir. |
1402 |
|
1403 |
def test_from_string_u5(self): |
|
|
5168.1.4
by Vincent Ladeuil
Final import fixes for bt.test_diff. |
1404 |
diff_obj = diff.DiffFromTool.from_string('diff "-u 5"', |
1405 |
None, None, None) |
|
|
3199.1.6
by Vincent Ladeuil
Fiz last leaking tmp dir. |
1406 |
self.addCleanup(diff_obj.finish) |
|
4603.1.20
by Aaron Bentley
Use string.Template substitution with @ as delimiter. |
1407 |
self.assertEqual(['diff', '-u 5', '@old_path', '@new_path'], |
|
3123.6.2
by Aaron Bentley
Implement diff --using natively |
1408 |
diff_obj.command_template) |
1409 |
self.assertEqual(['diff', '-u 5', 'old-path', 'new-path'], |
|
1410 |
diff_obj._get_command('old-path', 'new-path')) |
|
|
5168.1.2
by Vincent Ladeuil
Ckeanup some more imports. |
1411 |
|
|
4913.5.8
by Gordon Tyler
Added test_from_string_path_with_backslashes, which tests the actual scenario in bug 392428. |
1412 |
def test_from_string_path_with_backslashes(self): |
|
5241.2.2
by Robert Collins
Missed one test. |
1413 |
self.requireFeature(features.backslashdir_feature) |
|
4913.5.8
by Gordon Tyler
Added test_from_string_path_with_backslashes, which tests the actual scenario in bug 392428. |
1414 |
tool = 'C:\\Tools\\Diff.exe' |
|
5168.1.4
by Vincent Ladeuil
Final import fixes for bt.test_diff. |
1415 |
diff_obj = diff.DiffFromTool.from_string(tool, None, None, None) |
|
4913.5.8
by Gordon Tyler
Added test_from_string_path_with_backslashes, which tests the actual scenario in bug 392428. |
1416 |
self.addCleanup(diff_obj.finish) |
1417 |
self.assertEqual(['C:\\Tools\\Diff.exe', '@old_path', '@new_path'], |
|
1418 |
diff_obj.command_template) |
|
1419 |
self.assertEqual(['C:\\Tools\\Diff.exe', 'old-path', 'new-path'], |
|
1420 |
diff_obj._get_command('old-path', 'new-path')) |
|
|
3123.6.2
by Aaron Bentley
Implement diff --using natively |
1421 |
|
1422 |
def test_execute(self): |
|
|
6621.22.2
by Martin
Use BytesIO or StringIO from bzrlib.sixish |
1423 |
output = BytesIO() |
|
7141.1.1
by Jelmer Vernooij
Use sys.executable rather than python for ad-hoc tests. |
1424 |
diff_obj = diff.DiffFromTool([sys.executable, '-c', |
|
7019.1.1
by Jelmer Vernooij
Fix breezy.tests.test_diff.TestDiffFromTool.test_execute when default python is python3. |
1425 |
'print("@old_path @new_path")'], |
|
5168.1.4
by Vincent Ladeuil
Final import fixes for bt.test_diff. |
1426 |
None, None, output) |
|
3123.6.2
by Aaron Bentley
Implement diff --using natively |
1427 |
self.addCleanup(diff_obj.finish) |
1428 |
diff_obj._execute('old', 'new') |
|
|
6973.12.3
by Jelmer Vernooij
Fixes. |
1429 |
self.assertEqual(output.getvalue().rstrip(), b'old new') |
|
3123.6.2
by Aaron Bentley
Implement diff --using natively |
1430 |
|
|
6597.2.1
by Richard Wilbur
Split diff format option parser into a separate function, update to include all format options for GNU diff v3.2, and test parser. |
1431 |
def test_execute_missing(self): |
|
5168.1.4
by Vincent Ladeuil
Final import fixes for bt.test_diff. |
1432 |
diff_obj = diff.DiffFromTool(['a-tool-which-is-unlikely-to-exist'], |
1433 |
None, None, None) |
|
|
3145.1.1
by Aaron Bentley
Handle missing tools gracefully in diff --using |
1434 |
self.addCleanup(diff_obj.finish) |
|
5168.1.2
by Vincent Ladeuil
Ckeanup some more imports. |
1435 |
e = self.assertRaises(errors.ExecutableMissing, diff_obj._execute, |
1436 |
'old', 'new') |
|
|
3145.1.1
by Aaron Bentley
Handle missing tools gracefully in diff --using |
1437 |
self.assertEqual('a-tool-which-is-unlikely-to-exist could not be found' |
1438 |
' on this machine', str(e)) |
|
1439 |
||
|
3287.18.22
by Matt McClure
Reverts to prior decomposition of exercise and verification, as suggested |
1440 |
def test_prepare_files_creates_paths_readable_by_windows_tool(self): |
|
5967.12.1
by Martin Pool
Move all test features into bzrlib.tests.features |
1441 |
self.requireFeature(features.AttribFeature) |
|
6621.22.2
by Martin
Use BytesIO or StringIO from bzrlib.sixish |
1442 |
output = BytesIO() |
|
3287.18.10
by Matt McClure
Uses TestSkipped for test_execute_windows_tool on non-Windows platforms. |
1443 |
tree = self.make_branch_and_tree('tree') |
|
6855.4.1
by Jelmer Vernooij
Yet more bees. |
1444 |
self.build_tree_contents([('tree/file', b'content')]) |
1445 |
tree.add('file', b'file-id') |
|
|
3287.18.11
by Matt McClure
Removed unnecessary timestamp parameter. |
1446 |
tree.commit('old tree') |
|
3287.18.10
by Matt McClure
Uses TestSkipped for test_execute_windows_tool on non-Windows platforms. |
1447 |
tree.lock_read() |
1448 |
self.addCleanup(tree.unlock) |
|
|
4873.3.1
by John Arbash Meinel
Now that we return files directly from the working tree |
1449 |
basis_tree = tree.basis_tree() |
1450 |
basis_tree.lock_read() |
|
1451 |
self.addCleanup(basis_tree.unlock) |
|
|
7141.1.1
by Jelmer Vernooij
Use sys.executable rather than python for ad-hoc tests. |
1452 |
diff_obj = diff.DiffFromTool([sys.executable, '-c', |
|
5168.1.4
by Vincent Ladeuil
Final import fixes for bt.test_diff. |
1453 |
'print "@old_path @new_path"'], |
1454 |
basis_tree, tree, output) |
|
|
6855.4.1
by Jelmer Vernooij
Yet more bees. |
1455 |
diff_obj._prepare_files('file', 'file', file_id=b'file-id') |
|
4873.3.1
by John Arbash Meinel
Now that we return files directly from the working tree |
1456 |
# The old content should be readonly
|
1457 |
self.assertReadableByAttrib(diff_obj._root, 'old\\file', |
|
1458 |
r'R.*old\\file$') |
|
1459 |
# The new content should use the tree object, not a 'new' file anymore
|
|
1460 |
self.assertEndsWith(tree.basedir, 'work/tree') |
|
1461 |
self.assertReadableByAttrib(tree.basedir, 'file', r'work\\tree\\file$') |
|
|
3287.18.22
by Matt McClure
Reverts to prior decomposition of exercise and verification, as suggested |
1462 |
|
1463 |
def assertReadableByAttrib(self, cwd, relpath, regex): |
|
1464 |
proc = subprocess.Popen(['attrib', relpath], |
|
1465 |
stdout=subprocess.PIPE, |
|
1466 |
cwd=cwd) |
|
|
4873.3.1
by John Arbash Meinel
Now that we return files directly from the working tree |
1467 |
(result, err) = proc.communicate() |
1468 |
self.assertContainsRe(result.replace('\r\n', '\n'), regex) |
|
|
3287.18.9
by Matt McClure
Adds a test asserting that a Windows tool that understands forward slashes |
1469 |
|
|
3123.6.2
by Aaron Bentley
Implement diff --using natively |
1470 |
def test_prepare_files(self): |
|
6621.22.2
by Martin
Use BytesIO or StringIO from bzrlib.sixish |
1471 |
output = BytesIO() |
|
3123.6.2
by Aaron Bentley
Implement diff --using natively |
1472 |
tree = self.make_branch_and_tree('tree') |
|
6855.4.1
by Jelmer Vernooij
Yet more bees. |
1473 |
self.build_tree_contents([('tree/oldname', b'oldcontent')]) |
1474 |
self.build_tree_contents([('tree/oldname2', b'oldcontent2')]) |
|
1475 |
tree.add('oldname', b'file-id') |
|
1476 |
tree.add('oldname2', b'file2-id') |
|
|
5151.3.1
by Martin
Fix os.utime test failures, three on FAT filesystems and one with readonly files |
1477 |
# Earliest allowable date on FAT32 filesystems is 1980-01-01
|
1478 |
tree.commit('old tree', timestamp=315532800) |
|
|
3123.6.5
by Aaron Bentley
Symlink to real files if possible |
1479 |
tree.rename_one('oldname', 'newname') |
|
3287.18.23
by Matt McClure
Adds comments that document my understanding of |
1480 |
tree.rename_one('oldname2', 'newname2') |
|
6855.4.1
by Jelmer Vernooij
Yet more bees. |
1481 |
self.build_tree_contents([('tree/newname', b'newcontent')]) |
1482 |
self.build_tree_contents([('tree/newname2', b'newcontent2')]) |
|
|
3123.6.2
by Aaron Bentley
Implement diff --using natively |
1483 |
old_tree = tree.basis_tree() |
1484 |
old_tree.lock_read() |
|
1485 |
self.addCleanup(old_tree.unlock) |
|
|
3123.6.4
by Aaron Bentley
Set mtime (and atime) on files for --using |
1486 |
tree.lock_read() |
1487 |
self.addCleanup(tree.unlock) |
|
|
7141.1.1
by Jelmer Vernooij
Use sys.executable rather than python for ad-hoc tests. |
1488 |
diff_obj = diff.DiffFromTool([sys.executable, '-c', |
|
5168.1.4
by Vincent Ladeuil
Final import fixes for bt.test_diff. |
1489 |
'print "@old_path @new_path"'], |
1490 |
old_tree, tree, output) |
|
|
3123.6.2
by Aaron Bentley
Implement diff --using natively |
1491 |
self.addCleanup(diff_obj.finish) |
|
6681.2.10
by Jelmer Vernooij
Fix failures. |
1492 |
self.assertContainsRe(diff_obj._root, 'brz-diff-[^/]*') |
|
6809.4.15
by Jelmer Vernooij
Fix some more tests. |
1493 |
old_path, new_path = diff_obj._prepare_files( |
|
7206.6.2
by Jelmer Vernooij
Remove file_id from diff API. |
1494 |
'oldname', 'newname') |
|
3123.6.2
by Aaron Bentley
Implement diff --using natively |
1495 |
self.assertContainsRe(old_path, 'old/oldname$') |
|
5151.3.1
by Martin
Fix os.utime test failures, three on FAT filesystems and one with readonly files |
1496 |
self.assertEqual(315532800, os.stat(old_path).st_mtime) |
|
4845.2.1
by Gary van der Merwe
When launching an external diff app, don't write temporary files for a working tree. |
1497 |
self.assertContainsRe(new_path, 'tree/newname$') |
|
7029.4.2
by Jelmer Vernooij
Fix more merge tests. |
1498 |
self.assertFileEqual(b'oldcontent', old_path) |
1499 |
self.assertFileEqual(b'newcontent', new_path) |
|
|
3287.18.14
by Matt McClure
Extracted a host_os_dereferences_symlinks method. |
1500 |
if osutils.host_os_dereferences_symlinks(): |
|
3123.6.5
by Aaron Bentley
Symlink to real files if possible |
1501 |
self.assertTrue(os.path.samefile('tree/newname', new_path)) |
|
3123.6.2
by Aaron Bentley
Implement diff --using natively |
1502 |
# make sure we can create files with the same parent directories
|
|
7206.6.2
by Jelmer Vernooij
Remove file_id from diff API. |
1503 |
diff_obj._prepare_files('oldname2', 'newname2') |
|
4705.1.2
by Gary van der Merwe
Start on tests for get_trees_and_branches_to_diff. |
1504 |
|
1505 |
||
|
5074.5.4
by INADA Naoki
fix easy bug. |
1506 |
class TestDiffFromToolEncodedFilename(tests.TestCaseWithTransport): |
|
5074.5.2
by INADA Naoki
Add test for encoded filenames |
1507 |
|
1508 |
def test_encodable_filename(self): |
|
|
5074.5.9
by INADA Naoki
Make additional comments to clarify |
1509 |
# Just checks file path for external diff tool.
|
1510 |
# We cannot change CPython's internal encoding used by os.exec*.
|
|
1511 |
diffobj = diff.DiffFromTool(['dummy', '@old_path', '@new_path'], |
|
|
5074.5.7
by INADA Naoki
Test for filename encoding can't test subprocess execution because |
1512 |
None, None, None) |
|
5074.5.2
by INADA Naoki
Add test for encoded filenames |
1513 |
for _, scenario in EncodingAdapter.encoding_scenarios: |
1514 |
encoding = scenario['encoding'] |
|
|
6614.1.1
by Vincent Ladeuil
Fix assert_ being deprecated by using assertTrue. |
1515 |
dirname = scenario['info']['directory'] |
|
5074.5.2
by INADA Naoki
Add test for encoded filenames |
1516 |
filename = scenario['info']['filename'] |
|
5074.5.6
by INADA Naoki
Change directry name for each check. |
1517 |
|
|
5074.5.8
by INADA Naoki
Use tempfile when filepath in tree is not be able to encode with fsencoding. |
1518 |
self.overrideAttr(diffobj, '_fenc', lambda: encoding) |
1519 |
relpath = dirname + u'/' + filename |
|
1520 |
fullpath = diffobj._safe_filename('safe', relpath) |
|
|
6614.1.1
by Vincent Ladeuil
Fix assert_ being deprecated by using assertTrue. |
1521 |
self.assertEqual(fullpath, |
1522 |
fullpath.encode(encoding).decode(encoding)) |
|
1523 |
self.assertTrue(fullpath.startswith(diffobj._root + '/safe')) |
|
|
5074.5.2
by INADA Naoki
Add test for encoded filenames |
1524 |
|
1525 |
def test_unencodable_filename(self): |
|
|
5074.5.9
by INADA Naoki
Make additional comments to clarify |
1526 |
diffobj = diff.DiffFromTool(['dummy', '@old_path', '@new_path'], |
|
5074.5.7
by INADA Naoki
Test for filename encoding can't test subprocess execution because |
1527 |
None, None, None) |
|
5074.5.2
by INADA Naoki
Add test for encoded filenames |
1528 |
for _, scenario in EncodingAdapter.encoding_scenarios: |
1529 |
encoding = scenario['encoding'] |
|
|
6614.1.1
by Vincent Ladeuil
Fix assert_ being deprecated by using assertTrue. |
1530 |
dirname = scenario['info']['directory'] |
|
5074.5.2
by INADA Naoki
Add test for encoded filenames |
1531 |
filename = scenario['info']['filename'] |
1532 |
||
1533 |
if encoding == 'iso-8859-1': |
|
1534 |
encoding = 'iso-8859-2' |
|
1535 |
else: |
|
1536 |
encoding = 'iso-8859-1' |
|
|
5074.5.7
by INADA Naoki
Test for filename encoding can't test subprocess execution because |
1537 |
|
|
5074.5.8
by INADA Naoki
Use tempfile when filepath in tree is not be able to encode with fsencoding. |
1538 |
self.overrideAttr(diffobj, '_fenc', lambda: encoding) |
1539 |
relpath = dirname + u'/' + filename |
|
1540 |
fullpath = diffobj._safe_filename('safe', relpath) |
|
|
6614.1.1
by Vincent Ladeuil
Fix assert_ being deprecated by using assertTrue. |
1541 |
self.assertEqual(fullpath, |
1542 |
fullpath.encode(encoding).decode(encoding)) |
|
1543 |
self.assertTrue(fullpath.startswith(diffobj._root + '/safe')) |
|
|
5074.5.2
by INADA Naoki
Add test for encoded filenames |
1544 |
|
1545 |
||
|
5168.1.2
by Vincent Ladeuil
Ckeanup some more imports. |
1546 |
class TestGetTreesAndBranchesToDiffLocked(tests.TestCaseWithTransport): |
|
5147.3.7
by Andrew Bennetts
Expect DeprecationWarnings for get_trees_and_branches_to_diff in test_diff, and add corresponding test coverage for get_trees_and_branches_to_diff_locked. |
1547 |
|
1548 |
def call_gtabtd(self, path_list, revision_specs, old_url, new_url): |
|
|
6027.1.4
by Vincent Ladeuil
Remove ``diff.get_trees_and_branches_to_diff`` deprecated in 2.2.0 and the corrsponding tests. |
1549 |
"""Call get_trees_and_branches_to_diff_locked.""" |
|
5168.1.4
by Vincent Ladeuil
Final import fixes for bt.test_diff. |
1550 |
return diff.get_trees_and_branches_to_diff_locked( |
|
5147.3.7
by Andrew Bennetts
Expect DeprecationWarnings for get_trees_and_branches_to_diff in test_diff, and add corresponding test coverage for get_trees_and_branches_to_diff_locked. |
1551 |
path_list, revision_specs, old_url, new_url, self.addCleanup) |
|
4732.1.2
by Vincent Ladeuil
(trivial) Fix some PEP8 issues |
1552 |
|
|
4705.1.2
by Gary van der Merwe
Start on tests for get_trees_and_branches_to_diff. |
1553 |
def test_basic(self): |
1554 |
tree = self.make_branch_and_tree('tree') |
|
1555 |
(old_tree, new_tree, |
|
1556 |
old_branch, new_branch, |
|
|
5147.3.7
by Andrew Bennetts
Expect DeprecationWarnings for get_trees_and_branches_to_diff in test_diff, and add corresponding test coverage for get_trees_and_branches_to_diff_locked. |
1557 |
specific_files, extra_trees) = self.call_gtabtd( |
1558 |
['tree'], None, None, None) |
|
|
4732.1.2
by Vincent Ladeuil
(trivial) Fix some PEP8 issues |
1559 |
|
|
5168.1.2
by Vincent Ladeuil
Ckeanup some more imports. |
1560 |
self.assertIsInstance(old_tree, revisiontree.RevisionTree) |
1561 |
self.assertEqual(_mod_revision.NULL_REVISION, |
|
1562 |
old_tree.get_revision_id()) |
|
|
4705.1.2
by Gary van der Merwe
Start on tests for get_trees_and_branches_to_diff. |
1563 |
self.assertEqual(tree.basedir, new_tree.basedir) |
1564 |
self.assertEqual(tree.branch.base, old_branch.base) |
|
1565 |
self.assertEqual(tree.branch.base, new_branch.base) |
|
1566 |
self.assertIs(None, specific_files) |
|
1567 |
self.assertIs(None, extra_trees) |
|
1568 |
||
1569 |
def test_with_rev_specs(self): |
|
1570 |
tree = self.make_branch_and_tree('tree') |
|
|
6855.4.1
by Jelmer Vernooij
Yet more bees. |
1571 |
self.build_tree_contents([('tree/file', b'oldcontent')]) |
1572 |
tree.add('file', b'file-id') |
|
1573 |
tree.commit('old tree', timestamp=0, rev_id=b"old-id") |
|
1574 |
self.build_tree_contents([('tree/file', b'newcontent')]) |
|
|
6973.13.2
by Jelmer Vernooij
Fix some more tests. |
1575 |
tree.commit('new tree', timestamp=0, rev_id=b"new-id") |
|
4732.1.2
by Vincent Ladeuil
(trivial) Fix some PEP8 issues |
1576 |
|
|
5168.1.2
by Vincent Ladeuil
Ckeanup some more imports. |
1577 |
revisions = [revisionspec.RevisionSpec.from_string('1'), |
1578 |
revisionspec.RevisionSpec.from_string('2')] |
|
|
4705.1.2
by Gary van der Merwe
Start on tests for get_trees_and_branches_to_diff. |
1579 |
(old_tree, new_tree, |
1580 |
old_branch, new_branch, |
|
|
5147.3.7
by Andrew Bennetts
Expect DeprecationWarnings for get_trees_and_branches_to_diff in test_diff, and add corresponding test coverage for get_trees_and_branches_to_diff_locked. |
1581 |
specific_files, extra_trees) = self.call_gtabtd( |
1582 |
['tree'], revisions, None, None) |
|
|
4732.1.2
by Vincent Ladeuil
(trivial) Fix some PEP8 issues |
1583 |
|
|
5168.1.2
by Vincent Ladeuil
Ckeanup some more imports. |
1584 |
self.assertIsInstance(old_tree, revisiontree.RevisionTree) |
|
6973.13.2
by Jelmer Vernooij
Fix some more tests. |
1585 |
self.assertEqual(b"old-id", old_tree.get_revision_id()) |
|
5168.1.2
by Vincent Ladeuil
Ckeanup some more imports. |
1586 |
self.assertIsInstance(new_tree, revisiontree.RevisionTree) |
|
6973.13.2
by Jelmer Vernooij
Fix some more tests. |
1587 |
self.assertEqual(b"new-id", new_tree.get_revision_id()) |
|
4705.1.2
by Gary van der Merwe
Start on tests for get_trees_and_branches_to_diff. |
1588 |
self.assertEqual(tree.branch.base, old_branch.base) |
1589 |
self.assertEqual(tree.branch.base, new_branch.base) |
|
1590 |
self.assertIs(None, specific_files) |
|
|
4705.1.4
by Gary van der Merwe
Add newline to end of test_diff.py |
1591 |
self.assertEqual(tree.basedir, extra_trees[0].basedir) |