/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 bzrlib/tests/test_log.py

  • Committer: John Arbash Meinel
  • Date: 2008-10-14 21:35:27 UTC
  • mto: This revision was merged to the branch mainline in revision 3805.
  • Revision ID: john@arbash-meinel.com-20081014213527-4j9uc93aq1qmn43b
Add in a shortcut when we haven't cached much yet.

Document the current algorithm more completely, including the proper
justification for the various steps.

Show diffs side-by-side

added added

removed removed

Lines of Context:
1
 
# Copyright (C) 2005-2010 Canonical Ltd
 
1
# Copyright (C) 2005, 2006, 2007 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
12
12
#
13
13
# You should have received a copy of the GNU General Public License
14
14
# along with this program; if not, write to the Free Software
15
 
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
 
15
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
16
16
 
17
17
import os
18
18
from cStringIO import StringIO
19
19
 
20
 
from bzrlib import (
21
 
    branchbuilder,
22
 
    errors,
23
 
    log,
24
 
    registry,
25
 
    revision,
26
 
    revisionspec,
27
 
    symbol_versioning,
28
 
    tests,
29
 
    )
30
 
 
31
 
 
32
 
class TestLogMixin(object):
33
 
 
34
 
    def wt_commit(self, wt, message, **kwargs):
35
 
        """Use some mostly fixed values for commits to simplify tests.
36
 
 
37
 
        Tests can use this function to get some commit attributes. The time
38
 
        stamp is incremented at each commit.
39
 
        """
40
 
        if getattr(self, 'timestamp', None) is None:
41
 
            self.timestamp = 1132617600 # Mon 2005-11-22 00:00:00 +0000
42
 
        else:
43
 
            self.timestamp += 1 # 1 second between each commit
44
 
        kwargs.setdefault('timestamp', self.timestamp)
45
 
        kwargs.setdefault('timezone', 0) # UTC
46
 
        kwargs.setdefault('committer', 'Joe Foo <joe@foo.com>')
47
 
 
48
 
        return wt.commit(message, **kwargs)
49
 
 
50
 
 
51
 
class TestCaseForLogFormatter(tests.TestCaseWithTransport, TestLogMixin):
 
20
from bzrlib import log, registry
 
21
from bzrlib.tests import TestCase, TestCaseWithTransport
 
22
from bzrlib.log import (show_log,
 
23
                        get_view_revisions,
 
24
                        LogRevision,
 
25
                        LogFormatter,
 
26
                        LongLogFormatter,
 
27
                        ShortLogFormatter,
 
28
                        LineLogFormatter)
 
29
from bzrlib.branch import Branch
 
30
from bzrlib.errors import (
 
31
    BzrCommandError,
 
32
    InvalidRevisionNumber,
 
33
    )
 
34
from bzrlib.revision import Revision
 
35
from bzrlib.revisionspec import (
 
36
    RevisionInfo,
 
37
    RevisionSpec,
 
38
    )
 
39
 
 
40
 
 
41
class TestCaseWithoutPropsHandler(TestCaseWithTransport):
52
42
 
53
43
    def setUp(self):
54
 
        super(TestCaseForLogFormatter, self).setUp()
 
44
        super(TestCaseWithoutPropsHandler, self).setUp()
55
45
        # keep a reference to the "current" custom prop. handler registry
56
 
        self.properties_handler_registry = log.properties_handler_registry
57
 
        # Use a clean registry for log
 
46
        self.properties_handler_registry = \
 
47
            log.properties_handler_registry
 
48
        # clean up the registry in log
58
49
        log.properties_handler_registry = registry.Registry()
59
 
 
60
 
        def restore():
61
 
            log.properties_handler_registry = self.properties_handler_registry
62
 
        self.addCleanup(restore)
63
 
 
64
 
    def assertFormatterResult(self, result, branch, formatter_class,
65
 
                              formatter_kwargs=None, show_log_kwargs=None):
66
 
        logfile = self.make_utf8_encoded_stringio()
67
 
        if formatter_kwargs is None:
68
 
            formatter_kwargs = {}
69
 
        formatter = formatter_class(to_file=logfile, **formatter_kwargs)
70
 
        if show_log_kwargs is None:
71
 
            show_log_kwargs = {}
72
 
        log.show_log(branch, formatter, **show_log_kwargs)
73
 
        self.assertEqualDiff(result, logfile.getvalue())
74
 
 
75
 
    def make_standard_commit(self, branch_nick, **kwargs):
76
 
        wt = self.make_branch_and_tree('.')
77
 
        wt.lock_write()
78
 
        self.addCleanup(wt.unlock)
79
 
        self.build_tree(['a'])
80
 
        wt.add(['a'])
81
 
        wt.branch.nick = branch_nick
82
 
        kwargs.setdefault('committer', 'Lorem Ipsum <test@example.com>')
83
 
        kwargs.setdefault('authors', ['John Doe <jdoe@example.com>'])
84
 
        self.wt_commit(wt, 'add a', **kwargs)
85
 
        return wt
86
 
 
87
 
    def make_commits_with_trailing_newlines(self, wt):
88
 
        """Helper method for LogFormatter tests"""
89
 
        b = wt.branch
90
 
        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
 
        wt.add('b')
95
 
        self.wt_commit(wt, 'multiline\nlog\nmessage\n', rev_id='a2')
96
 
 
97
 
        self.build_tree_contents([('c', 'just another manic monday\n')])
98
 
        wt.add('c')
99
 
        self.wt_commit(wt, 'single line with trailing newline\n', rev_id='a3')
100
 
        return b
101
 
 
102
 
    def _prepare_tree_with_merges(self, with_tags=False):
103
 
        wt = self.make_branch_and_memory_tree('.')
104
 
        wt.lock_write()
105
 
        self.addCleanup(wt.unlock)
106
 
        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')
112
 
        if with_tags:
113
 
            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')
118
 
        return wt
119
 
 
120
 
class LogCatcher(log.LogFormatter):
121
 
    """Pull log messages into a list rather than displaying them.
122
 
 
123
 
    To simplify testing we save logged revisions here rather than actually
124
 
    formatting anything, so that we can precisely check the result without
125
 
    being dependent on the formatting.
 
50
        
 
51
    def _cleanup(self):
 
52
        super(TestCaseWithoutPropsHandler, self)._cleanup()
 
53
        # restore the custom properties handler registry
 
54
        log.properties_handler_registry = \
 
55
            self.properties_handler_registry
 
56
 
 
57
 
 
58
class LogCatcher(LogFormatter):
 
59
    """Pull log messages into list rather than displaying them.
 
60
 
 
61
    For ease of testing we save log messages here rather than actually
 
62
    formatting them, so that we can precisely check the result without
 
63
    being too dependent on the exact formatting.
 
64
 
 
65
    We should also test the LogFormatter.
126
66
    """
127
67
 
128
 
    supports_merge_revisions = True
129
68
    supports_delta = True
130
 
    supports_diff = True
131
 
    preferred_levels = 0
132
69
 
133
 
    def __init__(self, *args, **kwargs):
134
 
        kwargs.update(dict(to_file=None))
135
 
        super(LogCatcher, self).__init__(*args, **kwargs)
136
 
        self.revisions = []
 
70
    def __init__(self):
 
71
        super(LogCatcher, self).__init__(to_file=None)
 
72
        self.logs = []
137
73
 
138
74
    def log_revision(self, revision):
139
 
        self.revisions.append(revision)
140
 
 
141
 
 
142
 
class TestShowLog(tests.TestCaseWithTransport):
 
75
        self.logs.append(revision)
 
76
 
 
77
 
 
78
class TestShowLog(TestCaseWithTransport):
143
79
 
144
80
    def checkDelta(self, delta, **kw):
145
 
        """Check the filenames touched by a delta are as expected.
146
 
 
147
 
        Caller only have to pass in the list of files for each part, all
148
 
        unspecified parts are considered empty (and checked as such).
