← 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: Canonical.com Patch Queue Manager
  • Date: 2009-10-05 15:41:35 UTC
  • mfrom: (4725.2.1 integration)
  • Revision ID: pqm@pqm.ubuntu.com-20091005154135-k407ea8ssyhayv10
(vila) Disambiguate per_transport test IDs

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/tests/blackbox/test_commit.py
1
 
# Copyright (C) 2006-2011 Canonical Ltd
 
1
# Copyright (C) 2005, 2006, 2007, 2009 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
17
17
 
18
18
"""Tests for the commit CLI of bzr."""
19
19
 
20
 
import doctest
21
20
import os
22
 
import re
23
21
import sys
24
22
 
25
 
from testtools.matchers import DocTestMatches
26
 
 
27
23
from bzrlib import (
28
 
    config,
29
24
    osutils,
30
25
    ignores,
31
26
    msgeditor,
32
 
    tests,
 
27
    osutils,
33
28
    )
34
29
from bzrlib.bzrdir import BzrDir
35
30
from bzrlib.tests import (
36
31
    probe_bad_non_ascii,
37
 
    test_foreign,
38
32
    TestSkipped,
39
 
    features,
40
33
    )
41
 
from bzrlib.tests import TestCaseWithTransport
42
 
 
43
 
 
44
 
class TestCommit(TestCaseWithTransport):
 
34
from bzrlib.tests.blackbox import ExternalBase
 
35
 
 
36
 
 
37
class TestCommit(ExternalBase):
45
38
 
46
39
    def test_05_empty_commit(self):
47
40
        """Commit of tree with no versioned files should fail"""
50
43
        self.build_tree(['hello.txt'])
51
44
        out,err = self.run_bzr('commit -m empty', retcode=3)
52
45
        self.assertEqual('', out)
53
 
        # Two ugly bits here.
54
 
        # 1) We really don't want 'aborting commit write group' anymore.
55
 
        # 2) bzr: ERROR: is a really long line, so we wrap it with '\'
56
 
        self.assertThat(
57
 
            err,
58
 
            DocTestMatches("""\
59
 
Committing to: ...
60
 
bzr: ERROR: No changes to commit.\
61
 
 Please 'bzr add' the files you want to commit,\
62
 
 or use --unchanged to force an empty commit.
63
 
""", flags=doctest.ELLIPSIS|doctest.REPORT_UDIFF))
 
46
        self.assertContainsRe(err, 'bzr: ERROR: No changes to commit\.'
 
47
                                  ' Use --unchanged to commit anyhow.\n')
64
48
 
65
49
    def test_commit_success(self):
66
50
        """Successful commit should not leave behind a bzr-commit-* file"""
72
56
        self.run_bzr(["commit", "--unchanged", "-m", u'foo\xb5'])
73
57
        self.assertEqual('', self.run_bzr('unknowns')[0])
74
58
 
75
 
    def test_commit_lossy_native(self):
76
 
        """A --lossy option to commit is supported."""
77
 
        self.make_branch_and_tree('.')
78
 
        self.run_bzr('commit --lossy --unchanged -m message')
79
 
        self.assertEqual('', self.run_bzr('unknowns')[0])
80
 
 
81
 
    def test_commit_lossy_foreign(self):
82
 
        test_foreign.register_dummy_foreign_for_test(self)
83
 
        self.make_branch_and_tree('.',
84
 
            format=test_foreign.DummyForeignVcsDirFormat())
85
 
        self.run_bzr('commit --lossy --unchanged -m message')
86
 
        output = self.run_bzr('revision-info')[0]
87
 
        self.assertTrue(output.startswith('1 dummy-'))
88
 
 
89
59
    def test_commit_with_path(self):
90
60
        """Commit tree with path of root specified"""
91
61
        a_tree = self.make_branch_and_tree('a')
105
75
        self.run_bzr('resolved b/a_file')
106
76
        self.run_bzr(['commit', '-m', 'merge into b', 'b'])
107
77
 
 
78
 
108
79
    def test_10_verbose_commit(self):
109
80
        """Add one file and examine verbose commit output"""
110
81
        tree = self.make_branch_and_tree('.')
