← Back to branch summary

~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

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

  • Committer: Canonical.com Patch Queue Manager
  • Date: 2007-10-24 12:49:17 UTC
  • mfrom: (2935.1.1 ianc-integration)
  • Revision ID: pqm@pqm.ubuntu.com-20071024124917-xb75eckyxx6vkrlg
Makefile fixes - hooks.html generation & allow python to be overridden (Ian Clatworthy)

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/tests/blackbox/test_commit.py
1
 
# Copyright (C) 2005, 2006 by 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
18
18
"""Tests for the commit CLI of bzr."""
19
19
 
20
20
import os
21
 
import re
22
21
import sys
23
22
 
24
 
from bzrlib.branch import Branch
 
23
import bzrlib
 
24
from bzrlib import (
 
25
    osutils,
 
26
    ignores,
 
27
    osutils,
 
28
    )
25
29
from bzrlib.bzrdir import BzrDir
26
 
from bzrlib.errors import BzrCommandError
 
30
from bzrlib.tests import (
 
31
    probe_bad_non_ascii,
 
32
    TestSkipped,
 
33
    )
27
34
from bzrlib.tests.blackbox import ExternalBase
28
 
from bzrlib.workingtree import WorkingTree
29
35
 
30
36
 
31
37
class TestCommit(ExternalBase):
33
39
    def test_05_empty_commit(self):
34
40
        """Commit of tree with no versioned files should fail"""
35
41
        # If forced, it should succeed, but this is not tested here.
36
 
        self.run_bzr("init")
 
42
        self.make_branch_and_tree('.')
37
43
        self.build_tree(['hello.txt'])
38
 
        result = self.run_bzr("commit", "-m", "empty", retcode=3)
39
 
        self.assertEqual(('', 'bzr: ERROR: no changes to commit.'
40
 
                              ' use --unchanged to commit anyhow\n'),
41
 
                         result)
 
44
        out,err = self.run_bzr('commit -m empty', retcode=3)
 
45
        self.assertEqual('', out)
 
46
        self.assertContainsRe(err, 'bzr: ERROR: no changes to commit\.'
 
47
                                  ' use --unchanged to commit anyhow\n')
 
48
 
 
49
    def test_commit_success(self):
 
50
        """Successful commit should not leave behind a bzr-commit-* file"""
 
51
        self.make_branch_and_tree('.')
 
52
        self.run_bzr('commit --unchanged -m message')
 
53
        self.assertEqual('', self.run_bzr('unknowns')[0])
 
54
 
 
55
        # same for unicode messages
 
56
        self.run_bzr(["commit", "--unchanged", "-m", u'foo\xb5'])
 
57
        self.assertEqual('', self.run_bzr('unknowns')[0])
42
58
 
43
59
    def test_commit_with_path(self):
44
60
        """Commit tree with path of root specified"""
45
 
        self.run_bzr('init', 'a')
 
61
        a_tree = self.make_branch_and_tree('a')
46
62
        self.build_tree(['a/a_file'])
47
 
        self.run_bzr('add', 'a/a_file')
48
 
        self.run_bzr('commit', '-m', 'first commit', 'a')
 
63
        a_tree.add('a_file')
 
64
        self.run_bzr(['commit', '-m', 'first commit', 'a'])
49
65
 
50
 
        self.run_bzr('branch', 'a', 'b')
 
66
        b_tree = a_tree.bzrdir.sprout('b').open_workingtree()
51
67
        self.build_tree_contents([('b/a_file', 'changes in b')])
52
 
        self.run_bzr('commit', '-m', 'first commit in b', 'b')
 
68
        self.run_bzr(['commit', '-m', 'first commit in b', 'b'])
53
69
 
54
70
        self.build_tree_contents([('a/a_file', 'new contents')])
55
 
        self.run_bzr('commit', '-m', 'change in a', 'a')
 
71
        self.run_bzr(['commit', '-m', 'change in a', 'a'])
56
72
 
57
 
        os.chdir('b')
58
 
        self.run_bzr('merge', '../a', retcode=1) # will conflict
59
 
        os.chdir('..')
