/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-06-26 08:27:24 UTC
  • mto: This revision was merged to the branch mainline in revision 2554.
  • Revision ID: robertc@robertcollins.net-20070626082724-q95taws12ajox2io
Add helpers to get api versions from objects.

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/tests/blackbox/test_commit.py
1
 
# Copyright (C) 2005, 2006 by Canonical Ltd
2
 
 
 
1
# Copyright (C) 2005, 2006 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
15
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
17
17
 
18
18
"""Tests for the commit CLI of bzr."""
19
19
 
20
 
from cStringIO import StringIO
21
20
import os
22
21
import re
23
 
import shutil
24
22
import sys
25
23
 
 
24
from bzrlib import (
 
25
    ignores,
 
26
    )
26
27
from bzrlib.branch import Branch
27
 
import bzrlib.bzrdir as bzrdir
 
28
from bzrlib.bzrdir import BzrDir
28
29
from bzrlib.errors import BzrCommandError
29
30
from bzrlib.tests.blackbox import ExternalBase
30
31
from bzrlib.workingtree import WorkingTree
35
36
    def test_05_empty_commit(self):
36
37
        """Commit of tree with no versioned files should fail"""
37
38
        # If forced, it should succeed, but this is not tested here.
38
 
        self.runbzr("init")
 
39
        self.run_bzr("init")
39
40
        self.build_tree(['hello.txt'])
40
 
        self.runbzr("commit -m empty", retcode=3)
 
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.capture('unknowns'))
 
51
 
 
52
        # same for unicode messages
 
53
        self.run_bzr("commit", "--unchanged", "-m", u'foo\xb5')
 
54
        self.assertEqual('', self.capture('unknowns'))
 
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
 
41
76
 
42
77
    def test_10_verbose_commit(self):
43
78
        """Add one file and examine verbose commit output"""
50
85
                         'Committed revision 1.\n',
51
86
                         err)
52
87
 
53
 
    def test_15_verbose_commit_with_unknown(self):
 
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):
54
131
        """Unknown files should not be listed by default in verbose output"""
55
132
        # Is that really the best policy?
56
 
        self.runbzr("init")
 
133
        wt = BzrDir.create_standalone_workingtree('.')
57
134
        self.build_tree(['hello.txt', 'extra.txt'])
58
 
        self.runbzr("add hello.txt")
 
135
        wt.add(['hello.txt'])
59
136
        out,err = self.run_bzr("commit", "-m", "added")
60
137
        self.assertEqual('', out)
61
138
        self.assertEqual('added hello.txt\n'
62
139
                         'Committed revision 1.\n',
63
140
                         err)
64
141
 
65
 
    def test_16_verbose_commit_with_unchanged(self):
 
142
    def test_verbose_commit_with_unchanged(self):
66
143
        """Unchanged files should not be listed by default in verbose output"""
67
144
        self.runbzr("init")
68
145
        self.build_tree(['hello.txt', 'unchanged.txt'])
75
152
                         'Committed revision 2.\n',
76
153
                         err)
77
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
 
78
221
    def test_empty_commit_message(self):
79
222
        self.runbzr("init")
80
223
        file('foo.c', 'wt').write('int main() {}')
125
268
        self.assertEqualDiff('', out)
126
269
        self.assertEqualDiff('bzr: ERROR: Cannot perform local-only commits '
127
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')