← Back to branch summary

~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/tests/blackbox/test_log.py

  • Committer: Aaron Bentley
  • Date: 2009-11-03 15:45:56 UTC
  • mto: (4634.97.2 2.0)
  • mto: This revision was merged to the branch mainline in revision 4798.
  • Revision ID: aaron@aaronbentley.com-20091103154556-e953dmegqbinyokq
Improve patch binary section handling.

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/tests/blackbox/test_log.py
1
 
# Copyright (C) 2006-2010 Canonical Ltd
 
1
# Copyright (C) 2005, 2006, 2007, 2009 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
17
17
 
18
18
"""Black-box tests for bzr log."""
19
19
 
20
 
from itertools import izip
21
20
import os
22
21
import re
23
22
 
24
23
from bzrlib import (
25
 
    branchbuilder,
26
 
    errors,
27
 
    log,
28
24
    osutils,
29
25
    tests,
30
26
    )
31
 
from bzrlib.tests import (
32
 
    script,
33
 
    test_log,
34
 
    )
35
 
 
36
 
 
37
 
class TestLog(tests.TestCaseWithTransport, test_log.TestLogMixin):
 
27
from bzrlib.tests import test_log
 
28
 
 
29
 
 
30
class TestLog(tests.TestCaseWithTransport):
 
31
 
 
32
    def setUp(self):
 
33
        super(TestLog, self).setUp()
 
34
        self.timezone = 0 # UTC
 
35
        self.timestamp = 1132617600 # Mon 2005-11-22 00:00:00 +0000
38
36
 
39
37
    def make_minimal_branch(self, path='.', format=None):
40
38
        tree = self.make_branch_and_tree(path, format=format)
65
63
        tree.commit(message='merge')
66
64
        return tree
67
65
 
68
 
 
69
 
class TestLogWithLogCatcher(TestLog):
70
 
 
71
 
    def setUp(self):
72
 
        super(TestLogWithLogCatcher, self).setUp()
73
 
        # Capture log formatter creations
74
 
        class MyLogFormatter(test_log.LogCatcher):
75
 
 
76
 
            def __new__(klass, *args, **kwargs):
77
 
                self.log_catcher = test_log.LogCatcher(*args, **kwargs)
78
 
                # Always return our own log formatter
79
 
                return self.log_catcher
80
 
 
81
 
        def getme(branch):
82
 
                # Always return our own log formatter class hijacking the
83
 
                # default behavior (which requires setting up a config
84
 
                # variable)
85
 
            return MyLogFormatter
86
 
        self.overrideAttr(log.log_formatter_registry, 'get_default', getme)
87
 
 
88
 
    def get_captured_revisions(self):
89
 
        return self.log_catcher.revisions
90
 
 
91
 
    def assertLogRevnos(self, args, expected_revnos, working_dir='.'):
92
 
        self.run_bzr(['log'] + args, working_dir=working_dir)
93
 
        self.assertEqual(expected_revnos,
94
 
                         [r.revno for r in self.get_captured_revisions()])
95
 
 
96
 
    def assertLogRevnosAndDepths(self, args, expected_revnos_and_depths,
97
 
                                working_dir='.'):
98
 
        self.run_bzr(['log'] + args, working_dir=working_dir)
99
 
        self.assertEqual(expected_revnos_and_depths,
100
 
                         [(r.revno, r.merge_depth)
101
 
                           for r in self.get_captured_revisions()])
102
 
 
103
 
 
104
 
class TestLogRevSpecs(TestLogWithLogCatcher):
105
 
 
106
 
    def test_log_no_revspec(self):
107
 
        self.make_linear_branch()
108
 
        self.assertLogRevnos([], ['3', '2', '1'])
 
66
    def assertRevnos(self, log, must_have=(), must_not_have=()):
 
67
        """Check if revnos are in or not in the log output"""
 
68
        for revno in must_have:
 
69
            self.assertTrue(('revno: %s\n' % revno) in log,
 
70
                'Does not contain expected revno %s' % revno)
 
71
        for revno in must_not_have:
 
72
            self.assertFalse(('revno: %s\n' % revno) in log,
 
73
                'Contains unexpected revno %s' % revno)
 
74
 
 
75
    def commit_options(self):
 
76
        """Use some mostly fixed values for commits to simplify tests.
 
77
 
 
78
        Tests can use this function to get some commit attributes. The time
 
79
        stamp is incremented at each commit.
 
80
        """
 
81
        self.timestamp += 1 # 1 second between each commit
 
82
        return dict(committer='Lorem Ipsum <joe@foo.com>',
 
83
                 timezone=self.timezone,
 
84
                 timestamp=self.timestamp,
 
85
                 )
 
86
 
 
87
    def check_log(self, expected, args, working_dir='level0'):
 
88
        out, err = self.run_bzr(['log', '--timezone', 'utc'] + args,
 
89
                                working_dir=working_dir)
 
90
        self.assertEqual('', err)
 
91
        self.assertEqualDiff(expected, test_log.normalize_log(out))
 
92
 
 
93
 
 
94
class TestLogRevSpecs(TestLog):
109
95
 
110
96
    def test_log_null_end_revspec(self):
111
97
        self.make_linear_branch()
112
 
        self.assertLogRevnos(['-r1..'], ['3', '2', '1'])
 
98
        log = self.run_bzr(['log'])[0]
 
99
        self.assertTrue('revno: 1\n' in log)
 
100
        self.assertTrue('revno: 2\n' in log)
 
101
        self.assertTrue('revno: 3\n' in log)
 
102
        self.assertTrue('message:\n  message1\n' in log)
 
103
        self.assertTrue('message:\n  message2\n' in log)
 
104
        self.assertTrue('message:\n  message3\n' in log)
 
105
 
 
106
        full_log = self.run_bzr(['log'])[0]
 
107
        log = self.run_bzr("log -r 1..")[0]
 
108
        self.assertEqualDiff(log, full_log)
113
109
 
114
110
    def test_log_null_begin_revspec(self):
115
111
        self.make_linear_branch()
116
 
        self.assertLogRevnos(['-r..3'], ['3', '2', '1'])
 
112
        full_log = self.run_bzr(['log'])[0]
 
113
        log = self.run_bzr("log -r ..3")[0]
 
114
        self.assertEqualDiff(full_log, log)
117
115
 
118
116
    def test_log_null_both_revspecs(self):
119
117
        self.make_linear_branch()
120
 
        self.assertLogRevnos(['-r..'], ['3', '2', '1'])
 
118
        full_log = self.run_bzr(['log'])[0]
 
119
        log = self.run_bzr("log -r ..")[0]
 
120
        self.assertEqualDiff(full_log, log)
 
121
 
 
122
    def test_log_zero_revspec(self):
 
123
        self.make_minimal_branch()
 
124
        self.run_bzr_error(['bzr: ERROR: Logging revision 0 is invalid.'],
 
125
                           ['log', '-r0'])
 
126
 
 
127
    def test_log_zero_begin_revspec(self):
 
128
        self.make_linear_branch()
 
129
        self.run_bzr_error(['bzr: ERROR: Logging revision 0 is invalid.'],
 
130
                           ['log', '-r0..2'])
 
131
 
 
132
    def test_log_zero_end_revspec(self):
 
133
        self.make_linear_branch()
 
134
        self.run_bzr_error(['bzr: ERROR: Logging revision 0 is invalid.'],
 
135
                           ['log', '-r-2..0'])
121
136
 
122
137
    def test_log_negative_begin_revspec_full_log(self):
123
138
        self.make_linear_branch()
124
 
        self.assertLogRevnos(['-r-3..'], ['3', '2', '1'])
 
