/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

  • Committer: Robert Collins
  • Date: 2007-07-15 15:40:37 UTC
  • mto: (2592.3.33 repository)
  • mto: This revision was merged to the branch mainline in revision 2624.
  • Revision ID: robertc@robertcollins.net-20070715154037-3ar8g89decddc9su
Make GraphIndex accept nodes as key, value, references, so that the method
signature is closer to what a simple key->value index delivers. Also
change the behaviour when the reference list count is zero to accept
key, value as nodes, and emit key, value to make it identical in that case
to a simple key->value index. This may not be a good idea, but for now it
seems ok.

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/tests/blackbox/test_commit.py
1
 
# Copyright (C) 2006-2010 Canonical Ltd
 
1
# Copyright (C) 2005, 2006 Canonical Ltd
2
2
#
3
3
# This program is free software; you can redistribute it and/or modify
4
4
# it under the terms of the GNU General Public License as published by
12
12
#
13
13
# You should have received a copy of the GNU General Public License
14
14
# along with this program; if not, write to the Free Software
15
 
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
 
15
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
16
16
 
17
17
 
18
18
"""Tests for the commit CLI of bzr."""
22
22
import sys
23
23
 
24
24
from bzrlib import (
25
 
    osutils,
26
25
    ignores,
27
 
    msgeditor,
28
 
    osutils,
29
 
    tests,
30
26
    )
 
27
from bzrlib.branch import Branch
31
28
from bzrlib.bzrdir import BzrDir
32
 
from bzrlib.tests import (
33
 
    probe_bad_non_ascii,
34
 
    TestSkipped,
35
 
    )
 
29
from bzrlib.errors import BzrCommandError
36
30
from bzrlib.tests.blackbox import ExternalBase
 
31
from bzrlib.workingtree import WorkingTree
37
32
 
38
33
 
39
34
class TestCommit(ExternalBase):
41
36
    def test_05_empty_commit(self):
42
37
        """Commit of tree with no versioned files should fail"""
43
38
        # If forced, it should succeed, but this is not tested here.
44
 
        self.make_branch_and_tree('.')
 
39
        self.run_bzr("init")
45
40
        self.build_tree(['hello.txt'])
46
41
        out,err = self.run_bzr('commit -m empty', retcode=3)
47
42
        self.assertEqual('', out)
48
 
        self.assertContainsRe(err, 'bzr: ERROR: No changes to commit\.'
49
 
                                  ' Use --unchanged to commit anyhow.\n')
 
43
        self.assertStartsWith(err, 'bzr: ERROR: no changes to commit.'
 
44
                                  ' use --unchanged to commit anyhow\n')
50
45
 
51
46
    def test_commit_success(self):
52
47
        """Successful commit should not leave behind a bzr-commit-* file"""
53
 
        self.make_branch_and_tree('.')
 
48
        self.run_bzr("init")
54
49
        self.run_bzr('commit --unchanged -m message')
55
50
        self.assertEqual('', self.run_bzr('unknowns')[0])
56
51
 
60
55
 
61
56
    def test_commit_with_path(self):
62
57
        """Commit tree with path of root specified"""
63
 
        a_tree = self.make_branch_and_tree('a')
 
58
        self.run_bzr('init a')
64
59
        self.build_tree(['a/a_file'])
65
 
        a_tree.add('a_file')
 
60
        self.run_bzr('add a/a_file')
66
61
        self.run_bzr(['commit', '-m', 'first commit', 'a'])
67
62
 
68
 
        b_tree = a_tree.bzrdir.sprout('b').open_workingtree()
 
63
        self.run_bzr('branch a b')
69
64
        self.build_tree_contents([('b/a_file', 'changes in b')])
70
65
        self.run_bzr(['commit', '-m', 'first commit in b', 'b'])
71
66
 
72
67
        self.build_tree_contents([('a/a_file', 'new contents')])
73
68
        self.run_bzr(['commit', '-m', 'change in a', 'a'])
74
69
 
75
 
        b_tree.merge_from_branch(a_tree.branch)
76
 
        self.assertEqual(len(b_tree.conflicts()), 1)
 
70
        os.chdir('b')
 
71
        self.run_bzr('merge ../a', retcode=1) # will conflict
 
72
        os.chdir('..')
