/brz/remove-bazaar

To get this branch, use:
bzr branch http://gegoxaren.bato24.eu/bzr/brz/remove-bazaar

« back to all changes in this revision

Viewing changes to breezy/tests/test_log.py

  • Committer: Jelmer Vernooij
  • Date: 2018-07-08 14:45:27 UTC
  • mto: This revision was merged to the branch mainline in revision 7036.
  • Revision ID: jelmer@jelmer.uk-20180708144527-codhlvdcdg9y0nji
Fix a bunch of merge tests.

Show diffs side-by-side

added added

removed removed

Lines of Context:
breezy/tests/test_log.py
1
 
# Copyright (C) 2005-2010 Canonical Ltd
 
1
# Copyright (C) 2005-2013, 2016 Canonical Ltd
2
2
#
3
3
# This program is free software; you can redistribute it and/or modify
4
4
# it under the terms of the GNU General Public License as published by
15
15
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
16
16
 
17
17
import os
18
 
from cStringIO import StringIO
19
18
 
20
 
from bzrlib import (
 
19
from .. import (
21
20
    branchbuilder,
22
21
    errors,
23
22
    log,
24
23
    registry,
25
24
    revision,
26
25
    revisionspec,
27
 
    symbol_versioning,
28
26
    tests,
 
27
    gpg,
 
28
    trace,
 
29
    )
 
30
from ..sixish import (
 
31
    BytesIO,
29
32
    )
30
33
 
31
34
 
88
91
        """Helper method for LogFormatter tests"""
89
92
        b = wt.branch
90
93
        b.nick = 'test'
91
 
        self.build_tree_contents([('a', 'hello moto\n')])
92
 
        self.wt_commit(wt, 'simple log message', rev_id='a1')
93
 
        self.build_tree_contents([('b', 'goodbye\n')])
 
94
        self.build_tree_contents([('a', b'hello moto\n')])
 
95
        self.wt_commit(wt, 'simple log message', rev_id=b'a1')
 
96
        self.build_tree_contents([('b', b'goodbye\n')])
94
97
        wt.add('b')
95
 
        self.wt_commit(wt, 'multiline\nlog\nmessage\n', rev_id='a2')
 
98
        self.wt_commit(wt, 'multiline\nlog\nmessage\n', rev_id=b'a2')
96
99
 
97
 
        self.build_tree_contents([('c', 'just another manic monday\n')])
 
100
        self.build_tree_contents([('c', b'just another manic monday\n')])
98
101
        wt.add('c')
99
 
        self.wt_commit(wt, 'single line with trailing newline\n', rev_id='a3')
 
102
        self.wt_commit(wt, 'single line with trailing newline\n', rev_id=b'a3')
100
103
        return b
101
104
 
102
105
    def _prepare_tree_with_merges(self, with_tags=False):
104
107
        wt.lock_write()
105
108
        self.addCleanup(wt.unlock)
106
109
        wt.add('')
107
 
        self.wt_commit(wt, 'rev-1', rev_id='rev-1')
108
 
        self.wt_commit(wt, 'rev-merged', rev_id='rev-2a')
109
 
        wt.set_parent_ids(['rev-1', 'rev-2a'])
110
 
        wt.branch.set_last_revision_info(1, 'rev-1')
111
 
        self.wt_commit(wt, 'rev-2', rev_id='rev-2b')
 
110
        self.wt_commit(wt, 'rev-1', rev_id=b'rev-1')
 
111
        self.wt_commit(wt, 'rev-merged', rev_id=b'rev-2a')
 
112
        wt.set_parent_ids([b'rev-1', b'rev-2a'])
 
113
        wt.branch.set_last_revision_info(1, b'rev-1')
 
114
        self.wt_commit(wt, 'rev-2', rev_id=b'rev-2b')
112
115
        if with_tags:
113
116
            branch = wt.branch
114
 
            branch.tags.set_tag('v0.2', 'rev-2b')
115
 
            self.wt_commit(wt, 'rev-3', rev_id='rev-3')
116
 
            branch.tags.set_tag('v1.0rc1', 'rev-3')
117
 
            branch.tags.set_tag('v1.0', 'rev-3')
 
117
            branch.tags.set_tag('v0.2', b'rev-2b')
 
118
            self.wt_commit(wt, 'rev-3', rev_id=b'rev-3')
 
119
            branch.tags.set_tag('v1.0rc1', b'rev-3')
 
120
            branch.tags.set_tag('v1.0', b'rev-3')
118
121
        return wt
119
122
 
 
123
 
120
124
class LogCatcher(log.LogFormatter):
121
125
    """Pull log messages into a list rather than displaying them.
122
126
 
173
177
        self.assertInvalidRevisonNumber(b, 2, 1)
174
178
        self.assertInvalidRevisonNumber(b, 1, 2)
175
179
        self.assertInvalidRevisonNumber(b, 0, 2)
176
 
        self.assertInvalidRevisonNumber(b, 1, 0)
177
180
        self.assertInvalidRevisonNumber(b, -1, 1)
178
181
        self.assertInvalidRevisonNumber(b, 1, -1)
 
182
        self.assertInvalidRevisonNumber(b, 1, 0)
179
183
 
180
184
    def test_empty_branch(self):
181
185
        wt = self.make_branch_and_tree('.')
250
254
        wt.commit(message='add file1 and file2')
251
255
        self.run_bzr('branch parent child')
252
256
        os.unlink('child/file1')
253
 
        file('child/file2', 'wb').write('hello\n')
 
257
        with open('child/file2', 'wb') as f: f.write(b'hello\n')
254
258
        self.run_bzr(['commit', '-m', 'remove file1 and modify file2',
255
259
            'child'])
256
260
        os.chdir('parent')
280
284
        self.assertEqual('add file1 and file2', logentry.rev.message)
281
285
        self.checkDelta(logentry.delta, added=['file1', 'file2'])
282
286
 
 
287
    def test_bug_842695_log_restricted_to_dir(self):
 
288
        # Comments here indicate revision numbers in trunk  # VVVVV
 
289
        trunk = self.make_branch_and_tree('this')
 
290
        trunk.commit('initial trunk')                       # 1
 
291
        adder = trunk.controldir.sprout('adder').open_workingtree()
 
292
        merger = trunk.controldir.sprout('merger').open_workingtree()
 
293
        self.build_tree_contents([
 
294
            ('adder/dir/',),
 
295
            ('adder/dir/file', b'foo'),
 
296
            ])
 
297
        adder.add(['dir', 'dir/file'])
 
298
        adder.commit('added dir')                           # 1.1.1
 
299
        trunk.merge_from_branch(adder.branch)
 
300
        trunk.commit('merged adder into trunk')             # 2
 
301
        merger.merge_from_branch(trunk.branch)
 
302
        merger.commit('merged trunk into merger')           # 1.2.1
 
303
        # Commits are processed in increments of 200 revisions, so
 
304
        # make sure the two merges into trunk are in different chunks.
 
305
        for i in xrange(200):
 
306
            trunk.commit('intermediate commit %d' % i)      # 3-202
 
307
        trunk.merge_from_branch(merger.branch)
 
308
        trunk.commit('merged merger into trunk')            # 203
 
309
        file_id = trunk.path2id('dir')
 
310
        lf = LogCatcher()
 
311
        lf.supports_merge_revisions = True
 
312
        log.show_log(trunk.branch, lf, file_id)
 
313
        try:
 
314
            self.assertEqual(['2', '1.1.1'], [r.revno for r in lf.revisions])
 
315
        except AssertionError:
 
316
            raise tests.KnownFailure("bug #842695")
 
317
 
 
318
 
 
319
class TestFormatSignatureValidity(tests.TestCaseWithTransport):
 
320
 
 
321
    def verify_revision_signature(self, revid, gpg_strategy):
 
322
        return (gpg.SIGNATURE_VALID,
 
323
            u'UTF8 Test \xa1\xb1\xc1\xd1\xe1\xf1 <jrandom@example.com>')
 
324
 
 
325
    def test_format_signature_validity_utf(self):
 
326
        """Check that GPG signatures containing UTF-8 names are formatted
 
327
        correctly."""
 
328
        wt = self.make_branch_and_tree('.')
 
329
        revid = wt.commit('empty commit')
 
330
        repo = wt.branch.repository
 
331
        # Monkey patch out checking if this rev is actually signed, since we
 
332
        # can't sign it without a heavier TestCase and LoopbackGPGStrategy
 
333
        # doesn't care anyways.
 
334
        self.overrideAttr(repo, 'verify_revision_signature',
 
335
                self.verify_revision_signature)
 
336
        out = log.format_signature_validity(revid, wt.branch)
 
337
        self.assertEqual(
 
338
u'valid signature from UTF8 Test \xa1\xb1\xc1\xd1\xe1\xf1 <jrandom@example.com>',
 
339
                out)
 
340
 
283
341
 
284
342
class TestShortLogFormatter(TestCaseForLogFormatter):
285
343
 
322
380
    1 Joe Foo\t2005-11-22
323
381
      rev-1
324
382
 
325
 
Use --include-merges or -n0 to see merged revisions.
 
383
Use --include-merged or -n0 to see merged revisions.
326
384
""",
327
385
            wt.branch, log.ShortLogFormatter,
328
386
            formatter_kwargs=dict(show_advice=True))
