~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/tests/test_log.py

Streamline _walkdirs_utf8 for utf8 file systems, reducing time to traverse a mozilla tree from 1s to .6 seconds. (Robert Collins)

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