139
        full_log = self.run_bzr(['log'])[0]
 
140
        log = self.run_bzr("log -r -3..")[0]
 
141
        self.assertEqualDiff(full_log, log)
125
142
 
126
143
    def test_log_negative_both_revspec_full_log(self):
127
144
        self.make_linear_branch()
128
 
        self.assertLogRevnos(['-r-3..-1'], ['3', '2', '1'])
 
145
        full_log = self.run_bzr(['log'])[0]
 
146
        log = self.run_bzr("log -r -3..-1")[0]
 
147
        self.assertEqualDiff(full_log, log)
129
148
 
130
149
    def test_log_negative_both_revspec_partial(self):
131
150
        self.make_linear_branch()
132
 
        self.assertLogRevnos(['-r-3..-2'], ['2', '1'])
 
151
        log = self.run_bzr("log -r -3..-2")[0]
 
152
        self.assertTrue('revno: 1\n' in log)
 
153
        self.assertTrue('revno: 2\n' in log)
 
154
        self.assertTrue('revno: 3\n' not in log)
133
155
 
134
156
    def test_log_negative_begin_revspec(self):
135
157
        self.make_linear_branch()
136
 
        self.assertLogRevnos(['-r-2..'], ['3', '2'])
 
158
        log = self.run_bzr("log -r -2..")[0]
 
159
        self.assertTrue('revno: 1\n' not in log)
 
160
        self.assertTrue('revno: 2\n' in log)
 
161
        self.assertTrue('revno: 3\n' in log)
137
162
 
138
163
    def test_log_positive_revspecs(self):
139
164
        self.make_linear_branch()
140
 
        self.assertLogRevnos(['-r1..3'], ['3', '2', '1'])
 
165
        full_log = self.run_bzr(['log'])[0]
 
166
        log = self.run_bzr("log -r 1..3")[0]
 
167
        self.assertEqualDiff(full_log, log)
141
168
 
142
169
    def test_log_dotted_revspecs(self):
143
170
        self.make_merged_branch()
144
 
        self.assertLogRevnos(['-n0', '-r1..1.1.1'], ['1.1.1', '1'])
145
 
 
146
 
    def test_log_limit(self):
147
 
        tree = self.make_branch_and_tree('.')
148
 
        # We want more commits than our batch size starts at
149
 
        for pos in range(10):
150
 
            tree.commit("%s" % pos)
151
 
        self.assertLogRevnos(['--limit', '2'], ['10', '9'])
152
 
 
153
 
    def test_log_limit_short(self):
154
 
        self.make_linear_branch()
155
 
        self.assertLogRevnos(['-l', '2'], ['3', '2'])
156
 
 
157
 
    def test_log_change_revno(self):
158
 
        self.make_linear_branch()
159
 
        self.assertLogRevnos(['-c1'], ['1'])
160
 
 
161
 
 
162
 
class TestLogMergedLinearAncestry(TestLogWithLogCatcher):
163
 
 
164
 
    def setUp(self):
165
 
        super(TestLogMergedLinearAncestry, self).setUp()
166
 
        # FIXME: Using a MemoryTree would be even better here (but until we
167
 
        # stop calling run_bzr, there is no point) --vila 100118.
168
 
        builder = branchbuilder.BranchBuilder(self.get_transport())
169
 
        builder.start_series()
170
 
        # mainline
171
 
        builder.build_snapshot('1', None, [
172
 
            ('add', ('', 'root-id', 'directory', ''))])
173
 
        builder.build_snapshot('2', ['1'], [])
174
 
        # branch
175
 
        builder.build_snapshot('1.1.1', ['1'], [])
176
 
        # merge branch into mainline
177
 
        builder.build_snapshot('3', ['2', '1.1.1'], [])
178
 
        # new commits in branch
179
 
        builder.build_snapshot('1.1.2', ['1.1.1'], [])
180
 
        builder.build_snapshot('1.1.3', ['1.1.2'], [])
181
 
        # merge branch into mainline
182
 
        builder.build_snapshot('4', ['3', '1.1.3'], [])
183
 
        # merge mainline into branch
184
 
        builder.build_snapshot('1.1.4', ['1.1.3', '4'], [])
185
 
        # merge branch into mainline
186
 
        builder.build_snapshot('5', ['4', '1.1.4'], [])
187
 
        builder.finish_series()
188
 
 
189
 
    def test_n0(self):
190
 
        self.assertLogRevnos(['-n0', '-r1.1.1..1.1.4'],
191
 
                             ['1.1.4', '4', '1.1.3', '1.1.2', '3', '1.1.1'])
192
 
    def test_n0_forward(self):
193
 
        self.assertLogRevnos(['-n0', '-r1.1.1..1.1.4', '--forward'],
194
 
                             ['3', '1.1.1', '4', '1.1.2', '1.1.3', '1.1.4'])
195
 
 
196
 
    def test_n1(self):
197
 
        # starting from 1.1.4 we follow the left-hand ancestry
198
 
        self.assertLogRevnos(['-n1', '-r1.1.1..1.1.4'],
199
 
                             ['1.1.4', '1.1.3', '1.1.2', '1.1.1'])
200
 
 
201
 
    def test_n1_forward(self):
202
 
        self.assertLogRevnos(['-n1', '-r1.1.1..1.1.4', '--forward'],
203
 
                             ['1.1.1', '1.1.2', '1.1.3', '1.1.4'])
204
 
 
205
 
 
206
 
class Test_GenerateAllRevisions(TestLogWithLogCatcher):
207
 
 
208
 
    def make_branch_with_many_merges(self, path='.', format=None):
209
 
        builder = branchbuilder.BranchBuilder(self.get_transport())
210
 
        builder.start_series()
211
 
        # The graph below may look a bit complicated (and it may be but I've
212
 
        # banged my head enough on it) but the bug requires at least dotted
213
 
        # revnos *and* merged revisions below that.
214
 
        builder.build_snapshot('1', None, [
215
 
            ('add', ('', 'root-id', 'directory', ''))])
216
 
        builder.build_snapshot('2', ['1'], [])
217
 
        builder.build_snapshot('1.1.1', ['1'], [])
218
 
        builder.build_snapshot('2.1.1', ['2'], [])
219
 
        builder.build_snapshot('3', ['2', '1.1.1'], [])
220
 
        builder.build_snapshot('2.1.2', ['2.1.1'], [])
221
 
        builder.build_snapshot('2.2.1', ['2.1.1'], [])
222
 
        builder.build_snapshot('2.1.3', ['2.1.2', '2.2.1'], [])
223
 
        builder.build_snapshot('4', ['3', '2.1.3'], [])
224
 
        builder.build_snapshot('5', ['4', '2.1.2'], [])
225
 
        builder.finish_series()
226
 
        return builder
227
 
 
228
 
    def test_not_an_ancestor(self):
229
 
        builder = self.make_branch_with_many_merges()
230
 
        b = builder.get_branch()
231
 
        b.lock_read()
232
 
        self.addCleanup(b.unlock)
233
 
        self.assertRaises(errors.BzrCommandError,
234
 
                          log._generate_all_revisions,
235
 
                          b, '1.1.1', '2.1.3', 'reverse',
236
 
                          delayed_graph_generation=True)
237
 
 
238
 
    def test_wrong_order(self):
239
 
        builder = self.make_branch_with_many_merges()
240
 
        b = builder.get_branch()
241
 
        b.lock_read()
242
 
        self.addCleanup(b.unlock)
243
 
        self.assertRaises(errors.BzrCommandError,
244
 
                          log._generate_all_revisions,
245
 
                          b, '5', '2.1.3', 'reverse',
246
 
                          delayed_graph_generation=True)
