~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

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

  • Committer: Robert Collins
  • Date: 2007-07-05 01:09:11 UTC
  • mto: This revision was merged to the branch mainline in revision 2592.
  • Revision ID: robertc@robertcollins.net-20070705010911-q93psu3cxxhn8isr
* New method ``external_url`` on Transport for obtaining the url to
  hand to external processes. (Robert Collins)

Show diffs side-by-side

added added

removed removed

Lines of Context:
 
1
# Copyright (C) 2005, 2006 Canonical Ltd
 
2
#
 
3
# This program is free software; you can redistribute it and/or modify
 
4
# it under the terms of the GNU General Public License as published by
 
5
# the Free Software Foundation; either version 2 of the License, or
 
6
# (at your option) any later version.
 
7
#
 
8
# This program is distributed in the hope that it will be useful,
 
9
# but WITHOUT ANY WARRANTY; without even the implied warranty of
 
10
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 
11
# GNU General Public License for more details.
 
12
#
 
13
# You should have received a copy of the GNU General Public License
 
14
# along with this program; if not, write to the Free Software
 
15
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
 
16
 
 
17
 
 
18
"""Tests for the commit CLI of bzr."""
 
19
 
 
20
import os
 
21
import re
 
22
import sys
 
23
 
 
24
from bzrlib import (
 
25
    ignores,
 
26
    )
 
27
from bzrlib.branch import Branch
 
28
from bzrlib.bzrdir import BzrDir
 
29
from bzrlib.errors import BzrCommandError
 
30
from bzrlib.tests.blackbox import ExternalBase
 
31
from bzrlib.workingtree import WorkingTree
 
32
 
 
33
 
 
34
class TestCommit(ExternalBase):
 
35
 
 
36
    def test_05_empty_commit(self):
 
37
        """Commit of tree with no versioned files should fail"""
 
38
        # If forced, it should succeed, but this is not tested here.
 
39
        self.run_bzr("init")
 
40
        self.build_tree(['hello.txt'])
 
41
        out,err = self.run_bzr("commit", "-m", "empty", retcode=3)
 
42
        self.assertEqual('', out)
 
43
        self.assertStartsWith(err, 'bzr: ERROR: no changes to commit.'
 
44
                                  ' use --unchanged to commit anyhow\n')
 
45
 
 
46
    def test_commit_success(self):
 
47
        """Successful commit should not leave behind a bzr-commit-* file"""
 
48
        self.run_bzr("init")
 
49
        self.run_bzr("commit", "--unchanged", "-m", "message")
 
50
        self.assertEqual('', self.run_bzr(['unknowns'])[0])
 
51
 
 
52
        # same for unicode messages
 
53
        self.run_bzr("commit", "--unchanged", "-m", u'foo\xb5')
 
54
        self.assertEqual('', self.run_bzr(['unknowns'])[0])
 
55
 
 
56
    def test_commit_with_path(self):
 
57
        """Commit tree with path of root specified"""
 
58
        self.run_bzr('init', 'a')
 
59
        self.build_tree(['a/a_file'])
 
60
        self.run_bzr('add', 'a/a_file')
 
61
        self.run_bzr('commit', '-m', 'first commit', 'a')
 
62
 
 
63
        self.run_bzr('branch', 'a', 'b')
 
64
        self.build_tree_contents([('b/a_file', 'changes in b')])
 
65
        self.run_bzr('commit', '-m', 'first commit in b', 'b')
 
66
 
 
67
        self.build_tree_contents([('a/a_file', 'new contents')])
 
68
        self.run_bzr('commit', '-m', 'change in a', 'a')
 
69
 
 
70
        os.chdir('b')
 
71
        self.run_bzr('merge', '../a', retcode=1) # will conflict
 
72
        os.chdir('..')
 
73
        self.run_bzr('resolved', 'b/a_file')
 
74
        self.run_bzr('commit', '-m', 'merge into b', 'b')
 
