72
72
super(TestLogWithLogCatcher, self).setUp()
73
73
# Capture log formatter creations
74
75
class MyLogFormatter(test_log.LogCatcher):
76
77
def __new__(klass, *args, **kwargs):
77
78
self.log_catcher = test_log.LogCatcher(*args, **kwargs)
78
79
# Always return our own log formatter
79
80
return self.log_catcher
81
# Break cycle with closure over self on cleanup by removing method
82
self.addCleanup(setattr, MyLogFormatter, "__new__", None)
82
# Always return our own log formatter class hijacking the
83
# default behavior (which requires setting up a config
85
# Always return our own log formatter class hijacking the
86
# default behavior (which requires setting up a config
85
88
return MyLogFormatter
86
89
self.overrideAttr(log.log_formatter_registry, 'get_default', getme)
88
91
def get_captured_revisions(self):
89
92
return self.log_catcher.revisions
91
def assertLogRevnos(self, args, expected_revnos, working_dir='.'):
92
self.run_bzr(['log'] + args, working_dir=working_dir)
94
def assertLogRevnos(self, args, expected_revnos, working_dir='.',
96
actual_out, actual_err = self.run_bzr(['log'] + args,
97
working_dir=working_dir)
98
self.assertEqual(out, actual_out)
99
self.assertEqual(err, actual_err)
93
100
self.assertEqual(expected_revnos,
94
101
[r.revno for r in self.get_captured_revisions()])
96
103
def assertLogRevnosAndDepths(self, args, expected_revnos_and_depths,
98
105
self.run_bzr(['log'] + args, working_dir=working_dir)
99
106
self.assertEqual(expected_revnos_and_depths,
100
107
[(r.revno, r.merge_depth)
101
for r in self.get_captured_revisions()])
108
for r in self.get_captured_revisions()])
104
111
class TestLogRevSpecs(TestLogWithLogCatcher):
167
195
# stop calling run_bzr, there is no point) --vila 100118.
168
196
builder = branchbuilder.BranchBuilder(self.get_transport())
169
197
builder.start_series()
171
builder.build_snapshot('1', None, [
172
('add', ('', 'root-id', 'directory', ''))])
173
builder.build_snapshot('2', ['1'], [])
215
builder.build_snapshot(None, [
216
('add', ('', b'root-id', 'directory', ''))],
218
builder.build_snapshot([b'1'], [], revision_id=b'2')
175
builder.build_snapshot('1.1.1', ['1'], [])
220
builder.build_snapshot([b'1'], [], revision_id=b'1.1.1')
176
221
# merge branch into mainline
177
builder.build_snapshot('3', ['2', '1.1.1'], [])
222
builder.build_snapshot([b'2', b'1.1.1'], [], revision_id=b'3')
178
223
# new commits in branch
179
builder.build_snapshot('1.1.2', ['1.1.1'], [])
180
builder.build_snapshot('1.1.3', ['1.1.2'], [])
224
builder.build_snapshot([b'1.1.1'], [], revision_id=b'1.1.2')
225
builder.build_snapshot([b'1.1.2'], [], revision_id=b'1.1.3')
181
226
# merge branch into mainline
182
builder.build_snapshot('4', ['3', '1.1.3'], [])
227
builder.build_snapshot([b'3', b'1.1.3'], [], revision_id=b'4')
183
228
# merge mainline into branch
184
builder.build_snapshot('1.1.4', ['1.1.3', '4'], [])
229
builder.build_snapshot([b'1.1.3', b'4'], [], revision_id=b'1.1.4')
185
230
# merge branch into mainline
186
builder.build_snapshot('5', ['4', '1.1.4'], [])
231
builder.build_snapshot([b'4', b'1.1.4'], [], revision_id=b'5')
232
builder.build_snapshot([b'5'], [], revision_id=b'5.1.1')
233
builder.build_snapshot([b'5', b'5.1.1'], [], revision_id=b'6')
187
234
builder.finish_series()
189
236
def test_n0(self):
190
237
self.assertLogRevnos(['-n0', '-r1.1.1..1.1.4'],
191
238
['1.1.4', '4', '1.1.3', '1.1.2', '3', '1.1.1'])
192
240
def test_n0_forward(self):
193
241
self.assertLogRevnos(['-n0', '-r1.1.1..1.1.4', '--forward'],
194
242
['3', '1.1.1', '4', '1.1.2', '1.1.3', '1.1.4'])
219
272
# The graph below may look a bit complicated (and it may be but I've
220
273
# banged my head enough on it) but the bug requires at least dotted
221
274
# revnos *and* merged revisions below that.
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'], [])
288
builder.build_snapshot(None, [
289
('add', ('', b'root-id', 'directory', ''))], revision_id=b'1')
290
builder.build_snapshot([b'1'], [], revision_id=b'2')
291
builder.build_snapshot([b'1'], [], revision_id=b'1.1.1')
292
builder.build_snapshot([b'2'], [], revision_id=b'2.1.1')
293
builder.build_snapshot([b'2', b'1.1.1'], [], revision_id=b'3')
294
builder.build_snapshot([b'2.1.1'], [], revision_id=b'2.1.2')
295
builder.build_snapshot([b'2.1.1'], [], revision_id=b'2.2.1')
296
builder.build_snapshot([b'2.1.2', b'2.2.1'], [], revision_id=b'2.1.3')
297
builder.build_snapshot([b'3', b'2.1.3'], [], revision_id=b'4')
298
builder.build_snapshot([b'4', b'2.1.2'], [], revision_id=b'5')
233
299
builder.finish_series()
263
329
def test_log_revno_n_path_correct_order(self):
264
330
self.make_linear_branch('branch2')
265
331
self.assertLogRevnos(['-rrevno:1:branch2..revno:3:branch2'],
268
334
def test_log_revno_n_path(self):
269
335
self.make_linear_branch('branch2')
270
336
self.assertLogRevnos(['-rrevno:1:branch2'],
272
338
rev_props = self.log_catcher.revisions[0].rev.properties
273
self.assertEqual('branch2', rev_props['branch-nick'])
339
self.assertEqual('branch2', rev_props[u'branch-nick'])
276
342
class TestLogErrors(TestLog):
278
344
def test_log_zero_revspec(self):
279
345
self.make_minimal_branch()
280
self.run_bzr_error(['bzr: ERROR: Logging revision 0 is invalid.'],
346
self.run_bzr_error(['brz: ERROR: Logging revision 0 is invalid.'],
283
349
def test_log_zero_begin_revspec(self):
284
350
self.make_linear_branch()
285
self.run_bzr_error(['bzr: ERROR: Logging revision 0 is invalid.'],
351
self.run_bzr_error(['brz: ERROR: Logging revision 0 is invalid.'],
286
352
['log', '-r0..2'])
288
354
def test_log_zero_end_revspec(self):
289
355
self.make_linear_branch()
290
self.run_bzr_error(['bzr: ERROR: Logging revision 0 is invalid.'],
356
self.run_bzr_error(['brz: ERROR: Logging revision 0 is invalid.'],
291
357
['log', '-r-2..0'])
293
359
def test_log_nonexistent_revno(self):
294
360
self.make_minimal_branch()
295
self.run_bzr_error(["bzr: ERROR: Requested revision: '1234' "
361
self.run_bzr_error(["brz: ERROR: Requested revision: '1234' "
296
362
"does not exist in branch:"],
297
363
['log', '-r1234'])
299
365
def test_log_nonexistent_dotted_revno(self):
300
366
self.make_minimal_branch()
301
self.run_bzr_error(["bzr: ERROR: Requested revision: '123.123' "
367
self.run_bzr_error(["brz: ERROR: Requested revision: '123.123' "
302
368
"does not exist in branch:"],
303
['log', '-r123.123'])
369
['log', '-r123.123'])
305
371
def test_log_change_nonexistent_revno(self):
306
372
self.make_minimal_branch()
307
self.run_bzr_error(["bzr: ERROR: Requested revision: '1234' "
373
self.run_bzr_error(["brz: ERROR: Requested revision: '1234' "
308
374
"does not exist in branch:"],
311
377
def test_log_change_nonexistent_dotted_revno(self):
312
378
self.make_minimal_branch()
313
self.run_bzr_error(["bzr: ERROR: Requested revision: '123.123' "
379
self.run_bzr_error(["brz: ERROR: Requested revision: '123.123' "
314
380
"does not exist in branch:"],
315
381
['log', '-c123.123'])
317
383
def test_log_change_single_revno_only(self):
318
384
self.make_minimal_branch()
319
self.run_bzr_error(['bzr: ERROR: Option --change does not'
320
' accept revision ranges'],
385
self.run_bzr_error(['brz: ERROR: Option --change does not'
386
' accept revision ranges'],
321
387
['log', '--change', '2..3'])
323
389
def test_log_change_incompatible_with_revision(self):
324
self.run_bzr_error(['bzr: ERROR: --revision and --change'
325
' are mutually exclusive'],
390
self.run_bzr_error(['brz: ERROR: --revision and --change'
391
' are mutually exclusive'],
326
392
['log', '--change', '2', '--revision', '3'])
328
394
def test_log_nonexistent_file(self):
337
403
def test_log_reversed_revspecs(self):
338
404
self.make_linear_branch()
339
self.run_bzr_error(('bzr: ERROR: Start revision must be older than '
405
self.run_bzr_error(('brz: ERROR: Start revision must be older than '
340
406
'the end revision.\n',),
341
407
['log', '-r3..1'])
343
409
def test_log_reversed_dotted_revspecs(self):
344
410
self.make_merged_branch()
345
self.run_bzr_error(('bzr: ERROR: Start revision not found in '
346
'left-hand history of end revision.\n',),
411
self.run_bzr_error(('brz: ERROR: Start revision not found in '
412
'history of end revision.\n',),
347
413
"log -r 1.1.1..1")
349
415
def test_log_bad_message_re(self):
350
416
"""Bad --message argument gives a sensible message
352
418
See https://bugs.launchpad.net/bzr/+bug/251352
354
420
self.make_minimal_branch()
355
421
out, err = self.run_bzr(['log', '-m', '*'], retcode=3)
356
self.assertEqual("bzr: ERROR: Invalid regular expression"
357
" in log message filter"
359
": nothing to repeat\n", err)
360
self.assertEqual('', out)
422
self.assertContainsRe(err, "ERROR.*Invalid pattern.*nothing to repeat")
423
self.assertNotContainsRe(err, "Unprintable exception")
424
self.assertEqual(out, '')
362
426
def test_log_unsupported_timezone(self):
363
427
self.make_linear_branch()
364
self.run_bzr_error(['bzr: ERROR: Unsupported timezone format "foo", '
428
self.run_bzr_error(['brz: ERROR: Unsupported timezone format "foo", '
365
429
'options are "utc", "original", "local".'],
366
430
['log', '--timezone', 'foo'])
368
432
def test_log_exclude_ancestry_no_range(self):
369
433
self.make_linear_branch()
370
self.run_bzr_error(['bzr: ERROR: --exclude-common-ancestry'
434
self.run_bzr_error(['brz: ERROR: --exclude-common-ancestry'
371
435
' requires -r with two revisions'],
372
436
['log', '--exclude-common-ancestry'])
374
438
def test_log_exclude_ancestry_single_revision(self):
375
439
self.make_merged_branch()
376
self.run_bzr_error(['bzr: ERROR: --exclude-common-ancestry'
440
self.run_bzr_error(['brz: ERROR: --exclude-common-ancestry'
377
441
' requires two different revisions'],
378
442
['log', '--exclude-common-ancestry',
379
443
'-r1.1.1..1.1.1'])
381
446
class TestLogTags(TestLog):
383
448
def test_log_with_tags(self):
480
568
self.assertLogRevnos(['-n0'], ['2', '1.1.2', '1.2.1', '1.1.1', '1'],
481
569
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)
571
def test_include_merged(self):
572
# Confirm --include-merged gives the same output as -n0
573
expected = ['2', '1.1.2', '1.2.1', '1.1.1', '1']
574
self.assertLogRevnos(['--include-merged'],
575
expected, working_dir='level0')
576
self.assertLogRevnos(['--include-merged'],
577
expected, working_dir='level0')
498
579
def test_force_merge_revisions_N(self):
499
580
self.assertLogRevnos(['-n2'],
508
589
def test_merges_partial_range(self):
509
590
self.assertLogRevnosAndDepths(
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')
591
['-n0', '-r1.1.1..1.1.2'],
592
[('1.1.2', 0), ('1.2.1', 1), ('1.1.1', 0)],
593
working_dir='level0')
514
595
def test_merges_partial_range_ignore_before_lower_bound(self):
515
596
"""Dont show revisions before the lower bound's merged revs"""
516
597
self.assertLogRevnosAndDepths(
517
['-n0', '-r1.1.2..2'],
518
[('2', 0), ('1.1.2', 1), ('1.2.1', 2)],
519
working_dir='level0')
598
['-n0', '-r1.1.2..2'],
599
[('2', 0), ('1.1.2', 1), ('1.2.1', 2)],
600
working_dir='level0')
602
def test_omit_merges_with_sidelines(self):
603
self.assertLogRevnos(['--omit-merges', '-n0'], ['1.2.1', '1.1.1', '1'],
604
working_dir='level0')
606
def test_omit_merges_without_sidelines(self):
607
self.assertLogRevnos(['--omit-merges', '-n1'], ['1'],
608
working_dir='level0')
522
611
class TestLogDiff(TestLogWithLogCatcher):
625
713
def test_log_show_diff_non_ascii(self):
626
714
# Smoke test for bug #328007 UnicodeDecodeError on 'log -p'
627
715
message = u'Message with \xb5'
628
body = 'Body with \xb5\n'
716
body = b'Body with \xb5\n'
629
717
wt = self.make_branch_and_tree('.')
630
718
self.build_tree_contents([('foo', body)])
632
720
wt.commit(message=message)
633
721
# check that command won't fail with unicode error
634
722
# don't care about exact output because we have other tests for this
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')
723
out, err = self.run_bzr('log -p --long')
724
self.assertNotEqual('', out)
725
self.assertEqual('', err)
726
out, err = self.run_bzr('log -p --short')
727
self.assertNotEqual('', out)
728
self.assertEqual('', err)
729
out, err = self.run_bzr('log -p --line')
642
730
self.assertNotEqual('', out)
643
731
self.assertEqual('', err)
669
757
self.overrideAttr(osutils, '_cached_user_encoding')
671
759
def create_branch(self):
674
self.build_tree_contents([('a', 'some stuff\n')])
676
bzr(['commit', '-m', self._message])
762
self.build_tree_contents([('a', b'some stuff\n')])
764
brz(['commit', '-m', self._message])
678
766
def try_encoding(self, encoding, fail=False):
681
769
self.assertRaises(UnicodeEncodeError,
682
self._mu.encode, encoding)
770
self._mu.encode, encoding)
683
771
encoded_msg = self._message.encode(encoding, 'replace')
685
773
encoded_msg = self._message.encode(encoding)
687
775
old_encoding = osutils._cached_user_encoding
688
776
# This test requires that 'run_bzr' uses the current
689
# bzrlib, because we override user_encoding, and expect
777
# breezy, because we override user_encoding, and expect
692
780
osutils._cached_user_encoding = 'ascii'
693
781
# We should be able to handle any encoding
694
out, err = bzr('log', encoding=encoding)
782
out, err = brz('log', encoding=encoding)
696
784
# Make sure we wrote mu as we expected it to exist
697
self.assertNotEqual(-1, out.find(encoded_msg))
698
out_unicode = out.decode(encoding)
699
self.assertNotEqual(-1, out_unicode.find(self._message))
785
self.assertNotEqual(-1, out.find(self._message))
701
787
self.assertNotEqual(-1, out.find('Message with ?'))
893
979
self.prepare_tree()
895
981
self.assertLogRevnos(['dir2', 'file5'], ['5', '3'])
984
class MainlineGhostTests(TestLogWithLogCatcher):
987
super(MainlineGhostTests, self).setUp()
988
tree = self.make_branch_and_tree('')
989
tree.set_parent_ids([b"spooky"], allow_leftmost_as_ghost=True)
991
tree.commit('msg1', rev_id=b'rev1')
992
tree.commit('msg2', rev_id=b'rev2')
994
def test_log_range(self):
995
self.assertLogRevnos(["-r1..2"], ["2", "1"])
997
def test_log_norange(self):
998
self.assertLogRevnos([], ["2", "1"])
1000
def test_log_range_open_begin(self):
1001
(stdout, stderr) = self.run_bzr(['log', '-r..2'], retcode=3)
1002
self.assertEqual(["2", "1"],
1003
[r.revno for r in self.get_captured_revisions()])
1004
self.assertEqual("brz: ERROR: Further revision history missing.\n",
1007
def test_log_range_open_end(self):
1008
self.assertLogRevnos(["-r1.."], ["2", "1"])
1011
class TestLogMatch(TestLogWithLogCatcher):
1013
def prepare_tree(self):
1014
tree = self.make_branch_and_tree('')
1016
['/hello.txt', '/goodbye.txt'])
1017
tree.add('hello.txt')
1018
tree.commit(message='message1', committer='committer1',
1019
authors=['author1'])
1020
tree.add('goodbye.txt')
1021
tree.commit(message='message2', committer='committer2',
1022
authors=['author2'])
1024
def test_message(self):
1026
self.assertLogRevnos(["-m", "message1"], ["1"])
1027
self.assertLogRevnos(["-m", "message2"], ["2"])
1028
self.assertLogRevnos(["-m", "message"], ["2", "1"])
1029
self.assertLogRevnos(["-m", "message1", "-m", "message2"], ["2", "1"])
1030
self.assertLogRevnos(["--match-message", "message1"], ["1"])
1031
self.assertLogRevnos(["--match-message", "message2"], ["2"])
1032
self.assertLogRevnos(["--match-message", "message"], ["2", "1"])
1033
self.assertLogRevnos(["--match-message", "message1",
1034
"--match-message", "message2"], ["2", "1"])
1035
self.assertLogRevnos(["--message", "message1"], ["1"])
1036
self.assertLogRevnos(["--message", "message2"], ["2"])
1037
self.assertLogRevnos(["--message", "message"], ["2", "1"])
1038
self.assertLogRevnos(["--match-message", "message1",
1039
"--message", "message2"], ["2", "1"])
1040
self.assertLogRevnos(["--message", "message1",
1041
"--match-message", "message2"], ["2", "1"])
1043
def test_committer(self):
1045
self.assertLogRevnos(["-m", "committer1"], ["1"])
1046
self.assertLogRevnos(["-m", "committer2"], ["2"])
1047
self.assertLogRevnos(["-m", "committer"], ["2", "1"])
1048
self.assertLogRevnos(["-m", "committer1", "-m", "committer2"],
1050
self.assertLogRevnos(["--match-committer", "committer1"], ["1"])
1051
self.assertLogRevnos(["--match-committer", "committer2"], ["2"])
1052
self.assertLogRevnos(["--match-committer", "committer"], ["2", "1"])
1053
self.assertLogRevnos(["--match-committer", "committer1",
1054
"--match-committer", "committer2"], ["2", "1"])
1056
def test_author(self):
1058
self.assertLogRevnos(["-m", "author1"], ["1"])
1059
self.assertLogRevnos(["-m", "author2"], ["2"])
1060
self.assertLogRevnos(["-m", "author"], ["2", "1"])
1061
self.assertLogRevnos(["-m", "author1", "-m", "author2"],
1063
self.assertLogRevnos(["--match-author", "author1"], ["1"])
1064
self.assertLogRevnos(["--match-author", "author2"], ["2"])
1065
self.assertLogRevnos(["--match-author", "author"], ["2", "1"])
1066
self.assertLogRevnos(["--match-author", "author1",
1067
"--match-author", "author2"], ["2", "1"])
1070
class TestSmartServerLog(tests.TestCaseWithTransport):
1072
def test_standard_log(self):
1073
self.setup_smart_server_with_call_log()
1074
t = self.make_branch_and_tree('branch')
1075
self.build_tree_contents([('branch/foo', b'thecontents')])
1078
self.reset_smart_call_log()
1079
out, err = self.run_bzr(['log', self.get_url('branch')])
1080
# This figure represent the amount of work to perform this use case. It
1081
# is entirely ok to reduce this number if a test fails due to rpc_count
1082
# being too low. If rpc_count increases, more network roundtrips have
1083
# become necessary for this use case. Please do not adjust this number
1084
# upwards without agreement from bzr's network support maintainers.
1085
self.assertThat(self.hpss_calls, ContainsNoVfsCalls)
1086
self.assertLength(1, self.hpss_connections)
1087
self.assertLength(9, self.hpss_calls)
1089
def test_verbose_log(self):
1090
self.setup_smart_server_with_call_log()
1091
t = self.make_branch_and_tree('branch')
1092
self.build_tree_contents([('branch/foo', b'thecontents')])
1095
self.reset_smart_call_log()
1096
out, err = self.run_bzr(['log', '-v', self.get_url('branch')])
1097
# This figure represent the amount of work to perform this use case. It
1098
# is entirely ok to reduce this number if a test fails due to rpc_count
1099
# being too low. If rpc_count increases, more network roundtrips have
1100
# become necessary for this use case. Please do not adjust this number
1101
# upwards without agreement from bzr's network support maintainers.
1102
self.assertLength(10, self.hpss_calls)
1103
self.assertLength(1, self.hpss_connections)
1104
self.assertThat(self.hpss_calls, ContainsNoVfsCalls)
1106
def test_per_file(self):
1107
self.setup_smart_server_with_call_log()
1108
t = self.make_branch_and_tree('branch')
1109
self.build_tree_contents([('branch/foo', b'thecontents')])
1112
self.reset_smart_call_log()
1113
out, err = self.run_bzr(['log', '-v', self.get_url('branch') + "/foo"])
1114
# This figure represent the amount of work to perform this use case. It
1115
# is entirely ok to reduce this number if a test fails due to rpc_count
1116
# being too low. If rpc_count increases, more network roundtrips have
1117
# become necessary for this use case. Please do not adjust this number
1118
# upwards without agreement from bzr's network support maintainers.
1119
self.assertLength(14, self.hpss_calls)
1120
self.assertLength(1, self.hpss_connections)
1121
self.assertThat(self.hpss_calls, ContainsNoVfsCalls)