/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: Ian Clatworthy
  • Date: 2007-09-10 08:19:29 UTC
  • mto: (2818.1.1 ianc-integration)
  • mto: This revision was merged to the branch mainline in revision 2819.
  • Revision ID: ian.clatworthy@internode.on.net-20070910081929-9419oqg1zhva4e7d
put back KeyboardInterrupt trapping as needed for Python 2.5

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."""
19
19
 
20
20
import os
21
 
import sys
22
21
 
23
22
from bzrlib import (
24
 
    osutils,
25
23
    ignores,
26
 
    msgeditor,
27
 
    osutils,
28
 
    tests,
29
24
    )
30
25
from bzrlib.bzrdir import BzrDir
31
 
from bzrlib.tests import (
32
 
    probe_bad_non_ascii,
33
 
    TestSkipped,
34
 
    )
35
26
from bzrlib.tests.blackbox import ExternalBase
36
27
 
37
28
 
44
35
        self.build_tree(['hello.txt'])
45
36
        out,err = self.run_bzr('commit -m empty', retcode=3)
46
37
        self.assertEqual('', out)
47
 
        self.assertContainsRe(err, 'bzr: ERROR: No changes to commit\.'
48
 
                                  ' Use --unchanged to commit anyhow.\n')
 
38
        self.assertContainsRe(err, 'bzr: ERROR: no changes to commit\.'
 
39
                                  ' use --unchanged to commit anyhow\n')
49
40
 
50
41
    def test_commit_success(self):
51
42
        """Successful commit should not leave behind a bzr-commit-* file"""
84
75
        tree.add("hello.txt")
85
76
        out,err = self.run_bzr('commit -m added')
86
77
        self.assertEqual('', out)
87
 
        self.assertContainsRe(err, '^Committing to: .*\n'
 
78
        self.assertContainsRe(err, '^Committing revision 1 to ".*"\.\n'
88
79
                              'added hello.txt\n'
89
80
                              'Committed revision 1.\n$',)
90
81
 
103
94
        self.build_tree_contents([('hello.txt', 'new contents')])
104
95
        out, err = self.run_bzr('commit -m modified')
105
96
        self.assertEqual('', out)
106
 
        self.assertContainsRe(err, '^Committing to: .*\n'
 
97
        self.assertContainsRe(err, '^Committing revision 2 to ".*"\.\n'
107
98
                              'modified hello\.txt\n'
108
99
                              'Committed revision 2\.\n$')
109
100
 
110
 
    def test_warn_about_forgotten_commit_message(self):
111
 
        """Test that the lack of -m parameter is caught"""
112
 
        wt = self.make_branch_and_tree('.')
113
 
        self.build_tree(['one', 'two'])
114
 
        wt.add(['two'])
115
 
        out, err = self.run_bzr('commit -m one two')
116
 
        self.assertContainsRe(err, "The commit message is a file name")
117
 
 
118
101
    def test_verbose_commit_renamed(self):
119
102
        # Verbose commit of renamed file should say so
120
103
        wt = self.prepare_simple_history()
121
104
        wt.rename_one('hello.txt', 'gutentag.txt')
122
105
        out, err = self.run_bzr('commit -m renamed')
123
106
        self.assertEqual('', out)
124
 
        self.assertContainsRe(err, '^Committing to: .*\n'
 
107
        self.assertContainsRe(err, '^Committing revision 2 to ".*"\.\n'
125
108
                              'renamed hello\.txt => gutentag\.txt\n'
126
109
                              'Committed revision 2\.$\n')
127
110
 
133
116
        wt.rename_one('hello.txt', 'subdir/hello.txt')
134
117
        out, err = self.run_bzr('commit -m renamed')
135
118
        self.assertEqual('', out)
136
 
        self.assertEqual(set([
137
 
            'Committing to: %s/' % osutils.getcwd(),
138
 
            'added subdir',
139
 
            'renamed hello.txt => subdir/hello.txt',
140
 
            'Committed revision 2.',
141
 
            '',
142
 
            ]), set(err.split('\n')))
 
119
        self.assertContainsRe(err, '^Committing revision 2 to ".*"\.\n'
 
120
                              'added subdir\n'
 
121
                              'renamed hello\.txt => subdir/hello\.txt\n'
 
122
                              'Committed revision 2\.\n$')
143
123
 
144
124
    def test_verbose_commit_with_unknown(self):
145
125
        """Unknown files should not be listed by default in verbose output"""
149
129
        wt.add(['hello.txt'])
150
130
        out,err = self.run_bzr('commit -m added')
151
131
        self.assertEqual('', out)
152
 
        self.assertContainsRe(err, '^Committing to: .*\n'
 
132
        self.assertContainsRe(err, '^Committing revision 1 to ".*"\.\n'
153
133
                              'added hello\.txt\n'
154
134
                              'Committed revision 1\.\n$')
155
135
 
162
142
        tree.add("hello.txt")
163
143
        out,err = self.run_bzr('commit -m added')
164
144
        self.assertEqual('', out)
165
 
        self.assertContainsRe(err, '^Committing to: .*\n'
 
145
        self.assertContainsRe(err, '^Committing revision 2 to ".*"\.\n'
166
146
                              'added hello\.txt\n'
167
147
                              'Committed revision 2\.$\n')
168
148
 
174
154
        a_tree.commit(message='Initial message')
175
155
 
176
156
        b_tree = a_tree.branch.create_checkout('b')
177
 
        expected = "%s/" % (osutils.abspath('a'), )
 
157
        expected = "%s/" % (os.path.abspath('a'), )
178
158
        out, err = self.run_bzr('commit -m blah --unchanged', working_dir='b')
179
 
        self.assertEqual(err, 'Committing to: %s\n'
 
159
        self.assertEqual(err, 'Committing revision 2 to "%s".\n'
180
160
                         'Committed revision 2.\n' % expected)
181
161
 
182
 
    def test_commit_sanitizes_CR_in_message(self):
183
 
        # See bug #433779, basically Emacs likes to pass '\r\n' style line
184
 
        # endings to 'bzr commit -m ""' which breaks because we don't allow
185
 
        # '\r' in commit messages. (Mostly because of issues where XML style
186
 
        # formats arbitrarily strip it out of the data while parsing.)
187
 
        # To make life easier for users, we just always translate '\r\n' =>
188
 
        # '\n'. And '\r' => '\n'.
189
 
        a_tree = self.make_branch_and_tree('a')
190
 
        self.build_tree(['a/b'])
191
 
        a_tree.add('b')
192
 
        self.run_bzr(['commit',
193
 
                      '-m', 'a string\r\n\r\nwith mixed\r\rendings\n'],
194
 
                     working_dir='a')
195
 
        rev_id = a_tree.branch.last_revision()
196
 
        rev = a_tree.branch.repository.get_revision(rev_id)
197
 
        self.assertEqualDiff('a string\n\nwith mixed\n\nendings\n',
198
 
                             rev.message)
199
 
 
200
162
    def test_commit_merge_reports_all_modified_files(self):
201
163
        # the commit command should show all the files that are shown by
202
164
        # bzr diff or bzr status when committing, even when they were not
250
212
        os.chdir('this')
251
213
        out,err = self.run_bzr('commit -m added')
252
214
        self.assertEqual('', out)
253
 
        self.assertEqual(set([
254
 
            'Committing to: %s/' % osutils.getcwd(),
255
 
            'modified filetomodify',
256
 
            'added newdir',
257
 
            'added newfile',
258
 
            'renamed dirtorename => renameddir',
259
 
            'renamed filetorename => renamedfile',
260
 
            'renamed dirtoreparent => renameddir/reparenteddir',
261
 
            'renamed filetoreparent => renameddir/reparentedfile',
262
 
            'deleted dirtoremove',
263
 
            'deleted filetoremove',
264
 
            'Committed revision 2.',
265
 
            ''
266
 
            ]), set(err.split('\n')))
 
215
        expected = '%s/' % (os.getcwd(), )
 
216
        self.assertEqualDiff(
 
217
            'Committing revision 2 to "%s".\n'
 
218
            'modified filetomodify\n'
 
219
            'added newdir\n'
 
220
            'added newfile\n'
 
221
            'renamed dirtorename => renameddir\n'
 
222
            'renamed dirtoreparent => renameddir/reparenteddir\n'
 
223
            'renamed filetoreparent => renameddir/reparentedfile\n'
 
224
            'renamed filetorename => renamedfile\n'
 
225
            'deleted dirtoremove\n'
 
226
            'deleted filetoremove\n'
 
227
            'Committed revision 2.\n' % (expected, ),
 
228
            err)
267
229
 
268
230
    def test_empty_commit_message(self):
269
231
        tree = self.make_branch_and_tree('.')
272
234
        self.run_bzr('commit -m ""', retcode=3)
273
235
 
274
236
    def test_unsupported_encoding_commit_message(self):
275
 
        if sys.platform == 'win32':
276
 
            raise tests.TestNotApplicable('Win32 parses arguments directly'
277
 
                ' as Unicode, so we can\'t pass invalid non-ascii')
278
237
        tree = self.make_branch_and_tree('.')
279
238
        self.build_tree_contents([('foo.c', 'int main() {}')])
280
239
        tree.add('foo.c')
281
 
        # LANG env variable has no effect on Windows
282
 
        # but some characters anyway cannot be represented
283
 
        # in default user encoding
284
 
        char = probe_bad_non_ascii(osutils.get_user_encoding())
285
 
        if char is None:
286
 
            raise TestSkipped('Cannot find suitable non-ascii character'
287
 
                'for user_encoding (%s)' % osutils.get_user_encoding())
288
 
        out,err = self.run_bzr_subprocess('commit -m "%s"' % char,
289
 
                                          retcode=1,
290
 
                                          env_changes={'LANG': 'C'})
 
240
        out,err = self.run_bzr_subprocess('commit -m "\xff"', retcode=1,
 
241
                                                    env_changes={'LANG': 'C'})
291
242
        self.assertContainsRe(err, r'bzrlib.errors.BzrError: Parameter.*is '
292
243
                                    'unsupported by the current encoding.')
293
244
 
299
250
        self.build_tree_contents([
300
251
            ('branch/foo.c', 'int main() {}'),
301
252
            ('branch/bar.c', 'int main() {}')])
302
 
        inner_tree.add(['foo.c', 'bar.c'])
 
253
        inner_tree.add('foo.c')
 
254
        inner_tree.add('bar.c')
303
255
        # can't commit files in different trees; sane error
304
256
        self.run_bzr('commit -m newstuff branch/foo.c .', retcode=3)
305
 
        # can commit to branch - records foo.c only
306
257
        self.run_bzr('commit -m newstuff branch/foo.c')
307
 
        # can commit to branch - records bar.c
308
258
        self.run_bzr('commit -m newstuff branch')
309
 
        # No changes left
310
 
        self.run_bzr_error(["No changes to commit"], 'commit -m newstuff branch')
 
259
        self.run_bzr('commit -m newstuff branch', retcode=3)
311
260
 
312
261
    def test_out_of_date_tree_commit(self):
313
262
        # check we get an error code and a clear message committing with an out
337
286
    def test_commit_a_text_merge_in_a_checkout(self):
338
287
        # checkouts perform multiple actions in a transaction across bond
339
288
        # branches and their master, and have been observed to fail in the
340
 
        # past. This is a user story reported to fail in bug #43959 where
 
289
        # past. This is a user story reported to fail in bug #43959 where 
341
290
        # a merge done in a checkout (using the update command) failed to
342
291
        # commit correctly.
343
292
        trunk = self.make_branch_and_tree('trunk')
344
293
 
345
294
        u1 = trunk.branch.create_checkout('u1')
346
 
        self.build_tree_contents([('u1/hosts', 'initial contents\n')])
 
295
        self.build_tree_contents([('u1/hosts', 'initial contents')])
347
296
        u1.add('hosts')
348
297
        self.run_bzr('commit -m add-hosts u1')
349
298
 
350
299
        u2 = trunk.branch.create_checkout('u2')
351
 
        self.build_tree_contents([('u2/hosts', 'altered in u2\n')])
 
300
        self.build_tree_contents([('u2/hosts', 'altered in u2')])
352
301
        self.run_bzr('commit -m checkin-from-u2 u2')
353
302
 
354
303
        # make an offline commits
355
 
        self.build_tree_contents([('u1/hosts', 'first offline change in u1\n')])
 
304
        self.build_tree_contents([('u1/hosts', 'first offline change in u1')])
356
305
        self.run_bzr('commit -m checkin-offline --local u1')
357
306
 
358
307
        # now try to pull in online work from u2, and then commit our offline
359
308
        # work as a merge
360
309
        # retcode 1 as we expect a text conflict
361
310
        self.run_bzr('update u1', retcode=1)
362
 
        self.assertFileEqual('''\
363
 
<<<<<<< TREE
364
 
first offline change in u1
365
 
=======
366
 
altered in u2
367
 
>>>>>>> MERGE-SOURCE
368
 
''',
369
 
                             'u1/hosts')
370
 
 
371
311
        self.run_bzr('resolved u1/hosts')
372
312
        # add a text change here to represent resolving the merge conflicts in
373
313
        # favour of a new version of the file not identical to either the u1
375
315
        self.build_tree_contents([('u1/hosts', 'merge resolution\n')])
376
316
        self.run_bzr('commit -m checkin-merge-of-the-offline-work-from-u1 u1')
377
317
 
378
 
    def test_commit_exclude_excludes_modified_files(self):
379
 
        """Commit -x foo should ignore changes to foo."""
380
 
        tree = self.make_branch_and_tree('.')
381
 
        self.build_tree(['a', 'b', 'c'])
382
 
        tree.smart_add(['.'])
383
 
        out, err = self.run_bzr(['commit', '-m', 'test', '-x', 'b'])
384
 
        self.assertFalse('added b' in out)
385
 
        self.assertFalse('added b' in err)
386
 
        # If b was excluded it will still be 'added' in status.
387
 
        out, err = self.run_bzr(['added'])
388
 
        self.assertEqual('b\n', out)
389
 
        self.assertEqual('', err)
390
 
 
391
 
    def test_commit_exclude_twice_uses_both_rules(self):
392
 
        """Commit -x foo -x bar should ignore changes to foo and bar."""
393
 
        tree = self.make_branch_and_tree('.')
394
 
        self.build_tree(['a', 'b', 'c'])
395
 
        tree.smart_add(['.'])
396
 
        out, err = self.run_bzr(['commit', '-m', 'test', '-x', 'b', '-x', 'c'])
397
 
        self.assertFalse('added b' in out)
398
 
        self.assertFalse('added c' in out)
399
 
        self.assertFalse('added b' in err)
400
 
        self.assertFalse('added c' in err)
401
 
        # If b was excluded it will still be 'added' in status.
402
 
        out, err = self.run_bzr(['added'])
403
 
        self.assertTrue('b\n' in out)
404
 
        self.assertTrue('c\n' in out)
405
 
        self.assertEqual('', err)
406
 
 
407
318
    def test_commit_respects_spec_for_removals(self):
408
319
        """Commit with a file spec should only commit removals that match"""
409
320
        t = self.make_branch_and_tree('.')
436
347
 
437
348
        # With no changes, it should just be 'no changes'
438
349
        # Make sure that commit is failing because there is nothing to do
439
 
        self.run_bzr_error(['No changes to commit'],
 
350
        self.run_bzr_error(['no changes to commit'],
440
351
                           'commit --strict -m no-changes',
441
352
                           working_dir='tree')
442
353
 
470
381
        output, err = self.run_bzr(
471
382
            'commit -m hello --fixes=lp:23452 tree/hello.txt')
472
383
        self.assertEqual('', output)
473
 
        self.assertContainsRe(err, 'Committing to: .*\n'
 
384
        self.assertContainsRe(err, 'Committing revision 1 to ".*"\.\n'
474
385
                              'added hello\.txt\n'
475
386
                              'Committed revision 1\.\n')
476
387
 
561
472
        self.build_tree(['tree/hello.txt'])
562
473
        tree.add('hello.txt')
563
474
        self.run_bzr_error(
564
 
            ["Did not understand bug identifier orange: Must be an integer. "
565
 
             "See \"bzr help bugs\" for more information on this feature.\n"
566
 
             "Commit refused."],
 
475
            ["Invalid bug identifier for %s. Commit refused." % 'lp:orange'],
567
476
            'commit -m add-b --fixes=lp:orange',
568
477
            working_dir='tree')
569
478
 
573
482
        self.build_tree(['tree/hello.txt'])
574
483
        tree.add('hello.txt')
575
484
        self.run_bzr_error(
576
 
            [r"Invalid bug orange. Must be in the form of 'tracker:id'\. "
577
 
             r"See \"bzr help bugs\" for more information on this feature.\n"
 
485
            [r"Invalid bug orange. Must be in the form of 'tag:id'\. "
578
486
             r"Commit refused\."],
579
487
            'commit -m add-b --fixes=orange',
580
488
            working_dir='tree')
596
504
        tree = self.make_branch_and_tree('tree')
597
505
        self.build_tree(['tree/hello.txt'])
598
506
        tree.add('hello.txt')
599
 
        self.run_bzr(["commit", '-m', 'hello',
600
 
                      '--author', u'John D\xf6 <jdoe@example.com>',
601
 
                     "tree/hello.txt"])
 
507
        self.run_bzr("commit -m hello --author='John Doe <jdoe@example.com>' "
 
508
                     "tree/hello.txt")
602
509
        last_rev = tree.branch.repository.get_revision(tree.last_revision())
603
510
        properties = last_rev.properties
604
 
        self.assertEqual(u'John D\xf6 <jdoe@example.com>', properties['authors'])
 
511
        self.assertEqual('John Doe <jdoe@example.com>', properties['author'])
605
512
 
606
513
    def test_author_no_email(self):
607
514
        """Author's name without an email address is allowed, too."""