149
 
        """
 
81
        """Check the filenames touched by a delta are as expected."""
150
82
        for n in 'added', 'removed', 'renamed', 'modified', 'unchanged':
151
 
            # By default we expect an empty list
152
83
            expected = kw.get(n, [])
153
84
            # strip out only the path components
154
85
            got = [x[0] for x in getattr(delta, n)]
155
 
            self.assertEqual(expected, got)
156
 
 
157
 
    def assertInvalidRevisonNumber(self, br, start, end):
158
 
        lf = LogCatcher()
159
 
        self.assertRaises(errors.InvalidRevisionNumber,
160
 
                          log.show_log, br, lf,
161
 
                          start_revision=start, end_revision=end)
 
86
            self.assertEquals(expected, got)
162
87
 
163
88
    def test_cur_revno(self):
164
89
        wt = self.make_branch_and_tree('.')
166
91
 
167
92
        lf = LogCatcher()
168
93
        wt.commit('empty commit')
169
 
        log.show_log(b, lf, verbose=True, start_revision=1, end_revision=1)
170
 
 
171
 
        # Since there is a single revision in the branch all the combinations
172
 
        # below should fail.
173
 
        self.assertInvalidRevisonNumber(b, 2, 1)
174
 
        self.assertInvalidRevisonNumber(b, 1, 2)
175
 
        self.assertInvalidRevisonNumber(b, 0, 2)
176
 
        self.assertInvalidRevisonNumber(b, 1, 0)
177
 
        self.assertInvalidRevisonNumber(b, -1, 1)
178
 
        self.assertInvalidRevisonNumber(b, 1, -1)
179
 
 
180
 
    def test_empty_branch(self):
 
94
        show_log(b, lf, verbose=True, start_revision=1, end_revision=1)
 
95
        self.assertRaises(InvalidRevisionNumber, show_log, b, lf,
 
96
                          start_revision=2, end_revision=1) 
 
97
        self.assertRaises(InvalidRevisionNumber, show_log, b, lf,
 
98
                          start_revision=1, end_revision=2) 
 
99
        self.assertRaises(InvalidRevisionNumber, show_log, b, lf,
 
100
                          start_revision=0, end_revision=2) 
 
101
        self.assertRaises(InvalidRevisionNumber, show_log, b, lf,
 
102
                          start_revision=1, end_revision=0) 
 
103
        self.assertRaises(InvalidRevisionNumber, show_log, b, lf,
 
104
                          start_revision=-1, end_revision=1) 
 
105
        self.assertRaises(InvalidRevisionNumber, show_log, b, lf,
 
106
                          start_revision=1, end_revision=-1) 
 
107
 
 
108
    def test_simple_log(self):
 
109
        eq = self.assertEquals
 
110
        
181
111
        wt = self.make_branch_and_tree('.')
 
112
        b = wt.branch
182
113
 
183
114
        lf = LogCatcher()
184
 
        log.show_log(wt.branch, lf)
 
115
        show_log(b, lf)
185
116
        # no entries yet
186
 
        self.assertEqual([], lf.revisions)
187
 
 
188
 
    def test_empty_commit(self):
189
 
        wt = self.make_branch_and_tree('.')
 
117
        eq(lf.logs, [])
190
118
 
191
119
        wt.commit('empty commit')
192
120
        lf = LogCatcher()
193
 
        log.show_log(wt.branch, lf, verbose=True)
194
 
        revs = lf.revisions
195
 
        self.assertEqual(1, len(revs))
196
 
        self.assertEqual('1', revs[0].revno)
197
 
        self.assertEqual('empty commit', revs[0].rev.message)
198
 
        self.checkDelta(revs[0].delta)
 
121
        show_log(b, lf, verbose=True)
 
122
        eq(len(lf.logs), 1)
 
123
        eq(lf.logs[0].revno, '1')
 
124
        eq(lf.logs[0].rev.message, 'empty commit')
 
125
        d = lf.logs[0].delta
 
126
        self.log('log delta: %r' % d)
 
127
        self.checkDelta(d)
199
128
 
200
 
    def test_simple_commit(self):
201
 
        wt = self.make_branch_and_tree('.')
202
 
        wt.commit('empty commit')
203
129
        self.build_tree(['hello'])
204
130
        wt.add('hello')
205
131
        wt.commit('add one file',
206
132
                  committer=u'\u013d\xf3r\xe9m \xcdp\u0161\xfam '
207
133
                            u'<test@example.com>')
 
134
 
 
135
        lf = self.make_utf8_encoded_stringio()
 
136
        # log using regular thing
 
137
        show_log(b, LongLogFormatter(lf))
 
138
        lf.seek(0)
 
139
        for l in lf.readlines():
 
140
            self.log(l)
 
141
 
 
142
        # get log as data structure
208
143
        lf = LogCatcher()
209
 
        log.show_log(wt.branch, lf, verbose=True)
210
 
        self.assertEqual(2, len(lf.revisions))
 
144
        show_log(b, lf, verbose=True)
 
145
        eq(len(lf.logs), 2)
 
146
        self.log('log entries:')
 
147
        for logentry in lf.logs:
 
148
            self.log('%4s %s' % (logentry.revno, logentry.rev.message))
 
149
        
211
150
        # first one is most recent
212
 
        log_entry = lf.revisions[0]
213
 
        self.assertEqual('2', log_entry.revno)
214
 
        self.assertEqual('add one file', log_entry.rev.message)
215
 
        self.checkDelta(log_entry.delta, added=['hello'])
216
 
 
217
 
    def test_commit_message_with_control_chars(self):
218
 
        wt = self.make_branch_and_tree('.')
219
 
        msg = u"All 8-bit chars: " +  ''.join([unichr(x) for x in range(256)])
220
 
        msg = msg.replace(u'\r', u'\n')
 
151
        logentry = lf.logs[0]
 
152
        eq(logentry.revno, '2')
 
153
        eq(logentry.rev.message, 'add one file')
 
154
        d = logentry.delta
 
155
        self.log('log 2 delta: %r' % d)
 
156
        self.checkDelta(d, added=['hello'])
 
157
        
 
158
        # commit a log message with control characters
 
159
        msg = "All 8-bit chars: " +  ''.join([unichr(x) for x in range(256)])
 
160
        self.log("original commit message: %r", msg)
221
161
        wt.commit(msg)
222
162
        lf = LogCatcher()
223
 
        log.show_log(wt.branch, lf, verbose=True)
224
 
        committed_msg = lf.revisions[0].rev.message
225
 
        if wt.branch.repository._serializer.squashes_xml_invalid_characters:
226
 
            self.assertNotEqual(msg, committed_msg)
227
 
            self.assertTrue(len(committed_msg) > len(msg))
228
 
        else:
229
 
            self.assertEqual(msg, committed_msg)
 
163
        show_log(b, lf, verbose=True)
 
164
        committed_msg = lf.logs[0].rev.message
 
165
        self.log("escaped commit message: %r", committed_msg)
 
166
        self.assert_(msg != committed_msg)
 
167
        self.assert_(len(committed_msg) > len(msg))
230
168
 
231
 
    def test_commit_message_without_control_chars(self):
232
 
        wt = self.make_branch_and_tree('.')
 
169
        # Check that log message with only XML-valid characters isn't
233
170
        # escaped.  As ElementTree apparently does some kind of
234
171
        # newline conversion, neither LF (\x0A) nor CR (\x0D) are
235
172
        # included in the test commit message, even though they are
236
173
        # valid XML 1.0 characters.
237
174
        msg = "\x09" + ''.join([unichr(x) for x in range(0x20, 256)])
 
175
        self.log("original commit message: %r", msg)
238
176
        wt.commit(msg)
239
177
        lf = LogCatcher()
240
 
        log.show_log(wt.branch, lf, verbose=True)
241
 
        committed_msg = lf.revisions[0].rev.message
242
 
        self.assertEqual(msg, committed_msg)
 
178
        show_log(b, lf, verbose=True)
 
179
        committed_msg = lf.logs[0].rev.message
 
180
        self.log("escaped commit message: %r", committed_msg)
 
181
        self.assert_(msg == committed_msg)
243
182
 
244
183
    def test_deltas_in_merge_revisions(self):
245
184
        """Check deltas created for both mainline and merge revisions"""
 
185
        eq = self.assertEquals
246
186
        wt = self.make_branch_and_tree('parent')
247
187
        self.build_tree(['parent/file1', 'parent/file2', 'parent/file3'])
248
188
        wt.add('file1')
260
200
        b = wt.branch
261
201
        lf = LogCatcher()
262
202
        lf.supports_merge_revisions = True
263
 
        log.show_log(b, lf, verbose=True)
264
 
 
265
 
        revs = lf.revisions
266
 
        self.assertEqual(3, len(revs))
267
 
 
268
 
        logentry = revs[0]
269
 
        self.assertEqual('2', logentry.revno)
270
 
        self.assertEqual('merge child branch', logentry.rev.message)
271
 
        self.checkDelta(logentry.delta, removed=['file1'], modified=['file2'])
272
 
 
273
 
        logentry = revs[1]
274
 
        self.assertEqual('1.1.1', logentry.revno)
275
 
        self.assertEqual('remove file1 and modify file2', logentry.rev.message)
276
 
        self.checkDelta(logentry.delta, removed=['file1'], modified=['file2'])
277
 
 
278
 
        logentry = revs[2]
279
 
        self.assertEqual('1', logentry.revno)
280
 
        self.assertEqual('add file1 and file2', logentry.rev.message)
281
 
        self.checkDelta(logentry.delta, added=['file1', 'file2'])
282
 
 
283
 
 
284
 
class TestShortLogFormatter(TestCaseForLogFormatter):
 
203
        show_log(b, lf, verbose=True)
 
204
        eq(len(lf.logs),3)
 
205
        logentry = lf.logs[0]
 
206
        eq(logentry.revno, '2')
 
207
        eq(logentry.rev.message, 'merge child branch')
 
208
        d = logentry.delta
 
209
        self.checkDelta(d, removed=['file1'], modified=['file2'])
 
210
        logentry = lf.logs[1]
 
211
        eq(logentry.revno, '1.1.1')
 
212
        eq(logentry.rev.message, 'remove file1 and modify file2')
 
213
        d = logentry.delta
 
214
        self.checkDelta(d, removed=['file1'], modified=['file2'])
 
215
        logentry = lf.logs[2]
 
216
        eq(logentry.revno, '1')
 
217
        eq(logentry.rev.message, 'add file1 and file2')
 
218
        d = logentry.delta
 
219
        self.checkDelta(d, added=['file1', 'file2'])
 
220
 
 
221
    def test_merges_nonsupporting_formatter(self):
 
222
        """Tests that show_log will raise if the formatter doesn't
 
223
        support merge revisions."""
 
224
        wt = self.make_branch_and_memory_tree('.')
 
225
        wt.lock_write()
 
226
        try:
 
227
            wt.add('')
 
228
            wt.commit('rev-1', rev_id='rev-1',
 
229
                      timestamp=1132586655, timezone=36000,
 
230
                      committer='Joe Foo <joe@foo.com>')
 
231
            wt.commit('rev-merged', rev_id='rev-2a',
 
232
                      timestamp=1132586700, timezone=36000,
 
233
                      committer='Joe Foo <joe@foo.com>')
 
234
            wt.set_parent_ids(['rev-1', 'rev-2a'])
 
235
            wt.branch.set_last_revision_info(1, 'rev-1')
 
236
            wt.commit('rev-2', rev_id='rev-2b',
 
237
                      timestamp=1132586800, timezone=36000,
 
238
                      committer='Joe Foo <joe@foo.com>')
 
239
            logfile = self.make_utf8_encoded_stringio()
 
240
            formatter = ShortLogFormatter(to_file=logfile)
 
241
            wtb = wt.branch
 
242
            lf = LogCatcher()
 
243
            revspec = RevisionSpec.from_string('1.1.1')
 
244
            rev = revspec.in_history(wtb)
 
245
            self.assertRaises(BzrCommandError, show_log, wtb, lf,
 
246
                              start_revision=rev, end_revision=rev)
 
247
        finally:
 
248
            wt.unlock()
 
249
 
 
250
 
 
251
def make_commits_with_trailing_newlines(wt):
 
252
    """Helper method for LogFormatter tests"""    
 
253
    b = wt.branch
 
254
    b.nick='test'
 
255
    open('a', 'wb').write('hello moto\n')
 
256
    wt.add('a')
 
257
    wt.commit('simple log message', rev_id='a1',
 
258
              timestamp=1132586655.459960938, timezone=-6*3600,
 
259
              committer='Joe Foo <joe@foo.com>')
 
260
    open('b', 'wb').write('goodbye\n')
 
261
    wt.add('b')
 
262
    wt.commit('multiline\nlog\nmessage\n', rev_id='a2',
 
263
              timestamp=1132586842.411175966, timezone=-6*3600,
 
264
              committer='Joe Foo <joe@foo.com>',
 
265
              author='Joe Bar <joe@bar.com>')
 
266
 
 
267
    open('c', 'wb').write('just another manic monday\n')
 
268
    wt.add('c')
 
269
    wt.commit('single line with trailing newline\n', rev_id='a3',
 
270
              timestamp=1132587176.835228920, timezone=-6*3600,
 
271
              committer = 'Joe Foo <joe@foo.com>')
 
272
    return b
 
273
 
 
274
 
 
275
def normalize_log(log):
 
276
    """Replaces the variable lines of logs with fixed lines"""
 
277
    author = 'author: Dolor Sit <test@example.com>'
 
278
    committer = 'committer: Lorem Ipsum <test@example.com>'
 
279
    lines = log.splitlines(True)
 
280
    for idx,line in enumerate(lines):
 
281
        stripped_line = line.lstrip()
 
282
        indent = ' ' * (len(line) - len(stripped_line))
 
283
        if stripped_line.startswith('author:'):
 
284
            lines[idx] = indent + author + '\n'
 
285
        elif stripped_line.startswith('committer:'):
 
286
            lines[idx] = indent + committer + '\n'
 
287
        elif stripped_line.startswith('timestamp:'):
 
288
            lines[idx] = indent + 'timestamp: Just now\n'
 
289
    return ''.join(lines)
 
290
 
 
291
 
 
292
class TestShortLogFormatter(TestCaseWithTransport):
285
293
 
286
294
    def test_trailing_newlines(self):
287
295
        wt = self.make_branch_and_tree('.')
288
 
        b = self.make_commits_with_trailing_newlines(wt)
289
 
        self.assertFormatterResult("""\
290
 
    3 Joe Foo\t2005-11-22
 
296
        b = make_commits_with_trailing_newlines(wt)
 
297
        sio = self.make_utf8_encoded_stringio()
 
298
        lf = ShortLogFormatter(to_file=sio)
 
299
        show_log(b, lf)
 
300
        self.assertEqualDiff(sio.getvalue(), """\
 
301
    3 Joe Foo\t2005-11-21
291
302
      single line with trailing newline
292
303
 
293
 
    2 Joe Foo\t2005-11-22
 
304
    2 Joe Bar\t2005-11-21
294
305
      multiline
295
306
      log
296
307
      message
297
308
 
298
 
    1 Joe Foo\t2005-11-22
 
309
    1 Joe Foo\t2005-11-21
299
310
      simple log message
300
311
 
301
 
""",
302
 
            b, log.ShortLogFormatter)
 
312
""")
303
313
 
304
314
    def test_short_log_with_merges(self):
305
 
        wt = self._prepare_tree_with_merges()
306
 
        self.assertFormatterResult("""\
307
 
    2 Joe Foo\t2005-11-22 [merge]
308
 
      rev-2
309
 
 
310
 
    1 Joe Foo\t2005-11-22
311
 
      rev-1
312
 
 
313
 