77
73
        self.run_bzr('resolved b/a_file')
78
74
        self.run_bzr(['commit', '-m', 'merge into b', 'b'])
79
75
 
80
76
 
81
77
    def test_10_verbose_commit(self):
82
78
        """Add one file and examine verbose commit output"""
83
 
        tree = self.make_branch_and_tree('.')
 
79
        self.run_bzr("init")
84
80
        self.build_tree(['hello.txt'])
85
 
        tree.add("hello.txt")
 
81
        self.run_bzr("add hello.txt")
86
82
        out,err = self.run_bzr('commit -m added')
87
83
        self.assertEqual('', out)
88
 
        self.assertContainsRe(err, '^Committing to: .*\n'
89
 
                              'added hello.txt\n'
90
 
                              'Committed revision 1.\n$',)
 
84
        self.assertEqual('added hello.txt\n'
 
85
                         'Committed revision 1.\n',
 
86
                         err)
91
87
 
92
88
    def prepare_simple_history(self):
93
89
        """Prepare and return a working tree with one commit of one file"""
104
100
        self.build_tree_contents([('hello.txt', 'new contents')])
105
101
        out, err = self.run_bzr('commit -m modified')
106
102
        self.assertEqual('', out)
107
 
        self.assertContainsRe(err, '^Committing to: .*\n'
108
 
                              'modified hello\.txt\n'
109
 
                              'Committed revision 2\.\n$')
110
 
 
111
 
    def test_unicode_commit_message_is_filename(self):
112
 
        """Unicode commit message same as a filename (Bug #563646).
113
 
        """
114
 
        file_name = u'\N{euro sign}'
115
 
        self.run_bzr(['init'])
116
 
        open(file_name, 'w').write('hello world')
117
 
        self.run_bzr(['add'])
118
 
        out, err = self.run_bzr(['commit', '-m', file_name])
119
 
        reflags = re.MULTILINE|re.DOTALL|re.UNICODE
120
 
        te = osutils.get_terminal_encoding()
121
 
        self.assertContainsRe(err.decode(te),
122
 
            u'The commit message is a file name:',
123
 
            flags=reflags)
124
 
 
125
 
        # Run same test with a filename that causes encode
126
 
        # error for the terminal encoding. We do this
127
 
        # by forcing terminal encoding of ascii for
128
 
        # osutils.get_terminal_encoding which is used
129
 
        # by ui.text.show_warning
130
 
        default_get_terminal_enc = osutils.get_terminal_encoding
131
 
        try:
132
 
            osutils.get_terminal_encoding = lambda: 'ascii'
133
 
            file_name = u'foo\u1234'
134
 
            open(file_name, 'w').write('hello world')
135
 
            self.run_bzr(['add'])
136
 
            out, err = self.run_bzr(['commit', '-m', file_name])
137
 
            reflags = re.MULTILINE|re.DOTALL|re.UNICODE
138
 
            te = osutils.get_terminal_encoding()
139
 
            self.assertContainsRe(err.decode(te, 'replace'),
140
 
                u'The commit message is a file name:',
141
 
                flags=reflags)
142
 
        finally:
143
 
            osutils.get_terminal_encoding = default_get_terminal_enc
144
 
 
145
 
    def test_warn_about_forgotten_commit_message(self):
146
 
        """Test that the lack of -m parameter is caught"""
147
 
        wt = self.make_branch_and_tree('.')
148
 
        self.build_tree(['one', 'two'])
149
 
        wt.add(['two'])
150
 
        out, err = self.run_bzr('commit -m one two')
151
 
        self.assertContainsRe(err, "The commit message is a file name")
 
103
        self.assertEqual('modified hello.txt\n'
 
104
                         'Committed revision 2.\n',
 
105
                         err)
152
106
 
153
107
    def test_verbose_commit_renamed(self):
154
108
        # Verbose commit of renamed file should say so
156
110
        wt.rename_one('hello.txt', 'gutentag.txt')
157
111
        out, err = self.run_bzr('commit -m renamed')
158
112
        self.assertEqual('', out)
159
 
        self.assertContainsRe(err, '^Committing to: .*\n'
160
 
                              'renamed hello\.txt => gutentag\.txt\n'
161
 
                              'Committed revision 2\.$\n')
 
