← 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: Florent Gallaire
  • Date: 2017-03-17 10:39:02 UTC
  • mto: This revision was merged to the branch mainline in revision 6622.
  • Revision ID: fgallaire@gmail.com-20170317103902-xsmafws9vn8rczx9
Fix for Windows and 32-bit platforms buggy gmtime().

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-2012, 2016 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
30
30
    ignores,
31
31
    msgeditor,
32
32
    )
33
 
from bzrlib.bzrdir import BzrDir
 
33
from bzrlib.controldir import ControlDir
34
34
from bzrlib.tests import (
35
35
    test_foreign,
36
36
    features,
37
37
    )
38
38
from bzrlib.tests import TestCaseWithTransport
 
39
from bzrlib.tests.matchers import ContainsNoVfsCalls
39
40
 
40
41
 
41
42
class TestCommit(TestCaseWithTransport):
116
117
    def prepare_simple_history(self):
117
118
        """Prepare and return a working tree with one commit of one file"""
118
119
        # Commit with modified file should say so
119
 
        wt = BzrDir.create_standalone_workingtree('.')
 
120
        wt = ControlDir.create_standalone_workingtree('.')
120
121
        self.build_tree(['hello.txt', 'extra.txt'])
121
122
        wt.add(['hello.txt'])
122
123
        wt.commit(message='added')
138
139
        self.requireFeature(features.UnicodeFilenameFeature)
139
140
        file_name = u'\N{euro sign}'
140
141
        self.run_bzr(['init'])
141
 
        open(file_name, 'w').write('hello world')
 
142
        with open(file_name, 'w') as f: f.write('hello world')
142
143
        self.run_bzr(['add'])
143
144
        out, err = self.run_bzr(['commit', '-m', file_name])
144
145
        reflags = re.MULTILINE|re.DOTALL|re.UNICODE
156
157
        try:
157
158
            osutils.get_terminal_encoding = lambda trace=None: 'ascii'
158
159
            file_name = u'foo\u1234'
159
 
            open(file_name, 'w').write('hello world')
 
160
            with open(file_name, 'w') as f: f.write('hello world')
160
161
            self.run_bzr(['add'])
161
162
            out, err = self.run_bzr(['commit', '-m', file_name])
162
163
            reflags = re.MULTILINE|re.DOTALL|re.UNICODE
167
168
        finally:
168
169
            osutils.get_terminal_encoding = default_get_terminal_enc
169
170
 
 
171
    def test_non_ascii_file_unversioned_utf8(self):
 
172
        self.requireFeature(features.UnicodeFilenameFeature)
 
173
        tree = self.make_branch_and_tree(".")
 
174
        self.build_tree(["f"])
 
175
        tree.add(["f"])
 
176
        out, err = self.run_bzr(["commit", "-m", "Wrong filename", u"\xa7"],
 
177
            encoding="utf-8", retcode=3)
 
178
        self.assertContainsRe(err, "(?m)not versioned: \"\xc2\xa7\"$")
 
179
 
 
180
    def test_non_ascii_file_unversioned_iso_8859_5(self):
 
181
        self.requireFeature(features.UnicodeFilenameFeature)
 
182
        tree = self.make_branch_and_tree(".")
 
183
        self.build_tree(["f"])
 
184
        tree.add(["f"])
 
185
        out, err = self.run_bzr(["commit", "-m", "Wrong filename", u"\xa7"],
 
186
            encoding="iso-8859-5", retcode=3)
 
187
        self.expectFailure("Error messages are always written as UTF-8",
 
188
            self.assertNotContainsString, err, "\xc2\xa7")
 
189
        self.assertContainsRe(err, "(?m)not versioned: \"\xfd\"$")
 
190
 
170
191
    def test_warn_about_forgotten_commit_message(self):
171
192
        """Test that the lack of -m parameter is caught"""
172
193
        wt = self.make_branch_and_tree('.')
204
225
    def test_verbose_commit_with_unknown(self):
205
226
        """Unknown files should not be listed by default in verbose output"""
206
227
        # Is that really the best policy?
207
 
        wt = BzrDir.create_standalone_workingtree('.')
 
228
        wt = ControlDir.create_standalone_workingtree('.')
208
229
        self.build_tree(['hello.txt', 'extra.txt'])
209
230
        wt.add(['hello.txt'])
210
231
        out,err = self.run_bzr('commit -m added')
307
328
        finally:
308
329
            other_tree.unlock()
309
330
        this_tree.merge_from_branch(other_tree.branch)
310
 
        os.chdir('this')
311
 
        out,err = self.run_bzr('commit -m added')
 
331
        out, err = self.run_bzr('commit -m added', working_dir='this')
312
332
        self.assertEqual('', out)
