75
72
super(TestLogWithLogCatcher, self).setUp()
76
73
# Capture log formatter creations
78
74
class MyLogFormatter(test_log.LogCatcher):
80
76
def __new__(klass, *args, **kwargs):
81
77
self.log_catcher = test_log.LogCatcher(*args, **kwargs)
82
78
# Always return our own log formatter
83
79
return self.log_catcher
84
# Break cycle with closure over self on cleanup by removing method
85
self.addCleanup(setattr, MyLogFormatter, "__new__", None)
88
# Always return our own log formatter class hijacking the
89
# default behavior (which requires setting up a config
82
# Always return our own log formatter class hijacking the
83
# default behavior (which requires setting up a config
91
85
return MyLogFormatter
92
86
self.overrideAttr(log.log_formatter_registry, 'get_default', getme)
94
88
def get_captured_revisions(self):
95
89
return self.log_catcher.revisions
97
def assertLogRevnos(self, args, expected_revnos, working_dir='.',
99
actual_out, actual_err = self.run_bzr(['log'] + args,
100
working_dir=working_dir)
101
self.assertEqual(out, actual_out)
102
self.assertEqual(err, actual_err)
91
def assertLogRevnos(self, args, expected_revnos, working_dir='.'):
92
self.run_bzr(['log'] + args, working_dir=working_dir)
103
93
self.assertEqual(expected_revnos,
104
94
[r.revno for r in self.get_captured_revisions()])
106
96
def assertLogRevnosAndDepths(self, args, expected_revnos_and_depths,
108
98
self.run_bzr(['log'] + args, working_dir=working_dir)
109
99
self.assertEqual(expected_revnos_and_depths,
110
100
[(r.revno, r.merge_depth)
111
for r in self.get_captured_revisions()])
101
for r in self.get_captured_revisions()])
114
104
class TestLogRevSpecs(TestLogWithLogCatcher):
198
167
# stop calling run_bzr, there is no point) --vila 100118.
199
168
builder = branchbuilder.BranchBuilder(self.get_transport())
200
169
builder.start_series()
218
builder.build_snapshot(None, [
219
('add', ('', b'root-id', 'directory', ''))],
221
builder.build_snapshot([b'1'], [], revision_id=b'2')
171
builder.build_snapshot('1', None, [
172
('add', ('', 'root-id', 'directory', ''))])
173
builder.build_snapshot('2', ['1'], [])
223
builder.build_snapshot([b'1'], [], revision_id=b'1.1.1')
175
builder.build_snapshot('1.1.1', ['1'], [])
224
176
# merge branch into mainline
225
builder.build_snapshot([b'2', b'1.1.1'], [], revision_id=b'3')
177
builder.build_snapshot('3', ['2', '1.1.1'], [])
226
178
# new commits in branch
227
builder.build_snapshot([b'1.1.1'], [], revision_id=b'1.1.2')
228
builder.build_snapshot([b'1.1.2'], [], revision_id=b'1.1.3')
179
builder.build_snapshot('1.1.2', ['1.1.1'], [])
180
builder.build_snapshot('1.1.3', ['1.1.2'], [])
229
181
# merge branch into mainline
230
builder.build_snapshot([b'3', b'1.1.3'], [], revision_id=b'4')
182
builder.build_snapshot('4', ['3', '1.1.3'], [])
231
183
# merge mainline into branch
232
builder.build_snapshot([b'1.1.3', b'4'], [], revision_id=b'1.1.4')
184
builder.build_snapshot('1.1.4', ['1.1.3', '4'], [])
233
185
# merge branch into mainline
234
builder.build_snapshot([b'4', b'1.1.4'], [], revision_id=b'5')
235
builder.build_snapshot([b'5'], [], revision_id=b'5.1.1')
236
builder.build_snapshot([b'5', b'5.1.1'], [], revision_id=b'6')
186
builder.build_snapshot('5', ['4', '1.1.4'], [])
237
187
builder.finish_series()
239
189
def test_n0(self):
240
190
self.assertLogRevnos(['-n0', '-r1.1.1..1.1.4'],
241
191
['1.1.4', '4', '1.1.3', '1.1.2', '3', '1.1.1'])
243
192
def test_n0_forward(self):
244
193
self.assertLogRevnos(['-n0', '-r1.1.1..1.1.4', '--forward'],
245
194
['3', '1.1.1', '4', '1.1.2', '1.1.3', '1.1.4'])
275
219
# The graph below may look a bit complicated (and it may be but I've
276
220
# banged my head enough on it) but the bug requires at least dotted
277
221
# revnos *and* merged revisions below that.
291
builder.build_snapshot(None, [
292
('add', ('', b'root-id', 'directory', ''))], revision_id=b'1')
293
builder.build_snapshot([b'1'], [], revision_id=b'2')
294
builder.build_snapshot([b'1'], [], revision_id=b'1.1.1')
295
builder.build_snapshot([b'2'], [], revision_id=b'2.1.1')
296
builder.build_snapshot([b'2', b'1.1.1'], [], revision_id=b'3')
297
builder.build_snapshot([b'2.1.1'], [], revision_id=b'2.1.2')
298
builder.build_snapshot([b'2.1.1'], [], revision_id=b'2.2.1')
299
builder.build_snapshot([b'2.1.2', b'2.2.1'], [], revision_id=b'2.1.3')
300
builder.build_snapshot([b'3', b'2.1.3'], [], revision_id=b'4')
301
builder.build_snapshot([b'4', b'2.1.2'], [], revision_id=b'5')
222
builder.build_snapshot('1', None, [
223
('add', ('', 'root-id', 'directory', ''))])
224
builder.build_snapshot('2', ['1'], [])
225
builder.build_snapshot('1.1.1', ['1'], [])
226
builder.build_snapshot('2.1.1', ['2'], [])
227
builder.build_snapshot('3', ['2', '1.1.1'], [])
228
builder.build_snapshot('2.1.2', ['2.1.1'], [])
229
builder.build_snapshot('2.2.1', ['2.1.1'], [])
230
builder.build_snapshot('2.1.3', ['2.1.2', '2.2.1'], [])
231
builder.build_snapshot('4', ['3', '2.1.3'], [])
232
builder.build_snapshot('5', ['4', '2.1.2'], [])
302
233
builder.finish_series()
332
263
def test_log_revno_n_path_correct_order(self):
333
264
self.make_linear_branch('branch2')
334
265
self.assertLogRevnos(['-rrevno:1:branch2..revno:3:branch2'],
337
268
def test_log_revno_n_path(self):
338
269
self.make_linear_branch('branch2')
339
270
self.assertLogRevnos(['-rrevno:1:branch2'],
341
272
rev_props = self.log_catcher.revisions[0].rev.properties
342
self.assertEqual('branch2', rev_props[u'branch-nick'])
273
self.assertEqual('branch2', rev_props['branch-nick'])
345
276
class TestLogErrors(TestLog):
347
278
def test_log_zero_revspec(self):
348
279
self.make_minimal_branch()
349
self.run_bzr_error(['brz: ERROR: Logging revision 0 is invalid.'],
280
self.run_bzr_error(['bzr: ERROR: Logging revision 0 is invalid.'],
352
283
def test_log_zero_begin_revspec(self):
353
284
self.make_linear_branch()
354
self.run_bzr_error(['brz: ERROR: Logging revision 0 is invalid.'],
285
self.run_bzr_error(['bzr: ERROR: Logging revision 0 is invalid.'],
355
286
['log', '-r0..2'])
357
288
def test_log_zero_end_revspec(self):
358
289
self.make_linear_branch()
359
self.run_bzr_error(['brz: ERROR: Logging revision 0 is invalid.'],
290
self.run_bzr_error(['bzr: ERROR: Logging revision 0 is invalid.'],
360
291
['log', '-r-2..0'])
362
293
def test_log_nonexistent_revno(self):
363
294
self.make_minimal_branch()
364
self.run_bzr_error(["brz: ERROR: Requested revision: '1234' "
295
self.run_bzr_error(["bzr: ERROR: Requested revision: '1234' "
365
296
"does not exist in branch:"],
366
297
['log', '-r1234'])
368
299
def test_log_nonexistent_dotted_revno(self):
369
300
self.make_minimal_branch()
370
self.run_bzr_error(["brz: ERROR: Requested revision: '123.123' "
301
self.run_bzr_error(["bzr: ERROR: Requested revision: '123.123' "
371
302
"does not exist in branch:"],
372
['log', '-r123.123'])
303
['log', '-r123.123'])
374
305
def test_log_change_nonexistent_revno(self):
375
306
self.make_minimal_branch()
376
self.run_bzr_error(["brz: ERROR: Requested revision: '1234' "
307
self.run_bzr_error(["bzr: ERROR: Requested revision: '1234' "
377
308
"does not exist in branch:"],
380
311
def test_log_change_nonexistent_dotted_revno(self):
381
312
self.make_minimal_branch()
382
self.run_bzr_error(["brz: ERROR: Requested revision: '123.123' "
313
self.run_bzr_error(["bzr: ERROR: Requested revision: '123.123' "
383
314
"does not exist in branch:"],
384
315
['log', '-c123.123'])
386
317
def test_log_change_single_revno_only(self):
387
318
self.make_minimal_branch()
388
self.run_bzr_error(['brz: ERROR: Option --change does not'
389
' accept revision ranges'],
319
self.run_bzr_error(['bzr: ERROR: Option --change does not'
320
' accept revision ranges'],
390
321
['log', '--change', '2..3'])
392
323
def test_log_change_incompatible_with_revision(self):
393
self.run_bzr_error(['brz: ERROR: --revision and --change'
394
' are mutually exclusive'],
324
self.run_bzr_error(['bzr: ERROR: --revision and --change'
325
' are mutually exclusive'],
395
326
['log', '--change', '2', '--revision', '3'])
397
328
def test_log_nonexistent_file(self):
406
337
def test_log_reversed_revspecs(self):
407
338
self.make_linear_branch()
408
self.run_bzr_error(('brz: ERROR: Start revision must be older than '
339
self.run_bzr_error(('bzr: ERROR: Start revision must be older than '
409
340
'the end revision.\n',),
410
341
['log', '-r3..1'])
412
343
def test_log_reversed_dotted_revspecs(self):
413
344
self.make_merged_branch()
414
self.run_bzr_error(('brz: ERROR: Start revision not found in '
415
'history of end revision.\n',),
345
self.run_bzr_error(('bzr: ERROR: Start revision not found in '
346
'left-hand history of end revision.\n',),
416
347
"log -r 1.1.1..1")
418
349
def test_log_bad_message_re(self):
419
350
"""Bad --message argument gives a sensible message
421
352
See https://bugs.launchpad.net/bzr/+bug/251352
423
354
self.make_minimal_branch()
424
355
out, err = self.run_bzr(['log', '-m', '*'], retcode=3)
425
self.assertContainsRe(err, "ERROR.*Invalid pattern.*nothing to repeat")
426
self.assertNotContainsRe(err, "Unprintable exception")
427
self.assertEqual(out, '')
356
self.assertEqual("bzr: ERROR: Invalid regular expression"
357
" in log message filter"
359
": nothing to repeat\n", err)
360
self.assertEqual('', out)
429
362
def test_log_unsupported_timezone(self):
430
363
self.make_linear_branch()
431
self.run_bzr_error(['brz: ERROR: Unsupported timezone format "foo", '
364
self.run_bzr_error(['bzr: ERROR: Unsupported timezone format "foo", '
432
365
'options are "utc", "original", "local".'],
433
366
['log', '--timezone', 'foo'])
435
368
def test_log_exclude_ancestry_no_range(self):
436
369
self.make_linear_branch()
437
self.run_bzr_error(['brz: ERROR: --exclude-common-ancestry'
370
self.run_bzr_error(['bzr: ERROR: --exclude-common-ancestry'
438
371
' requires -r with two revisions'],
439
372
['log', '--exclude-common-ancestry'])
441
374
def test_log_exclude_ancestry_single_revision(self):
442
375
self.make_merged_branch()
443
self.run_bzr_error(['brz: ERROR: --exclude-common-ancestry'
376
self.run_bzr_error(['bzr: ERROR: --exclude-common-ancestry'
444
377
' requires two different revisions'],
445
378
['log', '--exclude-common-ancestry',
446
379
'-r1.1.1..1.1.1'])
449
381
class TestLogTags(TestLog):
451
383
def test_log_with_tags(self):
571
480
self.assertLogRevnos(['-n0'], ['2', '1.1.2', '1.2.1', '1.1.1', '1'],
572
481
working_dir='level0')
574
def test_include_merged(self):
575
# Confirm --include-merged gives the same output as -n0
576
expected = ['2', '1.1.2', '1.2.1', '1.1.1', '1']
577
self.assertLogRevnos(['--include-merged'],
578
expected, working_dir='level0')
579
self.assertLogRevnos(['--include-merged'],
580
expected, working_dir='level0')
483
def test_include_merges(self):
484
# Confirm --include-merges gives the same output as -n0
485
self.assertLogRevnos(['--include-merges'],
486
['2', '1.1.2', '1.2.1', '1.1.1', '1'],
487
working_dir='level0')
488
self.assertLogRevnos(['--include-merges'],
489
['2', '1.1.2', '1.2.1', '1.1.1', '1'],
490
working_dir='level0')
491
out_im, err_im = self.run_bzr('log --include-merges',
492
working_dir='level0')
493
out_n0, err_n0 = self.run_bzr('log -n0', working_dir='level0')
494
self.assertEqual('', err_im)
495
self.assertEqual('', err_n0)
496
self.assertEqual(out_im, out_n0)
582
498
def test_force_merge_revisions_N(self):
583
499
self.assertLogRevnos(['-n2'],
592
508
def test_merges_partial_range(self):
593
509
self.assertLogRevnosAndDepths(
594
['-n0', '-r1.1.1..1.1.2'],
595
[('1.1.2', 0), ('1.2.1', 1), ('1.1.1', 0)],
596
working_dir='level0')
510
['-n0', '-r1.1.1..1.1.2'],
511
[('1.1.2', 0), ('1.2.1', 1), ('1.1.1', 0)],
512
working_dir='level0')
598
514
def test_merges_partial_range_ignore_before_lower_bound(self):
599
515
"""Dont show revisions before the lower bound's merged revs"""
600
516
self.assertLogRevnosAndDepths(
601
['-n0', '-r1.1.2..2'],
602
[('2', 0), ('1.1.2', 1), ('1.2.1', 2)],
603
working_dir='level0')
605
def test_omit_merges_with_sidelines(self):
606
self.assertLogRevnos(['--omit-merges', '-n0'], ['1.2.1', '1.1.1', '1'],
607
working_dir='level0')
609
def test_omit_merges_without_sidelines(self):
610
self.assertLogRevnos(['--omit-merges', '-n1'], ['1'],
611
working_dir='level0')
517
['-n0', '-r1.1.2..2'],
518
[('2', 0), ('1.1.2', 1), ('1.2.1', 2)],
519
working_dir='level0')
614
522
class TestLogDiff(TestLogWithLogCatcher):
716
625
def test_log_show_diff_non_ascii(self):
717
626
# Smoke test for bug #328007 UnicodeDecodeError on 'log -p'
718
627
message = u'Message with \xb5'
719
body = b'Body with \xb5\n'
628
body = 'Body with \xb5\n'
720
629
wt = self.make_branch_and_tree('.')
721
630
self.build_tree_contents([('foo', body)])
723
632
wt.commit(message=message)
724
633
# check that command won't fail with unicode error
725
634
# don't care about exact output because we have other tests for this
726
out, err = self.run_bzr('log -p --long')
727
self.assertNotEqual('', out)
728
self.assertEqual('', err)
729
out, err = self.run_bzr('log -p --short')
730
self.assertNotEqual('', out)
731
self.assertEqual('', err)
732
out, err = self.run_bzr('log -p --line')
635
out,err = self.run_bzr('log -p --long')
636
self.assertNotEqual('', out)
637
self.assertEqual('', err)
638
out,err = self.run_bzr('log -p --short')
639
self.assertNotEqual('', out)
640
self.assertEqual('', err)
641
out,err = self.run_bzr('log -p --line')
733
642
self.assertNotEqual('', out)
734
643
self.assertEqual('', err)
760
669
self.overrideAttr(osutils, '_cached_user_encoding')
762
671
def create_branch(self):
765
self.build_tree_contents([('a', b'some stuff\n')])
767
brz(['commit', '-m', self._message])
674
self.build_tree_contents([('a', 'some stuff\n')])
676
bzr(['commit', '-m', self._message])
769
678
def try_encoding(self, encoding, fail=False):
772
681
self.assertRaises(UnicodeEncodeError,
773
self._mu.encode, encoding)
682
self._mu.encode, encoding)
774
683
encoded_msg = self._message.encode(encoding, 'replace')
776
685
encoded_msg = self._message.encode(encoding)
778
687
old_encoding = osutils._cached_user_encoding
779
688
# This test requires that 'run_bzr' uses the current
780
# breezy, because we override user_encoding, and expect
689
# bzrlib, because we override user_encoding, and expect
783
692
osutils._cached_user_encoding = 'ascii'
784
693
# We should be able to handle any encoding
785
out, err = brz('log', encoding=encoding)
694
out, err = bzr('log', encoding=encoding)
787
696
# Make sure we wrote mu as we expected it to exist
789
self.assertNotEqual(-1, out.find(encoded_msg))
790
out_unicode = out.decode(encoding)
697
self.assertNotEqual(-1, out.find(encoded_msg))
698
out_unicode = out.decode(encoding)
793
699
self.assertNotEqual(-1, out_unicode.find(self._message))
795
701
self.assertNotEqual(-1, out.find('Message with ?'))
987
893
self.prepare_tree()
989
895
self.assertLogRevnos(['dir2', 'file5'], ['5', '3'])
992
class MainlineGhostTests(TestLogWithLogCatcher):
995
super(MainlineGhostTests, self).setUp()
996
tree = self.make_branch_and_tree('')
997
tree.set_parent_ids([b"spooky"], allow_leftmost_as_ghost=True)
999
tree.commit('msg1', rev_id=b'rev1')
1000
tree.commit('msg2', rev_id=b'rev2')
1002
def test_log_range(self):
1003
self.assertLogRevnos(["-r1..2"], ["2", "1"])
1005
def test_log_norange(self):
1006
self.assertLogRevnos([], ["2", "1"])
1008
def test_log_range_open_begin(self):
1009
(stdout, stderr) = self.run_bzr(['log', '-r..2'], retcode=3)
1010
self.assertEqual(["2", "1"],
1011
[r.revno for r in self.get_captured_revisions()])
1012
self.assertEqual("brz: ERROR: Further revision history missing.\n",
1015
def test_log_range_open_end(self):
1016
self.assertLogRevnos(["-r1.."], ["2", "1"])
1019
class TestLogMatch(TestLogWithLogCatcher):
1021
def prepare_tree(self):
1022
tree = self.make_branch_and_tree('')
1024
['/hello.txt', '/goodbye.txt'])
1025
tree.add('hello.txt')
1026
tree.commit(message='message1', committer='committer1',
1027
authors=['author1'])
1028
tree.add('goodbye.txt')
1029
tree.commit(message='message2', committer='committer2',
1030
authors=['author2'])
1032
def test_message(self):
1034
self.assertLogRevnos(["-m", "message1"], ["1"])
1035
self.assertLogRevnos(["-m", "message2"], ["2"])
1036
self.assertLogRevnos(["-m", "message"], ["2", "1"])
1037
self.assertLogRevnos(["-m", "message1", "-m", "message2"], ["2", "1"])
1038
self.assertLogRevnos(["--match-message", "message1"], ["1"])
1039
self.assertLogRevnos(["--match-message", "message2"], ["2"])
1040
self.assertLogRevnos(["--match-message", "message"], ["2", "1"])
1041
self.assertLogRevnos(["--match-message", "message1",
1042
"--match-message", "message2"], ["2", "1"])
1043
self.assertLogRevnos(["--message", "message1"], ["1"])
1044
self.assertLogRevnos(["--message", "message2"], ["2"])
1045
self.assertLogRevnos(["--message", "message"], ["2", "1"])
1046
self.assertLogRevnos(["--match-message", "message1",
1047
"--message", "message2"], ["2", "1"])
1048
self.assertLogRevnos(["--message", "message1",
1049
"--match-message", "message2"], ["2", "1"])
1051
def test_committer(self):
1053
self.assertLogRevnos(["-m", "committer1"], ["1"])
1054
self.assertLogRevnos(["-m", "committer2"], ["2"])
1055
self.assertLogRevnos(["-m", "committer"], ["2", "1"])
1056
self.assertLogRevnos(["-m", "committer1", "-m", "committer2"],
1058
self.assertLogRevnos(["--match-committer", "committer1"], ["1"])
1059
self.assertLogRevnos(["--match-committer", "committer2"], ["2"])
1060
self.assertLogRevnos(["--match-committer", "committer"], ["2", "1"])
1061
self.assertLogRevnos(["--match-committer", "committer1",
1062
"--match-committer", "committer2"], ["2", "1"])
1064
def test_author(self):
1066
self.assertLogRevnos(["-m", "author1"], ["1"])
1067
self.assertLogRevnos(["-m", "author2"], ["2"])
1068
self.assertLogRevnos(["-m", "author"], ["2", "1"])
1069
self.assertLogRevnos(["-m", "author1", "-m", "author2"],
1071
self.assertLogRevnos(["--match-author", "author1"], ["1"])
1072
self.assertLogRevnos(["--match-author", "author2"], ["2"])
1073
self.assertLogRevnos(["--match-author", "author"], ["2", "1"])
1074
self.assertLogRevnos(["--match-author", "author1",
1075
"--match-author", "author2"], ["2", "1"])
1078
class TestSmartServerLog(tests.TestCaseWithTransport):
1080
def test_standard_log(self):
1081
self.setup_smart_server_with_call_log()
1082
t = self.make_branch_and_tree('branch')
1083
self.build_tree_contents([('branch/foo', b'thecontents')])
1086
self.reset_smart_call_log()
1087
out, err = self.run_bzr(['log', self.get_url('branch')])
1088
# This figure represent the amount of work to perform this use case. It
1089
# is entirely ok to reduce this number if a test fails due to rpc_count
1090
# being too low. If rpc_count increases, more network roundtrips have
1091
# become necessary for this use case. Please do not adjust this number
1092
# upwards without agreement from bzr's network support maintainers.
1093
self.assertThat(self.hpss_calls, ContainsNoVfsCalls)
1094
self.assertLength(1, self.hpss_connections)
1095
self.assertLength(9, self.hpss_calls)
1097
def test_verbose_log(self):
1098
self.setup_smart_server_with_call_log()
1099
t = self.make_branch_and_tree('branch')
1100
self.build_tree_contents([('branch/foo', b'thecontents')])
1103
self.reset_smart_call_log()
1104
out, err = self.run_bzr(['log', '-v', self.get_url('branch')])
1105
# This figure represent the amount of work to perform this use case. It
1106
# is entirely ok to reduce this number if a test fails due to rpc_count
1107
# being too low. If rpc_count increases, more network roundtrips have
1108
# become necessary for this use case. Please do not adjust this number
1109
# upwards without agreement from bzr's network support maintainers.
1110
self.assertLength(10, self.hpss_calls)
1111
self.assertLength(1, self.hpss_connections)
1112
self.assertThat(self.hpss_calls, ContainsNoVfsCalls)
1114
def test_per_file(self):
1115
self.setup_smart_server_with_call_log()
1116
t = self.make_branch_and_tree('branch')
1117
self.build_tree_contents([('branch/foo', b'thecontents')])
1120
self.reset_smart_call_log()
1121
out, err = self.run_bzr(['log', '-v', self.get_url('branch') + "/foo"])
1122
# This figure represent the amount of work to perform this use case. It
1123
# is entirely ok to reduce this number if a test fails due to rpc_count
1124
# being too low. If rpc_count increases, more network roundtrips have
1125
# become necessary for this use case. Please do not adjust this number
1126
# upwards without agreement from bzr's network support maintainers.
1127
self.assertLength(14, self.hpss_calls)
1128
self.assertLength(1, self.hpss_connections)
1129
self.assertThat(self.hpss_calls, ContainsNoVfsCalls)