/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-02-18 21:42:57 UTC
  • mto: This revision was merged to the branch mainline in revision 6859.
  • Revision ID: jelmer@jelmer.uk-20180218214257-jpevutp1wa30tz3v
Update TODO to reference Breezy, not Bazaar.

Show diffs side-by-side

added added

removed removed

Lines of Context:
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
 
117
120
            branch.tags.set_tag('v1.0', '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
 
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 file('child/file2', 'wb') as f: f.write('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', '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
    class UTFLoopbackGPGStrategy(gpg.LoopbackGPGStrategy):
 
321
        def verify(self, content, testament):
 
322
            return (gpg.SIGNATURE_VALID,
 
323
                u'UTF8 Test \xa1\xb1\xc1\xd1\xe1\xf1 <jrandom@example.com>')
 
324
 
 
325
    def has_signature_for_revision_id(self, revision_id):
 
326
        return True
 
327
 
 
328
    def get_signature_text(self, revision_id):
 
329
        return ''
 
330
 
 
331
    def test_format_signature_validity_utf(self):
 
332
        """Check that GPG signatures containing UTF-8 names are formatted
 
333
        correctly."""
 
334
        # Monkey patch to use our UTF-8 generating GPGStrategy
 
335
        self.overrideAttr(gpg, 'GPGStrategy', self.UTFLoopbackGPGStrategy)
 
336
        wt = self.make_branch_and_tree('.')
 
337
        revid = wt.commit('empty commit')
 
338
        repo = wt.branch.repository
 
339
        # Monkey patch out checking if this rev is actually signed, since we
 
340
        # can't sign it without a heavier TestCase and LoopbackGPGStrategy
 
341
        # doesn't care anyways.
 
342
        self.overrideAttr(repo, 'has_signature_for_revision_id',
 
343
                self.has_signature_for_revision_id)
 
344
        self.overrideAttr(repo, 'get_signature_text', self.get_signature_text)
 
345
        out = log.format_signature_validity(revid, wt.branch)
 
346
        self.assertEqual(
 
347
u'valid signature from UTF8 Test \xa1\xb1\xc1\xd1\xe1\xf1 <jrandom@example.com>',
 
348
                out)
 
349
 
283
350
 
284
351
class TestShortLogFormatter(TestCaseForLogFormatter):
285
352
 
322
389
    1 Joe Foo\t2005-11-22
323
390
      rev-1
324
391
 
325
 
Use --include-merges or -n0 to see merged revisions.
 
392
Use --include-merged or -n0 to see merged revisions.
326
393
""",
327
394
            wt.branch, log.ShortLogFormatter,
328
395
            formatter_kwargs=dict(show_advice=True))
371
438
            wt.branch, log.ShortLogFormatter,
372
439
            show_log_kwargs=dict(start_revision=rev, end_revision=rev))
373
440
 
 
441
    def test_show_ids(self):
 
442
        wt = self.make_branch_and_tree('parent')
 
443
        self.build_tree(['parent/f1', 'parent/f2'])
 
444
        wt.add(['f1', 'f2'])
 
445
        self.wt_commit(wt, 'first post', rev_id='a')
 
446
        child_wt = wt.controldir.sprout('child').open_workingtree()
 
447
        self.wt_commit(child_wt, 'branch 1 changes', rev_id='b')
 
448
        wt.merge_from_branch(child_wt.branch)
 
449
        self.wt_commit(wt, 'merge branch 1', rev_id='c')
 
450
        self.assertFormatterResult("""\
 
451
    2 Joe Foo\t2005-11-22 [merge]
 
452
      revision-id:c
 
453
      merge branch 1
 
454
 
 
455
          1.1.1 Joe Foo\t2005-11-22
 
456
                revision-id:b
 
457
                branch 1 changes
 
458
 
 
459
    1 Joe Foo\t2005-11-22
 
460
      revision-id:a
 
461
      first post
 
462
 
 
463
""",
 
464
            wt.branch, log.ShortLogFormatter,
 
465
            formatter_kwargs=dict(levels=0, show_ids=True))
 
466
 
374
467
 
375
468
class TestShortLogFormatterWithMergeRevisions(TestCaseForLogFormatter):
376
469
 
432
525
    def test_merges_are_indented_by_level(self):
433
526
        wt = self.make_branch_and_tree('parent')
434
527
        self.wt_commit(wt, 'first post')
435
 
        child_wt = wt.bzrdir.sprout('child').open_workingtree()
 
528
        child_wt = wt.controldir.sprout('child').open_workingtree()
436
529
        self.wt_commit(child_wt, 'branch 1')
437
 
        smallerchild_wt = wt.bzrdir.sprout('smallerchild').open_workingtree()
 
530
        smallerchild_wt = wt.controldir.sprout('smallerchild').open_workingtree()
438
531
        self.wt_commit(smallerchild_wt, 'branch 2')
439
532
        child_wt.merge_from_branch(smallerchild_wt.branch)
440
533
        self.wt_commit(child_wt, 'merge branch 2')
484
577
    def test_verbose_merge_revisions_contain_deltas(self):
485
578
        wt = self.make_branch_and_tree('parent')
486
579
        self.build_tree(['parent/f1', 'parent/f2'])
487
 
        wt.add(['f1','f2'])
 
580
        wt.add(['f1', 'f2'])
488
581
        self.wt_commit(wt, 'first post')
489
 
        child_wt = wt.bzrdir.sprout('child').open_workingtree()
 
582
        child_wt = wt.controldir.sprout('child').open_workingtree()
490
583
        os.unlink('child/f1')
491
584
        self.build_tree_contents([('child/f2', 'hello\n')])
492
585
        self.wt_commit(child_wt, 'removed f1 and modified f2')
632
725
        sio = self.make_utf8_encoded_stringio()
633
726
        formatter = log.LongLogFormatter(to_file=sio)
634
727
        def trivial_custom_prop_handler(revision):
635
 
            raise StandardError("a test error")
 
728
            raise Exception("a test error")
636
729
 
637
730
        log.properties_handler_registry.register(
638
731
            'trivial_custom_prop_handler',
639
732
            trivial_custom_prop_handler)
640
 
        self.assertRaises(StandardError, log.show_log, wt.branch, formatter,)
 
733
        self.assertRaises(Exception, log.show_log, wt.branch, formatter,)
641
734
 
642
735
    def test_properties_handler_bad_argument(self):
643
736
        wt = self.make_standard_commit('bad_argument',
659
752
        self.assertEqualDiff('''custom_prop_name: test_value\n''',
660
753
                             sio.getvalue())
661
754
 
 
755
    def test_show_ids(self):
 
756
        wt = self.make_branch_and_tree('parent')
 
757
        self.build_tree(['parent/f1', 'parent/f2'])
 
758
        wt.add(['f1', 'f2'])
 
759
        self.wt_commit(wt, 'first post', rev_id='a')
 
760
        child_wt = wt.controldir.sprout('child').open_workingtree()
 
761
        self.wt_commit(child_wt, 'branch 1 changes', rev_id='b')
 
762
        wt.merge_from_branch(child_wt.branch)
 
763
        self.wt_commit(wt, 'merge branch 1', rev_id='c')
 
764
        self.assertFormatterResult("""\
 
765
------------------------------------------------------------
 
766
revno: 2 [merge]
 
767
revision-id: c
 
768
parent: a
 
769
parent: b
 
770
committer: Joe Foo <joe@foo.com>
 
771
branch nick: parent
 
772
timestamp: Tue 2005-11-22 00:00:02 +0000
 
773
message:
 
774
  merge branch 1
 
775
    ------------------------------------------------------------
 
776
    revno: 1.1.1
 
777
    revision-id: b
 
778
    parent: a
 
779
    committer: Joe Foo <joe@foo.com>
 
780
    branch nick: child
 
781
    timestamp: Tue 2005-11-22 00:00:01 +0000
 
782
    message:
 
783
      branch 1 changes
 
784
------------------------------------------------------------
 
785
revno: 1
 
786
revision-id: a
 
787
committer: Joe Foo <joe@foo.com>
 
788
branch nick: parent
 
789
timestamp: Tue 2005-11-22 00:00:00 +0000
 
790
message:
 
791
  first post
 
792
""",
 
793
            wt.branch, log.LongLogFormatter,
 
794
            formatter_kwargs=dict(levels=0, show_ids=True))
 
795
 
662
796
 
663
797
class TestLongLogFormatterWithoutMergeRevisions(TestCaseForLogFormatter):
664
798
 
686
820
    def test_long_verbose_contain_deltas(self):
687
821
        wt = self.make_branch_and_tree('parent')
688
822
        self.build_tree(['parent/f1', 'parent/f2'])
689
 
        wt.add(['f1','f2'])
 
823
        wt.add(['f1', 'f2'])
690
824
        self.wt_commit(wt, 'first post')
691
 
        child_wt = wt.bzrdir.sprout('child').open_workingtree()
 
825
        child_wt = wt.controldir.sprout('child').open_workingtree()
692
826
        os.unlink('child/f1')
693
827
        self.build_tree_contents([('child/f2', 'hello\n')])
694
828
        self.wt_commit(child_wt, 'removed f1 and modified f2')
915
1049
            wt.branch, log.GnuChangelogLogFormatter,
916
1050
            show_log_kwargs=dict(verbose=True))
917
1051
 
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
1052
 
1262
1053
class TestShowChangedRevisions(tests.TestCaseWithTransport):
1263
1054
 
1434
1225
 
1435
1226
    def test_show_branch_change(self):
1436
1227
        tree = self.setup_ab_tree()
1437
 
        s = StringIO()
 
1228
        s = BytesIO()
1438
1229
        log.show_branch_change(tree.branch, s, 3, '3a')
1439
1230
        self.assertContainsRe(s.getvalue(),
1440
1231
            '[*]{60}\nRemoved Revisions:\n(.|\n)*2a(.|\n)*3a(.|\n)*'
1442
1233
 
1443
1234
    def test_show_branch_change_no_change(self):
1444
1235
        tree = self.setup_ab_tree()
1445
 
        s = StringIO()
 
1236
        s = BytesIO()
1446
1237
        log.show_branch_change(tree.branch, s, 3, '3b')
1447
1238
        self.assertEqual(s.getvalue(),
1448
1239
            'Nothing seems to have changed\n')
1449
1240
 
1450
1241
    def test_show_branch_change_no_old(self):
1451
1242
        tree = self.setup_ab_tree()
1452
 
        s = StringIO()
 
1243
        s = BytesIO()
1453
1244
        log.show_branch_change(tree.branch, s, 2, '2b')
1454
1245
        self.assertContainsRe(s.getvalue(), 'Added Revisions:')
1455
1246
        self.assertNotContainsRe(s.getvalue(), 'Removed Revisions:')
1457
1248
    def test_show_branch_change_no_new(self):
1458
1249
        tree = self.setup_ab_tree()
1459
1250
        tree.branch.set_last_revision_info(2, '2b')
1460
 
        s = StringIO()
 
1251
        s = BytesIO()
1461
1252
        log.show_branch_change(tree.branch, s, 3, '3b')
1462
1253
        self.assertContainsRe(s.getvalue(), 'Removed Revisions:')
1463
1254
        self.assertNotContainsRe(s.getvalue(), 'Added Revisions:')
1464
1255
 
1465
1256
 
 
1257
class TestRevisionNotInBranch(TestCaseForLogFormatter):
 
1258
 
 
1259
    def setup_a_tree(self):
 
1260
        tree = self.make_branch_and_tree('tree')
 
1261
        tree.lock_write()
 
1262
        self.addCleanup(tree.unlock)
 
1263
        kwargs = {
 
1264
            'committer': 'Joe Foo <joe@foo.com>',
 
1265
            'timestamp': 1132617600, # Mon 2005-11-22 00:00:00 +0000
 
1266
            'timezone': 0, # UTC
 
1267
        }
 
1268
        tree.commit('commit 1a', rev_id='1a', **kwargs)
 
1269
        tree.commit('commit 2a', rev_id='2a', **kwargs)
 
1270
        tree.commit('commit 3a', rev_id='3a', **kwargs)
 
1271
        return tree
 
1272
 
 
1273
    def setup_ab_tree(self):
 
1274
        tree = self.setup_a_tree()
 
1275
        tree.set_last_revision('1a')
 
1276
        tree.branch.set_last_revision_info(1, '1a')
 
1277
        kwargs = {
 
1278
            'committer': 'Joe Foo <joe@foo.com>',
 
1279
            'timestamp': 1132617600, # Mon 2005-11-22 00:00:00 +0000
 
1280
            'timezone': 0, # UTC
 
1281
        }
 
1282
        tree.commit('commit 2b', rev_id='2b', **kwargs)
 
1283
        tree.commit('commit 3b', rev_id='3b', **kwargs)
 
1284
        return tree
 
1285
 
 
1286
    def test_one_revision(self):
 
1287
        tree = self.setup_ab_tree()
 
1288
        lf = LogCatcher()
 
1289
        rev = revisionspec.RevisionInfo(tree.branch, None, '3a')
 
1290
        log.show_log(tree.branch, lf, verbose=True, start_revision=rev,
 
1291
                     end_revision=rev)
 
1292
        self.assertEqual(1, len(lf.revisions))
 
1293
        self.assertEqual(None, lf.revisions[0].revno)   # Out-of-branch
 
1294
        self.assertEqual('3a', lf.revisions[0].rev.revision_id)
 
1295
 
 
1296
    def test_many_revisions(self):
 
1297
        tree = self.setup_ab_tree()
 
1298
        lf = LogCatcher()
 
1299
        start_rev = revisionspec.RevisionInfo(tree.branch, None, '1a')
 
1300
        end_rev = revisionspec.RevisionInfo(tree.branch, None, '3a')
 
1301
        log.show_log(tree.branch, lf, verbose=True, start_revision=start_rev,
 
1302
                     end_revision=end_rev)
 
1303
        self.assertEqual(3, len(lf.revisions))
 
1304
        self.assertEqual(None, lf.revisions[0].revno)   # Out-of-branch
 
1305
        self.assertEqual('3a', lf.revisions[0].rev.revision_id)
 
1306
        self.assertEqual(None, lf.revisions[1].revno)   # Out-of-branch
 
1307
        self.assertEqual('2a', lf.revisions[1].rev.revision_id)
 
1308
        self.assertEqual('1', lf.revisions[2].revno)    # In-branch
 
1309
 
 
1310
    def test_long_format(self):
 
1311
        tree = self.setup_ab_tree()
 
1312
        start_rev = revisionspec.RevisionInfo(tree.branch, None, '1a')
 
1313
        end_rev = revisionspec.RevisionInfo(tree.branch, None, '3a')
 
1314
        self.assertFormatterResult("""\
 
1315
------------------------------------------------------------
 
1316
revision-id: 3a
 
1317
committer: Joe Foo <joe@foo.com>
 
1318
branch nick: tree
 
1319
timestamp: Tue 2005-11-22 00:00:00 +0000
 
1320
message:
 
1321
  commit 3a
 
1322
------------------------------------------------------------
 
1323
revision-id: 2a
 
1324
committer: Joe Foo <joe@foo.com>
 
1325
branch nick: tree
 
1326
timestamp: Tue 2005-11-22 00:00:00 +0000
 
1327
message:
 
1328
  commit 2a
 
1329
------------------------------------------------------------
 
1330
revno: 1
 
1331
committer: Joe Foo <joe@foo.com>
 
1332
branch nick: tree
 
1333
timestamp: Tue 2005-11-22 00:00:00 +0000
 
1334
message:
 
1335
  commit 1a
 
1336
""",
 
1337
            tree.branch, log.LongLogFormatter, show_log_kwargs={
 
1338
                'start_revision': start_rev, 'end_revision': end_rev
 
1339
            })
 
1340
 
 
1341
    def test_short_format(self):
 
1342
        tree = self.setup_ab_tree()
 
1343
        start_rev = revisionspec.RevisionInfo(tree.branch, None, '1a')
 
1344
        end_rev = revisionspec.RevisionInfo(tree.branch, None, '3a')
 
1345
        self.assertFormatterResult("""\
 
1346
      Joe Foo\t2005-11-22
 
1347
      revision-id:3a
 
1348
      commit 3a
 
1349
 
 
1350
      Joe Foo\t2005-11-22
 
1351
      revision-id:2a
 
1352
      commit 2a
 
1353
 
 
1354
    1 Joe Foo\t2005-11-22
 
1355
      commit 1a
 
1356
 
 
1357
""",
 
1358
            tree.branch, log.ShortLogFormatter, show_log_kwargs={
 
1359
                'start_revision': start_rev, 'end_revision': end_rev
 
1360
            })
 
1361
 
 
1362
    def test_line_format(self):
 
1363
        tree = self.setup_ab_tree()
 
1364
        start_rev = revisionspec.RevisionInfo(tree.branch, None, '1a')
 
1365
        end_rev = revisionspec.RevisionInfo(tree.branch, None, '3a')
 
1366
        self.assertFormatterResult("""\
 
1367
Joe Foo 2005-11-22 commit 3a
 
1368
Joe Foo 2005-11-22 commit 2a
 
1369
1: Joe Foo 2005-11-22 commit 1a
 
1370
""",
 
1371
            tree.branch, log.LineLogFormatter, show_log_kwargs={
 
1372
                'start_revision': start_rev, 'end_revision': end_rev
 
1373
            })
 
1374
 
1466
1375
 
1467
1376
class TestLogWithBugs(TestCaseForLogFormatter, TestLogMixin):
1468
1377
 
1469
1378
    def setUp(self):
1470
 
        TestCaseForLogFormatter.setUp(self)
 
1379
        super(TestLogWithBugs, self).setUp()
1471
1380
        log.properties_handler_registry.register(
1472
1381
            'bugs_properties_handler',
1473
1382
            log._bugs_properties_handler)
1492
1401
        self.assertFormatterResult("""\
1493
1402
------------------------------------------------------------
1494
1403
revno: 2
1495
 
fixes bug(s): test://bug/id test://bug/2
 
1404
fixes bugs: test://bug/id test://bug/2
1496
1405
author: Joe Bar <joe@bar.com>
1497
1406
committer: Joe Foo <joe@foo.com>
1498
1407
branch nick: work
1503
1412
  message
1504
1413
------------------------------------------------------------
1505
1414
revno: 1
1506
 
fixes bug(s): test://bug/id
 
1415
fixes bug: test://bug/id
1507
1416
committer: Joe Foo <joe@foo.com>
1508
1417
branch nick: work
1509
1418
timestamp: Tue 2005-11-22 00:00:00 +0000
1516
1425
        tree = self.make_commits_with_bugs()
1517
1426
        self.assertFormatterResult("""\
1518
1427
    2 Joe Bar\t2005-11-22
1519
 
      fixes bug(s): test://bug/id test://bug/2
 
1428
      fixes bugs: test://bug/id test://bug/2
1520
1429
      multiline
1521
1430
      log
1522
1431
      message
1523
1432
 
1524
1433
    1 Joe Foo\t2005-11-22
1525
 
      fixes bug(s): test://bug/id
 
1434
      fixes bug: test://bug/id
1526
1435
      simple log message
1527
1436
 
1528
1437
""",
1543
1452
    def test_bugs_handler_present(self):
1544
1453
        self.properties_handler_registry.get('bugs_properties_handler')
1545
1454
 
 
1455
 
 
1456
class TestLogForAuthors(TestCaseForLogFormatter):
 
1457
 
 
1458
    def setUp(self):
 
1459
        super(TestLogForAuthors, self).setUp()
 
1460
        self.wt = self.make_standard_commit('nicky',
 
1461
            authors=['John Doe <jdoe@example.com>',
 
1462
                     'Jane Rey <jrey@example.com>'])
 
1463
 
 
1464
    def assertFormatterResult(self, formatter, who, result):
 
1465
        formatter_kwargs = dict()
 
1466
        if who is not None:
 
1467
            author_list_handler = log.author_list_registry.get(who)
 
1468
            formatter_kwargs['author_list_handler'] = author_list_handler
 
1469
        TestCaseForLogFormatter.assertFormatterResult(self, result,
 
1470
            self.wt.branch, formatter, formatter_kwargs=formatter_kwargs)
 
1471
 
 
1472
    def test_line_default(self):
 
1473
        self.assertFormatterResult(log.LineLogFormatter, None, """\
 
1474
1: John Doe 2005-11-22 add a
 
1475
""")
 
1476
 
 
1477
    def test_line_committer(self):
 
1478
        self.assertFormatterResult(log.LineLogFormatter, 'committer', """\
 
1479
1: Lorem Ipsum 2005-11-22 add a
 
1480
""")
 
1481
 
 
1482
    def test_line_first(self):
 
1483
        self.assertFormatterResult(log.LineLogFormatter, 'first', """\
 
1484
1: John Doe 2005-11-22 add a
 
1485
""")
 
1486
 
 
1487
    def test_line_all(self):
 
1488
        self.assertFormatterResult(log.LineLogFormatter, 'all', """\
 
1489
1: John Doe, Jane Rey 2005-11-22 add a
 
1490
""")
 
1491
 
 
1492
 
 
1493
    def test_short_default(self):
 
1494
        self.assertFormatterResult(log.ShortLogFormatter, None, """\
 
1495
    1 John Doe\t2005-11-22
 
1496
      add a
 
1497
 
 
1498
""")
 
1499
 
 
1500
    def test_short_committer(self):
 
1501
        self.assertFormatterResult(log.ShortLogFormatter, 'committer', """\
 
1502
    1 Lorem Ipsum\t2005-11-22
 
1503
      add a
 
1504
 
 
1505
""")
 
1506
 
 
1507
    def test_short_first(self):
 
1508
        self.assertFormatterResult(log.ShortLogFormatter, 'first', """\
 
1509
    1 John Doe\t2005-11-22
 
1510
      add a
 
1511
 
 
1512
""")
 
1513
 
 
1514
    def test_short_all(self):
 
1515
        self.assertFormatterResult(log.ShortLogFormatter, 'all', """\
 
1516
    1 John Doe, Jane Rey\t2005-11-22
 
1517
      add a
 
1518
 
 
1519
""")
 
1520
 
 
1521
    def test_long_default(self):
 
1522
        self.assertFormatterResult(log.LongLogFormatter, None, """\
 
1523
------------------------------------------------------------
 
1524
revno: 1
 
1525
author: John Doe <jdoe@example.com>, Jane Rey <jrey@example.com>
 
1526
committer: Lorem Ipsum <test@example.com>
 
1527
branch nick: nicky
 
1528
timestamp: Tue 2005-11-22 00:00:00 +0000
 
1529
message:
 
1530
  add a
 
1531
""")
 
1532
 
 
1533
    def test_long_committer(self):
 
1534
        self.assertFormatterResult(log.LongLogFormatter, 'committer', """\
 
1535
------------------------------------------------------------
 
1536
revno: 1
 
1537
committer: Lorem Ipsum <test@example.com>
 
1538
branch nick: nicky
 
1539
timestamp: Tue 2005-11-22 00:00:00 +0000
 
1540
message:
 
1541
  add a
 
1542
""")
 
1543
 
 
1544
    def test_long_first(self):
 
1545
        self.assertFormatterResult(log.LongLogFormatter, 'first', """\
 
1546
------------------------------------------------------------
 
1547
revno: 1
 
1548
author: John Doe <jdoe@example.com>
 
1549
committer: Lorem Ipsum <test@example.com>
 
1550
branch nick: nicky
 
1551
timestamp: Tue 2005-11-22 00:00:00 +0000
 
1552
message:
 
1553
  add a
 
1554
""")
 
1555
 
 
1556
    def test_long_all(self):
 
1557
        self.assertFormatterResult(log.LongLogFormatter, 'all', """\
 
1558
------------------------------------------------------------
 
1559
revno: 1
 
1560
author: John Doe <jdoe@example.com>, Jane Rey <jrey@example.com>
 
1561
committer: Lorem Ipsum <test@example.com>
 
1562
branch nick: nicky
 
1563
timestamp: Tue 2005-11-22 00:00:00 +0000
 
1564
message:
 
1565
  add a
 
1566
""")
 
1567
 
 
1568
    def test_gnu_changelog_default(self):
 
1569
        self.assertFormatterResult(log.GnuChangelogLogFormatter, None, """\
 
1570
2005-11-22  John Doe  <jdoe@example.com>
 
1571
 
 
1572
\tadd a
 
1573
 
 
1574
""")
 
1575
 
 
1576
    def test_gnu_changelog_committer(self):
 
1577
        self.assertFormatterResult(log.GnuChangelogLogFormatter, 'committer', """\
 
1578
2005-11-22  Lorem Ipsum  <test@example.com>
 
1579
 
 
1580
\tadd a
 
1581
 
 
1582
""")
 
1583
 
 
1584
    def test_gnu_changelog_first(self):
 
1585
        self.assertFormatterResult(log.GnuChangelogLogFormatter, 'first', """\
 
1586
2005-11-22  John Doe  <jdoe@example.com>
 
1587
 
 
1588
\tadd a
 
1589
 
 
1590
""")
 
1591
 
 
1592
    def test_gnu_changelog_all(self):
 
1593
        self.assertFormatterResult(log.GnuChangelogLogFormatter, 'all', """\
 
1594
2005-11-22  John Doe  <jdoe@example.com>, Jane Rey  <jrey@example.com>
 
1595
 
 
1596
\tadd a
 
1597
 
 
1598
""")
 
1599
 
 
1600
 
1546
1601
class TestLogExcludeAncestry(tests.TestCaseWithTransport):
1547
1602
 
1548
1603
    def make_branch_with_alternate_ancestries(self, relpath='.'):
1549
1604
        # See test_merge_sorted_exclude_ancestry below for the difference with
1550
1605
        # bt.per_branch.test_iter_merge_sorted_revision.
1551
 
        # TestIterMergeSortedRevisionsBushyGraph. 
 
1606
        # TestIterMergeSortedRevisionsBushyGraph.
1552
1607
        # make_branch_with_alternate_ancestries
1553
1608
        # and test_merge_sorted_exclude_ancestry
1554
1609
        # See the FIXME in assertLogRevnos too.
1565
1620
        # | /
1566
1621
        # 3
1567
1622
        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'], [])
 
1623
        builder.build_snapshot(None, [
 
1624
            ('add', ('', 'TREE_ROOT', 'directory', '')),],
 
1625
            revision_id='1')
 
1626
        builder.build_snapshot(['1'], [], revision_id='1.1.1')
 
1627
        builder.build_snapshot(['1'], [], revision_id='2')
 
1628
        builder.build_snapshot(['1.1.1'], [], revision_id='1.2.1')
 
1629
        builder.build_snapshot(['1.1.1', '1.2.1'], [], revision_id='1.1.2')
 
1630
        builder.build_snapshot(['2', '1.1.2'], [], revision_id='3')
1575
1631
        builder.finish_series()
1576
1632
        br = builder.get_branch()
1577
1633
        br.lock_read()
1579
1635
        return br
1580
1636
 
1581
1637
    def assertLogRevnos(self, expected_revnos, b, start, end,
1582
 
                        exclude_common_ancestry):
 
1638
                        exclude_common_ancestry, generate_merge_revisions=True):
1583
1639
        # FIXME: the layering in log makes it hard to test intermediate levels,
1584
 
        # I wish adding filters with their parameters were easier...
 
1640
        # I wish adding filters with their parameters was easier...
1585
1641
        # -- vila 20100413
1586
1642
        iter_revs = log._calc_view_revisions(
1587
1643
            b, start, end, direction='reverse',
1588
 
            generate_merge_revisions=True,
 
1644
            generate_merge_revisions=generate_merge_revisions,
1589
1645
            exclude_common_ancestry=exclude_common_ancestry)
1590
1646
        self.assertEqual(expected_revnos,
1591
1647
                         [revid for revid, revno, depth in iter_revs])
1593
1649
    def test_merge_sorted_exclude_ancestry(self):
1594
1650
        b = self.make_branch_with_alternate_ancestries()
1595
1651
        self.assertLogRevnos(['3', '1.1.2', '1.2.1', '1.1.1', '2', '1'],
1596
 
                             b, '1', '3', False)
 
1652
                             b, '1', '3', exclude_common_ancestry=False)
1597
1653
        # '2' is part of the '3' ancestry but not part of '1.1.1' ancestry so
1598
1654
        # it should be mentioned even if merge_sort order will make it appear
1599
1655
        # after 1.1.1
1600
1656
        self.assertLogRevnos(['3', '1.1.2', '1.2.1', '2'],
1601
 
                             b, '1.1.1', '3', True)
1602
 
 
 
1657
                             b, '1.1.1', '3', exclude_common_ancestry=True)
 