113
        self.assertEqual('renamed hello.txt => gutentag.txt\n'
 
114
                         'Committed revision 2.\n',
 
115
                         err)
162
116
 
163
117
    def test_verbose_commit_moved(self):
164
118
        # Verbose commit of file moved to new directory should say so
168
122
        wt.rename_one('hello.txt', 'subdir/hello.txt')
169
123
        out, err = self.run_bzr('commit -m renamed')
170
124
        self.assertEqual('', out)
171
 
        self.assertEqual(set([
172
 
            'Committing to: %s/' % osutils.getcwd(),
173
 
            'added subdir',
174
 
            'renamed hello.txt => subdir/hello.txt',
175
 
            'Committed revision 2.',
176
 
            '',
177
 
            ]), set(err.split('\n')))
 
125
        self.assertEqualDiff('added subdir\n'
 
126
                             'renamed hello.txt => subdir/hello.txt\n'
 
127
                             'Committed revision 2.\n',
 
128
                             err)
178
129
 
179
130
    def test_verbose_commit_with_unknown(self):
180
131
        """Unknown files should not be listed by default in verbose output"""
184
135
        wt.add(['hello.txt'])
185
136
        out,err = self.run_bzr('commit -m added')
186
137
        self.assertEqual('', out)
187
 
        self.assertContainsRe(err, '^Committing to: .*\n'
188
 
                              'added hello\.txt\n'
189
 
                              'Committed revision 1\.\n$')
 
138
        self.assertEqual('added hello.txt\n'
 
139
                         'Committed revision 1.\n',
 
140
                         err)
190
141
 
191
142
    def test_verbose_commit_with_unchanged(self):
192
143
        """Unchanged files should not be listed by default in verbose output"""
193
 
        tree = self.make_branch_and_tree('.')
 
144
        self.run_bzr("init")
194
145
        self.build_tree(['hello.txt', 'unchanged.txt'])
195
 
        tree.add('unchanged.txt')
 
146
        self.run_bzr('add unchanged.txt')
196
147
        self.run_bzr('commit -m unchanged unchanged.txt')
197
 
        tree.add("hello.txt")
 
148
        self.run_bzr("add hello.txt")
198
149
        out,err = self.run_bzr('commit -m added')
199
150
        self.assertEqual('', out)
200
 
        self.assertContainsRe(err, '^Committing to: .*\n'
201
 
                              'added hello\.txt\n'
202
 
                              'Committed revision 2\.$\n')
203
 
 
204
 
    def test_verbose_commit_includes_master_location(self):
205
 
        """Location of master is displayed when committing to bound branch"""
206
 
        a_tree = self.make_branch_and_tree('a')
207
 
        self.build_tree(['a/b'])
208
 
        a_tree.add('b')
209
 
        a_tree.commit(message='Initial message')
210
 
 
211
 
        b_tree = a_tree.branch.create_checkout('b')
212
 
        expected = "%s/" % (osutils.abspath('a'), )
213
 
        out, err = self.run_bzr('commit -m blah --unchanged', working_dir='b')
214
 
        self.assertEqual(err, 'Committing to: %s\n'
215
 
                         'Committed revision 2.\n' % expected)
216
 
 
217
 
    def test_commit_sanitizes_CR_in_message(self):
218
 
        # See bug #433779, basically Emacs likes to pass '\r\n' style line
219
 
        # endings to 'bzr commit -m ""' which breaks because we don't allow
220
 
        # '\r' in commit messages. (Mostly because of issues where XML style
221
 
        # formats arbitrarily strip it out of the data while parsing.)
222
 
        # To make life easier for users, we just always translate '\r\n' =>
223
 
        # '\n'. And '\r' => '\n'.
224
 
        a_tree = self.make_branch_and_tree('a')
225
 
        self.build_tree(['a/b'])
226
 
        a_tree.add('b')
227
 
        self.run_bzr(['commit',
228
 
                      '-m', 'a string\r\n\r\nwith mixed\r\rendings\n'],
229
 
                     working_dir='a')
230
 
        rev_id = a_tree.branch.last_revision()
231
 
        rev = a_tree.branch.repository.get_revision(rev_id)
232
 
        self.assertEqualDiff('a string\n\nwith mixed\n\nendings\n',
233
 
                             rev.message)
 