612
519
                                "tree/hello.txt")
613
520
        last_rev = tree.branch.repository.get_revision(tree.last_revision())
614
521
        properties = last_rev.properties
615
 
        self.assertEqual('John Doe', properties['authors'])
616
 
 
617
 
    def test_multiple_authors(self):
618
 
        """Multiple authors can be specyfied, and all are stored."""
619
 
        tree = self.make_branch_and_tree('tree')
620
 
        self.build_tree(['tree/hello.txt'])
621
 
        tree.add('hello.txt')
622
 
        out, err = self.run_bzr("commit -m hello --author='John Doe' "
623
 
                                "--author='Jane Rey' tree/hello.txt")
624
 
        last_rev = tree.branch.repository.get_revision(tree.last_revision())
625
 
        properties = last_rev.properties
626
 
        self.assertEqual('John Doe\nJane Rey', properties['authors'])
627
 
 
628
 
    def test_commit_time(self):
629
 
        tree = self.make_branch_and_tree('tree')
630
 
        self.build_tree(['tree/hello.txt'])
631
 
        tree.add('hello.txt')
632
 
        out, err = self.run_bzr("commit -m hello "
633
 
            "--commit-time='2009-10-10 08:00:00 +0100' tree/hello.txt")
634
 
        last_rev = tree.branch.repository.get_revision(tree.last_revision())