247
 
 
248
 
 
249
 
class TestLogRevSpecsWithPaths(TestLogWithLogCatcher):
250
 
 
251
 
    def test_log_revno_n_path_wrong_namespace(self):
 
171
        log = self.run_bzr("log -n0 -r 1..1.1.1")[0]
 
172
        self.assertRevnos(log, (1, '1.1.1'), (2, 3, '1.1.2', 4))
 
173
 
 
174
    def test_log_reversed_revspecs(self):
 
175
        self.make_linear_branch()
 
176
        self.run_bzr_error(('bzr: ERROR: Start revision must be older than '
 
177
                            'the end revision.\n',),
 
178
                           ['log', '-r3..1'])
 
179
 
 
180
    def test_log_reversed_dotted_revspecs(self):
 
181
        self.make_merged_branch()
 
182
        self.run_bzr_error(('bzr: ERROR: Start revision not found in '
 
183
                            'left-hand history of end revision.\n',),
 
184
                           "log -r 1.1.1..1")
 
185
 
 
186
    def test_log_revno_n_path(self):
252
187
        self.make_linear_branch('branch1')
253
188
        self.make_linear_branch('branch2')
254
 
        # There is no guarantee that a path exist between two arbitrary
255
 
        # revisions.
256
 
        self.run_bzr("log -r revno:2:branch1..revno:3:branch2", retcode=3)
257
 
 
258
 
    def test_log_revno_n_path_correct_order(self):
259
 
        self.make_linear_branch('branch2')
260
 
        self.assertLogRevnos(['-rrevno:1:branch2..revno:3:branch2'],
261
 
                             ['3', '2','1'])
262
 
 
263
 
    def test_log_revno_n_path(self):
264
 
        self.make_linear_branch('branch2')
265
 
        self.assertLogRevnos(['-rrevno:1:branch2'],
266
 
                             ['1'])
267
 
        rev_props = self.log_catcher.revisions[0].rev.properties
268
 
        self.assertEqual('branch2', rev_props['branch-nick'])
269
 
 
270
 
 
271
 
class TestLogErrors(TestLog):
272
 
 
273
 
    def test_log_zero_revspec(self):
274
 
        self.make_minimal_branch()
275
 
        self.run_bzr_error(['bzr: ERROR: Logging revision 0 is invalid.'],
276
 
                           ['log', '-r0'])
277
 
 
278
 
    def test_log_zero_begin_revspec(self):
279
 
        self.make_linear_branch()
280
 
        self.run_bzr_error(['bzr: ERROR: Logging revision 0 is invalid.'],
281
 
                           ['log', '-r0..2'])
282
 
 
283
 
    def test_log_zero_end_revspec(self):
284
 
        self.make_linear_branch()
285
 
        self.run_bzr_error(['bzr: ERROR: Logging revision 0 is invalid.'],
286
 
                           ['log', '-r-2..0'])
 
189
        # Swapped revisions
 
190
        self.run_bzr("log -r revno:2:branch1..revno:3:branch2", retcode=3)[0]
 
191
        # Correct order
 
192
        log = self.run_bzr("log -r revno:1:branch2..revno:3:branch2")[0]
 
193
        full_log = self.run_bzr(['log'], working_dir='branch2')[0]
 
194
        self.assertEqualDiff(full_log, log)
 
195
        log = self.run_bzr("log -r revno:1:branch2")[0]
 
196
        self.assertTrue('revno: 1\n' in log)
 
197
        self.assertTrue('revno: 2\n' not in log)
 
198
        self.assertTrue('branch nick: branch2\n' in log)
 
199
        self.assertTrue('branch nick: branch1\n' not in log)
287
200
 
288
201
    def test_log_nonexistent_revno(self):
289
202
        self.make_minimal_branch()
290
 
        self.run_bzr_error(["bzr: ERROR: Requested revision: '1234' "
291
 
                            "does not exist in branch:"],
292
 
                           ['log', '-r1234'])
 
203
        (out, err) = self.run_bzr_error(
 
204
            ["bzr: ERROR: Requested revision: '1234' "
 
205
             "does not exist in branch:"],
 
206
            ['log', '-r1234'])
293
207
 
294
208
    def test_log_nonexistent_dotted_revno(self):
295
209
        self.make_minimal_branch()
296
 
        self.run_bzr_error(["bzr: ERROR: Requested revision: '123.123' "
297
 
                            "does not exist in branch:"],
298
 
                           ['log',  '-r123.123'])
 
210
        (out, err) = self.run_bzr_error(
 
211
            ["bzr: ERROR: Requested revision: '123.123' "
 
212
             "does not exist in branch:"],
 
213
            ['log',  '-r123.123'])
 
214
 
 
215
    def test_log_change_revno(self):
 
216
        self.make_linear_branch()
 
217
        expected_log = self.run_bzr("log -r 1")[0]
 
218
        log = self.run_bzr("log -c 1")[0]
 
219
        self.assertEqualDiff(expected_log, log)
299
220
 
300
221
    def test_log_change_nonexistent_revno(self):
301
222
        self.make_minimal_branch()
302
 
        self.run_bzr_error(["bzr: ERROR: Requested revision: '1234' "
303
 
                            "does not exist in branch:"],
304
 
                           ['log',  '-c1234'])
 
223
        (out, err) = self.run_bzr_error(
 
224
            ["bzr: ERROR: Requested revision: '1234' "
 
225
             "does not exist in branch:"],
 
226
            ['log',  '-c1234'])
305
227
 
306
228
    def test_log_change_nonexistent_dotted_revno(self):
307
229
        self.make_minimal_branch()
308
 
        self.run_bzr_error(["bzr: ERROR: Requested revision: '123.123' "
309
 
                            "does not exist in branch:"],
310
 
                           ['log', '-c123.123'])
 
230
        (out, err) = self.run_bzr_error(
 
231
            ["bzr: ERROR: Requested revision: '123.123' "
 
232
             "does not exist in branch:"],
 
233
            ['log', '-c123.123'])
311
234
 
312
235
    def test_log_change_single_revno_only(self):
313
236
        self.make_minimal_branch()
329
252
                              'Path unknown at end or start of revision range: '
330
253
                              'does-not-exist')
331
254
 
332
 
    def test_log_reversed_revspecs(self):
333
 
        self.make_linear_branch()
334
 
        self.run_bzr_error(('bzr: ERROR: Start revision must be older than '
335
 
                            'the end revision.\n',),
336
 
                           ['log', '-r3..1'])
337
 
 
338
 
    def test_log_reversed_dotted_revspecs(self):
339
 
        self.make_merged_branch()
340
 
        self.run_bzr_error(('bzr: ERROR: Start revision not found in '
341
 
                            'left-hand history of end revision.\n',),
342
 
                           "log -r 1.1.1..1")
343
 
 
344
 
    def test_log_bad_message_re(self):
345
 
        """Bad --message argument gives a sensible message
346
 
        
347
 
        See https://bugs.launchpad.net/bzr/+bug/251352
348
 
        """
349
 
        self.make_minimal_branch()
350
 
        out, err = self.run_bzr(['log', '-m', '*'], retcode=3)
351
 
        self.assertEqual("bzr: ERROR: Invalid regular expression"
352
 
            " in log message filter"
353
 
            ": '*'"
354
 
            ": nothing to repeat\n", err)
355
 
        self.assertEqual('', out)
356
 
 
357
 
    def test_log_unsupported_timezone(self):
358
 
        self.make_linear_branch()