60
 
        self.run_bzr('resolved', 'b/a_file')
61
 
        self.run_bzr('commit', '-m', 'merge into b', 'b')
 
73
        b_tree.merge_from_branch(a_tree.branch)
 
74
        self.assertEqual(len(b_tree.conflicts()), 1)
 
75
        self.run_bzr('resolved b/a_file')
 
76
        self.run_bzr(['commit', '-m', 'merge into b', 'b'])
62
77
 
63
78
 
64
79
    def test_10_verbose_commit(self):
65
80
        """Add one file and examine verbose commit output"""
66
 
        self.runbzr("init")
 
81
        tree = self.make_branch_and_tree('.')
67
82
        self.build_tree(['hello.txt'])
68
 
        self.runbzr("add hello.txt")
69
 
        out,err = self.run_bzr("commit", "-m", "added")
 
83
        tree.add("hello.txt")
 
84
        out,err = self.run_bzr('commit -m added')
70
85
        self.assertEqual('', out)
71
 
        self.assertEqual('added hello.txt\n'
72
 
                         'Committed revision 1.\n',
73
 
                         err)
 
86
        self.assertContainsRe(err, '^Committing revision 1 to ".*"\.\n'
 
87
                              'added hello.txt\n'
 
88
                              'Committed revision 1.\n$',)
74
89
 
75
90
    def prepare_simple_history(self):
76
91
        """Prepare and return a working tree with one commit of one file"""
85
100
        # Verbose commit of modified file should say so
86
101
        wt = self.prepare_simple_history()
87
102
        self.build_tree_contents([('hello.txt', 'new contents')])
88
 
        out, err = self.run_bzr("commit", "-m", "modified")
 
103
        out, err = self.run_bzr('commit -m modified')
89
104
        self.assertEqual('', out)
90
 
        self.assertEqual('modified hello.txt\n'
91
 
                         'Committed revision 2.\n',
92
 
                         err)
 
105
        self.assertContainsRe(err, '^Committing revision 2 to ".*"\.\n'
 
106
                              'modified hello\.txt\n'
 
107
                              'Committed revision 2\.\n$')
93
108
 
94
109
    def test_verbose_commit_renamed(self):
95
110
        # Verbose commit of renamed file should say so
96
111
        wt = self.prepare_simple_history()
97
112
        wt.rename_one('hello.txt', 'gutentag.txt')
98
 
        out, err = self.run_bzr("commit", "-m", "renamed")
 
113
        out, err = self.run_bzr('commit -m renamed')
99
114
        self.assertEqual('', out)
100
 
        self.assertEqual('renamed hello.txt => gutentag.txt\n'
101
 
                         'Committed revision 2.\n',
102
 
                         err)
 
115
        self.assertContainsRe(err, '^Committing revision 2 to ".*"\.\n'
 
116
                              'renamed hello\.txt => gutentag\.txt\n'
 
117
                              'Committed revision 2\.$\n')
103
118
 
104
119
    def test_verbose_commit_moved(self):
105
120
        # Verbose commit of file moved to new directory should say so
107
122
        os.mkdir('subdir')
108
123
        wt.add(['subdir'])
109
124
        wt.rename_one('hello.txt', 'subdir/hello.txt')
110
 
        out, err = self.run_bzr("commit", "-m", "renamed")
 
125
        out, err = self.run_bzr('commit -m renamed')
111
126
        self.assertEqual('', out)
112
 
        self.assertEqualDiff('added subdir\n'
113
 
                             'renamed hello.txt => subdir/hello.txt\n'
114
 
                             'Committed revision 2.\n',
115
 
                             err)
 
127
        self.assertContainsRe(err, '^Committing revision 2 to ".*"\.\n'
 
128
                              'added subdir\n'
 
129
                              'renamed hello\.txt => subdir/hello\.txt\n'
 
130
                              'Committed revision 2\.\n$')
116
131
 
117
132
    def test_verbose_commit_with_unknown(self):
118
133
        """Unknown files should not be listed by default in verbose output"""
120
135
        wt = BzrDir.create_standalone_workingtree('.')
