/brz/remove-bazaar

To get this branch, use:
bzr branch http://gegoxaren.bato24.eu/bzr/brz/remove-bazaar

« back to all changes in this revision

Viewing changes to bzrlib/tests/test_commit.py

First attempt to merge .dev and resolve the conflicts (but tests are 
failing)

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/tests/test_commit.py
1
 
# Copyright (C) 2005, 2006 by Canonical Ltd
 
1
# Copyright (C) 2005, 2006, 2008 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
import os
19
19
 
20
20
import bzrlib
21
 
from bzrlib.tests import TestCaseWithTransport
 
21
from bzrlib import (
 
22
    bzrdir,
 
23
    errors,
 
24
    lockdir,
 
25
    osutils,
 
26
    tests,
 
27
    )
22
28
from bzrlib.branch import Branch
23
29
from bzrlib.bzrdir import BzrDir, BzrDirMetaFormat1
24
 
from bzrlib.workingtree import WorkingTree
25
30
from bzrlib.commit import Commit, NullCommitReporter
26
31
from bzrlib.config import BranchConfig
27
32
from bzrlib.errors import (PointlessCommit, BzrError, SigningFailed, 
28
33
                           LockContention)
 
34
from bzrlib.tests import SymlinkFeature, TestCaseWithTransport
 
35
from bzrlib.workingtree import WorkingTree
29
36
 
30
37
 
31
38
# TODO: Test commit with some added, and added-but-missing files
64
71
    def renamed(self, change, old_path, new_path):
65
72
        self.calls.append(('renamed', change, old_path, new_path))
66
73
 
 
74
    def is_verbose(self):
 
75
        return True
 
76
 
67
77
 
68
78
class TestCommit(TestCaseWithTransport):
69
79
 
86
96
        eq(rev.message, 'add hello')
87
97
 
88
98
        tree1 = b.repository.revision_tree(rh[0])
 
99
        tree1.lock_read()
89
100
        text = tree1.get_file_text(file_id)
90
 
        eq(text, 'hello world')
 
101
        tree1.unlock()
 
102
        self.assertEqual('hello world', text)
91
103
 
92
104
        tree2 = b.repository.revision_tree(rh[1])
93
 
        eq(tree2.get_file_text(file_id), 'version 2')
 
105
        tree2.lock_read()
 
106
        text = tree2.get_file_text(file_id)
 
107
        tree2.unlock()
 
108
        self.assertEqual('version 2', text)
94
109
 
95
110
    def test_delete_commit(self):
96
111
        """Test a commit with a deleted file"""
106
121
        tree = b.repository.revision_tree('rev2')
107
122
        self.assertFalse(tree.has_id('hello-id'))
108
123
 
 
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
 
109
144
    def test_pointless_commit(self):
110
145
        """Commit refuses unless there are changes or it's forced."""
111
146
        wt = self.make_branch_and_tree('.')
159
194
        eq(b.revno(), 3)
160
195
 
161
196
        tree2 = b.repository.revision_tree('test@rev-2')
 
197
        tree2.lock_read()
 
198
        self.addCleanup(tree2.unlock)
162
199
        self.assertTrue(tree2.has_filename('hello'))
163
200
        self.assertEquals(tree2.get_file_text('hello-id'), 'hello')
164
201
        self.assertEquals(tree2.get_file_text('buongia-id'), 'new text')
165
202
        
166
203
        tree3 = b.repository.revision_tree('test@rev-3')
 
204
        tree3.lock_read()
 
205
        self.addCleanup(tree3.unlock)
167
206
        self.assertFalse(tree3.has_filename('hello'))
168
207
        self.assertEquals(tree3.get_file_text('buongia-id'), 'new text')
169
208
 
180
219
 
181
220
        eq = self.assertEquals
182
221
        tree1 = b.repository.revision_tree('test@rev-1')
 
222
        tree1.lock_read()
 
223
        self.addCleanup(tree1.unlock)
183
224
        eq(tree1.id2path('hello-id'), 'hello')
184
225
        eq(tree1.get_file_text('hello-id'), 'contents of hello\n')