""",
314
 
            wt.branch, log.ShortLogFormatter)
315
 
 
316
 
    def test_short_log_with_merges_and_advice(self):
317
 
        wt = self._prepare_tree_with_merges()
318
 
        self.assertFormatterResult("""\
319
 
    2 Joe Foo\t2005-11-22 [merge]
320
 
      rev-2
321
 
 
322
 
    1 Joe Foo\t2005-11-22
323
 
      rev-1
324
 
 
325
 
Use --include-merges or -n0 to see merged revisions.
326
 
""",
327
 
            wt.branch, log.ShortLogFormatter,
328
 
            formatter_kwargs=dict(show_advice=True))
329
 
 
330
 
    def test_short_log_with_merges_and_range(self):
331
 
        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')
336
 
        self.assertFormatterResult("""\
337
 
    3 Joe Foo\t2005-11-22 [merge]
338
 
      rev-3b
339
 
 
340
 
    2 Joe Foo\t2005-11-22 [merge]
341
 
      rev-2
342
 
 
343
 
""",
344
 
            wt.branch, log.ShortLogFormatter,
345
 
            show_log_kwargs=dict(start_revision=2, end_revision=3))
346
 
 
347
 
    def test_short_log_with_tags(self):
348
 
        wt = self._prepare_tree_with_merges(with_tags=True)
349
 
        self.assertFormatterResult("""\
350
 
    3 Joe Foo\t2005-11-22 {v1.0, v1.0rc1}
351
 
      rev-3
352
 
 
353
 
    2 Joe Foo\t2005-11-22 {v0.2} [merge]
354
 
      rev-2
355
 
 
356
 
    1 Joe Foo\t2005-11-22
357
 
      rev-1
358
 
 
359
 
""",
360
 
            wt.branch, log.ShortLogFormatter)
 
315
        wt = self.make_branch_and_memory_tree('.')
 
316
        wt.lock_write()
 
317
        try:
 
318
            wt.add('')
 
319
            wt.commit('rev-1', rev_id='rev-1',
 
320
                      timestamp=1132586655, timezone=36000,
 
321
                      committer='Joe Foo <joe@foo.com>')
 
322
            wt.commit('rev-merged', rev_id='rev-2a',
 
323
                      timestamp=1132586700, timezone=36000,
 
324
                      committer='Joe Foo <joe@foo.com>')
 
325
            wt.set_parent_ids(['rev-1', 'rev-2a'])
 
326
            wt.branch.set_last_revision_info(1, 'rev-1')
 
327
            wt.commit('rev-2', rev_id='rev-2b',
 
328
                      timestamp=1132586800, timezone=36000,
 
329
                      committer='Joe Foo <joe@foo.com>')
 
330
            logfile = self.make_utf8_encoded_stringio()
 
331
            formatter = ShortLogFormatter(to_file=logfile)
 
332
            show_log(wt.branch, formatter)
 
333
            self.assertEqualDiff(logfile.getvalue(), """\
 
334
    2 Joe Foo\t2005-11-22 [merge]
 
335
      rev-2
 
336
 
 
337
    1 Joe Foo\t2005-11-22
 
338
      rev-1
 
339
 
 
340
""")
 
341
        finally:
 
342
            wt.unlock()
361
343
 
362
344
    def test_short_log_single_merge_revision(self):
363
 
        wt = self._prepare_tree_with_merges()
364
 
        revspec = revisionspec.RevisionSpec.from_string('1.1.1')
365
 
        rev = revspec.in_history(wt.branch)
366
 
        self.assertFormatterResult("""\
367
 
      1.1.1 Joe Foo\t2005-11-22
368
 
            rev-merged
369
 
 
370
 
""",
371
 
            wt.branch, log.ShortLogFormatter,
372
 
            show_log_kwargs=dict(start_revision=rev, end_revision=rev))
373
 
 
374
 
 
375
 
class TestShortLogFormatterWithMergeRevisions(TestCaseForLogFormatter):
376
 
 
377
 
    def test_short_merge_revs_log_with_merges(self):
378
 
        wt = self._prepare_tree_with_merges()
379
 
        # Note that the 1.1.1 indenting is in fact correct given that
380
 
        # the revision numbers are right justified within 5 characters
381
 
        # for mainline revnos and 9 characters for dotted revnos.
382
 
        self.assertFormatterResult("""\
383
 
    2 Joe Foo\t2005-11-22 [merge]
384
 
      rev-2
385
 
 
386
 
          1.1.1 Joe Foo\t2005-11-22
387
 
                rev-merged
388
 
 
389
 
    1 Joe Foo\t2005-11-22
390
 
      rev-1
391
 
 
392
 
""",
393
 
            wt.branch, log.ShortLogFormatter,
394
 
            formatter_kwargs=dict(levels=0))
395
 
 
396
 
    def test_short_merge_revs_log_single_merge_revision(self):
397
 
        wt = self._prepare_tree_with_merges()
398
 
        revspec = revisionspec.RevisionSpec.from_string('1.1.1')
399
 
        rev = revspec.in_history(wt.branch)
400
 
        self.assertFormatterResult("""\
401
 
      1.1.1 Joe Foo\t2005-11-22
402
 
            rev-merged
403
 
 
404
 
""",
405
 
            wt.branch, log.ShortLogFormatter,
406
 
            formatter_kwargs=dict(levels=0),
407
 
            show_log_kwargs=dict(start_revision=rev, end_revision=rev))
408
 
 
409
 
 
410
 
class TestLongLogFormatter(TestCaseForLogFormatter):
 
345
        wt = self.make_branch_and_memory_tree('.')
 
346
        wt.lock_write()
 
347
        try:
 
348
            wt.add('')
 
349
            wt.commit('rev-1', rev_id='rev-1',
 
350
                      timestamp=1132586655, timezone=36000,
 
351
                      committer='Joe Foo <joe@foo.com>')
 
352
            wt.commit('rev-merged', rev_id='rev-2a',
 
353
                      timestamp=1132586700, timezone=36000,
 
354
                      committer='Joe Foo <joe@foo.com>')
 
355
            wt.set_parent_ids(['rev-1', 'rev-2a'])
 
356
            wt.branch.set_last_revision_info(1, 'rev-1')
 
357
            wt.commit('rev-2', rev_id='rev-2b',
 
358
                      timestamp=1132586800, timezone=36000,
 
359
                      committer='Joe Foo <joe@foo.com>')
 
360
            logfile = self.make_utf8_encoded_stringio()
 
361
            formatter = ShortLogFormatter(to_file=logfile)
 
362
            revspec = RevisionSpec.from_string('1.1.1')
 
363
            wtb = wt.branch
 
364
            rev = revspec.in_history(wtb)
 
365
            show_log(wtb, formatter, start_revision=rev, end_revision=rev)
 
366
            self.assertEqualDiff(logfile.getvalue(), """\
 
367
1.1.1 Joe Foo\t2005-11-22
 
368
      rev-merged
 
369
 
 
370
""")
 
371
        finally:
 
372
            wt.unlock()
 
373
 
 
374
 
 
375
class TestLongLogFormatter(TestCaseWithoutPropsHandler):
411
376
 
412
377
    def test_verbose_log(self):
413
378
        """Verbose log includes changed files
414
 
 
 
379
        
415
380
        bug #4676
416
381
        """
417
 
        wt = self.make_standard_commit('test_verbose_log', authors=[])
418
 
        self.assertFormatterResult('''\
 
382
        wt = self.make_branch_and_tree('.')
 
383
        b = wt.branch
 
384
        self.build_tree(['a'])
 
385
        wt.add('a')
 
386
        # XXX: why does a longer nick show up?
 
387
        b.nick = 'test_verbose_log'
 
388
        wt.commit(message='add a', 
 
389
                  timestamp=1132711707, 
 
390
                  timezone=36000,
 
391
                  committer='Lorem Ipsum <test@example.com>')
 
392
        logfile = file('out.tmp', 'w+')
 
393
        formatter = LongLogFormatter(to_file=logfile)
 
394
        show_log(b, formatter, verbose=True)
 
395
        logfile.flush()
 
396
        logfile.seek(0)
 
397
        log_contents = logfile.read()
 
398
        self.assertEqualDiff(log_contents, '''\
419
399
------------------------------------------------------------
420
400
revno: 1
421
401
committer: Lorem Ipsum <test@example.com>
422
402
branch nick: test_verbose_log
423
 
timestamp: Tue 2005-11-22 00:00:00 +0000
 
403
timestamp: Wed 2005-11-23 12:08:27 +1000
424
404
message:
425
405
  add a
426
406
added:
427
407
  a
428
 
''',
429
 
            wt.branch, log.LongLogFormatter,
430
 
            show_log_kwargs=dict(verbose=True))
 
408
''')
431
409
 
432
410
    def test_merges_are_indented_by_level(self):
433
411
        wt = self.make_branch_and_tree('parent')
434
 
        self.wt_commit(wt, 'first post')
435
 
        child_wt = wt.bzrdir.sprout('child').open_workingtree()
436
 
        self.wt_commit(child_wt, 'branch 1')
437
 
        smallerchild_wt = wt.bzrdir.sprout('smallerchild').open_workingtree()
438
 
        self.wt_commit(smallerchild_wt, 'branch 2')
439
 
        child_wt.merge_from_branch(smallerchild_wt.branch)
440
 
        self.wt_commit(child_wt, 'merge branch 2')
441
 
        wt.merge_from_branch(child_wt.branch)
442
 
        self.wt_commit(wt, 'merge branch 1')
443
 
        self.assertFormatterResult("""\
 
412
        wt.commit('first post')
 
413
        self.run_bzr('branch parent child')
 
414
        self.run_bzr(['commit', '-m', 'branch 1', '--unchanged', 'child'])
 
415
        self.run_bzr('branch child smallerchild')
 
416
        self.run_bzr(['commit', '-m', 'branch 2', '--unchanged',
 
417
            'smallerchild'])
 
418
        os.chdir('child')
 
419
        self.run_bzr('merge ../smallerchild')
 
420
        self.run_bzr(['commit', '-m', 'merge branch 2'])
 
421
        os.chdir('../parent')
 
422
        self.run_bzr('merge ../child')
 
423
        wt.commit('merge branch 1')
 
424
        b = wt.branch
 
425
        sio = self.make_utf8_encoded_stringio()
 
426
        lf = LongLogFormatter(to_file=sio)
 
427
        show_log(b, lf, verbose=True)
 
428
        log = normalize_log(sio.getvalue())
 
429
        self.assertEqualDiff(log, """\
444
430
------------------------------------------------------------
445
 
revno: 2 [merge]
446
 
committer: Joe Foo <joe@foo.com>
 
431
revno: 2
 
432
committer: Lorem Ipsum <test@example.com>
447
433
branch nick: parent
448
 
timestamp: Tue 2005-11-22 00:00:04 +0000
 
434
timestamp: Just now
449
435
message:
450
436
  merge branch 1
451
437
    ------------------------------------------------------------
452
 
    revno: 1.1.2 [merge]
453
 
    committer: Joe Foo <joe@foo.com>
 
438
    revno: 1.1.2
 
439
    committer: Lorem Ipsum <test@example.com>
454
440
    branch nick: child
455
 
    timestamp: Tue 2005-11-22 00:00:03 +0000
 
441
    timestamp: Just now
456
442
    message:
457
443
      merge branch 2
458
444
        ------------------------------------------------------------
459
445
        revno: 1.2.1
460
 
        committer: Joe Foo <joe@foo.com>
 
446
        committer: Lorem Ipsum <test@example.com>
461
447
        branch nick: smallerchild
462
 
        timestamp: Tue 2005-11-22 00:00:02 +0000
 
448
        timestamp: Just now
463
449
        message:
464
450
          branch 2
465
451
    ------------------------------------------------------------
466
452
    revno: 1.1.1
467
 
    committer: Joe Foo <joe@foo.com>
 
453
    committer: Lorem Ipsum <test@example.com>
468
454
    branch nick: child
469
 
    timestamp: Tue 2005-11-22 00:00:01 +0000
 
455
    timestamp: Just now
470
456
    message:
471
457
      branch 1
472
458
------------------------------------------------------------
473
459
revno: 1
474
 
committer: Joe Foo <joe@foo.com>
 
460
committer: Lorem Ipsum <test@example.com>
475
461
branch nick: parent
476
 
timestamp: Tue 2005-11-22 00:00:00 +0000
 
462
timestamp: Just now
477
463
message:
478
464
  first post
479
 
""",
480
 
            wt.branch, log.LongLogFormatter,
481
 
            formatter_kwargs=dict(levels=0),
482
 
            show_log_kwargs=dict(verbose=True))
 