121
136
        self.build_tree(['hello.txt', 'extra.txt'])
122
137
        wt.add(['hello.txt'])
123
 
        out,err = self.run_bzr("commit", "-m", "added")
 
138
        out,err = self.run_bzr('commit -m added')
124
139
        self.assertEqual('', out)
125
 
        self.assertEqual('added hello.txt\n'
126
 
                         'Committed revision 1.\n',
127
 
                         err)
 
140
        self.assertContainsRe(err, '^Committing revision 1 to ".*"\.\n'
 
141
                              'added hello\.txt\n'
 
142
                              'Committed revision 1\.\n$')
128
143
 
129
144
    def test_verbose_commit_with_unchanged(self):
130
145
        """Unchanged files should not be listed by default in verbose output"""
131
 
        self.runbzr("init")
 
146
        tree = self.make_branch_and_tree('.')
132
147
        self.build_tree(['hello.txt', 'unchanged.txt'])
133
 
        self.runbzr('add unchanged.txt')
134
 
        self.runbzr('commit -m unchanged unchanged.txt')
135
 
        self.runbzr("add hello.txt")
136
 
        out,err = self.run_bzr("commit", "-m", "added")
 
148
        tree.add('unchanged.txt')
 
149
        self.run_bzr('commit -m unchanged unchanged.txt')
 
150
        tree.add("hello.txt")
 
151
        out,err = self.run_bzr('commit -m added')
137
152
        self.assertEqual('', out)
138
 
        self.assertEqual('added hello.txt\n'
139
 
                         'Committed revision 2.\n',
140
 
                         err)
 
153
        self.assertContainsRe(err, '^Committing revision 2 to ".*"\.\n'
 
154
                              'added hello\.txt\n'
 
155
                              'Committed revision 2\.$\n')
 
156
 
 
157
    def test_verbose_commit_includes_master_location(self):
 
158
        """Location of master is displayed when committing to bound branch"""
 
159
        a_tree = self.make_branch_and_tree('a')
 
160
        self.build_tree(['a/b'])
 
161
        a_tree.add('b')
 
162
        a_tree.commit(message='Initial message')
 
163
 
 
164
        b_tree = a_tree.branch.create_checkout('b')
 
165
        expected = "%s/" % (osutils.abspath('a'), )
 
166
        out, err = self.run_bzr('commit -m blah --unchanged', working_dir='b')
 
167
        self.assertEqual(err, 'Committing revision 2 to "%s".\n'
 
168
                         'Committed revision 2.\n' % expected)
141
169
 
142
170
    def test_commit_merge_reports_all_modified_files(self):
143
171
        # the commit command should show all the files that are shown by
176
204
            other_tree.rename_one('dirtorename', 'renameddir')
177
205
            other_tree.rename_one('dirtoreparent', 'renameddir/reparenteddir')
178
206
            other_tree.rename_one('filetorename', 'renamedfile')
179
 
            other_tree.rename_one('filetoreparent', 'renameddir/reparentedfile')
 
207
            other_tree.rename_one('filetoreparent',
 
208
                                  'renameddir/reparentedfile')
180
209
            other_tree.remove(['dirtoremove', 'filetoremove'])
181
210
            self.build_tree_contents([
182
 
                ('other/newdir/', ),
 
211
                ('other/newdir/',),
183
212
                ('other/filetomodify', 'new content'),
184
213
                ('other/newfile', 'new file content')])
185
214
            other_tree.add('newfile')
187
216
            other_tree.commit('modify all sample files and dirs.')
188
217
        finally:
189
218
            other_tree.unlock()
190
 
        self.merge(other_tree.branch, this_tree)
 
219
        this_tree.merge_from_branch(other_tree.branch)
191
220
        os.chdir('this')
192
 
        out,err = self.run_bzr("commit", "-m", "added")
193
 
        os.chdir('..')
 
221
        out,err = self.run_bzr('commit -m added')
194
222
        self.assertEqual('', out)
 
223
        expected = '%s/' % (osutils.getcwd(), )