75
 
 
76
 
 
77
    def test_10_verbose_commit(self):
 
78
        """Add one file and examine verbose commit output"""
 
79
        self.run_bzr("init")
 
80
        self.build_tree(['hello.txt'])
 
81
        self.run_bzr("add hello.txt")
 
82
        out,err = self.run_bzr("commit", "-m", "added")
 
83
        self.assertEqual('', out)
 
84
        self.assertEqual('added hello.txt\n'
 
85
                         'Committed revision 1.\n',
 
86
                         err)
 
87
 
 
88
    def prepare_simple_history(self):
 
89
        """Prepare and return a working tree with one commit of one file"""
 
90
        # Commit with modified file should say so
 
91
        wt = BzrDir.create_standalone_workingtree('.')
 
92
        self.build_tree(['hello.txt', 'extra.txt'])
 
93
        wt.add(['hello.txt'])
 
94
        wt.commit(message='added')
 
95
        return wt
 
96
 
 
97
    def test_verbose_commit_modified(self):
 
98
        # Verbose commit of modified file should say so
 
99
        wt = self.prepare_simple_history()
 
100
        self.build_tree_contents([('hello.txt', 'new contents')])
 
101
        out, err = self.run_bzr("commit", "-m", "modified")
 
102
        self.assertEqual('', out)
 
103
        self.assertEqual('modified hello.txt\n'
 
104
                         'Committed revision 2.\n',
 
105
                         err)
 
106
 
 
107
    def test_verbose_commit_renamed(self):
 
108
        # Verbose commit of renamed file should say so
 
109
        wt = self.prepare_simple_history()
 
110
        wt.rename_one('hello.txt', 'gutentag.txt')
 
111
        out, err = self.run_bzr("commit", "-m", "renamed")
 
112
        self.assertEqual('', out)
 
113
        self.assertEqual('renamed hello.txt => gutentag.txt\n'
 
114
                         'Committed revision 2.\n',
 
115
                         err)
 
116
 
 
117
    def test_verbose_commit_moved(self):
 
118
        # Verbose commit of file moved to new directory should say so
 
119
        wt = self.prepare_simple_history()
 
120
        os.mkdir('subdir')
 
121
        wt.add(['subdir'])
 
122
        wt.rename_one('hello.txt', 'subdir/hello.txt')
 
123
        out, err = self.run_bzr("commit", "-m", "renamed")
 
124
        self.assertEqual('', out)
 
125
        self.assertEqualDiff('added subdir\n'
 
126
                             'renamed hello.txt => subdir/hello.txt\n'
 
127
                             'Committed revision 2.\n',
 
128
                             err)
 
129
 
 
130
    def test_verbose_commit_with_unknown(self):
 
131
        """Unknown files should not be listed by default in verbose output"""
 
132
        # Is that really the best policy?
 
133
        wt = BzrDir.create_standalone_workingtree('.')
 
134
        self.build_tree(['hello.txt', 'extra.txt'])
 
135
        wt.add(['hello.txt'])
 
136
        out,err = self.run_bzr("commit", "-m", "added")
 
137
        self.assertEqual('', out)
 
138
        self.assertEqual('added hello.txt\n'
 
139
                         'Committed revision 1.\n',
 
140
                         err)
 
141
 
 
142
    def test_verbose_commit_with_unchanged(self):
 
143
        """Unchanged files should not be listed by default in verbose output"""
 
144
        self.run_bzr("init")
 
145
        self.build_tree(['hello.txt', 'unchanged.txt'])
 
146
        self.run_bzr('add unchanged.txt')
 
147
        self.run_bzr('commit -m unchanged unchanged.txt')
 
148
        self.run_bzr("add hello.txt")
 
149
        out,err = self.run_bzr("commit", "-m", "added")
 
150
        self.assertEqual('', out)
 
151
        self.assertEqual('added hello.txt\n'
 
152
                         'Committed revision 2.\n',
 
153
                         err)
 