329
387
 
330
388
    def test_short_log_with_merges_and_range(self):
331
389
        wt = self._prepare_tree_with_merges()
332
 
        self.wt_commit(wt, 'rev-3a', rev_id='rev-3a')
333
 
        wt.branch.set_last_revision_info(2, 'rev-2b')
334
 
        wt.set_parent_ids(['rev-2b', 'rev-3a'])
335
 
        self.wt_commit(wt, 'rev-3b', rev_id='rev-3b')
 
390
        self.wt_commit(wt, 'rev-3a', rev_id=b'rev-3a')
 
391
        wt.branch.set_last_revision_info(2, b'rev-2b')
 
392
        wt.set_parent_ids([b'rev-2b', b'rev-3a'])
 
393
        self.wt_commit(wt, 'rev-3b', rev_id=b'rev-3b')
336
394
        self.assertFormatterResult("""\
337
395
    3 Joe Foo\t2005-11-22 [merge]
338
396
      rev-3b
371
429
            wt.branch, log.ShortLogFormatter,
372
430
            show_log_kwargs=dict(start_revision=rev, end_revision=rev))
373
431
 
 
432
    def test_show_ids(self):
 
433
        wt = self.make_branch_and_tree('parent')
 
434
        self.build_tree(['parent/f1', 'parent/f2'])
 
435
        wt.add(['f1', 'f2'])
 
436
        self.wt_commit(wt, 'first post', rev_id=b'a')
 
437
        child_wt = wt.controldir.sprout('child').open_workingtree()
 
438
        self.wt_commit(child_wt, 'branch 1 changes', rev_id=b'b')
 
439
        wt.merge_from_branch(child_wt.branch)
 
440
        self.wt_commit(wt, 'merge branch 1', rev_id=b'c')
 
441
        self.assertFormatterResult("""\
 
442
    2 Joe Foo\t2005-11-22 [merge]
 
443
      revision-id:c
 
444
      merge branch 1
 
445
 
 
446
          1.1.1 Joe Foo\t2005-11-22
 
447
                revision-id:b
 
448
                branch 1 changes
 
449
 
 
450
    1 Joe Foo\t2005-11-22
 
451
      revision-id:a
 
452
      first post
 
453
 
 
454
""",
 
455
            wt.branch, log.ShortLogFormatter,
 
456
            formatter_kwargs=dict(levels=0, show_ids=True))
 
457
 
374
458
 
375
459
class TestShortLogFormatterWithMergeRevisions(TestCaseForLogFormatter):
376
460
 
432
516
    def test_merges_are_indented_by_level(self):
433
517
        wt = self.make_branch_and_tree('parent')
434
518
        self.wt_commit(wt, 'first post')
435
 
        child_wt = wt.bzrdir.sprout('child').open_workingtree()
 
519
        child_wt = wt.controldir.sprout('child').open_workingtree()
436
520
        self.wt_commit(child_wt, 'branch 1')
437
 
        smallerchild_wt = wt.bzrdir.sprout('smallerchild').open_workingtree()
 
521
        smallerchild_wt = wt.controldir.sprout('smallerchild').open_workingtree()
438
522
        self.wt_commit(smallerchild_wt, 'branch 2')
439
523
        child_wt.merge_from_branch(smallerchild_wt.branch)
440
524
        self.wt_commit(child_wt, 'merge branch 2')
484
568
    def test_verbose_merge_revisions_contain_deltas(self):
485
569
        wt = self.make_branch_and_tree('parent')
486
570
        self.build_tree(['parent/f1', 'parent/f2'])
487
 
        wt.add(['f1','f2'])
 
571
        wt.add(['f1', 'f2'])
488
572
        self.wt_commit(wt, 'first post')
489
 
        child_wt = wt.bzrdir.sprout('child').open_workingtree()
 
573
        child_wt = wt.controldir.sprout('child').open_workingtree()
490
574
        os.unlink('child/f1')
491
 
        self.build_tree_contents([('child/f2', 'hello\n')])
 
575
        self.build_tree_contents([('child/f2', b'hello\n')])
492
576
        self.wt_commit(child_wt, 'removed f1 and modified f2')
493
577
        wt.merge_from_branch(child_wt.branch)
494
578
        self.wt_commit(wt, 'merge branch 1')
628
712
        handlers.
629
713
        """
630
714
        wt = self.make_standard_commit('error_in_properties_handler',
631
 
            revprops={'first_prop':'first_value'})
 
715
            revprops={u'first_prop':'first_value'})
632
716
        sio = self.make_utf8_encoded_stringio()
633
717
        formatter = log.LongLogFormatter(to_file=sio)
634
718
        def trivial_custom_prop_handler(revision):
635
 
            raise StandardError("a test error")
 
719
            raise Exception("a test error")
636
720
 
637
721
        log.properties_handler_registry.register(
638
722
            'trivial_custom_prop_handler',
639
723
            trivial_custom_prop_handler)
640
 
        self.assertRaises(StandardError, log.show_log, wt.branch, formatter,)
 
724
        self.assertRaises(Exception, log.show_log, wt.branch, formatter,)
641
725
 
642
726
    def test_properties_handler_bad_argument(self):
643
727
        wt = self.make_standard_commit('bad_argument',
644
 
              revprops={'a_prop':'test_value'})
 
728
              revprops={u'a_prop':'test_value'})
645
729
        sio = self.make_utf8_encoded_stringio()
646
730
        formatter = log.LongLogFormatter(to_file=sio)
647
731
        def bad_argument_prop_handler(revision):
659
743
        self.assertEqualDiff('''custom_prop_name: test_value\n''',
660
744
                             sio.getvalue())
661
745
 
 
746
    def test_show_ids(self):
 
747
        wt = self.make_branch_and_tree('parent')
 
748
        self.build_tree(['parent/f1', 'parent/f2'])
 
749
        wt.add(['f1', 'f2'])
 
750
        self.wt_commit(wt, 'first post', rev_id=b'a')
 
751
        child_wt = wt.controldir.sprout('child').open_workingtree()
 
752
        self.wt_commit(child_wt, 'branch 1 changes', rev_id=b'b')
 
753
        wt.merge_from_branch(child_wt.branch)
 
754
        self.wt_commit(wt, 'merge branch 1', rev_id=b'c')
 
755
        self.assertFormatterResult("""\
 
756
------------------------------------------------------------
 
757
revno: 2 [merge]
 
758
revision-id: c
 
759
parent: a
 
760
parent: b
 
761
committer: Joe Foo <joe@foo.com>
 
762
branch nick: parent
 
763
timestamp: Tue 2005-11-22 00:00:02 +0000
 
764
message:
 
765
  merge branch 1
 
766
    ------------------------------------------------------------
 
767
    revno: 1.1.1
 
768
    revision-id: b
 
769
    parent: a
 
770
    committer: Joe Foo <joe@foo.com>
 
771
    branch nick: child
 
772
    timestamp: Tue 2005-11-22 00:00:01 +0000
 
773
    message:
 
774
      branch 1 changes
 
775
------------------------------------------------------------
 
776
revno: 1
 
777
revision-id: a
 
778
committer: Joe Foo <joe@foo.com>
 
779
branch nick: parent
 
780
timestamp: Tue 2005-11-22 00:00:00 +0000
 
781
message:
 
782
  first post
 
783
""",
 
784
            wt.branch, log.LongLogFormatter,
 
785
            formatter_kwargs=dict(levels=0, show_ids=True))
 
