← Back to branch summary

~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/tests/test_commit.py

  • Committer: Canonical.com Patch Queue Manager
  • Date: 2007-08-09 15:19:06 UTC
  • mfrom: (2681.1.7 send-bundle)
  • Revision ID: pqm@pqm.ubuntu.com-20070809151906-hdn9oyslf2qib2op
Allow omitting -o for bundle, add --format

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/tests/test_commit.py
1
 
# Copyright (C) 2005, 2006, 2008 Canonical Ltd
 
1
# Copyright (C) 2005, 2006 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
12
12
#
13
13
# You should have received a copy of the GNU General Public License
14
14
# along with this program; if not, write to the Free Software
15
 
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
 
15
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
16
16
 
17
17
 
18
18
import os
19
19
 
20
20
import bzrlib
21
21
from bzrlib import (
22
 
    bzrdir,
23
22
    errors,
24
23
    lockdir,
25
24
    osutils,
29
28
from bzrlib.bzrdir import BzrDir, BzrDirMetaFormat1
30
29
from bzrlib.commit import Commit, NullCommitReporter
31
30
from bzrlib.config import BranchConfig
32
 
from bzrlib.errors import (PointlessCommit, BzrError, SigningFailed,
 
31
from bzrlib.errors import (PointlessCommit, BzrError, SigningFailed, 
33
32
                           LockContention)
34
 
from bzrlib.tests import SymlinkFeature, TestCaseWithTransport
 
33
from bzrlib.tests import TestCaseWithTransport
35
34
from bzrlib.workingtree import WorkingTree
36
35
 
37
36
 
71
70
    def renamed(self, change, old_path, new_path):
72
71
        self.calls.append(('renamed', change, old_path, new_path))
73
72
 
74
 
    def is_verbose(self):
75
 
        return True
76
 
 
77
73
 
78
74
class TestCommit(TestCaseWithTransport):
79
75
 
96
92
        eq(rev.message, 'add hello')
97
93
 
98
94
        tree1 = b.repository.revision_tree(rh[0])
99
 
        tree1.lock_read()
100
95
        text = tree1.get_file_text(file_id)
101
 
        tree1.unlock()
102
 
        self.assertEqual('hello world', text)
 
96
        eq(text, 'hello world')
103
97
 
104
98
        tree2 = b.repository.revision_tree(rh[1])
105
 
        tree2.lock_read()
106
 
        text = tree2.get_file_text(file_id)
107
 
        tree2.unlock()
108
 
        self.assertEqual('version 2', text)
 
99
        eq(tree2.get_file_text(file_id), 'version 2')
109
100
 
110
 
    def test_missing_commit(self):
111
 
        """Test a commit with a missing file"""
 
101
    def test_delete_commit(self):
 
102
        """Test a commit with a deleted file"""
112
103
        wt = self.make_branch_and_tree('.')
113
104
        b = wt.branch
114
105
        file('hello', 'w').write('hello world')
121
112
        tree = b.repository.revision_tree('rev2')
122
113
        self.assertFalse(tree.has_id('hello-id'))
123
114
 
124
 
    def test_partial_commit_move(self):
125
 
        """Test a partial commit where a file was renamed but not committed.
126
 
 
127
 
        https://bugs.launchpad.net/bzr/+bug/83039
128
 
 
129
 
        If not handled properly, commit will try to snapshot
130
 
        dialog.py with olive/ as a parent, while
131
 
        olive/ has not been snapshotted yet.
132
 
        """
133
 
        wt = self.make_branch_and_tree('.')
134
 
        b = wt.branch
135
 
        self.build_tree(['annotate/', 'annotate/foo.py',
136
 
                         'olive/', 'olive/dialog.py'
137
 
                        ])
138
 
        wt.add(['annotate', 'olive', 'annotate/foo.py', 'olive/dialog.py'])
139
 
        wt.commit(message='add files')
140
 
        wt.rename_one("olive/dialog.py", "aaa")
141
 
        self.build_tree_contents([('annotate/foo.py', 'modified\n')])
142
 
        wt.commit('renamed hello', specific_files=["annotate"])
143
 
 
144
115
    def test_pointless_commit(self):
145
116
        """Commit refuses unless there are changes or it's forced."""
146
117
        wt = self.make_branch_and_tree('.')
154
125
                          message='fails',
155
126
                          allow_pointless=False)
156
127
        self.assertEquals(b.revno(), 1)
157
 
 
 
128
        
158
129
    def test_commit_empty(self):
159
130
        """Commiting an empty tree works."""
160
131
        wt = self.make_branch_and_tree('.')
177
148
              ['hello-id', 'buongia-id'])
178
149
        wt.commit(message='add files',
179
150
                 rev_id='test@rev-1')
180
 
 
 
151
        
181
152
        os.remove('hello')
182
153
        file('buongia', 'w').write('new text')