154
 
 
155
    def test_commit_merge_reports_all_modified_files(self):
 
156
        # the commit command should show all the files that are shown by
 
157
        # bzr diff or bzr status when committing, even when they were not
 
158
        # changed by the user but rather through doing a merge.
 
159
        this_tree = self.make_branch_and_tree('this')
 
160
        # we need a bunch of files and dirs, to perform one action on each.
 
161
        self.build_tree([
 
162
            'this/dirtorename/',
 
163
            'this/dirtoreparent/',
 
164
            'this/dirtoleave/',
 
165
            'this/dirtoremove/',
 
166
            'this/filetoreparent',
 
167
            'this/filetorename',
 
168
            'this/filetomodify',
 
169
            'this/filetoremove',
 
170
            'this/filetoleave']
 
171
            )
 
172
        this_tree.add([
 
173
            'dirtorename',
 
174
            'dirtoreparent',
 
175
            'dirtoleave',
 
176
            'dirtoremove',
 
177
            'filetoreparent',
 
178
            'filetorename',
 
179
            'filetomodify',
 
180
            'filetoremove',
 
181
            'filetoleave']
 
182
            )
 
183
        this_tree.commit('create_files')
 
184
        other_dir = this_tree.bzrdir.sprout('other')
 
185
        other_tree = other_dir.open_workingtree()
 
186
        other_tree.lock_write()
 
187
        # perform the needed actions on the files and dirs.
 
188
        try:
 
189
            other_tree.rename_one('dirtorename', 'renameddir')
 
190
            other_tree.rename_one('dirtoreparent', 'renameddir/reparenteddir')
 
191
            other_tree.rename_one('filetorename', 'renamedfile')
 
192
            other_tree.rename_one('filetoreparent', 'renameddir/reparentedfile')
 
193
            other_tree.remove(['dirtoremove', 'filetoremove'])
 
194
            self.build_tree_contents([
 
195
                ('other/newdir/', ),
 
196
                ('other/filetomodify', 'new content'),
 
197
                ('other/newfile', 'new file content')])
 
198
            other_tree.add('newfile')
 
199
            other_tree.add('newdir/')
 
200
            other_tree.commit('modify all sample files and dirs.')
 
201
        finally:
 
202
            other_tree.unlock()
 
203
        this_tree.merge_from_branch(other_tree.branch)
 
204
        os.chdir('this')
 
205
        out,err = self.run_bzr("commit", "-m", "added")
 
206
        os.chdir('..')
 
207
        self.assertEqual('', out)
 
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)
 
220
 
 
221
    def test_empty_commit_message(self):
 
222
        self.run_bzr("init")
 
223
        file('foo.c', 'wt').write('int main() {}')
 
224
        self.run_bzr('add foo.c')
 
225
        self.run_bzr('commit -m ""', retcode=3)
 
226
 
 
227
    def test_other_branch_commit(self):
 
228
        # this branch is to ensure consistent behaviour, whether we're run
 
229
        # inside a branch, or not.
 
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'])
 
241
        # can't commit files in different trees; sane error
 
242
        self.run_bzr('commit -m newstuff branch/foo.c .', retcode=3)
 
243
        self.run_bzr('commit -m newstuff branch/foo.c')
 
244
        self.run_bzr('commit -m newstuff branch')
 
245
        self.run_bzr('commit -m newstuff branch', retcode=3)
 
246
 
 
247
    def test_out_of_date_tree_commit(self):
 
248
        # check we get an error code and a clear message committing with an out
 
249
        # of date checkout
 
250
        self.make_branch_and_tree('branch')
 
251
        # make a checkout
 
252
        self.run_bzr('checkout --lightweight branch checkout')
 
253
        # commit to the original branch to make the checkout out of date
 
254
        self.run_bzr('commit --unchanged -m message branch')
 
255
        # now commit to the checkout should emit
 
256
        # ERROR: Out of date with the branch, 'bzr update' is suggested
 
