1
# Copyright (C) 2005-2011 Canonical Ltd
3
# This program is free software; you can redistribute it and/or modify
4
# it under the terms of the GNU General Public License as published by
5
# the Free Software Foundation; either version 2 of the License, or
6
# (at your option) any later version.
8
# This program is distributed in the hope that it will be useful,
9
# but WITHOUT ANY WARRANTY; without even the implied warranty of
10
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
11
# GNU General Public License for more details.
13
# You should have received a copy of the GNU General Public License
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
26
from bzrlib.branch import Branch
27
from bzrlib.bzrdir import BzrDirMetaFormat1
28
from bzrlib.commit import Commit, NullCommitReporter
29
from bzrlib.errors import (
35
from bzrlib.tests import (
36
TestCaseWithTransport,
39
from bzrlib.tests.features import (
42
from bzrlib.tests.matchers import MatchesAncestry
45
# TODO: Test commit with some added, and added-but-missing files
47
class MustSignConfig(config.Stack):
49
def __init__(self, branch):
50
store = config.IniFileStore()
51
store._load_from_string('''
52
gpg_signing_command=cat -
53
create_signatures=always
55
super(MustSignConfig, self).__init__([store.get_sections])
56
# FIXME: Strictly speaking we should fallback to the no-name section in
57
# branch.conf but no tests need that so far -- vila 2011-12-14
60
class BranchWithHooks(config.Stack):
62
def __init__(self, branch):
63
store = config.IniFileStore()
64
store._load_from_string('post_commit=bzrlib.ahook bzrlib.ahook')
65
super(BranchWithHooks, self).__init__([store.get_sections])
66
# FIXME: Strictly speaking we should fallback to the no-name section in
67
# branch.conf but no tests need that so far -- vila 2011-12-14
70
class CapturingReporter(NullCommitReporter):
71
"""This reporter captures the calls made to it for evaluation later."""
74
# a list of the calls this received
77
def snapshot_change(self, change, path):
78
self.calls.append(('change', change, path))
80
def deleted(self, file_id):
81
self.calls.append(('deleted', file_id))
83
def missing(self, path):
84
self.calls.append(('missing', path))
86
def renamed(self, change, old_path, new_path):
87
self.calls.append(('renamed', change, old_path, new_path))
93
class TestCommit(TestCaseWithTransport):
95
def test_simple_commit(self):
96
"""Commit and check two versions of a single file."""
97
wt = self.make_branch_and_tree('.')
99
file('hello', 'w').write('hello world')
101
rev1 = wt.commit(message='add hello')
102
file_id = wt.path2id('hello')
104
file('hello', 'w').write('version 2')
105
rev2 = wt.commit(message='commit 2')
107
eq = self.assertEquals
109
rev = b.repository.get_revision(rev1)
110
eq(rev.message, 'add hello')
112
tree1 = b.repository.revision_tree(rev1)
114
text = tree1.get_file_text(file_id)
116
self.assertEqual('hello world', text)
118
tree2 = b.repository.revision_tree(rev2)
120
text = tree2.get_file_text(file_id)
122
self.assertEqual('version 2', text)
124
def test_commit_lossy_native(self):
125
"""Attempt a lossy commit to a native branch."""
126
wt = self.make_branch_and_tree('.')
128
file('hello', 'w').write('hello world')
130
revid = wt.commit(message='add hello', rev_id='revid', lossy=True)
131
self.assertEquals('revid', revid)
133
def test_commit_lossy_foreign(self):
134
"""Attempt a lossy commit to a foreign branch."""
135
test_foreign.register_dummy_foreign_for_test(self)
136
wt = self.make_branch_and_tree('.',
137
format=test_foreign.DummyForeignVcsDirFormat())
139
file('hello', 'w').write('hello world')
141
revid = wt.commit(message='add hello', lossy=True,
142
timestamp=1302659388, timezone=0)
143
self.assertEquals('dummy-v1:1302659388.0-0-UNKNOWN', revid)
145
def test_commit_bound_lossy_foreign(self):
146
"""Attempt a lossy commit to a bzr branch bound to a foreign branch."""
147
test_foreign.register_dummy_foreign_for_test(self)
148
foreign_branch = self.make_branch('foreign',
149
format=test_foreign.DummyForeignVcsDirFormat())
150
wt = foreign_branch.create_checkout("local")
152
file('local/hello', 'w').write('hello world')
154
revid = wt.commit(message='add hello', lossy=True,
155
timestamp=1302659388, timezone=0)
156
self.assertEquals('dummy-v1:1302659388.0-0-0', revid)
157
self.assertEquals('dummy-v1:1302659388.0-0-0',
158
foreign_branch.last_revision())
159
self.assertEquals('dummy-v1:1302659388.0-0-0',
160
wt.branch.last_revision())
162
def test_missing_commit(self):
163
"""Test a commit with a missing file"""
164
wt = self.make_branch_and_tree('.')
166
file('hello', 'w').write('hello world')
167
wt.add(['hello'], ['hello-id'])
168
wt.commit(message='add hello')
171
reporter = CapturingReporter()
172
wt.commit('removed hello', rev_id='rev2', reporter=reporter)
174
[('missing', u'hello'), ('deleted', u'hello')],
177
tree = b.repository.revision_tree('rev2')
178
self.assertFalse(tree.has_id('hello-id'))
180
def test_partial_commit_move(self):
181
"""Test a partial commit where a file was renamed but not committed.
183
https://bugs.launchpad.net/bzr/+bug/83039
185
If not handled properly, commit will try to snapshot
186
dialog.py with olive/ as a parent, while
187
olive/ has not been snapshotted yet.
189
wt = self.make_branch_and_tree('.')
191
self.build_tree(['annotate/', 'annotate/foo.py',
192
'olive/', 'olive/dialog.py'
194
wt.add(['annotate', 'olive', 'annotate/foo.py', 'olive/dialog.py'])
195
wt.commit(message='add files')
196
wt.rename_one("olive/dialog.py", "aaa")
197
self.build_tree_contents([('annotate/foo.py', 'modified\n')])
198
wt.commit('renamed hello', specific_files=["annotate"])
200
def test_pointless_commit(self):
201
"""Commit refuses unless there are changes or it's forced."""
202
wt = self.make_branch_and_tree('.')
204
file('hello', 'w').write('hello')
206
wt.commit(message='add hello')
207
self.assertEquals(b.revno(), 1)
208
self.assertRaises(PointlessCommit,
211
allow_pointless=False)
212
self.assertEquals(b.revno(), 1)
214
def test_commit_empty(self):
215
"""Commiting an empty tree works."""
216
wt = self.make_branch_and_tree('.')
218
wt.commit(message='empty tree', allow_pointless=True)
219
self.assertRaises(PointlessCommit,
221
message='empty tree',
222
allow_pointless=False)
223
wt.commit(message='empty tree', allow_pointless=True)
224
self.assertEquals(b.revno(), 2)
226
def test_selective_delete(self):
227
"""Selective commit in tree with deletions"""
228
wt = self.make_branch_and_tree('.')
230
file('hello', 'w').write('hello')
231
file('buongia', 'w').write('buongia')
232
wt.add(['hello', 'buongia'],
233
['hello-id', 'buongia-id'])
234
wt.commit(message='add files',
238
file('buongia', 'w').write('new text')
239
wt.commit(message='update text',
240
specific_files=['buongia'],
241
allow_pointless=False,
244
wt.commit(message='remove hello',
245
specific_files=['hello'],
246
allow_pointless=False,
249
eq = self.assertEquals
252
tree2 = b.repository.revision_tree('test@rev-2')
254
self.addCleanup(tree2.unlock)
255
self.assertTrue(tree2.has_filename('hello'))
256
self.assertEquals(tree2.get_file_text('hello-id'), 'hello')
257
self.assertEquals(tree2.get_file_text('buongia-id'), 'new text')
259
tree3 = b.repository.revision_tree('test@rev-3')
261
self.addCleanup(tree3.unlock)
262
self.assertFalse(tree3.has_filename('hello'))
263
self.assertEquals(tree3.get_file_text('buongia-id'), 'new text')
265
def test_commit_rename(self):
266
"""Test commit of a revision where a file is renamed."""
267
tree = self.make_branch_and_tree('.')
269
self.build_tree(['hello'], line_endings='binary')
270
tree.add(['hello'], ['hello-id'])
271
tree.commit(message='one', rev_id='test@rev-1', allow_pointless=False)
273
tree.rename_one('hello', 'fruity')
274
tree.commit(message='renamed', rev_id='test@rev-2', allow_pointless=False)
276
eq = self.assertEquals
277
tree1 = b.repository.revision_tree('test@rev-1')
279
self.addCleanup(tree1.unlock)
280
eq(tree1.id2path('hello-id'), 'hello')
281
eq(tree1.get_file_text('hello-id'), 'contents of hello\n')
282
self.assertFalse(tree1.has_filename('fruity'))
283
self.check_tree_shape(tree1, ['hello'])
284
eq(tree1.get_file_revision('hello-id'), 'test@rev-1')
286
tree2 = b.repository.revision_tree('test@rev-2')
288
self.addCleanup(tree2.unlock)
289
eq(tree2.id2path('hello-id'), 'fruity')
290
eq(tree2.get_file_text('hello-id'), 'contents of hello\n')
291
self.check_tree_shape(tree2, ['fruity'])
292
eq(tree2.get_file_revision('hello-id'), 'test@rev-2')
294
def test_reused_rev_id(self):
295
"""Test that a revision id cannot be reused in a branch"""
296
wt = self.make_branch_and_tree('.')
298
wt.commit('initial', rev_id='test@rev-1', allow_pointless=True)
299
self.assertRaises(Exception,
303
allow_pointless=True)
305
def test_commit_move(self):
306
"""Test commit of revisions with moved files and directories"""
307
eq = self.assertEquals
308
wt = self.make_branch_and_tree('.')
311
self.build_tree(['hello', 'a/', 'b/'])
312
wt.add(['hello', 'a', 'b'], ['hello-id', 'a-id', 'b-id'])
313
wt.commit('initial', rev_id=r1, allow_pointless=False)
314
wt.move(['hello'], 'a')
316
wt.commit('two', rev_id=r2, allow_pointless=False)
319
self.check_tree_shape(wt, ['a/', 'a/hello', 'b/'])
325
wt.commit('three', rev_id=r3, allow_pointless=False)
328
self.check_tree_shape(wt,
329
['a/', 'a/hello', 'a/b/'])
330
self.check_tree_shape(b.repository.revision_tree(r3),
331
['a/', 'a/hello', 'a/b/'])
335
wt.move(['a/hello'], 'a/b')
337
wt.commit('four', rev_id=r4, allow_pointless=False)
340
self.check_tree_shape(wt, ['a/', 'a/b/hello', 'a/b/'])
344
inv = b.repository.get_inventory(r4)
345
eq(inv['hello-id'].revision, r4)
346
eq(inv['a-id'].revision, r1)
347
eq(inv['b-id'].revision, r3)
349
def test_removed_commit(self):
350
"""Commit with a removed file"""
351
wt = self.make_branch_and_tree('.')
353
file('hello', 'w').write('hello world')
354
wt.add(['hello'], ['hello-id'])
355
wt.commit(message='add hello')
357
wt.commit('removed hello', rev_id='rev2')
359
tree = b.repository.revision_tree('rev2')
360
self.assertFalse(tree.has_id('hello-id'))
362
def test_committed_ancestry(self):
363
"""Test commit appends revisions to ancestry."""
364
wt = self.make_branch_and_tree('.')
368
file('hello', 'w').write((str(i) * 4) + '\n')
370
wt.add(['hello'], ['hello-id'])
371
rev_id = 'test@rev-%d' % (i+1)
372
rev_ids.append(rev_id)
373
wt.commit(message='rev %d' % (i+1),
376
self.assertThat(rev_ids[:i+1],
377
MatchesAncestry(b.repository, rev_ids[i]))
379
def test_commit_new_subdir_child_selective(self):
380
wt = self.make_branch_and_tree('.')
382
self.build_tree(['dir/', 'dir/file1', 'dir/file2'])
383
wt.add(['dir', 'dir/file1', 'dir/file2'],
384
['dirid', 'file1id', 'file2id'])
385
wt.commit('dir/file1', specific_files=['dir/file1'], rev_id='1')
386
inv = b.repository.get_inventory('1')
387
self.assertEqual('1', inv['dirid'].revision)
388
self.assertEqual('1', inv['file1id'].revision)
389
# FIXME: This should raise a KeyError I think, rbc20051006
390
self.assertRaises(BzrError, inv.__getitem__, 'file2id')
392
def test_strict_commit(self):
393
"""Try and commit with unknown files and strict = True, should fail."""
394
from bzrlib.errors import StrictCommitFailed
395
wt = self.make_branch_and_tree('.')
397
file('hello', 'w').write('hello world')
399
file('goodbye', 'w').write('goodbye cruel world!')
400
self.assertRaises(StrictCommitFailed, wt.commit,
401
message='add hello but not goodbye', strict=True)
403
def test_strict_commit_without_unknowns(self):
404
"""Try and commit with no unknown files and strict = True,
406
wt = self.make_branch_and_tree('.')
408
file('hello', 'w').write('hello world')
410
wt.commit(message='add hello', strict=True)
412
def test_nonstrict_commit(self):
413
"""Try and commit with unknown files and strict = False, should work."""
414
wt = self.make_branch_and_tree('.')
416
file('hello', 'w').write('hello world')
418
file('goodbye', 'w').write('goodbye cruel world!')
419
wt.commit(message='add hello but not goodbye', strict=False)
421
def test_nonstrict_commit_without_unknowns(self):
422
"""Try and commit with no unknown files and strict = False,
424
wt = self.make_branch_and_tree('.')
426
file('hello', 'w').write('hello world')
428
wt.commit(message='add hello', strict=False)
430
def test_signed_commit(self):
432
import bzrlib.commit as commit
433
oldstrategy = bzrlib.gpg.GPGStrategy
434
wt = self.make_branch_and_tree('.')
436
wt.commit("base", allow_pointless=True, rev_id='A')
437
self.assertFalse(branch.repository.has_signature_for_revision_id('A'))
439
from bzrlib.testament import Testament
440
# monkey patch gpg signing mechanism
441
bzrlib.gpg.GPGStrategy = bzrlib.gpg.LoopbackGPGStrategy
442
commit.Commit(config_stack=MustSignConfig(branch)
443
).commit(message="base", allow_pointless=True,
444
rev_id='B', working_tree=wt)
446
return bzrlib.gpg.LoopbackGPGStrategy(None).sign(text)
447
self.assertEqual(sign(Testament.from_revision(branch.repository,
448
'B').as_short_text()),
449
branch.repository.get_signature_text('B'))
451
bzrlib.gpg.GPGStrategy = oldstrategy
453
def test_commit_failed_signature(self):
455
import bzrlib.commit as commit
456
oldstrategy = bzrlib.gpg.GPGStrategy
457
wt = self.make_branch_and_tree('.')
459
wt.commit("base", allow_pointless=True, rev_id='A')
460
self.assertFalse(branch.repository.has_signature_for_revision_id('A'))
462
# monkey patch gpg signing mechanism
463
bzrlib.gpg.GPGStrategy = bzrlib.gpg.DisabledGPGStrategy
464
config = MustSignConfig(branch)
465
self.assertRaises(SigningFailed,
466
commit.Commit(config_stack=config).commit,
468
allow_pointless=True,
471
branch = Branch.open(self.get_url('.'))
472
self.assertEqual(branch.last_revision(), 'A')
473
self.assertFalse(branch.repository.has_revision('B'))
475
bzrlib.gpg.GPGStrategy = oldstrategy
477
def test_commit_invokes_hooks(self):
478
import bzrlib.commit as commit
479
wt = self.make_branch_and_tree('.')
482
def called(branch, rev_id):
483
calls.append('called')
484
bzrlib.ahook = called
486
config = BranchWithHooks(branch)
487
commit.Commit(config_stack=config).commit(
489
allow_pointless=True,
490
rev_id='A', working_tree = wt)
491
self.assertEqual(['called', 'called'], calls)
495
def test_commit_object_doesnt_set_nick(self):
496
# using the Commit object directly does not set the branch nick.
497
wt = self.make_branch_and_tree('.')
499
c.commit(working_tree=wt, message='empty tree', allow_pointless=True)
500
self.assertEquals(wt.branch.revno(), 1)
502
wt.branch.repository.get_revision(
503
wt.branch.last_revision()).properties)
505
def test_safe_master_lock(self):
507
master = BzrDirMetaFormat1().initialize('master')
508
master.create_repository()
509
master_branch = master.create_branch()
510
master.create_workingtree()
511
bound = master.sprout('bound')
512
wt = bound.open_workingtree()
513
wt.branch.set_bound_location(os.path.realpath('master'))
514
master_branch.lock_write()
516
self.assertRaises(LockContention, wt.commit, 'silly')
518
master_branch.unlock()
520
def test_commit_bound_merge(self):
521
# see bug #43959; commit of a merge in a bound branch fails to push
522
# the new commit into the master
523
master_branch = self.make_branch('master')
524
bound_tree = self.make_branch_and_tree('bound')
525
bound_tree.branch.bind(master_branch)
527
self.build_tree_contents([('bound/content_file', 'initial contents\n')])
528
bound_tree.add(['content_file'])
529
bound_tree.commit(message='woo!')
531
other_bzrdir = master_branch.bzrdir.sprout('other')
532
other_tree = other_bzrdir.open_workingtree()
534
# do a commit to the other branch changing the content file so
535
# that our commit after merging will have a merged revision in the
536
# content file history.
537
self.build_tree_contents([('other/content_file', 'change in other\n')])
538
other_tree.commit('change in other')
540
# do a merge into the bound branch from other, and then change the
541
# content file locally to force a new revision (rather than using the
542
# revision from other). This forces extra processing in commit.
543
bound_tree.merge_from_branch(other_tree.branch)
544
self.build_tree_contents([('bound/content_file', 'change in bound\n')])
546
# before #34959 was fixed, this failed with 'revision not present in
547
# weave' when trying to implicitly push from the bound branch to the master
548
bound_tree.commit(message='commit of merge in bound tree')
550
def test_commit_reporting_after_merge(self):
551
# when doing a commit of a merge, the reporter needs to still
552
# be called for each item that is added/removed/deleted.
553
this_tree = self.make_branch_and_tree('this')
554
# we need a bunch of files and dirs, to perform one action on each.
557
'this/dirtoreparent/',
560
'this/filetoreparent',
577
this_tree.commit('create_files')
578
other_dir = this_tree.bzrdir.sprout('other')
579
other_tree = other_dir.open_workingtree()
580
other_tree.lock_write()
581
# perform the needed actions on the files and dirs.
583
other_tree.rename_one('dirtorename', 'renameddir')
584
other_tree.rename_one('dirtoreparent', 'renameddir/reparenteddir')
585
other_tree.rename_one('filetorename', 'renamedfile')
586
other_tree.rename_one('filetoreparent', 'renameddir/reparentedfile')
587
other_tree.remove(['dirtoremove', 'filetoremove'])
588
self.build_tree_contents([
590
('other/filetomodify', 'new content'),
591
('other/newfile', 'new file content')])
592
other_tree.add('newfile')
593
other_tree.add('newdir/')
594
other_tree.commit('modify all sample files and dirs.')
597
this_tree.merge_from_branch(other_tree.branch)
598
reporter = CapturingReporter()
599
this_tree.commit('do the commit', reporter=reporter)
601
('change', 'modified', 'filetomodify'),
602
('change', 'added', 'newdir'),
603
('change', 'added', 'newfile'),
604
('renamed', 'renamed', 'dirtorename', 'renameddir'),
605
('renamed', 'renamed', 'filetorename', 'renamedfile'),
606
('renamed', 'renamed', 'dirtoreparent', 'renameddir/reparenteddir'),
607
('renamed', 'renamed', 'filetoreparent', 'renameddir/reparentedfile'),
608
('deleted', 'dirtoremove'),
609
('deleted', 'filetoremove'),
611
result = set(reporter.calls)
612
missing = expected - result
613
new = result - expected
614
self.assertEqual((set(), set()), (missing, new))
616
def test_commit_removals_respects_filespec(self):
617
"""Commit respects the specified_files for removals."""
618
tree = self.make_branch_and_tree('.')
619
self.build_tree(['a', 'b'])
621
tree.commit('added a, b')
622
tree.remove(['a', 'b'])
623
tree.commit('removed a', specific_files='a')
624
basis = tree.basis_tree()
627
self.assertIs(None, basis.path2id('a'))
628
self.assertFalse(basis.path2id('b') is None)
632
def test_commit_saves_1ms_timestamp(self):
633
"""Passing in a timestamp is saved with 1ms resolution"""
634
tree = self.make_branch_and_tree('.')
635
self.build_tree(['a'])
637
tree.commit('added a', timestamp=1153248633.4186721, timezone=0,
640
rev = tree.branch.repository.get_revision('a1')
641
self.assertEqual(1153248633.419, rev.timestamp)
643
def test_commit_has_1ms_resolution(self):
644
"""Allowing commit to generate the timestamp also has 1ms resolution"""
645
tree = self.make_branch_and_tree('.')
646
self.build_tree(['a'])
648
tree.commit('added a', rev_id='a1')
650
rev = tree.branch.repository.get_revision('a1')
651
timestamp = rev.timestamp
652
timestamp_1ms = round(timestamp, 3)
653
self.assertEqual(timestamp_1ms, timestamp)
655
def assertBasisTreeKind(self, kind, tree, file_id):
656
basis = tree.basis_tree()
659
self.assertEqual(kind, basis.kind(file_id))
663
def test_commit_kind_changes(self):
664
self.requireFeature(SymlinkFeature)
665
tree = self.make_branch_and_tree('.')
666
os.symlink('target', 'name')
667
tree.add('name', 'a-file-id')
668
tree.commit('Added a symlink')
669
self.assertBasisTreeKind('symlink', tree, 'a-file-id')
672
self.build_tree(['name'])
673
tree.commit('Changed symlink to file')
674
self.assertBasisTreeKind('file', tree, 'a-file-id')
677
os.symlink('target', 'name')
678
tree.commit('file to symlink')
679
self.assertBasisTreeKind('symlink', tree, 'a-file-id')
683
tree.commit('symlink to directory')
684
self.assertBasisTreeKind('directory', tree, 'a-file-id')
687
os.symlink('target', 'name')
688
tree.commit('directory to symlink')
689
self.assertBasisTreeKind('symlink', tree, 'a-file-id')
691
# prepare for directory <-> file tests
694
tree.commit('symlink to directory')
695
self.assertBasisTreeKind('directory', tree, 'a-file-id')
698
self.build_tree(['name'])
699
tree.commit('Changed directory to file')
700
self.assertBasisTreeKind('file', tree, 'a-file-id')
704
tree.commit('file to directory')
705
self.assertBasisTreeKind('directory', tree, 'a-file-id')
707
def test_commit_unversioned_specified(self):
708
"""Commit should raise if specified files isn't in basis or worktree"""
709
tree = self.make_branch_and_tree('.')
710
self.assertRaises(errors.PathsNotVersionedError, tree.commit,
711
'message', specific_files=['bogus'])
713
class Callback(object):
715
def __init__(self, message, testcase):
717
self.message = message
718
self.testcase = testcase
720
def __call__(self, commit_obj):
722
self.testcase.assertTrue(isinstance(commit_obj, Commit))
725
def test_commit_callback(self):
726
"""Commit should invoke a callback to get the message"""
728
tree = self.make_branch_and_tree('.')
732
self.assertTrue(isinstance(e, BzrError))
733
self.assertEqual('The message or message_callback keyword'
734
' parameter is required for commit().', str(e))
736
self.fail('exception not raised')
737
cb = self.Callback(u'commit 1', self)
738
tree.commit(message_callback=cb)
739
self.assertTrue(cb.called)
740
repository = tree.branch.repository
741
message = repository.get_revision(tree.last_revision()).message
742
self.assertEqual('commit 1', message)
744
def test_no_callback_pointless(self):
745
"""Callback should not be invoked for pointless commit"""
746
tree = self.make_branch_and_tree('.')
747
cb = self.Callback(u'commit 2', self)
748
self.assertRaises(PointlessCommit, tree.commit, message_callback=cb,
749
allow_pointless=False)
750
self.assertFalse(cb.called)
752
def test_no_callback_netfailure(self):
753
"""Callback should not be invoked if connectivity fails"""
754
tree = self.make_branch_and_tree('.')
755
cb = self.Callback(u'commit 2', self)
756
repository = tree.branch.repository
757
# simulate network failure
758
def raise_(self, arg, arg2, arg3=None, arg4=None):
759
raise errors.NoSuchFile('foo')
760
repository.add_inventory = raise_
761
repository.add_inventory_by_delta = raise_
762
self.assertRaises(errors.NoSuchFile, tree.commit, message_callback=cb)
763
self.assertFalse(cb.called)
765
def test_selected_file_merge_commit(self):
766
"""Ensure the correct error is raised"""
767
tree = self.make_branch_and_tree('foo')
768
# pending merge would turn into a left parent
769
tree.commit('commit 1')
770
tree.add_parent_tree_id('example')
771
self.build_tree(['foo/bar', 'foo/baz'])
772
tree.add(['bar', 'baz'])
773
err = self.assertRaises(errors.CannotCommitSelectedFileMerge,
774
tree.commit, 'commit 2', specific_files=['bar', 'baz'])
775
self.assertEqual(['bar', 'baz'], err.files)
776
self.assertEqual('Selected-file commit of merges is not supported'
777
' yet: files bar, baz', str(err))
779
def test_commit_ordering(self):
780
"""Test of corner-case commit ordering error"""
781
tree = self.make_branch_and_tree('.')
782
self.build_tree(['a/', 'a/z/', 'a/c/', 'a/z/x', 'a/z/y'])
783
tree.add(['a/', 'a/z/', 'a/c/', 'a/z/x', 'a/z/y'])
785
self.build_tree(['a/c/d/'])
787
tree.rename_one('a/z/x', 'a/c/d/x')
788
tree.commit('test', specific_files=['a/z/y'])
790
def test_commit_no_author(self):
791
"""The default kwarg author in MutableTree.commit should not add
792
the 'author' revision property.
794
tree = self.make_branch_and_tree('foo')
795
rev_id = tree.commit('commit 1')
796
rev = tree.branch.repository.get_revision(rev_id)
797
self.assertFalse('author' in rev.properties)
798
self.assertFalse('authors' in rev.properties)
800
def test_commit_author(self):
801
"""Passing a non-empty author kwarg to MutableTree.commit should add
802
the 'author' revision property.
804
tree = self.make_branch_and_tree('foo')
805
rev_id = self.callDeprecated(['The parameter author was '
806
'deprecated in version 1.13. Use authors instead'],
807
tree.commit, 'commit 1', author='John Doe <jdoe@example.com>')
808
rev = tree.branch.repository.get_revision(rev_id)
809
self.assertEqual('John Doe <jdoe@example.com>',
810
rev.properties['authors'])
811
self.assertFalse('author' in rev.properties)
813
def test_commit_empty_authors_list(self):
814
"""Passing an empty list to authors shouldn't add the property."""
815
tree = self.make_branch_and_tree('foo')
816
rev_id = tree.commit('commit 1', authors=[])
817
rev = tree.branch.repository.get_revision(rev_id)
818
self.assertFalse('author' in rev.properties)
819
self.assertFalse('authors' in rev.properties)
821
def test_multiple_authors(self):
822
tree = self.make_branch_and_tree('foo')
823
rev_id = tree.commit('commit 1',
824
authors=['John Doe <jdoe@example.com>',
825
'Jane Rey <jrey@example.com>'])
826
rev = tree.branch.repository.get_revision(rev_id)
827
self.assertEqual('John Doe <jdoe@example.com>\n'
828
'Jane Rey <jrey@example.com>', rev.properties['authors'])
829
self.assertFalse('author' in rev.properties)
831
def test_author_and_authors_incompatible(self):
832
tree = self.make_branch_and_tree('foo')
833
self.assertRaises(AssertionError, tree.commit, 'commit 1',
834
authors=['John Doe <jdoe@example.com>',
835
'Jane Rey <jrey@example.com>'],
836
author="Jack Me <jme@example.com>")
838
def test_author_with_newline_rejected(self):
839
tree = self.make_branch_and_tree('foo')
840
self.assertRaises(AssertionError, tree.commit, 'commit 1',
841
authors=['John\nDoe <jdoe@example.com>'])
843
def test_commit_with_checkout_and_branch_sharing_repo(self):
844
repo = self.make_repository('repo', shared=True)
845
# make_branch_and_tree ignores shared repos
846
branch = bzrdir.BzrDir.create_branch_convenience('repo/branch')
847
tree2 = branch.create_checkout('repo/tree2')
848
tree2.commit('message', rev_id='rev1')
849
self.assertTrue(tree2.branch.repository.has_revision('rev1'))
added
removed
bzrlib/tests/test_commit.py