151
        self.assertEqual('added hello.txt\n'
 
152
                         'Committed revision 2.\n',
 
153
                         err)
234
154
 
235
155
    def test_commit_merge_reports_all_modified_files(self):
236
156
        # the commit command should show all the files that are shown by
269
189
            other_tree.rename_one('dirtorename', 'renameddir')
270
190
            other_tree.rename_one('dirtoreparent', 'renameddir/reparenteddir')
271
191
            other_tree.rename_one('filetorename', 'renamedfile')
272
 
            other_tree.rename_one('filetoreparent',
273
 
                                  'renameddir/reparentedfile')
 
192
            other_tree.rename_one('filetoreparent', 'renameddir/reparentedfile')
274
193
            other_tree.remove(['dirtoremove', 'filetoremove'])
275
194
            self.build_tree_contents([
276
 
                ('other/newdir/',),
 
195
                ('other/newdir/', ),
277
196
                ('other/filetomodify', 'new content'),
278
197
                ('other/newfile', 'new file content')])
279
198
            other_tree.add('newfile')
284
203
        this_tree.merge_from_branch(other_tree.branch)
285
204
        os.chdir('this')
286
205
        out,err = self.run_bzr('commit -m added')
 
206
        os.chdir('..')
287
207
        self.assertEqual('', out)
288
 
        self.assertEqual(set([
289
 
            'Committing to: %s/' % osutils.getcwd(),
290
 
            'modified filetomodify',
291
 
            'added newdir',
292
 
            'added newfile',
293
 
            'renamed dirtorename => renameddir',
294
 
            'renamed filetorename => renamedfile',
295
 
            'renamed dirtoreparent => renameddir/reparenteddir',
296
 
            'renamed filetoreparent => renameddir/reparentedfile',
297
 
            'deleted dirtoremove',
298
 
            'deleted filetoremove',
299
 
            'Committed revision 2.',
300
 
            ''
301
 
            ]), set(err.split('\n')))
 
208
        self.assertEqualDiff(
 
209
            'modified filetomodify\n'
 
210
            'added newdir\n'
 
211
            'added newfile\n'
 
212
            'renamed dirtorename => renameddir\n'
 
213
            'renamed dirtoreparent => renameddir/reparenteddir\n'
 
214
            'renamed filetoreparent => renameddir/reparentedfile\n'
 
215
            'renamed filetorename => renamedfile\n'
 
216
            'deleted dirtoremove\n'
 
217
            'deleted filetoremove\n'
 
218
            'Committed revision 2.\n',
 
219
            err)
302
220
 
303
221
    def test_empty_commit_message(self):
304
 
        tree = self.make_branch_and_tree('.')
305
 
        self.build_tree_contents([('foo.c', 'int main() {}')])
306
 
        tree.add('foo.c')
 
222
        self.run_bzr("init")
 
223
        file('foo.c', 'wt').write('int main() {}')
 
224
        self.run_bzr('add foo.c')
307
225
        self.run_bzr('commit -m ""', retcode=3)
308
226
 
309
 
    def test_unsupported_encoding_commit_message(self):
310
 
        if sys.platform == 'win32':
311
 
            raise tests.TestNotApplicable('Win32 parses arguments directly'
312
 
                ' as Unicode, so we can\'t pass invalid non-ascii')
313
 
        tree = self.make_branch_and_tree('.')
314
 
        self.build_tree_contents([('foo.c', 'int main() {}')])
315
 
        tree.add('foo.c')
316
 
        # LANG env variable has no effect on Windows
317
 
        # but some characters anyway cannot be represented
318
 
        # in default user encoding
319
 
        char = probe_bad_non_ascii(osutils.get_user_encoding())
320
 
        if char is None:
321
 
            raise TestSkipped('Cannot find suitable non-ascii character'
322
 
                'for user_encoding (%s)' % osutils.get_user_encoding())
323
 
        out,err = self.run_bzr_subprocess('commit -m "%s"' % char,
324
 
                                          retcode=1,
325
 
                                          env_changes={'LANG': 'C'})
326
 
        self.assertContainsRe(err, r'bzrlib.errors.BzrError: Parameter.*is '
327
 
                                    'unsupported by the current encoding.')
328
 
 
329
227
    def test_other_branch_commit(self):