195
224
        self.assertEqualDiff(
 
225
            'Committing revision 2 to "%s".\n'
196
226
            'modified filetomodify\n'
197
227
            'added newdir\n'
198
228
            'added newfile\n'
199
229
            'renamed dirtorename => renameddir\n'
 
230
            'renamed filetorename => renamedfile\n'
200
231
            'renamed dirtoreparent => renameddir/reparenteddir\n'
201
232
            'renamed filetoreparent => renameddir/reparentedfile\n'
202
 
            'renamed filetorename => renamedfile\n'
203
233
            'deleted dirtoremove\n'
204
234
            'deleted filetoremove\n'
205
 
            'Committed revision 2.\n',
 
235
            'Committed revision 2.\n' % (expected, ),
206
236
            err)
207
237
 
208
238
    def test_empty_commit_message(self):
209
 
        self.runbzr("init")
210
 
        file('foo.c', 'wt').write('int main() {}')
211
 
        self.runbzr(['add', 'foo.c'])
212
 
        self.runbzr(["commit", "-m", ""] , retcode=3)
 
239
        tree = self.make_branch_and_tree('.')
 
240
        self.build_tree_contents([('foo.c', 'int main() {}')])
 
241
        tree.add('foo.c')
 
242
        self.run_bzr('commit -m ""', retcode=3)
 
243
 
 
244
    def test_unsupported_encoding_commit_message(self):
 
245
        tree = self.make_branch_and_tree('.')
 
246
        self.build_tree_contents([('foo.c', 'int main() {}')])
 
247
        tree.add('foo.c')
 
248
        # LANG env variable has no effect on Windows
 
249
        # but some characters anyway cannot be represented
 
250
        # in default user encoding
 
251
        char = probe_bad_non_ascii(bzrlib.user_encoding)
 
252
        if char is None:
 
253
            raise TestSkipped('Cannot find suitable non-ascii character'
 
254
                'for user_encoding (%s)' % bzrlib.user_encoding)
 
255
        out,err = self.run_bzr_subprocess('commit -m "%s"' % char,
 
256
                                          retcode=1,
 
257
                                          env_changes={'LANG': 'C'})
 
258
        self.assertContainsRe(err, r'bzrlib.errors.BzrError: Parameter.*is '
 
259
                                    'unsupported by the current encoding.')
213
260
 
214
261
    def test_other_branch_commit(self):
215
262
        # this branch is to ensure consistent behaviour, whether we're run
216
263
        # inside a branch, or not.
217
 
        os.mkdir('empty_branch')
218
 
        os.chdir('empty_branch')
219
 
        self.runbzr('init')
220
 
        os.mkdir('branch')
221
 
        os.chdir('branch')
222
 
        self.runbzr('init')
223
 
        file('foo.c', 'wt').write('int main() {}')
224
 
        file('bar.c', 'wt').write('int main() {}')
225
 
        os.chdir('..')
226
 
        self.runbzr(['add', 'branch/foo.c'])
227
 
        self.runbzr(['add', 'branch'])
 
264
        outer_tree = self.make_branch_and_tree('.')
 
265
        inner_tree = self.make_branch_and_tree('branch')
 
266
        self.build_tree_contents([
 
267
            ('branch/foo.c', 'int main() {}'),
 
268
            ('branch/bar.c', 'int main() {}')])
 
269
        inner_tree.add('foo.c')
 
270
        inner_tree.add('bar.c')
228
271
        # can't commit files in different trees; sane error
229
 
        self.runbzr('commit -m newstuff branch/foo.c .', retcode=3)
230
 
        self.runbzr('commit -m newstuff branch/foo.c')
231
 
        self.runbzr('commit -m newstuff branch')
232
 
        self.runbzr('commit -m newstuff branch', retcode=3)
 
272
        self.run_bzr('commit -m newstuff branch/foo.c .', retcode=3)
 
273
        self.run_bzr('commit -m newstuff branch/foo.c')
 
274
        self.run_bzr('commit -m newstuff branch')
 
275
        self.run_bzr('commit -m newstuff branch', retcode=3)
233
276
 
234
277
    def test_out_of_date_tree_commit(self):