257
        output = self.run_bzr('commit --unchanged -m checkout_message '
 
258
                             'checkout', retcode=3)
 
259
        self.assertEqual(output,
 
260
                         ('',
 
261
                          "bzr: ERROR: Working tree is out of date, please run "
 
262
                          "'bzr update'.\n"))
 
263
 
 
264
    def test_local_commit_unbound(self):
 
265
        # a --local commit on an unbound branch is an error
 
266
        self.make_branch_and_tree('.')
 
267
        out, err = self.run_bzr('commit', '--local', retcode=3)
 
268
        self.assertEqualDiff('', out)
 
269
        self.assertEqualDiff('bzr: ERROR: Cannot perform local-only commits '
 
270
                             'on unbound branches.\n', err)
 
271
 
 
272
    def test_commit_a_text_merge_in_a_checkout(self):
 
273
        # checkouts perform multiple actions in a transaction across bond
 
274
        # branches and their master, and have been observed to fail in the
 
275
        # past. This is a user story reported to fail in bug #43959 where 
 
276
        # a merge done in a checkout (using the update command) failed to
 
277
        # commit correctly.
 
278
        self.run_bzr('init', 'trunk')
 
279
 
 
280
        self.run_bzr('checkout', 'trunk', 'u1')
 
281
        self.build_tree_contents([('u1/hosts', 'initial contents')])
 
282
        self.run_bzr('add', 'u1/hosts')
 
283
        self.run_bzr('commit', '-m', 'add hosts', 'u1')
 
284
 
 
285
        self.run_bzr('checkout', 'trunk', 'u2')
 
286
        self.build_tree_contents([('u2/hosts', 'altered in u2')])
 
287
        self.run_bzr('commit', '-m', 'checkin from u2', 'u2')
 
288
 
 
289
        # make an offline commits
 
290
        self.build_tree_contents([('u1/hosts', 'first offline change in u1')])
 
291
        self.run_bzr('commit', '-m', 'checkin offline', '--local', 'u1')
 
292
 
 
293
        # now try to pull in online work from u2, and then commit our offline
 
294
        # work as a merge
 
295
        # retcode 1 as we expect a text conflict
 
296
        self.run_bzr('update', 'u1', retcode=1)
 
297
        self.run_bzr('resolved', 'u1/hosts')
 
298
        # add a text change here to represent resolving the merge conflicts in
 
299
        # favour of a new version of the file not identical to either the u1
 
300
        # version or the u2 version.
 
301
        self.build_tree_contents([('u1/hosts', 'merge resolution\n')])
 
302
        self.run_bzr('commit', '-m', 'checkin merge of the offline work from u1', 'u1')
 
303
 
 
304
    def test_commit_respects_spec_for_removals(self):
 
305
        """Commit with a file spec should only commit removals that match"""
 
306
        t = self.make_branch_and_tree('.')
 
307
        self.build_tree(['file-a', 'dir-a/', 'dir-a/file-b'])
 
308
        t.add(['file-a', 'dir-a', 'dir-a/file-b'])
 
309
        t.commit('Create')
 
310
        t.remove(['file-a', 'dir-a/file-b'])
 
311
        os.chdir('dir-a')
 
312
        result = self.run_bzr('commit', '.', '-m' 'removed file-b')[1]
 
313
        self.assertNotContainsRe(result, 'file-a')
 
314
        result = self.run_bzr('status')[0]
 
315
        self.assertContainsRe(result, 'removed:\n  file-a')
 
316
 
 
317
    def test_strict_commit(self):
 
318
        """Commit with --strict works if everything is known"""
 
319
        ignores._set_user_ignores([])
 
320
        tree = self.make_branch_and_tree('tree')
 
321
        self.build_tree(['tree/a'])
 
322
        tree.add('a')
 
323
        # A simple change should just work
 
324
        self.run_bzr('commit', '--strict', '-m', 'adding a',
 
325
                     working_dir='tree')
 
