13
13
# You should have received a copy of the GNU General Public License
14
14
# along with this program; if not, write to the Free Software
15
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
15
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA
18
19
from cStringIO import StringIO
23
from tempfile import TemporaryFile
29
revision as _mod_revision,
25
from bzrlib import tests
26
from bzrlib.diff import (
35
from bzrlib.symbol_versioning import deprecated_in
36
from bzrlib.tests import test_win32utils
39
class _AttribFeature(tests.Feature):
36
from bzrlib.errors import BinaryFile, NoDiff, ExecutableMissing
37
import bzrlib.osutils as osutils
38
import bzrlib.transform as transform
39
import bzrlib.patiencediff
40
import bzrlib._patiencediff_py
41
from bzrlib.tests import (Feature, TestCase, TestCaseWithTransport,
42
TestCaseInTempDir, TestSkipped)
45
class _AttribFeature(Feature):
42
48
if (sys.platform not in ('cygwin', 'win32')):
122
139
## "Unterminated hunk header for patch:\n%s" % "".join(lines)
124
141
def test_binary_lines(self):
126
uni_lines = [1023 * 'a' + '\x00']
127
self.assertRaises(errors.BinaryFile, udiff_lines, uni_lines , empty)
128
self.assertRaises(errors.BinaryFile, udiff_lines, empty, uni_lines)
129
udiff_lines(uni_lines , empty, allow_binary=True)
130
udiff_lines(empty, uni_lines, allow_binary=True)
142
self.assertRaises(BinaryFile, udiff_lines, [1023 * 'a' + '\x00'], [])
143
self.assertRaises(BinaryFile, udiff_lines, [], [1023 * 'a' + '\x00'])
144
udiff_lines([1023 * 'a' + '\x00'], [], allow_binary=True)
145
udiff_lines([], [1023 * 'a' + '\x00'], allow_binary=True)
132
147
def test_external_diff(self):
133
148
lines = external_udiff_lines(['boo\n'], ['goo\n'])
163
178
orig_path = os.environ['PATH']
165
180
os.environ['PATH'] = ''
166
self.assertRaises(errors.NoDiff, diff.external_diff,
181
self.assertRaises(NoDiff, external_diff,
167
182
'old', ['boo\n'], 'new', ['goo\n'],
168
183
StringIO(), diff_opts=['-u'])
170
185
os.environ['PATH'] = orig_path
172
187
def test_internal_diff_default(self):
173
188
# Default internal diff encoding is utf8
174
189
output = StringIO()
175
diff.internal_diff(u'old_\xb5', ['old_text\n'],
176
u'new_\xe5', ['new_text\n'], output)
190
internal_diff(u'old_\xb5', ['old_text\n'],
191
u'new_\xe5', ['new_text\n'], output)
177
192
lines = output.getvalue().splitlines(True)
178
193
self.check_patch(lines)
179
194
self.assertEquals(['--- old_\xc2\xb5\n',
220
235
def test_internal_diff_no_content(self):
221
236
output = StringIO()
222
diff.internal_diff(u'old', [], u'new', [], output)
237
internal_diff(u'old', [], u'new', [], output)
223
238
self.assertEqual('', output.getvalue())
225
240
def test_internal_diff_no_changes(self):
226
241
output = StringIO()
227
diff.internal_diff(u'old', ['text\n', 'contents\n'],
228
u'new', ['text\n', 'contents\n'],
242
internal_diff(u'old', ['text\n', 'contents\n'],
243
u'new', ['text\n', 'contents\n'],
230
245
self.assertEqual('', output.getvalue())
232
247
def test_internal_diff_returns_bytes(self):
234
249
output = StringIO.StringIO()
235
diff.internal_diff(u'old_\xb5', ['old_text\n'],
236
u'new_\xe5', ['new_text\n'], output)
250
internal_diff(u'old_\xb5', ['old_text\n'],
251
u'new_\xe5', ['new_text\n'], output)
237
252
self.failUnless(isinstance(output.getvalue(), str),
238
253
'internal_diff should return bytestrings')
241
class TestDiffFiles(tests.TestCaseInTempDir):
256
class TestDiffFiles(TestCaseInTempDir):
243
258
def test_external_diff_binary(self):
244
259
"""The output when using external diff should use diff's i18n error"""
414
428
tree.commit('one', rev_id='rev-1')
416
430
self.build_tree_contents([('tree/file', 'new contents\n')])
417
d = self.get_diff(tree.basis_tree(), tree)
418
self.assertContainsRe(d, "=== modified file 'file'\n")
419
self.assertContainsRe(d, '--- old/file\t')
420
self.assertContainsRe(d, '\\+\\+\\+ new/file\t')
421
self.assertContainsRe(d, '-contents\n'
431
diff = self.get_diff(tree.basis_tree(), tree)
432
self.assertContainsRe(diff, "=== modified file 'file'\n")
433
self.assertContainsRe(diff, '--- old/file\t')
434
self.assertContainsRe(diff, '\\+\\+\\+ new/file\t')
435
self.assertContainsRe(diff, '-contents\n'
424
438
def test_modified_file_in_renamed_dir(self):
425
439
"""Test when a file is modified in a renamed directory."""
432
446
tree.rename_one('dir', 'other')
433
447
self.build_tree_contents([('tree/other/file', 'new contents\n')])
434
d = self.get_diff(tree.basis_tree(), tree)
435
self.assertContainsRe(d, "=== renamed directory 'dir' => 'other'\n")
436
self.assertContainsRe(d, "=== modified file 'other/file'\n")
448
diff = self.get_diff(tree.basis_tree(), tree)
449
self.assertContainsRe(diff, "=== renamed directory 'dir' => 'other'\n")
450
self.assertContainsRe(diff, "=== modified file 'other/file'\n")
437
451
# XXX: This is technically incorrect, because it used to be at another
438
452
# location. What to do?
439
self.assertContainsRe(d, '--- old/dir/file\t')
440
self.assertContainsRe(d, '\\+\\+\\+ new/other/file\t')
441
self.assertContainsRe(d, '-contents\n'
453
self.assertContainsRe(diff, '--- old/dir/file\t')
454
self.assertContainsRe(diff, '\\+\\+\\+ new/other/file\t')
455
self.assertContainsRe(diff, '-contents\n'
444
458
def test_renamed_directory(self):
445
459
"""Test when only a directory is only renamed."""
508
522
tree.rename_one('c', 'new-c')
509
523
tree.rename_one('d', 'new-d')
511
d = self.get_diff(tree.basis_tree(), tree)
525
diff = self.get_diff(tree.basis_tree(), tree)
513
self.assertContainsRe(d, r"file 'a'.*\(properties changed:"
515
self.assertContainsRe(d, r"file 'b'.*\(properties changed:"
517
self.assertContainsRe(d, r"file 'c'.*\(properties changed:"
519
self.assertContainsRe(d, r"file 'd'.*\(properties changed:"
521
self.assertNotContainsRe(d, r"file 'e'")
522
self.assertNotContainsRe(d, r"file 'f'")
527
self.assertContainsRe(diff, r"file 'a'.*\(properties changed:.*\+x to -x.*\)")
528
self.assertContainsRe(diff, r"file 'b'.*\(properties changed:.*-x to \+x.*\)")
529
self.assertContainsRe(diff, r"file 'c'.*\(properties changed:.*\+x to -x.*\)")
530
self.assertContainsRe(diff, r"file 'd'.*\(properties changed:.*-x to \+x.*\)")
531
self.assertNotContainsRe(diff, r"file 'e'")
532
self.assertNotContainsRe(diff, r"file 'f'")
525
535
def test_binary_unicode_filenames(self):
541
551
tree.add([alpha], ['file-id'])
542
552
tree.add([omega], ['file-id-2'])
543
553
diff_content = StringIO()
544
diff.show_diff_trees(tree.basis_tree(), tree, diff_content)
545
d = diff_content.getvalue()
546
self.assertContainsRe(d, r"=== added file '%s'" % alpha_utf8)
547
self.assertContainsRe(d, "Binary files a/%s.*and b/%s.* differ\n"
548
% (alpha_utf8, alpha_utf8))
549
self.assertContainsRe(d, r"=== added file '%s'" % omega_utf8)
550
self.assertContainsRe(d, r"--- a/%s" % (omega_utf8,))
551
self.assertContainsRe(d, r"\+\+\+ b/%s" % (omega_utf8,))
554
show_diff_trees(tree.basis_tree(), tree, diff_content)
555
diff = diff_content.getvalue()
556
self.assertContainsRe(diff, r"=== added file '%s'" % alpha_utf8)
557
self.assertContainsRe(
558
diff, "Binary files a/%s.*and b/%s.* differ\n" % (alpha_utf8, alpha_utf8))
559
self.assertContainsRe(diff, r"=== added file '%s'" % omega_utf8)
560
self.assertContainsRe(diff, r"--- a/%s" % (omega_utf8,))
561
self.assertContainsRe(diff, r"\+\+\+ b/%s" % (omega_utf8,))
553
563
def test_unicode_filename(self):
554
564
"""Test when the filename are unicode."""
573
583
tree.add(['add_'+alpha], ['file-id'])
574
584
self.build_tree_contents([('tree/mod_'+alpha, 'contents_mod\n')])
576
d = self.get_diff(tree.basis_tree(), tree)
577
self.assertContainsRe(d,
586
diff = self.get_diff(tree.basis_tree(), tree)
587
self.assertContainsRe(diff,
578
588
"=== renamed file 'ren_%s' => 'ren_%s'\n"%(autf8, outf8))
579
self.assertContainsRe(d, "=== added file 'add_%s'"%autf8)
580
self.assertContainsRe(d, "=== modified file 'mod_%s'"%autf8)
581
self.assertContainsRe(d, "=== removed file 'del_%s'"%autf8)
584
class DiffWasIs(diff.DiffPath):
589
self.assertContainsRe(diff, "=== added file 'add_%s'"%autf8)
590
self.assertContainsRe(diff, "=== modified file 'mod_%s'"%autf8)
591
self.assertContainsRe(diff, "=== removed file 'del_%s'"%autf8)
594
class DiffWasIs(DiffPath):
586
596
def diff(self, file_id, old_path, new_path, old_kind, new_kind):
587
597
self.to_file.write('was: ')
647
657
self.assertContainsRe(self.differ.to_file.getvalue(), '\+contents')
649
659
def test_diff_symlink(self):
650
differ = diff.DiffSymlink(self.old_tree, self.new_tree, StringIO())
660
differ = DiffSymlink(self.old_tree, self.new_tree, StringIO())
651
661
differ.diff_symlink('old target', None)
652
662
self.assertEqual("=== target was 'old target'\n",
653
663
differ.to_file.getvalue())
655
differ = diff.DiffSymlink(self.old_tree, self.new_tree, StringIO())
665
differ = DiffSymlink(self.old_tree, self.new_tree, StringIO())
656
666
differ.diff_symlink(None, 'new target')
657
667
self.assertEqual("=== target is 'new target'\n",
658
668
differ.to_file.getvalue())
660
differ = diff.DiffSymlink(self.old_tree, self.new_tree, StringIO())
670
differ = DiffSymlink(self.old_tree, self.new_tree, StringIO())
661
671
differ.diff_symlink('old target', 'new target')
662
672
self.assertEqual("=== target changed 'old target' => 'new target'\n",
663
673
differ.to_file.getvalue())
750
760
'.*a-file(.|\n)*b-file')
753
class TestPatienceDiffLib(tests.TestCase):
763
class TestPatienceDiffLib(TestCase):
756
766
super(TestPatienceDiffLib, self).setUp()
757
self._unique_lcs = _patiencediff_py.unique_lcs_py
758
self._recurse_matches = _patiencediff_py.recurse_matches_py
767
self._unique_lcs = bzrlib._patiencediff_py.unique_lcs_py
768
self._recurse_matches = bzrlib._patiencediff_py.recurse_matches_py
759
769
self._PatienceSequenceMatcher = \
760
_patiencediff_py.PatienceSequenceMatcher_py
762
def test_diff_unicode_string(self):
763
a = ''.join([unichr(i) for i in range(4000, 4500, 3)])
764
b = ''.join([unichr(i) for i in range(4300, 4800, 2)])
765
sm = self._PatienceSequenceMatcher(None, a, b)
766
mb = sm.get_matching_blocks()
767
self.assertEquals(35, len(mb))
770
bzrlib._patiencediff_py.PatienceSequenceMatcher_py
769
772
def test_unique_lcs(self):
770
773
unique_lcs = self._unique_lcs
776
779
self.assertEquals(unique_lcs('ab', 'ab'), [(0,0), (1,1)])
777
780
self.assertEquals(unique_lcs('abcde', 'cdeab'), [(2,0), (3,1), (4,2)])
778
781
self.assertEquals(unique_lcs('cdeab', 'abcde'), [(0,2), (1,3), (2,4)])
779
self.assertEquals(unique_lcs('abXde', 'abYde'), [(0,0), (1,1),
782
self.assertEquals(unique_lcs('abXde', 'abYde'), [(0,0), (1,1),
781
784
self.assertEquals(unique_lcs('acbac', 'abc'), [(2,1)])
1118
1121
, list(unified_diff(txt_a, txt_b,
1119
1122
sequencematcher=psm)))
1121
def test_patience_unified_diff_with_dates(self):
1122
txt_a = ['hello there\n',
1124
'how are you today?\n']
1125
txt_b = ['hello there\n',
1126
'how are you today?\n']
1127
unified_diff = patiencediff.unified_diff
1128
psm = self._PatienceSequenceMatcher
1129
self.assertEquals(['--- a\t2008-08-08\n',
1130
'+++ b\t2008-09-09\n',
1131
'@@ -1,3 +1,2 @@\n',
1134
' how are you today?\n'
1136
, list(unified_diff(txt_a, txt_b,
1137
fromfile='a', tofile='b',
1138
fromfiledate='2008-08-08',
1139
tofiledate='2008-09-09',
1140
sequencematcher=psm)))
1143
1125
class TestPatienceDiffLib_c(TestPatienceDiffLib):
1145
_test_needs_features = [compiled_patiencediff_feature]
1127
_test_needs_features = [CompiledPatienceDiffFeature]
1147
1129
def setUp(self):
1148
1130
super(TestPatienceDiffLib_c, self).setUp()
1149
from bzrlib import _patiencediff_c
1150
self._unique_lcs = _patiencediff_c.unique_lcs_c
1151
self._recurse_matches = _patiencediff_c.recurse_matches_c
1131
import bzrlib._patiencediff_c
1132
self._unique_lcs = bzrlib._patiencediff_c.unique_lcs_c
1133
self._recurse_matches = bzrlib._patiencediff_c.recurse_matches_c
1152
1134
self._PatienceSequenceMatcher = \
1153
_patiencediff_c.PatienceSequenceMatcher_c
1135
bzrlib._patiencediff_c.PatienceSequenceMatcher_c
1155
1137
def test_unhashable(self):
1156
1138
"""We should get a proper exception here."""
1239
1221
class TestPatienceDiffLibFiles_c(TestPatienceDiffLibFiles):
1241
_test_needs_features = [compiled_patiencediff_feature]
1223
_test_needs_features = [CompiledPatienceDiffFeature]
1243
1225
def setUp(self):
1244
1226
super(TestPatienceDiffLibFiles_c, self).setUp()
1245
from bzrlib import _patiencediff_c
1227
import bzrlib._patiencediff_c
1246
1228
self._PatienceSequenceMatcher = \
1247
_patiencediff_c.PatienceSequenceMatcher_c
1250
class TestUsingCompiledIfAvailable(tests.TestCase):
1229
bzrlib._patiencediff_c.PatienceSequenceMatcher_c
1232
class TestUsingCompiledIfAvailable(TestCase):
1252
1234
def test_PatienceSequenceMatcher(self):
1253
if compiled_patiencediff_feature.available():
1235
if CompiledPatienceDiffFeature.available():
1254
1236
from bzrlib._patiencediff_c import PatienceSequenceMatcher_c
1255
1237
self.assertIs(PatienceSequenceMatcher_c,
1256
patiencediff.PatienceSequenceMatcher)
1238
bzrlib.patiencediff.PatienceSequenceMatcher)
1258
1240
from bzrlib._patiencediff_py import PatienceSequenceMatcher_py
1259
1241
self.assertIs(PatienceSequenceMatcher_py,
1260
patiencediff.PatienceSequenceMatcher)
1242
bzrlib.patiencediff.PatienceSequenceMatcher)
1262
1244
def test_unique_lcs(self):
1263
if compiled_patiencediff_feature.available():
1245
if CompiledPatienceDiffFeature.available():
1264
1246
from bzrlib._patiencediff_c import unique_lcs_c
1265
1247
self.assertIs(unique_lcs_c,
1266
patiencediff.unique_lcs)
1248
bzrlib.patiencediff.unique_lcs)
1268
1250
from bzrlib._patiencediff_py import unique_lcs_py
1269
1251
self.assertIs(unique_lcs_py,
1270
patiencediff.unique_lcs)
1252
bzrlib.patiencediff.unique_lcs)
1272
1254
def test_recurse_matches(self):
1273
if compiled_patiencediff_feature.available():
1255
if CompiledPatienceDiffFeature.available():
1274
1256
from bzrlib._patiencediff_c import recurse_matches_c
1275
1257
self.assertIs(recurse_matches_c,
1276
patiencediff.recurse_matches)
1258
bzrlib.patiencediff.recurse_matches)
1278
1260
from bzrlib._patiencediff_py import recurse_matches_py
1279
1261
self.assertIs(recurse_matches_py,
1280
patiencediff.recurse_matches)
1283
class TestDiffFromTool(tests.TestCaseWithTransport):
1262
bzrlib.patiencediff.recurse_matches)
1265
class TestDiffFromTool(TestCaseWithTransport):
1285
1267
def test_from_string(self):
1286
diff_obj = diff.DiffFromTool.from_string('diff', None, None, None)
1268
diff_obj = DiffFromTool.from_string('diff', None, None, None)
1287
1269
self.addCleanup(diff_obj.finish)
1288
self.assertEqual(['diff', '@old_path', '@new_path'],
1270
self.assertEqual(['diff', '%(old_path)s', '%(new_path)s'],
1289
1271
diff_obj.command_template)
1291
1273
def test_from_string_u5(self):
1292
diff_obj = diff.DiffFromTool.from_string('diff "-u 5"',
1274
diff_obj = DiffFromTool.from_string('diff -u\\ 5', None, None, None)
1294
1275
self.addCleanup(diff_obj.finish)
1295
self.assertEqual(['diff', '-u 5', '@old_path', '@new_path'],
1276
self.assertEqual(['diff', '-u 5', '%(old_path)s', '%(new_path)s'],
1296
1277
diff_obj.command_template)
1297
1278
self.assertEqual(['diff', '-u 5', 'old-path', 'new-path'],
1298
1279
diff_obj._get_command('old-path', 'new-path'))
1300
def test_from_string_path_with_backslashes(self):
1301
self.requireFeature(test_win32utils.BackslashDirSeparatorFeature)
1302
tool = 'C:\\Tools\\Diff.exe'
1303
diff_obj = diff.DiffFromTool.from_string(tool, None, None, None)
1304
self.addCleanup(diff_obj.finish)
1305
self.assertEqual(['C:\\Tools\\Diff.exe', '@old_path', '@new_path'],
1306
diff_obj.command_template)
1307
self.assertEqual(['C:\\Tools\\Diff.exe', 'old-path', 'new-path'],
1308
diff_obj._get_command('old-path', 'new-path'))
1310
1281
def test_execute(self):
1311
1282
output = StringIO()
1312
diff_obj = diff.DiffFromTool(['python', '-c',
1313
'print "@old_path @new_path"'],
1283
diff_obj = DiffFromTool(['python', '-c',
1284
'print "%(old_path)s %(new_path)s"'],
1315
1286
self.addCleanup(diff_obj.finish)
1316
1287
diff_obj._execute('old', 'new')
1317
1288
self.assertEqual(output.getvalue().rstrip(), 'old new')
1319
1290
def test_excute_missing(self):
1320
diff_obj = diff.DiffFromTool(['a-tool-which-is-unlikely-to-exist'],
1291
diff_obj = DiffFromTool(['a-tool-which-is-unlikely-to-exist'],
1322
1293
self.addCleanup(diff_obj.finish)
1323
e = self.assertRaises(errors.ExecutableMissing, diff_obj._execute,
1294
e = self.assertRaises(ExecutableMissing, diff_obj._execute, 'old',
1325
1296
self.assertEqual('a-tool-which-is-unlikely-to-exist could not be found'
1326
1297
' on this machine', str(e))
1334
1305
tree.commit('old tree')
1335
1306
tree.lock_read()
1336
1307
self.addCleanup(tree.unlock)
1337
basis_tree = tree.basis_tree()
1338
basis_tree.lock_read()
1339
self.addCleanup(basis_tree.unlock)
1340
diff_obj = diff.DiffFromTool(['python', '-c',
1341
'print "@old_path @new_path"'],
1342
basis_tree, tree, output)
1308
diff_obj = DiffFromTool(['python', '-c',
1309
'print "%(old_path)s %(new_path)s"'],
1343
1311
diff_obj._prepare_files('file-id', 'file', 'file')
1344
# The old content should be readonly
1345
self.assertReadableByAttrib(diff_obj._root, 'old\\file',
1347
# The new content should use the tree object, not a 'new' file anymore
1348
self.assertEndsWith(tree.basedir, 'work/tree')
1349
self.assertReadableByAttrib(tree.basedir, 'file', r'work\\tree\\file$')
1312
self.assertReadableByAttrib(diff_obj._root, 'old\\file', r'old\\file')
1313
self.assertReadableByAttrib(diff_obj._root, 'new\\file', r'new\\file')
1351
1315
def assertReadableByAttrib(self, cwd, relpath, regex):
1352
1316
proc = subprocess.Popen(['attrib', relpath],
1353
1317
stdout=subprocess.PIPE,
1355
(result, err) = proc.communicate()
1356
self.assertContainsRe(result.replace('\r\n', '\n'), regex)
1320
result = proc.stdout.read()
1321
self.assertContainsRe(result, regex)
1358
1323
def test_prepare_files(self):
1359
1324
output = StringIO()
1373
1337
self.addCleanup(old_tree.unlock)
1374
1338
tree.lock_read()
1375
1339
self.addCleanup(tree.unlock)
1376
diff_obj = diff.DiffFromTool(['python', '-c',
1377
'print "@old_path @new_path"'],
1378
old_tree, tree, output)
1340
diff_obj = DiffFromTool(['python', '-c',
1341
'print "%(old_path)s %(new_path)s"'],
1342
old_tree, tree, output)
1379
1343
self.addCleanup(diff_obj.finish)
1380
1344
self.assertContainsRe(diff_obj._root, 'bzr-diff-[^/]*')
1381
1345
old_path, new_path = diff_obj._prepare_files('file-id', 'oldname',
1383
1347
self.assertContainsRe(old_path, 'old/oldname$')
1384
self.assertEqual(315532800, os.stat(old_path).st_mtime)
1385
self.assertContainsRe(new_path, 'tree/newname$')
1348
self.assertEqual(0, os.stat(old_path).st_mtime)
1349
self.assertContainsRe(new_path, 'new/newname$')
1386
1350
self.assertFileEqual('oldcontent', old_path)
1387
1351
self.assertFileEqual('newcontent', new_path)
1388
1352
if osutils.host_os_dereferences_symlinks():
1389
1353
self.assertTrue(os.path.samefile('tree/newname', new_path))
1390
1354
# make sure we can create files with the same parent directories
1391
1355
diff_obj._prepare_files('file2-id', 'oldname2', 'newname2')
1394
class TestGetTreesAndBranchesToDiffLocked(tests.TestCaseWithTransport):
1396
def call_gtabtd(self, path_list, revision_specs, old_url, new_url):
1397
"""Call get_trees_and_branches_to_diff_locked. Overridden by
1398
TestGetTreesAndBranchesToDiff.
1400
return diff.get_trees_and_branches_to_diff_locked(
1401
path_list, revision_specs, old_url, new_url, self.addCleanup)
1403
def test_basic(self):
1404
tree = self.make_branch_and_tree('tree')
1405
(old_tree, new_tree,
1406
old_branch, new_branch,
1407
specific_files, extra_trees) = self.call_gtabtd(
1408
['tree'], None, None, None)
1410
self.assertIsInstance(old_tree, revisiontree.RevisionTree)
1411
self.assertEqual(_mod_revision.NULL_REVISION,
1412
old_tree.get_revision_id())
1413
self.assertEqual(tree.basedir, new_tree.basedir)
1414
self.assertEqual(tree.branch.base, old_branch.base)
1415
self.assertEqual(tree.branch.base, new_branch.base)
1416
self.assertIs(None, specific_files)
1417
self.assertIs(None, extra_trees)
1419
def test_with_rev_specs(self):
1420
tree = self.make_branch_and_tree('tree')
1421
self.build_tree_contents([('tree/file', 'oldcontent')])
1422
tree.add('file', 'file-id')
1423
tree.commit('old tree', timestamp=0, rev_id="old-id")
1424
self.build_tree_contents([('tree/file', 'newcontent')])
1425
tree.commit('new tree', timestamp=0, rev_id="new-id")
1427
revisions = [revisionspec.RevisionSpec.from_string('1'),
1428
revisionspec.RevisionSpec.from_string('2')]
1429
(old_tree, new_tree,
1430
old_branch, new_branch,
1431
specific_files, extra_trees) = self.call_gtabtd(
1432
['tree'], revisions, None, None)
1434
self.assertIsInstance(old_tree, revisiontree.RevisionTree)
1435
self.assertEqual("old-id", old_tree.get_revision_id())
1436
self.assertIsInstance(new_tree, revisiontree.RevisionTree)
1437
self.assertEqual("new-id", new_tree.get_revision_id())
1438
self.assertEqual(tree.branch.base, old_branch.base)
1439
self.assertEqual(tree.branch.base, new_branch.base)
1440
self.assertIs(None, specific_files)
1441
self.assertEqual(tree.basedir, extra_trees[0].basedir)
1444
class TestGetTreesAndBranchesToDiff(TestGetTreesAndBranchesToDiffLocked):
1445
"""Apply the tests for get_trees_and_branches_to_diff_locked to the
1446
deprecated get_trees_and_branches_to_diff function.
1449
def call_gtabtd(self, path_list, revision_specs, old_url, new_url):
1450
return self.applyDeprecated(
1451
deprecated_in((2, 2, 0)), diff.get_trees_and_branches_to_diff,
1452
path_list, revision_specs, old_url, new_url)