313
333
        self.assertEqual(set([
314
 
            'Committing to: %s/' % osutils.getcwd(),
 
334
            'Committing to: %s/' % osutils.pathjoin(osutils.getcwd(), 'this'),
315
335
            'modified filetomodify',
316
336
            'added newdir',
317
337
            'added newfile',
451
471
        t.add(['file-a', 'dir-a', 'dir-a/file-b'])
452
472
        t.commit('Create')
453
473
        t.remove(['file-a', 'dir-a/file-b'])
454
 
        os.chdir('dir-a')
455
 
        result = self.run_bzr('commit . -m removed-file-b')[1]
 
474
        result = self.run_bzr('commit . -m removed-file-b',
 
475
                              working_dir='dir-a')[1]
456
476
        self.assertNotContainsRe(result, 'file-a')
457
 
        result = self.run_bzr('status')[0]
 
477
        result = self.run_bzr('status', working_dir='dir-a')[0]
458
478
        self.assertContainsRe(result, 'removed:\n  file-a')
459
479
 
460
480
    def test_strict_commit(self):
464
484
        self.build_tree(['tree/a'])
465
485
        tree.add('a')
466
486
        # A simple change should just work
467
 
        self.run_bzr('commit --strict -m adding-a',
468
 
                     working_dir='tree')
 
487
        self.run_bzr('commit --strict -m adding-a', working_dir='tree')
469
488
 
470
489
    def test_strict_commit_no_changes(self):
471
490
        """commit --strict gives "no changes" if there is nothing to commit"""
609
628
            "Commit refused."],
610
629
            'commit -m add-b --fixes=123',
611
630
            working_dir='tree')
612
 
        tree.branch.get_config().set_user_option("bugtracker", "lp")
 
631
        tree.branch.get_config_stack().set("bugtracker", "lp")
613
632
        self.run_bzr('commit -m hello --fixes=234 tree/hello.txt')
614
633
        last_rev = tree.branch.repository.get_revision(tree.last_revision())
615
634
        self.assertEqual('https://launchpad.net/bugs/234 fixed',
694
713
        self.assertEqual(
695
714
            'Sat 2009-10-10 08:00:00 +0100',
696
715
            osutils.format_date(last_rev.timestamp, last_rev.timezone))
697
 
        
 
716
 
 
717
    def test_commit_time_negative_windows(self):
 
718
        tree = self.make_branch_and_tree('tree')
 
719
        self.build_tree(['tree/hello.txt'])
 
720
        tree.add('hello.txt')
 
721
        out, err = self.run_bzr("commit -m hello "
 
722
            "--commit-time='1969-10-10 00:00:00 +0000' tree/hello.txt")
 
723
        last_rev = tree.branch.repository.get_revision(tree.last_revision())
 
724
        self.assertEqual(
 
725
            'Fri 1969-10-10 00:00:00 +0000',
 
726
            osutils.format_date(last_rev.timestamp, last_rev.timezone))
 
727
 
 
728
    def test_commit_time_negative_32bit(self):
 
729
        tree = self.make_branch_and_tree('tree')
 
730
        self.build_tree(['tree/hello.txt'])
 
731
        tree.add('hello.txt')
 
732
        out, err = self.run_bzr("commit -m hello "
 
733
            "--commit-time='1900-01-01 00:00:00 +0000' tree/hello.txt")
 
734
        last_rev = tree.branch.repository.get_revision(tree.last_revision())
 
735
        self.assertEqual(
 
736
            'Mon 1900-01-01 00:00:00 +0000',
 
737
            osutils.format_date(last_rev.timestamp, last_rev.timezone))
 
738
 
 
739
    def test_commit_time_positive_32bit(self):
 
740
        tree = self.make_branch_and_tree('tree')
 
741
        self.build_tree(['tree/hello.txt'])
 
742
        tree.add('hello.txt')
 
743
        out, err = self.run_bzr("commit -m hello "
 
744
            "--commit-time='2039-01-01 00:00:00 +0000' tree/hello.txt")
 
745
        last_rev = tree.branch.repository.get_revision(tree.last_revision())
 
746
        self.assertEqual(
 
747
            'Sat 2039-01-01 00:00:00 +0000',
 
748
            osutils.format_date(last_rev.timestamp, last_rev.timezone))
 
749
 
698
750
    def test_commit_time_bad_time(self):
699
751
        tree = self.make_branch_and_tree('tree')
700
752
        self.build_tree(['tree/hello.txt'])
704
756
        self.assertStartsWith(
705
757
            err, "bzr: ERROR: Could not parse --commit-time:")
706
758
 
 
759
    def test_commit_time_missing_tz(self):
 
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 -m hello "
 
764
            "--commit-time='2009-10-10 08:00:00' tree/hello.txt", retcode=3)
 
765
        self.assertStartsWith(
 
766
            err, "bzr: ERROR: Could not parse --commit-time:")
 
767
        # Test that it is actually checking and does not simply crash with
 
768
        # some other exception
 
769
        self.assertContainsString(err, "missing a timezone offset")
 
770
 
707
771
    def test_partial_commit_with_renames_in_tree(self):
708
772
        # this test illustrates bug #140419
709
773
        t = self.make_branch_and_tree('.')
718
782
        self.build_tree_contents([('test', 'changes in test')])
719
783
        # partial commit
720
784
        out, err = self.run_bzr('commit test -m "partial commit"')
721
 
        self.assertEquals('', out)
 
785
        self.assertEqual('', out)
722
786
        self.assertContainsRe(err, r'modified test\nCommitted revision 2.')
723
787
 
724
788
    def test_commit_readonly_checkout(self):
726
790
        # "UnlockableTransport error trying to commit in checkout of readonly
727
791
        # branch"
728
792
        self.make_branch('master')
729
 
        master = BzrDir.open_from_transport(
 
793
        master = ControlDir.open_from_transport(
730
794
            self.get_readonly_transport('master')).open_branch()
731
795
        master.create_checkout('checkout')
732
796
        out, err = self.run_bzr(['commit', '--unchanged', '-mfoo', 'checkout'],
797
861
        """Ensure commit error if username is not set.
798
862
        """
799
863
        self.run_bzr(['init', 'foo'])
800
 
        os.chdir('foo')
801
 
        open('foo.txt', 'w').write('hello')
802
 
        self.run_bzr(['add'])
 
864
        with open('foo/foo.txt', 'w') as f:
 
865
            f.write('hello')
 
866
        self.run_bzr(['add'], working_dir='foo')
803
867
        self.overrideEnv('EMAIL', None)
804
868
        self.overrideEnv('BZR_EMAIL', None)
805
869
        # Also, make sure that it's not inferred from mailname.
806
870
        self.overrideAttr(config, '_auto_user_id',
807
871
            lambda: (None, None))
808
 
        out, err = self.run_bzr(['commit', '-m', 'initial'], 3)
809
 
        self.assertContainsRe(err, 'Unable to determine your name')
 
872
        self.run_bzr_error(
 
873
            ['Unable to determine your name'],
 
874
            ['commit', '-m', 'initial'], working_dir='foo')
810
875
 
811
876
    def test_commit_recursive_checkout(self):
812
877
        """Ensure that a commit to a recursive checkout fails cleanly.
813
878
        """
814
879
        self.run_bzr(['init', 'test_branch'])
815
880
        self.run_bzr(['checkout', 'test_branch', 'test_checkout'])
816
 
        os.chdir('test_checkout')
817
 
        self.run_bzr(['bind', '.']) # bind to self
818
 
        open('foo.txt', 'w').write('hello')
819
 
        self.run_bzr(['add'])
820
 
        out, err = self.run_bzr(['commit', '-m', 'addedfoo'], 3)
821
 
        self.assertEqual(out, '')
822
 
        self.assertContainsRe(err,
823
 
            'Branch.*test_checkout.*appears to be bound to itself')
 
881
        self.run_bzr(['bind', '.'], working_dir='test_checkout') # bind to self
 
882
        with open('test_checkout/foo.txt', 'w') as f:
 
883
            f.write('hello')
 
884
        self.run_bzr(['add'], working_dir='test_checkout')
 
885
        out, err = self.run_bzr_error(
 
886
            ['Branch.*test_checkout.*appears to be bound to itself'],
 
887
            ['commit', '-m', 'addedfoo'], working_dir='test_checkout')
 
888
 
 
889
    def test_mv_dirs_non_ascii(self):
 
890
        """Move directory with non-ascii name and containing files.
 
891
 
 
892
        Regression test for bug 185211.
 
893
        """
 
894
        tree = self.make_branch_and_tree('.')
 
895
        self.build_tree([u'abc\xa7/', u'abc\xa7/foo'])
 
896
 
 
897
        tree.add([u'abc\xa7/', u'abc\xa7/foo'])
 
898
        tree.commit('checkin')
 
899
 
 
900
        tree.rename_one(u'abc\xa7','abc')
 
901
 
 
902
        self.run_bzr('ci -m "non-ascii mv"')
 
903
 
 
904
 
 
905
class TestSmartServerCommit(TestCaseWithTransport):
 
906
 
 
907
    def test_commit_to_lightweight(self):
 
908
        self.setup_smart_server_with_call_log()
 
909
        t = self.make_branch_and_tree('from')
 
910
        for count in range(9):
 
911
            t.commit(message='commit %d' % count)
 
912
        out, err = self.run_bzr(['checkout', '--lightweight', self.get_url('from'),
 
913
            'target'])
 
914
        self.reset_smart_call_log()
 
915
        self.build_tree(['target/afile'])
 
916
        self.run_bzr(['add', 'target/afile'])
 
917
        out, err = self.run_bzr(['commit', '-m', 'do something', 'target'])
 
918
        # This figure represent the amount of work to perform this use case. It
 
919
        # is entirely ok to reduce this number if a test fails due to rpc_count
 
920
        # being too low. If rpc_count increases, more network roundtrips have
 
921
        # become necessary for this use case. Please do not adjust this number
 
922
        # upwards without agreement from bzr's network support maintainers.
 
923
        self.assertLength(211, self.hpss_calls)
 
924
        self.assertLength(2, self.hpss_connections)
 
925
        self.expectFailure("commit still uses VFS calls",
 
926
            self.assertThat, self.hpss_calls, ContainsNoVfsCalls)