359
 
        self.run_bzr_error(['bzr: ERROR: Unsupported timezone format "foo", '
360
 
                            'options are "utc", "original", "local".'],
361
 
                           ['log', '--timezone', 'foo'])
362
 
 
363
 
 
364
 
class TestLogTags(TestLog):
365
 
 
366
255
    def test_log_with_tags(self):
367
256
        tree = self.make_linear_branch(format='dirstate-tags')
368
257
        branch = tree.branch
393
282
        log = self.run_bzr("log -n0 -r3.1.1", working_dir='branch2')[0]
394
283
        self.assertContainsRe(log, r'tags: tag1')
395
284
 
 
285
    def test_log_limit(self):
 
286
        tree = self.make_branch_and_tree('.')
 
287
        # We want more commits than our batch size starts at
 
288
        for pos in range(10):
 
289
            tree.commit("%s" % pos)
 
290
        log = self.run_bzr("log --limit 2")[0]
 
291
        self.assertNotContainsRe(log, r'revno: 1\n')
 
292
        self.assertNotContainsRe(log, r'revno: 2\n')
 
293
        self.assertNotContainsRe(log, r'revno: 3\n')
 
294
        self.assertNotContainsRe(log, r'revno: 4\n')
 
295
        self.assertNotContainsRe(log, r'revno: 5\n')
 
296
        self.assertNotContainsRe(log, r'revno: 6\n')
 
297
        self.assertNotContainsRe(log, r'revno: 7\n')
 
298
        self.assertNotContainsRe(log, r'revno: 8\n')
 
299
        self.assertContainsRe(log, r'revno: 9\n')
 
300
        self.assertContainsRe(log, r'revno: 10\n')
 
301
 
 
302
    def test_log_limit_short(self):
 
303
        self.make_linear_branch()
 
304
        log = self.run_bzr("log -l 2")[0]
 
305
        self.assertNotContainsRe(log, r'revno: 1\n')
 
306
        self.assertContainsRe(log, r'revno: 2\n')
 
307
        self.assertContainsRe(log, r'revno: 3\n')
 
308
 
 
309
    def test_log_bad_message_re(self):
 
310
        """Bad --message argument gives a sensible message
 
311
        
 
312
        See https://bugs.launchpad.net/bzr/+bug/251352
 
313
        """
 
314
        self.make_minimal_branch()
 
315
        out, err = self.run_bzr(['log', '-m', '*'], retcode=3)
 
316
        self.assertEqual("bzr: ERROR: Invalid regular expression"
 
317
            " in log message filter"
 
318
            ": '*'"
 
319
            ": nothing to repeat\n", err)
 
320
        self.assertEqual('', out)
 
321
 
 
322
 
 
323
class TestLogTimeZone(TestLog):
 
324
 
 
325
    def test_log_unsupported_timezone(self):
 
326
        self.make_linear_branch()
 
327
        self.run_bzr_error(['bzr: ERROR: Unsupported timezone format "foo", '
 
328
                            'options are "utc", "original", "local".'],
 
329
                           ['log', '--timezone', 'foo'])
 
330
 
396
331
 
397
332
class TestLogVerbose(TestLog):
398
333
 
429
364
        self.assertUseLongDeltaFormat(['log', '--long', '-vv'])
430
365
 
431
366
 
432
 
class TestLogMerges(TestLogWithLogCatcher):
 
367
class TestLogMerges(TestLog):
433
368
 
434
369
    def setUp(self):
435
370
        super(TestLogMerges, self).setUp()
437
372
 
438
373
    def make_branches_with_merges(self):
439
374
        level0 = self.make_branch_and_tree('level0')
440
 
        self.wt_commit(level0, 'in branch level0')
 
375
        level0.commit(message='in branch level0', **self.commit_options())
 
376
 
441
377
        level1 = level0.bzrdir.sprout('level1').open_workingtree()
442
 
        self.wt_commit(level1, 'in branch level1')
 
378
        level1.commit(message='in branch level1', **self.commit_options())
 
379
 
443
380
        level2 = level1.bzrdir.sprout('level2').open_workingtree()
444
 
        self.wt_commit(level2, 'in branch level2')
 
381
        level2.commit(message='in branch level2', **self.commit_options())
 
382
 
445
383
        level1.merge_from_branch(level2.branch)
446
 
        self.wt_commit(level1, 'merge branch level2')
 
384
        level1.commit(message='merge branch level2', **self.commit_options())
 
385
 
447
386
        level0.merge_from_branch(level1.branch)
448
 
        self.wt_commit(level0, 'merge branch level1')
 
387
        level0.commit(message='merge branch level1', **self.commit_options())
449
388
 
450
389
    def test_merges_are_indented_by_level(self):
451
 
        self.run_bzr(['log', '-n0'], working_dir='level0')
452
 
        revnos_and_depth = [(r.revno, r.merge_depth)
453
 
                            for r in self.get_captured_revisions()]
454
 
        self.assertEqual([('2', 0), ('1.1.2', 1), ('1.2.1', 2), ('1.1.1', 1),
455
 
                          ('1', 0)],
456
 
                         [(r.revno, r.merge_depth)
457
 
                            for r in self.get_captured_revisions()])
 
390
        expected = """\
 
391
------------------------------------------------------------
 
392
revno: 2 [merge]
 
393
committer: Lorem Ipsum <test@example.com>
 
394
branch nick: level0
 
395
timestamp: Just now
 
396
message:
 
397
  merge branch level1
 
398
    ------------------------------------------------------------
 
399
    revno: 1.1.2 [merge]
 
400
    committer: Lorem Ipsum <test@example.com>
 
401
    branch nick: level1
 
402
    timestamp: Just now
 
403
    message:
 
404
      merge branch level2
 
405
        ------------------------------------------------------------
 
406
        revno: 1.2.1
 
407
        committer: Lorem Ipsum <test@example.com>
 
408
        branch nick: level2
 
409
        timestamp: Just now
 
410
        message:
 
411
          in branch level2
 
412
    ------------------------------------------------------------
 
413
    revno: 1.1.1
 
414
    committer: Lorem Ipsum <test@example.com>
 
415
    branch nick: level1
 
416
    timestamp: Just now
 
417
    message:
 
418
      in branch level1
 
419
------------------------------------------------------------
 
420
revno: 1
 
421
committer: Lorem Ipsum <test@example.com>
 
422
branch nick: level0
 
423
timestamp: Just now
 
424
message:
 
425
  in branch level0
 
426
"""
 
427
        self.check_log(expected, ['-n0'])
458
428
 
459
429
    def test_force_merge_revisions_off(self):
460
 
        self.assertLogRevnos(['-n1'], ['2', '1'], working_dir='level0')
 
430
        expected = """\
 
431
------------------------------------------------------------
 
432
revno: 2 [merge]
 
433
committer: Lorem Ipsum <test@example.com>
 
434
branch nick: level0
 
435
timestamp: Just now
 
436
message:
 
437
  merge branch level1
 
438
------------------------------------------------------------
 
439
revno: 1
 
440
committer: Lorem Ipsum <test@example.com>
 
441
branch nick: level0
 
442
timestamp: Just now
 
443
message:
 
444
  in branch level0
 
445
"""
 
446
        self.check_log(expected, ['--long', '-n1'])
461
447
 
462
448
    def test_force_merge_revisions_on(self):
463
 
        self.assertLogRevnos(['-n0'], ['2', '1.1.2', '1.2.1', '1.1.1', '1'],
464
 
                             working_dir='level0')
 