185
226
        self.assertFalse(tree1.has_filename('fruity'))
188
229
        eq(ie.revision, 'test@rev-1')
189
230
 
190
231
        tree2 = b.repository.revision_tree('test@rev-2')
 
232
        tree2.lock_read()
 
233
        self.addCleanup(tree2.unlock)
191
234
        eq(tree2.id2path('hello-id'), 'fruity')
192
235
        eq(tree2.get_file_text('hello-id'), 'contents of hello\n')
193
236
        self.check_inventory_shape(tree2.inventory, ['fruity'])
217
260
        wt.move(['hello'], 'a')
218
261
        r2 = 'test@rev-2'
219
262
        wt.commit('two', rev_id=r2, allow_pointless=False)
220
 
        self.check_inventory_shape(wt.read_working_inventory(),
221
 
                                   ['a', 'a/hello', 'b'])
 
263
        wt.lock_read()
 
264
        try:
 
265
            self.check_inventory_shape(wt.read_working_inventory(),
 
266
                                       ['a/', 'a/hello', 'b/'])
 
267
        finally:
 
268
            wt.unlock()
222
269
 
223
270
        wt.move(['b'], 'a')
224
271
        r3 = 'test@rev-3'
225
272
        wt.commit('three', rev_id=r3, allow_pointless=False)
226
 
        self.check_inventory_shape(wt.read_working_inventory(),
227
 
                                   ['a', 'a/hello', 'a/b'])
228
 
        self.check_inventory_shape(b.repository.get_revision_inventory(r3),
229
 
                                   ['a', 'a/hello', 'a/b'])
 
273
        wt.lock_read()
 
274
        try:
 
275
            self.check_inventory_shape(wt.read_working_inventory(),
 
276
                                       ['a/', 'a/hello', 'a/b/'])
 
277
            self.check_inventory_shape(b.repository.get_revision_inventory(r3),
 
278
                                       ['a/', 'a/hello', 'a/b/'])
 
279
        finally:
 
280
            wt.unlock()
230
281
 
231
282
        wt.move(['a/hello'], 'a/b')
232
283
        r4 = 'test@rev-4'
233
284
        wt.commit('four', rev_id=r4, allow_pointless=False)
234
 
        self.check_inventory_shape(wt.read_working_inventory(),
235
 
                                   ['a', 'a/b/hello', 'a/b'])
 
285
        wt.lock_read()
 
286
        try:
 
287
            self.check_inventory_shape(wt.read_working_inventory(),
 
288
                                       ['a/', 'a/b/hello', 'a/b/'])
 
289
        finally:
 
290
            wt.unlock()
236
291
 
237
292
        inv = b.repository.get_revision_inventory(r4)
238
293
        eq(inv['hello-id'].revision, r4)
239
294
        eq(inv['a-id'].revision, r1)
240
295
        eq(inv['b-id'].revision, r3)
241
 
        
 
296
 
242
297
    def test_removed_commit(self):
243
298
        """Commit with a removed file"""
244
299
        wt = self.make_branch_and_tree('.')
339
394
                                                      allow_pointless=True,
340
395
                                                      rev_id='B',
341
396
                                                      working_tree=wt)
342
 
            self.assertEqual(Testament.from_revision(branch.repository,
343
 
                             'B').as_short_text(),
 
397
            def sign(text):
 
398
                return bzrlib.gpg.LoopbackGPGStrategy(None).sign(text)
 
399
            self.assertEqual(sign(Testament.from_revision(branch.repository,
 
400
                             'B').as_short_text()),
344
401
                             branch.repository.get_signature_text('B'))
345
402
        finally:
346
403
            bzrlib.gpg.GPGStrategy = oldstrategy
501
558
            ('change', 'added', 'newdir'),
502
559
            ('change', 'added', 'newfile'),
503
560
            ('renamed', 'renamed', 'dirtorename', 'renameddir'),
 
561
            ('renamed', 'renamed', 'filetorename', 'renamedfile'),
504
562
            ('renamed', 'renamed', 'dirtoreparent', 'renameddir/reparenteddir'),
505
563
            ('renamed', 'renamed', 'filetoreparent', 'renameddir/reparentedfile'),
506
 
            ('renamed', 'renamed', 'filetorename', 'renamedfile'),
507
564
            ('deleted', 'dirtoremove'),
508
565
            ('deleted', 'filetoremove'),
509
566
            ],