235
278
        # check we get an error code and a clear message committing with an out
236
279
        # of date checkout
237
 
        self.make_branch_and_tree('branch')
 
280
        tree = self.make_branch_and_tree('branch')
238
281
        # make a checkout
239
 
        self.runbzr('checkout --lightweight branch checkout')
 
282
        checkout = tree.branch.create_checkout('checkout', lightweight=True)
240
283
        # commit to the original branch to make the checkout out of date
241
 
        self.runbzr('commit --unchanged -m message branch')
 
284
        tree.commit('message branch', allow_pointless=True)
242
285
        # now commit to the checkout should emit
243
286
        # ERROR: Out of date with the branch, 'bzr update' is suggested
244
 
        output = self.runbzr('commit --unchanged -m checkout_message '
 
287
        output = self.run_bzr('commit --unchanged -m checkout_message '
245
288
                             'checkout', retcode=3)
246
289
        self.assertEqual(output,
247
290
                         ('',
248
 
                          "bzr: ERROR: Working tree is out of date, please run "
249
 
                          "'bzr update'.\n"))
 
291
                          "bzr: ERROR: Working tree is out of date, please "
 
292
                          "run 'bzr update'.\n"))
250
293
 
251
294
    def test_local_commit_unbound(self):
252
295
        # a --local commit on an unbound branch is an error
253
296
        self.make_branch_and_tree('.')
254
 
        out, err = self.run_bzr('commit', '--local', retcode=3)
 
297
        out, err = self.run_bzr('commit --local', retcode=3)
255
298
        self.assertEqualDiff('', out)
256
299
        self.assertEqualDiff('bzr: ERROR: Cannot perform local-only commits '
257
300
                             'on unbound branches.\n', err)
262
305
        # past. This is a user story reported to fail in bug #43959 where 
263
306
        # a merge done in a checkout (using the update command) failed to
264
307
        # commit correctly.
265
 
        self.run_bzr('init', 'trunk')
 
308
        trunk = self.make_branch_and_tree('trunk')
266
309
 
267
 
        self.run_bzr('checkout', 'trunk', 'u1')
 
310
        u1 = trunk.branch.create_checkout('u1')
268
311
        self.build_tree_contents([('u1/hosts', 'initial contents')])
269
 
        self.run_bzr('add', 'u1/hosts')
270
 
        self.run_bzr('commit', '-m', 'add hosts', 'u1')
 
312
        u1.add('hosts')
 
313
        self.run_bzr('commit -m add-hosts u1')
271
314
 
272
 
        self.run_bzr('checkout', 'trunk', 'u2')
 
315
        u2 = trunk.branch.create_checkout('u2')
273
316
        self.build_tree_contents([('u2/hosts', 'altered in u2')])
274
 
        self.run_bzr('commit', '-m', 'checkin from u2', 'u2')
 
317
        self.run_bzr('commit -m checkin-from-u2 u2')
275
318
 
276
319
        # make an offline commits
277
320
        self.build_tree_contents([('u1/hosts', 'first offline change in u1')])
278
 
        self.run_bzr('commit', '-m', 'checkin offline', '--local', 'u1')
 
321
        self.run_bzr('commit -m checkin-offline --local u1')
279
322
 
280
323
        # now try to pull in online work from u2, and then commit our offline
281
324
        # work as a merge
282
325
        # retcode 1 as we expect a text conflict
283
 
        self.run_bzr('update', 'u1', retcode=1)
284
 
        self.run_bzr('resolved', 'u1/hosts')
 
326
        self.run_bzr('update u1', retcode=1)
 
327
        self.run_bzr('resolved u1/hosts')
285
328
        # add a text change here to represent resolving the merge conflicts in
286
329
        # favour of a new version of the file not identical to either the u1
287
330
        # version or the u2 version.
288
331
        self.build_tree_contents([('u1/hosts', 'merge resolution\n')])
289
 
        self.run_bzr('commit', '-m', 'checkin merge of the offline work from u1', 'u1')
 
332
        self.run_bzr('commit -m checkin-merge-of-the-offline-work-from-u1 u1')
