← 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-08-14 05:49:27 UTC
  • mfrom: (4476.3.86 inventory-delta)
  • Revision ID: pqm@pqm.ubuntu.com-20090814054927-k0k18dn46ax4b91f
(andrew) Add inventory-delta streaming for cross-format fetch.

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, 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
18
18
"""Tests for the commit CLI of bzr."""
19
19
 
20
20
import os
21
 
import re
22
21
import sys
23
22
 
24
23
from bzrlib import (
25
 
    bzrdir,
26
24
    osutils,
27
25
    ignores,
28
26
    msgeditor,
29
27
    osutils,
30
 
    tests,
31
28
    )
32
29
from bzrlib.bzrdir import BzrDir
33
30
from bzrlib.tests import (
34
31
    probe_bad_non_ascii,
35
32
    TestSkipped,
36
33
    )
37
 
from bzrlib.tests import TestCaseWithTransport
38
 
 
39
 
 
40
 
class TestCommit(TestCaseWithTransport):
 
34
from bzrlib.tests.blackbox import ExternalBase
 
35
 
 
36
 
 
37
class TestCommit(ExternalBase):
41
38
 
42
39
    def test_05_empty_commit(self):
43
40
        """Commit of tree with no versioned files should fail"""
109
106
                              'modified hello\.txt\n'
110
107
                              'Committed revision 2\.\n$')
111
108
 
112
 
    def test_unicode_commit_message_is_filename(self):
113
 
        """Unicode commit message same as a filename (Bug #563646).
114
 
        """
115
 
        file_name = u'\N{euro sign}'
116
 
        self.run_bzr(['init'])
117
 
        open(file_name, 'w').write('hello world')
118
 
        self.run_bzr(['add'])
119
 
        out, err = self.run_bzr(['commit', '-m', file_name])
120
 
        reflags = re.MULTILINE|re.DOTALL|re.UNICODE
121
 
        te = osutils.get_terminal_encoding()
122
 
        self.assertContainsRe(err.decode(te),
123
 
            u'The commit message is a file name:',
124
 
            flags=reflags)
125
 
 
126
 
        # Run same test with a filename that causes encode
127
 
        # error for the terminal encoding. We do this
128
 
        # by forcing terminal encoding of ascii for
129
 
        # osutils.get_terminal_encoding which is used
130
 
        # by ui.text.show_warning
131
 
        default_get_terminal_enc = osutils.get_terminal_encoding
132
 
        try:
133
 
            osutils.get_terminal_encoding = lambda trace=None: 'ascii'
134
 
            file_name = u'foo\u1234'
135
 
            open(file_name, 'w').write('hello world')
136
 
            self.run_bzr(['add'])
137
 
            out, err = self.run_bzr(['commit', '-m', file_name])
138
 
            reflags = re.MULTILINE|re.DOTALL|re.UNICODE
139
 
            te = osutils.get_terminal_encoding()
140
 
            self.assertContainsRe(err.decode(te, 'replace'),
141
 
                u'The commit message is a file name:',
142
 
                flags=reflags)
143
 
        finally:
144
 
            osutils.get_terminal_encoding = default_get_terminal_enc
145
 
 
146
 
    def test_warn_about_forgotten_commit_message(self):
147
 
        """Test that the lack of -m parameter is caught"""
148
 
        wt = self.make_branch_and_tree('.')
149
 
        self.build_tree(['one', 'two'])
150
 
        wt.add(['two'])
151
 
        out, err = self.run_bzr('commit -m one two')
152
 
        self.assertContainsRe(err, "The commit message is a file name")
153
 
 
154
109
    def test_verbose_commit_renamed(self):
155
110
        # Verbose commit of renamed file should say so
156
111
        wt = self.prepare_simple_history()
215
170
        self.assertEqual(err, 'Committing to: %s\n'
216
171
                         'Committed revision 2.\n' % expected)
217
172
 
218
 
    def test_commit_sanitizes_CR_in_message(self):
219
 
        # See bug #433779, basically Emacs likes to pass '\r\n' style line
220
 
        # endings to 'bzr commit -m ""' which breaks because we don't allow
221
 
        # '\r' in commit messages. (Mostly because of issues where XML style
222
 
        # formats arbitrarily strip it out of the data while parsing.)
223
 
        # To make life easier for users, we just always translate '\r\n' =>
224
 
        # '\n'. And '\r' => '\n'.
225
 
        a_tree = self.make_branch_and_tree('a')
226
 
        self.build_tree(['a/b'])
227
 
        a_tree.add('b')
228
 
        self.run_bzr(['commit',
229
 
                      '-m', 'a string\r\n\r\nwith mixed\r\rendings\n'],
230
 
                     working_dir='a')
231
 
        rev_id = a_tree.branch.last_revision()