786
 
662
787
 
663
788
class TestLongLogFormatterWithoutMergeRevisions(TestCaseForLogFormatter):
664
789
 
686
811
    def test_long_verbose_contain_deltas(self):
687
812
        wt = self.make_branch_and_tree('parent')
688
813
        self.build_tree(['parent/f1', 'parent/f2'])
689
 
        wt.add(['f1','f2'])
 
814
        wt.add(['f1', 'f2'])
690
815
        self.wt_commit(wt, 'first post')
691
 
        child_wt = wt.bzrdir.sprout('child').open_workingtree()
 
816
        child_wt = wt.controldir.sprout('child').open_workingtree()
692
817
        os.unlink('child/f1')
693
 
        self.build_tree_contents([('child/f2', 'hello\n')])
 
818
        self.build_tree_contents([('child/f2', b'hello\n')])
694
819
        self.wt_commit(child_wt, 'removed f1 and modified f2')
695
820
        wt.merge_from_branch(child_wt.branch)
696
821
        self.wt_commit(wt, 'merge branch 1')
915
1040
            wt.branch, log.GnuChangelogLogFormatter,
916
1041
            show_log_kwargs=dict(verbose=True))
917
1042
 
918
 
class TestGetViewRevisions(tests.TestCaseWithTransport, TestLogMixin):
919
 
 
920
 
    def _get_view_revisions(self, *args, **kwargs):
921
 
        return self.applyDeprecated(symbol_versioning.deprecated_in((2, 2, 0)),
922
 
                                    log.get_view_revisions, *args, **kwargs)
923
 
 
924
 
    def make_tree_with_commits(self):
925
 
        """Create a tree with well-known revision ids"""
926
 
        wt = self.make_branch_and_tree('tree1')
927
 
        self.wt_commit(wt, 'commit one', rev_id='1')
928
 
        self.wt_commit(wt, 'commit two', rev_id='2')
929
 
        self.wt_commit(wt, 'commit three', rev_id='3')
930
 
        mainline_revs = [None, '1', '2', '3']
931
 
        rev_nos = {'1': 1, '2': 2, '3': 3}
932
 
        return mainline_revs, rev_nos, wt
933
 
 
934
 
    def make_tree_with_merges(self):
935
 
        """Create a tree with well-known revision ids and a merge"""
936
 
        mainline_revs, rev_nos, wt = self.make_tree_with_commits()
937
 
        tree2 = wt.bzrdir.sprout('tree2').open_workingtree()
938
 
        self.wt_commit(tree2, 'four-a', rev_id='4a')
939
 
        wt.merge_from_branch(tree2.branch)
940
 
        self.wt_commit(wt, 'four-b', rev_id='4b')
941
 
        mainline_revs.append('4b')
942
 
        rev_nos['4b'] = 4
943
 
        # 4a: 3.1.1
944
 
        return mainline_revs, rev_nos, wt
945
 
 
946
 
    def make_branch_with_many_merges(self):
947
 
        """Create a tree with well-known revision ids"""
948
 
        builder = self.make_branch_builder('tree1')
949
 
        builder.start_series()
950
 
        builder.build_snapshot('1', None, [
951
 
            ('add', ('', 'TREE_ROOT', 'directory', '')),
952
 
            ('add', ('f', 'f-id', 'file', '1\n'))])
953
 
        builder.build_snapshot('2', ['1'], [])
954
 
        builder.build_snapshot('3a', ['2'], [
955
 
            ('modify', ('f-id', '1\n2\n3a\n'))])
956
 
        builder.build_snapshot('3b', ['2', '3a'], [
957
 
            ('modify', ('f-id', '1\n2\n3a\n'))])
958
 
        builder.build_snapshot('3c', ['2', '3b'], [
959
 
            ('modify', ('f-id', '1\n2\n3a\n'))])
960
 
        builder.build_snapshot('4a', ['3b'], [])
961
 
        builder.build_snapshot('4b', ['3c', '4a'], [])
962
 
        builder.finish_series()
963
 
 
964
 
        # 1
965
 
        # |
966
 
        # 2-.
967
 
        # |\ \
968
 
        # | | 3a
969
 
        # | |/
970
 
        # | 3b
971
 
        # |/|
972
 
        # 3c4a
973
 
        # |/
974
 
        # 4b
975
 
 
976
 
        mainline_revs = [None, '1', '2', '3c', '4b']
977
 
        rev_nos = {'1':1, '2':2, '3c': 3, '4b':4}
978
 
        full_rev_nos_for_reference = {
979
 
            '1': '1',
980
 
            '2': '2',
981
 
            '3a': '2.1.1', #first commit tree 3
982
 
            '3b': '2.2.1', # first commit tree 2
983
 
            '3c': '3', #merges 3b to main
984
 
            '4a': '2.2.2', # second commit tree 2
985
 
            '4b': '4', # merges 4a to main
986
 
            }
987
 
        return mainline_revs, rev_nos, builder.get_branch()
988
 
 
989
 
    def test_get_view_revisions_forward(self):
990
 
        """Test the get_view_revisions method"""
991
 
        mainline_revs, rev_nos, wt = self.make_tree_with_commits()
992
 
        wt.lock_read()
993
 
        self.addCleanup(wt.unlock)
994
 
        revisions = list(self._get_view_revisions(
995
 
                mainline_revs, rev_nos, wt.branch, 'forward'))
996
 
        self.assertEqual([('1', '1', 0), ('2', '2', 0), ('3', '3', 0)],
997
 
                         revisions)
998
 
        revisions2 = list(self._get_view_revisions(
999
 
                mainline_revs, rev_nos, wt.branch, 'forward',
1000
 
                include_merges=False))
1001
 
        self.assertEqual(revisions, revisions2)
1002
 
 
1003
 
    def test_get_view_revisions_reverse(self):
1004
 
        """Test the get_view_revisions with reverse"""
1005
 
        mainline_revs, rev_nos, wt = self.make_tree_with_commits()
1006
 
        wt.lock_read()
1007
 
        self.addCleanup(wt.unlock)
1008
 
        revisions = list(self._get_view_revisions(
1009
 
                mainline_revs, rev_nos, wt.branch, 'reverse'))
1010
 
        self.assertEqual([('3', '3', 0), ('2', '2', 0), ('1', '1', 0), ],
1011
 
                         revisions)
1012
 
        revisions2 = list(self._get_view_revisions(
1013
 
                mainline_revs, rev_nos, wt.branch, 'reverse',
1014
 
                include_merges=False))
1015
 
        self.assertEqual(revisions, revisions2)
1016
 
 
1017
 
    def test_get_view_revisions_merge(self):
1018
 
        """Test get_view_revisions when there are merges"""
1019
 
        mainline_revs, rev_nos, wt = self.make_tree_with_merges()
1020
 
        wt.lock_read()
1021
 
        self.addCleanup(wt.unlock)
1022
 
        revisions = list(self._get_view_revisions(
1023
 
                mainline_revs, rev_nos, wt.branch, 'forward'))
1024
 
        self.assertEqual([('1', '1', 0), ('2', '2', 0), ('3', '3', 0),
1025
 
                          ('4b', '4', 0), ('4a', '3.1.1', 1)],
1026
 
                         revisions)
1027
 
        revisions = list(self._get_view_revisions(
1028
 
                mainline_revs, rev_nos, wt.branch, 'forward',
1029
 
                include_merges=False))
1030
 
        self.assertEqual([('1', '1', 0), ('2', '2', 0), ('3', '3', 0),
1031
 
                          ('4b', '4', 0)],
1032
 
                         revisions)