465
""")
483
466
 
484
467
    def test_verbose_merge_revisions_contain_deltas(self):
485
468
        wt = self.make_branch_and_tree('parent')
486
469
        self.build_tree(['parent/f1', 'parent/f2'])
487
470
        wt.add(['f1','f2'])
488
 
        self.wt_commit(wt, 'first post')
489
 
        child_wt = wt.bzrdir.sprout('child').open_workingtree()
 
471
        wt.commit('first post')
 
472
        self.run_bzr('branch parent child')
490
473
        os.unlink('child/f1')
491
 
        self.build_tree_contents([('child/f2', 'hello\n')])
492
 
        self.wt_commit(child_wt, 'removed f1 and modified f2')
493
 
        wt.merge_from_branch(child_wt.branch)
494
 
        self.wt_commit(wt, 'merge branch 1')
495
 
        self.assertFormatterResult("""\
 
474
        file('child/f2', 'wb').write('hello\n')
 
475
        self.run_bzr(['commit', '-m', 'removed f1 and modified f2',
 
476
            'child'])
 
477
        os.chdir('parent')
 
478
        self.run_bzr('merge ../child')
 
479
        wt.commit('merge branch 1')
 
480
        b = wt.branch
 
481
        sio = self.make_utf8_encoded_stringio()
 
482
        lf = LongLogFormatter(to_file=sio)
 
483
        show_log(b, lf, verbose=True)
 
484
        log = normalize_log(sio.getvalue())
 
485
        self.assertEqualDiff(log, """\
496
486
------------------------------------------------------------
497
 
revno: 2 [merge]
498
 
committer: Joe Foo <joe@foo.com>
 
487
revno: 2
 
488
committer: Lorem Ipsum <test@example.com>
499
489
branch nick: parent
500
 
timestamp: Tue 2005-11-22 00:00:02 +0000
 
490
timestamp: Just now
501
491
message:
502
492
  merge branch 1
503
493
removed:
506
496
  f2
507
497
    ------------------------------------------------------------
508
498
    revno: 1.1.1
509
 
    committer: Joe Foo <joe@foo.com>
 
499
    committer: Lorem Ipsum <test@example.com>
510
500
    branch nick: child
511
 
    timestamp: Tue 2005-11-22 00:00:01 +0000
 
501
    timestamp: Just now
512
502
    message:
513
503
      removed f1 and modified f2
514
504
    removed:
517
507
      f2
518
508
------------------------------------------------------------
519
509
revno: 1
520
 
committer: Joe Foo <joe@foo.com>
 
510
committer: Lorem Ipsum <test@example.com>
521
511
branch nick: parent
522
 
timestamp: Tue 2005-11-22 00:00:00 +0000
 
512
timestamp: Just now
523
513
message:
524
514
  first post
525
515
added:
526
516
  f1
527
517
  f2
528
 
""",
529
 
            wt.branch, log.LongLogFormatter,
530
 
            formatter_kwargs=dict(levels=0),
531
 
            show_log_kwargs=dict(verbose=True))
 
518
""")
532
519
 
533
520
    def test_trailing_newlines(self):
534
521
        wt = self.make_branch_and_tree('.')
535
 
        b = self.make_commits_with_trailing_newlines(wt)
536
 
        self.assertFormatterResult("""\
 
522
        b = make_commits_with_trailing_newlines(wt)
 
523
        sio = self.make_utf8_encoded_stringio()
 
524
        lf = LongLogFormatter(to_file=sio)
 
525
        show_log(b, lf)
 
526
        self.assertEqualDiff(sio.getvalue(), """\
537
527
------------------------------------------------------------
538
528
revno: 3
539
529
committer: Joe Foo <joe@foo.com>
540
530
branch nick: test
541
 
timestamp: Tue 2005-11-22 00:00:02 +0000
 
531
timestamp: Mon 2005-11-21 09:32:56 -0600
542
532
message:
543
533
  single line with trailing newline
544
534
------------------------------------------------------------
545
535
revno: 2
 
536
author: Joe Bar <joe@bar.com>
546
537
committer: Joe Foo <joe@foo.com>
547
538
branch nick: test
548
 
timestamp: Tue 2005-11-22 00:00:01 +0000
 
539
timestamp: Mon 2005-11-21 09:27:22 -0600
549
540
message:
550
541
  multiline
551
542
  log
554
545
revno: 1
555
546
committer: Joe Foo <joe@foo.com>
556
547
branch nick: test
557
 
timestamp: Tue 2005-11-22 00:00:00 +0000
 
548
timestamp: Mon 2005-11-21 09:24:15 -0600
558
549
message:
559
550
  simple log message
560
 
""",
561
 
        b, log.LongLogFormatter)
 
551
""")
562
552
 
563
553
    def test_author_in_log(self):
564
554
        """Log includes the author name if it's set in
565
555
        the revision properties
566
556
        """
567
 
        wt = self.make_standard_commit('test_author_log',
568
 
            authors=['John Doe <jdoe@example.com>',
569
 
                     'Jane Rey <jrey@example.com>'])
570
 
        self.assertFormatterResult("""\
 
557
        wt = self.make_branch_and_tree('.')
 
558
        b = wt.branch
 
559
        self.build_tree(['a'])
 
560
        wt.add('a')
 
561
        b.nick = 'test_author_log'
 
562
        wt.commit(message='add a',
 
563
                  timestamp=1132711707,
 
564
                  timezone=36000,
 
565
                  committer='Lorem Ipsum <test@example.com>',
 
566
                  author='John Doe <jdoe@example.com>')
 
567
        sio = StringIO()
 
568
        formatter = LongLogFormatter(to_file=sio)
 
569
        show_log(b, formatter)
 
570
        self.assertEqualDiff(sio.getvalue(), '''\
571
571
------------------------------------------------------------
572
572
revno: 1
573
 
author: John Doe <jdoe@example.com>, Jane Rey <jrey@example.com>
 
573
author: John Doe <jdoe@example.com>
574
574
committer: Lorem Ipsum <test@example.com>
575
575
branch nick: test_author_log
576
 
timestamp: Tue 2005-11-22 00:00:00 +0000
 
576
timestamp: Wed 2005-11-23 12:08:27 +1000
577
577
message:
578
578
  add a
579
 
""",
580
 
        wt.branch, log.LongLogFormatter)
 
579
''')
581
580
 
582
581
    def test_properties_in_log(self):
583
 
        """Log includes the custom properties returned by the registered
 
582
        """Log includes the custom properties returned by the registered 
584
583
        handlers.
585
584
        """
586
 
        wt = self.make_standard_commit('test_properties_in_log')
587
 
        def trivial_custom_prop_handler(revision):
588
 
            return {'test_prop':'test_value'}
589
 
 
590
 
        # Cleaned up in setUp()
591
 
        log.properties_handler_registry.register(
592
 
            'trivial_custom_prop_handler',
593
 
            trivial_custom_prop_handler)
594
 
        self.assertFormatterResult("""\
 
585
        wt = self.make_branch_and_tree('.')
 
586
        b = wt.branch
 
587
        self.build_tree(['a'])
 
588
        wt.add('a')
 
589
        b.nick = 'test_properties_in_log'
 
590
        wt.commit(message='add a',
 
591
                  timestamp=1132711707,
 
592
                  timezone=36000,
 
593
                  committer='Lorem Ipsum <test@example.com>',
 
594
                  author='John Doe <jdoe@example.com>')
 
595
        sio = StringIO()
 
596
        formatter = LongLogFormatter(to_file=sio)
 
597
        try:
 
598
            def trivial_custom_prop_handler(revision):
 
599
                return {'test_prop':'test_value'}
 
600
            
 
601
            log.properties_handler_registry.register(
 
602
                'trivial_custom_prop_handler', 
 
603
                trivial_custom_prop_handler)
 
604
            show_log(b, formatter)
 
605
        finally:
 
606
            log.properties_handler_registry.remove(
 
607
                'trivial_custom_prop_handler')
 
608
            self.assertEqualDiff(sio.getvalue(), '''\
595
609
------------------------------------------------------------
596
610
revno: 1
597
611
test_prop: test_value
598
612
author: John Doe <jdoe@example.com>
599
613
committer: Lorem Ipsum <test@example.com>
600
614
branch nick: test_properties_in_log
601
 
timestamp: Tue 2005-11-22 00:00:00 +0000
 
615
timestamp: Wed 2005-11-23 12:08:27 +1000
602
616
message:
603
617
  add a
604
 
""",
605
 
            wt.branch, log.LongLogFormatter)
606
 
 
607
 
    def test_properties_in_short_log(self):
608
 
        """Log includes the custom properties returned by the registered
609
 
        handlers.
610
 
        """
611
 
        wt = self.make_standard_commit('test_properties_in_short_log')
612
 
        def trivial_custom_prop_handler(revision):
613
 
            return {'test_prop':'test_value'}
614
 
 
615
 
        log.properties_handler_registry.register(
616
 
            'trivial_custom_prop_handler',
617
 
            trivial_custom_prop_handler)
618
 
        self.assertFormatterResult("""\
619
 
    1 John Doe\t2005-11-22
620
 
      test_prop: test_value
621
 
      add a
622
 
 
623
 
""",
624
 
            wt.branch, log.ShortLogFormatter)
 
618
''')
625
619
 
626
620
    def test_error_in_properties_handler(self):
627
 
        """Log includes the custom properties returned by the registered
 
621
        """Log includes the custom properties returned by the registered 
628
622
        handlers.
629
623
        """
630
 
        wt = self.make_standard_commit('error_in_properties_handler',
631
 
            revprops={'first_prop':'first_value'})
632
 
        sio = self.make_utf8_encoded_stringio()
633
 
        formatter = log.LongLogFormatter(to_file=sio)
634
 
        def trivial_custom_prop_handler(revision):
635
 
            raise StandardError("a test error")
636
 
 
637
 
        log.properties_handler_registry.register(
638
 
            'trivial_custom_prop_handler',
639
 
            trivial_custom_prop_handler)
640
 
        self.assertRaises(StandardError, log.show_log, wt.branch, formatter,)
641
 
 
 
624
        wt = self.make_branch_and_tree('.')
 
625
        b = wt.branch
 
626
        self.build_tree(['a'])
 
627
        wt.add('a')
 
628
        b.nick = 'test_author_log'
 
629
        wt.commit(message='add a',
 
630
                  timestamp=1132711707,
 
631
                  timezone=36000,
 
632
                  committer='Lorem Ipsum <test@example.com>',
 
633
                  author='John Doe <jdoe@example.com>',
 
634
                  revprops={'first_prop':'first_value'})
 
635
        sio = StringIO()
 
636
        formatter = LongLogFormatter(to_file=sio)
 
637
        try:
 
638
            def trivial_custom_prop_handler(revision):
 
639
                raise StandardError("a test error")
 
640
            
 
641
            log.properties_handler_registry.register(
 
642
                'trivial_custom_prop_handler', 
 
643
                trivial_custom_prop_handler)
 
644
            self.assertRaises(StandardError, show_log, b, formatter,)
 
645
        finally:
 
646
            log.properties_handler_registry.remove(
 
647
                'trivial_custom_prop_handler')
 
648
                
642
649
    def test_properties_handler_bad_argument(self):
643
 
        wt = self.make_standard_commit('bad_argument',
644
 
              revprops={'a_prop':'test_value'})
645
 
        sio = self.make_utf8_encoded_stringio()
646
 
        formatter = log.LongLogFormatter(to_file=sio)
647
 
        def bad_argument_prop_handler(revision):
648
 
            return {'custom_prop_name':revision.properties['a_prop']}
649
 
 
650
 
        log.properties_handler_registry.register(
651
 
            'bad_argument_prop_handler',
652
 
            bad_argument_prop_handler)
653
 
 
654
 
        self.assertRaises(AttributeError, formatter.show_properties,
655
 
                          'a revision', '')
656
 
 
657
 
        revision = wt.branch.repository.get_revision(wt.branch.last_revision())
658
 
        formatter.show_properties(revision, '')
659
 
        self.assertEqualDiff('''custom_prop_name: test_value\n''',
660
 
                             sio.getvalue())
661
 
 
662
 
 
663
 
class TestLongLogFormatterWithoutMergeRevisions(TestCaseForLogFormatter):
664
 
 
665
 
    def test_long_verbose_log(self):
666
 
        """Verbose log includes changed files