330
228
        # this branch is to ensure consistent behaviour, whether we're run
331
229
        # inside a branch, or not.
332
 
        outer_tree = self.make_branch_and_tree('.')
333
 
        inner_tree = self.make_branch_and_tree('branch')
334
 
        self.build_tree_contents([
335
 
            ('branch/foo.c', 'int main() {}'),
336
 
            ('branch/bar.c', 'int main() {}')])
337
 
        inner_tree.add(['foo.c', 'bar.c'])
 
230
        os.mkdir('empty_branch')
 
231
        os.chdir('empty_branch')
 
232
        self.run_bzr('init')
 
233
        os.mkdir('branch')
 
234
        os.chdir('branch')
 
235
        self.run_bzr('init')
 
236
        file('foo.c', 'wt').write('int main() {}')
 
237
        file('bar.c', 'wt').write('int main() {}')
 
238
        os.chdir('..')
 
239
        self.run_bzr('add branch/foo.c')
 
240
        self.run_bzr('add branch')
338
241
        # can't commit files in different trees; sane error
339
242
        self.run_bzr('commit -m newstuff branch/foo.c .', retcode=3)
340
 
        # can commit to branch - records foo.c only
341
243
        self.run_bzr('commit -m newstuff branch/foo.c')
342
 
        # can commit to branch - records bar.c
343
244
        self.run_bzr('commit -m newstuff branch')
344
 
        # No changes left
345
 
        self.run_bzr_error(["No changes to commit"], 'commit -m newstuff branch')
 
245
        self.run_bzr('commit -m newstuff branch', retcode=3)
346
246
 
347
247
    def test_out_of_date_tree_commit(self):
348
248
        # check we get an error code and a clear message committing with an out
349
249
        # of date checkout
350
 
        tree = self.make_branch_and_tree('branch')
 
250
        self.make_branch_and_tree('branch')
351
251
        # make a checkout
352
 
        checkout = tree.branch.create_checkout('checkout', lightweight=True)
 
252
        self.run_bzr('checkout --lightweight branch checkout')
353
253
        # commit to the original branch to make the checkout out of date
354
 
        tree.commit('message branch', allow_pointless=True)
 
254
        self.run_bzr('commit --unchanged -m message branch')
355
255
        # now commit to the checkout should emit
356
256
        # ERROR: Out of date with the branch, 'bzr update' is suggested
357
257
        output = self.run_bzr('commit --unchanged -m checkout_message '
358
258
                             'checkout', retcode=3)
359
259
        self.assertEqual(output,
360
260
                         ('',
361
 
                          "bzr: ERROR: Working tree is out of date, please "
362
 
                          "run 'bzr update'.\n"))
 
261
                          "bzr: ERROR: Working tree is out of date, please run "
 
262
                          "'bzr update'.\n"))
363
263
 
364
264
    def test_local_commit_unbound(self):
365
265
        # a --local commit on an unbound branch is an error
372
272
    def test_commit_a_text_merge_in_a_checkout(self):
373
273
        # checkouts perform multiple actions in a transaction across bond
374
274
        # branches and their master, and have been observed to fail in the
375
 
        # past. This is a user story reported to fail in bug #43959 where
 
275
        # past. This is a user story reported to fail in bug #43959 where 
376
276
        # a merge done in a checkout (using the update command) failed to
377
277
        # commit correctly.
378
 
        trunk = self.make_branch_and_tree('trunk')
 
278
        self.run_bzr('init trunk')
379
279
 
380
 
        u1 = trunk.branch.create_checkout('u1')
381
 
        self.build_tree_contents([('u1/hosts', 'initial contents\n')])
382
 
        u1.add('hosts')
 
280
        self.run_bzr('checkout trunk u1')
 
281
        self.build_tree_contents([('u1/hosts', 'initial contents')])
 
282
        self.run_bzr('add u1/hosts')
383
283
        self.run_bzr('commit -m add-hosts u1')
384
284
 
385
 
        u2 = trunk.branch.create_checkout('u2')
386
 
        self.build_tree_contents([('u2/hosts', 'altered in u2\n')])
 
285
        self.run_bzr('checkout trunk u2')
 
286
        self.build_tree_contents([('u2/hosts', 'altered in u2')])
387
287
        self.run_bzr('commit -m checkin-from-u2 u2')