326
 
 
327
    def test_strict_commit_no_changes(self):
 
328
        """commit --strict gives "no changes" if there is nothing to commit"""
 
329
        tree = self.make_branch_and_tree('tree')
 
330
        self.build_tree(['tree/a'])
 
331
        tree.add('a')
 
332
        tree.commit('adding a')
 
333
 
 
334
        # With no changes, it should just be 'no changes'
 
335
        # Make sure that commit is failing because there is nothing to do
 
336
        self.run_bzr_error(['no changes to commit'],
 
337
                           'commit', '--strict', '-m', 'no changes',
 
338
                           working_dir='tree')
 
339
 
 
340
        # But --strict doesn't care if you supply --unchanged
 
341
        self.run_bzr('commit', '--strict', '--unchanged', '-m', 'no changes',
 
342
                     working_dir='tree')
 
343
 
 
344
    def test_strict_commit_unknown(self):
 
345
        """commit --strict fails if a file is unknown"""
 
346
        tree = self.make_branch_and_tree('tree')
 
347
        self.build_tree(['tree/a'])
 
348
        tree.add('a')
 
349
        tree.commit('adding a')
 
350
 
 
351
        # Add one file so there is a change, but forget the other
 
352
        self.build_tree(['tree/b', 'tree/c'])
 
353
        tree.add('b')
 
354
        self.run_bzr_error(['Commit refused because there are unknown files'],
 
355
                           'commit', '--strict', '-m', 'add b',
 
356
                           working_dir='tree')
 
357
 
 
358
        # --no-strict overrides --strict
 
359
        self.run_bzr('commit', '--strict', '-m', 'add b', '--no-strict',
 
360
                     working_dir='tree')
 
361
 
 
362
    def test_fixes_bug_output(self):
 
363
        """commit --fixes=lp:23452 succeeds without output."""
 
364
        tree = self.make_branch_and_tree('tree')
 
365
        self.build_tree(['tree/hello.txt'])
 
366
        tree.add('hello.txt')
 
367
        output, err = self.run_bzr(
 
368
            'commit', '-m', 'hello', '--fixes=lp:23452', 'tree/hello.txt')
 
369
        self.assertEqual('', output)
 
370
        self.assertEqual('added hello.txt\nCommitted revision 1.\n', err)
 
371
 
 
372
    def test_no_bugs_no_properties(self):
 
373
        """If no bugs are fixed, the bugs property is not set.
 
374
 
 
375
        see https://beta.launchpad.net/bzr/+bug/109613
 
376
        """
 
377
        tree = self.make_branch_and_tree('tree')
 
378
        self.build_tree(['tree/hello.txt'])
 
379
        tree.add('hello.txt')
 
380
        self.run_bzr( 'commit', '-m', 'hello', 'tree/hello.txt')
 
381
        # Get the revision properties, ignoring the branch-nick property, which
 
382
        # we don't care about for this test.
 
383
        last_rev = tree.branch.repository.get_revision(tree.last_revision())
 
384
        properties = dict(last_rev.properties)
 
385
        del properties['branch-nick']
 
386
        self.assertFalse('bugs' in properties)
 
387
 
 
388
    def test_fixes_bug_sets_property(self):
 
389
        """commit --fixes=lp:234 sets the lp:234 revprop to 'fixed'."""
 
390
        tree = self.make_branch_and_tree('tree')
 
391
        self.build_tree(['tree/hello.txt'])
 
392
        tree.add('hello.txt')
 
393
        self.run_bzr(
 
394
            'commit', '-m', 'hello', '--fixes=lp:234', 'tree/hello.txt')
 
395
 
 
396
        # Get the revision properties, ignoring the branch-nick property, which
 
397
        # we don't care about for this test.
 
398
        last_rev = tree.branch.repository.get_revision(tree.last_revision())
 
399
        properties = dict(last_rev.properties)
 
400
        del properties['branch-nick']
 
