/brz/remove-bazaar

To get this branch, use:
bzr branch http://gegoxaren.bato24.eu/bzr/brz/remove-bazaar

« back to all changes in this revision

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

First attempt to merge .dev and resolve the conflicts (but tests are 
failing)

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 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 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 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 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 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 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 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')
189
218
            other_tree.unlock()
190
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 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')
 
333
 
 
334
    def test_commit_exclude_excludes_modified_files(self):
 
335
        """Commit -x foo should ignore changes to foo."""
 
336
        tree = self.make_branch_and_tree('.')
 
337
        self.build_tree(['a', 'b', 'c'])
 
338
        tree.smart_add(['.'])
 
339
        out, err = self.run_bzr(['commit', '-m', 'test', '-x', 'b'])
 
340
        self.assertFalse('added b' in out)
 
341
        self.assertFalse('added b' in err)
 
342
        # If b was excluded it will still be 'added' in status.
 
343
        out, err = self.run_bzr(['added'])
 
344
        self.assertEqual('b\n', out)
 
345
        self.assertEqual('', err)
 
346
 
 
347
    def test_commit_exclude_twice_uses_both_rules(self):
 
348
        """Commit -x foo -x bar should ignore changes to foo and bar."""
 
349
        tree = self.make_branch_and_tree('.')
 
350
        self.build_tree(['a', 'b', 'c'])
 
351
        tree.smart_add(['.'])
 
352
        out, err = self.run_bzr(['commit', '-m', 'test', '-x', 'b', '-x', 'c'])
 
353
        self.assertFalse('added b' in out)
 
354
        self.assertFalse('added c' in out)
 
355
        self.assertFalse('added b' in err)
 
356
        self.assertFalse('added c' in err)
 
357
        # If b was excluded it will still be 'added' in status.
 
358
        out, err = self.run_bzr(['added'])
 
359
        self.assertTrue('b\n' in out)
 
360
        self.assertTrue('c\n' in out)
 
361
        self.assertEqual('', err)
290
362
 
291
363
    def test_commit_respects_spec_for_removals(self):
292
364
        """Commit with a file spec should only commit removals that match"""
296
368
        t.commit('Create')
297
369
        t.remove(['file-a', 'dir-a/file-b'])
298
370
        os.chdir('dir-a')
299
 
        result = self.run_bzr('commit', '.', '-m' 'removed file-b')[1]
 
371
        result = self.run_bzr('commit . -m removed-file-b')[1]
300
372
        self.assertNotContainsRe(result, 'file-a')
301
373
        result = self.run_bzr('status')[0]
302
374
        self.assertContainsRe(result, 'removed:\n  file-a')
 
375
 
 
376
    def test_strict_commit(self):
 
377
        """Commit with --strict works if everything is known"""
 
378
        ignores._set_user_ignores([])
 
379
        tree = self.make_branch_and_tree('tree')
 
380
        self.build_tree(['tree/a'])
 
381
        tree.add('a')
 
382
        # A simple change should just work
 
383
        self.run_bzr('commit --strict -m adding-a',
 
384
                     working_dir='tree')
 
385
 
 
386
    def test_strict_commit_no_changes(self):
 
387
        """commit --strict gives "no changes" if there is nothing to commit"""
 
388
        tree = self.make_branch_and_tree('tree')
 
389
        self.build_tree(['tree/a'])
 
390
        tree.add('a')
 
391
        tree.commit('adding a')
 
392
 
 
393
        # With no changes, it should just be 'no changes'
 
394
        # Make sure that commit is failing because there is nothing to do
 
395
        self.run_bzr_error(['no changes to commit'],
 
396
                           'commit --strict -m no-changes',
 
397
                           working_dir='tree')
 
398
 
 
399
        # But --strict doesn't care if you supply --unchanged
 
400
        self.run_bzr('commit --strict --unchanged -m no-changes',
 
401
                     working_dir='tree')
 
402
 
 
403
    def test_strict_commit_unknown(self):
 
404
        """commit --strict fails if a file is unknown"""
 
405
        tree = self.make_branch_and_tree('tree')
 
406
        self.build_tree(['tree/a'])
 
407
        tree.add('a')
 
408
        tree.commit('adding a')
 
409
 
 
410
        # Add one file so there is a change, but forget the other
 
411
        self.build_tree(['tree/b', 'tree/c'])
 
412
        tree.add('b')
 
413
        self.run_bzr_error(['Commit refused because there are unknown files'],
 
414
                           'commit --strict -m add-b',
 
415
                           working_dir='tree')
 
416
 
 
417
        # --no-strict overrides --strict
 
418
        self.run_bzr('commit --strict -m add-b --no-strict',
 
419
                     working_dir='tree')
 