667
 
 
668
 
        bug #4676
669
 
        """
670
 
        wt = self.make_standard_commit('test_long_verbose_log', authors=[])
671
 
        self.assertFormatterResult("""\
672
 
------------------------------------------------------------
673
 
revno: 1
674
 
committer: Lorem Ipsum <test@example.com>
675
 
branch nick: test_long_verbose_log
676
 
timestamp: Tue 2005-11-22 00:00:00 +0000
677
 
message:
678
 
  add a
679
 
added:
680
 
  a
681
 
""",
682
 
            wt.branch, log.LongLogFormatter,
683
 
            formatter_kwargs=dict(levels=1),
684
 
            show_log_kwargs=dict(verbose=True))
685
 
 
686
 
    def test_long_verbose_contain_deltas(self):
687
 
        wt = self.make_branch_and_tree('parent')
688
 
        self.build_tree(['parent/f1', 'parent/f2'])
689
 
        wt.add(['f1','f2'])
690
 
        self.wt_commit(wt, 'first post')
691
 
        child_wt = wt.bzrdir.sprout('child').open_workingtree()
692
 
        os.unlink('child/f1')
693
 
        self.build_tree_contents([('child/f2', 'hello\n')])
694
 
        self.wt_commit(child_wt, 'removed f1 and modified f2')
695
 
        wt.merge_from_branch(child_wt.branch)
696
 
        self.wt_commit(wt, 'merge branch 1')
697
 
        self.assertFormatterResult("""\
698
 
------------------------------------------------------------
699
 
revno: 2 [merge]
700
 
committer: Joe Foo <joe@foo.com>
701
 
branch nick: parent
702
 
timestamp: Tue 2005-11-22 00:00:02 +0000
703
 
message:
704
 
  merge branch 1
705
 
removed:
706
 
  f1
707
 
modified:
708
 
  f2
709
 
------------------------------------------------------------
710
 
revno: 1
711
 
committer: Joe Foo <joe@foo.com>
712
 
branch nick: parent
713
 
timestamp: Tue 2005-11-22 00:00:00 +0000
714
 
message:
715
 
  first post
716
 
added:
717
 
  f1
718
 
  f2
719
 
""",
720
 
            wt.branch, log.LongLogFormatter,
721
 
            formatter_kwargs=dict(levels=1),
722
 
            show_log_kwargs=dict(verbose=True))
723
 
 
724
 
    def test_long_trailing_newlines(self):
725
650
        wt = self.make_branch_and_tree('.')
726
 
        b = self.make_commits_with_trailing_newlines(wt)
727
 
        self.assertFormatterResult("""\
728
 
------------------------------------------------------------
729
 
revno: 3
730
 
committer: Joe Foo <joe@foo.com>
731
 
branch nick: test
732
 
timestamp: Tue 2005-11-22 00:00:02 +0000
733
 
message:
734
 
  single line with trailing newline
735
 
------------------------------------------------------------
736
 
revno: 2
737
 
committer: Joe Foo <joe@foo.com>
738
 
branch nick: test
739
 
timestamp: Tue 2005-11-22 00:00:01 +0000
740
 
message:
741
 
  multiline
742
 
  log
743
 
  message
744
 
------------------------------------------------------------
745
 
revno: 1
746
 
committer: Joe Foo <joe@foo.com>
747
 
branch nick: test
748
 
timestamp: Tue 2005-11-22 00:00:00 +0000
749
 
message:
750
 
  simple log message
751
 
""",
752
 
        b, log.LongLogFormatter,
753
 
        formatter_kwargs=dict(levels=1))
754
 
 
755
 
    def test_long_author_in_log(self):
756
 
        """Log includes the author name if it's set in
757
 
        the revision properties
758
 
        """
759
 
        wt = self.make_standard_commit('test_author_log')
760
 
        self.assertFormatterResult("""\
761
 
------------------------------------------------------------
762
 
revno: 1
763
 
author: John Doe <jdoe@example.com>
764
 
committer: Lorem Ipsum <test@example.com>
765
 
branch nick: test_author_log
766
 
timestamp: Tue 2005-11-22 00:00:00 +0000
767
 
message:
768
 
  add a
769
 
""",
770
 
            wt.branch, log.LongLogFormatter,
771
 
            formatter_kwargs=dict(levels=1))
772
 
 
773
 
    def test_long_properties_in_log(self):
774
 
        """Log includes the custom properties returned by the registered
775
 
        handlers.
776
 
        """
777
 
        wt = self.make_standard_commit('test_properties_in_log')
778
 
        def trivial_custom_prop_handler(revision):
779
 
            return {'test_prop':'test_value'}
780
 
 
781
 
        log.properties_handler_registry.register(
782
 
            'trivial_custom_prop_handler',
783
 
            trivial_custom_prop_handler)
784
 
        self.assertFormatterResult("""\
785
 
------------------------------------------------------------
786
 
revno: 1
787
 
test_prop: test_value
788
 
author: John Doe <jdoe@example.com>
789
 
committer: Lorem Ipsum <test@example.com>
790
 
branch nick: test_properties_in_log
791
 
timestamp: Tue 2005-11-22 00:00:00 +0000
792
 
message:
793
 
  add a
794
 
""",
795
 
            wt.branch, log.LongLogFormatter,
796
 
            formatter_kwargs=dict(levels=1))
797
 
 
798
 
 
799
 
class TestLineLogFormatter(TestCaseForLogFormatter):
 
651
        b = wt.branch
 
652
        self.build_tree(['a'])
 
653
        wt.add('a')
 
654
        b.nick = 'test_author_log'
 
655
        wt.commit(message='add a',
 
656
                  timestamp=1132711707,
 
657
                  timezone=36000,
 
658
                  committer='Lorem Ipsum <test@example.com>',
 
659
                  author='John Doe <jdoe@example.com>',
 
660
                  revprops={'a_prop':'test_value'})
 
661
        sio = StringIO()
 
662
        formatter = LongLogFormatter(to_file=sio)
 
663
        try:
 
664
            def bad_argument_prop_handler(revision):
 
665
                return {'custom_prop_name':revision.properties['a_prop']}
 
666
                
 
667
            log.properties_handler_registry.register(
 
668
                'bad_argument_prop_handler', 
 
669
                bad_argument_prop_handler)
 
670
            
 
671
            self.assertRaises(AttributeError, formatter.show_properties, 
 
672
                'a revision', '')
 
673
            
 
674
            revision = b.repository.get_revision(b.last_revision())
 
675
            formatter.show_properties(revision, '')
 
676
            self.assertEqualDiff(sio.getvalue(),
 
677
                '''custom_prop_name: test_value\n''')
 
678
        finally:
 
679
            log.properties_handler_registry.remove(
 
680
                'bad_argument_prop_handler')
 
681
 
 
682
 
 
683
class TestLineLogFormatter(TestCaseWithTransport):
800
684
 
801
685
    def test_line_log(self):
802
686
        """Line log should show revno
803
 
 
 
687
        
804
688
        bug #5162
805
689
        """
806
 
        wt = self.make_standard_commit('test-line-log',
807
 
                committer='Line-Log-Formatter Tester <test@line.log>',
808
 
                authors=[])
809
 
        self.assertFormatterResult("""\
810
 
1: Line-Log-Formatte... 2005-11-22 add a
811
 
""",
812
 
            wt.branch, log.LineLogFormatter)
 
690
        wt = self.make_branch_and_tree('.')
 
691
        b = wt.branch
 
692
        self.build_tree(['a'])
 
693
        wt.add('a')
 
694
        b.nick = 'test-line-log'
 
695
        wt.commit(message='add a',
 
696
                  timestamp=1132711707,
 
697
                  timezone=36000,
 
698
                  committer='Line-Log-Formatter Tester <test@line.log>')
 
699
        logfile = file('out.tmp', 'w+')
 
700
        formatter = LineLogFormatter(to_file=logfile)
 
701
        show_log(b, formatter)
 
702
        logfile.flush()
 
703
        logfile.seek(0)
 
704
        log_contents = logfile.read()
 
705
        self.assertEqualDiff(log_contents,
 
706
            '1: Line-Log-Formatte... 2005-11-23 add a\n')
813
707
 
814
708
    def test_trailing_newlines(self):
815
709
        wt = self.make_branch_and_tree('.')
816
 
        b = self.make_commits_with_trailing_newlines(wt)
817
 
        self.assertFormatterResult("""\
818
 
3: Joe Foo 2005-11-22 single line with trailing newline
819
 
2: Joe Foo 2005-11-22 multiline
820
 
1: Joe Foo 2005-11-22 simple log message
821
 
""",
822
 
            b, log.LineLogFormatter)
 
710
        b = make_commits_with_trailing_newlines(wt)
 
711
        sio = self.make_utf8_encoded_stringio()
 
712
        lf = LineLogFormatter(to_file=sio)
 
713
        show_log(b, lf)
 
714
        self.assertEqualDiff(sio.getvalue(), """\
 
715
3: Joe Foo 2005-11-21 single line with trailing newline
 
716
2: Joe Bar 2005-11-21 multiline
 
717
1: Joe Foo 2005-11-21 simple log message
 
718
""")
823
719
 
824
720
    def test_line_log_single_merge_revision(self):
825
 
        wt = self._prepare_tree_with_merges()
826
 
        revspec = revisionspec.RevisionSpec.from_string('1.1.1')
827
 
        rev = revspec.in_history(wt.branch)
828
 
        self.assertFormatterResult("""\
829
 
1.1.1: Joe Foo 2005-11-22 rev-merged
830
 
""",
831
 
            wt.branch, log.LineLogFormatter,
832
 
            show_log_kwargs=dict(start_revision=rev, end_revision=rev))
833
 
 
834
 
    def test_line_log_with_tags(self):
835
 
        wt = self._prepare_tree_with_merges(with_tags=True)
836
 
        self.assertFormatterResult("""\
837
 
3: Joe Foo 2005-11-22 {v1.0, v1.0rc1} rev-3
838
 
2: Joe Foo 2005-11-22 [merge] {v0.2} rev-2
839
 
1: Joe Foo 2005-11-22 rev-1
840
 
""",
841
 
            wt.branch, log.LineLogFormatter)
842
 
 
843
 
 
844
 
class TestLineLogFormatterWithMergeRevisions(TestCaseForLogFormatter):
845
 
 
846
 
    def test_line_merge_revs_log(self):
847
 
        """Line log should show revno
848
 
 
849
 
        bug #5162