290
333
 
291
334
    def test_commit_respects_spec_for_removals(self):
292
335
        """Commit with a file spec should only commit removals that match"""
296
339
        t.commit('Create')
297
340
        t.remove(['file-a', 'dir-a/file-b'])
298
341
        os.chdir('dir-a')
299
 
        result = self.run_bzr('commit', '.', '-m' 'removed file-b')[1]
 
342
        result = self.run_bzr('commit . -m removed-file-b')[1]
300
343
        self.assertNotContainsRe(result, 'file-a')
301
344
        result = self.run_bzr('status')[0]
302
345
        self.assertContainsRe(result, 'removed:\n  file-a')
 
346
 
 
347
    def test_strict_commit(self):
 
348
        """Commit with --strict works if everything is known"""
 
349
        ignores._set_user_ignores([])
 
350
        tree = self.make_branch_and_tree('tree')
 
351
        self.build_tree(['tree/a'])
 
352
        tree.add('a')
 
353
        # A simple change should just work
 
354
        self.run_bzr('commit --strict -m adding-a',
 
355
                     working_dir='tree')
 
356
 
 
357
    def test_strict_commit_no_changes(self):
 
358
        """commit --strict gives "no changes" if there is nothing to commit"""
 
359
        tree = self.make_branch_and_tree('tree')
 
360
        self.build_tree(['tree/a'])
 
361
        tree.add('a')
 
362
        tree.commit('adding a')
 
363
 
 
364
        # With no changes, it should just be 'no changes'
 
365
        # Make sure that commit is failing because there is nothing to do
 
366
        self.run_bzr_error(['no changes to commit'],
 
367
                           'commit --strict -m no-changes',
 
368
                           working_dir='tree')
 
369
 
 
370
        # But --strict doesn't care if you supply --unchanged
 
371
        self.run_bzr('commit --strict --unchanged -m no-changes',
 
372
                     working_dir='tree')
 
373
 
 
374
    def test_strict_commit_unknown(self):
 
375
        """commit --strict fails if a file is unknown"""
 
376
        tree = self.make_branch_and_tree('tree')
 
377
        self.build_tree(['tree/a'])
 
378
        tree.add('a')
 
379
        tree.commit('adding a')
 
380
 
 
381
        # Add one file so there is a change, but forget the other
 
382
        self.build_tree(['tree/b', 'tree/c'])
 
383
        tree.add('b')
 
384
        self.run_bzr_error(['Commit refused because there are unknown files'],
 
385
                           'commit --strict -m add-b',
 
386
                           working_dir='tree')
 
387
 
 
388
        # --no-strict overrides --strict
 
389
        self.run_bzr('commit --strict -m add-b --no-strict',
 
390
                     working_dir='tree')
 
391
 
 
392
    def test_fixes_bug_output(self):
 
393
        """commit --fixes=lp:23452 succeeds without output."""
 
394
        tree = self.make_branch_and_tree('tree')
 
395
        self.build_tree(['tree/hello.txt'])
 
396
        tree.add('hello.txt')
 
397
        output, err = self.run_bzr(
 
398
            'commit -m hello --fixes=lp:23452 tree/hello.txt')
 
399
        self.assertEqual('', output)
 
400
        self.assertContainsRe(err, 'Committing revision 1 to ".*"\.\n'
 
401
                              'added hello\.txt\n'
 
402
                              'Committed revision 1\.\n')
 
403
 
 
404
    def test_no_bugs_no_properties(self):
 
405
        """If no bugs are fixed, the bugs property is not set.
 
406
 
 
407
        see https://beta.launchpad.net/bzr/+bug/109613
 
408
        """
 
409
        tree = self.make_branch_and_tree('tree')
 
410
        self.build_tree(['tree/hello.txt'])
 
411
        tree.add('hello.txt')
 
412
        self.run_bzr( 'commit -m hello tree/hello.txt')
 
413
        # Get the revision properties, ignoring the branch-nick property, which
 
414
        # we don't care about for this test.
 
415
        last_rev = tree.branch.repository.get_revision(tree.last_revision())
 