449
        expected = """\
 
450
    2 Lorem Ipsum\t2005-11-22 [merge]
 
451
      merge branch level1
 
452
 
 
453
          1.1.2 Lorem Ipsum\t2005-11-22 [merge]
 
454
                merge branch level2
 
455
 
 
456
              1.2.1 Lorem Ipsum\t2005-11-22
 
457
                    in branch level2
 
458
 
 
459
          1.1.1 Lorem Ipsum\t2005-11-22
 
460
                in branch level1
 
461
 
 
462
    1 Lorem Ipsum\t2005-11-22
 
463
      in branch level0
 
464
 
 
465
"""
 
466
        self.check_log(expected, ['--short', '-n0'])
465
467
 
466
468
    def test_include_merges(self):
467
469
        # Confirm --include-merges gives the same output as -n0
468
 
        self.assertLogRevnos(['--include-merges'],
469
 
                             ['2', '1.1.2', '1.2.1', '1.1.1', '1'],
470
 
                             working_dir='level0')
471
 
        self.assertLogRevnos(['--include-merges'],
472
 
                             ['2', '1.1.2', '1.2.1', '1.1.1', '1'],
473
 
                             working_dir='level0')
474
470
        out_im, err_im = self.run_bzr('log --include-merges',
475
471
                                      working_dir='level0')
476
472
        out_n0, err_n0 = self.run_bzr('log -n0', working_dir='level0')
479
475
        self.assertEqual(out_im, out_n0)
480
476
 
481
477
    def test_force_merge_revisions_N(self):
482
 
        self.assertLogRevnos(['-n2'],
483
 
                             ['2', '1.1.2', '1.1.1', '1'],
484
 
                             working_dir='level0')
 
478
        expected = """\
 
479
    2 Lorem Ipsum\t2005-11-22 [merge]
 
480
      merge branch level1
 
481
 
 
482
          1.1.2 Lorem Ipsum\t2005-11-22 [merge]
 
483
                merge branch level2
 
484
 
 
485
          1.1.1 Lorem Ipsum\t2005-11-22
 
486
                in branch level1
 
487
 
 
488
    1 Lorem Ipsum\t2005-11-22
 
489
      in branch level0
 
490
 
 
491
"""
 
492
        self.check_log(expected, ['--short', '-n2'])
485
493
 
486
494
    def test_merges_single_merge_rev(self):
487
 
        self.assertLogRevnosAndDepths(['-n0', '-r1.1.2'],
488
 
                                      [('1.1.2', 0), ('1.2.1', 1)],
489
 
                                      working_dir='level0')
 
495
        expected = """\
 
496
------------------------------------------------------------
 
497
revno: 1.1.2 [merge]
 
498
committer: Lorem Ipsum <test@example.com>
 
499
branch nick: level1
 
500
timestamp: Just now
 
501
message:
 
502
  merge branch level2
 
503
    ------------------------------------------------------------
 
504
    revno: 1.2.1
 
505
    committer: Lorem Ipsum <test@example.com>
 
506
    branch nick: level2
 
507
    timestamp: Just now
 
508
    message:
 
509
      in branch level2
 
510
"""
 
511
        self.check_log(expected, ['-n0', '-r1.1.2'])
490
512
 
491
513
    def test_merges_partial_range(self):
492
 
        self.assertLogRevnosAndDepths(
493
 
                ['-n0', '-r1.1.1..1.1.2'],
494
 
                [('1.1.2', 0), ('1.2.1', 1), ('1.1.1', 0)],
495
 
                working_dir='level0')
496
 
 
497
 
    def test_merges_partial_range_ignore_before_lower_bound(self):
498
 
        """Dont show revisions before the lower bound's merged revs"""
499
 
        self.assertLogRevnosAndDepths(
500
 
                ['-n0', '-r1.1.2..2'],
501
 
                [('2', 0), ('1.1.2', 1), ('1.2.1', 2)],
502
 
                working_dir='level0')
503
 
 
504
 
 
505
 
class TestLogDiff(TestLogWithLogCatcher):
506
 
 
507
 
    # FIXME: We need specific tests for each LogFormatter about how the diffs
508
 
    # are displayed: --long indent them by depth, --short use a fixed
509
 
    # indent and --line does't display them. -- vila 10019
 
514
        expected = """\
 
515
------------------------------------------------------------
 
516
revno: 1.1.2 [merge]
 
517
committer: Lorem Ipsum <test@example.com>
 
518
branch nick: level1
 
519
timestamp: Just now
 
520
message:
 
521
  merge branch level2
 
522
    ------------------------------------------------------------
 
523
    revno: 1.2.1
 
524
    committer: Lorem Ipsum <test@example.com>
 
525
    branch nick: level2
 
526
    timestamp: Just now
 
527
    message:
 
528
      in branch level2
 
529
------------------------------------------------------------
 
530
revno: 1.1.1
 
531
committer: Lorem Ipsum <test@example.com>
 
532
branch nick: level1
 
533
timestamp: Just now
 
534
message:
 
535
  in branch level1
 
536
"""
 
537
        self.check_log(expected, ['-n0', '-r1.1.1..1.1.2'])
 
538
 
 
539
 
 
540
class TestLogDiff(TestLog):
510
541
 
511
542
    def setUp(self):
512
543
        super(TestLogDiff, self).setUp()
517
548
        self.build_tree(['level0/file1', 'level0/file2'])
518
549
        level0.add('file1')
519
550
        level0.add('file2')
520
 
        self.wt_commit(level0, 'in branch level0')
 
551
        level0.commit(message='in branch level0', **self.commit_options())
521
552
 
522
553
        level1 = level0.bzrdir.sprout('level1').open_workingtree()
523
554
        self.build_tree_contents([('level1/file2', 'hello\n')])
524
 
        self.wt_commit(level1, 'in branch level1')
 
555
        level1.commit(message='in branch level1', **self.commit_options())
525
556
        level0.merge_from_branch(level1.branch)
526
 
        self.wt_commit(level0, 'merge branch level1')
 
557
        level0.commit(message='merge branch level1', **self.commit_options())
527
558
 
528
 
    def _diff_file1_revno1(self):
529
 
        return """=== added file 'file1'
 
559
    def test_log_show_diff_long_with_merges(self):
 
560
        out,err = self.run_bzr('log -p -n0')
 
561
        self.assertEqual('', err)
 
562
        log = test_log.normalize_log(out)
 
563
        expected = """\
 
564
------------------------------------------------------------
 
565
revno: 2 [merge]
 
566
committer: Lorem Ipsum <test@example.com>
 
567
branch nick: level0
 
568
timestamp: Just now
 
569
message:
 
570
  merge branch level1
 
571
diff:
 
572
=== modified file 'file2'
 
573
--- file2\t2005-11-22 00:00:01 +0000
 
574
+++ file2\t2005-11-22 00:00:02 +0000
 
575
@@ -1,1 +1,1 @@
 
576
-contents of level0/file2
 
577
+hello
 
578
    ------------------------------------------------------------
 
579
    revno: 1.1.1
 
580
    committer: Lorem Ipsum <test@example.com>
 
581
    branch nick: level1
 
582
    timestamp: Just now
 
583
    message:
 
584
      in branch level1
 
585
    diff:
 
586
    === modified file 'file2'
 
587
    --- file2\t2005-11-22 00:00:01 +0000
 
588
    +++ file2\t2005-11-22 00:00:02 +0000
 
589
    @@ -1,1 +1,1 @@
 
590
    -contents of level0/file2
 
591
    +hello
 
592
------------------------------------------------------------
 
593
revno: 1
 
594
committer: Lorem Ipsum <test@example.com>
 
595
branch nick: level0
 
596
timestamp: Just now
 
597
message:
 
598
  in branch level0
 