135
106
                              'modified hello\.txt\n'
136
107
                              'Committed revision 2\.\n$')
137
108
 
138
 
    def test_unicode_commit_message_is_filename(self):
139
 
        """Unicode commit message same as a filename (Bug #563646).
140
 
        """
141
 
        self.requireFeature(features.UnicodeFilenameFeature)
142
 
        file_name = u'\N{euro sign}'
143
 
        self.run_bzr(['init'])
144
 
        open(file_name, 'w').write('hello world')
145
 
        self.run_bzr(['add'])
146
 
        out, err = self.run_bzr(['commit', '-m', file_name])
147
 
        reflags = re.MULTILINE|re.DOTALL|re.UNICODE
148
 
        te = osutils.get_terminal_encoding()
149
 
        self.assertContainsRe(err.decode(te),
150
 
            u'The commit message is a file name:',
151
 
            flags=reflags)
152
 
 
153
 
        # Run same test with a filename that causes encode
154
 
        # error for the terminal encoding. We do this
155
 
        # by forcing terminal encoding of ascii for
156
 
        # osutils.get_terminal_encoding which is used
157
 
        # by ui.text.show_warning
158
 
        default_get_terminal_enc = osutils.get_terminal_encoding
159
 
        try:
160
 
            osutils.get_terminal_encoding = lambda trace=None: 'ascii'
161
 
            file_name = u'foo\u1234'
162
 
            open(file_name, 'w').write('hello world')
163
 
            self.run_bzr(['add'])
164
 
            out, err = self.run_bzr(['commit', '-m', file_name])
165
 
            reflags = re.MULTILINE|re.DOTALL|re.UNICODE
166
 
            te = osutils.get_terminal_encoding()
167
 
            self.assertContainsRe(err.decode(te, 'replace'),
168
 
                u'The commit message is a file name:',
169
 
                flags=reflags)
170
 
        finally:
171
 
            osutils.get_terminal_encoding = default_get_terminal_enc
172
 
 
173
 
    def test_warn_about_forgotten_commit_message(self):
174
 
        """Test that the lack of -m parameter is caught"""
175
 
        wt = self.make_branch_and_tree('.')
176
 
        self.build_tree(['one', 'two'])
177
 
        wt.add(['two'])
178
 
        out, err = self.run_bzr('commit -m one two')
179
 
        self.assertContainsRe(err, "The commit message is a file name")
180
 
 
181
109
    def test_verbose_commit_renamed(self):
182
110
        # Verbose commit of renamed file should say so
183
111
        wt = self.prepare_simple_history()
242
170
        self.assertEqual(err, 'Committing to: %s\n'
243
171
                         'Committed revision 2.\n' % expected)
244
172
 
245
 
    def test_commit_sanitizes_CR_in_message(self):
246
 
        # See bug #433779, basically Emacs likes to pass '\r\n' style line
247
 
        # endings to 'bzr commit -m ""' which breaks because we don't allow
248
 
        # '\r' in commit messages. (Mostly because of issues where XML style
249
 
        # formats arbitrarily strip it out of the data while parsing.)
250
 
        # To make life easier for users, we just always translate '\r\n' =>
251
 
        # '\n'. And '\r' => '\n'.
252
 
        a_tree = self.make_branch_and_tree('a')
253
 
        self.build_tree(['a/b'])
254
 
        a_tree.add('b')
255
 
        self.run_bzr(['commit',
256
 
                      '-m', 'a string\r\n\r\nwith mixed\r\rendings\n'],
257
 
                     working_dir='a')
258
 
        rev_id = a_tree.branch.last_revision()
259
 
        rev = a_tree.branch.repository.get_revision(rev_id)
260
 
        self.assertEqualDiff('a string\n\nwith mixed\n\nendings\n',
261
 
                             rev.message)
262
 
 
263
173
    def test_commit_merge_reports_all_modified_files(self):
264
174
        # the commit command should show all the files that are shown by
265
175
        # bzr diff or bzr status when committing, even when they were not
334
244
        tree.add('foo.c')
335
245
        self.run_bzr('commit -m ""', retcode=3)
336
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.')
 
267
 
337
268
    def test_other_branch_commit(self):