1033
 
 
1034
 
    def test_get_view_revisions_merge_reverse(self):
1035
 
        """Test get_view_revisions in reverse when there are merges"""
1036
 
        mainline_revs, rev_nos, wt = self.make_tree_with_merges()
1037
 
        wt.lock_read()
1038
 
        self.addCleanup(wt.unlock)
1039
 
        revisions = list(self._get_view_revisions(
1040
 
                mainline_revs, rev_nos, wt.branch, 'reverse'))
1041
 
        self.assertEqual([('4b', '4', 0), ('4a', '3.1.1', 1),
1042
 
                          ('3', '3', 0), ('2', '2', 0), ('1', '1', 0)],
1043
 
                         revisions)
1044
 
        revisions = list(self._get_view_revisions(
1045
 
                mainline_revs, rev_nos, wt.branch, 'reverse',
1046
 
                include_merges=False))
1047
 
        self.assertEqual([('4b', '4', 0), ('3', '3', 0), ('2', '2', 0),
1048
 
                          ('1', '1', 0)],
1049
 
                         revisions)
1050
 
 
1051
 
    def test_get_view_revisions_merge2(self):
1052
 
        """Test get_view_revisions when there are merges"""
1053
 
        mainline_revs, rev_nos, b = self.make_branch_with_many_merges()
1054
 
        b.lock_read()
1055
 
        self.addCleanup(b.unlock)
1056
 
        revisions = list(self._get_view_revisions(
1057
 
                mainline_revs, rev_nos, b, 'forward'))
1058
 
        expected = [('1', '1', 0), ('2', '2', 0), ('3c', '3', 0),
1059
 
                    ('3b', '2.2.1', 1), ('3a', '2.1.1', 2), ('4b', '4', 0),
1060
 
                    ('4a', '2.2.2', 1)]
1061
 
        self.assertEqual(expected, revisions)
1062
 
        revisions = list(self._get_view_revisions(
1063
 
                mainline_revs, rev_nos, b, 'forward',
1064
 
                include_merges=False))
1065
 
        self.assertEqual([('1', '1', 0), ('2', '2', 0), ('3c', '3', 0),
1066
 
                          ('4b', '4', 0)],
1067
 
                         revisions)
1068
 
 
1069
 
    def test_file_id_for_range(self):
1070
 
        mainline_revs, rev_nos, b = self.make_branch_with_many_merges()
1071
 
        b.lock_read()
1072
 
        self.addCleanup(b.unlock)
1073
 
 
1074
 
        def rev_from_rev_id(revid, branch):
1075
 
            revspec = revisionspec.RevisionSpec.from_string('revid:%s' % revid)
1076
 
            return revspec.in_history(branch)
1077
 
 
1078
 
        def view_revs(start_rev, end_rev, file_id, direction):
1079
 
            revs = self.applyDeprecated(
1080
 
                symbol_versioning.deprecated_in((2, 2, 0)),
1081
 
                log.calculate_view_revisions,
1082
 
                b,
1083
 
                start_rev, # start_revision
1084
 
                end_rev, # end_revision
1085
 
                direction, # direction
1086
 
                file_id, # specific_fileid
1087
 
                True, # generate_merge_revisions
1088
 
                )
1089
 
            return revs
1090
 
 
1091
 
        rev_3a = rev_from_rev_id('3a', b)
1092
 
        rev_4b = rev_from_rev_id('4b', b)
1093
 
        self.assertEqual([('3c', '3', 0), ('3b', '2.2.1', 1),
1094
 
                          ('3a', '2.1.1', 2)],
1095
 
                          view_revs(rev_3a, rev_4b, 'f-id', 'reverse'))
1096
 
        # Note: 3c still appears before 3a here because of depth-based sorting
1097
 
        self.assertEqual([('3c', '3', 0), ('3b', '2.2.1', 1),
1098
 
                          ('3a', '2.1.1', 2)],
1099
 
                          view_revs(rev_3a, rev_4b, 'f-id', 'forward'))
1100
 
 
1101
 
 
1102
 
class TestGetRevisionsTouchingFileID(tests.TestCaseWithTransport):
1103
 
 
1104
 
    def get_view_revisions(self, *args):
1105
 
        return self.applyDeprecated(symbol_versioning.deprecated_in((2, 2, 0)),
1106
 
                                    log.get_view_revisions, *args)
1107
 
 
1108
 
    def create_tree_with_single_merge(self):
1109
 
        """Create a branch with a moderate layout.
1110
 
 
1111
 
        The revision graph looks like:
1112
 
 
1113
 
           A
1114
 
           |\
1115
 
           B C
1116
 
           |/
1117
 
           D
1118
 
 
1119
 
        In this graph, A introduced files f1 and f2 and f3.
1120
 
        B modifies f1 and f3, and C modifies f2 and f3.
1121
 
        D merges the changes from B and C and resolves the conflict for f3.
1122
 
        """
1123
 
        # TODO: jam 20070218 This seems like it could really be done
1124
 
        #       with make_branch_and_memory_tree() if we could just
1125
 
        #       create the content of those files.
1126
 
        # TODO: jam 20070218 Another alternative is that we would really
1127
 
        #       like to only create this tree 1 time for all tests that
1128
 
        #       use it. Since 'log' only uses the tree in a readonly
1129
 
        #       fashion, it seems a shame to regenerate an identical
1130
 
        #       tree for each test.
1131
 
        # TODO: vila 20100122 One way to address the shame above will be to
1132
 
        #       create a memory tree during test parametrization and give a
1133
 
        #       *copy* of this tree to each test. Copying a memory tree ought
1134
 
        #       to be cheap, at least cheaper than creating them with such
1135
 
        #       complex setups.
1136
 
        tree = self.make_branch_and_tree('tree')
1137
 
        tree.lock_write()
1138
 
        self.addCleanup(tree.unlock)
1139
 
 
1140
 
        self.build_tree_contents([('tree/f1', 'A\n'),
1141
 
                                  ('tree/f2', 'A\n'),
1142
 
                                  ('tree/f3', 'A\n'),
1143
 
                                 ])
1144
 
        tree.add(['f1', 'f2', 'f3'], ['f1-id', 'f2-id', 'f3-id'])
1145
 
        tree.commit('A', rev_id='A')
1146
 
 
1147
 
        self.build_tree_contents([('tree/f2', 'A\nC\n'),
1148
 
                                  ('tree/f3', 'A\nC\n'),
1149
 
                                 ])
1150
 
        tree.commit('C', rev_id='C')
1151
 
        # Revert back to A to build the other history.
1152
 
        tree.set_last_revision('A')
1153
 
        tree.branch.set_last_revision_info(1, 'A')
1154
 
        self.build_tree_contents([('tree/f1', 'A\nB\n'),
1155
 
                                  ('tree/f2', 'A\n'),
1156
 
                                  ('tree/f3', 'A\nB\n'),
1157
 
                                 ])
1158
 
        tree.commit('B', rev_id='B')
1159
 
        tree.set_parent_ids(['B', 'C'])
1160
 
        self.build_tree_contents([('tree/f1', 'A\nB\n'),
1161
 
                                  ('tree/f2', 'A\nC\n'),
1162
 
                                  ('tree/f3', 'A\nB\nC\n'),
1163
 
                                 ])
1164
 
        tree.commit('D', rev_id='D')
1165
 
 
1166
 
        # Switch to a read lock for this tree.
1167
 
        # We still have an addCleanup(tree.unlock) pending
1168
 
        tree.unlock()
1169
 
        tree.lock_read()
1170
 
        return tree
1171
 
 
1172
 
    def check_delta(self, delta, **kw):
1173
 
        """Check the filenames touched by a delta are as expected.
1174
 
 
1175
 
        Caller only have to pass in the list of files for each part, all
1176
 
        unspecified parts are considered empty (and checked as such).
1177
 
        """
1178
 
        for n in 'added', 'removed', 'renamed', 'modified', 'unchanged':
1179
 
            # By default we expect an empty list
1180
 
            expected = kw.get(n, [])
1181
 
            # strip out only the path components
1182
 
            got = [x[0] for x in getattr(delta, n)]