1658
 
 
1659
    def test_merge_sorted_simple_revnos_exclude_ancestry(self):
 
1660
        b = self.make_branch_with_alternate_ancestries()
 
1661
        self.assertLogRevnos(['3', '2'],
 
1662
                             b, '1', '3', exclude_common_ancestry=True,
 
1663
                             generate_merge_revisions=False)
 
1664
        self.assertLogRevnos(['3', '1.1.2', '1.2.1', '1.1.1', '2'],
 
1665
                             b, '1', '3', exclude_common_ancestry=True,
 
1666
                             generate_merge_revisions=True)
 
1667
 
 
1668
 
 
1669
class TestLogDefaults(TestCaseForLogFormatter):
 
1670
    def test_default_log_level(self):
 
1671
        """
 
1672
        Test to ensure that specifying 'levels=1' to make_log_request_dict
 
1673
        doesn't get overwritten when using a LogFormatter that supports more
 
1674
        detail.
 
1675
        Fixes bug #747958.
 
1676
        """
 
1677
        wt = self._prepare_tree_with_merges()
 
1678
        b = wt.branch
 
1679
 
 
1680
        class CustomLogFormatter(log.LogFormatter):
 
1681
            def __init__(self, *args, **kwargs):
 
1682
                super(CustomLogFormatter, self).__init__(*args, **kwargs)
 
1683
                self.revisions = []
 
1684
            def get_levels(self):
 
1685
                # log formatter supports all levels:
 
1686
                return 0
 
1687
            def log_revision(self, revision):
 
1688
                self.revisions.append(revision)
 
1689
 
 
1690
        log_formatter = LogCatcher()
 
1691
        # First request we don't specify number of levels, we should get a
 
1692
        # sensible default (whatever the LogFormatter handles - which in this
 
1693
        # case is 0/everything):
 
1694
        request = log.make_log_request_dict(limit=10)
 
1695
        log.Logger(b, request).show(log_formatter)
 
1696
        # should have all three revisions:
 
1697
        self.assertEqual(len(log_formatter.revisions), 3)
 
1698
 
 
1699
        del log_formatter
 
1700
        log_formatter = LogCatcher()
 
1701
        # now explicitly request mainline revisions only:
 
1702
        request = log.make_log_request_dict(limit=10, levels=1)
 
1703
        log.Logger(b, request).show(log_formatter)
 
1704
        # should now only have 2 revisions:
 
1705
        self.assertEqual(len(log_formatter.revisions), 2)
1603
1706