420
 
 
421
    def test_fixes_bug_output(self):
 
422
        """commit --fixes=lp:23452 succeeds without output."""
 
423
        tree = self.make_branch_and_tree('tree')
 
424
        self.build_tree(['tree/hello.txt'])
 
425
        tree.add('hello.txt')
 
426
        output, err = self.run_bzr(
 
427
            'commit -m hello --fixes=lp:23452 tree/hello.txt')
 
428
        self.assertEqual('', output)
 
429
        self.assertContainsRe(err, 'Committing to: .*\n'
 
430
                              'added hello\.txt\n'
 
431
                              'Committed revision 1\.\n')
 
432
 
 
433
    def test_no_bugs_no_properties(self):
 
434
        """If no bugs are fixed, the bugs property is not set.
 
435
 
 
436
        see https://beta.launchpad.net/bzr/+bug/109613
 
437
        """
 
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 tree/hello.txt')
 
442
        # Get the revision properties, ignoring the branch-nick property, which
 
443
        # we don't care about for this test.
 
444
        last_rev = tree.branch.repository.get_revision(tree.last_revision())
 
445
        properties = dict(last_rev.properties)
 
446
        del properties['branch-nick']
 
447
        self.assertFalse('bugs' in properties)
 
448
 
 
449
    def test_fixes_bug_sets_property(self):
 
450
        """commit --fixes=lp:234 sets the lp:234 revprop to 'fixed'."""
 
451
        tree = self.make_branch_and_tree('tree')
 
452
        self.build_tree(['tree/hello.txt'])
 
453
        tree.add('hello.txt')
 
454
        self.run_bzr('commit -m hello --fixes=lp:234 tree/hello.txt')
 
455
 
 
456
        # Get the revision properties, ignoring the branch-nick property, which
 
457
        # we don't care about for this test.
 
458
        last_rev = tree.branch.repository.get_revision(tree.last_revision())
 
459
        properties = dict(last_rev.properties)
 
460
        del properties['branch-nick']
 
461
 
 
462
        self.assertEqual({'bugs': 'https://launchpad.net/bugs/234 fixed'},
 
463
                         properties)
 
464
 
 
465
    def test_fixes_multiple_bugs_sets_properties(self):
 
466
        """--fixes can be used more than once to show that bugs are fixed."""
 
467
        tree = self.make_branch_and_tree('tree')
 
468
        self.build_tree(['tree/hello.txt'])
 
469
        tree.add('hello.txt')
 
470
        self.run_bzr('commit -m hello --fixes=lp:123 --fixes=lp:235'
 
471
                     ' tree/hello.txt')
 
472
 
 
473
        # Get the revision properties, ignoring the branch-nick property, which
 
474
        # we don't care about for this test.
 
475
        last_rev = tree.branch.repository.get_revision(tree.last_revision())
 
476
        properties = dict(last_rev.properties)
 
477
        del properties['branch-nick']
 
478
 
 
479
        self.assertEqual(
 
480
            {'bugs': 'https://launchpad.net/bugs/123 fixed\n'
 
481
                     'https://launchpad.net/bugs/235 fixed'},
 
482
            properties)
 
483
 
 
484
    def test_fixes_bug_with_alternate_trackers(self):
 
485
        """--fixes can be used on a properly configured branch to mark bug
 
486
        fixes on multiple trackers.
 
487
        """
 
488
        tree = self.make_branch_and_tree('tree')
 
489
        tree.branch.get_config().set_user_option(
 
490
            'trac_twisted_url', 'http://twistedmatrix.com/trac')
 
491
        self.build_tree(['tree/hello.txt'])
 
492
        tree.add('hello.txt')
 
493
        self.run_bzr('commit -m hello --fixes=lp:123 --fixes=twisted:235 tree/')
 
494
 
 
495
        # Get the revision properties, ignoring the branch-nick property, which
 
496
        # we don't care about for this test.
 
497
        last_rev = tree.branch.repository.get_revision(tree.last_revision())
 
498
        properties = dict(last_rev.properties)
 
499
        del properties['branch-nick']
 
500
 
 
501
        self.assertEqual(
 
502
            {'bugs': 'https://launchpad.net/bugs/123 fixed\n'
 
503
                     'http://twistedmatrix.com/trac/ticket/235 fixed'},
 
504
            properties)
 
505
 
 
506
    def test_fixes_unknown_bug_prefix(self):
 
507
        tree = self.make_branch_and_tree('tree')
 
508
        self.build_tree(['tree/hello.txt'])
 
509
        tree.add('hello.txt')
 
