← 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: Martin von Gagern
  • Date: 2010-04-22 06:25:26 UTC
  • mfrom: (0.27.39 trunk)
  • mto: This revision was merged to the branch mainline in revision 5240.
  • Revision ID: martin.vgagern@gmx.net-20100422062526-4lyy2yoor932u80w
Join bzr-bash-completion plugin into core bzr tree.

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) 2006-2010 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,
 
27
    osutils,
32
28
    tests,
33
29
    )
34
30
from bzrlib.bzrdir import BzrDir
35
31
from bzrlib.tests import (
36
32
    probe_bad_non_ascii,
37
 
    test_foreign,
38
33
    TestSkipped,
39
 
    UnicodeFilenameFeature,
40
34
    )
41
 
from bzrlib.tests import TestCaseWithTransport
42
 
 
43
 
 
44
 
class TestCommit(TestCaseWithTransport):
 
35
from bzrlib.tests.blackbox import ExternalBase
 
36
 
 
37
 
 
38
class TestCommit(ExternalBase):
45
39
 
46
40
    def test_05_empty_commit(self):
47
41
        """Commit of tree with no versioned files should fail"""
50
44
        self.build_tree(['hello.txt'])
51
45
        out,err = self.run_bzr('commit -m empty', retcode=3)
52
46
        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))
 
47
        self.assertContainsRe(err, 'bzr: ERROR: No changes to commit\.'
 
48
                                  ' Use --unchanged to commit anyhow.\n')
64
49
 
65
50
    def test_commit_success(self):
66
51
        """Successful commit should not leave behind a bzr-commit-* file"""
72
57
        self.run_bzr(["commit", "--unchanged", "-m", u'foo\xb5'])
73
58
        self.assertEqual('', self.run_bzr('unknowns')[0])
74
59
 
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
60
    def test_commit_with_path(self):
90
61
        """Commit tree with path of root specified"""
91
62
        a_tree = self.make_branch_and_tree('a')
105
76
        self.run_bzr('resolved b/a_file')
106
77
        self.run_bzr(['commit', '-m', 'merge into b', 'b'])
107
78
 
 
79
 
108
80
    def test_10_verbose_commit(self):
109
81
        """Add one file and examine verbose commit output"""
110
82
        tree = self.make_branch_and_tree('.')
135
107
                              'modified hello\.txt\n'
136
108
                              'Committed revision 2\.\n$')
137
109
 
138
 
    def test_unicode_commit_message_is_filename(self):
139
 
        """Unicode commit message same as a filename (Bug #563646).
140
 
        """
141
 
        self.requireFeature(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
110
    def test_warn_about_forgotten_commit_message(self):
174
111
        """Test that the lack of -m parameter is caught"""
175
112
        wt = self.make_branch_and_tree('.')
334
271
        tree.add('foo.c')
335
272
        self.run_bzr('commit -m ""', retcode=3)
336
273
 
 
274
    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
        tree = self.make_branch_and_tree('.')
 
279
        self.build_tree_contents([('foo.c', 'int main() {}')])
 
280
        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'})
 
291
        self.assertContainsRe(err, r'bzrlib.errors.BzrError: Parameter.*is '
 
292
                                    'unsupported by the current encoding.')
 
293
 
337
294
    def test_other_branch_commit(self):
338
295
        # this branch is to ensure consistent behaviour, whether we're run
339
296
        # inside a branch, or not.
706
663
        self.assertContainsRe(err, r'modified test\nCommitted revision 2.')
707
664
 
708
665
    def test_commit_readonly_checkout(self):
709
 
        # https://bugs.launchpad.net/bzr/+bug/129701
 
666
        # https://bugs.edge.launchpad.net/bzr/+bug/129701
710
667
        # "UnlockableTransport error trying to commit in checkout of readonly
711
668
        # branch"
712
669
        self.make_branch('master')
724
681
            f = file('fed.bat', 'w')
725
682
            f.write('@rem dummy fed')
726
683
            f.close()
727
 
            self.overrideEnv('BZR_EDITOR', "fed.bat")
 
684
            osutils.set_or_unset_env('BZR_EDITOR', "fed.bat")
728
685
        else:
729
686
            f = file('fed.sh', 'wb')
730
687
            f.write('#!/bin/sh\n')
731
688
            f.close()
732
689
            os.chmod('fed.sh', 0755)
733
 
            self.overrideEnv('BZR_EDITOR', "./fed.sh")
 
690
            osutils.set_or_unset_env('BZR_EDITOR', "./fed.sh")
734
691
 
735
692
    def setup_commit_with_template(self):
736
693
        self.setup_editor()
753
710
        out, err = self.run_bzr_error(["empty commit message"],
754
711
            "commit tree/hello.txt", stdin="n\n")
755
712
        self.assertEqual(expected, tree.last_revision())
756
 
 
757
 
    def test_set_commit_message(self):
758
 
        msgeditor.hooks.install_named_hook("set_commit_message",
759
 
                lambda commit_obj, msg: "save me some typing\n", None)
760
 
        tree = self.make_branch_and_tree('tree')
761
 
        self.build_tree(['tree/hello.txt'])
762
 
        tree.add('hello.txt')
763
 
        out, err = self.run_bzr("commit tree/hello.txt")
764
 
        last_rev = tree.branch.repository.get_revision(tree.last_revision())
765
 
        self.assertEqual('save me some typing\n', last_rev.message)
766
 
 
767
 
    def test_commit_without_username(self):
768
 
        """Ensure commit error if username is not set.
769
 
        """
770
 
        self.run_bzr(['init', 'foo'])
771
 
        os.chdir('foo')
772
 
        open('foo.txt', 'w').write('hello')
773
 
        self.run_bzr(['add'])
774
 
        self.overrideEnv('EMAIL', None)
775
 
        self.overrideEnv('BZR_EMAIL', None)
776
 
        # Also, make sure that it's not inferred from mailname.
777
 
        self.overrideAttr(config, '_auto_user_id',
778
 
            lambda: (None, None))
779
 
        out, err = self.run_bzr(['commit', '-m', 'initial'], 3)
780
 
        self.assertContainsRe(err, 'Unable to determine your name')
781
 
 
782
 
    def test_commit_recursive_checkout(self):
783
 
        """Ensure that a commit to a recursive checkout fails cleanly.
784
 
        """
785
 
        self.run_bzr(['init', 'test_branch'])
786
 
        self.run_bzr(['checkout', 'test_branch', 'test_checkout'])
787
 
        os.chdir('test_checkout')
788
 
        self.run_bzr(['bind', '.']) # bind to self
789
 
        open('foo.txt', 'w').write('hello')
790
 
        self.run_bzr(['add'])
791
 
        out, err = self.run_bzr(['commit', '-m', 'addedfoo'], 3)
792
 
        self.assertEqual(out, '')
793
 
        self.assertContainsRe(err,
794
 
            'Branch.*test_checkout.*appears to be bound to itself')