635
 
        self.assertEqual(
636
 
            'Sat 2009-10-10 08:00:00 +0100',
637
 
            osutils.format_date(last_rev.timestamp, last_rev.timezone))
638
 
        
639
 
    def test_commit_time_bad_time(self):
640
 
        tree = self.make_branch_and_tree('tree')
641
 
        self.build_tree(['tree/hello.txt'])
642
 
        tree.add('hello.txt')
643
 
        out, err = self.run_bzr("commit -m hello "
644
 
            "--commit-time='NOT A TIME' tree/hello.txt", retcode=3)
645
 
        self.assertStartsWith(
646
 
            err, "bzr: ERROR: Could not parse --commit-time:")
647
 
 
648
 
    def test_partial_commit_with_renames_in_tree(self):
649
 
        # this test illustrates bug #140419
650
 
        t = self.make_branch_and_tree('.')
651
 
        self.build_tree(['dir/', 'dir/a', 'test'])
652
 
        t.add(['dir/', 'dir/a', 'test'])
653
 
        t.commit('initial commit')
654
 
        # important part: file dir/a should change parent
655
 
        # and should appear before old parent
656
 
        # then during partial commit we have error
657
 
        # parent_id {dir-XXX} not in inventory
658
 
        t.rename_one('dir/a', 'a')