510
        self.run_bzr_error(
 
511
            ["Unrecognized bug %s. Commit refused." % 'xxx:123'],
 
512
            'commit -m add-b --fixes=xxx:123',
 
513
            working_dir='tree')
 
514
 
 
515
    def test_fixes_invalid_bug_number(self):
 
516
        tree = self.make_branch_and_tree('tree')
 
517
        self.build_tree(['tree/hello.txt'])
 
518
        tree.add('hello.txt')
 
519
        self.run_bzr_error(
 
520
            ["Invalid bug identifier for %s. Commit refused." % 'lp:orange'],
 
521
            'commit -m add-b --fixes=lp:orange',
 
522
            working_dir='tree')
 
523
 
 
524
    def test_fixes_invalid_argument(self):
 
525
        """Raise an appropriate error when the fixes argument isn't tag:id."""
 
526
        tree = self.make_branch_and_tree('tree')
 
527
        self.build_tree(['tree/hello.txt'])
 
528
        tree.add('hello.txt')
 
529
        self.run_bzr_error(
 
530
            [r"Invalid bug orange. Must be in the form of 'tag:id'\. "
 
531
             r"Commit refused\."],
 
532
            'commit -m add-b --fixes=orange',
 
533
            working_dir='tree')
 
534
 
 
535
    def test_no_author(self):
 
536
        """If the author is not specified, the author property is not set."""
 
537
        tree = self.make_branch_and_tree('tree')
 
538
        self.build_tree(['tree/hello.txt'])
 
539
        tree.add('hello.txt')
 
540
        self.run_bzr( 'commit -m hello tree/hello.txt')
 
541
        last_rev = tree.branch.repository.get_revision(tree.last_revision())
 
542
        properties = last_rev.properties
 
543
        self.assertFalse('author' in properties)
 
544
 
 
545
    def test_author_sets_property(self):
 
546
        """commit --author='John Doe <jdoe@example.com>' sets the author
 
547
           revprop.
 
548
        """
 
549
        tree = self.make_branch_and_tree('tree')
 
550
        self.build_tree(['tree/hello.txt'])
 
551
        tree.add('hello.txt')
 
552
        self.run_bzr(["commit", '-m', 'hello',
 
553
                      '--author', u'John D\xf6 <jdoe@example.com>',
 
554
                     "tree/hello.txt"])
 
555
        last_rev = tree.branch.repository.get_revision(tree.last_revision())
 
556
        properties = last_rev.properties
 
557
        self.assertEqual(u'John D\xf6 <jdoe@example.com>', properties['author'])
 
558
 
 
559
    def test_author_no_email(self):
 
560
        """Author's name without an email address is allowed, too."""
 
561
        tree = self.make_branch_and_tree('tree')
 
562
        self.build_tree(['tree/hello.txt'])
 
563
        tree.add('hello.txt')
 
564
        out, err = self.run_bzr("commit -m hello --author='John Doe' "
 
565
                                "tree/hello.txt")
 
566
        last_rev = tree.branch.repository.get_revision(tree.last_revision())
 
567
        properties = last_rev.properties
 
568
        self.assertEqual('John Doe', properties['author'])
 
569
 
 
570
    def test_partial_commit_with_renames_in_tree(self):
 
571
        # this test illustrates bug #140419
 
572
        t = self.make_branch_and_tree('.')
 
573
        self.build_tree(['dir/', 'dir/a', 'test'])
 
574
        t.add(['dir/', 'dir/a', 'test'])
 
575
        t.commit('initial commit')
 
576
        # important part: file dir/a should change parent
 
577
        # and should appear before old parent
 
578
        # then during partial commit we have error
 
579
        # parent_id {dir-XXX} not in inventory
 
580
        t.rename_one('dir/a', 'a')
 
581
        self.build_tree_contents([('test', 'changes in test')])
 
582
        # partial commit
 
583
        out, err = self.run_bzr('commit test -m "partial commit"')
 
584
        self.assertEquals('', out)
 
585
        self.assertContainsRe(err, r'modified test\nCommitted revision 2.')
 
586
 
 
587
    def test_commit_readonly_checkout(self):
 
588
        # https://bugs.edge.launchpad.net/bzr/+bug/129701
 
589
        # "UnlockableTransport error trying to commit in checkout of readonly
 
590
        # branch"
 
591
        self.make_branch('master')
 
592
        master = BzrDir.open_from_transport(
 
593
            self.get_readonly_transport('master')).open_branch()
 
594
        master.create_checkout('checkout')
 
595
        out, err = self.run_bzr(['commit', '--unchanged', '-mfoo', 'checkout'],
 
596
            retcode=3)
 
597
        self.assertContainsRe(err,
 
598
            r'^bzr: ERROR: Cannot lock.*readonly transport')