599
diff:
 
600
=== added file 'file1'
530
601
--- file1\t1970-01-01 00:00:00 +0000
531
 
+++ file1\t2005-11-22 00:00:00 +0000
 
602
+++ file1\t2005-11-22 00:00:01 +0000
532
603
@@ -0,0 +1,1 @@
533
604
+contents of level0/file1
534
605
 
535
 
"""
536
 
 
537
 
    def _diff_file2_revno2(self):
538
 
        return """=== modified file 'file2'
539
 
--- file2\t2005-11-22 00:00:00 +0000
540
 
+++ file2\t2005-11-22 00:00:01 +0000
541
 
@@ -1,1 +1,1 @@
542
 
-contents of level0/file2
543
 
+hello
544
 
 
545
 
"""
546
 
 
547
 
    def _diff_file2_revno1_1_1(self):
548
 
        return """=== modified file 'file2'
549
 
--- file2\t2005-11-22 00:00:00 +0000
550
 
+++ file2\t2005-11-22 00:00:01 +0000
551
 
@@ -1,1 +1,1 @@
552
 
-contents of level0/file2
553
 
+hello
554
 
 
555
 
"""
556
 
 
557
 
    def _diff_file2_revno1(self):
558
 
        return """=== added file 'file2'
 
606
=== added file 'file2'
559
607
--- file2\t1970-01-01 00:00:00 +0000
560
 
+++ file2\t2005-11-22 00:00:00 +0000
 
608
+++ file2\t2005-11-22 00:00:01 +0000
561
609
@@ -0,0 +1,1 @@
562
610
+contents of level0/file2
563
 
 
564
 
"""
565
 
 
566
 
    def assertLogRevnosAndDiff(self, args, expected,
567
 
                            working_dir='.'):
568
 
        self.run_bzr(['log', '-p'] + args, working_dir=working_dir)
569
 
        expected_revnos_and_depths = [
570
 
            (revno, depth) for revno, depth, diff in expected]
571
 
        # Check the revnos and depths first to make debugging easier
572
 
        self.assertEqual(expected_revnos_and_depths,
573
 
                         [(r.revno, r.merge_depth)
574
 
                           for r in self.get_captured_revisions()])
575
 
        # Now check the diffs, adding the revno  in case of failure
576
 
        fmt = 'In revno %s\n%s'
577
 
        for expected_rev, actual_rev in izip(expected,
578
 
                                             self.get_captured_revisions()):
579
 
            revno, depth, expected_diff = expected_rev
580
 
            actual_diff = actual_rev.diff
581
 
            self.assertEqualDiff(fmt % (revno, expected_diff),
582
 
                                 fmt % (revno, actual_diff))
583
 
 
584
 
    def test_log_diff_with_merges(self):
585
 
        self.assertLogRevnosAndDiff(
586
 
            ['-n0'],
587
 
            [('2', 0, self._diff_file2_revno2()),
588
 
             ('1.1.1', 1, self._diff_file2_revno1_1_1()),
589
 
             ('1', 0, self._diff_file1_revno1()
590
 
              + self._diff_file2_revno1())],
591
 
            working_dir='level0')
592
 
 
593
 
 
594
 
    def test_log_diff_file1(self):
595
 
        self.assertLogRevnosAndDiff(['-n0', 'file1'],
596
 
                                    [('1', 0, self._diff_file1_revno1())],
597
 
                                    working_dir='level0')
598
 
 
599
 
    def test_log_diff_file2(self):
600
 
        self.assertLogRevnosAndDiff(['-n1', 'file2'],
601
 
                                    [('2', 0, self._diff_file2_revno2()),
602
 
                                     ('1', 0, self._diff_file2_revno1())],
603
 
                                    working_dir='level0')
 
611
"""
 
612
        self.check_log(expected, ['-p', '-n0'])
 
613
 
 
614
    def test_log_show_diff_short(self):
 
615
        expected = """\
 
616
    2 Lorem Ipsum\t2005-11-22 [merge]
 
617
      merge branch level1
 
618
      === modified file 'file2'
 
619
      --- file2\t2005-11-22 00:00:01 +0000
 
620
      +++ file2\t2005-11-22 00:00:02 +0000
 
621
      @@ -1,1 +1,1 @@
 
622
      -contents of level0/file2
 
623
      +hello
 
624
 
 
625
    1 Lorem Ipsum\t2005-11-22
 
626
      in branch level0
 
627
      === added file 'file1'
 
628
      --- file1\t1970-01-01 00:00:00 +0000
 
629
      +++ file1\t2005-11-22 00:00:01 +0000
 
630
      @@ -0,0 +1,1 @@
 
631
      +contents of level0/file1
 
632
\x20\x20\x20\x20\x20\x20
 
633
      === added file 'file2'
 
634
      --- file2\t1970-01-01 00:00:00 +0000
 
635
      +++ file2\t2005-11-22 00:00:01 +0000
 
636
      @@ -0,0 +1,1 @@
 
637
      +contents of level0/file2
 
638
 
 
639
Use --include-merges or -n0 to see merged revisions.
 
640
"""
 
641
        self.check_log(expected, ['-p', '--short'])
 
642
 
 
643
    def test_log_show_diff_line(self):
 
644
        # Not supported by this formatter so expect plain output
 
645
        expected = """\
 
646
2: Lorem Ipsum 2005-11-22 [merge] merge branch level1
 
647
1: Lorem Ipsum 2005-11-22 in branch level0
 
648
"""
 
649
        self.check_log(expected, ['-p', '--line'])
 
650
 
 
651
    def test_log_show_diff_file1(self):
 
652
        """Only the diffs for the given file are to be shown"""
 
653
        expected = """\
 
654
    1 Lorem Ipsum\t2005-11-22
 
655
      in branch level0
 
656
      === added file 'file1'
 
657
      --- file1\t1970-01-01 00:00:00 +0000
 
658
      +++ file1\t2005-11-22 00:00:01 +0000
 
659
      @@ -0,0 +1,1 @@
 
660
      +contents of level0/file1
 
661
 
 
662
"""
 
663
        self.check_log(expected, ['-p', '--short', 'file1'])
 
664
 
 
665
    def test_log_show_diff_file2(self):
 
666
        """Only the diffs for the given file are to be shown"""
 
667
        expected = """\
 
668
    2 Lorem Ipsum\t2005-11-22 [merge]
 
669
      merge branch level1
 
670
      === modified file 'file2'
 
671
      --- file2\t2005-11-22 00:00:01 +0000
 
672
      +++ file2\t2005-11-22 00:00:02 +0000
 
673
      @@ -1,1 +1,1 @@
 
674
      -contents of level0/file2
 
675
      +hello
 
676
 
 
677
    1 Lorem Ipsum\t2005-11-22
 
678
      in branch level0
 
679
      === added file 'file2'
 
680
      --- file2\t1970-01-01 00:00:00 +0000
 
681
      +++ file2\t2005-11-22 00:00:01 +0000
 
682
      @@ -0,0 +1,1 @@
 
683
      +contents of level0/file2
 
684
 
 
685
Use --include-merges or -n0 to see merged revisions.
 
686
"""
 
687
        self.check_log(expected, ['-p', '--short', 'file2'])
604
688
 
605
689
 
606
690
class TestLogUnicodeDiff(TestLog):
649
733
 
650
734
    def setUp(self):
651
735
        super(TestLogEncodings, self).setUp()
652
 
        self.overrideAttr(osutils, '_cached_user_encoding')
 
736
        self.user_encoding = osutils._cached_user_encoding
 
737
        def restore():
 
738
            osutils._cached_user_encoding = self.user_encoding
 
