← 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: Andrew Bennetts
  • Date: 2010-07-29 11:17:57 UTC
  • mfrom: (5050.3.17 2.2)
  • mto: This revision was merged to the branch mainline in revision 5365.
  • Revision ID: andrew.bennetts@canonical.com-20100729111757-018h3pcefo7z0dnq
Merge lp:bzr/2.2 into lp:bzr.

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
21
import re
23
22
import sys
24
23
 
25
 
from testtools.matchers import DocTestMatches
26
 
 
27
24
from bzrlib import (
28
 
    config,
 
25
    bzrdir,
29
26
    osutils,
30
27
    ignores,
31
28
    msgeditor,
 
29
    osutils,
 
30
    tests,
32
31
    )
33
32
from bzrlib.bzrdir import BzrDir
34
33
from bzrlib.tests import (
35
 
    test_foreign,
36
 
    features,
 
34
    probe_bad_non_ascii,
 
35
    TestSkipped,
37
36
    )
38
37
from bzrlib.tests import TestCaseWithTransport
39
38
 
47
46
        self.build_tree(['hello.txt'])
48
47
        out,err = self.run_bzr('commit -m empty', retcode=3)
49
48
        self.assertEqual('', out)
50
 
        # Two ugly bits here.
51
 
        # 1) We really don't want 'aborting commit write group' anymore.
52
 
        # 2) bzr: ERROR: is a really long line, so we wrap it with '\'
53
 
        self.assertThat(
54
 
            err,
55
 
            DocTestMatches("""\
56
 
Committing to: ...
57
 
bzr: ERROR: No changes to commit.\
58
 
 Please 'bzr add' the files you want to commit,\
59
 
 or use --unchanged to force an empty commit.
60
 
""", flags=doctest.ELLIPSIS|doctest.REPORT_UDIFF))
 
49
        self.assertContainsRe(err, 'bzr: ERROR: No changes to commit\.'
 
50
                                  ' Use --unchanged to commit anyhow.\n')
61
51
 
62
52
    def test_commit_success(self):
63
53
        """Successful commit should not leave behind a bzr-commit-* file"""
69
59
        self.run_bzr(["commit", "--unchanged", "-m", u'foo\xb5'])
70
60
        self.assertEqual('', self.run_bzr('unknowns')[0])
71
61
 
72
 
    def test_commit_lossy_native(self):
73
 
        """A --lossy option to commit is supported."""
74
 
        self.make_branch_and_tree('.')
75
 
        self.run_bzr('commit --lossy --unchanged -m message')
76
 
        self.assertEqual('', self.run_bzr('unknowns')[0])
77
 
 
78
 
    def test_commit_lossy_foreign(self):
79
 
        test_foreign.register_dummy_foreign_for_test(self)
80
 
        self.make_branch_and_tree('.',
81
 
            format=test_foreign.DummyForeignVcsDirFormat())
82
 
        self.run_bzr('commit --lossy --unchanged -m message')
83
 
        output = self.run_bzr('revision-info')[0]
84
 
        self.assertTrue(output.startswith('1 dummy-'))
85
 
 
86
62
    def test_commit_with_path(self):
87
63
        """Commit tree with path of root specified"""
88
64
        a_tree = self.make_branch_and_tree('a')
102
78
        self.run_bzr('resolved b/a_file')
103
79
        self.run_bzr(['commit', '-m', 'merge into b', 'b'])
104
80
 
 
81
 
105
82
    def test_10_verbose_commit(self):
106
83
        """Add one file and examine verbose commit output"""
107
84
        tree = self.make_branch_and_tree('.')
135
112
    def test_unicode_commit_message_is_filename(self):
136
113
        """Unicode commit message same as a filename (Bug #563646).
137
114
        """
138
 
        self.requireFeature(features.UnicodeFilenameFeature)
139
115
        file_name = u'\N{euro sign}'
140
116
        self.run_bzr(['init'])
141
117
        open(file_name, 'w').write('hello world')
329
305
        tree = self.make_branch_and_tree('.')
330
306
        self.build_tree_contents([('foo.c', 'int main() {}')])
331
307
        tree.add('foo.c')
332
 
        self.run_bzr('commit -m ""')
 
308
        self.run_bzr('commit -m ""', retcode=3)
 
309
 
 
310
    def test_unsupported_encoding_commit_message(self):
 
311
        if sys.platform == 'win32':
 
312
            raise tests.TestNotApplicable('Win32 parses arguments directly'
 
313
                ' as Unicode, so we can\'t pass invalid non-ascii')
 
314
        tree = self.make_branch_and_tree('.')
 
315
        self.build_tree_contents([('foo.c', 'int main() {}')])
 
316
        tree.add('foo.c')
 
317
        # LANG env variable has no effect on Windows
 
318
        # but some characters anyway cannot be represented
 
319
        # in default user encoding
 
320
        char = probe_bad_non_ascii(osutils.get_user_encoding())
 
321
        if char is None:
 
322
            raise TestSkipped('Cannot find suitable non-ascii character'
 
323
                'for user_encoding (%s)' % osutils.get_user_encoding())
 
324
        out,err = self.run_bzr_subprocess('commit -m "%s"' % char,
 
325
                                          retcode=1,
 
326
                                          env_changes={'LANG': 'C'})
 
327
        self.assertContainsRe(err, r'bzrlib.errors.BzrError: Parameter.*is '
 
328
                                    'unsupported by the current encoding.')
333
329
 
334
330
    def test_other_branch_commit(self):
335
331
        # this branch is to ensure consistent behaviour, whether we're run
596
592
            'commit -m add-b --fixes=xxx:123',
597
593
            working_dir='tree')
598
594
 
599
 
    def test_fixes_bug_with_default_tracker(self):