850
 
        """
851
 
        wt = self.make_standard_commit('test-line-log',
852
 
                committer='Line-Log-Formatter Tester <test@line.log>',
853
 
                authors=[])
854
 
        self.assertFormatterResult("""\
855
 
1: Line-Log-Formatte... 2005-11-22 add a
856
 
""",
857
 
            wt.branch, log.LineLogFormatter)
858
 
 
859
 
    def test_line_merge_revs_log_single_merge_revision(self):
860
 
        wt = self._prepare_tree_with_merges()
861
 
        revspec = revisionspec.RevisionSpec.from_string('1.1.1')
862
 
        rev = revspec.in_history(wt.branch)
863
 
        self.assertFormatterResult("""\
864
 
1.1.1: Joe Foo 2005-11-22 rev-merged
865
 
""",
866
 
            wt.branch, log.LineLogFormatter,
867
 
            formatter_kwargs=dict(levels=0),
868
 
            show_log_kwargs=dict(start_revision=rev, end_revision=rev))
869
 
 
870
 
    def test_line_merge_revs_log_with_merges(self):
871
 
        wt = self._prepare_tree_with_merges()
872
 
        self.assertFormatterResult("""\
873
 
2: Joe Foo 2005-11-22 [merge] rev-2
874
 
  1.1.1: Joe Foo 2005-11-22 rev-merged
875
 
1: Joe Foo 2005-11-22 rev-1
876
 
""",
877
 
            wt.branch, log.LineLogFormatter,
878
 
            formatter_kwargs=dict(levels=0))
879
 
 
880
 
 
881
 
class TestGnuChangelogFormatter(TestCaseForLogFormatter):
882
 
 
883
 
    def test_gnu_changelog(self):
884
 
        wt = self.make_standard_commit('nicky', authors=[])
885
 
        self.assertFormatterResult('''\
886
 
2005-11-22  Lorem Ipsum  <test@example.com>
887
 
 
888
 
\tadd a
889
 
 
890
 
''',
891
 
            wt.branch, log.GnuChangelogLogFormatter)
892
 
 
893
 
    def test_with_authors(self):
894
 
        wt = self.make_standard_commit('nicky',
895
 
            authors=['Fooa Fooz <foo@example.com>',
896
 
                     'Bari Baro <bar@example.com>'])
897
 
        self.assertFormatterResult('''\
898
 
2005-11-22  Fooa Fooz  <foo@example.com>
899
 
 
900
 
\tadd a
901
 
 
902
 
''',
903
 
            wt.branch, log.GnuChangelogLogFormatter)
904
 
 
905
 
    def test_verbose(self):
906
 
        wt = self.make_standard_commit('nicky')
907
 
        self.assertFormatterResult('''\
908
 
2005-11-22  John Doe  <jdoe@example.com>
909
 
 
910
 
\t* a:
911
 
 
912
 
\tadd a
913
 
 
914
 
''',
915
 
            wt.branch, log.GnuChangelogLogFormatter,
916
 
            show_log_kwargs=dict(verbose=True))
917
 
 
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)
 
721
        wt = self.make_branch_and_memory_tree('.')
 
722
        wt.lock_write()
 
723
        try:
 
724
            wt.add('')
 
725
            wt.commit('rev-1', rev_id='rev-1',
 
726
                      timestamp=1132586655, timezone=36000,
 
727
                      committer='Joe Foo <joe@foo.com>')
 
728
            wt.commit('rev-merged', rev_id='rev-2a',
 
729
                      timestamp=1132586700, timezone=36000,
 
730
                      committer='Joe Foo <joe@foo.com>')
 
731
            wt.set_parent_ids(['rev-1', 'rev-2a'])
 
732
            wt.branch.set_last_revision_info(1, 'rev-1')
 
733
            wt.commit('rev-2', rev_id='rev-2b',
 
734
                      timestamp=1132586800, timezone=36000,
 
735
                      committer='Joe Foo <joe@foo.com>')
 
736
            logfile = self.make_utf8_encoded_stringio()
 
737
            formatter = LineLogFormatter(to_file=logfile)
 
738
            revspec = RevisionSpec.from_string('1.1.1')
 
739
            wtb = wt.branch
 
740
            rev = revspec.in_history(wtb)
 
741
            show_log(wtb, formatter, start_revision=rev, end_revision=rev)
 
742
            self.assertEqualDiff(logfile.getvalue(), """\
 
743
1.1.1: Joe Foo 2005-11-22 rev-merged
 
744
""")
 
745
        finally:
 
746
            wt.unlock()
 
747
 
 
748
 
 
749
 
 
750
class TestGetViewRevisions(TestCaseWithTransport):
923
751
 
924
752
    def make_tree_with_commits(self):
925
753
        """Create a tree with well-known revision ids"""
926
754
        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')
 
755
        wt.commit('commit one', rev_id='1')
 
756
        wt.commit('commit two', rev_id='2')
 
757
        wt.commit('commit three', rev_id='3')
930
758
        mainline_revs = [None, '1', '2', '3']
931
759
        rev_nos = {'1': 1, '2': 2, '3': 3}
932
760
        return mainline_revs, rev_nos, wt
935
763
        """Create a tree with well-known revision ids and a merge"""
936
764
        mainline_revs, rev_nos, wt = self.make_tree_with_commits()
937
765
        tree2 = wt.bzrdir.sprout('tree2').open_workingtree()
938
 
        self.wt_commit(tree2, 'four-a', rev_id='4a')
 
766
        tree2.commit('four-a', rev_id='4a')
939
767
        wt.merge_from_branch(tree2.branch)
940
 
        self.wt_commit(wt, 'four-b', rev_id='4b')
 
768
        wt.commit('four-b', rev_id='4b')
941
769
        mainline_revs.append('4b')
942
770
        rev_nos['4b'] = 4
943
771
        # 4a: 3.1.1
944
772
        return mainline_revs, rev_nos, wt
945
773
 
946
 
    def make_branch_with_many_merges(self):
 
774
    def make_tree_with_many_merges(self):
947
775
        """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
 
 
 
776
        wt = self.make_branch_and_tree('tree1')
 
777
        wt.commit('commit one', rev_id='1')
 
778
        wt.commit('commit two', rev_id='2')
 
779
        tree3 = wt.bzrdir.sprout('tree3').open_workingtree()
 
780
        tree3.commit('commit three a', rev_id='3a')
 
781
        tree2 = wt.bzrdir.sprout('tree2').open_workingtree()
 
782
        tree2.merge_from_branch(tree3.branch)
 
783
        tree2.commit('commit three b', rev_id='3b')
 
784
        wt.merge_from_branch(tree2.branch)
 
785
        wt.commit('commit three c', rev_id='3c')
 
786
        tree2.commit('four-a', rev_id='4a')
 
787
        wt.merge_from_branch(tree2.branch)
 
788
        wt.commit('four-b', rev_id='4b')
976
789
        mainline_revs = [None, '1', '2', '3c', '4b']
977
790
        rev_nos = {'1':1, '2':2, '3c': 3, '4b':4}
978
791
        full_rev_nos_for_reference = {
984
797
            '4a': '2.2.2', # second commit tree 2
985
798
            '4b': '4', # merges 4a to main
986
799
            }
987
 
        return mainline_revs, rev_nos, builder.get_branch()
 
800
        return mainline_revs, rev_nos, wt
988
801
 
989
802
    def test_get_view_revisions_forward(self):
990
803
        """Test the get_view_revisions method"""
991
804
        mainline_revs, rev_nos, wt = self.make_tree_with_commits()
992
805
        wt.lock_read()
993
806
        self.addCleanup(wt.unlock)
994
 
        revisions = list(self._get_view_revisions(
995
 
                mainline_revs, rev_nos, wt.branch, 'forward'))
 
807
        revisions = list(get_view_revisions(mainline_revs, rev_nos, wt.branch,
 
808
                                            'forward'))
996
809
        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))
 
810
            revisions)
 
811
        revisions2 = list(get_view_revisions(mainline_revs, rev_nos, wt.branch,
 
812
                                             'forward', include_merges=False))
1001
813
        self.assertEqual(revisions, revisions2)
1002
814
 
1003
815
    def test_get_view_revisions_reverse(self):
1005
817
        mainline_revs, rev_nos, wt = self.make_tree_with_commits()
1006
818
        wt.lock_read()
1007
819
        self.addCleanup(wt.unlock)
1008
 
        revisions = list(self._get_view_revisions(
1009
 
                mainline_revs, rev_nos, wt.branch, 'reverse'))
 
820
        revisions = list(get_view_revisions(mainline_revs, rev_nos, wt.branch,
 
821
                                            'reverse'))
1010
822
        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))
 
823
            revisions)
 
824
        revisions2 = list(get_view_revisions(mainline_revs, rev_nos, wt.branch,
 
825
                                             'reverse', include_merges=False))
1015
826
        self.assertEqual(revisions, revisions2)
1016
827
 
1017
828
    def test_get_view_revisions_merge(self):
1019
830
        mainline_revs, rev_nos, wt = self.make_tree_with_merges()
1020
831
        wt.lock_read()
1021
832
        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)
 
833
        revisions = list(get_view_revisions(mainline_revs, rev_nos, wt.branch,
 
834
                                            'forward'))
 
835
        self.assertEqual([('1', '1', 0), ('2', '2', 0), ('3', '3', 0),
 
836
            ('4b', '4', 0), ('4a', '3.1.1', 1)],
 
837
            revisions)
 
838
        revisions = list(get_view_revisions(mainline_revs, rev_nos, wt.branch,
 
839
                                             'forward', include_merges=False))
 
840
        self.assertEqual([('1', '1', 0), ('2', '2', 0), ('3', '3', 0),
 
841
            ('4b', '4', 0)],
 
842
            revisions)
1033
843
 
1034
844
    def test_get_view_revisions_merge_reverse(self):
1035
845
        """Test get_view_revisions in reverse when there are merges"""
1036
846
        mainline_revs, rev_nos, wt = self.make_tree_with_merges()
1037
847
        wt.lock_read()
1038
848
        self.addCleanup(wt.unlock)
1039
 
        revisions = list(self._get_view_revisions(
1040
 
                mainline_revs, rev_nos, wt.branch, 'reverse'))
 
849
        revisions = list(get_view_revisions(mainline_revs, rev_nos, wt.branch,
 
850
                                            'reverse'))
1041
851
        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))
 
852
            ('3', '3', 0), ('2', '2', 0), ('1', '1', 0)],
 
853
            revisions)
 
854
        revisions = list(get_view_revisions(mainline_revs, rev_nos, wt.branch,
 
855
                                             'reverse', include_merges=False))
1047
856
        self.assertEqual([('4b', '4', 0), ('3', '3', 0), ('2', '2', 0),
1048
 
                          ('1', '1', 0)],
1049
 
                         revisions)
 
857
            ('1', '1', 0)],
 
858
            revisions)
1050
859
 
1051
860
    def test_get_view_revisions_merge2(self):
1052
861
        """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'))
 
862
        mainline_revs, rev_nos, wt = self.make_tree_with_many_merges()
 
863
        wt.lock_read()
 
864
        self.addCleanup(wt.unlock)
 
865
        revisions = list(get_view_revisions(mainline_revs, rev_nos, wt.branch,
 
866
                                            'forward'))
1058
867
        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)]
 
868
            ('3a', '2.1.1', 1), ('3b', '2.2.1', 1), ('4b', '4', 0),
 
869
            ('4a', '2.2.2', 1)]
1061
870
        self.assertEqual(expected, revisions)
1062
 
        revisions = list(self._get_view_revisions(
1063
 
                mainline_revs, rev_nos, b, 'forward',
1064
 
                include_merges=False))
 
871
        revisions = list(get_view_revisions(mainline_revs, rev_nos, wt.branch,
 
872
                                             'forward', include_merges=False))
1065
873
        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)
 
874
            ('4b', '4', 0)],
 
875
            revisions)
 
876
 
 
877
 
 
878
class TestGetRevisionsTouchingFileID(TestCaseWithTransport):
1107
879
 
1108
880
    def create_tree_with_single_merge(self):
1109
881
        """Create a branch with a moderate layout.
