← 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-06-22 17:11:20 UTC
  • mfrom: (4398.8.10 1.16-commit-fulltext)
  • Revision ID: pqm@pqm.ubuntu.com-20090622171120-fuxez9ylfqpxynqn
(jam) Add VF._add_text and reduce memory overhead during commit (see
        bug #109114)

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 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
        out,err = self.run_bzr_subprocess('commit -m "%s"' % char,
 
259
                                          retcode=1,
 
260
                                          env_changes={'LANG': 'C'})
 
261
        self.assertContainsRe(err, r'bzrlib.errors.BzrError: Parameter.*is '
 
262
                                    'unsupported by the current encoding.')
 
263
 
337
264
    def test_other_branch_commit(self):
338
265
        # this branch is to ensure consistent behaviour, whether we're run
339
266
        # inside a branch, or not.
342
269
        self.build_tree_contents([
343
270
            ('branch/foo.c', 'int main() {}'),
344
271
            ('branch/bar.c', 'int main() {}')])
345
 
        inner_tree.add(['foo.c', 'bar.c'])
 
272
        inner_tree.add('foo.c')
 
273
        inner_tree.add('bar.c')
346
274
        # can't commit files in different trees; sane error
347
275
        self.run_bzr('commit -m newstuff branch/foo.c .', retcode=3)
348
 
        # can commit to branch - records foo.c only
349
276
        self.run_bzr('commit -m newstuff branch/foo.c')
350
 
        # can commit to branch - records bar.c
351
277
        self.run_bzr('commit -m newstuff branch')
352
 
        # No changes left
353
 
        self.run_bzr_error(["No changes to commit"], 'commit -m newstuff branch')
 
278
        self.run_bzr('commit -m newstuff branch', retcode=3)
354
279
 
355
280
    def test_out_of_date_tree_commit(self):
356
281
        # check we get an error code and a clear message committing with an out
386
311
        trunk = self.make_branch_and_tree('trunk')
387
312
 
388
313
        u1 = trunk.branch.create_checkout('u1')
389
 
        self.build_tree_contents([('u1/hosts', 'initial contents\n')])
 
314
        self.build_tree_contents([('u1/hosts', 'initial contents')])
390
315
        u1.add('hosts')
391
316
        self.run_bzr('commit -m add-hosts u1')
392
317
 
393
318
        u2 = trunk.branch.create_checkout('u2')
394
 
        self.build_tree_contents([('u2/hosts', 'altered in u2\n')])
 
319
        self.build_tree_contents([('u2/hosts', 'altered in u2')])
395
320
        self.run_bzr('commit -m checkin-from-u2 u2')
396
321
 
397
322
        # make an offline commits
398
 
        self.build_tree_contents([('u1/hosts', 'first offline change in u1\n')])
 
323
        self.build_tree_contents([('u1/hosts', 'first offline change in u1')])
399
324
        self.run_bzr('commit -m checkin-offline --local u1')
400
325
 
401
326
        # now try to pull in online work from u2, and then commit our offline
402
327
        # work as a merge
403
328
        # retcode 1 as we expect a text conflict
404
329
        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
330
        self.run_bzr('resolved u1/hosts')
415
331
        # add a text change here to represent resolving the merge conflicts in
416
332
        # favour of a new version of the file not identical to either the u1
668
584
        properties = last_rev.properties
669
585
        self.assertEqual('John Doe\nJane Rey', properties['authors'])
670
586
 
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
587
    def test_partial_commit_with_renames_in_tree(self):
692
588
        # this test illustrates bug #140419
693
589
        t = self.make_branch_and_tree('.')
706
602
        self.assertContainsRe(err, r'modified test\nCommitted revision 2.')
707
603
 
708
604
    def test_commit_readonly_checkout(self):
709
 
        # https://bugs.launchpad.net/bzr/+bug/129701
 
605
        # https://bugs.edge.launchpad.net/bzr/+bug/129701
710
606
        # "UnlockableTransport error trying to commit in checkout of readonly
711
607
        # branch"
712
608
        self.make_branch('master')
718
614
        self.assertContainsRe(err,
719
615
            r'^bzr: ERROR: Cannot lock.*readonly transport')
720
616
 
721
 
    def setup_editor(self):
 
617
    def test_commit_hook_template(self):
722
618
        # Test that commit template hooks work
 
619
        def restoreDefaults():
 
620
            msgeditor.hooks['commit_message_template'] = []
 
621
            osutils.set_or_unset_env('BZR_EDITOR', default_editor)
723
622
        if sys.platform == "win32":
724
623
            f = file('fed.bat', 'w')
725
624
            f.write('@rem dummy fed')
726
625
            f.close()
727
 
            self.overrideEnv('BZR_EDITOR', "fed.bat")
 
626
            default_editor = osutils.set_or_unset_env('BZR_EDITOR', "fed.bat")
728
627
        else:
729
628
            f = file('fed.sh', 'wb')
730
629
            f.write('#!/bin/sh\n')
731
630
            f.close()
732
631
            os.chmod('fed.sh', 0755)
733
 
            self.overrideEnv('BZR_EDITOR', "./fed.sh")
734
 
 
735
 
    def setup_commit_with_template(self):
736
 
        self.setup_editor()
 
632
            default_editor = osutils.set_or_unset_env('BZR_EDITOR', "./fed.sh")
 
633
        self.addCleanup(restoreDefaults)
737
634
        msgeditor.hooks.install_named_hook("commit_message_template",
738
635
                lambda commit_obj, msg: "save me some typing\n", None)
739
636
        tree = self.make_branch_and_tree('tree')
740
637
        self.build_tree(['tree/hello.txt'])
741
638
        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
639
        out, err = self.run_bzr("commit tree/hello.txt")
767
640
        last_rev = tree.branch.repository.get_revision(tree.last_revision())
768
641
        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')