401
 
 
402
        self.assertEqual({'bugs': 'https://launchpad.net/bugs/234 fixed'},
 
403
                         properties)
 
404
 
 
405
    def test_fixes_multiple_bugs_sets_properties(self):
 
406
        """--fixes can be used more than once to show that bugs are fixed."""
 
407
        tree = self.make_branch_and_tree('tree')
 
408
        self.build_tree(['tree/hello.txt'])
 
409
        tree.add('hello.txt')
 
410
        self.run_bzr(
 
411
            'commit', '-m', 'hello', '--fixes=lp:123', '--fixes=lp:235',
 
412
            'tree/hello.txt')
 
413
 
 
414
        # Get the revision properties, ignoring the branch-nick property, which
 
415
        # we don't care about for this test.
 
416
        last_rev = tree.branch.repository.get_revision(tree.last_revision())
 
417
        properties = dict(last_rev.properties)
 
418
        del properties['branch-nick']
 
419
 
 
420
        self.assertEqual(
 
421
            {'bugs': 'https://launchpad.net/bugs/123 fixed\n'
 
422
                     'https://launchpad.net/bugs/235 fixed'},
 
423
            properties)
 
424
 
 
425
    def test_fixes_bug_with_alternate_trackers(self):
 
426
        """--fixes can be used on a properly configured branch to mark bug
 
427
        fixes on multiple trackers.
 
428
        """
 
429
        tree = self.make_branch_and_tree('tree')
 
430
        tree.branch.get_config().set_user_option(
 
431
            'trac_twisted_url', 'http://twistedmatrix.com/trac')
 
432
        self.build_tree(['tree/hello.txt'])
 
433
        tree.add('hello.txt')
 
434
        self.run_bzr(
 
435
            'commit', '-m', 'hello', '--fixes=lp:123',
 
436
            '--fixes=twisted:235', 'tree/')
 
437
 
 
438
        # Get the revision properties, ignoring the branch-nick property, which
 
439
        # we don't care about for this test.
 
440
        last_rev = tree.branch.repository.get_revision(tree.last_revision())
 
441
        properties = dict(last_rev.properties)
 
442
        del properties['branch-nick']
 
443
 
 
444
        self.assertEqual(
 
445
            {'bugs': 'https://launchpad.net/bugs/123 fixed\n'
 
446
                     'http://twistedmatrix.com/trac/ticket/235 fixed'},
 
447
            properties)
 
448
 
 
449
    def test_fixes_unknown_bug_prefix(self):
 
450
        tree = self.make_branch_and_tree('tree')
 
451
        self.build_tree(['tree/hello.txt'])
 
452
        tree.add('hello.txt')
 
453
        self.run_bzr_error(
 
454
            ["Unrecognized bug %s. Commit refused." % 'xxx:123'],
 
455
            'commit', '-m', 'add b', '--fixes=xxx:123',
 
456
            working_dir='tree')
 
457
 
 
458
    def test_fixes_invalid_bug_number(self):
 
459
        tree = self.make_branch_and_tree('tree')
 
460
        self.build_tree(['tree/hello.txt'])
 
461
        tree.add('hello.txt')
 
462
        self.run_bzr_error(
 
463
            ["Invalid bug identifier for %s. Commit refused." % 'lp:orange'],
 
464
            'commit', '-m', 'add b', '--fixes=lp:orange',
 
465
            working_dir='tree')
 
466
 
 
467
    def test_fixes_invalid_argument(self):
 
468
        """Raise an appropriate error when the fixes argument isn't tag:id."""
 
469
        tree = self.make_branch_and_tree('tree')
 
470
        self.build_tree(['tree/hello.txt'])
 
471
        tree.add('hello.txt')
 
472
        self.run_bzr_error(
 
473
            [r"Invalid bug orange. Must be in the form of 'tag:id'\. "
 
474
             r"Commit refused\."],
 
475
            'commit', '-m', 'add b', '--fixes=orange',
 
476
            working_dir='tree')