388
288
 
389
289
        # make an offline commits
390
 
        self.build_tree_contents([('u1/hosts', 'first offline change in u1\n')])
 
290
        self.build_tree_contents([('u1/hosts', 'first offline change in u1')])
391
291
        self.run_bzr('commit -m checkin-offline --local u1')
392
292
 
393
293
        # now try to pull in online work from u2, and then commit our offline
394
294
        # work as a merge
395
295
        # retcode 1 as we expect a text conflict
396
296
        self.run_bzr('update u1', retcode=1)
397
 
        self.assertFileEqual('''\
398
 
<<<<<<< TREE
399
 
first offline change in u1
400
 
=======
401
 
altered in u2
402
 
>>>>>>> MERGE-SOURCE
403
 
''',
404
 
                             'u1/hosts')
405
 
 
406
297
        self.run_bzr('resolved u1/hosts')
407
298
        # add a text change here to represent resolving the merge conflicts in
408
299
        # favour of a new version of the file not identical to either the u1
410
301
        self.build_tree_contents([('u1/hosts', 'merge resolution\n')])
411
302
        self.run_bzr('commit -m checkin-merge-of-the-offline-work-from-u1 u1')
412
303
 
413
 
    def test_commit_exclude_excludes_modified_files(self):
414
 
        """Commit -x foo should ignore changes to foo."""
415
 
        tree = self.make_branch_and_tree('.')
416
 
        self.build_tree(['a', 'b', 'c'])
417
 
        tree.smart_add(['.'])
418
 
        out, err = self.run_bzr(['commit', '-m', 'test', '-x', 'b'])
419
 
        self.assertFalse('added b' in out)
420
 
        self.assertFalse('added b' in err)
421
 
        # If b was excluded it will still be 'added' in status.
422
 
        out, err = self.run_bzr(['added'])
423
 
        self.assertEqual('b\n', out)
424
 
        self.assertEqual('', err)
425
 
 
426
 
    def test_commit_exclude_twice_uses_both_rules(self):
427
 
        """Commit -x foo -x bar should ignore changes to foo and bar."""
428
 
        tree = self.make_branch_and_tree('.')
429
 
        self.build_tree(['a', 'b', 'c'])
430
 
        tree.smart_add(['.'])
431
 
        out, err = self.run_bzr(['commit', '-m', 'test', '-x', 'b', '-x', 'c'])
432
 
        self.assertFalse('added b' in out)
433
 
        self.assertFalse('added c' in out)
434
 
        self.assertFalse('added b' in err)
435
 
        self.assertFalse('added c' in err)
436
 
        # If b was excluded it will still be 'added' in status.
437
 
        out, err = self.run_bzr(['added'])
438
 
        self.assertTrue('b\n' in out)
439
 
        self.assertTrue('c\n' in out)
440
 
        self.assertEqual('', err)
441
 
 
442
304
    def test_commit_respects_spec_for_removals(self):
443
305
        """Commit with a file spec should only commit removals that match"""
444
306
        t = self.make_branch_and_tree('.')
471
333
 
472
334
        # With no changes, it should just be 'no changes'
473
335
        # Make sure that commit is failing because there is nothing to do