517
574
        tree.commit('added a, b')
518
575
        tree.remove(['a', 'b'])
519
576
        tree.commit('removed a', specific_files='a')
520
 
        basis = tree.basis_tree().inventory
521
 
        self.assertIs(None, basis.path2id('a'))
522
 
        self.assertFalse(basis.path2id('b') is None)
 
577
        basis = tree.basis_tree()
 
578
        tree.lock_read()
 
579
        try:
 
580
            self.assertIs(None, basis.path2id('a'))
 
581
            self.assertFalse(basis.path2id('b') is None)
 
582
        finally:
 
583
            tree.unlock()
523
584
 
524
585
    def test_commit_saves_1ms_timestamp(self):
525
586
        """Passing in a timestamp is saved with 1ms resolution"""
543
604
        timestamp = rev.timestamp
544
605
        timestamp_1ms = round(timestamp, 3)
545
606
        self.assertEqual(timestamp_1ms, timestamp)
 
607
 
 
608
    def assertBasisTreeKind(self, kind, tree, file_id):
 
609
        basis = tree.basis_tree()
 
610
        basis.lock_read()
 
611
        try:
 
612
            self.assertEqual(kind, basis.kind(file_id))
 
613
        finally:
 
614
            basis.unlock()
 
615
 
 
616
    def test_commit_kind_changes(self):
 
617
        self.requireFeature(SymlinkFeature)
 
618
        tree = self.make_branch_and_tree('.')
 
619
        os.symlink('target', 'name')
 
620
        tree.add('name', 'a-file-id')
 
621
        tree.commit('Added a symlink')
 
622
        self.assertBasisTreeKind('symlink', tree, 'a-file-id')
 
623
 
 
624
        os.unlink('name')
 
625
        self.build_tree(['name'])
 
626
        tree.commit('Changed symlink to file')
 
627
        self.assertBasisTreeKind('file', tree, 'a-file-id')
 
628
 
 
629
        os.unlink('name')
 
630
        os.symlink('target', 'name')
 
631
        tree.commit('file to symlink')
 
632
        self.assertBasisTreeKind('symlink', tree, 'a-file-id')
 
633
 
 
634
        os.unlink('name')
 
635
        os.mkdir('name')
 
636
        tree.commit('symlink to directory')
 
637
        self.assertBasisTreeKind('directory', tree, 'a-file-id')
 
638
 
 
639
        os.rmdir('name')
 
640
        os.symlink('target', 'name')
 
641
        tree.commit('directory to symlink')
 
642
        self.assertBasisTreeKind('symlink', tree, 'a-file-id')
 
643
 
 
644
        # prepare for directory <-> file tests
 
645
        os.unlink('name')
 
646
        os.mkdir('name')
 
647
        tree.commit('symlink to directory')
 
648
        self.assertBasisTreeKind('directory', tree, 'a-file-id')
 
649
 
 
650
        os.rmdir('name')
 
651
        self.build_tree(['name'])
 
652
        tree.commit('Changed directory to file')
 
653
        self.assertBasisTreeKind('file', tree, 'a-file-id')
 
654
 
 
655
        os.unlink('name')
 
656
        os.mkdir('name')
 
657
        tree.commit('file to directory')
 
658
        self.assertBasisTreeKind('directory', tree, 'a-file-id')
 
659
 
 
660
    def test_commit_unversioned_specified(self):
 
661
        """Commit should raise if specified files isn't in basis or worktree"""
 
662
        tree = self.make_branch_and_tree('.')
 
663
        self.assertRaises(errors.PathsNotVersionedError, tree.commit, 
 
664
                          'message', specific_files=['bogus'])
 
665
 
 
666
    class Callback(object):
 