416
        properties = dict(last_rev.properties)
 
417
        del properties['branch-nick']
 
418
        self.assertFalse('bugs' in properties)
 
419
 
 
420
    def test_fixes_bug_sets_property(self):
 
421
        """commit --fixes=lp:234 sets the lp:234 revprop to 'fixed'."""
 
422
        tree = self.make_branch_and_tree('tree')
 
423
        self.build_tree(['tree/hello.txt'])
 
424
        tree.add('hello.txt')
 
425
        self.run_bzr('commit -m hello --fixes=lp:234 tree/hello.txt')
 
426
 
 
427
        # Get the revision properties, ignoring the branch-nick property, which
 
428
        # we don't care about for this test.
 
429
        last_rev = tree.branch.repository.get_revision(tree.last_revision())
 
430
        properties = dict(last_rev.properties)
 
431
        del properties['branch-nick']
 
432
 
 
433
        self.assertEqual({'bugs': 'https://launchpad.net/bugs/234 fixed'},
 
434
                         properties)
 
435
 
 
436
    def test_fixes_multiple_bugs_sets_properties(self):
 
437
        """--fixes can be used more than once to show that bugs are fixed."""
 
438
        tree = self.make_branch_and_tree('tree')
 
439
        self.build_tree(['tree/hello.txt'])
 
440
        tree.add('hello.txt')
 
441
        self.run_bzr('commit -m hello --fixes=lp:123 --fixes=lp:235'
 
442
                     ' tree/hello.txt')
 
443
 
 
444
        # Get the revision properties, ignoring the branch-nick property, which
 
445
        # we don't care about for this test.
 
446
        last_rev = tree.branch.repository.get_revision(tree.last_revision())
 
447
        properties = dict(last_rev.properties)
 
448
        del properties['branch-nick']
 
449
 
 
450
        self.assertEqual(
 
451
            {'bugs': 'https://launchpad.net/bugs/123 fixed\n'
 
452
                     'https://launchpad.net/bugs/235 fixed'},
 
453
            properties)
 
454
 
 
455
    def test_fixes_bug_with_alternate_trackers(self):
 
456
        """--fixes can be used on a properly configured branch to mark bug
 
457
        fixes on multiple trackers.
 
458
        """
 
459
        tree = self.make_branch_and_tree('tree')
 
460
        tree.branch.get_config().set_user_option(
 
461
            'trac_twisted_url', 'http://twistedmatrix.com/trac')
 
462
        self.build_tree(['tree/hello.txt'])
 
463
        tree.add('hello.txt')
 
464
        self.run_bzr('commit -m hello --fixes=lp:123 --fixes=twisted:235 tree/')
 
465
 
 
466
        # Get the revision properties, ignoring the branch-nick property, which
 
467
        # we don't care about for this test.
 
468
        last_rev = tree.branch.repository.get_revision(tree.last_revision())
 
469
        properties = dict(last_rev.properties)
 
470
        del properties['branch-nick']
 
471
 
 
472
        self.assertEqual(
 
473
            {'bugs': 'https://launchpad.net/bugs/123 fixed\n'
 
474
                     'http://twistedmatrix.com/trac/ticket/235 fixed'},
 
475
            properties)
 
476
 
 
477
    def test_fixes_unknown_bug_prefix(self):
 
478
        tree = self.make_branch_and_tree('tree')
 
479
        self.build_tree(['tree/hello.txt'])
 
480
        tree.add('hello.txt')
 
481
        self.run_bzr_error(
 
482
            ["Unrecognized bug %s. Commit refused." % 'xxx:123'],
 
483
            'commit -m add-b --fixes=xxx:123',
 
484
            working_dir='tree')
 
485
 
 
486
    def test_fixes_invalid_bug_number(self):
 
487
        tree = self.make_branch_and_tree('tree')
 
488
        self.build_tree(['tree/hello.txt'])
 
489
        tree.add('hello.txt')
 
490
        self.run_bzr_error(
 
491
            ["Invalid bug identifier for %s. Commit refused." % 'lp:orange'],
 
492
            'commit -m add-b --fixes=lp:orange',
 
493
            working_dir='tree')
 