739
        self.addCleanup(restore)
653
740
 
654
741
    def create_branch(self):
655
742
        bzr = self.run_bzr
656
743
        bzr('init')
657
 
        self.build_tree_contents([('a', 'some stuff\n')])
 
744
        open('a', 'wb').write('some stuff\n')
658
745
        bzr('add a')
659
746
        bzr(['commit', '-m', self._message])
660
747
 
723
810
        self.assertEquals(-1, stdout.find(test_in_cp1251))
724
811
 
725
812
 
726
 
class TestLogFile(TestLogWithLogCatcher):
 
813
class TestLogFile(tests.TestCaseWithTransport):
727
814
 
728
815
    def test_log_local_branch_file(self):
729
816
        """We should be able to log files in local treeless branches"""
758
845
            tree.commit('remove file1')
759
846
        os.chdir('parent')
760
847
 
761
 
    # FIXME: It would be good to parametrize the following tests against all
762
 
    # formatters. But the revisions selection is not *currently* part of the
763
 
    # LogFormatter contract, so using LogCatcher is sufficient -- vila 100118
764
 
    def test_log_file1(self):
765
 
        self.prepare_tree()
766
 
        self.assertLogRevnos(['-n0', 'file1'], ['1'])
767
 
 
768
 
    def test_log_file2(self):
769
 
        self.prepare_tree()
770
 
        # file2 full history
771
 
        self.assertLogRevnos(['-n0', 'file2'], ['4', '3.1.1', '2'])
772
 
        # file2 in a merge revision
773
 
        self.assertLogRevnos(['-n0', '-r3.1.1', 'file2'], ['3.1.1'])
774
 
        # file2 in a mainline revision
775
 
        self.assertLogRevnos(['-n0', '-r4', 'file2'], ['4', '3.1.1'])
776
 
        # file2 since a revision
777
 
        self.assertLogRevnos(['-n0', '-r3..', 'file2'], ['4', '3.1.1'])
778
 
        # file2 up to a revision
779
 
        self.assertLogRevnos(['-n0', '-r..3', 'file2'], ['2'])
780
 
 
781
 
    def test_log_file3(self):
782
 
        self.prepare_tree()
783
 
        self.assertLogRevnos(['-n0', 'file3'], ['3'])
 
848
    def test_log_file(self):
 
849
        """The log for a particular file should only list revs for that file"""
 
850
        self.prepare_tree()
 
851
        log = self.run_bzr('log -n0 file1')[0]
 
852
        self.assertContainsRe(log, 'revno: 1\n')
 
853
        self.assertNotContainsRe(log, 'revno: 2\n')
 
854
        self.assertNotContainsRe(log, 'revno: 3\n')
 
855
        self.assertNotContainsRe(log, 'revno: 3.1.1\n')
 
856
        self.assertNotContainsRe(log, 'revno: 4 ')
 
857
        log = self.run_bzr('log -n0 file2')[0]
 
858
        self.assertNotContainsRe(log, 'revno: 1\n')
 
859
        self.assertContainsRe(log, 'revno: 2\n')
 
860
        self.assertNotContainsRe(log, 'revno: 3\n')
 
861
        self.assertContainsRe(log, 'revno: 3.1.1\n')
 
862
        self.assertContainsRe(log, 'revno: 4 ')
 
863
        log = self.run_bzr('log -n0 file3')[0]
 
864
        self.assertNotContainsRe(log, 'revno: 1\n')
 
865
        self.assertNotContainsRe(log, 'revno: 2\n')
 
866
        self.assertContainsRe(log, 'revno: 3\n')
 
867
        self.assertNotContainsRe(log, 'revno: 3.1.1\n')
 
868
        self.assertNotContainsRe(log, 'revno: 4 ')
 
869
        log = self.run_bzr('log -n0 -r3.1.1 file2')[0]
 
870
        self.assertNotContainsRe(log, 'revno: 1\n')
 
871
        self.assertNotContainsRe(log, 'revno: 2\n')
 
872
        self.assertNotContainsRe(log, 'revno: 3\n')
 
873
        self.assertContainsRe(log, 'revno: 3.1.1\n')
 
874
        self.assertNotContainsRe(log, 'revno: 4 ')
 
875
        log = self.run_bzr('log -n0 -r4 file2')[0]
 
876
        self.assertNotContainsRe(log, 'revno: 1\n')
 
877
        self.assertNotContainsRe(log, 'revno: 2\n')
 
878
        self.assertNotContainsRe(log, 'revno: 3\n')
 
879
        self.assertContainsRe(log, 'revno: 3.1.1\n')
 
880
        self.assertContainsRe(log, 'revno: 4 ')
 
881
        log = self.run_bzr('log -n0 -r3.. file2')[0]
 
882
        self.assertNotContainsRe(log, 'revno: 1\n')
 
883
        self.assertNotContainsRe(log, 'revno: 2\n')
 
884
        self.assertNotContainsRe(log, 'revno: 3\n')
 
885
        self.assertContainsRe(log, 'revno: 3.1.1\n')
 
886
        self.assertContainsRe(log, 'revno: 4 ')
 
887
        log = self.run_bzr('log -n0 -r..3 file2')[0]
 
888
        self.assertNotContainsRe(log, 'revno: 1\n')
 
889
        self.assertContainsRe(log, 'revno: 2\n')
 
890
        self.assertNotContainsRe(log, 'revno: 3\n')
 
891
        self.assertNotContainsRe(log, 'revno: 3.1.1\n')
 
892
        self.assertNotContainsRe(log, 'revno: 4 ')
784
893
 
785
894
    def test_log_file_historical_missing(self):
786
895
        # Check logging a deleted file gives an error if the
794
903
        # Check logging a deleted file is ok if the file existed
795
904
        # at the end the revision range
796
905
        self.prepare_tree(complex=True)
797
 
        self.assertLogRevnos(['-n0', '-r..4', 'file2'], ['4', '3.1.1', '2'])
 
906
        log, err = self.run_bzr('log -n0 -r..4 file2')
 
907
        self.assertEquals('', err)
 
908
        self.assertNotContainsRe(log, 'revno: 1\n')
 
909
        self.assertContainsRe(log, 'revno: 2\n')
 
910
        self.assertNotContainsRe(log, 'revno: 3\n')
 
911
        self.assertContainsRe(log, 'revno: 3.1.1\n')
 
912
        self.assertContainsRe(log, 'revno: 4 ')
798
913
 
799
914
    def test_log_file_historical_start(self):
800
915
        # Check logging a deleted file is ok if the file existed
801
916
        # at the start of the revision range
802
917
        self.prepare_tree(complex=True)
803
 
        self.assertLogRevnos(['file1'], ['1'])
 
918
        log, err = self.run_bzr('log file1')
 
919
        self.assertEquals('', err)
 
920
        self.assertContainsRe(log, 'revno: 1\n')
 
921
        self.assertNotContainsRe(log, 'revno: 2\n')
 
922
        self.assertNotContainsRe(log, 'revno: 3\n')
 
923
        self.assertNotContainsRe(log, 'revno: 3.1.1\n')
 
924
        self.assertNotContainsRe(log, 'revno: 4 ')
804
925
 
805
926
    def test_log_file_renamed(self):
806
927
        """File matched against revision range, not current tree."""
812
933
        self.assertContainsRe(err, err_msg)
813
934
 
814
935
        # Check we can see a renamed file if we give the right end revision
815
 
        self.assertLogRevnos(['-r..4', 'file3'], ['3'])
816
 
 
817
 
 
818
 
class TestLogMultiple(TestLogWithLogCatcher):
 