667
        
 
668
        def __init__(self, message, testcase):
 
669
            self.called = False
 
670
            self.message = message
 
671
            self.testcase = testcase
 
672
 
 
673
        def __call__(self, commit_obj):
 
674
            self.called = True
 
675
            self.testcase.assertTrue(isinstance(commit_obj, Commit))
 
676
            return self.message
 
677
 
 
678
    def test_commit_callback(self):
 
679
        """Commit should invoke a callback to get the message"""
 
680
 
 
681
        tree = self.make_branch_and_tree('.')
 
682
        try:
 
683
            tree.commit()
 
684
        except Exception, e:
 
685
            self.assertTrue(isinstance(e, BzrError))
 
686
            self.assertEqual('The message or message_callback keyword'
 
687
                             ' parameter is required for commit().', str(e))
 
688
        else:
 
689
            self.fail('exception not raised')
 
690
        cb = self.Callback(u'commit 1', self)
 
691
        tree.commit(message_callback=cb)
 
692
        self.assertTrue(cb.called)
 
693
        repository = tree.branch.repository
 
694
        message = repository.get_revision(tree.last_revision()).message
 
695
        self.assertEqual('commit 1', message)
 
696
 
 
697
    def test_no_callback_pointless(self):
 
698
        """Callback should not be invoked for pointless commit"""
 
699
        tree = self.make_branch_and_tree('.')
 
700
        cb = self.Callback(u'commit 2', self)
 
701
        self.assertRaises(PointlessCommit, tree.commit, message_callback=cb, 
 
702
                          allow_pointless=False)
 
703
        self.assertFalse(cb.called)
 
704
 
 
705
    def test_no_callback_netfailure(self):
 
706
        """Callback should not be invoked if connectivity fails"""
 
707
        tree = self.make_branch_and_tree('.')
 
708
        cb = self.Callback(u'commit 2', self)
 
709
        repository = tree.branch.repository
 
710
        # simulate network failure
 
711
        def raise_(self, arg, arg2):
 
712
            raise errors.NoSuchFile('foo')
 
713
        repository.add_inventory = raise_
 
714
        self.assertRaises(errors.NoSuchFile, tree.commit, message_callback=cb)
 
715
        self.assertFalse(cb.called)
 
716
 
 
717
    def test_selected_file_merge_commit(self):
 
718
        """Ensure the correct error is raised"""
 
719
        tree = self.make_branch_and_tree('foo')
 
720
        # pending merge would turn into a left parent
 
721
        tree.commit('commit 1')
 
722
        tree.add_parent_tree_id('example')
 
723
        self.build_tree(['foo/bar', 'foo/baz'])
 
724
        tree.add(['bar', 'baz'])
 
725
        err = self.assertRaises(errors.CannotCommitSelectedFileMerge,
 
726
            tree.commit, 'commit 2', specific_files=['bar', 'baz'])
 
727
        self.assertEqual(['bar', 'baz'], err.files)
 
728
        self.assertEqual('Selected-file commit of merges is not supported'
 
729
                         ' 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
 
 
751
    def test_commit_author(self):
 
752
        """Passing a non-empty author kwarg to MutableTree.commit should add
 
753
        the 'author' revision property.
 
754
        """
 
755
        tree = self.make_branch_and_tree('foo')
 
756
        rev_id = tree.commit('commit 1', author='John Doe <jdoe@example.com>')
 
757
        rev = tree.branch.repository.get_revision(rev_id)
 
758
        self.assertEqual('John Doe <jdoe@example.com>',
 
759
                         rev.properties['author'])
 
760
 
 
761
    def test_commit_with_checkout_and_branch_sharing_repo(self):
 
762
        repo = self.make_repository('repo', shared=True)
 
763
        # make_branch_and_tree ignores shared repos
 
764
        branch = bzrdir.BzrDir.create_branch_convenience('repo/branch')
 
765
        tree2 = branch.create_checkout('repo/tree2')
 
766
        tree2.commit('message', rev_id='rev1')
 
767
        self.assertTrue(tree2.branch.repository.has_revision('rev1'))