338
269
        # this branch is to ensure consistent behaviour, whether we're run
339
270
        # inside a branch, or not.
386
317
        trunk = self.make_branch_and_tree('trunk')
387
318
 
388
319
        u1 = trunk.branch.create_checkout('u1')
389
 
        self.build_tree_contents([('u1/hosts', 'initial contents\n')])
 
320
        self.build_tree_contents([('u1/hosts', 'initial contents')])
390
321
        u1.add('hosts')
391
322
        self.run_bzr('commit -m add-hosts u1')
392
323
 
393
324
        u2 = trunk.branch.create_checkout('u2')
394
 
        self.build_tree_contents([('u2/hosts', 'altered in u2\n')])
 
325
        self.build_tree_contents([('u2/hosts', 'altered in u2')])
395
326
        self.run_bzr('commit -m checkin-from-u2 u2')
396
327
 
397
328
        # make an offline commits
398
 
        self.build_tree_contents([('u1/hosts', 'first offline change in u1\n')])
 
329
        self.build_tree_contents([('u1/hosts', 'first offline change in u1')])
399
330
        self.run_bzr('commit -m checkin-offline --local u1')
400
331
 
401
332
        # now try to pull in online work from u2, and then commit our offline
402
333
        # work as a merge
403
334
        # retcode 1 as we expect a text conflict
404
335
        self.run_bzr('update u1', retcode=1)
405
 
        self.assertFileEqual('''\
406
 
<<<<<<< TREE
407
 
first offline change in u1
408
 
=======
409
 
altered in u2
410
 
>>>>>>> MERGE-SOURCE
411
 
''',
412
 
                             'u1/hosts')
413
 
 
414
336
        self.run_bzr('resolved u1/hosts')
415
337
        # add a text change here to represent resolving the merge conflicts in
416
338
        # favour of a new version of the file not identical to either the u1
668
590
        properties = last_rev.properties
669
591
        self.assertEqual('John Doe\nJane Rey', properties['authors'])
670
592
 
671
 
    def test_commit_time(self):
672
 
        tree = self.make_branch_and_tree('tree')
673
 
        self.build_tree(['tree/hello.txt'])
674
 
        tree.add('hello.txt')
675
 
        out, err = self.run_bzr("commit -m hello "
676
 
            "--commit-time='2009-10-10 08:00:00 +0100' tree/hello.txt")
677
 
        last_rev = tree.branch.repository.get_revision(tree.last_revision())
678
 
        self.assertEqual(
679
 
            'Sat 2009-10-10 08:00:00 +0100',
680
 
            osutils.format_date(last_rev.timestamp, last_rev.timezone))
681
 
        
682
 
    def test_commit_time_bad_time(self):
683
 
        tree = self.make_branch_and_tree('tree')
684
 
        self.build_tree(['tree/hello.txt'])
685
 
        tree.add('hello.txt')
686
 
        out, err = self.run_bzr("commit -m hello "
687
 
            "--commit-time='NOT A TIME' tree/hello.txt", retcode=3)
688
 
        self.assertStartsWith(
689
 
            err, "bzr: ERROR: Could not parse --commit-time:")
690
 
 
691
593
    def test_partial_commit_with_renames_in_tree(self):
692
594
        # this test illustrates bug #140419
693
595
        t = self.make_branch_and_tree('.')
706
608
        self.assertContainsRe(err, r'modified test\nCommitted revision 2.')
707
609
 
708
610
    def test_commit_readonly_checkout(self):
709
 
        # https://bugs.launchpad.net/bzr/+bug/129701
 
611
        # https://bugs.edge.launchpad.net/bzr/+bug/129701
710
612
        # "UnlockableTransport error trying to commit in checkout of readonly
711
613
        # branch"
712
614
        self.make_branch('master')
718
620
        self.assertContainsRe(err,
719
621
            r'^bzr: ERROR: Cannot lock.*readonly transport')
720
622
 
721
 
    def setup_editor(self):
 
623
    def test_commit_hook_template(self):
722
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)
723
628
        if sys.platform == "win32":
724
629
            f = file('fed.bat', 'w')
725
630
            f.write('@rem dummy fed')