936
        log, err = self.run_bzr('log -r..4 file3')
 
937
        self.assertEquals('', err)
 
938
        self.assertNotContainsRe(log, 'revno: 1\n')
 
939
        self.assertNotContainsRe(log, 'revno: 2\n')
 
940
        self.assertContainsRe(log, 'revno: 3\n')
 
941
        self.assertNotContainsRe(log, 'revno: 3.1.1\n')
 
942
        self.assertNotContainsRe(log, 'revno: 4 ')
 
943
 
 
944
    def test_line_log_file(self):
 
945
        """The line log for a file should only list relevant mainline revs"""
 
946
        # Note: this also implicitly  covers the short logging case.
 
947
        # We test using --line in preference to --short because matching
 
948
        # revnos in the output of --line is more reliable.
 
949
        self.prepare_tree()
 
950
 
 
951
        # full history of file1
 
952
        log = self.run_bzr('log --line file1')[0]
 
953
        self.assertContainsRe(log, '^1:', re.MULTILINE)
 
954
        self.assertNotContainsRe(log, '^2:', re.MULTILINE)
 
955
        self.assertNotContainsRe(log, '^3:', re.MULTILINE)
 
956
        self.assertNotContainsRe(log, '^3.1.1:', re.MULTILINE)
 
957
        self.assertNotContainsRe(log, '^4:', re.MULTILINE)
 
958
 
 
959
        # full history of file2
 
960
        log = self.run_bzr('log --line file2')[0]
 
961
        self.assertNotContainsRe(log, '^1:', re.MULTILINE)
 
962
        self.assertContainsRe(log, '^2:', re.MULTILINE)
 
963
        self.assertNotContainsRe(log, '^3:', re.MULTILINE)
 
964
        self.assertNotContainsRe(log, '^3.1.1:', re.MULTILINE)
 
965
        self.assertContainsRe(log, '^4:', re.MULTILINE)
 
966
 
 
967
        # full history of file3
 
968
        log = self.run_bzr('log --line file3')[0]
 
969
        self.assertNotContainsRe(log, '^1:', re.MULTILINE)
 
970
        self.assertNotContainsRe(log, '^2:', re.MULTILINE)
 
971
        self.assertContainsRe(log, '^3:', re.MULTILINE)
 
972
        self.assertNotContainsRe(log, '^3.1.1:', re.MULTILINE)
 
973
        self.assertNotContainsRe(log, '^4:', re.MULTILINE)
 
974
 
 
975
        # file in a merge revision
 
976
        log = self.run_bzr('log --line -r3.1.1 file2')[0]
 
977
        self.assertNotContainsRe(log, '^1:', re.MULTILINE)
 
978
        self.assertNotContainsRe(log, '^2:', re.MULTILINE)
 
979
        self.assertNotContainsRe(log, '^3:', re.MULTILINE)
 
980
        self.assertContainsRe(log, '^3.1.1:', re.MULTILINE)
 
981
        self.assertNotContainsRe(log, '^4:', re.MULTILINE)
 
982
 
 
983
        # file in a mainline revision
 
984
        log = self.run_bzr('log --line -r4 file2')[0]
 
985
        self.assertNotContainsRe(log, '^1:', re.MULTILINE)
 
986
        self.assertNotContainsRe(log, '^2:', re.MULTILINE)
 
987
        self.assertNotContainsRe(log, '^3:', re.MULTILINE)
 
988
        self.assertNotContainsRe(log, '^3.1.1:', re.MULTILINE)
 
989
        self.assertContainsRe(log, '^4:', re.MULTILINE)
 
990
 
 
991
        # file since a revision
 
992
        log = self.run_bzr('log --line -r3.. file2')[0]
 
993
        self.assertNotContainsRe(log, '^1:', re.MULTILINE)
 
994
        self.assertNotContainsRe(log, '^2:', re.MULTILINE)
 
995
        self.assertNotContainsRe(log, '^3:', re.MULTILINE)
 
996
        self.assertNotContainsRe(log, '^3.1.1:', re.MULTILINE)
 
997
        self.assertContainsRe(log, '^4:', re.MULTILINE)
 
998
 
 
999
        # file up to a revision
 
1000
        log = self.run_bzr('log --line -r..3 file2')[0]
 
1001
        self.assertNotContainsRe(log, '^1:', re.MULTILINE)
 
1002
        self.assertContainsRe(log, '^2:', re.MULTILINE)
 
1003
        self.assertNotContainsRe(log, '^3:', re.MULTILINE)
 
1004
        self.assertNotContainsRe(log, '^3.1.1:', re.MULTILINE)
 
1005
        self.assertNotContainsRe(log, '^4:', re.MULTILINE)
 
1006
 
 
1007
 
 
1008
class TestLogMultiple(tests.TestCaseWithTransport):
819
1009
 
820
1010
    def prepare_tree(self):
821
1011
        tree = self.make_branch_and_tree('parent')
844
1034
        tree.commit(message='merge child branch')
845
1035
        os.chdir('parent')
846
1036
 
 
1037
    def assertRevnos(self, paths_str, expected_revnos):
 
1038
        # confirm the revision numbers in log --line output are those expected
 
1039
        out, err = self.run_bzr('log --line -n0 %s' % (paths_str,))
 
1040
        self.assertEqual('', err)
 
1041
        revnos = [s.split(':', 1)[0].lstrip() for s in out.splitlines()]
 
1042
        self.assertEqual(expected_revnos, revnos)
 
1043
 
847
1044
    def test_log_files(self):
848
1045
        """The log for multiple file should only list revs for those files"""
849
1046
        self.prepare_tree()
850
 
        self.assertLogRevnos(['file1', 'file2', 'dir1/dir2/file3'],
851
 
                             ['6', '5.1.1', '3', '2', '1'])
 
1047
        self.assertRevnos('file1 file2 dir1/dir2/file3',
 
1048
            ['6', '5.1.1', '3', '2', '1'])
852
1049
 
853
1050
    def test_log_directory(self):
854
1051
        """The log for a directory should show all nested files."""
855
1052
        self.prepare_tree()
856
 
        self.assertLogRevnos(['dir1'], ['5', '3'])
 
1053
        self.assertRevnos('dir1', ['5', '3'])
857
1054
 
858
1055
    def test_log_nested_directory(self):
859
1056
        """The log for a directory should show all nested files."""
860
1057
        self.prepare_tree()
861
 
        self.assertLogRevnos(['dir1/dir2'], ['3'])
 
1058
        self.assertRevnos('dir1/dir2', ['3'])
862
1059
 
863
1060
    def test_log_in_nested_directory(self):
864
1061
        """The log for a directory should show all nested files."""
865
1062
        self.prepare_tree()
866
1063
        os.chdir("dir1")
867
 
        self.assertLogRevnos(['.'], ['5', '3'])
 
1064
        self.assertRevnos('.', ['5', '3'])
868
1065
 
869
1066
    def test_log_files_and_directories(self):
870
1067
        """Logging files and directories together should be fine."""
871
1068
        self.prepare_tree()
872
 
        self.assertLogRevnos(['file4', 'dir1/dir2'], ['4', '3'])
 
1069
        self.assertRevnos('file4 dir1/dir2', ['4', '3'])
873
1070
 
874
1071
    def test_log_files_and_dirs_in_nested_directory(self):
875
1072
        """The log for a directory should show all nested files."""
876
1073
        self.prepare_tree()
877
1074
        os.chdir("dir1")
878
 
        self.assertLogRevnos(['dir2', 'file5'], ['5', '3'])
 
1075
        self.assertRevnos('dir2 file5', ['5', '3'])