1128
900
        #       use it. Since 'log' only uses the tree in a readonly
1129
901
        #       fashion, it seems a shame to regenerate an identical
1130
902
        #       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
903
        tree = self.make_branch_and_tree('tree')
1137
904
        tree.lock_write()
1138
905
        self.addCleanup(tree.unlock)
1164
931
        tree.commit('D', rev_id='D')
1165
932
 
1166
933
        # Switch to a read lock for this tree.
1167
 
        # We still have an addCleanup(tree.unlock) pending
 
934
        # We still have addCleanup(unlock)
1168
935
        tree.unlock()
1169
936
        tree.lock_read()
1170
937
        return tree
1171
938
 
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
939
    def test_tree_with_single_merge(self):
1186
940
        """Make sure the tree layout is correct."""
1187
941
        tree = self.create_tree_with_single_merge()
1188
942
        rev_A_tree = tree.branch.repository.revision_tree('A')
1189
943
        rev_B_tree = tree.branch.repository.revision_tree('B')
 
944
 
 
945
        f1_changed = (u'f1', 'f1-id', 'file', True, False)
 
946
        f2_changed = (u'f2', 'f2-id', 'file', True, False)
 
947
        f3_changed = (u'f3', 'f3-id', 'file', True, False)
 
948
 
 
949
        delta = rev_B_tree.changes_from(rev_A_tree)
 
950
        self.assertEqual([f1_changed, f3_changed], delta.modified)
 
951
        self.assertEqual([], delta.renamed)
 
952
        self.assertEqual([], delta.added)
 
953
        self.assertEqual([], delta.removed)
 
954
 
1190
955
        rev_C_tree = tree.branch.repository.revision_tree('C')
 
956
        delta = rev_C_tree.changes_from(rev_A_tree)
 
957
        self.assertEqual([f2_changed, f3_changed], delta.modified)
 
958
        self.assertEqual([], delta.renamed)
 
959
        self.assertEqual([], delta.added)
 
960
        self.assertEqual([], delta.removed)
 
961
 
1191
962
        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'])
 
963
        delta = rev_D_tree.changes_from(rev_B_tree)
 
964
        self.assertEqual([f2_changed, f3_changed], delta.modified)
 
965
        self.assertEqual([], delta.renamed)
 
966
        self.assertEqual([], delta.added)
 
967
        self.assertEqual([], delta.removed)
 
968
 
 
969
        delta = rev_D_tree.changes_from(rev_C_tree)
 
970
        self.assertEqual([f1_changed, f3_changed], delta.modified)
 
971
        self.assertEqual([], delta.renamed)
 
972
        self.assertEqual([], delta.added)
 
973
        self.assertEqual([], delta.removed)
1204
974
 
1205
975
    def assertAllRevisionsForFileID(self, tree, file_id, revisions):
1206
 
        """Ensure _filter_revisions_touching_file_id returns the right values.
 
976
        """Make sure _filter_revisions_touching_file_id returns the right values.
1207
977
 
1208
978
        Get the return value from _filter_revisions_touching_file_id and make
1209
979
        sure they are correct.
1210
980
        """
1211
 
        # The api for _filter_revisions_touching_file_id is a little crazy.
 
981
        # The api for _get_revisions_touching_file_id is a little crazy,
1212
982
        # So we do the setup here.
1213
983
        mainline = tree.branch.revision_history()
1214
984
        mainline.insert(0, None)
1215
985
        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)
 
986
        view_revs_iter = log.get_view_revisions(mainline, revnos, tree.branch,
 
987
                                                'reverse', True)
1218
988
        actual_revs = log._filter_revisions_touching_file_id(
1219
 
            tree.branch, file_id, list(view_revs_iter))
 
989
                            tree.branch,
 
990
                            file_id,
 
991
                            list(view_revs_iter),
 
992
                            'reverse')
1220
993
        self.assertEqual(revisions, [r for r, revno, depth in actual_revs])
1221
994
 
1222
995
    def test_file_id_f1(self):
1233
1006
    def test_file_id_f3(self):
1234
1007
        tree = self.create_tree_with_single_merge()
1235
1008
        # f3 should be marked as modified by revisions A, B, C, and D
1236
 
        self.assertAllRevisionsForFileID(tree, 'f3-id', ['D', 'C', 'B', 'A'])
 
1009
        self.assertAllRevisionsForFileID(tree, 'f2-id', ['D', 'C', 'A'])
1237
1010
 
1238
1011
    def test_file_id_with_ghosts(self):
1239
1012
        # This is testing bug #209948, where having a ghost would cause
1240
1013
        # _filter_revisions_touching_file_id() to fail.
1241
1014
        tree = self.create_tree_with_single_merge()
1242
1015
        # We need to add a revision, so switch back to a write-locked tree
1243
 
        # (still a single addCleanup(tree.unlock) pending).
1244
1016
        tree.unlock()
1245
1017
        tree.lock_write()
1246
1018
        first_parent = tree.last_revision()
1250
1022
        self.assertAllRevisionsForFileID(tree, 'f1-id', ['XX', 'B', 'A'])
1251
1023
        self.assertAllRevisionsForFileID(tree, 'f2-id', ['D', 'C', 'A'])
1252
1024
 
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
 
 
1262
 
class TestShowChangedRevisions(tests.TestCaseWithTransport):
 
1025
 
 
1026
class TestShowChangedRevisions(TestCaseWithTransport):
1263
1027
 
1264
1028
    def test_show_changed_revisions_verbose(self):
1265
1029
        tree = self.make_branch_and_tree('tree_a')
1272
1036
        self.assertNotContainsRe(s.getvalue(), 'foo')
1273
1037
 
1274
1038
 
1275
 
class TestLogFormatter(tests.TestCase):
1276
 
 
1277
 
    def setUp(self):
1278
 
        super(TestLogFormatter, self).setUp()
1279
 
        self.rev = revision.Revision('a-id')
1280
 
        self.lf = log.LogFormatter(None)
 
1039
class TestLogFormatter(TestCase):
1281
1040
 
1282
1041
    def test_short_committer(self):
1283
 
        def assertCommitter(expected, committer):
1284
 
            self.rev.committer = committer
1285
 
            self.assertEqual(expected, self.lf.short_committer(self.rev))
1286
 
 
1287
 
        assertCommitter('John Doe', 'John Doe <jdoe@example.com>')
1288
 
        assertCommitter('John Smith', 'John Smith <jsmith@example.com>')
1289
 
        assertCommitter('John Smith', 'John Smith')
1290
 
        assertCommitter('jsmith@example.com', 'jsmith@example.com')
1291
 
        assertCommitter('jsmith@example.com', '<jsmith@example.com>')
1292
 
        assertCommitter('John Smith', 'John Smith jsmith@example.com')
 
1042
        rev = Revision('a-id')
 
1043
        rev.committer = 'John Doe <jdoe@example.com>'
 
1044
        lf = LogFormatter(None)
 
1045
        self.assertEqual('John Doe', lf.short_committer(rev))
 
1046
        rev.committer = 'John Smith <jsmith@example.com>'
 
1047
        self.assertEqual('John Smith', lf.short_committer(rev))
 
1048
        rev.committer = 'John Smith'
 
1049
        self.assertEqual('John Smith', lf.short_committer(rev))
 
1050
        rev.committer = 'jsmith@example.com'
 
1051
        self.assertEqual('jsmith@example.com', lf.short_committer(rev))
 
1052
        rev.committer = '<jsmith@example.com>'
 
1053
        self.assertEqual('jsmith@example.com', lf.short_committer(rev))
 
1054
        rev.committer = 'John Smith jsmith@example.com'
 
1055
        self.assertEqual('John Smith', lf.short_committer(rev))
1293
1056
 
1294
1057
    def test_short_author(self):
1295
 
        def assertAuthor(expected, author):
1296
 
            self.rev.properties['author'] = author
1297
 
            self.assertEqual(expected, self.lf.short_author(self.rev))
1298
 
 
1299
 
        assertAuthor('John Smith', 'John Smith <jsmith@example.com>')
1300
 
        assertAuthor('John Smith', 'John Smith')
1301
 
        assertAuthor('jsmith@example.com', 'jsmith@example.com')
1302
 
        assertAuthor('jsmith@example.com', '<jsmith@example.com>')
1303
 
        assertAuthor('John Smith', 'John Smith jsmith@example.com')
1304
 
 
1305
 
    def test_short_author_from_committer(self):
1306
 
        self.rev.committer = 'John Doe <jdoe@example.com>'
1307
 
        self.assertEqual('John Doe', self.lf.short_author(self.rev))
1308
 
 
1309
 
    def test_short_author_from_authors(self):
1310
 
        self.rev.properties['authors'] = ('John Smith <jsmith@example.com>\n'
1311
 
                                          'Jane Rey <jrey@example.com>')
1312
 
        self.assertEqual('John Smith', self.lf.short_author(self.rev))
1313
 
 
1314
 
 
1315
 
class TestReverseByDepth(tests.TestCase):
1316
 
    """Test reverse_by_depth behavior.
1317
 
 
1318
 
    This is used to present revisions in forward (oldest first) order in a nice
1319
 
    layout.
1320
 
 
1321
 
    The tests use lighter revision description to ease reading.
1322
 
    """
1323
 
 
1324
 
    def assertReversed(self, forward, backward):
1325
 
        # Transform the descriptions to suit the API: tests use (revno, depth),
1326
 
        # while the API expects (revid, revno, depth)
1327
 
        def complete_revisions(l):
1328
 
            """Transform the description to suit the API.
1329
 
 
1330
 
            Tests use (revno, depth) whil the API expects (revid, revno, depth).
1331
 
            Since the revid is arbitrary, we just duplicate revno
1332
 
            """
1333
 
            return [ (r, r, d) for r, d in l]
1334
 
        forward = complete_revisions(forward)
1335
 
        backward= complete_revisions(backward)
1336
 
        self.assertEqual(forward, log.reverse_by_depth(backward))
1337
 
 
1338
 
 
1339
 
    def test_mainline_revisions(self):
1340
 
        self.assertReversed([( '1', 0), ('2', 0)],
1341
 
                            [('2', 0), ('1', 0)])
1342
 
 
1343
 
    def test_merged_revisions(self):
1344
 
        self.assertReversed([('1', 0), ('2', 0), ('2.2', 1), ('2.1', 1),],
1345
 
                            [('2', 0), ('2.1', 1), ('2.2', 1), ('1', 0),])
1346
 
    def test_shifted_merged_revisions(self):
1347
 
        """Test irregular layout.
1348
 
 
1349
 
        Requesting revisions touching a file can produce "holes" in the depths.
1350
 
        """
1351
 
        self.assertReversed([('1', 0), ('2', 0), ('1.1', 2), ('1.2', 2),],
1352
 
                            [('2', 0), ('1.2', 2), ('1.1', 2), ('1', 0),])
1353
 
 
1354
 
    def test_merged_without_child_revisions(self):
1355
 
        """Test irregular layout.
1356
 
 
1357
 
        Revision ranges can produce "holes" in the depths.