726
631
            f.close()
727
 
            self.overrideEnv('BZR_EDITOR', "fed.bat")
 
632
            default_editor = osutils.set_or_unset_env('BZR_EDITOR', "fed.bat")
728
633
        else:
729
634
            f = file('fed.sh', 'wb')
730
635
            f.write('#!/bin/sh\n')
731
636
            f.close()
732
637
            os.chmod('fed.sh', 0755)
733
 
            self.overrideEnv('BZR_EDITOR', "./fed.sh")
734
 
 
735
 
    def setup_commit_with_template(self):
736
 
        self.setup_editor()
 
638
            default_editor = osutils.set_or_unset_env('BZR_EDITOR', "./fed.sh")
 
639
        self.addCleanup(restoreDefaults)
737
640
        msgeditor.hooks.install_named_hook("commit_message_template",
738
641
                lambda commit_obj, msg: "save me some typing\n", None)
739
642
        tree = self.make_branch_and_tree('tree')
740
643
        self.build_tree(['tree/hello.txt'])
741
644
        tree.add('hello.txt')
742
 
        return tree
743
 
 
744
 
    def test_commit_hook_template_accepted(self):
745
 
        tree = self.setup_commit_with_template()
746
 
        out, err = self.run_bzr("commit tree/hello.txt", stdin="y\n")
747
 
        last_rev = tree.branch.repository.get_revision(tree.last_revision())
748
 
        self.assertEqual('save me some typing\n', last_rev.message)
749
 
 
750
 
    def test_commit_hook_template_rejected(self):
751
 
        tree = self.setup_commit_with_template()
752
 
        expected = tree.last_revision()
753
 
        out, err = self.run_bzr_error(["Empty commit message specified."
754
 
                  " Please specify a commit message with either"
755
 
                  " --message or --file or leave a blank message"
756
 
                  " with --message \"\"."],
757
 
            "commit tree/hello.txt", stdin="n\n")
758
 
        self.assertEqual(expected, tree.last_revision())
759
 
 
760
 
    def test_set_commit_message(self):
761
 
        msgeditor.hooks.install_named_hook("set_commit_message",
762
 
                lambda commit_obj, msg: "save me some typing\n", None)
763
 
        tree = self.make_branch_and_tree('tree')
764
 
        self.build_tree(['tree/hello.txt'])
765
 
        tree.add('hello.txt')
766
645
        out, err = self.run_bzr("commit tree/hello.txt")
767
646
        last_rev = tree.branch.repository.get_revision(tree.last_revision())
768
647
        self.assertEqual('save me some typing\n', last_rev.message)
769
 
 
770
 
    def test_commit_without_username(self):
771
 
        """Ensure commit error if username is not set.
772
 
        """
773
 
        self.run_bzr(['init', 'foo'])
774
 
        os.chdir('foo')
775
 
        open('foo.txt', 'w').write('hello')
776
 
        self.run_bzr(['add'])
777
 
        self.overrideEnv('EMAIL', None)
778
 
        self.overrideEnv('BZR_EMAIL', None)
779
 
        # Also, make sure that it's not inferred from mailname.
780
 
        self.overrideAttr(config, '_auto_user_id',
781
 
            lambda: (None, None))
782
 
        out, err = self.run_bzr(['commit', '-m', 'initial'], 3)
783
 
        self.assertContainsRe(err, 'Unable to determine your name')
784
 
 
785
 
    def test_commit_recursive_checkout(self):
786
 
        """Ensure that a commit to a recursive checkout fails cleanly.
787
 
        """
788
 
        self.run_bzr(['init', 'test_branch'])
789
 
        self.run_bzr(['checkout', 'test_branch', 'test_checkout'])
790
 
        os.chdir('test_checkout')
791
 
        self.run_bzr(['bind', '.']) # bind to self
792
 
        open('foo.txt', 'w').write('hello')
793
 
        self.run_bzr(['add'])
794
 
        out, err = self.run_bzr(['commit', '-m', 'addedfoo'], 3)
795
 
        self.assertEqual(out, '')
796
 
        self.assertContainsRe(err,
797
 
            'Branch.*test_checkout.*appears to be bound to itself')