← 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: Aaron Bentley
  • Date: 2006-06-21 14:30:57 UTC
  • mfrom: (1801.1.1 bzr.dev)
  • mto: This revision was merged to the branch mainline in revision 1803.
  • Revision ID: abentley@panoramicfeedback.com-20060621143057-776e4b8d707e430e
Install benchmarks. (Jelmer Vernooij)

Show diffs side-by-side

added added

removed removed

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