600
 
        """commit --fixes=234 uses the default bug tracker."""
601
 
        tree = self.make_branch_and_tree('tree')
602
 
        self.build_tree(['tree/hello.txt'])
603
 
        tree.add('hello.txt')
604
 
        self.run_bzr_error(
605
 
            ["bzr: ERROR: No tracker specified for bug 123. Use the form "
606
 
            "'tracker:id' or specify a default bug tracker using the "
607
 
            "`bugtracker` option.\n"
608
 
            "See \"bzr help bugs\" for more information on this feature. "
609
 
            "Commit refused."],
610
 
            'commit -m add-b --fixes=123',
611
 
            working_dir='tree')
612
 
        tree.branch.get_config().set_user_option("bugtracker", "lp")
613
 
        self.run_bzr('commit -m hello --fixes=234 tree/hello.txt')
614
 
        last_rev = tree.branch.repository.get_revision(tree.last_revision())
615
 
        self.assertEqual('https://launchpad.net/bugs/234 fixed',
616
 
                         last_rev.properties['bugs'])
617
 
 
618
595
    def test_fixes_invalid_bug_number(self):
619
596
        tree = self.make_branch_and_tree('tree')
620
597
        self.build_tree(['tree/hello.txt'])
632
609
        self.build_tree(['tree/hello.txt'])
633
610
        tree.add('hello.txt')
634
611
        self.run_bzr_error(
635
 
            [r"Invalid bug orange:apples:bananas. Must be in the form of "
636
 
             r"'tracker:id'\. See \"bzr help bugs\" for more information on "
637
 
             r"this feature.\nCommit refused\."],
638
 
            'commit -m add-b --fixes=orange:apples:bananas',
 
612
            [r"Invalid bug orange. Must be in the form of 'tracker:id'\. "
 
613
             r"See \"bzr help bugs\" for more information on this feature.\n"
 
614
             r"Commit refused\."],
 
615
            'commit -m add-b --fixes=orange',
639
616
            working_dir='tree')
640
617
 
641
618
    def test_no_author(self):
740
717
            f = file('fed.bat', 'w')
741
718
            f.write('@rem dummy fed')
742
719
            f.close()
743
 
            self.overrideEnv('BZR_EDITOR', "fed.bat")
 
720
            osutils.set_or_unset_env('BZR_EDITOR', "fed.bat")
744
721
        else:
745
722
            f = file('fed.sh', 'wb')
746
723
            f.write('#!/bin/sh\n')
747
724
            f.close()
748
725
            os.chmod('fed.sh', 0755)
749
 
            self.overrideEnv('BZR_EDITOR', "./fed.sh")
 
726
            osutils.set_or_unset_env('BZR_EDITOR', "./fed.sh")
750
727
 
751
728
    def setup_commit_with_template(self):
752
729
        self.setup_editor()
757
734
        tree.add('hello.txt')
758
735
        return tree
759
736
 
760
 
    def test_edit_empty_message(self):
761
 
        tree = self.make_branch_and_tree('tree')
762
 
        self.setup_editor()
763
 
        self.build_tree(['tree/hello.txt'])
764
 
        tree.add('hello.txt')
765
 
        out, err = self.run_bzr("commit tree/hello.txt", retcode=3,
766
 
            stdin="y\n")
767
 
        self.assertContainsRe(err,
768
 
            "bzr: ERROR: Empty commit message specified")
769
 
 
770
737
    def test_commit_hook_template_accepted(self):
771
738
        tree = self.setup_commit_with_template()
772
739
        out, err = self.run_bzr("commit tree/hello.txt", stdin="y\n")
776
743
    def test_commit_hook_template_rejected(self):
777
744
        tree = self.setup_commit_with_template()
778
745
        expected = tree.last_revision()
779
 
        out, err = self.run_bzr_error(["Empty commit message specified."
780
 
                  " Please specify a commit message with either"
781
 
                  " --message or --file or leave a blank message"
782
 
                  " with --message \"\"."],
 
746
        out, err = self.run_bzr_error(["empty commit message"],
783
747
            "commit tree/hello.txt", stdin="n\n")
784
748
        self.assertEqual(expected, tree.last_revision())
785
749
 
786
 
    def test_set_commit_message(self):
787
 
        msgeditor.hooks.install_named_hook("set_commit_message",
788
 
                lambda commit_obj, msg: "save me some typing\n", None)
789
 
        tree = self.make_branch_and_tree('tree')
790
 
        self.build_tree(['tree/hello.txt'])
791
 
        tree.add('hello.txt')
792
 
        out, err = self.run_bzr("commit tree/hello.txt")
793
 
        last_rev = tree.branch.repository.get_revision(tree.last_revision())
794
 
        self.assertEqual('save me some typing\n', last_rev.message)
795
 
 
796
750
    def test_commit_without_username(self):
797
751
        """Ensure commit error if username is not set.
798
752
        """
800
754
        os.chdir('foo')
801
755
        open('foo.txt', 'w').write('hello')
802
756
        self.run_bzr(['add'])
803
 
        self.overrideEnv('EMAIL', None)
804
 
        self.overrideEnv('BZR_EMAIL', None)
805
 
        # Also, make sure that it's not inferred from mailname.
806
 
        self.overrideAttr(config, '_auto_user_id',
807
 
            lambda: (None, None))
 
757
        osutils.set_or_unset_env('EMAIL', None)
 
758
        osutils.set_or_unset_env('BZR_EMAIL', None)
808
759
        out, err = self.run_bzr(['commit', '-m', 'initial'], 3)
809
760
        self.assertContainsRe(err, 'Unable to determine your name')
810
761
 
821
772
        self.assertEqual(out, '')
822
773
        self.assertContainsRe(err,
823
774
            'Branch.*test_checkout.*appears to be bound to itself')
 
775