232
 
        rev = a_tree.branch.repository.get_revision(rev_id)
233
 
        self.assertEqualDiff('a string\n\nwith mixed\n\nendings\n',
234
 
                             rev.message)
235
 
 
236
173
    def test_commit_merge_reports_all_modified_files(self):
237
174
        # the commit command should show all the files that are shown by
238
175
        # bzr diff or bzr status when committing, even when they were not
308
245
        self.run_bzr('commit -m ""', retcode=3)
309
246
 
310
247
    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
248
        tree = self.make_branch_and_tree('.')
315
249
        self.build_tree_contents([('foo.c', 'int main() {}')])
316
250
        tree.add('foo.c')
321
255
        if char is None:
322
256
            raise TestSkipped('Cannot find suitable non-ascii character'
323
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.
324
262
        out,err = self.run_bzr_subprocess('commit -m "%s"' % char,
325
263
                                          retcode=1,
326
264
                                          env_changes={'LANG': 'C'})
335
273
        self.build_tree_contents([
336
274
            ('branch/foo.c', 'int main() {}'),
337
275
            ('branch/bar.c', 'int main() {}')])
338
 
        inner_tree.add(['foo.c', 'bar.c'])
 
276
        inner_tree.add('foo.c')
 
277
        inner_tree.add('bar.c')
339
278
        # can't commit files in different trees; sane error
340
279
        self.run_bzr('commit -m newstuff branch/foo.c .', retcode=3)
341
 
        # can commit to branch - records foo.c only
342
280
        self.run_bzr('commit -m newstuff branch/foo.c')
343
 
        # can commit to branch - records bar.c
344
281
        self.run_bzr('commit -m newstuff branch')
345
 
        # No changes left
346
 
        self.run_bzr_error(["No changes to commit"], 'commit -m newstuff branch')
 
282
        self.run_bzr('commit -m newstuff branch', retcode=3)
347
283
 
348
284
    def test_out_of_date_tree_commit(self):
349
285
        # check we get an error code and a clear message committing with an out
379
315
        trunk = self.make_branch_and_tree('trunk')
380
316
 
381
317
        u1 = trunk.branch.create_checkout('u1')
382
 
        self.build_tree_contents([('u1/hosts', 'initial contents\n')])
 
318
        self.build_tree_contents([('u1/hosts', 'initial contents')])
383
319
        u1.add('hosts')
384
320
        self.run_bzr('commit -m add-hosts u1')
385
321
 
386
322
        u2 = trunk.branch.create_checkout('u2')
387
 
        self.build_tree_contents([('u2/hosts', 'altered in u2\n')])
 
323
        self.build_tree_contents([('u2/hosts', 'altered in u2')])
388
324
        self.run_bzr('commit -m checkin-from-u2 u2')
389
325
 
390
326
        # make an offline commits
391
 
        self.build_tree_contents([('u1/hosts', 'first offline change in u1\n')])
 
327
        self.build_tree_contents([('u1/hosts', 'first offline change in u1')])
392
328
        self.run_bzr('commit -m checkin-offline --local u1')
393
329
 
394
330
        # now try to pull in online work from u2, and then commit our offline
395
331
        # work as a merge
396
332
        # retcode 1 as we expect a text conflict
397
333
        self.run_bzr('update u1', retcode=1)
398
 
        self.assertFileEqual('''\
399
 
<<<<<<< TREE
400
 
first offline change in u1
401
 
=======
402
 
altered in u2
403
 
>>>>>>> MERGE-SOURCE
404
 
''',
405
 
                             'u1/hosts')
406
 
 
407
334
        self.run_bzr('resolved u1/hosts')
408
335
        # add a text change here to represent resolving the merge conflicts in
409
336
        # favour of a new version of the file not identical to either the u1
661
588
        properties = last_rev.properties
662
589
        self.assertEqual('John Doe\nJane Rey', properties['authors'])
663
590
 
664
 
    def test_commit_time(self):
665
 
        tree = self.make_branch_and_tree('tree')
666
 
        self.build_tree(['tree/hello.txt'])
667
 
        tree.add('hello.txt')
668
 
        out, err = self.run_bzr("commit -m hello "
669
 
            "--commit-time='2009-10-10 08:00:00 +0100' tree/hello.txt")
670
 
        last_rev = tree.branch.repository.get_revision(tree.last_revision())
671
 
        self.assertEqual(
672
 
            'Sat 2009-10-10 08:00:00 +0100',
673
 
            osutils.format_date(last_rev.timestamp, last_rev.timezone))
674
 
        
675
 
    def test_commit_time_bad_time(self):
676
 
        tree = self.make_branch_and_tree('tree')
677
 
        self.build_tree(['tree/hello.txt'])
678
 
        tree.add('hello.txt')
679
 
        out, err = self.run_bzr("commit -m hello "
680
 
            "--commit-time='NOT A TIME' tree/hello.txt", retcode=3)
681
 
        self.assertStartsWith(
682
 
            err, "bzr: ERROR: Could not parse --commit-time:")
683
 
 
684
591
    def test_partial_commit_with_renames_in_tree(self):
685
592
        # this test illustrates bug #140419
686
593
        t = self.make_branch_and_tree('.')
699
606
        self.assertContainsRe(err, r'modified test\nCommitted revision 2.')
700
607
 
701
608
    def test_commit_readonly_checkout(self):
702
 
        # https://bugs.launchpad.net/bzr/+bug/129701
 
609
        # https://bugs.edge.launchpad.net/bzr/+bug/129701
703
610
        # "UnlockableTransport error trying to commit in checkout of readonly
704
611
        # branch"
705
612
        self.make_branch('master')
711
618
        self.assertContainsRe(err,
712
619
            r'^bzr: ERROR: Cannot lock.*readonly transport')
713
620
 
714
 
    def setup_editor(self):
 
621
    def test_commit_hook_template(self):
715
622
        # Test that commit template hooks work
 
623
        def restoreDefaults():
 
624
            msgeditor.hooks['commit_message_template'] = []
 
625
            osutils.set_or_unset_env('BZR_EDITOR', default_editor)
716
626
        if sys.platform == "win32":
717
627
            f = file('fed.bat', 'w')
718
628
            f.write('@rem dummy fed')
719
629
            f.close()
720
 
            osutils.set_or_unset_env('BZR_EDITOR', "fed.bat")
 
630
            default_editor = osutils.set_or_unset_env('BZR_EDITOR', "fed.bat")
721
631
        else:
722
632
            f = file('fed.sh', 'wb')
723
633
            f.write('#!/bin/sh\n')
724
634
            f.close()
725
635
            os.chmod('fed.sh', 0755)
726
 
            osutils.set_or_unset_env('BZR_EDITOR', "./fed.sh")
727
 
 
728
 
    def setup_commit_with_template(self):
729
 
        self.setup_editor()
 
636
            default_editor = osutils.set_or_unset_env('BZR_EDITOR', "./fed.sh")
 
637
        self.addCleanup(restoreDefaults)
730
638
        msgeditor.hooks.install_named_hook("commit_message_template",
731
639
                lambda commit_obj, msg: "save me some typing\n", None)
732
640
        tree = self.make_branch_and_tree('tree')
733
641
        self.build_tree(['tree/hello.txt'])
734
642
        tree.add('hello.txt')
735
 
        return tree
736
 
 
737
 
    def test_commit_hook_template_accepted(self):
738
 
        tree = self.setup_commit_with_template()
739
 
        out, err = self.run_bzr("commit tree/hello.txt", stdin="y\n")
 
643
        out, err = self.run_bzr("commit tree/hello.txt")
740
644
        last_rev = tree.branch.repository.get_revision(tree.last_revision())
741
645
        self.assertEqual('save me some typing\n', last_rev.message)
742
 
 
743
 
    def test_commit_hook_template_rejected(self):
744
 
        tree = self.setup_commit_with_template()
745
 
        expected = tree.last_revision()
746
 
        out, err = self.run_bzr_error(["empty commit message"],
747
 
            "commit tree/hello.txt", stdin="n\n")
748
 
        self.assertEqual(expected, tree.last_revision())
749
 
 
750
 
    def test_commit_without_username(self):
751
 
        """Ensure commit error if username is not set.
752
 
        """
753
 
        self.run_bzr(['init', 'foo'])
754
 
        os.chdir('foo')
755
 
        open('foo.txt', 'w').write('hello')
756
 
        self.run_bzr(['add'])
757
 
        osutils.set_or_unset_env('EMAIL', None)
758
 
        osutils.set_or_unset_env('BZR_EMAIL', None)
759
 
        out, err = self.run_bzr(['commit', '-m', 'initial'], 3)
760
 
        self.assertContainsRe(err, 'Unable to determine your name')
761
 
 
762
 
    def test_commit_recursive_checkout(self):
763
 
        """Ensure that a commit to a recursive checkout fails cleanly.
764
 
        """
765
 
        self.run_bzr(['init', 'test_branch'])
766
 
        self.run_bzr(['checkout', 'test_branch', 'test_checkout'])
767
 
        os.chdir('test_checkout')
768
 
        self.run_bzr(['bind', '.']) # bind to self
769
 
        open('foo.txt', 'w').write('hello')
770
 
        self.run_bzr(['add'])
771
 
        out, err = self.run_bzr(['commit', '-m', 'addedfoo'], 3)
772
 
        self.assertEqual(out, '')
773
 
        self.assertContainsRe(err,
774
 
            'Branch.*test_checkout.*appears to be bound to itself')
775