1183
 
            self.assertEqual(expected, got)
1184
 
 
1185
 
    def test_tree_with_single_merge(self):
1186
 
        """Make sure the tree layout is correct."""
1187
 
        tree = self.create_tree_with_single_merge()
1188
 
        rev_A_tree = tree.branch.repository.revision_tree('A')
1189
 
        rev_B_tree = tree.branch.repository.revision_tree('B')
1190
 
        rev_C_tree = tree.branch.repository.revision_tree('C')
1191
 
        rev_D_tree = tree.branch.repository.revision_tree('D')
1192
 
 
1193
 
        self.check_delta(rev_B_tree.changes_from(rev_A_tree),
1194
 
                         modified=['f1', 'f3'])
1195
 
 
1196
 
        self.check_delta(rev_C_tree.changes_from(rev_A_tree),
1197
 
                         modified=['f2', 'f3'])
1198
 
 
1199
 
        self.check_delta(rev_D_tree.changes_from(rev_B_tree),
1200
 
                         modified=['f2', 'f3'])
1201
 
 
1202
 
        self.check_delta(rev_D_tree.changes_from(rev_C_tree),
1203
 
                         modified=['f1', 'f3'])
1204
 
 
1205
 
    def assertAllRevisionsForFileID(self, tree, file_id, revisions):
1206
 
        """Ensure _filter_revisions_touching_file_id returns the right values.
1207
 
 
1208
 
        Get the return value from _filter_revisions_touching_file_id and make
1209
 
        sure they are correct.
1210
 
        """
1211
 
        # The api for _filter_revisions_touching_file_id is a little crazy.
1212
 
        # So we do the setup here.
1213
 
        mainline = tree.branch.revision_history()
1214
 
        mainline.insert(0, None)
1215
 
        revnos = dict((rev, idx+1) for idx, rev in enumerate(mainline))
1216
 
        view_revs_iter = self.get_view_revisions(
1217
 
            mainline, revnos, tree.branch, 'reverse', True)
1218
 
        actual_revs = log._filter_revisions_touching_file_id(
1219
 
            tree.branch, file_id, list(view_revs_iter))
1220
 
        self.assertEqual(revisions, [r for r, revno, depth in actual_revs])
1221
 
 
1222
 
    def test_file_id_f1(self):
1223
 
        tree = self.create_tree_with_single_merge()
1224
 
        # f1 should be marked as modified by revisions A and B
1225
 
        self.assertAllRevisionsForFileID(tree, 'f1-id', ['B', 'A'])
1226
 
 
1227
 
    def test_file_id_f2(self):
1228
 
        tree = self.create_tree_with_single_merge()
1229
 
        # f2 should be marked as modified by revisions A, C, and D
1230
 
        # because D merged the changes from C.
1231
 
        self.assertAllRevisionsForFileID(tree, 'f2-id', ['D', 'C', 'A'])
1232
 
 
1233
 
    def test_file_id_f3(self):
1234
 
        tree = self.create_tree_with_single_merge()
1235
 
        # f3 should be marked as modified by revisions A, B, C, and D
1236
 
        self.assertAllRevisionsForFileID(tree, 'f3-id', ['D', 'C', 'B', 'A'])
1237
 
 
1238
 
    def test_file_id_with_ghosts(self):
1239
 
        # This is testing bug #209948, where having a ghost would cause
1240
 
        # _filter_revisions_touching_file_id() to fail.
1241
 
        tree = self.create_tree_with_single_merge()
1242
 
        # We need to add a revision, so switch back to a write-locked tree
1243
 
        # (still a single addCleanup(tree.unlock) pending).
1244
 
        tree.unlock()
1245
 
        tree.lock_write()
1246
 
        first_parent = tree.last_revision()
1247
 
        tree.set_parent_ids([first_parent, 'ghost-revision-id'])
1248
 
        self.build_tree_contents([('tree/f1', 'A\nB\nXX\n')])
1249
 
        tree.commit('commit with a ghost', rev_id='XX')
1250
 
        self.assertAllRevisionsForFileID(tree, 'f1-id', ['XX', 'B', 'A'])
1251
 
        self.assertAllRevisionsForFileID(tree, 'f2-id', ['D', 'C', 'A'])
1252
 
 
1253
 
    def test_unknown_file_id(self):
1254
 
        tree = self.create_tree_with_single_merge()
1255
 
        self.assertAllRevisionsForFileID(tree, 'unknown', [])
1256
 
 
1257
 
    def test_empty_branch_unknown_file_id(self):
1258
 
        tree = self.make_branch_and_tree('tree')
1259
 
        self.assertAllRevisionsForFileID(tree, 'unknown', [])
1260
 
 
1261
1043
 
1262
1044
class TestShowChangedRevisions(tests.TestCaseWithTransport):
1263
1045
 
1265
1047
        tree = self.make_branch_and_tree('tree_a')
1266
1048
        self.build_tree(['tree_a/foo'])
1267
1049
        tree.add('foo')
1268
 
        tree.commit('bar', rev_id='bar-id')
 
1050
        tree.commit('bar', rev_id=b'bar-id')
1269
1051
        s = self.make_utf8_encoded_stringio()
1270
 
        log.show_changed_revisions(tree.branch, [], ['bar-id'], s)
 
1052
        log.show_changed_revisions(tree.branch, [], [b'bar-id'], s)
1271
1053
        self.assertContainsRe(s.getvalue(), 'bar')
1272
1054
        self.assertNotContainsRe(s.getvalue(), 'foo')
1273
1055
 
1276
1058
 
1277
1059
    def setUp(self):
1278
1060
        super(TestLogFormatter, self).setUp()
1279
 
        self.rev = revision.Revision('a-id')
 
1061
        self.rev = revision.Revision(b'a-id')
1280
1062
        self.lf = log.LogFormatter(None)
1281
1063
 
1282
1064
    def test_short_committer(self):
1372
1154
        tree = self.make_branch_and_tree('tree')
1373
1155
        tree.lock_write()
1374
1156
        self.addCleanup(tree.unlock)
1375
 
        tree.commit('1a', rev_id='1a')
1376
 
        tree.commit('2a', rev_id='2a')
1377
 
        tree.commit('3a', rev_id='3a')
 
1157
        tree.commit('1a', rev_id=b'1a')
 
1158
        tree.commit('2a', rev_id=b'2a')
 
1159
        tree.commit('3a', rev_id=b'3a')
1378
1160
        return tree
1379
1161
 
1380
1162
    def setup_ab_tree(self):
1381
1163
        tree = self.setup_a_tree()
1382
 
        tree.set_last_revision('1a')
1383
 
        tree.branch.set_last_revision_info(1, '1a')
1384
 
        tree.commit('2b', rev_id='2b')
1385
 
        tree.commit('3b', rev_id='3b')
 
1164
        tree.set_last_revision(b'1a')
 
1165
        tree.branch.set_last_revision_info(1, b'1a')
 
1166
        tree.commit('2b', rev_id=b'2b')
 
1167
        tree.commit('3b', rev_id=b'3b')
1386
1168
        return tree
1387
1169
 
1388
1170
    def setup_ac_tree(self):
1389
1171
        tree = self.setup_a_tree()
1390
1172
        tree.set_last_revision(revision.NULL_REVISION)
1391
1173
        tree.branch.set_last_revision_info(0, revision.NULL_REVISION)
1392
 
        tree.commit('1c', rev_id='1c')
1393
 
        tree.commit('2c', rev_id='2c')
1394
 
        tree.commit('3c', rev_id='3c')
 
1174
        tree.commit('1c', rev_id=b'1c')
 
1175
        tree.commit('2c', rev_id=b'2c')
 
1176
        tree.commit('3c', rev_id=b'3c')
1395
1177
        return tree
1396
1178
 
1397
1179
    def test_all_new(self):
1434
1216
 
1435
1217
    def test_show_branch_change(self):
1436
1218
        tree = self.setup_ab_tree()
1437
 
        s = StringIO()
 
1219
        s = BytesIO()
1438
1220
        log.show_branch_change(tree.branch, s, 3, '3a')