474
 
        self.run_bzr_error(['No changes to commit'],
 
336
        self.run_bzr_error(['no changes to commit'],
475
337
                           'commit --strict -m no-changes',
476
338
                           working_dir='tree')
477
339
 
505
367
        output, err = self.run_bzr(
506
368
            'commit -m hello --fixes=lp:23452 tree/hello.txt')
507
369
        self.assertEqual('', output)
508
 
        self.assertContainsRe(err, 'Committing to: .*\n'
509
 
                              'added hello\.txt\n'
510
 
                              'Committed revision 1\.\n')
 
370
        self.assertEqual('added hello.txt\nCommitted revision 1.\n', err)
511
371
 
512
372
    def test_no_bugs_no_properties(self):
513
373
        """If no bugs are fixed, the bugs property is not set.
596
456
        self.build_tree(['tree/hello.txt'])
597
457
        tree.add('hello.txt')
598
458
        self.run_bzr_error(
599
 
            ["Did not understand bug identifier orange: Must be an integer. "
600
 
             "See \"bzr help bugs\" for more information on this feature.\n"
601
 
             "Commit refused."],
 
459
            ["Invalid bug identifier for %s. Commit refused." % 'lp:orange'],
602
460
            'commit -m add-b --fixes=lp:orange',
603
461
            working_dir='tree')
604
462
 
608
466
        self.build_tree(['tree/hello.txt'])
609
467
        tree.add('hello.txt')
610
468
        self.run_bzr_error(
611
 
            [r"Invalid bug orange. Must be in the form of 'tracker:id'\. "
612
 
             r"See \"bzr help bugs\" for more information on this feature.\n"
 
469
            [r"Invalid bug orange. Must be in the form of 'tag:id'\. "
613
470
             r"Commit refused\."],
614
471
            'commit -m add-b --fixes=orange',
615
472
            working_dir='tree')
616
 
 
617
 
    def test_no_author(self):
618
 
        """If the author is not specified, the author property is not set."""
619
 
        tree = self.make_branch_and_tree('tree')
620
 
        self.build_tree(['tree/hello.txt'])
621
 
        tree.add('hello.txt')
622
 
        self.run_bzr( 'commit -m hello tree/hello.txt')
623
 
        last_rev = tree.branch.repository.get_revision(tree.last_revision())
624
 
        properties = last_rev.properties
625
 
        self.assertFalse('author' in properties)
626
 
 
627
 
    def test_author_sets_property(self):
628
 
        """commit --author='John Doe <jdoe@example.com>' sets the author
629
 
           revprop.
630
 
        """
631
 
        tree = self.make_branch_and_tree('tree')
632
 
        self.build_tree(['tree/hello.txt'])
633
 
        tree.add('hello.txt')
634
 
        self.run_bzr(["commit", '-m', 'hello',
635
 
                      '--author', u'John D\xf6 <jdoe@example.com>',
636
 
                     "tree/hello.txt"])
637
 
        last_rev = tree.branch.repository.get_revision(tree.last_revision())
638
 
        properties = last_rev.properties
639
 
        self.assertEqual(u'John D\xf6 <jdoe@example.com>', properties['authors'])
640
 
 
641
 
    def test_author_no_email(self):
642
 
        """Author's name without an email address is allowed, too."""
643
 
        tree = self.make_branch_and_tree('tree')
644
 
        self.build_tree(['tree/hello.txt'])
645
 
        tree.add('hello.txt')
646
 
        out, err = self.run_bzr("commit -m hello --author='John Doe' "
647
 
                                "tree/hello.txt")
648
 
        last_rev = tree.branch.repository.get_revision(tree.last_revision())
649
 
        properties = last_rev.properties
650
 
        self.assertEqual('John Doe', properties['authors'])
651
 
 
652
 
    def test_multiple_authors(self):
653
 
        """Multiple authors can be specyfied, and all are stored."""
654
 
        tree = self.make_branch_and_tree('tree')
655
 
        self.build_tree(['tree/hello.txt'])
656
 
        tree.add('hello.txt')
657
 
        out, err = self.run_bzr("commit -m hello --author='John Doe' "
658
 
                                "--author='Jane Rey' tree/hello.txt")
659
 
        last_rev = tree.branch.repository.get_revision(tree.last_revision())
660
 
        properties = last_rev.properties
661
 
        self.assertEqual('John Doe\nJane Rey', properties['authors'])
662
 
 
663
 
    def test_commit_time(self):
664
 
        tree = self.make_branch_and_tree('tree')
665
 
        self.build_tree(['tree/hello.txt'])
666
 
        tree.add('hello.txt')
667
 
        out, err = self.run_bzr("commit -m hello "
668
 
            "--commit-time='2009-10-10 08:00:00 +0100' tree/hello.txt")
669
 
        last_rev = tree.branch.repository.get_revision(tree.last_revision())
670
 
        self.assertEqual(
671
 
            'Sat 2009-10-10 08:00:00 +0100',
672
 
            osutils.format_date(last_rev.timestamp, last_rev.timezone))
673
 
        
674
 
    def test_commit_time_bad_time(self):
675
 
        tree = self.make_branch_and_tree('tree')
676
 
        self.build_tree(['tree/hello.txt'])
677
 
        tree.add('hello.txt')
678
 
        out, err = self.run_bzr("commit -m hello "
679
 
            "--commit-time='NOT A TIME' tree/hello.txt", retcode=3)
680
 
        self.assertStartsWith(
681
 
            err, "bzr: ERROR: Could not parse --commit-time:")
682
 
 
683
 
    def test_partial_commit_with_renames_in_tree(self):
684
 
        # this test illustrates bug #140419
685
 
        t = self.make_branch_and_tree('.')
686
 
        self.build_tree(['dir/', 'dir/a', 'test'])
687
 
        t.add(['dir/', 'dir/a', 'test'])
688
 
        t.commit('initial commit')
689
 
        # important part: file dir/a should change parent
690
 
        # and should appear before old parent
691
 
        # then during partial commit we have error
692
 
        # parent_id {dir-XXX} not in inventory
693
 
        t.rename_one('dir/a', 'a')
694
 
        self.build_tree_contents([('test', 'changes in test')])
695
 
        # partial commit
696
 
        out, err = self.run_bzr('commit test -m "partial commit"')
697
 
        self.assertEquals('', out)
698
 
        self.assertContainsRe(err, r'modified test\nCommitted revision 2.')
699
 
 
700
 
    def test_commit_readonly_checkout(self):
701
 
        # https://bugs.edge.launchpad.net/bzr/+bug/129701
702
 
        # "UnlockableTransport error trying to commit in checkout of readonly
703
 
        # branch"
704
 
        self.make_branch('master')
705
 
        master = BzrDir.open_from_transport(
706
 
            self.get_readonly_transport('master')).open_branch()
707
 
        master.create_checkout('checkout')
708
 
        out, err = self.run_bzr(['commit', '--unchanged', '-mfoo', 'checkout'],
709
 
            retcode=3)
710
 
        self.assertContainsRe(err,
711
 
            r'^bzr: ERROR: Cannot lock.*readonly transport')
712
 
 
713
 
    def setup_editor(self):
714
 
        # Test that commit template hooks work
715
 
        if sys.platform == "win32":
716
 
            f = file('fed.bat', 'w')
717
 
            f.write('@rem dummy fed')
718
 
            f.close()
719
 
            osutils.set_or_unset_env('BZR_EDITOR', "fed.bat")
720
 
        else:
721
 
            f = file('fed.sh', 'wb')
722
 
            f.write('#!/bin/sh\n')
723
 
            f.close()
724
 
            os.chmod('fed.sh', 0755)
725
 
            osutils.set_or_unset_env('BZR_EDITOR', "./fed.sh")
726
 
 
727
 
    def setup_commit_with_template(self):
728
 
        self.setup_editor()
729
 
        msgeditor.hooks.install_named_hook("commit_message_template",
730
 
                lambda commit_obj, msg: "save me some typing\n", None)
731
 
        tree = self.make_branch_and_tree('tree')
732
 
        self.build_tree(['tree/hello.txt'])
733
 
        tree.add('hello.txt')
734
 
        return tree
735
 
 
736
 
    def test_commit_hook_template_accepted(self):
737
 
        tree = self.setup_commit_with_template()
738
 
        out, err = self.run_bzr("commit tree/hello.txt", stdin="y\n")
739
 
        last_rev = tree.branch.repository.get_revision(tree.last_revision())
740
 
        self.assertEqual('save me some typing\n', last_rev.message)
741
 
 
742
 
    def test_commit_hook_template_rejected(self):
743
 
        tree = self.setup_commit_with_template()
744
 
        expected = tree.last_revision()
745
 
        out, err = self.run_bzr_error(["empty commit message"],
746
 
            "commit tree/hello.txt", stdin="n\n")
747
 
        self.assertEqual(expected, tree.last_revision())
748
 
 
749
 
    def test_commit_without_username(self):
750
 
        """Ensure commit error if username is not set.
751
 
        """
752
 
        self.run_bzr(['init', 'foo'])
753
 
        os.chdir('foo')
754
 
        open('foo.txt', 'w').write('hello')
755
 
        self.run_bzr(['add'])
756
 
        osutils.set_or_unset_env('EMAIL', None)
757
 
        osutils.set_or_unset_env('BZR_EMAIL', None)
758
 
        out, err = self.run_bzr(['commit', '-m', 'initial'], 3)
759
 
        self.assertContainsRe(err, 'Unable to determine your name')