183
154
        wt.commit(message='update text',
194
165
        eq(b.revno(), 3)
195
166
 
196
167
        tree2 = b.repository.revision_tree('test@rev-2')
197
 
        tree2.lock_read()
198
 
        self.addCleanup(tree2.unlock)
199
168
        self.assertTrue(tree2.has_filename('hello'))
200
169
        self.assertEquals(tree2.get_file_text('hello-id'), 'hello')
201
170
        self.assertEquals(tree2.get_file_text('buongia-id'), 'new text')
202
 
 
 
171
        
203
172
        tree3 = b.repository.revision_tree('test@rev-3')
204
 
        tree3.lock_read()
205
 
        self.addCleanup(tree3.unlock)
206
173
        self.assertFalse(tree3.has_filename('hello'))
207
174
        self.assertEquals(tree3.get_file_text('buongia-id'), 'new text')
208
175
 
219
186
 
220
187
        eq = self.assertEquals
221
188
        tree1 = b.repository.revision_tree('test@rev-1')
222
 
        tree1.lock_read()
223
 
        self.addCleanup(tree1.unlock)
224
189
        eq(tree1.id2path('hello-id'), 'hello')
225
190
        eq(tree1.get_file_text('hello-id'), 'contents of hello\n')
226
191
        self.assertFalse(tree1.has_filename('fruity'))
229
194
        eq(ie.revision, 'test@rev-1')
230
195
 
231
196
        tree2 = b.repository.revision_tree('test@rev-2')
232
 
        tree2.lock_read()
233
 
        self.addCleanup(tree2.unlock)
234
197
        eq(tree2.id2path('hello-id'), 'fruity')
235
198
        eq(tree2.get_file_text('hello-id'), 'contents of hello\n')
236
199
        self.check_inventory_shape(tree2.inventory, ['fruity'])
464
427
        bound = master.sprout('bound')
465
428
        wt = bound.open_workingtree()
466
429
        wt.branch.set_bound_location(os.path.realpath('master'))
 
430
 
 
431
        orig_default = lockdir._DEFAULT_TIMEOUT_SECONDS
467
432
        master_branch.lock_write()
468
433
        try:
 
434
            lockdir._DEFAULT_TIMEOUT_SECONDS = 1
469
435
            self.assertRaises(LockContention, wt.commit, 'silly')
470
436
        finally:
 
437
            lockdir._DEFAULT_TIMEOUT_SECONDS = orig_default
471
438
            master_branch.unlock()
472
439
 
473
440
    def test_commit_bound_merge(self):
501
468
        bound_tree.commit(message='commit of merge in bound tree')
502
469
 
503
470
    def test_commit_reporting_after_merge(self):
504
 
        # when doing a commit of a merge, the reporter needs to still
 
471
        # when doing a commit of a merge, the reporter needs to still 
505
472
        # be called for each item that is added/removed/deleted.
506
473
        this_tree = self.make_branch_and_tree('this')
507
474
        # we need a bunch of files and dirs, to perform one action on each.
550
517
        this_tree.merge_from_branch(other_tree.branch)
551
518
        reporter = CapturingReporter()
552
519
        this_tree.commit('do the commit', reporter=reporter)
553
 
        expected = set([
 
520
        self.assertEqual([
 
521
            ('change', 'unchanged', ''),
 
522
            ('change', 'unchanged', 'dirtoleave'),
 
523
            ('change', 'unchanged', 'filetoleave'),
554
524
            ('change', 'modified', 'filetomodify'),
555
525
            ('change', 'added', 'newdir'),
556
526
            ('change', 'added', 'newfile'),
557
527
            ('renamed', 'renamed', 'dirtorename', 'renameddir'),
558
 
            ('renamed', 'renamed', 'filetorename', 'renamedfile'),
559
528
            ('renamed', 'renamed', 'dirtoreparent', 'renameddir/reparenteddir'),
560
529
            ('renamed', 'renamed', 'filetoreparent', 'renameddir/reparentedfile'),
 
530
            ('renamed', 'renamed', 'filetorename', 'renamedfile'),
561
531
            ('deleted', 'dirtoremove'),
562
532
            ('deleted', 'filetoremove'),
563
 
            ])
564
 
        result = set(reporter.calls)
565
 
        missing = expected - result
566
 
        new = result - expected
567
 
        self.assertEqual((set(), set()), (missing, new))
 
533
            ],
 
534
            reporter.calls)
568
535
 
569
536
    def test_commit_removals_respects_filespec(self):
570
537
        """Commit respects the specified_files for removals."""
614
581
            basis.unlock()
615
582
 
616
583
    def test_commit_kind_changes(self):
617
 
        self.requireFeature(SymlinkFeature)
 
584
        if not osutils.has_symlinks():
 
585
            raise tests.TestSkipped('Test requires symlink support')
618
586
        tree = self.make_branch_and_tree('.')
619
587
        os.symlink('target', 'name')
620
588
        tree.add('name', 'a-file-id')
660
628
    def test_commit_unversioned_specified(self):
661
629
        """Commit should raise if specified files isn't in basis or worktree"""
662
630
        tree = self.make_branch_and_tree('.')
663
 
        self.assertRaises(errors.PathsNotVersionedError, tree.commit,
 
631
        self.assertRaises(errors.PathsNotVersionedError, tree.commit, 
664
632
                          'message', specific_files=['bogus'])
665
633
 
666
634
    class Callback(object):
667
 
 
 
635
        
668
636
        def __init__(self, message, testcase):
669
637
            self.called = False
670
638
            self.message = message
698
666
        """Callback should not be invoked for pointless commit"""
699
667
        tree = self.make_branch_and_tree('.')
700
668
        cb = self.Callback(u'commit 2', self)
701
 
        self.assertRaises(PointlessCommit, tree.commit, message_callback=cb,
 
669
        self.assertRaises(PointlessCommit, tree.commit, message_callback=cb, 
702
670
                          allow_pointless=False)
703
671
        self.assertFalse(cb.called)
704
672
 
727
695
        self.assertEqual(['bar', 'baz'], err.files)
728
696
        self.assertEqual('Selected-file commit of merges is not supported'
729
697
                         ' yet: files bar, baz', str(err))
730
 
 
731
 
    def test_commit_ordering(self):
732
 
        """Test of corner-case commit ordering error"""
733
 
        tree = self.make_branch_and_tree('.')
734
 
        self.build_tree(['a/', 'a/z/', 'a/c/', 'a/z/x', 'a/z/y'])
735
 
        tree.add(['a/', 'a/z/', 'a/c/', 'a/z/x', 'a/z/y'])
736
 
        tree.commit('setup')
737
 
        self.build_tree(['a/c/d/'])
738
 
        tree.add('a/c/d')
739
 
        tree.rename_one('a/z/x', 'a/c/d/x')
740
 
        tree.commit('test', specific_files=['a/z/y'])
741
 
 
742
 
    def test_commit_no_author(self):
743
 
        """The default kwarg author in MutableTree.commit should not add
744
 
        the 'author' revision property.
745
 
        """
746
 
        tree = self.make_branch_and_tree('foo')
747
 
        rev_id = tree.commit('commit 1')
748
 
        rev = tree.branch.repository.get_revision(rev_id)
749
 
        self.assertFalse('author' in rev.properties)
750
 
        self.assertFalse('authors' in rev.properties)
751
 
 
752
 
    def test_commit_author(self):
753
 
        """Passing a non-empty author kwarg to MutableTree.commit should add
754
 
        the 'author' revision property.
755
 
        """
756
 
        tree = self.make_branch_and_tree('foo')
757
 
        rev_id = self.callDeprecated(['The parameter author was '
758
 
                'deprecated in version 1.13. Use authors instead'],
759
 
                tree.commit, 'commit 1', author='John Doe <jdoe@example.com>')
760
 
        rev = tree.branch.repository.get_revision(rev_id)
761
 
        self.assertEqual('John Doe <jdoe@example.com>',
762
 
                         rev.properties['authors'])
763
 
        self.assertFalse('author' in rev.properties)
764
 
 
765
 
    def test_commit_empty_authors_list(self):
766
 
        """Passing an empty list to authors shouldn't add the property."""
767
 
        tree = self.make_branch_and_tree('foo')
768
 
        rev_id = tree.commit('commit 1', authors=[])
769
 
        rev = tree.branch.repository.get_revision(rev_id)
770
 
        self.assertFalse('author' in rev.properties)
771
 
        self.assertFalse('authors' in rev.properties)
772
 
 
773
 
    def test_multiple_authors(self):
774
 
        tree = self.make_branch_and_tree('foo')
775
 
        rev_id = tree.commit('commit 1',
776
 
                authors=['John Doe <jdoe@example.com>',
777
 
                         'Jane Rey <jrey@example.com>'])
778
 
        rev = tree.branch.repository.get_revision(rev_id)
779
 
        self.assertEqual('John Doe <jdoe@example.com>\n'
780
 
                'Jane Rey <jrey@example.com>', rev.properties['authors'])
781
 
        self.assertFalse('author' in rev.properties)
782
 
 
783
 
    def test_author_and_authors_incompatible(self):
784
 
        tree = self.make_branch_and_tree('foo')
785
 
        self.assertRaises(AssertionError, tree.commit, 'commit 1',
786
 
                authors=['John Doe <jdoe@example.com>',
787
 
                         'Jane Rey <jrey@example.com>'],
788
 
                author="Jack Me <jme@example.com>")
789
 
 
790
 
    def test_author_with_newline_rejected(self):
791
 
        tree = self.make_branch_and_tree('foo')
792
 
        self.assertRaises(AssertionError, tree.commit, 'commit 1',
793
 
                authors=['John\nDoe <jdoe@example.com>'])
794
 
 
795
 
    def test_commit_with_checkout_and_branch_sharing_repo(self):
796
 
        repo = self.make_repository('repo', shared=True)
797
 
        # make_branch_and_tree ignores shared repos
798
 
        branch = bzrdir.BzrDir.create_branch_convenience('repo/branch')
799
 
        tree2 = branch.create_checkout('repo/tree2')
800
 
        tree2.commit('message', rev_id='rev1')
801
 
        self.assertTrue(tree2.branch.repository.has_revision('rev1'))