494
 
 
495
    def test_fixes_invalid_argument(self):
 
496
        """Raise an appropriate error when the fixes argument isn't tag:id."""
 
497
        tree = self.make_branch_and_tree('tree')
 
498
        self.build_tree(['tree/hello.txt'])
 
499
        tree.add('hello.txt')
 
500
        self.run_bzr_error(
 
501
            [r"Invalid bug orange. Must be in the form of 'tag:id'\. "
 
502
             r"Commit refused\."],
 
503
            'commit -m add-b --fixes=orange',
 
504
            working_dir='tree')
 
505
 
 
506
    def test_no_author(self):
 
507
        """If the author is not specified, the author property is not set."""
 
508
        tree = self.make_branch_and_tree('tree')
 
509
        self.build_tree(['tree/hello.txt'])
 
510
        tree.add('hello.txt')
 
511
        self.run_bzr( 'commit -m hello tree/hello.txt')
 
512
        last_rev = tree.branch.repository.get_revision(tree.last_revision())
 
513
        properties = last_rev.properties
 
514
        self.assertFalse('author' in properties)
 
515
 
 
516
    def test_author_sets_property(self):
 
517
        """commit --author='John Doe <jdoe@example.com>' sets the author
 
518
           revprop.
 
519
        """
 
520
        tree = self.make_branch_and_tree('tree')
 
521
        self.build_tree(['tree/hello.txt'])
 
522
        tree.add('hello.txt')
 
523
        self.run_bzr("commit -m hello --author='John Doe <jdoe@example.com>' "
 
524
                     "tree/hello.txt")
 
525
        last_rev = tree.branch.repository.get_revision(tree.last_revision())
 
526
        properties = last_rev.properties
 
527
        self.assertEqual('John Doe <jdoe@example.com>', properties['author'])
 
528
 
 
529
    def test_author_no_email(self):
 
530
        """Author's name without an email address is allowed, too."""
 
531
        tree = self.make_branch_and_tree('tree')
 
532
        self.build_tree(['tree/hello.txt'])
 
533
        tree.add('hello.txt')
 
534
        out, err = self.run_bzr("commit -m hello --author='John Doe' "
 
535
                                "tree/hello.txt")
 
536
        last_rev = tree.branch.repository.get_revision(tree.last_revision())
 
537
        properties = last_rev.properties
 
538
        self.assertEqual('John Doe', properties['author'])
 
539
 
 
540
    def test_partial_commit_with_renames_in_tree(self):
 
541
        # this test illustrates bug #140419
 
542
        t = self.make_branch_and_tree('.')
 
543
        self.build_tree(['dir/', 'dir/a', 'test'])
 
544
        t.add(['dir/', 'dir/a', 'test'])
 
545
        t.commit('initial commit')
 
546
        # important part: file dir/a should change parent
 
547
        # and should appear before old parent
 
548
        # then during partial commit we have error
 
549
        # parent_id {dir-XXX} not in inventory
 
550
        t.rename_one('dir/a', 'a')
 
551
        self.build_tree_contents([('test', 'changes in test')])
 
552
        # partial commit
 
553
        out, err = self.run_bzr('commit test -m "partial commit"')
 
554
        self.assertEquals('', out)
 
555
        self.assertContainsRe(err, r'modified test\nCommitted revision 2.')
 
556
 
 
557
    def test_commit_readonly_checkout(self):
 
558
        # https://bugs.edge.launchpad.net/bzr/+bug/129701
 
559
        # "UnlockableTransport error trying to commit in checkout of readonly
 
560
        # branch"
 
561
        self.make_branch('master')
 
562
        master = BzrDir.open_from_transport(
 
563
            self.get_readonly_transport('master')).open_branch()
 
564
        master.create_checkout('checkout')
 
565
        out, err = self.run_bzr(['commit', '--unchanged', '-mfoo', 'checkout'],
 
566
            retcode=3)
 
567
        self.assertContainsRe(err,
 
568
            r'^bzr: ERROR: Cannot lock.*readonly transport')