1358
 
        """
1359
 
        # When a revision of higher depth doesn't follow one of lower depth, we
1360
 
        # assume a lower depth one is virtually there
1361
 
        self.assertReversed([('1', 2), ('2', 2), ('3', 3), ('4', 4)],
1362
 
                            [('4', 4), ('3', 3), ('2', 2), ('1', 2),])
1363
 
        # So we get the same order after reversing below even if the original
1364
 
        # revisions are not in the same order.
1365
 
        self.assertReversed([('1', 2), ('2', 2), ('3', 3), ('4', 4)],
1366
 
                            [('3', 3), ('4', 4), ('2', 2), ('1', 2),])
1367
 
 
1368
 
 
1369
 
class TestHistoryChange(tests.TestCaseWithTransport):
1370
 
 
1371
 
    def setup_a_tree(self):
1372
 
        tree = self.make_branch_and_tree('tree')
1373
 
        tree.lock_write()
1374
 
        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')
1378
 
        return tree
1379
 
 
1380
 
    def setup_ab_tree(self):
1381
 
        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')
1386
 
        return tree
1387
 
 
1388
 
    def setup_ac_tree(self):
1389
 
        tree = self.setup_a_tree()
1390
 
        tree.set_last_revision(revision.NULL_REVISION)
1391
 
        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')
1395
 
        return tree
1396
 
 
1397
 
    def test_all_new(self):
1398
 
        tree = self.setup_ab_tree()
1399
 
        old, new = log.get_history_change('1a', '3a', tree.branch.repository)
1400
 
        self.assertEqual([], old)
1401
 
        self.assertEqual(['2a', '3a'], new)
1402
 
 
1403
 
    def test_all_old(self):
1404
 
        tree = self.setup_ab_tree()
1405
 
        old, new = log.get_history_change('3a', '1a', tree.branch.repository)
1406
 
        self.assertEqual([], new)
1407
 
        self.assertEqual(['2a', '3a'], old)
1408
 
 
1409
 
    def test_null_old(self):
1410
 
        tree = self.setup_ab_tree()
1411
 
        old, new = log.get_history_change(revision.NULL_REVISION,
1412
 
                                          '3a', tree.branch.repository)
1413
 
        self.assertEqual([], old)
1414
 
        self.assertEqual(['1a', '2a', '3a'], new)
1415
 
 
1416
 
    def test_null_new(self):
1417
 
        tree = self.setup_ab_tree()
1418
 
        old, new = log.get_history_change('3a', revision.NULL_REVISION,
1419
 
                                          tree.branch.repository)
1420
 
        self.assertEqual([], new)
1421
 
        self.assertEqual(['1a', '2a', '3a'], old)
1422
 
 
1423
 
    def test_diverged(self):
1424
 
        tree = self.setup_ab_tree()
1425
 
        old, new = log.get_history_change('3a', '3b', tree.branch.repository)
1426
 
        self.assertEqual(old, ['2a', '3a'])
1427
 
        self.assertEqual(new, ['2b', '3b'])
1428
 
 
1429
 
    def test_unrelated(self):
1430
 
        tree = self.setup_ac_tree()
1431
 
        old, new = log.get_history_change('3a', '3c', tree.branch.repository)
1432
 
        self.assertEqual(old, ['1a', '2a', '3a'])
1433
 
        self.assertEqual(new, ['1c', '2c', '3c'])
1434
 
 
1435
 
    def test_show_branch_change(self):
1436
 
        tree = self.setup_ab_tree()
1437
 
        s = StringIO()
1438
 
        log.show_branch_change(tree.branch, s, 3, '3a')
1439
 
        self.assertContainsRe(s.getvalue(),
1440
 
            '[*]{60}\nRemoved Revisions:\n(.|\n)*2a(.|\n)*3a(.|\n)*'
1441
 
            '[*]{60}\n\nAdded Revisions:\n(.|\n)*2b(.|\n)*3b')
1442
 
 
1443
 
    def test_show_branch_change_no_change(self):
1444
 
        tree = self.setup_ab_tree()
1445
 
        s = StringIO()
1446
 
        log.show_branch_change(tree.branch, s, 3, '3b')
1447
 
        self.assertEqual(s.getvalue(),
1448
 
            'Nothing seems to have changed\n')
1449
 
 
1450
 
    def test_show_branch_change_no_old(self):
1451
 
        tree = self.setup_ab_tree()
1452
 
        s = StringIO()
1453
 
        log.show_branch_change(tree.branch, s, 2, '2b')
1454
 
        self.assertContainsRe(s.getvalue(), 'Added Revisions:')
1455
 
        self.assertNotContainsRe(s.getvalue(), 'Removed Revisions:')
1456
 
 
1457
 
    def test_show_branch_change_no_new(self):
1458
 
        tree = self.setup_ab_tree()
1459
 
        tree.branch.set_last_revision_info(2, '2b')
1460
 
        s = StringIO()
1461
 
        log.show_branch_change(tree.branch, s, 3, '3b')
1462
 
        self.assertContainsRe(s.getvalue(), 'Removed Revisions:')
1463
 
        self.assertNotContainsRe(s.getvalue(), 'Added Revisions:')
1464
 
 
1465
 
 
1466
 
 
1467
 
class TestLogWithBugs(TestCaseForLogFormatter, TestLogMixin):
1468
 
 
1469
 
    def setUp(self):
1470
 
        TestCaseForLogFormatter.setUp(self)
1471
 
        log.properties_handler_registry.register(
1472
 
            'bugs_properties_handler',
1473
 
            log._bugs_properties_handler)
1474
 
 
1475
 
    def make_commits_with_bugs(self):
1476
 
        """Helper method for LogFormatter tests"""
1477
 
        tree = self.make_branch_and_tree(u'.')
1478
 
        self.build_tree(['a', 'b'])
1479
 
        tree.add('a')
1480
 
        self.wt_commit(tree, 'simple log message', rev_id='a1',
1481
 
                       revprops={'bugs': 'test://bug/id fixed'})
1482
 
        tree.add('b')
1483
 
        self.wt_commit(tree, 'multiline\nlog\nmessage\n', rev_id='a2',
1484
 
                       authors=['Joe Bar <joe@bar.com>'],
1485
 
                       revprops={'bugs': 'test://bug/id fixed\n'
1486
 
                                 'test://bug/2 fixed'})
1487
 
        return tree
1488
 
 
1489
 
 
1490
 
    def test_long_bugs(self):
1491
 
        tree = self.make_commits_with_bugs()
1492
 
        self.assertFormatterResult("""\
1493
 
------------------------------------------------------------
1494
 
revno: 2
1495
 
fixes bug(s): test://bug/id test://bug/2
1496
 
author: Joe Bar <joe@bar.com>
1497
 
committer: Joe Foo <joe@foo.com>
1498
 
branch nick: work
1499
 
timestamp: Tue 2005-11-22 00:00:01 +0000
1500
 
message:
1501
 
  multiline
1502
 
  log
1503
 
  message
1504
 
------------------------------------------------------------
1505
 
revno: 1
1506
 
fixes bug(s): test://bug/id
1507
 
committer: Joe Foo <joe@foo.com>
1508
 
branch nick: work
1509
 
timestamp: Tue 2005-11-22 00:00:00 +0000
1510
 
message:
1511
 
  simple log message
1512
 
""",
1513
 
            tree.branch, log.LongLogFormatter)
1514
 
 
1515
 
    def test_short_bugs(self):
1516
 
        tree = self.make_commits_with_bugs()
1517
 
        self.assertFormatterResult("""\
1518
 
    2 Joe Bar\t2005-11-22
1519
 
      fixes bug(s): test://bug/id test://bug/2
1520
 
      multiline
1521
 
      log
1522
 
      message
1523
 
 
1524
 
    1 Joe Foo\t2005-11-22
1525
 
      fixes bug(s): test://bug/id
1526
 
      simple log message
1527
 
 
1528
 
""",
1529
 
            tree.branch, log.ShortLogFormatter)
1530
 
 
1531
 
    def test_wrong_bugs_property(self):
1532
 
        tree = self.make_branch_and_tree(u'.')
1533
 
        self.build_tree(['foo'])
1534
 
        self.wt_commit(tree, 'simple log message', rev_id='a1',
1535
 
                       revprops={'bugs': 'test://bug/id invalid_value'})
1536
 
        self.assertFormatterResult("""\
1537
 
    1 Joe Foo\t2005-11-22
1538
 
      simple log message
1539
 
 
1540
 
""",
1541
 
            tree.branch, log.ShortLogFormatter)
1542
 
 
1543
 
    def test_bugs_handler_present(self):
1544
 
        self.properties_handler_registry.get('bugs_properties_handler')
1545
 
 
1546
 
class TestLogExcludeAncestry(tests.TestCaseWithTransport):
1547
 
 
1548
 
    def make_branch_with_alternate_ancestries(self, relpath='.'):
1549
 
        # See test_merge_sorted_exclude_ancestry below for the difference with
1550
 
        # bt.per_branch.test_iter_merge_sorted_revision.
1551
 
        # TestIterMergeSortedRevisionsBushyGraph. 
1552
 
        # make_branch_with_alternate_ancestries
1553
 
        # and test_merge_sorted_exclude_ancestry
1554
 
        # See the FIXME in assertLogRevnos too.
1555
 
        builder = branchbuilder.BranchBuilder(self.get_transport(relpath))
1556
 
        # 1
1557
 
        # |\
1558
 
        # 2 \
1559
 
        # |  |
1560
 
        # |  1.1.1
1561
 
        # |  | \
1562
 
        # |  |  1.2.1
1563
 
        # |  | /
1564
 
        # |  1.1.2
1565
 
        # | /
1566
 
        # 3
1567
 
        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'], [])
1575
 
        builder.finish_series()
1576
 
        br = builder.get_branch()
1577
 
        br.lock_read()
1578
 
        self.addCleanup(br.unlock)
1579
 
        return br
1580
 
 
1581
 
    def assertLogRevnos(self, expected_revnos, b, start, end,
1582
 
                        exclude_common_ancestry):
1583
 
        # FIXME: the layering in log makes it hard to test intermediate levels,
1584
 
        # I wish adding filters with their parameters were easier...
1585
 
        # -- vila 20100413
1586
 
        iter_revs = log._calc_view_revisions(
1587
 
            b, start, end, direction='reverse',
1588
 
            generate_merge_revisions=True,
1589
 
            exclude_common_ancestry=exclude_common_ancestry)
1590
 
        self.assertEqual(expected_revnos,
1591
 
                         [revid for revid, revno, depth in iter_revs])
1592
 
 
1593
 
    def test_merge_sorted_exclude_ancestry(self):
1594
 
        b = self.make_branch_with_alternate_ancestries()
1595
 
        self.assertLogRevnos(['3', '1.1.2', '1.2.1', '1.1.1', '2', '1'],
1596
 
                             b, '1', '3', False)
1597
 
        # '2' is part of the '3' ancestry but not part of '1.1.1' ancestry so
1598
 
        # it should be mentioned even if merge_sort order will make it appear
1599
 
        # after 1.1.1
1600
 
        self.assertLogRevnos(['3', '1.1.2', '1.2.1', '2'],
1601
 
                             b, '1.1.1', '3', True)
1602
 
 
1603
 
 
 
1058
        rev = Revision('a-id')
 
1059
        rev.committer = 'John Doe <jdoe@example.com>'
 
1060
        lf = LogFormatter(None)
 
1061
        self.assertEqual('John Doe', lf.short_author(rev))
 
1062
        rev.properties['author'] = 'John Smith <jsmith@example.com>'
 
1063
        self.assertEqual('John Smith', lf.short_author(rev))
 
1064
        rev.properties['author'] = 'John Smith'
 
1065
        self.assertEqual('John Smith', lf.short_author(rev))
 
1066
        rev.properties['author'] = 'jsmith@example.com'
 
1067
        self.assertEqual('jsmith@example.com', lf.short_author(rev))
 
1068
        rev.properties['author'] = '<jsmith@example.com>'
 
1069
        self.assertEqual('jsmith@example.com', lf.short_author(rev))
 
1070
        rev.properties['author'] = 'John Smith jsmith@example.com'
 
1071
        self.assertEqual('John Smith', lf.short_author(rev))