659
 
        self.build_tree_contents([('test', 'changes in test')])
660
 
        # partial commit
661
 
        out, err = self.run_bzr('commit test -m "partial commit"')
662
 
        self.assertEquals('', out)
663
 
        self.assertContainsRe(err, r'modified test\nCommitted revision 2.')
664
 
 
665
 
    def test_commit_readonly_checkout(self):
666
 
        # https://bugs.edge.launchpad.net/bzr/+bug/129701
667
 
        # "UnlockableTransport error trying to commit in checkout of readonly
668
 
        # branch"
669
 
        self.make_branch('master')
670
 
        master = BzrDir.open_from_transport(
671
 
            self.get_readonly_transport('master')).open_branch()
672
 
        master.create_checkout('checkout')
673
 
        out, err = self.run_bzr(['commit', '--unchanged', '-mfoo', 'checkout'],
674
 
            retcode=3)
675
 
        self.assertContainsRe(err,
676
 
            r'^bzr: ERROR: Cannot lock.*readonly transport')
677
 
 
678
 
    def setup_editor(self):
679
 
        # Test that commit template hooks work
680
 
        if sys.platform == "win32":
681
 
            f = file('fed.bat', 'w')
682
 
            f.write('@rem dummy fed')
683
 
            f.close()
684
 
            osutils.set_or_unset_env('BZR_EDITOR', "fed.bat")