1439
1221
        self.assertContainsRe(s.getvalue(),
1440
1222
            '[*]{60}\nRemoved Revisions:\n(.|\n)*2a(.|\n)*3a(.|\n)*'
1442
1224
 
1443
1225
    def test_show_branch_change_no_change(self):
1444
1226
        tree = self.setup_ab_tree()
1445
 
        s = StringIO()
 
1227
        s = BytesIO()
1446
1228
        log.show_branch_change(tree.branch, s, 3, '3b')
1447
1229
        self.assertEqual(s.getvalue(),
1448
1230
            'Nothing seems to have changed\n')
1449
1231
 
1450
1232
    def test_show_branch_change_no_old(self):
1451
1233
        tree = self.setup_ab_tree()
1452
 
        s = StringIO()
 
1234
        s = BytesIO()
1453
1235
        log.show_branch_change(tree.branch, s, 2, '2b')
1454
1236
        self.assertContainsRe(s.getvalue(), 'Added Revisions:')
1455
1237
        self.assertNotContainsRe(s.getvalue(), 'Removed Revisions:')
1456
1238
 
1457
1239
    def test_show_branch_change_no_new(self):
1458
1240
        tree = self.setup_ab_tree()
1459
 
        tree.branch.set_last_revision_info(2, '2b')
1460
 
        s = StringIO()
 
1241
        tree.branch.set_last_revision_info(2, b'2b')
 
1242
        s = BytesIO()
1461
1243
        log.show_branch_change(tree.branch, s, 3, '3b')
1462
1244
        self.assertContainsRe(s.getvalue(), 'Removed Revisions:')
1463
1245
        self.assertNotContainsRe(s.getvalue(), 'Added Revisions:')
1464
1246
 
1465
1247
 
 
1248
class TestRevisionNotInBranch(TestCaseForLogFormatter):
 
1249
 
 
1250
    def setup_a_tree(self):
 
1251
        tree = self.make_branch_and_tree('tree')
 
1252
        tree.lock_write()
 
1253
        self.addCleanup(tree.unlock)
 
1254
        kwargs = {
 
1255
            'committer': 'Joe Foo <joe@foo.com>',
 
1256
            'timestamp': 1132617600, # Mon 2005-11-22 00:00:00 +0000
 
1257
            'timezone': 0, # UTC
 
1258
        }
 
1259
        tree.commit('commit 1a', rev_id=b'1a', **kwargs)
 
1260
        tree.commit('commit 2a', rev_id=b'2a', **kwargs)
 
1261
        tree.commit('commit 3a', rev_id=b'3a', **kwargs)
 
1262
        return tree
 
1263
 
 
1264
    def setup_ab_tree(self):
 
1265
        tree = self.setup_a_tree()
 
1266
        tree.set_last_revision(b'1a')
 
1267
        tree.branch.set_last_revision_info(1, b'1a')
 
1268
        kwargs = {
 
1269
            'committer': 'Joe Foo <joe@foo.com>',
 
1270
            'timestamp': 1132617600, # Mon 2005-11-22 00:00:00 +0000
 
1271
            'timezone': 0, # UTC
 
1272
        }
 
1273
        tree.commit('commit 2b', rev_id=b'2b', **kwargs)
 
1274
        tree.commit('commit 3b', rev_id=b'3b', **kwargs)
 
1275
        return tree
 
1276
 
 
1277
    def test_one_revision(self):
 
1278
        tree = self.setup_ab_tree()
 
1279
        lf = LogCatcher()
 
1280
        rev = revisionspec.RevisionInfo(tree.branch, None, '3a')
 
1281
        log.show_log(tree.branch, lf, verbose=True, start_revision=rev,
 
1282
                     end_revision=rev)
 
1283
        self.assertEqual(1, len(lf.revisions))
 
1284
        self.assertEqual(None, lf.revisions[0].revno)   # Out-of-branch
 
1285
        self.assertEqual('3a', lf.revisions[0].rev.revision_id)
 
1286
 
 
1287
    def test_many_revisions(self):
 
1288
        tree = self.setup_ab_tree()
 
1289
        lf = LogCatcher()
 
1290
        start_rev = revisionspec.RevisionInfo(tree.branch, None, '1a')
 
1291
        end_rev = revisionspec.RevisionInfo(tree.branch, None, '3a')
 
1292
        log.show_log(tree.branch, lf, verbose=True, start_revision=start_rev,
 
1293
                     end_revision=end_rev)
 
1294
        self.assertEqual(3, len(lf.revisions))
 
1295
        self.assertEqual(None, lf.revisions[0].revno)   # Out-of-branch
 
1296
        self.assertEqual('3a', lf.revisions[0].rev.revision_id)
 
1297
        self.assertEqual(None, lf.revisions[1].revno)   # Out-of-branch
 
1298
        self.assertEqual('2a', lf.revisions[1].rev.revision_id)
 
1299
        self.assertEqual('1', lf.revisions[2].revno)    # In-branch
 
1300
 
 
1301
    def test_long_format(self):
 
1302
        tree = self.setup_ab_tree()
 
1303
        start_rev = revisionspec.RevisionInfo(tree.branch, None, '1a')
 
1304
        end_rev = revisionspec.RevisionInfo(tree.branch, None, '3a')
 
1305
        self.assertFormatterResult("""\
 
1306
------------------------------------------------------------
 
1307
revision-id: 3a
 
1308
committer: Joe Foo <joe@foo.com>
 
1309
branch nick: tree
 
1310
timestamp: Tue 2005-11-22 00:00:00 +0000
 
1311
message:
 
1312
  commit 3a
 
1313
------------------------------------------------------------
 
1314
revision-id: 2a
 
1315
committer: Joe Foo <joe@foo.com>
 
1316
branch nick: tree
 
1317
timestamp: Tue 2005-11-22 00:00:00 +0000
 
1318
message:
 
1319
  commit 2a
 
1320
------------------------------------------------------------
 
1321
revno: 1
 
1322
committer: Joe Foo <joe@foo.com>
 
1323
branch nick: tree
 
1324
timestamp: Tue 2005-11-22 00:00:00 +0000
 
1325
message:
 
1326
  commit 1a
 
1327
""",
 
1328
            tree.branch, log.LongLogFormatter, show_log_kwargs={
 
1329
                'start_revision': start_rev, 'end_revision': end_rev
 
1330
            })
 
1331
 
 
1332
    def test_short_format(self):
 
1333
        tree = self.setup_ab_tree()
 
1334
        start_rev = revisionspec.RevisionInfo(tree.branch, None, '1a')
 
1335
        end_rev = revisionspec.RevisionInfo(tree.branch, None, '3a')
 
1336
        self.assertFormatterResult("""\
 
1337
      Joe Foo\t2005-11-22
 
1338
      revision-id:3a
 
1339
      commit 3a
 
1340
 
 
1341
      Joe Foo\t2005-11-22
 
1342
      revision-id:2a
 
1343
      commit 2a
 
1344
 
 
1345
    1 Joe Foo\t2005-11-22
 
1346
      commit 1a
 
1347
 
 
1348
""",
 
1349
            tree.branch, log.ShortLogFormatter, show_log_kwargs={
 
1350
                'start_revision': start_rev, 'end_revision': end_rev
 
1351
            })
 
1352
 
 
1353
    def test_line_format(self):
 
1354
        tree = self.setup_ab_tree()
 
1355
        start_rev = revisionspec.RevisionInfo(tree.branch, None, '1a')
 
1356
        end_rev = revisionspec.RevisionInfo(tree.branch, None, '3a')
 
1357
        self.assertFormatterResult("""\
 
1358
Joe Foo 2005-11-22 commit 3a
 
1359
Joe Foo 2005-11-22 commit 2a
 
1360
1: Joe Foo 2005-11-22 commit 1a
 
1361
""",
 
1362
            tree.branch, log.LineLogFormatter, show_log_kwargs={
 
1363
                'start_revision': start_rev, 'end_revision': end_rev
 
1364
            })
 
1365
 
1466
1366
 
1467
1367
class TestLogWithBugs(TestCaseForLogFormatter, TestLogMixin):
1468
1368
 
1469
1369
    def setUp(self):
1470
 
        TestCaseForLogFormatter.setUp(self)
 
1370
        super(TestLogWithBugs, self).setUp()
1471
1371
        log.properties_handler_registry.register(
1472
1372
            'bugs_properties_handler',
1473
1373
            log._bugs_properties_handler)
1477
1377
        tree = self.make_branch_and_tree(u'.')
1478
1378
        self.build_tree(['a', 'b'])
1479
1379
        tree.add('a')
1480
 
        self.wt_commit(tree, 'simple log message', rev_id='a1',
1481
 
                       revprops={'bugs': 'test://bug/id fixed'})
 
1380
        self.wt_commit(tree, 'simple log message', rev_id=b'a1',
 
1381
                       revprops={u'bugs': 'test://bug/id fixed'})
1482
1382
        tree.add('b')
1483
 
        self.wt_commit(tree, 'multiline\nlog\nmessage\n', rev_id='a2',
 
1383
        self.wt_commit(tree, 'multiline\nlog\nmessage\n', rev_id=b'a2',
1484
1384
                       authors=['Joe Bar <joe@bar.com>'],
1485
 
                       revprops={'bugs': 'test://bug/id fixed\n'
 
1385
                       revprops={u'bugs': 'test://bug/id fixed\n'
1486
1386
                                 'test://bug/2 fixed'})
1487
1387
        return tree
1488
1388
 
1492
1392
        self.assertFormatterResult("""\
1493
1393
------------------------------------------------------------
1494
1394
revno: 2
1495
 
fixes bug(s): test://bug/id test://bug/2
 
1395
fixes bugs: test://bug/id test://bug/2
1496
1396
author: Joe Bar <joe@bar.com>
1497
1397
committer: Joe Foo <joe@foo.com>
1498
1398
branch nick: work
1503
1403
  message
1504
1404
------------------------------------------------------------
1505
1405
revno: 1
1506
 
fixes bug(s): test://bug/id
 
1406
fixes bug: test://bug/id
1507
1407
committer: Joe Foo <joe@foo.com>
1508
1408
branch nick: work
1509
1409
timestamp: Tue 2005-11-22 00:00:00 +0000
1516
1416
        tree = self.make_commits_with_bugs()
1517
1417
        self.assertFormatterResult("""\
1518
1418
    2 Joe Bar\t2005-11-22
1519
 
      fixes bug(s): test://bug/id test://bug/2
 
1419
      fixes bugs: test://bug/id test://bug/2
1520
1420
      multiline
1521
1421
      log
1522
1422
      message
1523
1423
 
1524
1424
    1 Joe Foo\t2005-11-22
1525
 
      fixes bug(s): test://bug/id
 
1425
      fixes bug: test://bug/id
1526
1426
      simple log message
1527
1427
 
1528
1428
""",
1531
1431
    def test_wrong_bugs_property(self):
1532
1432
        tree = self.make_branch_and_tree(u'.')
1533
1433
        self.build_tree(['foo'])
1534
 
        self.wt_commit(tree, 'simple log message', rev_id='a1',
1535
 
                       revprops={'bugs': 'test://bug/id invalid_value'})
 
1434
        self.wt_commit(tree, 'simple log message', rev_id=b'a1',
 
1435
                       revprops={u'bugs': 'test://bug/id invalid_value'})
1536
1436
        self.assertFormatterResult("""\
1537
1437
    1 Joe Foo\t2005-11-22
1538
1438
      simple log message
1543
1443
    def test_bugs_handler_present(self):
1544
1444
        self.properties_handler_registry.get('bugs_properties_handler')
1545
1445
 
 
1446
 
 
1447
class TestLogForAuthors(TestCaseForLogFormatter):
 
1448
 
 
1449
    def setUp(self):
 
1450
        super(TestLogForAuthors, self).setUp()
 
1451
        self.wt = self.make_standard_commit('nicky',
 
1452
            authors=['John Doe <jdoe@example.com>',
 
1453
                     'Jane Rey <jrey@example.com>'])
 
1454
 
 
1455
    def assertFormatterResult(self, formatter, who, result):
 
1456
        formatter_kwargs = dict()
 
1457
        if who is not None:
 
1458
            author_list_handler = log.author_list_registry.get(who)
 
1459
            formatter_kwargs['author_list_handler'] = author_list_handler
 
1460
        TestCaseForLogFormatter.assertFormatterResult(self, result,
 
1461
            self.wt.branch, formatter, formatter_kwargs=formatter_kwargs)
 
1462
 
 
1463
    def test_line_default(self):
 
1464
        self.assertFormatterResult(log.LineLogFormatter, None, """\
 
1465
1: John Doe 2005-11-22 add a
 
1466
""")
 
1467
 
 
1468
    def test_line_committer(self):
 
1469
        self.assertFormatterResult(log.LineLogFormatter, 'committer', """\
 
1470
1: Lorem Ipsum 2005-11-22 add a
 
1471
""")
 
1472
 
 
1473
    def test_line_first(self):
 
1474
        self.assertFormatterResult(log.LineLogFormatter, 'first', """\
 
1475
1: John Doe 2005-11-22 add a
 
1476
""")
 
1477
 
 
1478
    def test_line_all(self):
 
1479
        self.assertFormatterResult(log.LineLogFormatter, 'all', """\
 
1480
1: John Doe, Jane Rey 2005-11-22 add a
 
1481
""")
 
1482
 
 
1483
 
 
1484
    def test_short_default(self):
 
1485
        self.assertFormatterResult(log.ShortLogFormatter, None, """\
 
1486
    1 John Doe\t2005-11-22
 
1487
      add a
 
1488
 
 
1489
""")
 
1490
 
 
1491
    def test_short_committer(self):
 
1492
        self.assertFormatterResult(log.ShortLogFormatter, 'committer', """\
 
1493
    1 Lorem Ipsum\t2005-11-22
 
1494
      add a
 
1495
 
 
1496
""")
 
1497
 
 
1498
    def test_short_first(self):
 
1499
        self.assertFormatterResult(log.ShortLogFormatter, 'first', """\
 
1500
    1 John Doe\t2005-11-22
 
1501
      add a
 
1502
 
 
1503
""")
 
1504
 
 
1505
    def test_short_all(self):
 
1506
        self.assertFormatterResult(log.ShortLogFormatter, 'all', """\
 
1507
    1 John Doe, Jane Rey\t2005-11-22
 
1508
      add a
 
1509
 
 
1510
""")
 
1511
 
 
1512
    def test_long_default(self):
 
1513
        self.assertFormatterResult(log.LongLogFormatter, None, """\
 
1514
------------------------------------------------------------
 
1515
revno: 1
 
1516
author: John Doe <jdoe@example.com>, Jane Rey <jrey@example.com>
 
1517
committer: Lorem Ipsum <test@example.com>
 
1518
branch nick: nicky
 
1519
timestamp: Tue 2005-11-22 00:00:00 +0000
 
1520
message:
 
1521
  add a
 
1522
""")
 
1523
 
 
1524
    def test_long_committer(self):
 
1525
        self.assertFormatterResult(log.LongLogFormatter, 'committer', """\
 
1526
------------------------------------------------------------
 
1527
revno: 1
 
1528
committer: Lorem Ipsum <test@example.com>
 
1529
branch nick: nicky
 
1530
timestamp: Tue 2005-11-22 00:00:00 +0000
 
1531
message:
 
1532
  add a
 
1533
""")
 
1534
 
 
1535
    def test_long_first(self):
 
1536
        self.assertFormatterResult(log.LongLogFormatter, 'first', """\
 
1537
------------------------------------------------------------
 
1538
revno: 1
 
1539
author: John Doe <jdoe@example.com>
 
1540
committer: Lorem Ipsum <test@example.com>
 
1541
branch nick: nicky
 
1542
timestamp: Tue 2005-11-22 00:00:00 +0000
 
1543
message:
 
1544
  add a
 
1545
""")
 
1546
 
 
1547
    def test_long_all(self):
 
1548
        self.assertFormatterResult(log.LongLogFormatter, 'all', """\
 
1549
------------------------------------------------------------
 
1550
revno: 1
 
1551
author: John Doe <jdoe@example.com>, Jane Rey <jrey@example.com>
 
1552
committer: Lorem Ipsum <test@example.com>
 
1553
branch nick: nicky
 
1554
timestamp: Tue 2005-11-22 00:00:00 +0000
 
1555
message:
 
1556
  add a
 
1557
""")
 
1558
 
 
1559
    def test_gnu_changelog_default(self):
 
1560
        self.assertFormatterResult(log.GnuChangelogLogFormatter, None, """\
 
1561
2005-11-22  John Doe  <jdoe@example.com>
 
1562
 
 
1563
\tadd a
 
1564
 
 
1565
""")
 
1566
 
 
1567
    def test_gnu_changelog_committer(self):
 
1568
        self.assertFormatterResult(log.GnuChangelogLogFormatter, 'committer', """\
 
1569
2005-11-22  Lorem Ipsum  <test@example.com>
 
1570
 
 
1571
\tadd a
 
1572
 
 
1573
""")
 
1574
 
 
1575
    def test_gnu_changelog_first(self):
 
1576
        self.assertFormatterResult(log.GnuChangelogLogFormatter, 'first', """\
 
1577
2005-11-22  John Doe  <jdoe@example.com>
 
1578
 
 
1579
\tadd a
 
1580
 
 
1581
""")
 
1582
 
 
1583
    def test_gnu_changelog_all(self):
 
1584
        self.assertFormatterResult(log.GnuChangelogLogFormatter, 'all', """\
 
1585
2005-11-22  John Doe  <jdoe@example.com>, Jane Rey  <jrey@example.com>
 
1586
 
 
1587
\tadd a
 
1588
 
 
1589
""")
 
1590
 
 
1591
 
1546
1592
class TestLogExcludeAncestry(tests.TestCaseWithTransport):
1547
1593
 
1548
1594
    def make_branch_with_alternate_ancestries(self, relpath='.'):
1549
1595
        # See test_merge_sorted_exclude_ancestry below for the difference with
1550
1596
        # bt.per_branch.test_iter_merge_sorted_revision.
1551
 
        # TestIterMergeSortedRevisionsBushyGraph. 
 
1597
        # TestIterMergeSortedRevisionsBushyGraph.
1552
1598
        # make_branch_with_alternate_ancestries
1553
1599
        # and test_merge_sorted_exclude_ancestry
1554
1600
        # See the FIXME in assertLogRevnos too.
1565
1611
        # | /
1566
1612
        # 3
1567
1613
        builder.start_series()
1568
 
        builder.build_snapshot('1', None, [
1569
 
            ('add', ('', 'TREE_ROOT', 'directory', '')),])
1570
 
        builder.build_snapshot('1.1.1', ['1'], [])
1571
 
        builder.build_snapshot('2', ['1'], [])
1572
 
        builder.build_snapshot('1.2.1', ['1.1.1'], [])
1573
 
        builder.build_snapshot('1.1.2', ['1.1.1', '1.2.1'], [])
1574
 
        builder.build_snapshot('3', ['2', '1.1.2'], [])
 
1614
        builder.build_snapshot(None, [
 
1615
            ('add', ('', b'TREE_ROOT', 'directory', '')),],
 
1616
            revision_id=b'1')
 
1617
        builder.build_snapshot([b'1'], [], revision_id=b'1.1.1')
 
1618
        builder.build_snapshot([b'1'], [], revision_id=b'2')
 
1619
        builder.build_snapshot([b'1.1.1'], [], revision_id=b'1.2.1')
 
1620
        builder.build_snapshot([b'1.1.1', b'1.2.1'], [], revision_id=b'1.1.2')
 
1621
        builder.build_snapshot([b'2', b'1.1.2'], [], revision_id=b'3')
1575
1622
        builder.finish_series()
1576
1623
        br = builder.get_branch()
1577
1624
        br.lock_read()
1579
1626
        return br
1580
1627
 
1581
1628
    def assertLogRevnos(self, expected_revnos, b, start, end,
1582
 
                        exclude_common_ancestry):
 
1629
                        exclude_common_ancestry, generate_merge_revisions=True):
1583
1630
        # FIXME: the layering in log makes it hard to test intermediate levels,
1584
 
        # I wish adding filters with their parameters were easier...
 
1631
        # I wish adding filters with their parameters was easier...
1585
1632
        # -- vila 20100413
1586
1633
        iter_revs = log._calc_view_revisions(
1587
1634
            b, start, end, direction='reverse',
1588
 
            generate_merge_revisions=True,
 
1635
            generate_merge_revisions=generate_merge_revisions,
1589
1636
            exclude_common_ancestry=exclude_common_ancestry)
1590
1637
        self.assertEqual(expected_revnos,
1591
1638
                         [revid for revid, revno, depth in iter_revs])
1593
1640
    def test_merge_sorted_exclude_ancestry(self):
1594
1641
        b = self.make_branch_with_alternate_ancestries()
1595
1642
        self.assertLogRevnos(['3', '1.1.2', '1.2.1', '1.1.1', '2', '1'],
1596
 
                             b, '1', '3', False)
 
1643
                             b, '1', '3', exclude_common_ancestry=False)
1597
1644
        # '2' is part of the '3' ancestry but not part of '1.1.1' ancestry so
1598
1645
        # it should be mentioned even if merge_sort order will make it appear
1599
1646
        # after 1.1.1
1600
1647
        self.assertLogRevnos(['3', '1.1.2', '1.2.1', '2'],
1601
 
                             b, '1.1.1', '3', True)
1602
 
 
 
1648
                             b, '1.1.1', '3', exclude_common_ancestry=True)
 
1649
 
 
1650
    def test_merge_sorted_simple_revnos_exclude_ancestry(self):
 
1651
        b = self.make_branch_with_alternate_ancestries()
 
1652
        self.assertLogRevnos(['3', '2'],
 
1653
                             b, '1', '3', exclude_common_ancestry=True,
 
1654
                             generate_merge_revisions=False)
 
1655
        self.assertLogRevnos(['3', '1.1.2', '1.2.1', '1.1.1', '2'],
 
1656
                             b, '1', '3', exclude_common_ancestry=True,
 
1657
                             generate_merge_revisions=True)
 
1658
 
 
1659
 
 
1660
class TestLogDefaults(TestCaseForLogFormatter):
 
1661
    def test_default_log_level(self):
 
1662
        """
 
1663
        Test to ensure that specifying 'levels=1' to make_log_request_dict
 
1664
        doesn't get overwritten when using a LogFormatter that supports more
 
1665
        detail.
 
1666
        Fixes bug #747958.
 
1667
        """
 
1668
        wt = self._prepare_tree_with_merges()
 
1669
        b = wt.branch
 
1670
 
 
1671
        class CustomLogFormatter(log.LogFormatter):
 
1672
            def __init__(self, *args, **kwargs):
 
1673
                super(CustomLogFormatter, self).__init__(*args, **kwargs)
 
1674
                self.revisions = []
 
1675
            def get_levels(self):
 
1676
                # log formatter supports all levels:
 
1677
                return 0
 
1678
            def log_revision(self, revision):
 
1679
                self.revisions.append(revision)
 
1680
 
 
1681
        log_formatter = LogCatcher()
 
1682
        # First request we don't specify number of levels, we should get a
 
1683
        # sensible default (whatever the LogFormatter handles - which in this
 
1684
        # case is 0/everything):
 
1685
        request = log.make_log_request_dict(limit=10)
 
1686
        log.Logger(b, request).show(log_formatter)
 
1687
        # should have all three revisions:
 
1688
        self.assertEqual(len(log_formatter.revisions), 3)
 
1689
 
 
1690
        del log_formatter
 
1691
        log_formatter = LogCatcher()
 
1692
        # now explicitly request mainline revisions only:
 
1693
        request = log.make_log_request_dict(limit=10, levels=1)
 
1694
        log.Logger(b, request).show(log_formatter)
 
1695
        # should now only have 2 revisions:
 
1696
        self.assertEqual(len(log_formatter.revisions), 2)
1603
1697