685
 
        else:
686
 
            f = file('fed.sh', 'wb')
687
 
            f.write('#!/bin/sh\n')
688
 
            f.close()
689
 
            os.chmod('fed.sh', 0755)
690
 
            osutils.set_or_unset_env('BZR_EDITOR', "./fed.sh")
691
 
 
692
 
    def setup_commit_with_template(self):
693
 
        self.setup_editor()
694
 
        msgeditor.hooks.install_named_hook("commit_message_template",
695
 
                lambda commit_obj, msg: "save me some typing\n", None)
696
 
        tree = self.make_branch_and_tree('tree')
697
 
        self.build_tree(['tree/hello.txt'])
698
 
        tree.add('hello.txt')
699
 
        return tree
700
 
 
701
 
    def test_commit_hook_template_accepted(self):
702
 
        tree = self.setup_commit_with_template()
703
 
        out, err = self.run_bzr("commit tree/hello.txt", stdin="y\n")
704
 
        last_rev = tree.branch.repository.get_revision(tree.last_revision())
705
 
        self.assertEqual('save me some typing\n', last_rev.message)
706
 
 
707
 
    def test_commit_hook_template_rejected(self):
708
 
        tree = self.setup_commit_with_template()
709
 
        expected = tree.last_revision()
710
 
        out, err = self.run_bzr_error(["empty commit message"],
711
 
            "commit tree/hello.txt", stdin="n\n")
712
 
        self.assertEqual(expected, tree.last_revision())
 
522
        self.assertEqual('John Doe', properties['author'])